Repository navigation
Conversation
A parked Run has no open Workflow Task, so the server dispatches a query on a task of its own and Core allows nothing beside the answer there. The instance still reported the registered wait set, Core refused the completion, and the query timed out.
The id is Core's, and nothing in this file said where it is defined, so a rename over there would be found by a failing query rather than by grep.
2 of 3 tasks
…8-s1-py-16-parked-run-query
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.
This PR fixes queries against a workflow parked on an external stream.
What changed?
When a completion answers a legacy query,
_emit_external_stream_commandsreturns before it adds any external stream command, so neither the stream snapshot nor the progress marker goes out beside the answer._LEGACY_QUERY_IDis held in step with Core'sLEGACY_QUERY_ID. A new integration case parks a run on an empty stream, queries it, then publishes and checks that the run finishes with no failed Workflow Task.Part of AI-198 (epic AI-37).
Why?
A parked run has no open Workflow Task, so the server sends a query on a task of its own. Core allows nothing beside the answer on that task. The instance still added the wait set it had registered, so Core refused the completion and the query timed out. That activation ran no workflow code, so there's nothing new to report anyway.
It's a bug in Max's prototype, and it bites any query against a parked run. The earlier series placed it before a channel wake PR that made every completion carry a report, which turned every legacy query on a channel server into a failure. That wake is paused with the notification channel per the 2026-10-05 design review, so the fix now stands on its own.
How did you test it?
Link to a test plan if any -
The new case, the whole external stream suite and
tests/streamspass on the dev server the fixtures start. The lint set is clean.