feat: add Type() to Provider interface and configurable instance Name

Author: ssncferreira

bridge.go

@@ -198,7 +198,8 @@ func newInterceptionProcessor(p provider.Provider, cbs *circuitbreaker.ProviderC
 			InitiatorID:           actor.ID,
 			Metadata:              actor.Metadata,
 			Model:                 interceptor.Model(),
-			Provider:              p.Name(),
+			Provider:              p.Type(),
+			ProviderName:          p.Name(),
 			UserAgent:             r.UserAgent(),
 			Client:                string(client),
 			ClientSessionID:       sessionID,

config/config.go

@@ -9,6 +9,8 @@ const (
 )
 
 type Anthropic struct {
+	// Name is the provider instance name. If empty, defaults to ProviderAnthropic.
+	Name             string
 	BaseURL          string
 	Key              string
 	APIDumpDir       string
@@ -32,6 +34,8 @@ type AWSBedrock struct {
 }
 
 type OpenAI struct {
+	// Name is the provider instance name. If empty, defaults to ProviderOpenAI.
+	Name             string
 	BaseURL          string
 	Key              string
 	APIDumpDir       string
@@ -69,6 +73,8 @@ func DefaultCircuitBreaker() CircuitBreaker {
 }
 
 type Copilot struct {
+	// Name is the provider instance name. If empty, defaults to ProviderCopilot.
+	Name           string
 	BaseURL        string
 	APIDumpDir     string
 	CircuitBreaker *CircuitBreaker

internal/testutil/mockprovider.go

@@ -17,6 +17,7 @@ type MockProvider struct {
 	InterceptorFunc func(w http.ResponseWriter, r *http.Request, tracer trace.Tracer) (intercept.Interceptor, error)
 }
 
+func (m *MockProvider) Type() string                                 { return m.Name_ }
 func (m *MockProvider) Name() string                                 { return m.Name_ }
 func (m *MockProvider) BaseURL() string                              { return m.URL }
 func (m *MockProvider) RoutePrefix() string                          { return fmt.Sprintf("/%s", m.Name_) }

provider/anthropic.go

@@ -48,6 +48,9 @@ var anthropicIsFailure = func(statusCode int) bool {
 }
 
 func NewAnthropic(cfg config.Anthropic, bedrockCfg *config.AWSBedrock) *Anthropic {
+	if cfg.Name == "" {
+		cfg.Name = config.ProviderAnthropic
+	}
 	if cfg.BaseURL == "" {
 		cfg.BaseURL = "https://api.anthropic.com/"
 	}
@@ -68,10 +71,14 @@ func NewAnthropic(cfg config.Anthropic, bedrockCfg *config.AWSBedrock) *Anthropi
 	}
 }
 
-func (p *Anthropic) Name() string {
+func (p *Anthropic) Type() string {
 	return config.ProviderAnthropic
 }
 
+func (p *Anthropic) Name() string {
+	return p.cfg.Name
+}
+
 func (p *Anthropic) RoutePrefix() string {
 	return fmt.Sprintf("/%s", p.Name())
 }

provider/copilot.go

@@ -52,6 +52,9 @@ type Copilot struct {
 var _ Provider = &Copilot{}
 
 func NewCopilot(cfg config.Copilot) *Copilot {
+	if cfg.Name == "" {
+		cfg.Name = config.ProviderCopilot
+	}
 	if cfg.BaseURL == "" {
 		cfg.BaseURL = copilotBaseURL
 	}
@@ -67,10 +70,14 @@ func NewCopilot(cfg config.Copilot) *Copilot {
 	}
 }
 
-func (p *Copilot) Name() string {
+func (p *Copilot) Type() string {
 	return config.ProviderCopilot
 }
 
+func (p *Copilot) Name() string {
+	return p.cfg.Name
+}
+
 func (p *Copilot) BaseURL() string {
 	return p.cfg.BaseURL
 }

provider/openai.go

@@ -37,6 +37,9 @@ type OpenAI struct {
 var _ Provider = &OpenAI{}
 
 func NewOpenAI(cfg config.OpenAI) *OpenAI {
+	if cfg.Name == "" {
+		cfg.Name = config.ProviderOpenAI
+	}
 	if cfg.BaseURL == "" {
 		cfg.BaseURL = "https://api.openai.com/v1/"
 	}
@@ -56,10 +59,14 @@ func NewOpenAI(cfg config.OpenAI) *OpenAI {
 	}
 }
 
-func (p *OpenAI) Name() string {
+func (p *OpenAI) Type() string {
 	return config.ProviderOpenAI
 }
 
+func (p *OpenAI) Name() string {
+	return p.cfg.Name
+}
+
 func (p *OpenAI) RoutePrefix() string {
 	// Route prefix includes version to match default OpenAI base URL.
 	// More detailed explanation: https://github.com/coder/aibridge/pull/174#discussion_r2782320152

provider/provider.go

@@ -44,7 +44,11 @@ var UnknownRoute = errors.New("unknown route")
 // OpenAI includes the version '/v1' in the base url while Anthropic does not.
 // More details/examples: https://github.com/coder/aibridge/pull/174#discussion_r2782320152
 type Provider interface {
-	// Name returns the provider's name.
+	// Type returns the provider type: "copilot", "openai", or "anthropic".
+	// Multiple provider instances can share the same type.
+	Type() string
+	// Name returns the provider instance name.
+	// Defaults to Type() when not explicitly configured.
 	Name() string
 	// BaseURL defines the base URL endpoint for this provider's API.
 	BaseURL() string

recorder/types.go

@@ -33,6 +33,7 @@ type InterceptionRecord struct {
 	Metadata              Metadata
 	Model                 string
 	Provider              string
+	ProviderName          string
 	StartedAt             time.Time
 	ClientSessionID       *string
 	Client                string