Skip to content

Remove Claude Code Permissions #2269

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

Open
wants to merge 3 commits into
base: canary
Choose a base branch
from

Conversation

ActuallyTrent
Copy link

@ActuallyTrent ActuallyTrent commented Aug 7, 2025

The permissions file allows all bash commands (Bash(*)), but even if it didn't it still allows many dangerous commands (Bash(rm:*), Bash(node:*)).

While I don't think this in and of itself is really all that exploitable, I do think it increases the exposure risk of community members.

People who want to contribute to the repository may become at risk to something like a compromised MCP server or a model going a bit wild, even if they would normally manually approve commands.

My personal take is that a public repository should not prescribe permissions for something as powerful as Claude Code.

Anthropic's documentation suggest adding .claude/settings.local.json to the .gitignore, as well.


Important

Remove all allowed Bash commands from .claude/settings.local.json to reduce exposure risk.

  • Permissions:
    • Remove all allowed Bash commands from permissions in .claude/settings.local.json, leaving allow list empty.
    • No changes to deny list.

This description was created by Ellipsis for e6cc466. You can customize this summary. It will automatically update as commits are pushed.

Copy link

vercel bot commented Aug 7, 2025

@ActuallyTrent is attempting to deploy a commit to the Boundary Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to e6cc466 in 41 seconds. Click for details.
  • Reviewed 57 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .claude/settings.local.json:4
  • Draft comment:
    Empty 'allow' array: Ensure that disabling all commands is intentional and won’t break expected behavior. Consider adding an inline comment explaining the rationale.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 50% <= threshold 50% The comment is asking the author to ensure that disabling all commands is intentional, which is not allowed. However, it also suggests adding an inline comment explaining the rationale, which is a specific suggestion to improve code clarity. The first part of the comment is not allowed, but the second part is useful.
2. .claude/settings.local.json:7
  • Draft comment:
    Missing trailing newline at end of file. Please add one to follow best practices.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_UnRMeIxDqGv9KJwY

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

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