fix(claude): keep permissions.defaultMode on auto - #6
Merged
Conversation
The modify_ script merges its managed block over whatever is on disk, managed winning, so it re-asserted permissions.defaultMode=bypassPermissions on every apply while the live file sat on auto. That was the last standing dot-doctor drift warning, and it was a disagreement about permission posture rather than drift: chezmoi and the running tool each wanted a different answer. auto is the answer. The script now asserts it, so the two agree and the warning cannot recur. skipDangerousModePermissionPrompt is left as it was. It suppresses the confirmation when bypass mode is entered deliberately and does not select the mode, so it is unaffected by this change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
phall1
enabled auto-merge (squash)
August 17, 2026 14:34
phall1
pushed a commit
that referenced
this pull request
Aug 19, 2026
Reverses #6. Requested explicitly; full-autonomy mode is the intended default for this workstation. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The last standing
dot-doctorwarning, resolved in favour ofauto.dot_claude/modify_settings.jsonmerges its managed block over whatever is on disk with managed winning (. * $managed), so it re-assertedpermissions.defaultMode=bypassPermissionson every apply while the live file sat onauto. That is a disagreement about permission posture, not drift — chezmoi and the running tool each wanted a different answer, andchezmoi re-addcannot capture it because amodify_script computes its output.The script now asserts
auto, so the two agree and the warning cannot recur.skipDangerousModePermissionPromptis left as-is: it suppresses the confirmation when bypass mode is entered deliberately and does not select the mode, so it is unaffected.Verification
herdr's
SessionStarthook survives the merge, which is the property the script exists to protect.Expected after merge:
dot-doctor→ no warnings, no failures.🤖 Generated with Claude Code