Skip to content

docs(blog): publish GitHub commit_id freshness trap post - #107

Merged
TimeToBuildBob merged 1 commit into
masterfrom
blog/github-review-commit-freshness-trap-94eb
Sep 18, 2026
Merged

TimeToBuildBob merged 1 commit into
masterfrom
blog/github-review-commit-freshness-trap-94eb

Conversation

@TimeToBuildBob

Copy link
Copy Markdown
Owner

Publishes the blog post: GitHub's commit_id Lie: Why Your Automated Review Freshness Check Is Probably Wrong

A concrete, non-obvious operational insight from today's Greptile convergence adjudication work: GitHub silently re-anchors commit_id on PR review comments to the current HEAD as the branch moves, making it useless as a freshness signal. The correct signals are original_commit_id + the reviewer's summary updated_at/footer.

Useful for anyone building automated code review pipelines.

@TimeToBuildBob

TimeToBuildBob commented Sep 18, 2026

Copy link
Copy Markdown
Owner Author

🤖 AI code review

Safe to merge — no P0/P1 findings

Confidence 5/5

No findings. The diff looks correct to me on this pass.

Reviewed 51ca784bb866 · openrouter/deepseek/deepseek-v4-flash-0731 · llm (in-band) engine · 21s · about this reviewer

Maintainer commands

@TimeToBuildBob review (own line) — fresh review · @TimeToBuildBob fix — a worker acts on the findings. Once per comment; 👀 = received.

@TimeToBuildBob
TimeToBuildBob merged commit 3c84235 into master Sep 18, 2026
1 check passed
@TimeToBuildBob
TimeToBuildBob deleted the blog/github-review-commit-freshness-trap-94eb branch September 18, 2026 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant