Skip to content

🧈 [claude] allow gh repo#116

Merged
chicks-net merged 1 commit intomainfrom
chicks/2026-03-21-allow-gh-repo
Mar 21, 2026
Merged

🧈 [claude] allow gh repo#116
chicks-net merged 1 commit intomainfrom
chicks/2026-03-21-allow-gh-repo

Conversation

@chicks-net
Copy link
Copy Markdown
Member

Done

  • 🧈 [claude] allow gh reop

Meta

(Automated in .just/gh-process.just.)

Copilot AI review requested due to automatic review settings March 21, 2026 18:00
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Claude tool-execution allowlist to permit running additional GitHub CLI (gh) repo commands, likely to support existing automation that queries repository metadata.

Changes:

  • Add Bash(gh repo:*) to .claude/settings.local.json permissions allowlist.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"Bash(gh pr view:*)",
"Bash(gh pr:*)",
"Bash(gh release list:*)",
"Bash(gh repo:*)",
Copy link

Copilot AI Mar 21, 2026

Choose a reason for hiding this comment

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

Bash(gh repo:*) is an extremely broad permission in an allowlist and includes destructive subcommands (e.g., gh repo delete, gh repo deploy-key add). Please narrow this to only the specific gh repo … subcommands/flags needed (for example, gh repo view for .just/compliance.just) to avoid granting unnecessary write/delete capabilities.

Suggested change
"Bash(gh repo:*)",
"Bash(gh repo view:*)",

Copilot uses AI. Check for mistakes.
@chicks-net chicks-net merged commit 5e8e296 into main Mar 21, 2026
26 of 32 checks passed
@chicks-net chicks-net deleted the chicks/2026-03-21-allow-gh-repo branch March 21, 2026 18:04
@chicks-net chicks-net changed the title 🧈 [claude] allow gh reop 🧈 [claude] allow gh repo Mar 21, 2026
@chicks-net
Copy link
Copy Markdown
Member Author

oops merged, with typo left in title 😢

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.

2 participants