pi-open-agents 0.1.22 → 0.1.23

package/README.md

@@ -175,6 +175,8 @@ This means:
 single call — `subagent({ agent: "deep-research", task: "...", model: "zai/glm-5.3-flash" })`.
 Handy for "run this one task on the cheap model" without forking an agent
 definition. Malformed values (`no-slash`) are rejected before any spawn.
+Aggregator providers work too — the model id may contain slashes:
+`model: "openrouter/anthropic/claude-opus-5.5"`.
 
 The `--tools` whitelist is derived from both explicit `tools:` arrays and
 permission allow-lists. An agent with `permission: { read: allow, edit: allow }`

package/package.json

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

package/src/subagent/tool.ts

@@ -66,6 +66,18 @@ function availableAgentsText(agents: AgentDefinition[]): string {
   );
 }
 
+/**
+ * Validate a per-call model spec: "provider/model-id".
+ *
+ * The model id may itself contain slashes — aggregator providers use
+ * `vendor/model` ids (e.g. "openrouter/anthropic/claude-opus-5.5"), so the
+ * split is on the FIRST slash and only the separator is validated.
+ */
+function isValidModelSpec(spec: string): boolean {
+  const slash = spec.indexOf("/");
+  return slash > 0 && slash < spec.length - 1 && !/\s/.test(spec);
+}
+
 function toToolResult(result: AgentResult) {
   return {
     content: [{ type: "text" as const, text: result.output || "(no output)" }],
@@ -213,7 +225,7 @@ export function registerSubagentTool(
       ),
       model: Type.Optional(
         Type.String({
-          description: 'Per-call model override in "provider/model-id" format (e.g. "anthropic/claude-sonnet-4-5"). Takes precedence over the agent definition default.',
+          description: 'Per-call model override in "provider/model-id" format (e.g. "anthropic/claude-sonnet-4-5"). Model ids may contain slashes for aggregator providers ("openrouter/anthropic/claude-opus-5.5"). Takes precedence over the agent definition default.',
         }),
       ),
     }),
@@ -244,9 +256,9 @@ export function registerSubagentTool(
 
       const childCwd = params.cwd ?? ctx.cwd;
 
-      if (params.model !== undefined && !/^[^/\s]+\/[^/\s]+$/.test(params.model)) {
+      if (params.model !== undefined && !isValidModelSpec(params.model)) {
         throw new Error(
-          `Invalid model "${params.model}": expected "provider/model-id" format (e.g. "anthropic/claude-sonnet-4-5").`,
+          `Invalid model "${params.model}": expected "provider/model-id" format (e.g. "anthropic/claude-sonnet-4-5" or "openrouter/anthropic/claude-opus-5.5").`,
         );
       }