Skip to content

ci: expand claude-code-action allowlist and enable progress tracking#501

Merged
kronosapiens merged 1 commit intomainfrom
kronosapiens/claude-action-tools
Apr 22, 2026
Merged

ci: expand claude-code-action allowlist and enable progress tracking#501
kronosapiens merged 1 commit intomainfrom
kronosapiens/claude-action-tools

Conversation

@kronosapiens
Copy link
Copy Markdown
Collaborator

Follow-up to #500. The previous run on issue #498 still produced no branch or PR — logs (now visible thanks to show_full_output: true) show 6 denials: four git checkout -b/git switch -c attempts and two Edit calls on docs/pages/framework/systems/index.md.

Root causes:

  • --allowedTools is treated as a strict allowlist in the Claude Code SDK, so setting it to only Bash(pnpm:*),Bash(gh:*) silently dropped Edit, Write, Read, etc. The docs' own example re-lists them, which I missed.
  • Even with use_commit_signing: false, Claude uses raw git checkout -b to create the working branch (MCP file ops only handle commits, not branching), so Bash(git:*) is required.

Also enables track_progress: true so @claude mentions get the eyes reaction and a live sticky status comment — matching the UX in other repos.

The previous allowlist ("Bash(pnpm:*),Bash(gh:*)") is treated as a
strict set — it silently dropped Edit, Write, Read, and the git-branch
commands Claude uses before committing. Re-list the file-ops tools
explicitly, add Bash(git:*) for branch creation, and enable
track_progress so @claude mentions get the eyes reaction and a live
sticky status comment.
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
book Ready Ready Preview Apr 22, 2026 6:57pm

Request Review

@kronosapiens kronosapiens merged commit 59b147c into main Apr 22, 2026
2 checks passed
@kronosapiens kronosapiens deleted the kronosapiens/claude-action-tools branch April 22, 2026 19:13
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