Skip to content

Commit 276d57b

Browse files
committed
refactor: use class property for provider name in OpenAiHandler
- Replace hardcoded 'OpenAI' strings with this.providerName property - Follows same pattern as other provider classes for consistency
1 parent 4bbfb89 commit 276d57b

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

src/api/providers/openai.ts

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ import { handleOpenAIError } from "./utils/openai-error-handler"
3232
export class OpenAiHandler extends BaseProvider implements SingleCompletionHandler {
3333
protected options: ApiHandlerOptions
3434
private client: OpenAI
35+
private readonly providerName = "OpenAI"
3536

3637
constructor(options: ApiHandlerOptions) {
3738
super()
@@ -182,7 +183,7 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
182183
isAzureAiInference ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
183184
)
184185
} catch (error) {
185-
throw handleOpenAIError(error, "OpenAI")
186+
throw handleOpenAIError(error, this.providerName)
186187
}
187188

188189
const matcher = new XmlMatcher(
@@ -249,7 +250,7 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
249250
this._isAzureAiInference(modelUrl) ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
250251
)
251252
} catch (error) {
252-
throw handleOpenAIError(error, "OpenAI")
253+
throw handleOpenAIError(error, this.providerName)
253254
}
254255

255256
yield {
@@ -299,13 +300,13 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
299300
isAzureAiInference ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
300301
)
301302
} catch (error) {
302-
throw handleOpenAIError(error, "OpenAI")
303+
throw handleOpenAIError(error, this.providerName)
303304
}
304305

305306
return response.choices[0]?.message.content || ""
306307
} catch (error) {
307308
if (error instanceof Error) {
308-
throw new Error(`OpenAI completion error: ${error.message}`)
309+
throw new Error(`${this.providerName} completion error: ${error.message}`)
309310
}
310311

311312
throw error
@@ -350,7 +351,7 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
350351
methodIsAzureAiInference ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
351352
)
352353
} catch (error) {
353-
throw handleOpenAIError(error, "OpenAI")
354+
throw handleOpenAIError(error, this.providerName)
354355
}
355356

356357
yield* this.handleStreamResponse(stream)
@@ -380,7 +381,7 @@ export class OpenAiHandler extends BaseProvider implements SingleCompletionHandl
380381
methodIsAzureAiInference ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
381382
)
382383
} catch (error) {
383-
throw handleOpenAIError(error, "OpenAI")
384+
throw handleOpenAIError(error, this.providerName)
384385
}
385386

386387
yield {

0 commit comments

Comments
 (0)