Skip to content

Add namespace to Nexus operation info#1416

Open
Quinn-With-Two-Ns wants to merge 3 commits intotemporalio:mainfrom
Quinn-With-Two-Ns:NEXUS-283
Open

Add namespace to Nexus operation info#1416
Quinn-With-Two-Ns wants to merge 3 commits intotemporalio:mainfrom
Quinn-With-Two-Ns:NEXUS-283

Conversation

@Quinn-With-Two-Ns
Copy link
Copy Markdown
Contributor

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Apr 6, 2026

What was changed

Add namespace to Nexus operation info

Why?

So an operation handler knows what namespace it is in, we already include this in other SDKs, was just missed in Python

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Low Risk
Small, additive context propagation change; primary risk is minor compatibility impact for any external code that instantiates Info directly due to the new required namespace field.

Overview
Nexus operation handlers can now access the worker’s Temporal namespace via nexus.info().namespace (added to temporalio.nexus.Info alongside task_queue).

The Nexus worker plumbing is updated to propagate the client namespace into both start and cancel operation contexts, and a new test asserts the returned Info includes the expected namespace and task queue.

Reviewed by Cursor Bugbot for commit d76eb40. Bugbot is set up for automated code reviews on this repo. Configure here.

@Quinn-With-Two-Ns Quinn-With-Two-Ns requested a review from a team as a code owner April 6, 2026 21:10
@Quinn-With-Two-Ns Quinn-With-Two-Ns force-pushed the NEXUS-283 branch 2 times, most recently from 91ef578 to 2be8831 Compare April 7, 2026 17:05
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.

4 participants