Generate nexus service definitions#245
Conversation
|
|
|
|
||
| ##### Compile proto files for go ##### | ||
| grpc: http-api-docs go-grpc copy-helpers | ||
| NEXUS_RPC_GEN_SRC ?= $(HOME)/git/github.com/nexus-rpc/nexus-rpc-gen |
There was a problem hiding this comment.
Fix this before merging please.
| grpc: http-api-docs go-grpc copy-helpers | ||
| NEXUS_RPC_GEN_SRC ?= $(HOME)/git/github.com/nexus-rpc/nexus-rpc-gen | ||
| NEXUS_SCHEMA_ROOT := $(PROTO_ROOT)/nexus | ||
| NEXUS_OUT := workflowservice/v1/workflowservicenexus |
There was a problem hiding this comment.
We are going to have more nexus services going into the same schema. You'll have to find a better solution to generate all of them.
There was a problem hiding this comment.
Also note that both the server and the Go SDK are going to use the generated files here. You'll want to come up with a directory structure that supports both outputs.
There was a problem hiding this comment.
The server files are generated to workflowservice/v1/workflowservicenexus/service_nexus.pb.go and the go sdk JSON is in workflowservice/v1/workflowservicenexus/json/service_nexus.go, following @tconley1428 's PR to add the sdk files
| .idea | ||
| .gobincache | ||
| .go-helpers-installed | ||
| .go-nexus-installed |
There was a problem hiding this comment.
Not anymore, removing
| @cd $(NEXUS_RPC_GEN_SRC)/src && pnpm run build | ||
| @cd $(NEXUS_RPC_GEN_SRC)/src/packages/nexus-rpc-gen && npm link |
There was a problem hiding this comment.
Eventually we'll want to install it via our CDN. Add a TODO please.
For now it's okay if we clone the repo since the generator is still WIP.
| @go install google.golang.org/protobuf/cmd/protoc-gen-go@latest | ||
| grpc-install: nexus-rpc-gen-install | ||
| @printf $(COLOR) "Install/update grpc plugins..." | ||
| @go install google.golang.org/protobuf/cmd/protoc-gen-go@latest |
There was a problem hiding this comment.
Thanks for removing the whitespace. The print was okay though. It's not all GRPC plugins. You can maybe change it to protoc plugins.
| @go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest | ||
| @go install github.com/grpc-ecosystem/grpc-gateway/v2/protoc-gen-grpc-gateway@latest | ||
|
|
||
| nexus-rpc-gen-install: |
There was a problem hiding this comment.
I would mark this as a PHONY target
Bumps proto/api submodule to f5b514e (master HEAD) to pick up the merged temporalio/api PR #736 (system nexus endpoint "exposed" op) and regenerates the Go bindings. Also fixes a non-constant format string in cmd/protogen/main.go that Go 1.25's stricter vet flagged, which broke the Update Proto CI job. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Resolves rawDesc conflict in workflowservice/v1/service.pb.go by regenerating from the merged submodule (proto/api at f5b514e). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
spkane31
changed the title
4 participants
What changed?
Adds Nexus service definition generation for
WorkflowService(workflowservice/v1/workflowservicenexus/service_nexus.pb.go) driven bynexus-rpc-genreading thegithub.com/temporalio/api:nexus/temporal-proto-models-nexusrpc.yamlfiles which definesSignalWithStartWorkflowExecution.Add a
nexus-genMakefile target for generation and wiring into thegrpctarget.Why?
Temporal needs to expose WorkflowService RPCs as Nexus operations (those marked as
exposed).How did you test it?
Server PR: temporalio/temporal#9833
Potential risks