chore(proto): regenerate worker v1 Go protobuf/grpc stubs#66
chore(proto): regenerate worker v1 Go protobuf/grpc stubs#66
Conversation
Updates generated *.pb.go / *_grpc.pb.go for admin/payload/worker: - align imports (protoreflect/protoimpl) and ordering - minor formatting/whitespace cleanups in grpc server stubs No intended functional changes.
There was a problem hiding this comment.
Pull request overview
Regenerates the Go protobuf and gRPC stubs for worker/v1 to reflect updated generator output and formatting, with no intended behavioral changes.
Changes:
- Regenerated
*.pb.gofiles with updated import ordering and minor code layout changes. - Regenerated
*_grpc.pb.goserver/client stubs with whitespace/formatting adjustments (notably import grouping and spacing between methods). - Minor reshaping of generated
vardeclarations (e.g., splittingvar (...)blocks into separatevarstatements).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/worker/v1/worker_grpc.pb.go | Regenerated WorkerService gRPC stubs; whitespace/import formatting updates. |
| pkg/worker/v1/worker.pb.go | Regenerated worker proto types; import ordering and generated var layout changes. |
| pkg/worker/v1/payload.pb.go | Regenerated payload proto types; import ordering and generated var layout changes. |
| pkg/worker/v1/admin_grpc.pb.go | Regenerated AdminService gRPC stubs; whitespace/import formatting updates. |
| pkg/worker/v1/admin.pb.go | Regenerated admin proto types; import ordering and generated var layout changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -8,7 +8,6 @@ package workerpb | |||
|
|
|||
| import ( | |||
| context "context" | |||
There was a problem hiding this comment.
This generated gRPC stub removed the blank line separating stdlib and third-party imports. gofmt/gofumpt will re-introduce the import grouping; since this repo runs gofumpt in make lint, please run gofumpt/gofmt after regeneration to keep the generated output stable.
| context "context" | |
| context "context" |
| @@ -8,7 +8,6 @@ package workerpb | |||
|
|
|||
| import ( | |||
| context "context" | |||
There was a problem hiding this comment.
This generated gRPC stub removed the blank line separating stdlib and third-party imports. gofmt/gofumpt will reformat this; since make lint runs gofumpt over GOFILES, please run gofumpt/gofmt after regeneration so the committed generated output matches formatting expectations.
| context "context" | |
| context "context" |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | ||
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | ||
| anypb "google.golang.org/protobuf/types/known/anypb" | ||
| durationpb "google.golang.org/protobuf/types/known/durationpb" | ||
| reflect "reflect" | ||
| sync "sync" | ||
| unsafe "unsafe" |
There was a problem hiding this comment.
The import block is no longer gofmt/gofumpt-compliant (stdlib imports like reflect are placed after third-party protobuf imports and there’s no stdlib/third-party grouping). Since this repo runs gofumpt over GOFILES in make lint, this will be rewritten and can cause recurring diffs. Please regenerate and/or run gofumpt (or gofmt) on the generated stubs so imports are grouped and ordered consistently.
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | |
| anypb "google.golang.org/protobuf/types/known/anypb" | |
| durationpb "google.golang.org/protobuf/types/known/durationpb" | |
| reflect "reflect" | |
| sync "sync" | |
| unsafe "unsafe" | |
| reflect "reflect" | |
| sync "sync" | |
| unsafe "unsafe" | |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | |
| anypb "google.golang.org/protobuf/types/known/anypb" | |
| durationpb "google.golang.org/protobuf/types/known/durationpb" |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | ||
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | ||
| reflect "reflect" | ||
| sync "sync" | ||
| unsafe "unsafe" |
There was a problem hiding this comment.
The import block is not gofmt/gofumpt-compliant (protobuf imports are mixed with stdlib imports without grouping). Given make lint runs gofumpt across GOFILES, this will be rewritten. Please run gofumpt/gofmt after regenerating so the import grouping is stable.
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | |
| reflect "reflect" | |
| sync "sync" | |
| unsafe "unsafe" | |
| "reflect" | |
| "sync" | |
| "unsafe" | |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | ||
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | ||
| reflect "reflect" | ||
| sync "sync" | ||
| unsafe "unsafe" |
There was a problem hiding this comment.
The import block is not gofmt/gofumpt-compliant (protobuf imports are placed before stdlib imports with no blank line grouping). Since make lint runs gofumpt, this will be rewritten and may cause repeated churn. Please run gofumpt/gofmt on the regenerated file so imports are grouped consistently.
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" | |
| reflect "reflect" | |
| sync "sync" | |
| unsafe "unsafe" | |
| reflect "reflect" | |
| sync "sync" | |
| unsafe "unsafe" | |
| protoreflect "google.golang.org/protobuf/reflect/protoreflect" | |
| protoimpl "google.golang.org/protobuf/runtime/protoimpl" |
1 participant
Updates generated *.pb.go / *_grpc.pb.go for admin/payload/worker: