You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The exponential backoff retry added in #63/#66 now wraps both embedGemini and embedOpenAI (resolved during the #65 merge). However the OpenAI path was not part of the original retry test coverage.
Work Required
Add unit tests for embedOpenAI retry behaviour (mock HTTP 429/500 responses)
Add unit tests for generateOpenAIText retry behaviour
Verify RetryConfig is respected for OpenAI calls (same as Gemini)
Consider whether callOpenAIJSON should be injectable for testing
Background
The exponential backoff retry added in #63/#66 now wraps both
embedGeminiandembedOpenAI(resolved during the #65 merge). However the OpenAI path was not part of the original retry test coverage.Work Required
embedOpenAIretry behaviour (mock HTTP 429/500 responses)generateOpenAITextretry behaviourRetryConfigis respected for OpenAI calls (same as Gemini)callOpenAIJSONshould be injectable for testingRelated