Skip to content

Support connect-simple - #51

Open
majst01 wants to merge 2 commits into
redpanda-data:mainfrom
majst01:support-connect-simple
Open

Support connect-simple#51
majst01 wants to merge 2 commits into
redpanda-data:mainfrom
majst01:support-connect-simple

Conversation

@majst01

@majst01 majst01 commented Aug 3, 2026

Copy link
Copy Markdown

Hi Redpanda Team,

first of all, thanks very much for this compiler, love it.
We are building our api with connect-go, but with the "simple" flag to their compiler [1] which reduces the generic boiler plate code dramatically. With this enabled, the Interfaces your library generates are not compatible anymore.

The Interfaces connect-go simple generates look like:

// AuditServiceClient is a client for the metalstack.api.v2.AuditService service.
type AuditServiceClient interface {
	// Returns the audit trace with the specified UUID.
	Get(context.Context, *v2.AuditServiceGetRequest) (*v2.AuditServiceGetResponse, error)
	// Returns the list of audit traces.
	List(context.Context, *v2.AuditServiceListRequest) (*v2.AuditServiceListResponse, error)
}

In contrast, your compiler generates:

// AuditServiceClient is compatible with the grpc-go client interface.
type AuditServiceClient interface {
	Get(ctx context.Context, req *v2.AuditServiceGetRequest, opts ...grpc.CallOption) (*v2.AuditServiceGetResponse, error)
	List(ctx context.Context, req *v2.AuditServiceListRequest, opts ...grpc.CallOption) (*v2.AuditServiceListResponse, error)
}

I tried the following PR and thought i did it correct, but no luck. Maybe i am missing something how your compiler works.

Any help is appreciated.

[1] connectrpc/connect-go#851

@majst01 majst01 mentioned this pull request Aug 3, 2026
5 tasks
@majst01
majst01 force-pushed the support-connect-simple branch from 3f10928 to 61abf07 Compare August 4, 2026 05:45
@majst01

majst01 commented Aug 21, 2026

Copy link
Copy Markdown
Author

Hi,

Is there any interest in getting this PR upstream?
Or can i add additional information to clarify the need for this?

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