Skip to content

Add Buf required annotations to standalone Nexus APIs - #879

Draft
stephanos wants to merge 2 commits into
temporalio:mainfrom
stephanos:stephanos/proto-required
Draft

stephanos wants to merge 2 commits into
temporalio:mainfrom
stephanos:stephanos/proto-required

Conversation

@stephanos

@stephanos stephanos commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

What changed?

Add standard Buf required annotations to the eight standalone Nexus requests and Start response run ID, vendor the pinned upstream schema, and document its use.

Why?

Declare zero/empty-field constraints in the API for automatic validation by consuming applications.

Breaking changes

No protobuf wire changes; enforcing the annotations rejects invalid requests, and optional fields retain their existing contracts.

Server PR

API-Go#320 generates the annotations consumed by Server#12364; merge and publish in API → API-Go → Server order.

@@ -0,0 +1,5136 @@
// Copyright 2023-2026 Buf Technologies, Inc.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Vendored just like the other ones like google/.

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