fix: make it possible to inject http client into factories (#155)

Author: chr-hertel

src/Bridge/Anthropic/PlatformFactory.php

@@ -6,12 +6,18 @@
 
 use PhpLlm\LlmChain\Platform;
 use Symfony\Component\HttpClient\EventSourceHttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final readonly class PlatformFactory
 {
-    public static function create(#[\SensitiveParameter] string $apiKey, string $version = '2023-06-01'): Platform
-    {
-        $responseHandler = new ModelHandler(new EventSourceHttpClient(), $apiKey, $version);
+    public static function create(
+        #[\SensitiveParameter]
+        string $apiKey,
+        string $version = '2023-06-01',
+        ?HttpClientInterface $httpClient = null,
+    ): Platform {
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
+        $responseHandler = new ModelHandler($httpClient, $apiKey, $version);
 
         return new Platform([$responseHandler], [$responseHandler]);
     }

src/Bridge/Azure/OpenAI/PlatformFactory.php

@@ -8,16 +8,19 @@
 use PhpLlm\LlmChain\Bridge\OpenAI\GPT\ResponseConverter;
 use PhpLlm\LlmChain\Platform;
 use Symfony\Component\HttpClient\EventSourceHttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final readonly class PlatformFactory
 {
     public static function create(
         string $baseUrl,
         string $deployment,
         string $apiVersion,
-        #[\SensitiveParameter] string $apiKey,
+        #[\SensitiveParameter]
+        string $apiKey,
+        ?HttpClientInterface $httpClient = null,
     ): Platform {
-        $httpClient = new EventSourceHttpClient();
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
         $embeddingsResponseFactory = new EmbeddingsModelClient($httpClient, $baseUrl, $deployment, $apiVersion, $apiKey);
         $GPTResponseFactory = new GPTModelClient($httpClient, $baseUrl, $deployment, $apiVersion, $apiKey);
 

src/Bridge/Ollama/PlatformFactory.php

@@ -5,13 +5,17 @@
 namespace PhpLlm\LlmChain\Bridge\Ollama;
 
 use PhpLlm\LlmChain\Platform;
-use Symfony\Component\HttpClient\HttpClient;
+use Symfony\Component\HttpClient\EventSourceHttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final class PlatformFactory
 {
-    public static function create(string $hostUrl = 'http://localhost:11434'): Platform
-    {
-        $handler = new LlamaModelHandler(HttpClient::create(), $hostUrl);
+    public static function create(
+        string $hostUrl = 'http://localhost:11434',
+        ?HttpClientInterface $httpClient = null,
+    ): Platform {
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
+        $handler = new LlamaModelHandler($httpClient, $hostUrl);
 
         return new Platform([$handler], [$handler]);
     }

src/Bridge/OpenAI/PlatformFactory.php

@@ -10,12 +10,16 @@
 use PhpLlm\LlmChain\Bridge\OpenAI\GPT\ResponseConverter as GPTResponseConverter;
 use PhpLlm\LlmChain\Platform;
 use Symfony\Component\HttpClient\EventSourceHttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final readonly class PlatformFactory
 {
-    public static function create(#[\SensitiveParameter] string $apiKey): Platform
-    {
-        $httpClient = new EventSourceHttpClient();
+    public static function create(
+        #[\SensitiveParameter]
+        string $apiKey,
+        ?HttpClientInterface $httpClient = null,
+    ): Platform {
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
 
         return new Platform(
             [new GPTModelClient($httpClient, $apiKey), new EmbeddingsModelClient($httpClient, $apiKey)],

src/Bridge/Replicate/PlatformFactory.php

@@ -7,12 +7,19 @@
 use PhpLlm\LlmChain\Bridge\Meta\LlamaPromptConverter;
 use PhpLlm\LlmChain\Platform;
 use Symfony\Component\Clock\Clock;
+use Symfony\Component\HttpClient\EventSourceHttpClient;
 use Symfony\Component\HttpClient\HttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final class PlatformFactory
 {
-    public static function create(string $apiKey): Platform
-    {
+    public static function create(
+        #[\SensitiveParameter]
+        string $apiKey,
+        ?HttpClientInterface $httpClient = null,
+    ): Platform {
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
+
         return new Platform(
             [new LlamaModelClient(new Client(HttpClient::create(), new Clock(), $apiKey), new LlamaPromptConverter())],
             [new LlamaResponseConverter()],

src/Bridge/Voyage/PlatformFactory.php

@@ -5,13 +5,18 @@
 namespace PhpLlm\LlmChain\Bridge\Voyage;
 
 use PhpLlm\LlmChain\Platform;
-use Symfony\Component\HttpClient\HttpClient;
+use Symfony\Component\HttpClient\EventSourceHttpClient;
+use Symfony\Contracts\HttpClient\HttpClientInterface;
 
 final class PlatformFactory
 {
-    public static function create(string $apiKey): Platform
-    {
-        $handler = new ModelHandler(HttpClient::create(), $apiKey);
+    public static function create(
+        #[\SensitiveParameter]
+        string $apiKey,
+        ?HttpClientInterface $httpClient = null,
+    ): Platform {
+        $httpClient = $httpClient instanceof EventSourceHttpClient ? $httpClient : new EventSourceHttpClient($httpClient);
+        $handler = new ModelHandler($httpClient, $apiKey);
 
         return new Platform([$handler], [$handler]);
     }