Skip to content

chore(perms): allow brew install + nix CLI + darwin-rebuild#4904

Merged
AceHack merged 1 commit into
mainfrom
chore/allow-brew-nix-perms-2026-05-24
May 25, 2026
Merged

chore(perms): allow brew install + nix CLI + darwin-rebuild#4904
AceHack merged 1 commit into
mainfrom
chore/allow-brew-nix-perms-2026-05-24

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 25, 2026

Summary

Adds 12 Bash permission patterns to `.claude/settings.json` so the agent can run `brew install` (Nix), the full nix CLI surface, and `darwin-rebuild` for nix-darwin's `linux-builder` activation.

These are needed to build the installer ISO from the flake locally on Apple Silicon Macs via the nix-darwin `linux-builder` path — uses Apple's Virtualization.framework + Rosetta 2 for Rosetta-accelerated x86_64 Linux builds, no third-party VM software required.

Permissions added

Pattern Why
`Bash(brew install *)` Install Nix and any future cluster CLI tooling
`Bash(brew install --cask *)` Cask installs if needed (e.g. orbstack alternative)
`Bash(brew upgrade *)`, `Bash(brew list *)`, `Bash(brew --version)` Brew maintenance
`Bash(nix *)` The main Nix CLI (`nix build`, `nix flake`, etc.)
`Bash(nix-env *)`, `Bash(nix-shell *)`, `Bash(nix-build *)`, `Bash(nix-store *)`, `Bash(nix-channel *)` Legacy nix command surface
`Bash(darwin-rebuild *)` Activate nix-darwin config changes (linux-builder enable, etc.)

Test plan

  • markdownlint passes (settings.json is JSON, not markdown)
  • Post-merge: agent successfully runs `brew install nix` and proceeds with the ISO build workflow

Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

Copilot AI review requested due to automatic review settings May 25, 2026 03:15
@AceHack AceHack enabled auto-merge (squash) May 25, 2026 03:15
@AceHack AceHack merged commit 7565e93 into main May 25, 2026
26 of 27 checks passed
@AceHack AceHack deleted the chore/allow-brew-nix-perms-2026-05-24 branch May 25, 2026 03:16
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 942546248a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .claude/settings.json
@@ -1,10 +1,4 @@
{
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restore _ip_risk_acceptance attribution block

This change drops the only _*_acceptance metadata from .claude/settings.json, so edits to docs/research/ip-questionable/** can still proceed (via the existing broad Bash(gh api *) allow rule) without the named-human risk attribution required by .claude/rules/human-audit-and-legal-risk-acceptance-pattern-in-settings.md (“MUST attach a named human” with operator/scope/policy/see_also). That creates an auditability regression: the risk-acceptance trail documented for this scope is no longer present at the command-permission surface where it was previously anchored.

Useful? React with 👍 / 👎.

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