Skip to content

Commit 220b8df

Browse files
ahrzbclaude
andcommitted
ci: actually post the review — grant comment tools + instruct posting
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>
1 parent 3bb3987 commit 220b8df

1 file changed

Lines changed: 16 additions & 7 deletions

File tree

.github/workflows/claude-review.yml

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,14 +28,23 @@ jobs:
2828
allowed_bots: claude-agent-ahrzb # agents push feature branches as this App bot
2929
plugin_marketplaces: https://github.com/anthropics/claude-plugins-official.git
3030
plugins: superpowers@claude-plugins-official
31-
claude_args: --model claude-sonnet-5 --max-turns 20
32-
track_progress: true # post the review to the PR as a comment
33-
use_sticky_comment: true # reuse one comment per PR instead of one per push
31+
# Grant the comment tools — without these the agent has no way to
32+
# write anything back to the PR (this was the "nothing gets posted" bug).
33+
claude_args: >-
34+
--model claude-sonnet-5 --max-turns 30
35+
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"
36+
track_progress: true # tracking comment + PR context
3437
prompt: |
35-
Review this pull request.
38+
Review this pull request. Use the superpowers:requesting-code-review
39+
skill for the review methodology, diffing the PR branch against
40+
${{ github.event.pull_request.base.ref }}.
3641
37-
Use the superpowers:requesting-code-review skill and follow it exactly.
38-
Diff the PR branch against ${{ github.event.pull_request.base.ref }}.
42+
Then POST the review to GitHub — a review that isn't posted is useless:
43+
- inline comments (mcp__github_inline_comment__create_inline_comment)
44+
for specific, line-level issues
45+
- one top-level PR comment (`gh pr comment`) for the overall summary
46+
and assessment
3947
4048
Report only what you verified in the diff. No speculative findings, no
41-
style nits. If nothing needs changing, say so in one line.
49+
style nits. If nothing needs changing, post a one-line top-level
50+
comment saying so.

0 commit comments

Comments
 (0)