Skip to content

Allow Read, Glob, and Grep tools in Claude settings#23

Merged
agarcher merged 1 commit intomainfrom
allow-read-search-permissions
Jan 12, 2026
Merged

Allow Read, Glob, and Grep tools in Claude settings#23
agarcher merged 1 commit intomainfrom
allow-read-search-permissions

Conversation

@agarcher
Copy link
Owner

@agarcher agarcher commented Jan 12, 2026

Summary

  • Add permissions for Read(**), Glob(**), and Grep(**) tools in Claude Code settings
  • Uses ** pattern to allow file reading and searching within the current directory tree
  • Prevents Claude from getting stuck waiting for permission prompts during codebase exploration

Test plan

  • Verify Claude Code can read files without permission prompts
  • Verify Glob/Grep searches work within the repo
  • Confirm operations outside the repo still require permission

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Expanded allowed operations in configuration settings to support additional Read, Glob, and Grep operations.

✏️ Tip: You can customize this high-level summary in your review settings.

Add permissions for file reading and searching within the current
directory tree. Uses ** pattern to allow operations relative to
the working directory without hardcoding paths.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Jan 12, 2026

📝 Walkthrough

Walkthrough

Three new permissions (Read, Glob, and Grep) are added to the allow list in the .claude/settings.json configuration file, expanding the set of permitted operations without modifying existing entries or control flow.

Changes

Cohort / File(s) Summary
Configuration Permissions
\.claude/settings\.json
Added three new permission patterns: Read(**), Glob(**), and Grep(**) to the allow list, extending permitted operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰✨ Three new keys we gently place,
Read, Glob, Grep—permissions grace!
Our settings now dance, expanded and wide,
New powers unlocked, nothing denied! 🔓

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change—adding Read, Glob, and Grep tool permissions to Claude settings file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e518431 and 4b655f1.

📒 Files selected for processing (1)
  • .claude/settings.json
🔇 Additional comments (1)
.claude/settings.json (1)

4-6: LGTM! Permissions appropriately scoped for codebase exploration.

The ** recursive glob pattern grants Claude Code unrestricted read and search access within the working directory tree, which aligns with the PR objective of preventing permission prompts during codebase exploration. This is a reasonable trade-off for developer productivity.

Per the test plan, verify that operations targeting paths outside the repository root still trigger permission prompts as expected.


Comment @coderabbitai help to get the list of available commands and usage tips.

@agarcher agarcher merged commit 3ea82dd into main Jan 12, 2026
5 checks passed
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.

1 participant