Repository navigation
fix(tui): surface silent background task completions - #222
Merged
kingsword09 merged 3 commits intoOct 19, 2026
Merged
Conversation
Two paths let a finished background task disappear silently. A turn handed off from a completed task is task-scoped out of the main transcript, so its response was only reachable from /tasks and the assistant appeared to ignore the completion notice; render the settled handoff response in the transcript instead. And on session resume, restored tasks whose terminal notifications were already delivered in the conversation were re-announced as fresh system events; classify restored tasks against the transcript and skip the ones a prior turn already consumed. Change-Id: Ia18ac9696142b07e0e7a95c4fdba446cec93c76f
tae9898
marked this pull request as draft
October 19, 2026 02:26
The feature smoke guarded the old contract where every task-scoped string stayed out of the foreground transcript, so the handoff rendering from f8b393b fails CI on its own fixture text. Assert the settled responses appear in the transcript now, while reasoning, tool names, and nested agent trees must still stay scoped to /tasks. Change-Id: I5c6b840901247c352d518842d31b7338240031d0
Contributor
|
pkg-pr-new preview for commit npx --yes https://pkg.pr.new/zcode-app-cli@b8b709eThis preview uses the selected commit. PR updates publish automatically after CI succeeds; kingsword09 can also request a preview with |
Contributor
Author
|
The feature smoke asserted the pre-existing contract that no task-scoped string reaches the foreground |
tae9898
marked this pull request as ready for review
October 19, 2026 04:32
Contributor
|
pkg-pr-new preview for commit npx --yes https://pkg.pr.new/zcode-app-cli@50685c8This preview uses the selected commit. PR updates publish automatically after CI succeeds; kingsword09 can also request a preview with |
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
2 participants
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.
Two paths let a finished background task disappear silently. A turn handed off from a completed task is task-scoped out of the main transcript, so its response was only reachable from /tasks and the assistant appeared to ignore the completion notice; render the settled handoff response in the transcript instead. And on session resume, restored tasks whose terminal notifications were already delivered in the conversation were re-announced as fresh system events; classify restored tasks against the transcript and skip the ones a prior turn already consumed.