Anthropic request headers helper

[enzowritescode]

Description

• Add helper function to set common headers for Anthropic requests
• Fix variables colliding with imported package

[crspeller]

Might be nicer to use the variable below instead of putting the p.GetMetrics() into the if

[enzowritescode]

The metrics variable was colliding with the metrics package and the variable was only being used for the if statement.

[enzowritescode]

Are you saying like this? This is something I had actually wondered about since I saw p.GetMetrics() being called several times

func (p *Plugin) metricsMiddleware(c *gin.Context) {
	llmMetrics := p.GetMetrics()
	if llmMetrics == nil {
		c.Next()
		return
	}
	llmMetrics.IncrementHTTPRequests()
	now := time.Now()

	c.Next()

	elapsed := float64(time.Since(now)) / float64(time.Second)

	status := c.Writer.Status()

	if status < 200 || status > 299 {
		llmMetrics.IncrementHTTPErrors()
	}

	endpoint := c.HandlerName()
	llmMetrics.ObserveAPIEndpointDuration(endpoint, c.Request.Method, strconv.Itoa(status), elapsed)
}

[crspeller]

Yup like that.

[crspeller]

Not sure this helper is necessary. But if we do want to keep it maybe call it setAnthropicHeaders and pull in the streaming ones with a boolean parameter.

[enzowritescode]

That sounds good. Yeah it was repeated logic so I figured not repeating it would be better, especially because then there were less hard-coded strings and consolidating into one place would eliminate the argument for making constants to reduce the number of hard-coded strings.

[enzowritescode]

I just remembered my reasoning for originally calling it prepareRequestHeaders().
using "set" typically denotes that you are setting an instance variable on the calling object, whereas in this case I am setting values of something being passed in.