Skip to content

rt-client: gate answers carry an optional text replacement (0.30.0) - #376

Merged
m4ttheweric merged 3 commits into
mainfrom
answer-text-field
Sep 23, 2026
Merged

m4ttheweric merged 3 commits into
mainfrom
answer-text-field

Conversation

@m4ttheweric

Copy link
Copy Markdown
Collaborator

Gate answers carry an optional text replacement

The board's respond post step is gaining an edit button for a drafted reply. The edited reply travels on the answer's object form as text, beside value and note, so this declares and validates it.

What changed

rt-client (0.29.0 -> 0.30.0)

  • GateAnswer answers accept { value, note?, text? }
  • validateGateAnswers rejects a non-string text (question <id> text must be a string) and a whitespace-only one (question <id> text must not be empty)
  • note keeps its meaning (a comment to the agent); text is replacement text for something the gate offered

MCP

  • gate_answer's answer schema allows text in the object form

Also

  • bun.lock records the workspace's new version

The daemon already stores and returns an answers object verbatim, so an older daemon carries text untouched; this release adds the validation.

Testing

  • 5 new validator tests and a schema test; bun test packages lib/mcp green; dist rebuilt, dist-freshness green

🤖 Generated with Claude Code

m4ttheweric and others added 2 commits September 22, 2026 23:42
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 23 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 82 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: 2e1a0520-3424-441c-9286-6f2eb05e8273

📥 Commits

Reviewing files that changed from the base of the PR and between 937dc03 and ddebc20.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (7)
  • lib/daemon/__tests__/gates-handlers.test.ts
  • lib/mcp/__tests__/tools.test.ts
  • lib/mcp/tools.ts
  • packages/rt-client/package.json
  • packages/rt-client/src/commands.ts
  • packages/rt-client/src/gate-answers.ts
  • packages/rt-client/test/gate-answers.test.ts

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

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@m4ttheweric
m4ttheweric merged commit cde0f07 into main Sep 23, 2026
6 checks passed
@m4ttheweric
m4ttheweric deleted the answer-text-field branch September 23, 2026 13:16
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