Handle response errors

[Iyadhfaleh]

I think json_decode is better to handle responses APIs .
"toArray" method will throw exception when status code >= 300 .

[chr-hertel]

So, first of all thanks and I do agree that error handling is currently not in great shape.

the code that you added is mostly sitting on top of the expected json/array structure - which is a valid addition, but i doesn't help with 4xx/5xx situations, since everything is converted to a runtime exception and context of the original error or unexpected behavior would get lost.

i consider the structural changes that you check and error handling with RuntimeException a valuable addition, but would still keep the toArray() with throwing exceptions. nevertheless, this should be improved but rather in a different layer. and without loosing the original exception information.

this was the original handling, which is broken since i introduced the AsyncResponse i guess...
https://github.com/php-llm/llm-chain/blob/main/src/Platform.php#L44-L54

[chr-hertel]

Let's focus on the additional RuntimeExceptions - they are a good addition and provide value. Thanks for that!

[Iyadhfaleh]

Thank you for your review and your feedbacks @chr-hertel. I applied your suggestions

[chr-hertel]

Thank you @Iyadhfaleh