Skip to content

fix(code-review): align README with actual workflow and document required permissions#33006

Open
daniel-dallimore wants to merge 1 commit intoanthropics:mainfrom
daniel-dallimore:fix/code-review-plugin-permissions-docs
Open

fix(code-review): align README with actual workflow and document required permissions#33006
daniel-dallimore wants to merge 1 commit intoanthropics:mainfrom
daniel-dallimore:fix/code-review-plugin-permissions-docs

Conversation

@daniel-dallimore
Copy link
Copy Markdown

Fixes #32459

Summary

  • Corrects outdated documentation: the README described an old version of the plugin (confidence scoring 0-100, threshold 80, Agent 4 as git blame/history analyzer) that no longer matches commands/code-review.md. Updated to reflect the current validation-based architecture.
  • Adds Required Permissions section: documents which tools subagents need (Bash(git *), Read, Glob, Grep) and provides a ready-to-copy settings.json snippet.
  • Surfaces silent failure cause in Troubleshooting: permission denial is now listed as the first and most common cause of "no review output", with a direct link to the new Required Permissions section.

Test plan

  • Read commands/code-review.md and verify the README now accurately describes the workflow (validation agents, not confidence scoring)
  • Confirm the Required Permissions section matches the tools actually used by subagents
  • Confirm "No review output" troubleshooting now mentions permission denial as the primary cause

🤖 Generated with Claude Code

…ired permissions

The README described an outdated version of the plugin (confidence scoring
0-100, threshold 80, git blame agent) that no longer matches the command.
Update it to reflect the current validation-based architecture.

Also adds a Required Permissions section explaining which tools subagents
need, and surfaces permission denial as the primary cause of silent failures
in the Troubleshooting section.

Fixes anthropics#32459

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

code-review plugin silently fails to post PR comments when project settings restrict tool permissions

1 participant