Skip to content

Add Standalone Nexus Operation links#2932

Open
Quinn-With-Two-Ns wants to merge 4 commits into
temporalio:masterfrom
Quinn-With-Two-Ns:NEXUS-445
Open

Add Standalone Nexus Operation links#2932
Quinn-With-Two-Ns wants to merge 4 commits into
temporalio:masterfrom
Quinn-With-Two-Ns:NEXUS-445

Conversation

@Quinn-With-Two-Ns

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What was changed

Add Standalone nexus operation links

Why?

So SANO has nice links in the UI

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

Note

Medium Risk
Changes how Nexus and workflow RPCs attach links (including completion callbacks), which affects UI correlation; behavior is well-tested but mistakes could drop or mis-route links across SANO, workflows, and signal-with-start.

Overview
Adds bidirectional link plumbing for standalone Nexus operations (SANO): the SDK now converts and forwards Link.NexusOperation alongside existing Link.WorkflowEvent links through the Nexus handler path.

LinkConverter gains nexusOperationToNexusLink / nexusLinkToNexusOperation (using temporal:///…/nexus-operations/…/details URLs), plus linkToNexusLink and nexusLinkToLink dispatchers. NexusTaskHandlerImpl, InternalUtils (Nexus-bound workflow start), and NexusStartWorkflowHelper are switched to these APIs so inbound SANO links attach to handler RPCs (e.g. signal-with-start) and response workflow-event links round-trip on StartOperationResponse.

Integration tests cover SANO ↔ backing-workflow links and SANO-triggered signal-with-start back/forward links (external server only). Unit coverage for Nexus operation URL encoding/parsing is in LinkConverterTest.

Reviewed by Cursor Bugbot for commit 52c8fe6. 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 June 26, 2026 22:38
@Quinn-With-Two-Ns Quinn-With-Two-Ns changed the title Nexus 445 Add Standalone Nexus Operation links Jun 26, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

Reviewed by Cursor Bugbot for commit 487b9df. Configure here.

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