@esso0428/pi-subagents 0.17.24 → 0.17.26

package/CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.17.26] - 2026-10-02
+
+### Fixed
+- **Drew the last agents roster row as `└─` when the roster overflows**: the branch fixup only ran on the path where every row fits, so a clipped or scrolled roster kept `├─` on its final visible entry. The bounded viewport also pads the body with blank lines, so the row is not at the end of the rendered output; both render paths now share a `markLastBranch()` helper that resolves the final visible row explicitly and also blanks the `│` continuation under it.
+
+## [0.17.25] - 2026-10-02
+
+### Fixed
+- **Suspended the roster key listener for the remaining widget-owned dialogs**: v0.17.24 routed `ctx.ui.select`, `ctx.ui.editor`, and `ctx.ui.custom` through `withModal()` but left `ctx.ui.input` and `ctx.ui.confirm` outside it. Those dialogs are reached only after a wrapped modal has already cleared roster navigation, so the listener could still swallow a stray `↓`, and any future path that opens one directly would lose `escape`/`↑`/`↓`/`enter` entirely. Every remaining modal call now runs inside `withModal()` as well.
+
 ## [0.17.24] - 2026-10-02
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@esso0428/pi-subagents",
-  "version": "0.17.24",
+  "version": "0.17.26",
   "description": "A pi extension that brings smart Claude Code-style autonomous sub-agents to pi, with npm:pi-subagents-style JSON agent overrides.",
   "author": "ESSO0428",
   "repository": {

package/src/index.ts

@@ -2158,7 +2158,7 @@ Terse command-style prompts produce shallow, generic work.
       }
     } else if (choice === "Delete") {
       if (file) {
-        const confirmed = await ctx.ui.confirm("Delete agent", `Delete ${name} from ${file.location} (${file.path})?`);
+        const confirmed = await withModal(() => ctx.ui.confirm("Delete agent", `Delete ${name} from ${file.location} (${file.path})?`));
         if (confirmed) {
           unlinkSync(file.path);
           reloadCustomAgents();
@@ -2166,7 +2166,7 @@ Terse command-style prompts produce shallow, generic work.
         }
       }
     } else if (choice === "Reset to default" && file) {
-      const confirmed = await ctx.ui.confirm("Reset to default", `Delete override ${file.path} and restore embedded default?`);
+      const confirmed = await withModal(() => ctx.ui.confirm("Reset to default", `Delete override ${file.path} and restore embedded default?`));
       if (confirmed) {
         unlinkSync(file.path);
         reloadCustomAgents();
@@ -2194,7 +2194,7 @@ Terse command-style prompts produce shallow, generic work.
 
     const targetPath = join(targetDir, `${name}.md`);
     if (existsSync(targetPath)) {
-      const overwrite = await ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`);
+      const overwrite = await withModal(() => ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`));
       if (!overwrite) return;
     }
 
@@ -2307,17 +2307,17 @@ Terse command-style prompts produce shallow, generic work.
   }
 
   async function showGenerateWizard(ctx: ExtensionCommandContext, targetDir: string) {
-    const description = await ctx.ui.input("Describe what this agent should do");
+    const description = await withModal(() => ctx.ui.input("Describe what this agent should do"));
     if (!description) return;
 
-    const name = await ctx.ui.input("Agent name (filename, no spaces)");
+    const name = await withModal(() => ctx.ui.input("Agent name (filename, no spaces)"));
     if (!name) return;
 
     mkdirSync(targetDir, { recursive: true });
 
     const targetPath = join(targetDir, `${name}.md`);
     if (existsSync(targetPath)) {
-      const overwrite = await ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`);
+      const overwrite = await withModal(() => ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`));
       if (!overwrite) return;
     }
 
@@ -2384,11 +2384,11 @@ Write the file using the write tool. Only write the file, nothing else.`;
 
   async function showManualWizard(ctx: ExtensionCommandContext, targetDir: string) {
     // 1. Name
-    const name = await ctx.ui.input("Agent name (filename, no spaces)");
+    const name = await withModal(() => ctx.ui.input("Agent name (filename, no spaces)"));
     if (!name) return;
 
     // 2. Description
-    const description = await ctx.ui.input("Description (one line)");
+    const description = await withModal(() => ctx.ui.input("Description (one line)"));
     if (!description) return;
 
     // 3. Tools
@@ -2403,7 +2403,7 @@ Write the file using the write tool. Only write the file, nothing else.`;
     } else if (toolChoice.startsWith("read-only")) {
       tools = "read, bash, grep, find, ls";
     } else {
-      const customTools = await ctx.ui.input("Tools (comma-separated)", BUILTIN_TOOL_NAMES.join(", "));
+      const customTools = await withModal(() => ctx.ui.input("Tools (comma-separated)", BUILTIN_TOOL_NAMES.join(", ")));
       if (!customTools) return;
       tools = customTools;
     }
@@ -2423,7 +2423,7 @@ Write the file using the write tool. Only write the file, nothing else.`;
     else if (modelChoice === "sonnet") modelLine = "\nmodel: anthropic/claude-sonnet-4-6";
     else if (modelChoice === "opus") modelLine = "\nmodel: anthropic/claude-opus-4-6";
     else if (modelChoice === "custom...") {
-      const customModel = await ctx.ui.input("Model (provider/modelId)");
+      const customModel = await withModal(() => ctx.ui.input("Model (provider/modelId)"));
       if (customModel) modelLine = `\nmodel: ${customModel}`;
     }
 
@@ -2453,7 +2453,7 @@ ${systemPrompt}
     const targetPath = join(targetDir, `${name}.md`);
 
     if (existsSync(targetPath)) {
-      const overwrite = await ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`);
+      const overwrite = await withModal(() => ctx.ui.confirm("Overwrite", `${targetPath} already exists. Overwrite?`));
       if (!overwrite) return;
     }
 
@@ -2680,7 +2680,7 @@ ${systemPrompt}
 
       // Loop until user enters a valid integer or cancels (Esc / null).
       // Silently trims whitespace; rejects non-numeric input by re-prompting.
-      let input: string | undefined = await ctx.ui.input(label, current);
+      let input: string | undefined = await withModal(() => ctx.ui.input(label, current));
       while (input != null) {
         const trimmed = input.trim();
         const n = Number(trimmed);
@@ -2690,7 +2690,7 @@ ${systemPrompt}
           return;
         }
         // Invalid — re-prompt with the user's last entry so they can edit it
-        input = await ctx.ui.input(label, trimmed);
+        input = await withModal(() => ctx.ui.input(label, trimmed));
       }
     }
   }

package/src/ui/agent-widget.ts

@@ -238,6 +238,21 @@ export function describeActivity(activeTools: Map<string, string>, responseText?
   return "thinking…";
 }
 
+/**
+ * Mark the final visible roster row as the tree's last branch.
+ *
+ * `bodyIndex` is the offset of that row's header line inside the assembled body,
+ * which is not simply `body.length - 1`: the bounded viewport pads the body
+ * with blank lines, and two-line rows carry a `│` continuation underneath.
+ */
+function markLastBranch(body: string[], bodyIndex: number): void {
+  const header = body[bodyIndex];
+  if (header === undefined) return;
+  body[bodyIndex] = header.replace("├─", "└─");
+  const continuation = body[bodyIndex + 1];
+  if (continuation?.includes("│  ")) body[bodyIndex + 1] = continuation.replace("│  ", "   ");
+}
+
 // ---- Widget manager ----
 
 export class AgentWidget {
@@ -631,11 +646,7 @@ export class AgentWidget {
       for (const row of rows) lines.push(...row.lines);
       if (rows.length > 0) {
         const lastRow = rows[rows.length - 1];
-        const lastStart = lines.length - lastRow.lines.length;
-        lines[lastStart] = lines[lastStart].replace("├─", "└─");
-        if (lastRow.lines.length === 2) {
-          lines[lastStart + 1] = lines[lastStart + 1].replace("│  ", "   ");
-        }
+        markLastBranch(lines, lines.length - lastRow.lines.length);
       }
     } else {
       // Reserve one line for a directional overflow summary. The viewport is
@@ -675,8 +686,10 @@ export class AgentWidget {
 
       let used = 0;
       let end = start;
+      let lastRowLine = -1;
       const visibleBody: string[] = [];
       while (end < rows.length && used + heightAt(end) <= contentBudget) {
+        lastRowLine = visibleBody.length;
         visibleBody.push(...rows[end].lines);
         used += heightAt(end);
         end++;
@@ -687,6 +700,7 @@ export class AgentWidget {
         visibleBody.push(rows[start].lines[0]);
         end = start + 1;
       }
+      markLastBranch(visibleBody, lastRowLine);
 
       const hiddenBefore = start;
       const hiddenAfter = Math.max(0, rows.length - end);