-

Author: OskarStark

src/Platform/Bridge/Albert/EmbeddingsModelClient.php

@@ -36,7 +36,7 @@ public function request(Model $model, array|string $payload, array $options = []
     {
         return $this->httpClient->request('POST', rtrim($this->baseUrl, '/').'/embeddings', [
             'auth_bearer' => $this->apiKey,
-            'json' => is_array($payload) ? array_merge($payload, $options) : $payload,
+            'json' => \is_array($payload) ? array_merge($payload, $options) : $payload,
         ]);
     }
 }

src/Platform/Bridge/Albert/GPTModelClient.php

@@ -40,7 +40,7 @@ public function request(Model $model, array|string $payload, array $options = []
     {
         return $this->httpClient->request('POST', rtrim($this->baseUrl, '/').'/chat/completions', [
             'auth_bearer' => $this->apiKey,
-            'json' => is_array($payload) ? array_merge($payload, $options) : $payload,
+            'json' => \is_array($payload) ? array_merge($payload, $options) : $payload,
         ]);
     }
 }

tests/Platform/Bridge/Albert/EmbeddingsModelClientTest.php

@@ -79,6 +79,7 @@ public function requestSendsCorrectHttpRequest(array|string $payload, array $opt
         $capturedRequest = null;
         $httpClient = new MockHttpClient(function ($method, $url, $options) use (&$capturedRequest) {
             $capturedRequest = ['method' => $method, 'url' => $url, 'options' => $options];
+
             return new JsonMockResponse(['data' => []]);
         });
 
@@ -97,7 +98,7 @@ public function requestSendsCorrectHttpRequest(array|string $payload, array $opt
         self::assertArrayHasKey('normalized_headers', $capturedRequest['options']);
         self::assertArrayHasKey('authorization', $capturedRequest['options']['normalized_headers']);
         self::assertStringContainsString('Bearer test-api-key', $capturedRequest['options']['normalized_headers']['authorization'][0]);
-        
+
         // Check JSON body - it might be in 'body' after processing
         if (isset($capturedRequest['options']['body'])) {
             $actualJson = json_decode($capturedRequest['options']['body'], true);
@@ -140,6 +141,7 @@ public function requestHandlesBaseUrlWithoutTrailingSlash(): void
         $capturedUrl = null;
         $httpClient = new MockHttpClient(function ($method, $url) use (&$capturedUrl) {
             $capturedUrl = $url;
+
             return new JsonMockResponse(['data' => []]);
         });
 
@@ -161,6 +163,7 @@ public function requestHandlesBaseUrlWithTrailingSlash(): void
         $capturedUrl = null;
         $httpClient = new MockHttpClient(function ($method, $url) use (&$capturedUrl) {
             $capturedUrl = $url;
+
             return new JsonMockResponse(['data' => []]);
         });
 
@@ -175,4 +178,4 @@ public function requestHandlesBaseUrlWithTrailingSlash(): void
 
         self::assertSame('https://albert.example.com/v1/embeddings', $capturedUrl);
     }
-}
+}

tests/Platform/Bridge/Albert/GPTModelClientTest.php

@@ -51,7 +51,7 @@ public function constructorThrowsExceptionForEmptyBaseUrl(): void
     public function constructorWrapsHttpClientInEventSourceHttpClient(): void
     {
         $mockHttpClient = new MockHttpClient();
-        
+
         $client = new GPTModelClient(
             $mockHttpClient,
             'test-api-key',
@@ -74,7 +74,7 @@ public function constructorAcceptsEventSourceHttpClient(): void
     {
         $mockHttpClient = new MockHttpClient();
         $eventSourceClient = new EventSourceHttpClient($mockHttpClient);
-        
+
         $client = new GPTModelClient(
             $eventSourceClient,
             'test-api-key',
@@ -124,6 +124,7 @@ public function requestSendsCorrectHttpRequest(array|string $payload, array $opt
         $capturedRequest = null;
         $httpClient = new MockHttpClient(function ($method, $url, $options) use (&$capturedRequest) {
             $capturedRequest = ['method' => $method, 'url' => $url, 'options' => $options];
+
             return new JsonMockResponse(['choices' => []]);
         });
 
@@ -142,7 +143,7 @@ public function requestSendsCorrectHttpRequest(array|string $payload, array $opt
         self::assertArrayHasKey('normalized_headers', $capturedRequest['options']);
         self::assertArrayHasKey('authorization', $capturedRequest['options']['normalized_headers']);
         self::assertStringContainsString('Bearer test-api-key', $capturedRequest['options']['normalized_headers']['authorization'][0]);
-        
+
         // Check JSON body - it might be in 'body' after processing
         if (isset($capturedRequest['options']['body'])) {
             $actualJson = json_decode($capturedRequest['options']['body'], true);
@@ -191,6 +192,7 @@ public function requestHandlesBaseUrlWithoutTrailingSlash(): void
         $capturedUrl = null;
         $httpClient = new MockHttpClient(function ($method, $url) use (&$capturedUrl) {
             $capturedUrl = $url;
+
             return new JsonMockResponse(['choices' => []]);
         });
 
@@ -212,6 +214,7 @@ public function requestHandlesBaseUrlWithTrailingSlash(): void
         $capturedUrl = null;
         $httpClient = new MockHttpClient(function ($method, $url) use (&$capturedUrl) {
             $capturedUrl = $url;
+
             return new JsonMockResponse(['choices' => []]);
         });
 
@@ -226,4 +229,4 @@ public function requestHandlesBaseUrlWithTrailingSlash(): void
 
         self::assertSame('https://albert.example.com/v1/chat/completions', $capturedUrl);
     }
-}
+}