Skip to content

Route handleErrorResponse logging through logDebug - #13

Open
egigoka wants to merge 1 commit into
dkraemerwork:mainfrom
egigoka:fix-error-response-logDebug
Open

egigoka wants to merge 1 commit into
dkraemerwork:mainfrom
egigoka:fix-error-response-logDebug

Conversation

@egigoka

@egigoka egigoka commented Sep 24, 2026

Copy link
Copy Markdown

handleErrorResponse() logs every non-ok response with unconditional console.error, so expected rate-limit 429s (including usage_limit_reached 404s mapped to 429) always surface as [openai-codex-plugin] Error 429 noise in opencode UI, even without debug logging enabled.

Route the error-body log through logDebug() instead, so it only prints when DEBUG_CODEX_PLUGIN=1 / ENABLE_PLUGIN_REQUEST_LOGGING=1. Error mapping and return behavior unchanged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant