Skip to content

fix: remove allowedTools restriction blocking review posting - #7

Merged
toolpathguy merged 1 commit into
mainfrom
fix/claude-review-tools
Jul 22, 2026
Merged

fix: remove allowedTools restriction blocking review posting#7
toolpathguy merged 1 commit into
mainfrom
fix/claude-review-tools

Conversation

@toolpathguy

Copy link
Copy Markdown
Owner

Relates to #5

Problem

The --allowedTools flag in the SDK context acts as a tool registry — it restricts which tools the agent can see at all. Our allowlist only included file-reading tools, which blocked Claude from using the GitHub tools the action injects for posting PR reviews and comments. This caused 13 permission denials per run and max-turns exhaustion ($1+ wasted each time).

Fix

Removed --allowedTools entirely. The prompt already instructs Claude not to run build commands (CI handles that), and the action needs its injected GitHub tools to function. --max-turns 20 remains as cost control.

Risk

Without an allowlist, Claude could theoretically run arbitrary bash commands. However:

  • The action runs with contents: read permissions
  • The prompt explicitly says not to run npm/build commands
  • The 20-turn limit caps total spend
  • The real security boundary is the workflow permissions, not tool restrictions

@toolpathguy
toolpathguy merged commit 4a4af8d into main Jul 22, 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