pi-open-agents 0.1.28 → 0.1.29

package/README.md

@@ -398,7 +398,7 @@ opening an issue or pull request.
 
 ```bash
 npm install
-npm test          # 191 tests
+npm test          # 197 tests
 npm run typecheck # tsc --noEmit
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-open-agents",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "description": "Unified agent and subagent management for pi coding agent, with OpenCode-compatible agent definitions.",
   "keywords": [
     "pi-package",

package/src/subagent/executor.ts

@@ -440,10 +440,28 @@ function usageFromMessage(
   };
 }
 
+/**
+ * Extract the display model string from an assistant message (issue #27).
+ *
+ * `AssistantMessage` always carries both `provider` and `model`, so prefix the
+ * bare id (`provider/model`) to match the `model:` frontmatter format and the
+ * per-call override (`parseModelRef` splits on the first slash, so a
+ * namespaced id such as `anthropic/claude-opus-5.5` under `openrouter`
+ * round-trips exactly). No `provider` → bare id, as before.
+ *
+ * The `startsWith` guard prevents `provider/provider/...` when a host echoes
+ * an already-qualified id. Deliberately NOT `responseModel`: that is the
+ * provider-native answering snapshot (alias → dated version), a different
+ * granularity from "which provider ran the agent".
+ */
 function modelFromMessage(message: unknown): string | undefined {
   if (!message || typeof message !== "object") return undefined;
-  const model = (message as { model?: unknown }).model;
-  return typeof model === "string" ? model : undefined;
+  const data = message as { provider?: unknown; model?: unknown };
+  if (typeof data.model !== "string") return undefined;
+  if (typeof data.provider === "string" && !data.model.startsWith(`${data.provider}/`)) {
+    return `${data.provider}/${data.model}`;
+  }
+  return data.model;
 }
 
 function updateUsage(target: AgentUsage, update: Partial<AgentUsage> | undefined) {