chore: drop unmatched MultiEdit permission rule from .claude/settings.json - #1352
Conversation
….json Claude Code gates every file-editing tool on Edit(path) rules and no longer matches MultiEdit(path). The MultiEdit(reports/**) entry is dead config and prints a warning on every launch: Permission allow rule (.claude\settings.json): MultiEdit(reports/**) is not matched by file permission checks — only Edit(path) rules are. Use Edit(reports/**) instead (Edit rules cover all file-editing tools). Edit(reports/**) is already present, so behavior is unchanged.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe Claude permissions configuration removes ChangesClaude permissions
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This removes a redundant permission entry while preserving the existing Edit permission for reports, so no functional access change or actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description follows the repository template. It explains the dead permission rule, lists the change, identifies the refactoring type, records testing as not applicable, and includes compatibility, documentation, issue, and additional notes. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thanks! Good to know that the rule no longer fits |
Description
Claude Code gates every file-editing tool (Edit, Write, MultiEdit, NotebookEdit) on
Edit(path)permission rules and no longer matchesMultiEdit(path)rules. TheMultiEdit(reports/**)entry in.claude/settings.jsonis dead config, and current Claude Code builds print this on every launch in the repo:Edit(reports/**)is already in the allow list, so removing the line changes nothing about what is permitted.Type of Change
Changes Made
"MultiEdit(reports/**)"frompermissions.allowin.claude/settings.jsonCompatibility / Package Source
Testing/Screenshots/Recordings
Documentation Updates
Related Issues
None.
Additional Notes
Config-only change to the Claude Code project settings; no package or server code touched. The remaining
Edit(reports/**)rule is the exact replacement the warning itself recommends.Summary by CodeRabbit