FE-559: Migrate: @anthropic-ai/claude-agent-sdk query() -> @anthropic-ai/sdk client.messages.stream()#25
Merged
Merged
Conversation
FE-559 Migrate: `@anthropic-ai/claude-agent-sdk query()` -> `@anthropic-ai/sdk client.messages.stream()`
An ad-hoc migration and re-adaptation due to discovering significant known bugs in claude agent SDK |
This was referenced Apr 3, 2026
Contributor
Author
This was referenced Apr 3, 2026
lunelson
changed the title
|
This pull request is abnormally large and would use a significant amount of tokens to review. If you still wish to review it, comment |
Contributor
Author
lunelson
changed the base branch from
ln/fe-538-entity-sidebar
to
graphite-base/25
Agent SDK MCP tool registration broken (A2 invalidated). Spike proves @anthropic-ai/sdk client.messages.stream() with tool_choice forcing reliably calls ask_question. 3/3 tests pass: 1. Forced tool_choice → structured question args 2. Raw stream events match content_block_start/delta/stop format 3. Observer JSON extraction works via direct API call New: A26 (validated), D30. Invalidated: A2, A24, A25. Unblocks slices 7+ (phase transitions and beyond). Amp-Thread-ID: https://ampcode.com/threads/T-019d4eaf-1eb6-75cf-ab50-c2acf1c59c14 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
…ipping Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4ecf-de8a-7258-b074-08887c6a66bf Co-authored-by: Amp <amp@ampcode.com>
lunelson
force-pushed
the
ln/fe-559-migrate-sdk
branch
from
a3c46d1 to
5014c7c
Compare
lunelson
added a commit
that referenced
this pull request
Apr 7, 2026
## Stack Context This stack continues from PR #25 by remodeling choice-turn replies into structured turn responses. PR #25 establishes the migration and live-turn-card seam; this PR carries the flexible response model above it. ## What? - persist choice-turn replies as structured `data-turn-response` parts with a human-readable summary seam - support zero, one, or many selected options plus optional free-text - keep transcript hydration, resume, and interviewer-context projection aligned with the structured response shape ## Why? The old flow assumed one immediate option selection mapped to one scalar answer string. This branch retires that assumption so the interview UI can capture richer responses without losing persistence, hydration, or context coherence. It also formalizes the split from `a3c46d1`, so the response-model work now lives on its own FE-564 branch and PR.
This was referenced Apr 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stack Context
This stack finishes the SDK migration branch and then layers the flexible turn-response model on top of it.
PR #25 is the parent branch for that work; PR #27 carries the response-model changes.
What?
Why?
The FE-564 response-model work needed to be split out on top of
a3c46d1.That makes this PR the foundation branch again: it now owns the migration, workspace stabilization, and live turn-card projection that the child branch depends on.
Updating the metadata keeps the stack legible after the branch split.