@bacnh85/pi-subagent 0.16.0 → 0.16.1

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 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/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);
@@ -387,10 +401,17 @@ export default function (pi: ExtensionAPI) {
         }
         const current = readSubagentRolesGlobal();
         const working = buildRolesPanelCfg(discovery.agents, current);
+        const panelOptions = {
+          models: () => {
+            try { return ctx.modelRegistry.getAvailable().map((m) => `${m.provider}/${m.id}`); }
+            catch { return []; }
+          },
+          roles: () => [...Object.keys(DEFAULT_ROLES), ...Object.keys(readSubagentRolesGlobal().roles)],
+        };
         await openConfigPanel({
           ctx,
           cfg: working,
-          build: (cfg) => buildRows(cfg, discovery.agents),
+          build: (cfg) => buildRows(cfg, discovery.agents, panelOptions),
           title: "Subagent model roles",
           onSave: (saved, editedKeys) => {
             if (!(saved && editedKeys && editedKeys.size > 0)) return;

package/extensions/roles-panel.ts

@@ -14,6 +14,15 @@ 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";
+
+// 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";
 
@@ -83,6 +92,14 @@ 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[];
+}
+
 /** 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,21 +115,32 @@ 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[] {
+/** 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). */
+export function buildRows(cfg: RolesPanelCfg, agents: AgentConfig[], options?: RolesPanelOptions): 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 agentRows = agents.map((agent) =>
     row(`agent.${agent.name}`, agent.name, "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()])),
   );
   return [
     { key: "roles", label: "Model roles (chain, blank = default)", rows: roleRows },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-subagent",
-  "version": "0.16.0",
+  "version": "0.16.1",
   "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",