@bacnh85/pi-subagent 0.16.0 → 0.17.0

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # Changelog
 
+## 0.17.0 (2026-08-31)
+
+### Added
+
+- **Add/remove model roles from the `/subagent` panel** — new `+ Add role` /
+  `− Remove role` action rows (two-prompt flow: name → chain). Custom roles are
+  deletable; built-in roles (`fast`/`coder`/`smart`) reset to their bundled
+  default. New names validate against `[A-Za-z0-9._-]{1,64}` with
+  case-insensitive collision checks.
+- **Per-agent default display** — each override row now shows the agent's
+  default (no-override) chain, e.g. `scout  (default: @fast → zai-coding-cn/
+  glm-5-turbo, …)`, so blank = inherit is meaningful. Labels track live role
+  edits and freshly added roles appear in `@role` completions immediately.
+- Panel save is now guarded by a working-copy content diff (action-only
+  sessions — add/remove without row edits — previously left `editedKeys` empty
+  and silently skipped persistence).
+
+## 0.16.1 (2026-08-29)
+
+### Added
+
+- `/subagent` argument completion: keywords (`list|all|agents|roles|reload|
+  refresh|history`), discovered agent names, and `@role` refs.
+- Roles editor rows now offer inline model suggestions (Tab to pick, Enter
+  keeps typed text) when run against @bacnh85/pi-config-panel >= 0.1.1; the
+  package stays compilable and fully functional on 0.1.0 (suggestions simply
+  absent), so no dependency floor bump is required.
+
 ## 0.16.0 (2026-08-23)
 
 ### Features

package/README.md

@@ -34,9 +34,11 @@ in `~/.pi/agent/settings.json` under `subagent.roles`:
 
 `/subagent` opens the interactive role editor (TUI panel via the shared
 `@bacnh85/pi-config-panel` kernel; prints the effective mapping headless).
-`/subagent list` lists agents, `/subagent <name>` shows an agent's resolved
-chain, and `/subagent @role` (or `/subagent fast`) shows a role's chain and
-the agents using it.
+The panel supports `+ Add role` / `− Remove role` (custom roles are deletable,
+built-ins reset to their bundled default) and shows each agent's default
+(no-override) chain on its row. `/subagent list` lists agents,
+`/subagent <name>` shows an agent's resolved chain, and `/subagent @role`
+(or `/subagent fast`) shows a role's chain and the agents using it.
 
 ## Live progress widget
 

package/extensions/index.ts

@@ -329,6 +329,20 @@ export default function (pi: ExtensionAPI) {
   });
   pi.registerCommand("subagent", {
     description: "Configure model roles (/subagent), list agents (/subagent list), agent details (/subagent <name>), role detail (/subagent @role), reload definitions (/subagent reload), history (/subagent history)",
+    getArgumentCompletions: (prefix) => {
+      const ctx = currentCtx;
+      const keywords = ["list", "all", "agents", "roles", "reload", "refresh", "history"];
+      const vocab = [...keywords];
+      if (ctx) {
+        const discovery = discoverAgents(ctx.cwd, "both", bundledAgentsDir);
+        vocab.push(...discovery.agents.map((a) => a.name));
+        try { vocab.push(...Object.keys(readSubagentRoles(ctx).roles).map((r) => `@${r}`)); } catch { /* roles optional */ }
+      }
+      const q = prefix.trim().toLowerCase();
+      const items = vocab.filter((v) => v.toLowerCase().startsWith(q))
+        .map((v) => ({ value: v, label: v, description: keywords.includes(v) ? "subagent command" : "agent / role" }));
+      return items.length > 0 ? items : null;
+    },
     handler: async (args, ctx) => {
       const cmd = args.trim().toLowerCase();
       const discovery = discoverAgents(ctx.cwd, "both", bundledAgentsDir);
@@ -364,7 +378,7 @@ export default function (pi: ExtensionAPI) {
 
       const openRolesEditor = async (): Promise<void> => {
         // Role mapping editor: panel in TUI, plain text otherwise.
-        const [{ openConfigPanel }, { buildRows, buildRolesPanelCfg, cfgToPatch, preserveUnknownAgentModels, writeSubagentSection }] = await Promise.all([
+        const [{ openConfigPanel }, { buildRows, buildRolesPanelCfg, cfgToPatch, makeAddRoleAction, makeRemoveRoleAction, preserveUnknownAgentModels, writeSubagentSection }] = await Promise.all([
           import("@bacnh85/pi-config-panel"),
           import("./roles-panel.ts"),
         ]);
@@ -387,13 +401,31 @@ export default function (pi: ExtensionAPI) {
         }
         const current = readSubagentRolesGlobal();
         const working = buildRolesPanelCfg(discovery.agents, current);
+        // Actions (add/remove role) set model.dirty but not editedKeys — guard
+        // the save on a working-copy diff instead (pi-a2a pattern).
+        const before = JSON.stringify([working.roles, working.agentModels]);
+        const notify = (message: string, kind?: "info" | "warning" | "error") => ctx.ui.notify(message, kind ?? "info");
+        const panelOptions = {
+          models: () => {
+            try { return ctx.modelRegistry.getAvailable().map((m) => `${m.provider}/${m.id}`); }
+            catch { return []; }
+          },
+          // Working copy: tracks live edits + freshly added (unsaved) roles.
+          roles: () => Object.keys(working.roles),
+          effectiveRoles: working.roles,
+        };
+        const actions = {
+          addRole: makeAddRoleAction(working, { notify }),
+          removeRole: makeRemoveRoleAction(working, { notify }),
+        };
         await openConfigPanel({
           ctx,
           cfg: working,
-          build: (cfg) => buildRows(cfg, discovery.agents),
+          actions,
+          build: (cfg, panelActions) => buildRows(cfg, discovery.agents, panelOptions, panelActions),
           title: "Subagent model roles",
-          onSave: (saved, editedKeys) => {
-            if (!(saved && editedKeys && editedKeys.size > 0)) return;
+          onSave: (saved) => {
+            if (!saved || JSON.stringify([working.roles, working.agentModels]) === before) return;
             const patch = cfgToPatch(working);
             patch.agentModels = preserveUnknownAgentModels(
               patch.agentModels,

package/extensions/roles-panel.ts

@@ -13,9 +13,19 @@ import { existsSync, mkdirSync, readFileSync, renameSync, writeFileSync } from "
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
 import { row } from "@bacnh85/pi-config-panel";
-import type { PanelGroup } from "@bacnh85/pi-config-panel";
+import type { PanelAction, PanelGroup, PanelRow } from "@bacnh85/pi-config-panel";
+
+// ponytail: local structural type — the kernel only reads value/label/
+// description, so this stays compatible with the published 0.1.0 range while
+// completion support ships in 0.1.1 (no static import of new kernel symbols).
+interface CompletionItem {
+  value: string;
+  label?: string;
+  description?: string;
+}
 import type { AgentConfig } from "./agents.ts";
-import { DEFAULT_ROLES, readSubagentRoles, type RolesConfig } from "./roles.ts";
+import { getModelCandidates } from "./agents.ts";
+import { DEFAULT_ROLES, readSubagentRoles, resolveAgentModelChain, type RoleMap, type RolesConfig } from "./roles.ts";
 
 // ---------------------------------------------------------------------------
 // Settings persistence (global only — repo .pi/settings.json is read-only)
@@ -83,6 +93,17 @@ export interface RolesPanelCfg {
   agentModels: Record<string, string>;
 }
 
+/** Completion sources for the panel's model rows (lazy — resolved per keypress). */
+export interface RolesPanelOptions {
+  /** Available model refs (`provider/id`), sorted; may be empty before registry sync. */
+  models: () => string[];
+  /** Known role names (defaults + configured), offered as `@role` on agent rows. */
+  roles: () => string[];
+  /** Effective role chains used to render each agent's default (falls back to
+   *  DEFAULT_ROLES). Pass the working copy's roles so labels track live edits. */
+  effectiveRoles?: RoleMap;
+}
+
 /** Seed a working config from current effective settings + bundled agents. */
 export function buildRolesPanelCfg(agents: AgentConfig[], current: RolesConfig): RolesPanelCfg {
   const roleNames = new Set([...Object.keys(DEFAULT_ROLES), ...Object.keys(current.roles)]);
@@ -98,24 +119,62 @@ export function buildRolesPanelCfg(agents: AgentConfig[], current: RolesConfig):
   return cfg;
 }
 
-/** Build panel groups. Role rows first, then one override row per agent. */
-export function buildRows(cfg: RolesPanelCfg, agents: AgentConfig[]): PanelGroup[] {
+/** Human-readable default (no-override) chain for an agent's panel row label. */
+export function defaultChainLabel(
+  agent: Pick<AgentConfig, "name" | "model" | "models">,
+  roles: RoleMap,
+): string {
+  const raw = getModelCandidates(agent);
+  const alias = raw.filter((m) => m.startsWith("@")).join(", ");
+  const { candidates } = resolveAgentModelChain(agent, { roles, agentModels: {} });
+  const chain = candidates.length > 0 ? candidates.join(", ") : "parent fallback";
+  // alias is @-prefixed, candidates are expanded model ids — never equal.
+  return alias ? `default: ${alias} → ${chain}` : `default: ${chain}`;
+}
+
+/** Build panel groups. Role rows first, then one override row per agent.
+ *  `options` adds inline model/@role completions when provided (optional so
+ *  existing unit tests and non-TUI callers stay unchanged). `actions` appends
+ *  the + Add role / − Remove role action rows when provided. */
+export function buildRows(
+  cfg: RolesPanelCfg,
+  agents: AgentConfig[],
+  options?: RolesPanelOptions,
+  actions?: Record<string, PanelAction>,
+): PanelGroup[] {
   const defaultChain = (name: string) => Array.isArray(DEFAULT_ROLES[name]) ? (DEFAULT_ROLES[name] as string[]).join(", ") : String(DEFAULT_ROLES[name] ?? "");
+  const modelItems = (): CompletionItem[] =>
+    (options?.models() ?? []).sort().map((ref) => ({ value: ref }));
+  const roleItems = (): CompletionItem[] =>
+    (options?.roles() ?? []).map((name) => ({ value: `@${name}`, description: "role chain" }));
+  // ponytail: opts spread keeps this compilable against kernel 0.1.0 (whose
+  // row() opts type lacks `completions`); the runtime contract is additive and
+  // 0.1.1+ consumes the field. Drop the cast when the dep floor moves to 0.1.1.
+  const withCompletions = (completions: () => CompletionItem[]) =>
+    ({ completions }) as unknown as { mask?: boolean };
   const roleRows = Object.keys(cfg.roles).sort().map((name) => {
     // Label shows the default chain so blank is meaningful.
     return row(`role.${name}`, `@${name}  (default: ${defaultChain(name) || "none"})`, "string", cfg.roles[name], (v) => {
       cfg.roles[name] = String(v ?? "").trim();
-    });
+    }, withCompletions(modelItems));
   });
+  const effective = options?.effectiveRoles ?? DEFAULT_ROLES;
   const agentRows = agents.map((agent) =>
-    row(`agent.${agent.name}`, agent.name, "string", cfg.agentModels[agent.name] ?? "", (v) => {
+    row(`agent.${agent.name}`, `${agent.name}  (${defaultChainLabel(agent, effective)})`, "string", cfg.agentModels[agent.name] ?? "", (v) => {
       const value = String(v ?? "").trim();
       if (value) cfg.agentModels[agent.name] = value;
       else delete cfg.agentModels[agent.name];
-    }),
+    }, withCompletions(() => [...modelItems(), ...roleItems()])),
   );
+  const actionRows: PanelRow[] = [];
+  if (actions?.addRole) {
+    actionRows.push({ key: "action.addRole", label: "+ Add role", kind: "action", value: undefined, set: (p) => actions.addRole!.run(p as never) });
+  }
+  if (actions?.removeRole) {
+    actionRows.push({ key: "action.removeRole", label: "− Remove role", kind: "action", value: undefined, set: (p) => actions.removeRole!.run(p as never) });
+  }
   return [
-    { key: "roles", label: "Model roles (chain, blank = default)", rows: roleRows },
+    { key: "roles", label: "Model roles (chain, blank = default)", rows: [...roleRows, ...actionRows] },
     { key: "agents", label: "Per-agent overrides (blank = inherit)", rows: agentRows },
   ];
 }
@@ -137,6 +196,88 @@ export function cfgToPatch(cfg: RolesPanelCfg): { roles: RolesConfig["roles"]; a
   return { roles, agentModels };
 }
 
+/** Validate a new role name; returns an error message or null when OK. */
+export function validateNewRoleName(name: string, known: readonly string[]): string | null {
+  const trimmed = name.trim();
+  if (!trimmed) return "Role name is empty.";
+  if (!/^[A-Za-z0-9._-]{1,64}$/.test(trimmed)) {
+    return `Invalid role name "${trimmed}" — use letters, digits, dot, dash, underscore (max 64).`;
+  }
+  const clash = known.find((k) => k.toLowerCase() === trimmed.toLowerCase());
+  if (clash) return `Role @${clash} already exists.`;
+  return null;
+}
+
+/** Apply a role removal to the working config. Built-in roles (DEFAULT_ROLES)
+ *  can only be reset to their bundled chain (blank); custom roles are deleted
+ *  (row disappears on rebuild, key dropped by cfgToPatch). Unknown → null. */
+export function removeRoleFromCfg(cfg: RolesPanelCfg, name: string): "reset" | "deleted" | null {
+  const key = Object.keys(cfg.roles).find((k) => k.toLowerCase() === name.trim().toLowerCase());
+  if (!key) return null;
+  if (DEFAULT_ROLES[key] !== undefined) {
+    cfg.roles[key] = "";
+    return "reset";
+  }
+  delete cfg.roles[key];
+  return "deleted";
+}
+
+export interface RoleActionOpts {
+  /** User feedback (defaults to no-op so tests stay quiet). */
+  notify?: (message: string, kind?: "info" | "warning" | "error") => void;
+}
+
+/** "+ Add role" panel action: prompt name → validate → prompt chain → mutate
+ *  the working config (kernel rebuilds rows + marks dirty after the action). */
+export function makeAddRoleAction(cfg: RolesPanelCfg, opts: RoleActionOpts = {}): PanelAction {
+  return {
+    label: "Add role",
+    run: (prompt) => new Promise<void>((resolve) => {
+      prompt("Role name (e.g. writer)", (name) => {
+        const trimmed = (name ?? "").trim();
+        if (!trimmed) return resolve();
+        const err = validateNewRoleName(trimmed, Object.keys(cfg.roles));
+        if (err) {
+          opts.notify?.(err, "warning");
+          return resolve();
+        }
+        prompt("Model chain (comma-separated; @role or * allowed)", (chain) => {
+          const value = (chain ?? "").trim();
+          if (!value) {
+            opts.notify?.("Chain required — role not added.", "warning");
+            return resolve();
+          }
+          cfg.roles[trimmed] = value;
+          resolve();
+        });
+      });
+    }),
+  };
+}
+
+/** "− Remove role" panel action: prompt pick from known roles, then reset
+ *  (built-in) or delete (custom) via removeRoleFromCfg. */
+export function makeRemoveRoleAction(cfg: RolesPanelCfg, opts: RoleActionOpts = {}): PanelAction {
+  return {
+    label: "Remove role",
+    run: (prompt) => new Promise<void>((resolve) => {
+      const names = Object.keys(cfg.roles).sort();
+      if (names.length === 0) {
+        opts.notify?.("No roles to remove.", "warning");
+        return resolve();
+      }
+      prompt(`Remove role (${names.join(", ")})`, (pick) => {
+        if (!pick) return resolve();
+        const result = removeRoleFromCfg(cfg, pick);
+        if (result === "reset") opts.notify?.(`@${pick.trim()} reset to bundled default`, "info");
+        else if (result === "deleted") opts.notify?.(`@${pick.trim()} removed`, "info");
+        else opts.notify?.(`No role named "${pick.trim()}".`, "warning");
+        resolve();
+      });
+    }),
+  };
+}
+
 /** Keep overrides for agents NOT shown in the panel (e.g. overrides for
  *  project-local agents saved globally from another project) so a panel save
  *  doesn't wipe them. Discovered-agent entries always follow the panel. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-subagent",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "In-process subagents for Pi with isolated SDK sessions, parallel and chained delegation, and inspectable threads.",
   "type": "module",
   "license": "MIT",
@@ -62,9 +62,9 @@
     "check": "npm run typecheck && npm test"
   },
   "peerDependencies": {
-    "@earendil-works/pi-coding-agent": ">=0.80.0 <0.85.0",
-    "@earendil-works/pi-ai": ">=0.80.0 <0.85.0",
     "@earendil-works/pi-agent-core": ">=0.80.0 <0.85.0",
+    "@earendil-works/pi-ai": ">=0.80.0 <0.85.0",
+    "@earendil-works/pi-coding-agent": ">=0.80.0 <0.85.0",
     "@earendil-works/pi-tui": ">=0.80.0 <0.85.0",
     "typebox": ">=1.3.0 <2.0.0"
   },
@@ -74,14 +74,14 @@
   "devDependencies": {
     "@earendil-works/pi-agent-core": "^0.84.0",
     "@earendil-works/pi-ai": "^0.84.0",
-    "@earendil-works/pi-coding-agent": "^0.84.0",
-    "@earendil-works/pi-tui": "^0.84.0",
+    "@earendil-works/pi-coding-agent": "^0.84.3",
+    "@earendil-works/pi-tui": "^0.84.3",
     "@types/mocha": "^10.0.10",
     "@types/node": "^20.19.43",
     "mocha": "^11.8.0",
     "tsx": "^4.22.4",
-    "typescript": "^5.9.3",
-    "typebox": "^1.3.1"
+    "typebox": "^1.3.1",
+    "typescript": "^5.9.3"
   },
   "overrides": {
     "serialize-javascript@>=5.0.0 <7.0.5": "^7.0.5",