Repository navigation
Conversation
Pins proto/api to the fork commit that adds Nexus operation progress and the stream notifier RPCs, so the server and SDKs can build against them. The submodule points at moetemp/api because the commit exists only there.
Picks up the api fix that makes progress and NotifyStream carry a string position and string metadata, so the payload visitor no longer walks them.
The notifier WIT now uses WIT maps, an optional run id, a namespace taken from the workflow and an omitted identity.
Picks up the attach request's operation token and start time, which the notifier sends back to the caller, and the 1 KiB position bound.
The api N2 and N3 heads changed only comments and the nexgen pin, so only generated comments and descriptors move.
moetemp
marked this pull request as draft
October 12, 2026 06:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed?
This PR regenerates api-go for the Nexus progress messages and the stream notifier RPCs.
proto/apipoints at api5f7ed18d, withclose_failure..gitmodulesnames moetemp/api while the commit exists only there.stream/v1package, the progress messages, the workflowservice client, server and mocks, and the WIT copy undersystemnexus/.NotifyStreamRequest.close_resultandclose_failure.Why?
The server and the SDK tooling take the API through api-go, and the System Nexus code comes from the same WIT (DD-36).
How did you test it?
Link to a test plan if any -
make proto,go build ./...,go test -tags protolegacy ./...andmake checkpass. The live lane's server builds on this pin.