fix(web): PR file stats ignore the hide-whitespace toggle

[flamboh]

Note

🤖 Claude Opus 5.5 on behalf of Oliver

With "Hide whitespace changes" on, the per-file stat in a pull request's Code tab disagreed with the panel header. On #15561, its only file showed +98 −46 while the header and GitHub showed +115 −63. The file row was counting hunks after each one had been re-diffed with whitespace removed, so lines that only changed indentation dropped out of the count.

The file row now counts the patch as sent, using the unfiltered sourceFiles that getRenderablePatch already returns. The toggle still hides whitespace-only lines in the diff, but the numbers stay GitHub's and match the header. This is also how GitHub's own "Hide whitespace" works: it hides the lines without changing the file's stats. Files whose patch the host withheld still show the stats the host reported.

Before / After

With "Hide whitespace changes" on, on #15561:

Before: file row +98 −46, header +115 −63

After: file row and header both +115 −63

GitHub for reference: file row and header both +115 −63

Validation

• Ran fix(web): PR code toolbar no longer overlaps in narrow panels #15561's gh pr diff patch through getRenderablePatch: 115/63 unfiltered and 98/46 with whitespace hidden, which matches the reported mismatch exactly.
• tsc --noEmit for apps/web, lint on the changed file (no new warnings), and vp test run src/lib/diffRendering.test.ts src/components/pullRequest/PullRequestDetailPanel.test.tsx (50 passed).

Made with Claude Opus 5.5 in Claude Code via T3 Code.

[macroscopeapp]

Approvability

Verdict: Approved at 2c88db1

Macroscope's review found this PR approvable — This is a small, self-contained web bug fix that corrects pull-request file statistics when whitespace hiding is enabled. It changes only displayed metadata, preserves existing defaults and diff behavior, and introduces no sensitive, schema, deployment, or static-analysis changes.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 64febbe7-fc08-44ea-8483-e435e4bc19c2

📥 Commits

Reviewing files that changed from the base of the PR and between 3e6b450 and 2c88db1.

📒 Files selected for processing (1)

• apps/web/src/components/pullRequest/PullRequestCodeTab.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

The code tab now calculates memoized addition and deletion counts from each parsed slice’s source file. File headers use those counts when available, then fall back to displayed-diff counts and the existing omitted-file statistics.

Changes

Pull request file statistics

Layer / File(s)	Summary
Calculate and display file statistics apps/web/src/components/pullRequest/PullRequestCodeTab.tsx	The code tab memoizes statistics by rendered file identity, using the parsed source file when available. Header counts use those statistics or fall back to displayed-diff counts; the existing omitted-file fallback remains. The header metadata callback tracks the statistics in its dependencies.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 2c88d

No actionable PR-introduced risk remains. File statistics stay associated with the rendered block; the separate same-path ordering behavior is not shown to regress in this change.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the fix to pull request file statistics when whitespace changes are hidden.
Description check	✅ Passed	The description explains the problem and change, provides before-and-after screenshots, and reports specific validation results. It does not include the template’s Scope and approval section or explic…

✨ Finishing Touches 💡 1

🧪 Generate unit tests (beta)

• Create a new PR

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

_{Comment @coderabbitai help to get the list of available commands.}