Code Issues Pull Requests Actions Security InsightsSettings

fix(pulls): merged PRs no longer show an empty Files-changed tab #5445

Merged⚡ AI-generatedXSccantynz wants to mergefix/merged-pr-empty-diff→mainopened 24d ago

ccantynzcommented 24d ago

The Files-changed tab diffs base...head live; once a PR merges the merge-base becomes head and the diff is empty — every merged PR rendered zero files, which read as "these PRs contain no code."

When a merged PR yields an empty live diff, the tab now recovers the real change set: find the merge commit on the base branch (pull request #N message convention) and diff first-parent..commit; fall back to the surviving head ref for fast-forward merges. Best-effort — failures degrade to the old behavior.

Follow-up (separate): persist merge_sha at merge time in the shared pr-merge-gated chain so recovery never depends on message conventions.

🤖 Generated with Claude Code

ccantynzAI Reviewcommented 24d ago

AI Triage

(no summary)

Priority: medium Risk area: mixed

Suggested labels: (no label suggestions) Suggested reviewers: (no reviewer suggestions)

Suggestions only — nothing has been applied. The PR author stays in control.

gluecron[bot]🤖 botAI Reviewcommented 24d ago

AI review unavailable

The platform's AI balance is exhausted, so AI generation is temporarily unavailable. Nothing was lost. You can queue this as a repair for the internal agent from the repository's Health page, or try again once the balance is restored. The PR is otherwise unchanged.

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →

⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts