@bermudi/pi-delegate 0.1.17 → 0.1.18

package/model.ts

@@ -14,13 +14,45 @@ function resolveModelReference(
   spec: string,
   registry: ModelRegistry,
 ): Model<Api> | undefined {
+  // Pi's own reference grammar is case-insensitive (model-resolver.ts
+  // lowercases provider and id on every match), and callers echo model
+  // strings in whatever casing they saw — e.g. a task model of
+  // `modal/zai-org/glm-5.3-flash:max` against a registry id of
+  // `zai-org/GLM-5.3-Flash`. Matching stricter than pi about case rejects
+  // references pi itself accepts. This is still exact matching: no fuzzy,
+  // partial, or fallback-model resolution — a typo must fail loudly rather
+  // than silently select a different model.
+  const target = spec.toLowerCase();
   const idx = spec.indexOf("/");
   if (idx === -1) {
     // Bare id — match against available models
-    const match = registry.getAvailable().find((m) => m.id === spec);
+    const match = registry
+      .getAvailable()
+      .find((m) => m.id.toLowerCase() === target);
     return match ?? undefined;
   }
-  return registry.find(spec.slice(0, idx), spec.slice(idx + 1)) ?? undefined;
+  const provider = spec.slice(0, idx);
+  const id = spec.slice(idx + 1);
+  // Exact match first (preserves resolution of unauthenticated models via
+  // find), then a case-insensitive scan over the same all-models universe
+  // registry.find() searches.
+  const exact = registry.find(provider, id);
+  if (exact) return exact;
+  const providerLower = provider.toLowerCase();
+  const idLower = id.toLowerCase();
+  const models = registry.getAll();
+  const splitMatch = models.find(
+    (m) =>
+      m.provider.toLowerCase() === providerLower &&
+      m.id.toLowerCase() === idLower,
+  );
+  if (splitMatch) return splitMatch;
+  // Pi-grammar fallback: model ids may themselves contain slashes
+  // (openrouter-style, or namespaced ids like `zai-org/GLM-5.3-Flash`). When
+  // the provider-split reading matches nothing, try the whole spec as an id.
+  // The split reading wins when the prefix is a real provider, matching pi's
+  // own preference order.
+  return models.find((m) => m.id.toLowerCase() === target) ?? undefined;
 }
 
 /** Resolve a model reference, tolerating a Pi-style `:<thinking-level>`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bermudi/pi-delegate",
-  "version": "0.1.17",
+  "version": "0.1.18",
   "description": "Delegate tool for the Pi coding agent.",
   "keywords": [
     "pi-package"