Skip to content

feat: proxy the feedback command to the Restless CLI - #1232

Merged
erunion merged 1 commit into
mainfrom
proxy-feedback-debug-cli-975ba8d8
Oct 22, 2026
Merged

erunion merged 1 commit into
mainfrom
proxy-feedback-debug-cli-975ba8d8

Conversation

@mjcuva

@mjcuva mjcuva commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

We added a feedback command in restless to make it easy for agents to submit feedback automatically. Before this was only surfaced in the mcp server, but we are adding it to the cli and surfacing it via a response header (just like the x-api-debug one we already had). This just makes the api module forward api feedback to the restless CLI just like the debug one does.


What

  • packages/api/src/router.ts: feedback joins debug in the set of commands dispatched to npx -y restless@latest. Everything else still falls through to our Commander program.
  • test/router.test.ts: classification cases for feedback, feedback <id>, feedback <id> --message …, and feedback --help.
  • test/bin.test.ts: spawns the real bin with feedback req_12345 --message … against the stubbed npx and checks the arguments and exit code are forwarded. The "other commands" test now names both proxied commands.
  • Changeset: patch for api.

Publish order

restless@0.10.0 already has the feedback command live.

Verification

  • router.test.ts and bin.test.ts: 33 tests, green.
  • oxlint, oxfmt, and tsc --noEmit for packages/api are clean.

🤖 Generated with Claude Code

Add `feedback` to `RESTLESS_COMMANDS` so `npx api feedback <request-id>`
is dispatched to `restless`, the same way `debug` already is. The
command itself lands in restlesshq/onboarding#58 (RES-15).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@mjcuva
mjcuva requested a review from erunion as a code owner October 22, 2026 18:31
@changeset-bot

changeset-bot Bot commented Oct 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a8b4a0b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@erunion
erunion merged commit 6d97886 into main Oct 22, 2026
14 checks passed
@erunion
erunion deleted the proxy-feedback-debug-cli-975ba8d8 branch October 22, 2026 18:52
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