fix: remove --permission-mode plan from reviewer args
Plan mode causes Claude to spend all time on tool calls (Read/Grep) in -p mode, producing empty stdout. Reviewers receive full context (diff, plan, checklist) via the prompt, so file access is not required. Without --permission-mode, -p mode defaults to read-allowed, write-denied. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -62,10 +62,7 @@ _CLAUDE_CODER_ARGS = list(_CLAUDE_BASE_ARGS) + [
|
||||
"bypassPermissions",
|
||||
]
|
||||
|
||||
_CLAUDE_REVIEW_ARGS = list(_CLAUDE_BASE_ARGS) + [
|
||||
"--permission-mode",
|
||||
"plan",
|
||||
]
|
||||
_CLAUDE_REVIEW_ARGS = list(_CLAUDE_BASE_ARGS)
|
||||
|
||||
_CODER_SYSTEM_PROMPT = (
|
||||
"You are a senior software engineer implementing code changes.\n"
|
||||
|
||||
Reference in New Issue
Block a user