diff options
| author | hathach <[email protected]> | 2026-06-30 17:04:32 +0700 |
|---|---|---|
| committer | hathach <[email protected]> | 2026-06-30 17:04:32 +0700 |
| commit | 2bedc60e8335e33abe6c0ed24a1785e5b0e90a8d (patch) | |
| tree | aec162022e06433cb61d8c516f142626d43bebce | |
| parent | c52a4a37f0505bb020dd4f121dfe3908b8b30783 (diff) | |
ci(claude-review): trim allowlist to Bash (the only gated tool)
Every blocked call in the review log was a compound Bash pipeline; Read/Grep/
Glob/Task already ran un-prompted, so only bare Bash needs allowlisting.
Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
| -rw-r--r-- | .github/workflows/claude-code-review.yml | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/.github/workflows/claude-code-review.yml b/.github/workflows/claude-code-review.yml index 88d435c00..18659e2c7 100644 --- a/.github/workflows/claude-code-review.yml +++ b/.github/workflows/claude-code-review.yml @@ -53,15 +53,14 @@ jobs: # TEMPORARY: expose the full Claude transcript in the Actions log for # debugging. Revert to remove once done. show_full_output: true - # /code-review needs git/gh (Bash), file search, and the sub-agents it - # fans out (Task). This job runs ONLY on same-repo PRs (the `if` above) - # with a contents:read token that cannot push — so, unlike claude.yml's - # fork-exposed @claude job, allowlisting these is safe. Without it the - # headless run stalls on per-tool approval and can't read the diff or - # spawn reviewers. (Bash is broad; scope it to git/gh/grep if preferred.) + # The review's only blocked calls were COMPOUND Bash commands (sed/python/ + # grep pipelines): default permission mode splits a multi-op command and + # gates the parts not on its allowlist. Bare `Bash` allows them all; + # Read/Grep/Glob/Task already run un-prompted. Safe here (unlike claude.yml): + # same-repo-only + contents:read token that cannot push. claude_args: | --max-turns 50 --model claude-opus-4-8 --effort max - --allowedTools Bash,Read,Grep,Glob,Task,WebFetch,WebSearch,TodoWrite + --allowedTools Bash # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md |
