-
-
Notifications
You must be signed in to change notification settings - Fork 177
chore(repo): Add Claude Code settings with basic permissions #959
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Adds .claude/settings.json to enable Claude Code with: - Basic bash commands (find, ls, git, grep, mv) - WebFetch access to GitHub and Sentry documentation sites Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
|
I am not sure how I feel about this change; I think permissions settings are something that developers should configure for themselves. @lcian what do you think? |
|
@szokeasaurusrex, this is a handy default set of permissions for Claude across all SDKs and developers. You can still keep personal permissions locally for every dev. Sharing defaults helps everyone get started faster, but it’s totally optional—use them if you want, or just close the PR if not. |
|
I think as long as the commands cannot introduce any security problems in the developer's environment, this is not a big deal. |
szokeasaurusrex
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, you have convinced me that having defaults in the repo makes sense. But, I do have some opinions on some of the permissions.
I myself typically am okay with the agent taking more permissions, but I am not using YOLO mode. I also know that other devs may not want their agents to have that many permissions, so the defaults should be conservative imo.
Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
Adds
.claude/settings.jsonto enable Claude Code with:This follows the same pattern used in other Sentry SDK repositories like sentry-javascript, sentry-dart, and sentry-react-native.
Note: This adds .claude/settings.json with -f flag since .claude is in .gitignore for local settings.