Skip to content

fix: Write tool needs its own permission rule on Claude Code 2.1.204#1051

Merged
lroolle merged 1 commit into
mainfrom
fix/write-tool-permission
Jul 15, 2026
Merged

fix: Write tool needs its own permission rule on Claude Code 2.1.204#1051
lroolle merged 1 commit into
mainfrom
fix/write-tool-permission

Conversation

@lroolle

@lroolle lroolle commented Jul 15, 2026

Copy link
Copy Markdown
Member

Run 29441419112 transcript: the decider agent wrote correct .decision files but all Write calls were denied. Edit(path) rules only cover Write from Claude Code 2.1.210; the pinned action runs 2.1.204. Added Write(.decision/**) + Bash(mkdir:*).

🤖 Generated with Claude Code

Run 29441419112: agent correctly wrote .decision/ files but Write was
denied - Edit(path) rules do not cover the Write tool until 2.1.210
(whose changelog deprecates Write(path) rules, confirming the mapping is
new). The action's pinned runtime is 2.1.204, so keep both rule forms.
Also allow mkdir (agent creates .decision/ before writing).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 15, 2026 18:49
@lroolle
lroolle merged commit f088ec7 into main Jul 15, 2026
@lroolle
lroolle deleted the fix/write-tool-permission branch July 15, 2026 18:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project’s Claude Code permissions configuration to unblock the pinned Claude Code Action (v2.1.204) from writing .decision files during automated runs.

Changes:

  • Add an explicit Write(.decision/**) permission rule (needed for Claude Code 2.1.204 behavior).
  • Add a Bash(mkdir:*) allow rule to permit directory creation during runs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .claude/settings.json
Comment on lines 20 to 23
"Edit(.decision/**)",
"Write(.decision/**)",
"Bash(mkdir:*)",
"mcp__barkme__notify"
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.

3 participants