Skip to content

Commit 8fae3b5

Browse files
igerberclaude
andcommitted
Exclude large data files from AI review diff to avoid input limit
Golden JSON files (4.3MB) and notebook outputs exceeded Codex's 1MB character limit. Exclude benchmarks/data/**/*.json, *.csv, and docs/tutorials/*.ipynb from the unified diff while keeping them in the --stat summary so the reviewer knows they changed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b634294 commit 8fae3b5

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ai_pr_review.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,11 @@ jobs:
148148
git --no-pager diff --name-status "$BASE_SHA" "$HEAD_SHA"
149149
echo ""
150150
echo "Unified diff (context=5):"
151-
git --no-pager diff --unified=5 "$BASE_SHA" "$HEAD_SHA"
151+
# Exclude large generated data files from the full diff to stay
152+
# within the model's input limit. The --stat above still lists them.
153+
git --no-pager diff --unified=5 "$BASE_SHA" "$HEAD_SHA" \
154+
-- . ':!benchmarks/data/**/*.json' ':!benchmarks/data/**/*.csv' \
155+
':!docs/tutorials/*.ipynb'
152156
} >> "$PROMPT"
153157
154158
- name: Run Codex

0 commit comments

Comments
 (0)