Update permission patterns to Claude Code v4 format#4
Open
jaypaulb wants to merge 3 commits intodwillitzer:mainfrom
Open
Update permission patterns to Claude Code v4 format#4jaypaulb wants to merge 3 commits intodwillitzer:mainfrom
jaypaulb wants to merge 3 commits intodwillitzer:mainfrom
Conversation
- Create organized directory structure (config/, docs/, scripts/, src/) - Move configuration files to config/ - Move documentation to docs/ - Move scripts to scripts/ - Move wrappers to src/wrappers/ - Add tools directory for future development - Update .gitignore for development files - Update README.md with new structure documentation
- Add dedicated Claude tools section - Implement new tool patterns - Enhance security with deny patterns - Improve documentation and structure
Convert all permission patterns from old format (e.g., "Bash(command *)") to new Claude Code v4 compatible format (e.g., "Bash(command:*)"). - Updated 1,006+ allow and deny list patterns - Ensures compatibility with latest Claude Code settings schema - All 893 allow rules and 149 deny rules converted - Tested and validated JSON structure This addresses the incompatibility between the settings-up feature branch and the current Claude Code settings validation schema.
|
Seems worth merging this :) |
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
3 participants
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.
Summary
This PR updates all permission patterns in
settings.jsonandsettings.jsoncto be compatible with Claude Code v4's new settings schema.Changes
Converted 1,006+ permission patterns from old format to new Claude Code v4 format
Bash(command *)orBash(command*)Bash(command:*)(using:*for prefix matching)Updated files:
settings.json- 893 allow rules, 149 deny rulessettings.jsonc- Same rules with documentation commentsWhy This Matters
The current Claude Code settings schema (used in latest versions) requires the
:*suffix for prefix matching instead of just*. The old format in this repository causes validation errors when users try to use these settings. This conversion makes the settings compatible with the current Claude Code version.Testing
Tool(args:*)formatExample Conversions
Bash(docker ps *)Bash(docker ps:*)Bash(npm run *)Bash(npm run:*)Bash(git commit *)Bash(git commit:*)Bash(curl *)Bash(curl:*)This update ensures the repository's settings work seamlessly with current Claude Code installations.