fix(server): keep checked options when ask answer has custom text #217

Merged
lucas merged 1 commit from agent/issue-216-ask-multi-other into main 2026-11-03 12:03:28 +01:00
Collaborator

The agent-facing ask result dropped checked options whenever "Other" text was present. Adds formatAskAnswer producing selected: A, B; custom: <text> (empty parts omitted), used for both single- and multi-question results. details and the web UI are unchanged.

Tests: bun test apps/server/src/bridge/deck-tools.test.ts (3 pass). tsc --noEmit in apps/server reports only pre-existing errors in session-policy.test.ts, unrelated to this change. Manual in-deck check not performed.

Closes: #216

The agent-facing `ask` result dropped checked options whenever "Other" text was present. Adds `formatAskAnswer` producing `selected: A, B; custom: <text>` (empty parts omitted), used for both single- and multi-question results. `details` and the web UI are unchanged. Tests: `bun test apps/server/src/bridge/deck-tools.test.ts` (3 pass). `tsc --noEmit` in apps/server reports only pre-existing errors in `session-policy.test.ts`, unrelated to this change. Manual in-deck check not performed. Closes: #216
Multi-select ask answers that combined checked options with "Other"
text reported only the custom text to the agent. The result now reads
`selected: A, B; custom: <text>`, omitting whichever part is empty.

Co-Authored-By: Claude Opus 5.5 via omp-deck
lucas merged commit ab70833dc1 into main 2026-11-03 12:03:28 +01:00
lucas deleted branch agent/issue-216-ask-multi-other 2026-11-03 12:03:29 +01:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
lucas/omp-deck!217
No description provided.