Jsburckhardt/287 azure openai list models

[jsburckhardt]

what

• feat(models): include flow for listing models with azure openai endpoint

why

• working on issue Azure OpenAI list models #287

how to test

• go test
  

• or run test TestAzureListModels
  

[codecov]

Codecov Report

Merging #290 (c446225) into master (af9ff51) will increase coverage by 0.49%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
+ Coverage   92.43%   92.93%   +0.49%     
==========================================
  Files          22       22              
  Lines         661      665       +4     
==========================================
+ Hits          611      618       +7     
+ Misses         36       35       -1     
+ Partials       14       12       -2     

Impacted Files	Coverage Δ
client.go	91.34% <100.00%> (+3.34%)	⬆️

... and 1 file with indirect coverage changes

[sashabaranov]

@jsburckhardt thank you for the PR! I wonder if this is covered by the logic in the fullURL? If not, maybe we should improve the fullURL

go-openai/client.go

Lines 102 to 108 in af9ff51

	// /openai/deployments/{engine}/chat/completions?api-version={api_version}
	if c.config.APIType == APITypeAzure || c.config.APIType == APITypeAzureAD {
	baseURL := c.config.BaseURL
	baseURL = strings.TrimRight(baseURL, "/")
	return fmt.Sprintf("%s/%s/%s/%s%s?api-version=%s",
	baseURL, azureAPIPrefix, azureDeploymentsPrefix, c.config.Engine, suffix, c.config.APIVersion)
	}

[jsburckhardt]

I'll give it a shot updating fullURL, the thing is the URL/API changes between /models and /completions and fullURL isn't aware of the service that is getting called. Leave it with me I'll check what I can do.
completions-> {endpoint}/openai/deployments/{engine}/chat/completions?api-version={api_version}
models -> {endpoint}/openai/models?api-version=2022-12-01

[sashabaranov]

@jsburckhardt got it! ThefullURL knows about what endpoint is passed to it, so we can make a special case there

[jsburckhardt]

@sashabaranov updated by adding some logic into the fullURL, should be ok now. Let me know ;)

[sashabaranov]

Thank you!