Skip to content

Backported two upstream changes for the latest dependency set. - #23

Open
moedash wants to merge 2 commits into
task/python-sdk-streamingfrom
moe/AI-198-ch-pyext-0-backports
Open

moedash wants to merge 2 commits into
task/python-sdk-streamingfrom
moe/AI-198-ch-pyext-0-backports

Conversation

@moedash

@moedash moedash commented Oct 3, 2026

Copy link
Copy Markdown
Owner

This PR backports two upstream changes onto task/python-sdk-streaming so the branch builds and tests on today's dependency set.

What changed?

  • The google-genai>=2.21.0 file download fix in contrib/google_genai, and the openai_agents testing fix for the separate httpx and the one-call-id rule, with their tests and the lockfile.
  • main's scripts/gen_nexus_system_api.py, scripts/gen_payload_visitor.py and scripts/nex_gen_support.py, with their regenerated output under temporalio/nexus/system/workflow_service/ and in temporalio/bridge/_visitor.py.

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

Why?

The branch predates the versions CI resolves now, so the contrib suites fail before any streaming code runs. Its Nexus generator also calls a binary nexgen doesn't install. Both changes are plain upstream code, and every later PR on this chain needs a green base. Once the branch is refreshed onto a newer main, this PR can go.

How did you test it?

Link to a test plan if any -

  • Unit Tests
  • Staging
  • End to End Tests

The bridge builds and poe lint is clean. The Gemini, ADK streaming, system Nexus, payload visitor and OpenAI Agents suites pass on a local dev server. The OpenAI cases that need a real key were skipped. The visitor generator gives back the committed file. The Nexus generator doesn't run on this branch's Core pin, because nexgen rejects that pin's workflow-service.wit. The output here comes from the later pin, and the Core-side WIT fix lands lower in the external Core chain.

The branch predates google-genai 2.21 file downloads and the test fixes that
came with the latest dependency set, and CI runs both. In addition, openai
now carries its own httpx, and the Agents SDK rejects two invocations sharing
one completed call id.

(cherry picked from commit 186987b)
The branch's own Nexus generator shells out to a binary name the nexgen crate
does not install, and the payload visitor script still looked for the service
module under its old filename, so generation died before it could diff
anything.

(cherry picked from commit b7cb0b1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant