Skip to content

fix(coordinator): ask through ask_questions and say its card sits below - #129

Merged
TheGreatAxios merged 11 commits into
mainfrom
cl-11868-preserve-structured-ask_questions-options-when-stripping-raw
Nov 4, 2026
Merged

TheGreatAxios merged 11 commits into
mainfrom
cl-11868-preserve-structured-ask_questions-options-when-stripping-raw

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • The coordinator's opening and constraints steps say only to ask with one ask_questions call; the opening broad question stays in the reply. The per-step caps, ordering and preselect guidance are gone; the schema's limit of five stays.
  • How to ask lives in the tool: its description and its result say the questions show on one card below the reply, which introduces them without repeating them. The coordinator no longer points at "the questions above".

The written-JSON fallback first built here is removed: no stored run shows the model writing questions as JSON, and each review round found new edge cases. Its code, tests and notes are kept in Shelved: written ask_questions JSON fallback on CL-11868.

Verification

  • bun run lint, format:check, typecheck, test (227 pass).
  • Not yet proved on a live model: 3–5 fresh openings on the Anthropic default profile should each show a card and no "above" wording.

Closes CL-11868

@linear-code

linear-code Bot commented Nov 4, 2026

Copy link
Copy Markdown

CL-11868

@corbits-triage corbits-triage Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Triage requests changes:

  • Errors kept: added code matches a code pattern
    • apps/web/src/pages/workspace/visible-chat.ts:152-154 bare-catch: A catch without a binding drops the error; check for the expected failure, or catch it and rethrow the rest.
  • Invented fallbacks: added code matches a code pattern
    • apps/web/src/pages/workspace/visible-chat.ts:145 empty-fallback: Invented fallback (an empty string, list, object or zero); handle the missing value instead.

@corbits-triage

corbits-triage Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@corbits-triage
corbits-triage Bot dismissed their stale review November 4, 2026 06:22

Nothing triage asked to change still fails, so this request no longer stands.

@TheGreatAxios TheGreatAxios changed the title fix: show questions the coordinator writes as JSON as a questions card fix(coordinator): ask through ask_questions and say its card sits below Nov 4, 2026

@corbits-triage corbits-triage Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No defects reported.

@TheGreatAxios
TheGreatAxios added this pull request to the merge queue Nov 4, 2026
Merged via the queue into main with commit fa83ff8 Nov 4, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant