Skip to content

[ENHANCEMENT] Remove deprecated openai-error-handler shim #766

Description

@daewoongoh

Problem (one or two sentences)

openai-error-handler remains as a deprecated compatibility shim that wraps error-handler, which has been the canonical source since RooCodeInc/Roo-Code#10204. Keeping this shim creates duplicate import paths and duplicate tests for the same handleOpenAIError behavior, increasing maintenance overhead.

Context (who is affected and when)

Provider and embedder code that uses OpenAI-compatible error handling currently has references to the deprecated openai-error-handler shim, while the actual canonical implementation lives in error-handler.

This can make it unclear which module should be used when updating or reviewing error handling logic. It also leaves deprecated code and duplicate tests in the codebase even though the canonical implementation already exists.

Desired behavior (conceptual, not technical)

OpenAI-compatible error handling should be managed through a single canonical utility: error-handler.

The deprecated openai-error-handler shim and its duplicate test file should be removed, and all related imports should reference error-handler directly. The existing handleOpenAIError behavior should remain unchanged.

Constraints / preferences (optional)

No response

Request checklist

  • I've searched existing Issues and Discussions for duplicates
  • This describes a specific problem with clear context and impact

Zoo Code Task Links (optional)

No response

Acceptance criteria (optional)

No response

Proposed approach (optional)

No response

Trade-offs / risks (optional)

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions