Skip to content

fix(claude): drop unmatched Write() deny rule for CHANGELOG.md - #327

Merged
laurigates merged 1 commit into
mainfrom
fix/changelog-deny-edit-rule
Jul 16, 2026
Merged

laurigates merged 1 commit into
mainfrom
fix/changelog-deny-edit-rule

Conversation

@laurigates

Copy link
Copy Markdown
Owner

What

Removes Write(**/CHANGELOG.md) from the permissions.deny overlay in exact_dot_claude/modify_settings.json, keeping only Edit(**/CHANGELOG.md), and updates the deny-backstop example in rules/claude-code-auto-mode.md accordingly.

Why

Claude Code now warns at every launch:

Permission deny rule: Write(/CHANGELOG.md) is not matched by file permission checks — only Edit(path) rules are. Use Edit(/CHANGELOG.md) instead (Edit rules cover all file-editing tools).

Edit(path) rules cover all file-editing tools (Write, Edit, NotebookEdit), so the Write entry was redundant and unmatched. Applied to the live ~/.claude/settings.json via chezmoi apply; the warning no longer fires.

🤖 Generated with Claude Code

https://claude.ai/code/session_012ybMwaDwhwhXsvMfBLk8Gq

Claude Code warns at launch that Write(**/CHANGELOG.md) is not matched
by file permission checks — only Edit(path) rules are, and they cover
all file-editing tools including Write. Keep the Edit rule, drop the
Write duplicate, and update the deny-backstop example in
claude-code-auto-mode.md to match.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012ybMwaDwhwhXsvMfBLk8Gq
@github-actions github-actions Bot added documentation Improvements or additions to documentation claude size/xs labels Jul 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Broken Links Detected

Summary

Status Count
🔍 Total 170
✅ Successful 160
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 9
❓ Unknown 0
🚫 Errors 1

Errors per input

Errors in exact_dot_claude/docs/prds/daily-catchup.PRD.md


Please fix the broken links before merging.

@github-actions

Copy link
Copy Markdown
Contributor

🤖 Claude analyzed the CI failures but determined no code changes are needed.

Failed workflow: https://github.com/laurigates/dotfiles/actions/runs/29499761964

This may indicate:

  • Flaky tests
  • Infrastructure issues
  • Transient failures
  • Configuration problems outside the codebase

Please review the failure logs manually.

laurigates added a commit that referenced this pull request Jul 16, 2026
… files (#328)

## What

Adds two entries to `exact_dot_claude/.chezmoiignore` under the
runtime-files section: `.credentials.json` and `settings.json.backup.*`.

## Why

`~/.claude` is an `exact_` tree, so any unlisted top-level entry is
deleted on `chezmoi apply`. While preparing #327, `chezmoi status
~/.claude` showed pending `D` lines for:

- `.credentials.json` — Claude Code auth state; deleting it would log
the CLI out
- `settings.json.backup.<timestamp>` — written by Claude Code
(2026-07-16) before migrating settings

Neither was registered in `.chezmoiignore`, so the next apply would have
silently removed both. This is the same failure class as the
`friction-reports/` deletion incident documented in
`chezmoi-conventions.md`.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_012ybMwaDwhwhXsvMfBLk8Gq

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
@laurigates
laurigates merged commit 437e1ec into main Jul 16, 2026
6 of 8 checks passed
@laurigates
laurigates deleted the fix/changelog-deny-edit-rule branch July 16, 2026 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude documentation Improvements or additions to documentation size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant