Skip to content

fix(prompt): handle exceptions from decodeResponse() in AbstractOpenAILLMClient#2012

Merged
Amaneusz merged 1 commit into
developfrom
amanowicz/develop/KG-834_openai_decode_response_bug
May 15, 2026
Merged

fix(prompt): handle exceptions from decodeResponse() in AbstractOpenAILLMClient#2012
Amaneusz merged 1 commit into
developfrom
amanowicz/develop/KG-834_openai_decode_response_bug

Conversation

@Amaneusz
Copy link
Copy Markdown
Collaborator

@Amaneusz Amaneusz commented May 15, 2026 •
edited
Loading

AbstractOpenAILLMClient: wraps exceptions thrown in getResponse() in LLMClientException

Fixes: #1978

@Amaneusz Amaneusz changed the title fix(prompt): handle exceptions from AbstractOpenAILLMClient.decodeResponse() fix(prompt): handle exceptions from decodeResponse() in AbstractOpenAILLMClient May 15, 2026
@Amaneusz Amaneusz marked this pull request as ready for review May 15, 2026 08:22
@Amaneusz Amaneusz requested review from Ololoshechkin and sdubov May 15, 2026 08:23
@Amaneusz Amaneusz merged commit f70ec8f into develop May 15, 2026
24 checks passed
@Amaneusz Amaneusz deleted the amanowicz/develop/KG-834_openai_decode_response_bug branch May 15, 2026 09:32
@Amaneusz Amaneusz mentioned this pull request May 15, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EugeneTheDev EugeneTheDev EugeneTheDev approved these changes

@Ololoshechkin Ololoshechkin Awaiting requested review from Ololoshechkin

@sdubov sdubov Awaiting requested review from sdubov

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Non-streaming getResponse: decodeResponse() outside try/catch bypasses LLMClientException wrapping

2 participants

@Amaneusz @EugeneTheDev