@tonyclaw/agent-inspector 2.0.20 → 2.0.22

package/src/proxy/formats/index.ts

@@ -5,6 +5,7 @@ import "./openai/index";
 // Import providers to trigger self-registration
 import "./anthropic/anthropicProvider";
 import "./openai/provider";
+import "./openai/zhipuProvider";
 import "./openai/alibabaProvider";
 
 export type { FormatHandler, ParsedRequest } from "./handler";

package/src/proxy/formats/openai/index.ts

@@ -22,4 +22,5 @@ export { extractOpenAIStream } from "./stream";
 
 // Providers
 export { openaiProvider } from "./provider";
+export { zhipuProvider } from "./zhipuProvider";
 export { alibabaProvider } from "./alibabaProvider";

package/src/proxy/formats/openai/zhipuProvider.ts

@@ -0,0 +1,31 @@
+import type { CapturedLog } from "../../schemas";
+import type { ProviderProtocol } from "../protocol";
+import { registry } from "../providers";
+import { extractOpenAIStream } from "./stream";
+
+const DEFAULT_ZHIPU_UPSTREAM = "https://open.bigmodel.cn/api/paas/v4";
+const ZHIPU_MODEL_PREFIXES = ["glm-", "charglm-", "cogview-", "embedding-", "rerank-"];
+
+export const zhipuProvider: ProviderProtocol = {
+  name: "zhipu",
+
+  matches(model: string): boolean {
+    const normalized = model.toLowerCase().replace(/\s+/g, "-");
+    return ZHIPU_MODEL_PREFIXES.some((prefix) => normalized.startsWith(prefix));
+  },
+
+  getUpstreamBase(_isChatCompletions: boolean, providerConfig?: { baseUrl?: string }): string {
+    return providerConfig?.baseUrl ?? DEFAULT_ZHIPU_UPSTREAM;
+  },
+
+  extractStream(
+    raw: string,
+    log: CapturedLog,
+    fallbackModel?: string,
+    collectChunks?: boolean,
+  ): string {
+    return extractOpenAIStream(raw, log, fallbackModel, collectChunks);
+  },
+};
+
+registry.register(zhipuProvider);

package/src/proxy/providerImporters.ts

@@ -17,6 +17,10 @@ export type ExternalProvider = {
   alreadyExists: boolean;
 };
 
+const ZHIPU_OPENAI_BASE_URL = "https://open.bigmodel.cn/api/paas/v4";
+const ZHIPU_CODING_OPENAI_BASE_URL = "https://open.bigmodel.cn/api/coding/paas/v4";
+const ZHIPU_CODING_ANTHROPIC_BASE_URL = "https://open.bigmodel.cn/api/anthropic";
+
 function readJsonSafe(filePath: string): Record<string, unknown> | null {
   try {
     if (!existsSync(filePath)) return null;
@@ -51,6 +55,9 @@ const KNOWN_PROVIDER_NAMES: Record<string, string> = {
   azure: "Azure",
   moonshot: "Moonshot",
   zhipu: "ZhipuAI",
+  zhipuai: "ZhipuAI",
+  bigmodel: "ZhipuAI",
+  zai: "Z.AI",
   qwen: "Qwen",
   baichuan: "Baichuan",
   iflytek: "iFlytek",
@@ -399,6 +406,46 @@ function detectMiMoCodeProviders(): ExternalProvider[] {
         openaiBaseUrl: "https://api.together.xyz",
         format: "openai",
       },
+      zhipu: {
+        anthropicBaseUrl: "",
+        openaiBaseUrl: ZHIPU_OPENAI_BASE_URL,
+        format: "openai",
+      },
+      zhipuai: {
+        anthropicBaseUrl: "",
+        openaiBaseUrl: ZHIPU_OPENAI_BASE_URL,
+        format: "openai",
+      },
+      bigmodel: {
+        anthropicBaseUrl: "",
+        openaiBaseUrl: ZHIPU_OPENAI_BASE_URL,
+        format: "openai",
+      },
+      zai: {
+        anthropicBaseUrl: "",
+        openaiBaseUrl: ZHIPU_OPENAI_BASE_URL,
+        format: "openai",
+      },
+      glm: {
+        anthropicBaseUrl: ZHIPU_CODING_ANTHROPIC_BASE_URL,
+        openaiBaseUrl: ZHIPU_CODING_OPENAI_BASE_URL,
+        format: "anthropic",
+      },
+      "glm-coding-plan": {
+        anthropicBaseUrl: ZHIPU_CODING_ANTHROPIC_BASE_URL,
+        openaiBaseUrl: ZHIPU_CODING_OPENAI_BASE_URL,
+        format: "anthropic",
+      },
+      "zhipu-coding-plan": {
+        anthropicBaseUrl: ZHIPU_CODING_ANTHROPIC_BASE_URL,
+        openaiBaseUrl: ZHIPU_CODING_OPENAI_BASE_URL,
+        format: "anthropic",
+      },
+      "zai-coding-plan": {
+        anthropicBaseUrl: ZHIPU_CODING_ANTHROPIC_BASE_URL,
+        openaiBaseUrl: ZHIPU_CODING_OPENAI_BASE_URL,
+        format: "anthropic",
+      },
     };
 
     for (const [key, cred] of Object.entries(auth)) {

package/src/proxy/providers.ts

@@ -665,6 +665,56 @@ function normalizeModelName(name: string): string {
   return name.toLowerCase().replace(/\s+/g, "-");
 }
 
+const ZHIPU_MODEL_PREFIXES = ["glm-", "charglm-", "cogview-", "embedding-", "rerank-"];
+const ZHIPU_PROVIDER_ALIASES = ["zhipu", "zhipuai", "bigmodel", "zai", "glm"];
+const ZHIPU_BASE_URL_MARKERS = [
+  "open.bigmodel.cn/api/paas/v4",
+  "open.bigmodel.cn/api/coding/paas/v4",
+  "open.bigmodel.cn/api/anthropic",
+];
+
+function normalizeProviderAlias(name: string): string {
+  return name.toLowerCase().replace(/[^a-z0-9]/g, "");
+}
+
+function providerRoutingUrls(provider: ProviderConfig): string[] {
+  const urls: string[] = [];
+  if (provider.anthropicBaseUrl !== undefined && provider.anthropicBaseUrl.trim() !== "") {
+    urls.push(provider.anthropicBaseUrl);
+  }
+  if (provider.openaiBaseUrl !== undefined && provider.openaiBaseUrl.trim() !== "") {
+    urls.push(provider.openaiBaseUrl);
+  }
+  if (provider.baseUrl !== undefined && provider.baseUrl.trim() !== "") {
+    urls.push(provider.baseUrl);
+  }
+  return urls;
+}
+
+function isZhipuProvider(provider: ProviderConfig): boolean {
+  const normalizedName = normalizeProviderAlias(provider.name);
+  const nameMatches = ZHIPU_PROVIDER_ALIASES.some((alias) => {
+    const normalizedAlias = normalizeProviderAlias(alias);
+    return normalizedName.includes(normalizedAlias) || normalizedAlias.includes(normalizedName);
+  });
+  if (nameMatches) return true;
+
+  return providerRoutingUrls(provider).some((url) => {
+    const normalizedUrl = url.toLowerCase().replace(/\/+$/, "");
+    return ZHIPU_BASE_URL_MARKERS.some((marker) => normalizedUrl.includes(marker));
+  });
+}
+
+function modelMatchesKnownProviderPrefix(
+  modelNormalized: string,
+  provider: ProviderConfig,
+): boolean {
+  if (isZhipuProvider(provider)) {
+    return ZHIPU_MODEL_PREFIXES.some((prefix) => modelNormalized.startsWith(prefix));
+  }
+  return false;
+}
+
 /**
  * Finds a provider by model name using three strategies:
  * 1. Case-insensitive prefix match against "{provider.name}-" (e.g., "deepseek-" matches "deepseek-*")
@@ -683,6 +733,11 @@ export function findProviderByModel(model: string): ProviderConfig | null {
     if (modelLower.startsWith(providerPrefix)) {
       return provider;
     }
+    // Strategy 1b: known model-family prefixes for providers whose public
+    // model names do not start with the provider brand, e.g. ZhipuAI -> glm-*.
+    if (modelMatchesKnownProviderPrefix(modelNormalized, provider)) {
+      return provider;
+    }
     // Strategy 2: match against provider.models array with normalization
     const modelList = provider.models ?? (provider.model !== undefined ? [provider.model] : []);
     for (const m of modelList) {

package/src/proxy/upstream.ts

@@ -60,6 +60,10 @@ export function selectUpstreamBase(route: ApiRoute, provider: ProviderConfig | n
 export function buildUpstreamUrl(upstreamBase: string, normalizedPath: string): string {
   const base = upstreamBase.endsWith("/") ? upstreamBase.slice(0, -1) : upstreamBase;
 
+  if (isZhipuVersionedOpenAIBase(base) && isVersionedChatCompletionsPath(normalizedPath)) {
+    return base + normalizedPath.slice(3);
+  }
+
   // Many OpenAI-compatible base URLs already include /v1. Avoid producing /v1/v1.
   if (base.endsWith("/v1") && normalizedPath.startsWith("/v1/")) {
     return base + normalizedPath.slice(3);
@@ -67,6 +71,27 @@ export function buildUpstreamUrl(upstreamBase: string, normalizedPath: string):
   return base + normalizedPath;
 }
 
+function isVersionedChatCompletionsPath(normalizedPath: string): boolean {
+  return (
+    normalizedPath === PATH_V1_CHAT_COMPLETIONS ||
+    normalizedPath.startsWith(`${PATH_V1_CHAT_COMPLETIONS}?`) ||
+    normalizedPath.startsWith(`${PATH_V1_CHAT_COMPLETIONS}/`)
+  );
+}
+
+function isZhipuVersionedOpenAIBase(base: string): boolean {
+  try {
+    const parsed = new URL(base);
+    const path = parsed.pathname.replace(/\/+$/, "");
+    return (
+      parsed.hostname === "open.bigmodel.cn" &&
+      (path === "/api/paas/v4" || path === "/api/coding/paas/v4")
+    );
+  } catch {
+    return false;
+  }
+}
+
 export function setUpstreamHost(headers: Headers, upstreamBase: string): void {
   try {
     headers.set(HEADER_HOST, new URL(upstreamBase).host);

package/src/routes/api/config.ts

@@ -1,6 +1,7 @@
 import { createFileRoute } from "@tanstack/react-router";
 import { z } from "zod";
 import {
+  MAX_PROVIDER_TEST_TIMEOUT_SECONDS,
   MAX_SLOW_RESPONSE_THRESHOLD_SECONDS,
   TimeDisplayFormatSchema,
 } from "../../lib/runtimeConfig";
@@ -17,6 +18,12 @@ const RuntimeConfigPatchSchema = z
       .min(0)
       .max(MAX_SLOW_RESPONSE_THRESHOLD_SECONDS)
       .optional(),
+    providerTestTimeoutSeconds: z
+      .number()
+      .int()
+      .min(1)
+      .max(MAX_PROVIDER_TEST_TIMEOUT_SECONDS)
+      .optional(),
     timeDisplayFormat: TimeDisplayFormatSchema.optional(),
   })
   .strict()

package/src/routes/api/providers.$providerId.test.log.ts

@@ -13,6 +13,22 @@ function hasSuccessField(result: ProviderTestState): result is TestResult {
   return Object.prototype.hasOwnProperty.call(result, "success");
 }
 
+function getResultResponseStatus(result: TestResult): number {
+  if (result.debug?.responseStatus !== undefined) {
+    return result.debug.responseStatus;
+  }
+
+  return result.success ? 200 : 500;
+}
+
+function getResultResponseText(result: TestResult): string {
+  return result.rawResponse ?? result.debug?.responseBody ?? JSON.stringify(result);
+}
+
+function getResultHeaders(result: TestResult): Record<string, string> {
+  return result.requestHeaders ?? result.debug?.requestHeaders ?? {};
+}
+
 async function logModelResults(
   displayName: string,
   providerName: string,
@@ -42,8 +58,8 @@ async function logModelResults(
         model: nonStreamingResult.model ?? displayName,
         sessionId: null,
         rawRequestBody: requestBody,
-        responseStatus: nonStreamingResult.success ? 200 : 500,
-        responseText: nonStreamingResult.rawResponse ?? JSON.stringify(nonStreamingResult),
+        responseStatus: getResultResponseStatus(nonStreamingResult),
+        responseText: getResultResponseText(nonStreamingResult),
         inputTokens: nonStreamingResult.inputTokens ?? null,
         outputTokens: nonStreamingResult.outputTokens ?? null,
         cacheCreationInputTokens: nonStreamingResult.cacheCreationInputTokens ?? null,
@@ -55,7 +71,7 @@ async function logModelResults(
         apiFormat: "anthropic",
         isTest: true,
         providerName,
-        headers: captureFullDetails ? (nonStreamingResult.requestHeaders ?? {}) : undefined,
+        headers: captureFullDetails ? getResultHeaders(nonStreamingResult) : undefined,
       });
 
       const streamingRequestBody = JSON.stringify(
@@ -69,8 +85,8 @@ async function logModelResults(
         model: streamingResult.model ?? displayName,
         sessionId: null,
         rawRequestBody: streamingRequestBody,
-        responseStatus: streamingResult.success ? 200 : 500,
-        responseText: streamingResult.rawResponse ?? JSON.stringify(streamingResult),
+        responseStatus: getResultResponseStatus(streamingResult),
+        responseText: getResultResponseText(streamingResult),
         inputTokens: streamingResult.inputTokens ?? null,
         outputTokens: streamingResult.outputTokens ?? null,
         cacheCreationInputTokens: streamingResult.cacheCreationInputTokens ?? null,
@@ -83,7 +99,7 @@ async function logModelResults(
         apiFormat: "anthropic",
         isTest: true,
         providerName,
-        headers: captureFullDetails ? (streamingResult.requestHeaders ?? {}) : undefined,
+        headers: captureFullDetails ? getResultHeaders(streamingResult) : undefined,
       });
     }
   }
@@ -104,8 +120,8 @@ async function logModelResults(
         model: nonStreamingResult.model ?? displayName,
         sessionId: null,
         rawRequestBody: requestBody,
-        responseStatus: nonStreamingResult.success ? 200 : 500,
-        responseText: nonStreamingResult.rawResponse ?? JSON.stringify(nonStreamingResult),
+        responseStatus: getResultResponseStatus(nonStreamingResult),
+        responseText: getResultResponseText(nonStreamingResult),
         inputTokens: nonStreamingResult.inputTokens ?? null,
         outputTokens: nonStreamingResult.outputTokens ?? null,
         cacheCreationInputTokens: null,
@@ -117,7 +133,7 @@ async function logModelResults(
         apiFormat: "openai",
         isTest: true,
         providerName,
-        headers: captureFullDetails ? (nonStreamingResult.requestHeaders ?? {}) : undefined,
+        headers: captureFullDetails ? getResultHeaders(nonStreamingResult) : undefined,
       });
 
       const streamingRequestBody = JSON.stringify(
@@ -131,8 +147,8 @@ async function logModelResults(
         model: streamingResult.model ?? displayName,
         sessionId: null,
         rawRequestBody: streamingRequestBody,
-        responseStatus: streamingResult.success ? 200 : 500,
-        responseText: streamingResult.rawResponse ?? JSON.stringify(streamingResult),
+        responseStatus: getResultResponseStatus(streamingResult),
+        responseText: getResultResponseText(streamingResult),
         inputTokens: streamingResult.inputTokens ?? null,
         outputTokens: streamingResult.outputTokens ?? null,
         cacheCreationInputTokens: null,
@@ -145,7 +161,7 @@ async function logModelResults(
         apiFormat: "openai",
         isTest: true,
         providerName,
-        headers: captureFullDetails ? (streamingResult.requestHeaders ?? {}) : undefined,
+        headers: captureFullDetails ? getResultHeaders(streamingResult) : undefined,
       });
     }
   }