Skip to content

fix: improve CI reviewer to catch misplaced files and missed findings#1117

Merged
max-sixty merged 4 commits intomainfrom
reviewer
Feb 20, 2026
Merged

fix: improve CI reviewer to catch misplaced files and missed findings#1117
max-sixty merged 4 commits intomainfrom
reviewer

Conversation

@max-sixty
Copy link
Copy Markdown
Owner

Summary

  • pr-review skill: adds a substep prompting the reviewer to assess the PR's shape (what files are being added/changed) before diving into code details — so obvious misplacements like a scratch file at the repo root don't slip through
  • nightly review-reviewers: adds a cross-check step that compares bot review verdicts against subsequent commits and human comments on the same PR, identifying things the bot should have caught

Triggered by the bot approving PR #1004 without flagging an IDEAS.md design document at the repo root.

Test plan

  • Next claude-review run: verify the reviewer pauses to assess file list before code review
  • Next claude-nightly-review-reviewers run: verify it cross-checks review sessions against PR outcomes

This was written by Claude Code on behalf of max-sixty

🤖 Generated with Claude Code

max-sixty and others added 2 commits February 19, 2026 14:40
… code

The CI reviewer was approving PRs without noticing obviously misplaced
files (e.g. IDEAS.md at repo root in PR #1004). It read the diff but
went straight to Rust code review without stepping back to look at the
change holistically. Add a substep prompting the reviewer to look at
what files are being added/changed before reading them in detail.

Co-Authored-By: Claude <noreply@anthropic.com>
The nightly review-reviewers workflow previously only compared session
logs against CLAUDE.md guidelines. Now it also checks what happened
after each review — subsequent commits and human comments — to find
things the bot should have caught.

Co-Authored-By: Claude <noreply@anthropic.com>
@max-sixty max-sixty enabled auto-merge (squash) February 19, 2026 23:01
@max-sixty max-sixty merged commit 4befe46 into main Feb 20, 2026
20 checks passed
@max-sixty max-sixty deleted the reviewer branch February 20, 2026 00:08
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.

2 participants