summaryrefslogtreecommitdiff
path: root/.github/workflows
diff options
context:
space:
mode:
authorHa Thach <[email protected]>2026-06-02 12:08:56 +0700
committerGitHub <[email protected]>2026-06-02 12:08:56 +0700
commitf78715d8f383ce8962f31e1d9a7f02fb2d8268ec (patch)
tree024f74a625ad2e0fb21a0dcb6b60c5cb6213e1fb /.github/workflows
parent8e0f2bd1c7b234282e0a98b6069780cff21cabe9 (diff)
parent2fc46b690996f776e3b36bcaa9865e7d2a8f8c89 (diff)
Merge pull request #3663 from hathach/claude-workflow-fix-bugs
ci(claude): enable @claude to fix bugs and commit from comments
Diffstat (limited to '.github/workflows')
-rw-r--r--.github/workflows/claude.yml21
1 files changed, 15 insertions, 6 deletions
diff --git a/.github/workflows/claude.yml b/.github/workflows/claude.yml
index bf7a401e4..66e36897c 100644
--- a/.github/workflows/claude.yml
+++ b/.github/workflows/claude.yml
@@ -50,10 +50,19 @@ jobs:
additional_permissions: |
actions: read
- # Optional: Give a custom prompt to Claude. If this is not specified, Claude will perform the instructions specified in the comment that tagged it.
- # prompt: 'Update the pull request description to include a summary of changes.'
+ # Sign the bot's commits so they show as "Verified". The action commits
+ # automatically — on a PR comment it pushes to that PR's branch; on an
+ # issue comment it opens a new claude/* branch + PR with the fix.
+ use_commit_signing: true
- # Optional: Add claude_args to customize behavior and configuration
- # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
- # or https://code.claude.com/docs/en/cli-reference for available options
- # claude_args: '--allowed-tools Bash(gh pr:*)'
+ # No custom prompt: Claude performs the instructions in the @claude comment.
+
+ # Deliberately NO Bash in the tool allowlist. @claude can be summoned on a
+ # fork PR (claude-code-review.yml even directs fork PRs here), and this job
+ # holds the OAuth secret + a write token. Any build/interpreter command
+ # (python -c, cmake/make custom targets, etc.) run against attacker-
+ # controlled PR content is arbitrary code + network execution, so no
+ # command allowlist can safely contain it. Claude still edits files and
+ # the action commits/opens the PR; the resulting commit is verified by the
+ # repo's CircleCI matrix. --max-turns gives room to investigate + fix.
+ claude_args: '--max-turns 30'