Repository navigation
fix(server): reveal buffered prose before tool activity - #857
Conversation
|
Deployment failed for project pylon-marketing with the following error: Learn More: https://vercel.com/pylon-code?upgradeToPro=build-rate-limit |
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
…-tool-boundary-20260925
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Refreshed onto current |
When a provider emits an assistant paragraph and then starts a tool before completing the assistant item, Pylon can show the tool activity while the prose stays buffered. This makes the stream appear to stop until a later paragraph or item completion. Flush that turn's buffered assistant text immediately before appending a
tool.startedactivity, preserving the current Markdown-aware buffering and normal delivery rate for prose outside tool boundaries.A regression drives Claude-shaped runtime events through the real ingestion harness: one unfinished paragraph, a tool start, then more assistant text and completion. It asserts the first prose is visible before the tool activity and the final message is complete without duplication. This is a server projection fix shared by web, desktop, and mobile; it does not change provider protocol handling or the user setting for legacy token streaming.
Validation: all 124
ProviderRuntimeIngestiontests, server typecheck, scoped lint and format, andgit diff --checkpassed. One pre-existing unused-import lint warning remains in the touched server file. No live Claude turn was run, so this addresses the identified buffer/tool ordering path rather than claiming every cause of a missing reply is resolved.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.