@d3ara1n/pi-subagent 3.3.1 → 3.4.0

package/README.md

@@ -64,7 +64,7 @@ Observations on how main models behave with this plugin, one family per subsecti
 
 | Command | Description |
 |---------|-------------|
-| `/subagent:view` | Open the live view: a tabbed overlay with a per-run activity feed and a brief detail page (inputs, files, stats), plus modal steer input for the focused run — doubles as the session's run archive |
+| `/subagent:view` | Open the live view: a tabbed overlay with a per-run activity feed and a brief detail page (inputs, files, stats), plus modal steer input for the focused run — doubles as the session's run archive. Also available as the native palette entry **Subagent: Activity View** (Ctrl+Shift+P via [`pi-command-palette`](../pi-command-palette)), which opens the view directly without needing an empty editor |
 | `/subagent:doctor` | Diagnose pi invocation, model-role resolution, configuration, and role references |
 | `/subagent:status` | List background runs and their current state |
 | `/subagent:cancel <id\|all> [reason]` | Cancel a live background run (or every live run); the optional reason is recorded with the run |
@@ -82,6 +82,7 @@ Steer input is modal so keys never conflict with typing: in browse mode `s` open
 ## Dependencies
 
 - [`@d3ara1n/pi-model-roles`](../pi-model-roles) — model role resolution
+- [`@d3ara1n/pi-command-palette-core`](../pi-command-palette-core) — native command-palette registry (pure npm library, installed automatically; the palette entry appears when [`pi-command-palette`](../pi-command-palette) is installed)
 
 ## Installation
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@d3ara1n/pi-subagent",
-  "version": "3.3.1",
+  "version": "3.4.0",
   "type": "module",
   "description": "Role-based subagent orchestration for pi — delegates tasks to specialized pi child processes with configurable model roles",
   "main": "src/index.ts",
@@ -27,6 +27,7 @@
     }
   },
   "dependencies": {
+    "@d3ara1n/pi-command-palette-core": "*",
     "@d3ara1n/pi-model-roles": "*"
   },
   "pi": {

package/src/activity.test.ts

@@ -42,11 +42,10 @@ test("queued items share the visible limit and consumption changes only the mark
   assert.equal(buildDisplayItems(log).length, 7);
 });
 
-test("queued rendering uses status for every display type and steers retain accent", () => {
+test("queued rendering uses status for every display type and steer color follows delivery status", () => {
   const color = (name: string, text: string) => `<${name}>${text}</${name}>`;
-  for (const status of ["queued", "done"] as const) {
-    assert.match(formatDisplayItem({ type: "steer", status, text: "input" }, color), /^<accent>/);
-  }
+  assert.match(formatDisplayItem({ type: "steer", status: "queued", text: "input" }, color), /^<accent>/);
+  assert.match(formatDisplayItem({ type: "steer", status: "done", text: "input" }, color), /^<dim>/);
   assert.equal(formatDisplayItem({ type: "steer", status: "done", text: "queued" }, plain), "\u21a9 steer: queued");
   assert.match(formatDisplayItem({ type: "thinking", status: "queued" }, plain), /\(queued\)/);
   assert.match(formatDisplayItem({ type: "toolCall", name: "read", args: {}, status: "queued" }, plain), /\(queued\)/);

package/src/index.ts

@@ -13,9 +13,10 @@
  * - Accurate, concise output for the main model
  */
 
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { getModelRolesAPI } from "@d3ara1n/pi-model-roles";
+import { paletteCommandRegistry } from "@d3ara1n/pi-command-palette-core";
 import type { SubagentConfig, SubagentResult, SubagentRole } from "./types.ts";
 import { DEFAULT_CONFIG } from "./types.ts";
 import { loadSubagentConfig } from "./config.ts";
@@ -713,8 +714,7 @@ export default function subagentExtension(pi: ExtensionAPI) {
     parameters: Type.Object({
       id: Type.String({ description: "Run id returned by a background delegate call" }),
       message: Type.String({
-        description:
-          "The concrete correction or updated requirement for the child to follow.",
+        description: "The concrete correction or updated requirement for the child to follow.",
       }),
     }),
 
@@ -809,38 +809,50 @@ export default function subagentExtension(pi: ExtensionAPI) {
     renderResult: renderCancelResult,
   });
 
-  pi.registerCommand("subagent:view", {
-    description: "Open the subagent activity view (watch runs, steer, browse the session's archive)",
-    handler: async (_args, ctx) => {
-      // Union of every known run: the background registry — append-only for
-      // the whole session, the view doubles as the run archive and derives
-      // nothing from delivery state — plus live in-flight runs (foreground
-      // delegate calls included, visible only while in flight). Dedupe by
-      // id — background runs appear in both.
-      const runsProvider = () => {
-        const seen = new Set<string>();
-        const out: RunHandle[] = [];
-        for (const r of [...backgroundRuns.values(), ...liveRuns]) {
-          if (!seen.has(r.id)) {
-            seen.add(r.id);
-            out.push(r);
-          }
+  // ── Subagent activity view ─────────────────────────────────────────
+  // Shared by the /subagent:view command and the native command-palette
+  // entry — both open the same overlay.
+  //
+  // Union of every known run: the background registry — append-only for
+  // the whole session, the view doubles as the run archive and derives
+  // nothing from delivery state — plus live in-flight runs (foreground
+  // delegate calls included, visible only while in flight). Dedupe by
+  // id — background runs appear in both.
+  async function openSubagentView(ctx: ExtensionContext): Promise<void> {
+    const runsProvider = () => {
+      const seen = new Set<string>();
+      const out: RunHandle[] = [];
+      for (const r of [...backgroundRuns.values(), ...liveRuns]) {
+        if (!seen.has(r.id)) {
+          seen.add(r.id);
+          out.push(r);
         }
-        return out;
-      };
-      if (runsProvider().length === 0) {
-        ctx.ui.notify("No subagent runs yet.", "info");
-        return;
       }
-      await ctx.ui.custom(
-        (tui, theme, _keybindings, done) =>
-          createViewPanel(runsProvider, tui, theme, () => done(undefined)),
-        {
-          overlay: true,
-          overlayOptions: { anchor: "center", width: "90%", maxHeight: "85%" },
-        },
-      );
-    },
+      return out;
+    };
+    await ctx.ui.custom(
+      (tui, theme, _keybindings, done) =>
+        createViewPanel(runsProvider, tui, theme, () => done(undefined)),
+      {
+        overlay: true,
+        overlayOptions: { anchor: "center", width: "90%", maxHeight: "85%" },
+      },
+    );
+  }
+
+  pi.registerCommand("subagent:view", {
+    description:
+      "Open the subagent activity view (watch runs, steer, browse the session's archive)",
+    handler: (_args, ctx) => openSubagentView(ctx),
+  });
+
+  // Native command-palette entry — opens the view directly, no editor
+  // round-trip, so it works mid-draft.
+  paletteCommandRegistry.register({
+    id: "subagent:view",
+    label: "Subagent: Activity View",
+    description: "Watch subagent runs, steer, browse the session's archive",
+    run: (_pi, ctx) => openSubagentView(ctx),
   });
 
   pi.registerCommand("subagent:doctor", {

package/src/utils.ts

@@ -288,7 +288,10 @@ export function formatDisplayItem(
   const queued = item.status === "queued";
   const marker = queued ? " (queued)" : "";
   if (item.type === "steer") {
-    return fg("accent", `\u21a9 steer${marker}: ${oneLine(item.text)}`);
+    // Color follows delivery status like every other entry (queued = accent,
+    // consumed = dim); the ↩ glyph alone carries the external-origin marker.
+    const steerColor = queued ? "accent" : "dim";
+    return fg(steerColor, `\u21a9 steer${marker}: ${oneLine(item.text)}`);
   }
   if (queued) {
     const body = item.type === "thinking" ? "thinking" : formatToolCall(item.name, item.args, (_c, text) => text);