Connect RPC import path being the Buf-tour path

[sudowoodo200]

Describe the bug

.connect.go file imports .pb.go file as v1 "github.com/bufbuild/buf-tour/gen/v1". Totally wrong. I believe it's because my proto directory has "v1"

To Reproduce

buf.gen.yaml

   - plugin: buf.build/protocolbuffers/go
    out: go/pkgs/bindings/proto
    opt:
      - paths=source_relative
  - plugin: buf.build/connectrpc/go
    out: go/pkgs/bindings/proto
    opt:
      - paths=source_relative

command

buf generate --path ./v1/*.proto

Results:

// Code generated by protoc-gen-connect-go. DO NOT EDIT.
//
// Source: v1/test.proto

package v1connect

import (
	connect "connectrpc.com/connect"
	context "context"
	errors "errors"
	v1 "github.com/bufbuild/buf-tour/gen/v1"
	http "net/http"
	strings "strings"
)

Additional context
Add any other context about the problem here.

[hirasawayuki]

@sudowoodo200
Could you please provide the contents of your *.proto file as well?
This will help in diagnosing the issue more accurately. The current description lacks the *.proto file details, making it unclear if the issue is due to a bug or the specific configuration of the proto file.

[sudowoodo200]

Yup here you go:

syntax = "proto3";
package foo;

enum Gender {
  MALE = 0;
  FEMALE = 1;
}

message Response {
  string id = 1;
  string text = 2;
}

message Person {
  string name = 1;
  Gender gender = 2;
}

service TestService {
  rpc Hello (Person) returns (Response);
  rpc StreamHello (Person) returns (stream Response);
  rpc Echo (stream Person) returns (stream Response);
}

[hirasawayuki]

The issue seems to be caused by not specifying the go_package option.
I suggest trying to add the following to your proto file:

option go_package = "{module_name}/go/pkgs/bindings/proto/v1;foov1";

[jhump]

@sudowoodo200, this configuration looks incomplete. As @hirasawayuki points out, the proto has no Go package option. And the buf.gen.yaml snippet doesn't indicate any managed mode configuration. Did you omit that when you pasted the config?

Also, you don't actually state what you were expecting as the import path. What is the intent? Why is the generated import "totally wrong"?

[sudowoodo200]

@jhump , @hirasawayuki -- extremely helpful, thank you! Both are bona fide omissions. Let me go try it. I simply ported over the code that I used for grpc-go and was taken aback by the buf-tour import.

Update: Solved! My bad, sorry guys!