pi-open-agents 0.1.21 → 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 }`
@@ -271,7 +273,13 @@ Agents are loaded from multiple locations (project overrides global by name):
 | `~/.opencode/{agent,agents,mode}/*.md` | Global | OpenCode |
 | `.pi/agents/*.md` | Project | pi |
 | `.opencode/{agent,agents,mode}/*.md` | Project | OpenCode |
-| `.agents/*.md` | Project | Shared |
+| `.agents/agents/*.md` | Project | Shared |
+
+> **⚠️ Migration note (v0.1.22):** discovery in the shared `.agents/` directory is now
+> scoped to `​.agents/agents/*.md` — previously *every* markdown file under `.agents/`
+> (including `skills/` and `commands/`) was parsed as an agent and showed up in `/agents`
+> and the `subagent` tool. If you relied on the loose loading, move your agent files into
+> `.agents/agents/`. Skills and commands are picked up natively by pi as before. (#14)
 
 ---
 

package/package.json

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

package/src/config/paths.ts

@@ -88,7 +88,7 @@ export function getDiscoveryPaths(cwd: string, agentDir: string): DiscoveryPath[
     dir: path.join(cwd, ".agents"),
     source: "project",
     family: "shared",
-    subdirs: [""],
+    subdirs: ["agents"],
   });
 
   return paths;
@@ -99,6 +99,7 @@ export function getDiscoveryPaths(cwd: string, agentDir: string): DiscoveryPath[
  *
  * For pi/shared: uses the filename without extension
  *   .pi/agents/my-agent.md → "my-agent"
+ *   .agents/agents/my-agent.md → "my-agent"
  *
  * For OpenCode: strips the subdir prefix
  *   .opencode/agent/triage.md → "triage"

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").`,
         );
       }