fix: address panic when listing models with default model provider

thedadams · thedadams · commit 4197b39ed68f · 2024-08-12T15:56:11.000-04:00
Apparently, this code path is exercised by listing models and not by
using the provider with LLM calls.

Signed-off-by: Donnie Adams &lt;donnie@acorn.io&gt;
diff --git a/pkg/remote/remote.go b/pkg/remote/remote.go
@@ -189,7 +189,7 @@ func (c *Client) load(ctx context.Context, toolName string) (*openai.Client, err
 		client: oClient,
 		url:    url,
 	}
-	return client.client, nil
+	return oClient, nil
 }
 
 func (c *Client) retrieveAPIKey(ctx context.Context, env, url string) (string, error) {

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,7 @@ func (c Client) load(ctx context.Context, toolName string) (openai.Client, err`
`189`	`189`	`client: oClient,`
`190`	`190`	`url: url,`
`191`	`191`	`}`
`192`		`- return client.client, nil`
	`192`	`+ return oClient, nil`
`193`	`193`	`}`
`194`	`194`
`195`	`195`	`func (c *Client) retrieveAPIKey(ctx context.Context, env, url string) (string, error) {`