Skip to content

ci: let claude code review read the pr diff - #90

Merged
dimslaev merged 2 commits into
masterfrom
ci/claude-review-reads-diff
Oct 19, 2026
Merged

dimslaev merged 2 commits into
masterfrom
ci/claude-review-reads-diff

Conversation

@dimslaev

Copy link
Copy Markdown
Owner
  • The review allowed only the inline comment tool. Its subagents couldn't run gh pr view or gh pr diff, so each run ended after 3 turns and about 5 seconds with nothing posted (run 39159627553 on refactor: move shared backend mechanism into app/core #82: num_turns: 3, duration_ms: 4494).
  • Allows gh pr diff, gh pr view, gh pr comment (the plugin posts its "no issues found" summary with it), plus Read, Grep and Glob.
  • Leaves out the plugin's gh search and gh issue patterns, because its steps don't use them and they would let PR text start reads or comments elsewhere without a prompt.
  • fetch-depth: 1 stays. The plugin reads the diff through gh pr diff and the files from the checked-out merge ref, so it needs no history.
  • Pin comments on both claude-code-action uses now say # v1.0.252, the tag at the pinned commit, so zizmor's ref-version-mismatch audit passes on master.

This PR can't review itself. The action skips any PR whose workflow file differs from master's ("Workflow validation failed. The workflow file must exist and have identical content to the version on the repository's default branch"). The first real check is the first PR opened or pushed after this merges.


Generated by Claude Code

@dimslaev
dimslaev merged commit 071d4e7 into master Oct 19, 2026
6 checks passed
@dimslaev
dimslaev deleted the ci/claude-review-reads-diff branch October 19, 2026 22:36
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