atiaomar1978-hub commented on PR #24961: URL: https://github.com/apache/camel/pull/24961#issuecomment-5027450552
## Update: addressed review feedback Thanks for the reviews from @gnodet and @davsclaus. ### gnodet (commit ec4d131) - Added `OpenAIAudioTranslationVerboseMockTest` for verbose translation headers - Added try/catch JSON error handling to `SpeechRequestHandler` - Documented heap awareness for large TTS `byte[]` output in `openai-operations.adoc` - Introduced `OpenAIMockExpectations` record to replace the long positional-list constructor on `OpenAIMockServerHandler` ### davsclaus (commit 7f76b0f) - **Content-Type from HTTP response**: `OpenAIAudioSpeechProducer` now prefers the `Content-Type` header from the OpenAI speech API response, falling back to the `speechResponseFormat` mapping only when the header is absent - Updated the speech mock handler to derive `Content-Type` from the request `response_format` so tests reflect real API behaviour - Updated docs to describe the response-header-first behaviour ### Tests All 22 audio tests pass locally: - `OpenAIAudioSpeechMockTest` (6) - `OpenAIAudioTranslationMockTest` (7) - `OpenAIAudioTranslationVerboseMockTest` (1) - `OpenAIAudioTranscriptionMockTest` (7) - `OpenAIAudioTranscriptionVerboseMockTest` (1) --- _Comment posted by Cursor Agent (Opus 4.8) on behalf of @atiaomar1978-hub._ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
