Sessions: infer wrapper provider for legacy slash models

gumadeiras · gumadeiras · commit f0953a72845f · 2026-02-25T00:35:50.000-05:00
diff --git a/src/agents/model-selection.test.ts b/src/agents/model-selection.test.ts
@@ -193,6 +193,25 @@ describe("model-selection", () => {
         }),
       ).toBeUndefined();
     });
+
+    it("infers provider for slash-containing model id when allowlist match is unique", () => {
+      const cfg = {
+        agents: {
+          defaults: {
+            models: {
+              "vercel-ai-gateway/anthropic/claude-sonnet-4-6": {},
+            },
+          },
+        },
+      } as OpenClawConfig;
+
+      expect(
+        inferUniqueProviderFromConfiguredModels({
+          cfg,
+          model: "anthropic/claude-sonnet-4-6",
+        }),
+      ).toBe("vercel-ai-gateway");
+    });
   });
 
   describe("buildModelAliasIndex", () => {
diff --git a/src/agents/model-selection.ts b/src/agents/model-selection.ts
@@ -176,7 +176,7 @@ export function inferUniqueProviderFromConfiguredModels(params: {
   model: string;
 }): string | undefined {
   const model = params.model.trim();
-  if (!model || model.includes("/")) {
+  if (!model) {
     return undefined;
   }
   const configuredModels = params.cfg.agents?.defaults?.models;
diff --git a/src/gateway/session-utils.test.ts b/src/gateway/session-utils.test.ts
@@ -468,6 +468,31 @@ describe("resolveSessionModelIdentityRef", () => {
 
     expect(resolved).toEqual({ provider: "anthropic", model: "claude-sonnet-4-6" });
   });
+
+  test("infers wrapper provider for slash-prefixed runtime model when allowlist match is unique", () => {
+    const cfg = {
+      agents: {
+        defaults: {
+          model: { primary: "google-gemini-cli/gemini-3-pro-preview" },
+          models: {
+            "vercel-ai-gateway/anthropic/claude-sonnet-4-6": {},
+          },
+        },
+      },
+    } as OpenClawConfig;
+
+    const resolved = resolveSessionModelIdentityRef(cfg, {
+      sessionId: "slash-model",
+      updatedAt: Date.now(),
+      model: "anthropic/claude-sonnet-4-6",
+      modelProvider: undefined,
+    });
+
+    expect(resolved).toEqual({
+      provider: "vercel-ai-gateway",
+      model: "anthropic/claude-sonnet-4-6",
+    });
+  });
 });
 
 describe("deriveSessionTitle", () => {
@@ -719,6 +744,38 @@ describe("listSessionsFromStore search", () => {
     expect(result.sessions[0]?.model).toBe("claude-sonnet-4-6");
   });
 
+  test("infers wrapper provider for slash-prefixed legacy runtime model when allowlist match is unique", () => {
+    const cfg = {
+      session: { mainKey: "main" },
+      agents: {
+        defaults: {
+          model: { primary: "google-gemini-cli/gemini-3-pro-preview" },
+          models: {
+            "vercel-ai-gateway/anthropic/claude-sonnet-4-6": {},
+          },
+        },
+      },
+    } as OpenClawConfig;
+    const now = Date.now();
+    const store: Record<string, SessionEntry> = {
+      "agent:main:main": {
+        sessionId: "sess-main",
+        updatedAt: now,
+        model: "anthropic/claude-sonnet-4-6",
+      } as SessionEntry,
+    };
+
+    const result = listSessionsFromStore({
+      cfg,
+      storePath: "/tmp/sessions.json",
+      store,
+      opts: {},
+    });
+
+    expect(result.sessions[0]?.modelProvider).toBe("vercel-ai-gateway");
+    expect(result.sessions[0]?.model).toBe("anthropic/claude-sonnet-4-6");
+  });
+
   test("exposes unknown totals when freshness is stale or missing", () => {
     const now = Date.now();
     const store: Record<string, SessionEntry> = {
diff --git a/src/gateway/session-utils.ts b/src/gateway/session-utils.ts
@@ -706,20 +706,21 @@ export function resolveSessionModelIdentityRef(
     if (runtimeProvider) {
       return { provider: runtimeProvider, model: runtimeModel };
     }
+    const inferredProvider = inferUniqueProviderFromConfiguredModels({
+      cfg,
+      model: runtimeModel,
+    });
+    if (inferredProvider) {
+      return { provider: inferredProvider, model: runtimeModel };
+    }
     if (runtimeModel.includes("/")) {
       const parsedRuntime = parseModelRef(runtimeModel, DEFAULT_PROVIDER);
       if (parsedRuntime) {
         return { provider: parsedRuntime.provider, model: parsedRuntime.model };
       }
       return { model: runtimeModel };
     }
-    const inferredProvider = inferUniqueProviderFromConfiguredModels({
-      cfg,
-      model: runtimeModel,
-    });
-    return inferredProvider
-      ? { provider: inferredProvider, model: runtimeModel }
-      : { model: runtimeModel };
+    return { model: runtimeModel };
   }
   const resolved = resolveSessionModelRef(cfg, entry, agentId);
   return { provider: resolved.provider, model: resolved.model };

Original file line number	Diff line number	Diff line change
`@@ -176,7 +176,7 @@ export function inferUniqueProviderFromConfiguredModels(params: {`
`176`	`176`	`model: string;`
`177`	`177`	`}): string \| undefined {`
`178`	`178`	`const model = params.model.trim();`
`179`		`- if (!model \|\| model.includes("/")) {`
	`179`	`+ if (!model) {`
`180`	`180`	`return undefined;`
`181`	`181`	`}`
`182`	`182`	`const configuredModels = params.cfg.agents?.defaults?.models;`