Skip to content

Make Claude code review post PR comments - #13

Merged
daniel-munoz merged 1 commit into
mainfrom
fix-claude-review-comments
Jul 26, 2026
Merged

Make Claude code review post PR comments#13
daniel-munoz merged 1 commit into
mainfrom
fix-claude-review-comments

Conversation

@daniel-munoz

Copy link
Copy Markdown
Owner

Summary

The Claude Code Review workflow ran successfully on PRs but never posted any comments. Investigation of run 30175987844 (PR #10) showed three compounding causes, all fixed here:

  • Missing --comment flag: the code-review plugin explicitly stops before posting anything to GitHub unless invoked with --comment — findings only went to the workflow log.
  • Read-only permissions: the job granted pull-requests: read / issues: read, so gh pr comment would have failed anyway. Bumped to write, matching the official pr-review-comprehensive.yml example from anthropics/claude-code-action.
  • No tool allowlist: the run logged permission_denials_count: 8. Added --allowedTools via claude_args covering the tools the plugin uses (mcp__github_inline_comment__create_inline_comment, gh pr comment/diff/view/list, etc.).

With --comment, the plugin posts inline comments when it finds issues and a "No issues found" summary comment when the review comes back clean — so every reviewed PR gets visible feedback.

Test plan

🤖 Generated with Claude Code

The review workflow ran but never posted anything to the PR:
- The code-review plugin only posts to GitHub when invoked with
  --comment; without it, findings go to the workflow log only.
- The job token had read-only pull-requests/issues permissions.
- Comment-posting tools were not in the allowed tools list.

Add --comment (which also posts a "No issues found" summary when the
review comes back clean), bump permissions to write, and allowlist the
tools the plugin uses.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@daniel-munoz
daniel-munoz merged commit ce0c3ef into main Jul 26, 2026
2 checks passed
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