@dougbots/pi-agents 0.1.0 → 0.1.1

package/README.md

@@ -69,6 +69,19 @@ PI_AGENT=jockey pi -c
 
 When `avenor_spawn(agent: "reviewer", backend: "pi")` is called, avenor spawns `pi --mode rpc` with `PI_AGENT=reviewer`. The agents extension applies the full profile (model, systemPrompt, tools, permissions) automatically.
 
+Pass `model` as well to override only the profile's default model while retaining its prompt, tools, and permissions:
+
+```text
+avenor_spawn(
+  agent: "reviewer",
+  model: "anthropic/claude-opus-4",
+  backend: "pi",
+  prompt: "Review the outage recovery changes",
+)
+```
+
+Explicit `--model` selection takes precedence over the profile default. Avenor forwards its `model` option to the pi subprocess as `--model`, so no agent config changes are needed for a one-off override.
+
 Requires [avenor](https://github.com/sdougbrown/avenor) with the pi backend (v0.3.3+). Model resolution falls back to `~/.pi/agent/agents.json` when the agent is not found in opencode config.
 
 ## Profile vs. agent

package/extensions/agents/index.ts

@@ -1,8 +1,8 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { Type } from "typebox";
 import { readFileSync, existsSync } from "node:fs";
 import { join } from "node:path";
+import { findCliModelOverride, parseModelSpecifier } from "./model-selection.ts";
 
 /* ------------------------------------------------------------------ */
 /*  Types                                                              */
@@ -99,6 +99,10 @@ function checkBashPermission(
 
 export default async function (pi: ExtensionAPI) {
   const agents = loadAgentsConfig();
+  // Built-in flags are not exposed through pi.getFlag(). Avenor forwards its
+  // model option to pi as --model, so argv is the extension-level signal that
+  // the caller's model should take precedence over the profile default.
+  const cliModelOverride = findCliModelOverride(process.argv.slice(2));
   let activeAgent: { name: string; profile: AgentProfile } | null = null;
 
   /* ---- Apply agent profile to the current session ---- */
@@ -109,14 +113,23 @@ export default async function (pi: ExtensionAPI) {
       return false;
     }
 
-    // Resolve model
-    const [provider, modelId] = profile.model.split("/");
-    const model = ctx.modelRegistry.find(provider, modelId);
-    if (!model) {
-      ctx.ui.notify(`Model "${profile.model}" not found in registry`, "error");
-      return false;
+    // Explicit CLI selection wins over the profile default. This preserves
+    // `avenor_spawn(agent: ..., model: ..., backend: "pi")` overrides because
+    // Avenor forwards model to the pi subprocess as --model.
+    if (!cliModelOverride) {
+      const modelSpecifier = parseModelSpecifier(profile.model);
+      if (!modelSpecifier) {
+        ctx.ui.notify(`Invalid model "${profile.model}"; expected provider/model-id`, "error");
+        return false;
+      }
+
+      const model = ctx.modelRegistry.find(modelSpecifier.provider, modelSpecifier.modelId);
+      if (!model) {
+        ctx.ui.notify(`Model "${profile.model}" not found in registry`, "error");
+        return false;
+      }
+      await pi.setModel(model);
     }
-    await pi.setModel(model);
 
     // Thinking level
     if (profile.thinkingLevel) {

package/extensions/agents/model-selection.ts

@@ -0,0 +1,29 @@
+export interface ModelSpecifier {
+  provider: string;
+  modelId: string;
+}
+
+/**
+ * Return the model explicitly supplied to pi, if any.
+ *
+ * Pi consumes --model itself, so extension flags cannot expose this value via
+ * pi.getFlag(). Inspecting argv lets agent profiles respect normal CLI
+ * precedence, including models forwarded by Avenor.
+ */
+export function findCliModelOverride(argv: readonly string[]): string | undefined {
+  for (let index = 0; index < argv.length - 1; index += 1) {
+    if (argv[index] === "--model") return argv[index + 1];
+  }
+  return undefined;
+}
+
+/** Split provider/model-id at the first slash so model IDs may contain slashes. */
+export function parseModelSpecifier(specifier: string): ModelSpecifier | undefined {
+  const separator = specifier.indexOf("/");
+  if (separator <= 0 || separator === specifier.length - 1) return undefined;
+
+  return {
+    provider: specifier.slice(0, separator),
+    modelId: specifier.slice(separator + 1),
+  };
+}

package/package.json

@@ -1,7 +1,10 @@
 {
   "name": "@dougbots/pi-agents",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Agent profiles for pi — preconfigured model, system prompt, tool restrictions, and permission gates",
+  "scripts": {
+    "test": "node --experimental-strip-types --test tests/*.test.mts"
+  },
   "keywords": ["pi-package", "pi", "pi-extension", "agents"],
   "license": "MIT",
   "repository": {