ci: on-demand PR review via /review comment - #6
Merged
Conversation
The requesting-code-review skill reads code-reviewer.md from the plugin cache (~/.claude/plugins), which claude-code-action's repo-only sandbox blocks. Grant read access via project settings (settingSources: project), loaded without touching the workflow file. Track the shared settings file via a .gitignore negation instead of force-add. Verified: review ran full (21 turns, is_error:false) on this branch. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Agents push feature branches as claude-agent-ahrzb (App identity), which claude-code-action blocks by default. Scope allowed_bots to that bot only (not '*') so only our agents' pushes trigger a review. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Public repo — show_full_output publishes the review agent's full transcript to public Actions logs. It was a temporary capture flag; remove it so the merge to master also clears it there. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The review agent produced its findings as final text but never called a GitHub comment tool, and every auto-post flag was off, so nothing landed on the PR. track_progress posts the report as a tracking comment; use_sticky_comment keeps it to one comment per PR across pushes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
track_progress alone was not enough. Per Anthropic's canonical pr-review-comprehensive.yml example, the agent needs the comment tools granted in --allowedTools (mcp__github_inline_comment + gh pr comment) and an explicit instruction to post; otherwise it reviews and exits without writing anything to the PR. Drop use_sticky_comment (not in the recipe). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ahrzb
changed the title
This was referenced Jul 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Turns the review workflow into an on-demand action: comment
/reviewon any PR and Claude reviews it and posts the results.Why
/reviewand not@claude: the official Claude app is already installed on this repo and matches the@claudesubstring — reusing it would fire two reviews per mention./reviewis collision-free.Changes to
.github/workflows/claude-review.yml:pull_request(every push) →issue_comment/pull_request_review_commentcontaining/review, gated to PRs only.mcp__github_inline_comment__create_inline_comment,gh pr comment/diff/view), setstrack_progress: true, and the prompt now explicitly tells the agent to post inline comments + a top-level summary. Matches Anthropic'spr-review-comprehensive.ymlrecipe.show_full_output(public-log transcript leak) anduse_sticky_comment.Can't be exercised on a branch (workflow-triggered runs use the default-branch copy), so it proves out on the first
/reviewafter merge.🤖 Generated with Claude Code