fix(azure): narrow Foundry Responses client detection

vincentkoc · vincentkoc · commit c5171625369e · 2026-06-09T16:45:14.000+09:00
diff --git a/src/llm/providers/azure-openai-responses.test.ts b/src/llm/providers/azure-openai-responses.test.ts
@@ -45,6 +45,11 @@ describe("azure-openai-responses", () => {
         "https://project.services.ai.azure.com/openai/v1",
       ),
     ).toBe(true);
+    expect(
+      testing.isOpenAICompatibleAzureResponsesBaseUrl(
+        "https://eastus.api.cognitive.microsoft.com/openai/v1",
+      ),
+    ).toBe(true);
   });
 
   it("does not treat non-v1 custom endpoints as OpenAI-compatible Responses bases", () => {
@@ -54,4 +59,15 @@ describe("azure-openai-responses", () => {
       ),
     ).toBe(false);
   });
+
+  it("keeps private or APIM Azure OpenAI-compatible paths on the AzureOpenAI client path", () => {
+    expect(testing.isOpenAICompatibleAzureResponsesBaseUrl("https://aoai.internal/openai/v1")).toBe(
+      false,
+    );
+    expect(
+      testing.isOpenAICompatibleAzureResponsesBaseUrl(
+        "https://gateway.example.com/proxy/openai/v1",
+      ),
+    ).toBe(false);
+  });
 });
diff --git a/src/shared/azure-openai-responses-client-compat.ts b/src/shared/azure-openai-responses-client-compat.ts
@@ -16,6 +16,14 @@ export function isOpenAICompatibleAzureResponsesBaseUrl(baseUrl: string): boolea
     return false;
   }
 
+  const hostname = url.hostname.toLowerCase();
+  const isFoundryHost =
+    hostname.endsWith(".services.ai.azure.com") ||
+    hostname.endsWith(".api.cognitive.microsoft.com");
+  if (!isFoundryHost) {
+    return false;
+  }
+
   const normalizedPath = url.pathname.replace(/\/+$/, "");
   return normalizedPath === "/openai/v1" || normalizedPath.endsWith("/openai/v1");
 }

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,14 @@ export function isOpenAICompatibleAzureResponsesBaseUrl(baseUrl: string): boolea`
`16`	`16`	`return false;`
`17`	`17`	`}`
`18`	`18`
	`19`	`+ const hostname = url.hostname.toLowerCase();`
	`20`	`+ const isFoundryHost =`
	`21`	`+ hostname.endsWith(".services.ai.azure.com") \|\|`
	`22`	`+ hostname.endsWith(".api.cognitive.microsoft.com");`
	`23`	`+ if (!isFoundryHost) {`
	`24`	`+ return false;`
	`25`	`+ }`
	`26`	`+`
`19`	`27`	`const normalizedPath = url.pathname.replace(/\/+$/, "");`
`20`	`28`	`return normalizedPath === "/openai/v1" \|\| normalizedPath.endsWith("/openai/v1");`
`21`	`29`	`}`