refactor: remove deprecated openai-error-handler shim and use error-handler (#767)

* refactor: remove deprecated openai-error-handler and consolidate into error-handler

Replace all imports of the deprecated `openai-error-handler` shim with
`error-handler`, which has been the canonical source since #10204.
Delete `openai-error-handler.ts` and its duplicate test file, as
`error-handler.spec.ts` already covers the same behavior.

Signed-off-by: daewoongoh <dw.oh@samsung.com>

* test(deepseek): add test for API error handling in createMessage

Cover the catch block that wraps API errors via handleOpenAIError,
fixing 0% patch coverage on the error path.

Signed-off-by: daewoongoh <dw.oh@samsung.com>

* refactor(deepseek): convert dynamic import to static for handleOpenAIError

Replace dynamic `await import()` with a static import for `handleOpenAIError`
in the catch block, and update the error handler test to also assert the HTTP
status code (401) on the thrown error.

Signed-off-by: daewoongoh <dw.oh@samsung.com>

---------

Signed-off-by: daewoongoh <dw.oh@samsung.com>

Author: daewoongoh

src/api/providers/__tests__/deepseek.spec.ts

@@ -426,6 +426,17 @@ describe("DeepSeekHandler", () => {
 			expect(chunks).toContainEqual({ type: "reasoning", text: "router-style thought" })
 		})
 
+		it("should throw a wrapped error when the API call fails", async () => {
+			const apiError = Object.assign(new Error("Invalid API key"), { status: 401 })
+			mockCreate.mockRejectedValueOnce(apiError)
+
+			const stream = handler.createMessage(systemPrompt, messages)
+			const err = await stream.next().catch((e) => e)
+
+			expect(err.message).toBe("DeepSeek completion error: Invalid API key")
+			expect((err as any).status).toBe(401)
+		})
+
 		it("prefers delta.reasoning_content over delta.reasoning when both are present", async () => {
 			mockCreate.mockImplementationOnce(async () => ({
 				[Symbol.asyncIterator]: async function* () {

src/api/providers/base-openai-compatible-provider.ts

@@ -11,7 +11,7 @@ import { convertToOpenAiMessages } from "../transform/openai-format"
 import type { SingleCompletionHandler, ApiHandlerCreateMessageMetadata } from "../index"
 import { DEFAULT_HEADERS } from "./constants"
 import { BaseProvider } from "./base-provider"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 import { calculateApiCostOpenAI } from "../../shared/cost"
 import { extractReasoningFromDelta } from "./utils/extract-reasoning"
 

src/api/providers/deepseek.ts

@@ -18,6 +18,7 @@ import { convertToR1Format } from "../transform/r1-format"
 import { OpenAiHandler } from "./openai"
 import { extractReasoningFromDelta } from "./utils/extract-reasoning"
 import type { ApiHandlerCreateMessageMetadata } from "../index"
+import { handleOpenAIError } from "./utils/error-handler"
 
 // Custom interface for DeepSeek params to support thinking mode
 type DeepSeekChatCompletionParams = Omit<OpenAI.Chat.ChatCompletionCreateParamsStreaming, "reasoning_effort"> & {
@@ -137,7 +138,6 @@ export class DeepSeekHandler extends OpenAiHandler {
 				isAzureAiInference ? { path: OPENAI_AZURE_AI_INFERENCE_PATH } : {},
 			)
 		} catch (error) {
-			const { handleOpenAIError } = await import("./utils/openai-error-handler")
 			throw handleOpenAIError(error, "DeepSeek")
 		}
 

src/api/providers/lm-studio.ts

@@ -15,7 +15,7 @@ import { ApiStream } from "../transform/stream"
 import { BaseProvider } from "./base-provider"
 import type { SingleCompletionHandler, ApiHandlerCreateMessageMetadata } from "../index"
 import { getModelsFromCache } from "./fetchers/modelCache"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 
 export class LmStudioHandler extends BaseProvider implements SingleCompletionHandler {
 	protected options: ApiHandlerOptions

src/api/providers/openai.ts

@@ -22,7 +22,7 @@ import { getModelParams } from "../transform/model-params"
 import { DEFAULT_HEADERS } from "./constants"
 import { BaseProvider } from "./base-provider"
 import type { SingleCompletionHandler, ApiHandlerCreateMessageMetadata } from "../index"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 import { extractReasoningFromDelta } from "./utils/extract-reasoning"
 
 // TODO: Rename this to OpenAICompatibleHandler. Also, I think the

src/api/providers/openrouter.ts

@@ -36,7 +36,7 @@ import { getModelEndpoints } from "./fetchers/modelEndpointCache"
 import { DEFAULT_HEADERS } from "./constants"
 import { BaseProvider } from "./base-provider"
 import type { ApiHandlerCreateMessageMetadata, SingleCompletionHandler } from "../index"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 import { generateImageWithProvider, ImageGenerationResult } from "./utils/image-generation"
 import { applyRouterToolPreferences } from "./utils/router-tool-preferences"
 

src/api/providers/requesty.ts

@@ -16,7 +16,7 @@ import { getModels } from "./fetchers/modelCache"
 import { BaseProvider } from "./base-provider"
 import type { SingleCompletionHandler, ApiHandlerCreateMessageMetadata } from "../index"
 import { toRequestyServiceUrl } from "../../shared/utils/requesty"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 import { applyRouterToolPreferences } from "./utils/router-tool-preferences"
 import { extractReasoningFromDelta } from "./utils/extract-reasoning"
 

src/api/providers/unbound.ts

@@ -15,7 +15,7 @@ import { DEFAULT_HEADERS } from "./constants"
 import { getModels } from "./fetchers/modelCache"
 import { BaseProvider } from "./base-provider"
 import type { SingleCompletionHandler, ApiHandlerCreateMessageMetadata } from "../index"
-import { handleOpenAIError } from "./utils/openai-error-handler"
+import { handleOpenAIError } from "./utils/error-handler"
 import { applyRouterToolPreferences } from "./utils/router-tool-preferences"
 import { extractReasoningFromDelta } from "./utils/extract-reasoning"