fix(azure/audio): use model param for deployments (#1297)

Author: deyaaeldeen

Diff for: src/core.ts

@@ -814,6 +814,7 @@ export type RequestOptions<
   signal?: AbortSignal | undefined | null;
   idempotencyKey?: string;
 
+  __metadata?: Record<string, unknown>;
   __binaryRequest?: boolean | undefined;
   __binaryResponse?: boolean | undefined;
   __streamClass?: typeof Stream;
@@ -836,6 +837,7 @@ const requestOptionsKeys: KeysEnum<RequestOptions> = {
   signal: true,
   idempotencyKey: true,
 
+  __metadata: true,
   __binaryRequest: true,
   __binaryResponse: true,
   __streamClass: true,

Diff for: src/index.ts

@@ -590,7 +590,7 @@ export class AzureOpenAI extends OpenAI {
       if (!Core.isObj(options.body)) {
         throw new Error('Expected request body to be an object');
       }
-      const model = this.deploymentName || options.body['model'];
+      const model = this.deploymentName || options.body['model'] || options.__metadata?.['model'];
       if (model !== undefined && !this.baseURL.includes('/deployments')) {
         options.path = `/deployments/${model}${options.path}`;
       }

Diff for: src/resources/audio/transcriptions.ts

@@ -25,7 +25,10 @@ export class Transcriptions extends APIResource {
     body: TranscriptionCreateParams,
     options?: Core.RequestOptions,
   ): Core.APIPromise<TranscriptionCreateResponse | string> {
-    return this._client.post('/audio/transcriptions', Core.multipartFormRequestOptions({ body, ...options }));
+    return this._client.post(
+      '/audio/transcriptions',
+      Core.multipartFormRequestOptions({ body, ...options, __metadata: { model: body.model } }),
+    );
   }
 }
 

Diff for: src/resources/audio/translations.ts

@@ -26,7 +26,10 @@ export class Translations extends APIResource {
     body: TranslationCreateParams,
     options?: Core.RequestOptions,
   ): Core.APIPromise<TranslationCreateResponse | string> {
-    return this._client.post('/audio/translations', Core.multipartFormRequestOptions({ body, ...options }));
+    return this._client.post(
+      '/audio/translations',
+      Core.multipartFormRequestOptions({ body, ...options, __metadata: { model: body.model } }),
+    );
   }
 }
 

Diff for: tests/lib/azure.test.ts

@@ -495,21 +495,23 @@ describe('azure request building', () => {
         );
       });
 
-      test('Audio translations is not handled', async () => {
+      test('handles audio translations', async () => {
         const { url } = (await client.audio.translations.create({
           model: deployment,
           file: { url: 'https://example.com', blob: () => 0 as any },
         })) as any;
-        expect(url).toStrictEqual(`https://example.com/openai/audio/translations?api-version=${apiVersion}`);
+        expect(url).toStrictEqual(
+          `https://example.com/openai/deployments/${deployment}/audio/translations?api-version=${apiVersion}`,
+        );
       });
 
-      test('Audio transcriptions is not handled', async () => {
+      test('handles audio transcriptions', async () => {
         const { url } = (await client.audio.transcriptions.create({
           model: deployment,
           file: { url: 'https://example.com', blob: () => 0 as any },
         })) as any;
         expect(url).toStrictEqual(
-          `https://example.com/openai/audio/transcriptions?api-version=${apiVersion}`,
+          `https://example.com/openai/deployments/${deployment}/audio/transcriptions?api-version=${apiVersion}`,
         );
       });