Skip to content

Pinned the stream Core and regenerated the stream api protos. - #51

Open
moedash wants to merge 2 commits into
moe/AI-198-if-py-11-query-publish-refusalfrom
moe/AI-198-st-py-1-core-pin
Open

moedash wants to merge 2 commits into
moe/AI-198-if-py-11-query-publish-refusalfrom
moe/AI-198-st-py-1-core-pin

Conversation

@moedash

@moedash moedash commented Oct 3, 2026

Copy link
Copy Markdown
Owner

This PR pins the Core that carries server-side streams and regenerates the Python protos from it.

What changed?

  • temporalio/bridge/sdk-core points at the stream Core layer.
  • Everything else is what poe gen-protos produces from that pin: StreamSlice, StreamRange and StreamStartPosition in temporal.api.stream.v1, the two stream commands, events and failed causes, stream_slices and consumed_stream_ranges on the poll and completion messages, the DeliverStreamRecords job, and the SubscribeStream and AppendStreamRecords commands. The pin forces the bridge visitor changes too.

Part of AI-198 (epic AI-37).

Why?

The native stream layers need these types before any of their code can exist. Keeping the pin and its regen apart means the next PRs show only hand-written code. The numbers follow the channel's: the stream values sit right after the channel values everywhere both touch.

How did you test it?

Link to a test plan if any -

  • Unit Tests
  • Staging
  • End to End Tests

The bridge builds, and cargo clippy -- -D warnings and poe lint are clean. A second poe gen-protos leaves the tree as committed. The client, activity, workflow, replayer, visitor and streams suites pass on the dev server the fixtures start. Nothing uses the new types yet.

The pinned Core carries the stream commands, events, job and api types on their final numbers, so the regen brings them into Python ahead of the native wire.
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