refactor: move error handling to chain + logging (#182)

Author: chr-hertel

src/Chain.php

@@ -11,10 +11,15 @@
 use PhpLlm\LlmChain\Chain\OutputProcessor;
 use PhpLlm\LlmChain\Exception\InvalidArgumentException;
 use PhpLlm\LlmChain\Exception\MissingModelSupport;
+use PhpLlm\LlmChain\Exception\RuntimeException;
 use PhpLlm\LlmChain\Model\LanguageModel;
 use PhpLlm\LlmChain\Model\Message\MessageBagInterface;
 use PhpLlm\LlmChain\Model\Response\AsyncResponse;
 use PhpLlm\LlmChain\Model\Response\ResponseInterface;
+use Psr\Log\LoggerInterface;
+use Psr\Log\NullLogger;
+use Symfony\Contracts\HttpClient\Exception\ClientExceptionInterface;
+use Symfony\Contracts\HttpClient\Exception\HttpExceptionInterface;
 
 final readonly class Chain implements ChainInterface
 {
@@ -37,6 +42,7 @@ public function __construct(
         private LanguageModel $llm,
         iterable $inputProcessor = [],
         iterable $outputProcessor = [],
+        private LoggerInterface $logger = new NullLogger(),
     ) {
         $this->inputProcessor = $this->initializeProcessors($inputProcessor, InputProcessor::class);
         $this->outputProcessor = $this->initializeProcessors($outputProcessor, OutputProcessor::class);
@@ -58,10 +64,21 @@ public function call(MessageBagInterface $messages, array $options = []): Respon
             throw MissingModelSupport::forImageInput($llm::class);
         }
 
-        $response = $this->platform->request($llm, $messages, $options);
+        try {
+            $response = $this->platform->request($llm, $messages, $options);
 
-        if ($response instanceof AsyncResponse) {
-            $response = $response->unwrap();
+            if ($response instanceof AsyncResponse) {
+                $response = $response->unwrap();
+            }
+        } catch (ClientExceptionInterface $e) {
+            $message = $e->getMessage();
+            $content = $e->getResponse()->toArray(false);
+
+            $this->logger->debug($message, $content);
+
+            throw new InvalidArgumentException('' === $message ? 'Invalid request to model or platform' : $message, 0, $e);
+        } catch (HttpExceptionInterface $e) {
+            throw new RuntimeException('Failed to request model', 0, $e);
         }
 
         $output = new Output($llm, $response, $messages, $options);

src/Platform.php

@@ -4,15 +4,12 @@
 
 namespace PhpLlm\LlmChain;
 
-use PhpLlm\LlmChain\Exception\InvalidArgumentException;
 use PhpLlm\LlmChain\Exception\RuntimeException;
 use PhpLlm\LlmChain\Model\Model;
 use PhpLlm\LlmChain\Model\Response\AsyncResponse;
 use PhpLlm\LlmChain\Model\Response\ResponseInterface;
 use PhpLlm\LlmChain\Platform\ModelClient;
 use PhpLlm\LlmChain\Platform\ResponseConverter;
-use Symfony\Contracts\HttpClient\Exception\ClientExceptionInterface;
-use Symfony\Contracts\HttpClient\Exception\HttpExceptionInterface;
 use Symfony\Contracts\HttpClient\ResponseInterface as HttpResponse;
 
 final readonly class Platform implements PlatformInterface
@@ -41,17 +38,9 @@ public function request(Model $model, array|string|object $input, array $options
     {
         $options = array_merge($model->getOptions(), $options);
 
-        try {
-            $response = $this->doRequest($model, $input, $options);
+        $response = $this->doRequest($model, $input, $options);
 
-            return $this->convertResponse($model, $input, $response, $options);
-        } catch (ClientExceptionInterface $e) {
-            $message = $e->getMessage();
-
-            throw new InvalidArgumentException('' === $message ? 'Invalid request to model or platform' : $message, 0, $e);
-        } catch (HttpExceptionInterface $e) {
-            throw new RuntimeException('Failed to request model', 0, $e);
-        }
+        return $this->convertResponse($model, $input, $response, $options);
     }
 
     /**