Skip to content

Answered a query against a parked external-stream workflow. - #43

Open
moedash wants to merge 2 commits into
moe/AI-198-if-pyext-6-channel-wakefrom
moe/AI-198-if-pyext-7-parked-run-query
Open

moedash wants to merge 2 commits into
moe/AI-198-if-pyext-6-channel-wakefrom
moe/AI-198-if-pyext-7-parked-run-query

Conversation

@moedash

@moedash moedash commented Oct 3, 2026

Copy link
Copy Markdown
Owner

This PR fixes queries against a workflow parked on an external stream.

What changed?

  • When a completion answers a legacy query, _emit_external_stream_commands returns before it adds the stream snapshot, the progress marker or the channel report.
  • _LEGACY_QUERY_ID is held in step with Core's LEGACY_QUERY_ID.
  • A new integration case queries a parked run and gets its answer.

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.

How did you test it?

Link to a test plan if any -

  • Unit Tests
  • Staging
  • End to End Tests

Without the fix, the new case times out. With it, the case and the whole external stream suite pass on the dev server, and poe lint is clean.

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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Changelog entry rides another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant