fix: consistent Whisper task option for OpenAI and Azure bridges

src/platform/src/Bridge/Azure/OpenAI/WhisperModelClient.php

@@ -12,6 +12,7 @@
 namespace Symfony\AI\Platform\Bridge\Azure\OpenAI;
 
 use Symfony\AI\Platform\Bridge\OpenAI\Whisper;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper\Task;
 use Symfony\AI\Platform\Model;
 use Symfony\AI\Platform\ModelClientInterface;
 use Symfony\Component\HttpClient\EventSourceHttpClient;
@@ -48,7 +49,11 @@ public function supports(Model $model): bool
 
     public function request(Model $model, array|string $payload, array $options = []): ResponseInterface
     {
-        $url = \sprintf('https://%s/openai/deployments/%s/audio/translations', $this->baseUrl, $this->deployment);
+        $task = $options['task'] ?? Task::TRANSCRIPTION;
+        $endpoint = Task::TRANSCRIPTION === $task ? 'transcriptions' : 'translations';
+        $url = \sprintf('https://%s/openai/deployments/%s/audio/%s', $this->baseUrl, $this->deployment, $endpoint);
+
+        unset($options['task']);
 
         return $this->httpClient->request('POST', $url, [
             'headers' => [

src/platform/src/Bridge/OpenAI/Whisper/ModelClient.php

@@ -38,7 +38,11 @@ public function supports(Model $model): bool
 
     public function request(Model $model, array|string $payload, array $options = []): ResponseInterface
     {
-        return $this->httpClient->request('POST', 'https://api.openai.com/v1/audio/transcriptions', [
+        $task = $options['task'] ?? Task::TRANSCRIPTION;
+        $endpoint = Task::TRANSCRIPTION === $task ? 'transcriptions' : 'translations';
+        unset($options['task']);
+
+        return $this->httpClient->request('POST', \sprintf('https://api.openai.com/v1/audio/%s', $endpoint), [
             'auth_bearer' => $this->apiKey,
             'headers' => ['Content-Type' => 'multipart/form-data'],
             'body' => array_merge($options, $payload, ['model' => $model->getName()]),

src/platform/src/Bridge/OpenAI/Whisper/Task.php

@@ -0,0 +1,21 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\AI\Platform\Bridge\OpenAI\Whisper;
+
+/**
+ * @author Christopher Hertel <[email protected]>
+ */
+interface Task
+{
+    public const TRANSCRIPTION = 'transcription';
+    public const TRANSLATION = 'translation';
+}

src/platform/tests/Bridge/Azure/OpenAI/WhisperModelClientTest.php

@@ -0,0 +1,107 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\AI\Platform\Tests\Bridge\Azure\OpenAI;
+
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Small;
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+use Symfony\AI\Platform\Bridge\Azure\OpenAI\WhisperModelClient;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper\Task;
+use Symfony\Component\HttpClient\MockHttpClient;
+use Symfony\Component\HttpClient\Response\MockResponse;
+
+#[CoversClass(WhisperModelClient::class)]
+#[Small]
+final class WhisperModelClientTest extends TestCase
+{
+    #[Test]
+    public function itSupportsWhisperModel(): void
+    {
+        $client = new WhisperModelClient(
+            new MockHttpClient(),
+            'test.openai.azure.com',
+            'whisper-deployment',
+            '2023-12-01-preview',
+            'test-key'
+        );
+        $model = new Whisper();
+
+        self::assertTrue($client->supports($model));
+    }
+
+    #[Test]
+    public function itUsesTranscriptionEndpointByDefault(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://test.azure.com/openai/deployments/whspr/audio/transcriptions?api-version=2023-12', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new WhisperModelClient($httpClient, 'test.azure.com', 'whspr', '2023-12', 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+
+        $client->request($model, $payload);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+
+    #[Test]
+    public function itUsesTranscriptionEndpointWhenTaskIsSpecified(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://test.azure.com/openai/deployments/whspr/audio/transcriptions?api-version=2023-12', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new WhisperModelClient($httpClient, 'test.azure.com', 'whspr', '2023-12', 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+        $options = ['task' => Task::TRANSCRIPTION];
+
+        $client->request($model, $payload, $options);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+
+    #[Test]
+    public function itUsesTranslationEndpointWhenTaskIsSpecified(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://test.azure.com/openai/deployments/whspr/audio/translations?api-version=2023-12', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new WhisperModelClient($httpClient, 'test.azure.com', 'whspr', '2023-12', 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+        $options = ['task' => Task::TRANSLATION];
+
+        $client->request($model, $payload, $options);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+}

src/platform/tests/Bridge/OpenAI/Whisper/ModelClientTest.php

@@ -0,0 +1,101 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\AI\Platform\Tests\Bridge\OpenAI\Whisper;
+
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Small;
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper\ModelClient;
+use Symfony\AI\Platform\Bridge\OpenAI\Whisper\Task;
+use Symfony\Component\HttpClient\MockHttpClient;
+use Symfony\Component\HttpClient\Response\MockResponse;
+
+#[CoversClass(ModelClient::class)]
+#[Small]
+final class ModelClientTest extends TestCase
+{
+    #[Test]
+    public function itSupportsWhisperModel(): void
+    {
+        $client = new ModelClient(new MockHttpClient(), 'test-key');
+        $model = new Whisper();
+
+        self::assertTrue($client->supports($model));
+    }
+
+    #[Test]
+    public function itUsesTranscriptionEndpointByDefault(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://api.openai.com/v1/audio/transcriptions', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new ModelClient($httpClient, 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+
+        $client->request($model, $payload);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+
+    #[Test]
+    public function itUsesTranscriptionEndpointWhenTaskIsSpecified(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://api.openai.com/v1/audio/transcriptions', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new ModelClient($httpClient, 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+        $options = ['task' => Task::TRANSCRIPTION];
+
+        $client->request($model, $payload, $options);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+
+    #[Test]
+    public function itUsesTranslationEndpointWhenTaskIsSpecified(): void
+    {
+        $httpClient = new MockHttpClient([
+            function ($method, $url): MockResponse {
+                self::assertSame('POST', $method);
+                self::assertSame('https://api.openai.com/v1/audio/translations', $url);
+
+                return new MockResponse('{"text": "Hello World"}');
+            },
+        ]);
+
+        $client = new ModelClient($httpClient, 'test-key');
+        $model = new Whisper();
+        $payload = ['file' => 'audio-data'];
+        $options = ['task' => Task::TRANSLATION];
+
+        $client->request($model, $payload, $options);
+
+        self::assertSame(1, $httpClient->getRequestsCount());
+    }
+}