Add support for Stand Alone Nexus Operations by Quinn-With-Two-Ns · Pull Request #2280 · temporalio/sdk-go

Quinn-With-Two-Ns · 2026-04-08T16:47:56Z

Add standalone Nexus operation support

Adds the ability to start, poll, describe, cancel, terminate, list, and count Nexus operations directly from the
client, outside of workflow context. This mirrors the standalone activity pattern and is based on the API we already reviewed the cross SDK design for.

API

client.NewNexusClient(options) — creates a NexusClient bound to an endpoint+service
NexusClient.ExecuteOperation(ctx, operation, input, options) — starts an operation and returns a handle
NexusOperationHandle.GetID() / GetRunID() / Get() / Describe() / Cancel() / Terminate() — handle operations
client.GetNexusOperationHandle(options) — creates a handle to an existing operation (no network call)
client.ListNexusOperations(ctx, options) — lists operations via visibility query
client.CountNexusOperations(ctx, options) — counts operations via visibility query

Interceptor support

Adds 6 methods to ClientOutboundInterceptor: ExecuteNexusOperation, GetNexusOperationHandle, CancelNexusOperation,
TerminateNexusOperation, DescribeNexusOperation, PollNexusOperationResult. List and Count bypass the interceptor,
consistent with standalone activities.

Testing

Unit tests for interceptor header propagation, client validation, and operation name resolution
Integration tests covering execute+get, describe, get handle, cancel, terminate, list, count, and validation

Note

Medium Risk
Adds new experimental client APIs and interceptor hooks for starting/polling/canceling Nexus operations, which touches core client surface area and gRPC request handling. Risk is mitigated by new unit/integration coverage but may impact compatibility for custom interceptors/mocks.

Overview
Adds experimental standalone Nexus operation support to the public client API, including NewNexusClient, NexusClient.ExecuteOperation, NexusOperationHandle (get/describe/cancel/terminate), plus ListNexusOperations and CountNexusOperations visibility queries.

Implements the feature end-to-end in internal/internal_nexus_client.go with new gRPC calls, typed options/metadata structs, operation-name resolution, and long-poll result retrieval; extends ClientOutboundInterceptor with Nexus-related methods and updates mocks/tests accordingly.

Updates integration/unit tests to cover the new standalone flow (including endpoint provisioning, handle behaviors, cancel/terminate, list/count) and adjusts CI/dev-server config to use a Nexus-capable dev server and to optionally disable the new tests.

^{Reviewed by Cursor Bugbot for commit 8408739. Bugbot is set up for automated code reviews on this repo. Configure here.}

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3fe42e708a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-04-08T17:22:34Z

+		// ID - The business identifier of the operation.
+		//
+		// Mandatory: No default.
+		ID string


Rename start option field to OperationID for API consistency

ClientStartNexusOperationOptions introduces an ID field while the rest of the new Nexus API uses OperationID (GetNexusOperationHandleOptions, metadata structs, and the integration test callsites). This inconsistency makes normal usage patterns fail at compile time (unknown field OperationID) and effectively breaks the newly added standalone Nexus start API for callers who follow the surrounding API naming. Aligning this field name now avoids shipping an immediately incompatible public surface.

Useful? React with 👍 / 👎.

This is to be consistent with Stand alone activities naming approach

Quinn-With-Two-Ns · 2026-04-21T22:09:32Z

Note this PR is missing support for otel I was planning on addressing that in a separate PR to keep the scope down

jmaeagle99 · 2026-05-15T19:26:14Z

+	}
+}
+
+func (w *workflowClientInterceptor) DescribeNexusOperation(


Should cancel, describe, and terminate check that the operation ID is not empty?

VegetarianOrc

Couple of questions, but looks good!

VegetarianOrc · 2026-05-15T19:15:15Z

          DISABLE_SERVER_1_27_TESTS: "1"
          DISABLE_PRIORITY_TESTS: "1"
          DISABLE_STANDALONE_ACTIVITY_TESTS: "1"
+          DISABLE_STANDALONE_NEXUS_TESTS: "1"


This is following the existing pattern, but should we be defaulting to exclude all these tests in CI? Or maybe I am misunderstanding the setup.

We should definitely revisit how we do this, last time I checked there was definitely some that could be removed at this point. I can add to TODO for myself

The logic here just excludes when we are using docker to test since the docker tests don't use the pre-release CLI

VegetarianOrc · 2026-05-15T21:04:28Z

 		_, err := nexusclient.ExecuteOperation(ctx, nc, syncOp, "timeout", nexus.StartOperationOptions{
 			// Force shorter timeout to speed up the test and get a response back.
-			Header: nexus.Header{nexus.HeaderRequestTimeout: "300ms"},
+			Header: nexus.Header{nexus.HeaderRequestTimeout: "5s"},


Is this change needed/related? Was 300ms too aggressive and causing flakes?

Yeah, specifically the endpoint registry is eventually consistent right now so under load it can take too long for the endpoint to become visible

yuandrew · 2026-05-18T17:14:23Z

+
+	// NexusClient is the client for starting Nexus operations bound to a specific endpoint and service.
+	// This is for standalone Nexus operations outside of workflow context.
+	// For Nexus operations within workflows, use workflow.NexusClient.


Do we want to also mention in workflow.NexusClient that we have client.NexusClient for standalone operations?

cursor

Cursor Bugbot has reviewed your changes using default mode and found 1 potential issue.

^{Reviewed by Cursor Bugbot for commit fe11924. Configure here.}

Quinn-With-Two-Ns marked this pull request as ready for review April 8, 2026 17:16

Quinn-With-Two-Ns requested a review from a team as a code owner April 8, 2026 17:16

cursor Bot reviewed Apr 8, 2026

View reviewed changes

Comment thread mocks/Client.go

chatgpt-codex-connector Bot reviewed Apr 8, 2026

View reviewed changes

jmaeagle99 reviewed Apr 10, 2026

View reviewed changes

cursor Bot reviewed Apr 10, 2026

View reviewed changes

Comment thread internal/internal_nexus_client.go

jmaeagle99 reviewed Apr 13, 2026

View reviewed changes

Comment thread test/integration_test.go

Comment thread test/integration_test.go

chrsmith self-assigned this Apr 23, 2026

Quinn-With-Two-Ns force-pushed the NEXUS-289 branch from d296109 to 8d6e09e Compare May 7, 2026 15:34

Quinn-With-Two-Ns requested a review from jmaeagle99 May 13, 2026 14:03

Quinn-With-Two-Ns force-pushed the NEXUS-289 branch 2 times, most recently from e77a1be to 0fb81e4 Compare May 14, 2026 17:33

cursor Bot reviewed May 14, 2026

View reviewed changes

Comment thread internal/internal_nexus_client.go

Quinn-With-Two-Ns force-pushed the NEXUS-289 branch from 0fb81e4 to 1da6fb7 Compare May 15, 2026 15:46

jmaeagle99 approved these changes May 15, 2026

View reviewed changes

VegetarianOrc approved these changes May 15, 2026

View reviewed changes

yuandrew approved these changes May 18, 2026

View reviewed changes

Quinn-With-Two-Ns force-pushed the NEXUS-289 branch from 1da6fb7 to fe11924 Compare May 19, 2026 16:07

cursor Bot reviewed May 19, 2026

View reviewed changes

Comment thread internal/internal_nexus_client.go

Quinn-With-Two-Ns added 10 commits May 19, 2026 10:38

Add support for Stand Alone Nexus Operations

63e7d29

Fix some naming issues

619f5fe

Fix ID in test

a1286f9

Respond to PR comments

ebc15e2

Update API

c828f3e

Use real test version

3a79139

Remove DISABLE_STANDALONE_NEXUS_TESTS for dev server test

5329557

Fix test failure

61324bb

Add async test

3e1b7fc

Fix DC

5ec68ba

Quinn-With-Two-Ns added 2 commits May 19, 2026 10:38

Fix CI

88e2e03

Respond to PR comments

8408739

Quinn-With-Two-Ns force-pushed the NEXUS-289 branch from fe11924 to 8408739 Compare May 19, 2026 17:38

Quinn-With-Two-Ns enabled auto-merge (squash) May 19, 2026 17:46

Quinn-With-Two-Ns merged commit edfc506 into temporalio:main May 19, 2026
56 of 59 checks passed

Conversation

Quinn-With-Two-Ns commented Apr 8, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Add standalone Nexus operation support

API

Interceptor support

Testing

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector Bot Apr 8, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Quinn-With-Two-Ns commented Apr 21, 2026

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

VegetarianOrc left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Quinn-With-Two-Ns commented Apr 8, 2026 •

edited by cursor Bot

Loading