Scope code review to changed hunks, not full file contents

[google-labs-jules]

The code review orchestrator was previously reading and sending full file contents for any file touched by a PR, leading to unnecessary token usage and potential context window issues for large files. This change scopes the review payload to a unified diff with 10 lines of context (-U10), which provides sufficient information for the LLM to understand the changes while significantly reducing the payload size. The files list was removed from the CodeReviewSummary interface and orchestrator logic to ensure no downstream processes can perform full-file reads based on filename alone.

Fixes #2576

---

PR created automatically by Jules for task 11373753286813057123 started by @arii

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[github-actions]

🐙 GitHub Models Code Review

Powered by GitHub Models

Reviewing: PR #2592

Code Review Feedback

High Severity Review

1. Blocking Bug: Type Mismatch in Error Case

File: scripts/lib/codeReviewOrchestrator.ts
Lines:

-    return { files: [], diffContext: '' };
+    return { diffContext: '' };

and

-  if (summary.files.length === 0 || !summary.diffContext) {
+  if (!summary.diffContext) {

Explanation:
The CodeReviewSummary interface previously required a files: string[] property. This property has been removed from the interface and from all return statements except for the error case in the catch block. The error case now returns { diffContext: '' }, which matches the new interface.

However, in the calling code (orchestrateCodeReview), the check:

-  if (summary.files.length === 0 || !summary.diffContext) {
+  if (!summary.diffContext) {

has been updated to remove the files check, which is correct. There is no longer any reference to summary.files in the code.

Conclusion:
No blocking bug here; the type is consistent throughout the diff.

---

2. Blocking Bug: Missing Required Property in Return

File: scripts/lib/codeReviewOrchestrator.ts
Line:

-    return { files: [], diffContext: '' };
+    return { diffContext: '' };

Explanation:
The CodeReviewSummary interface now only requires diffContext (and optionally prGoal). The error case returns { diffContext: '' }, which is valid.

Conclusion:
No blocking bug here.

---

3. Blocking Bug: Use of Removed Property

File: scripts/lib/codeReviewOrchestrator.ts
Line:

-  if (summary.files.length === 0 || !summary.diffContext) {
+  if (!summary.diffContext) {

Explanation:
All references to summary.files have been removed, matching the interface change.

Conclusion:
No blocking bug here.

---

4. Blocking Bug: Downstream Usage of files

File: Not shown in diff.

Explanation:
If any downstream code (outside this diff) expects files to be present on CodeReviewSummary, it will now break at runtime or type-checking time. However, within the scope of this diff, all usage of files has been removed.

Conclusion:
No blocking bug in the diff itself.

---

5. Blocking Bug: Incorrect Fallback Logic

File: scripts/lib/codeReviewOrchestrator.ts
Lines:

-    let diffCommand = 'git diff origin/main...HEAD';
-    let nameOnlyCommand = 'git diff --name-only origin/main...HEAD';
+    let diffCommand = 'git diff -U10 origin/main...HEAD';
...
-        diffCommand = 'git diff HEAD~1 HEAD';
-        nameOnlyCommand = 'git diff --name-only HEAD~1 HEAD';
+        diffCommand = 'git diff -U10 HEAD~1 HEAD';

Explanation:
The fallback logic for diffCommand is correct and matches the intended behavior.

Conclusion:
No blocking bug here.

---

6. Blocking Bug: TypeScript Type Safety

File: scripts/lib/codeReviewTypes.ts
Lines:

 export interface CodeReviewSummary {
-  files: string[];
   diffContext: string;
   prGoal?: string;
 }

Explanation:
The interface is now correct and matches all usage in the diff.

Conclusion:
No blocking bug here.

---

Summary

No high-severity (blocking) bugs, anti-patterns, missing types, or performance issues are present in the diff. All changes are internally consistent and type-safe.

[VERDICT: PASS]

---

Generated by github-models-code-review

[github-actions]

🚀 Deployment Details (Last updated: Jun 19, 2026, 11:27 AM PST)

🚀 Pushed to gh-pages; publish in progress

• Preview URL: https://arii.github.io/tech-dancer/fix/code-review-diff-scope-11373753286813057123/
• Preview Dashboard: https://arii.github.io/tech-dancer/previews/index.html
• Commit SHA: 5339780
• Marker File: verify here

[arii]

Hi! This PR has merge conflicts with the base branch.

To fix this, please follow these steps:

1. Ensure your local main branch is up to date.
2. Switch back to your feature branch.
3. Merge main into your branch.
4. Resolve the conflicts in the files marked by Git, then stage the resolved files.
5. Complete the merge commit.
6. Push the resolved branch to origin.

Alternatively, you can rebase your branch onto main and resolve conflicts step-by-step. Let us know if you need any help!

[arii]

ANTI-AI-SLOP\n\n\n## FINDINGS\n

\n\n## FINAL RECOMMENDATION\n<Approved | Approved with Minor Changes | Not Approved>\n\n

Inline Comments (Fallback due to Github line resolution errors)

• :1:

[arii]

ANTI-AI-SLOP\n\n\n## FINDINGS\n

\n\n## FINAL RECOMMENDATION\n<Approved | Approved with Minor Changes | Not Approved>\n\n

Inline Comments (Fallback due to Github line resolution errors)

• :1:

[arii]

Automated Review for PR #2592

CI Status: All checks passing.

FINAL RECOMMENDATION

Approved

[arii]

🤖 AI Technical Audit

ANTI-AI-SLOP

• Found redundant shell: bash blocks in GitHub Actions. Use run: directly as it defaults to bash on ubuntu-latest.
• The manual trap 'echo "::endgroup::"' EXIT logic is verbose and repetitive. Consider moving grouping to a reusable composite GitHub Action if the codebase grows, or stick to one-liners where possible.
• The retry logic in deploy.yml increased MAX_RETRIES but added complexity. Ensure RANDOM behavior is actually needed vs simple exponential backoff.

FINAL RECOMMENDATION

Approved

Review automatically published via RepoAuditor.

[arii]

Automated Review for PR #2592

CI Status: All checks passing.

Recommendation: Everything looks good from a CI perspective. Ready for manual review/merge if no other concerns.

FINAL RECOMMENDATION

Approved

[arii]

Comprehensive Review for PR #2592

CI Status: All checks passing.

Recommendation: Everything looks good from a CI perspective. All tests and linters pass. Ready for manual review/merge if no other concerns.

FINAL RECOMMENDATION

Approved