Skip to content

RT-150: rt gate ask CLI - #276

Merged
m4ttheweric merged 6 commits into
mainfrom
rt-150
Sep 15, 2026
Merged

m4ttheweric merged 6 commits into
mainfrom
rt-150

Conversation

@m4ttheweric

@m4ttheweric m4ttheweric commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

CLI verb over the daemon's gate:ask command, so callers can open a gate and ask a shared-presentation question from the command line instead of only through the daemon API.

  • rt gate ask: new command-tree leaf wired to daemon gate:ask
  • Always-JSON envelope on the response, with e2e coverage for the success path
  • Refusal-path e2e tests and a usage header line for the new verb
  • Regenerated docs reference (website/docs/reference/gate/)

Note: this verb is inert until the wave-2 daemon restart; RT-164's spawn keys off this merge plus that restart.

Review

CodeRabbit was rate-limited (org cap: 1 review/hour) with zero comments posted; the usual fallback model (fable) was also out of usage credits, so this PR's substitute adversarial review ran on Claude Opus 5. No blocking findings; two doc-nit fixes (unresolvable process-artifact citations) applied as a follow-up commit.

🤖 Generated with Claude Code

m4ttheweric and others added 5 commits September 15, 2026 15:10
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Task 2 added the ask tree node but never regenerated
website/docs/reference/gate/, so bun run docs:check failed. Regenerate via
bun scripts/gen-docs.ts.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Add e2e cases for the two refusals the CLI itself authors via askFail:
malformed --questions JSON and missing --questions. Both happen before any
daemon call, unlike the existing three cases which all exercise askFail
only via the daemon-rejection path.

Also add the ask usage line to the verb summary comment at the top of
commands/gate.ts, which listed every other verb but this one.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 6 seconds.

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: 1733cf02-fd03-46de-9967-fec8e849b276

📥 Commits

Reviewing files that changed from the base of the PR and between 00cb755 and 733d03e.

📒 Files selected for processing (6)
  • commands/__tests__/gate-ask-cli.test.ts
  • commands/gate.ts
  • e2e/tests/gate-ask-cli.test.ts
  • lib/command-tree-def.ts
  • website/docs/reference/gate/ask.mdx
  • website/docs/reference/gate/index.mdx

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

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@m4ttheweric
m4ttheweric merged commit 3b50b70 into main Sep 15, 2026
4 checks passed
@m4ttheweric
m4ttheweric deleted the rt-150 branch September 15, 2026 22:09
m4ttheweric added a commit that referenced this pull request Sep 17, 2026
* gate ask: CLI verb over daemon gate:ask (RT-150)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* gate ask: tree node

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

* gate ask: e2e for the always-JSON envelope

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* gate ask: regenerate docs reference

Task 2 added the ask tree node but never regenerated
website/docs/reference/gate/, so bun run docs:check failed. Regenerate via
bun scripts/gen-docs.ts.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* gate ask: refusal-path e2e tests, usage header line

Add e2e cases for the two refusals the CLI itself authors via askFail:
malformed --questions JSON and missing --questions. Both happen before any
daemon call, unlike the existing three cases which all exercise askFail
only via the daemon-rejection path.

Also add the ask usage line to the verb summary comment at the top of
commands/gate.ts, which listed every other verb but this one.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* rt-150: drop unresolvable process-artifact citations from comments

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

---------

Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
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