Skip to content

RT-246 stopgap: allow rt runs and rt gate in the baseline Claude permissions - #395

Merged
m4ttheweric merged 1 commit into
mainfrom
rt-246-baseline-runs-gate
Sep 24, 2026
Merged

m4ttheweric merged 1 commit into
mainfrom
rt-246-baseline-runs-gate

Conversation

@m4ttheweric

Copy link
Copy Markdown
Collaborator

On a fresh Mac every rt runs and rt gate call a pipeline skill makes from Bash prompts in a board-launched pane, and nobody is watching it. Two baseline entries stop that for the first teammate test; the rt_verb env allowlist on RT-246 stays the end state.

What changed

  • lib/setup/base-permissions.ts: Bash(rt runs *) and Bash(rt gate *), with the why beside the list.
  • lib/setup/__tests__/steps-c.test.ts: the pinned list has ten entries.

Follow-up

  • Existing installs do not receive new baseline entries (RT-245).

Verification

steps-c suite 106/106.

🤖 Generated with Claude Code

…RT-246 stopgap)

The pipeline skills call both from Bash in board-launched panes nobody watches, and on a fresh Mac every call prompted. Both write only the caller's own run and gates, and the mattstack MCP server already exposes the gate verbs. The rt_verb env allowlist (RT-246) remains the end state.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 84 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 4e710e79-026d-4d97-8ee2-4e8c5316b7cf

📥 Commits

Reviewing files that changed from the base of the PR and between f490d84 and 3df62fc.

📒 Files selected for processing (2)
  • lib/setup/__tests__/steps-c.test.ts
  • lib/setup/base-permissions.ts

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

@m4ttheweric
m4ttheweric merged commit 410f227 into main Sep 24, 2026
6 checks passed
@m4ttheweric
m4ttheweric deleted the rt-246-baseline-runs-gate branch September 24, 2026 01:22
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