@bacnh85/pi-subagent 0.15.0 → 0.15.2

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## 0.15.2 (2026-08-18)
+
+### Improvements
+
+- Colored borders around thread viewer overlay using agent color.
+- Colored scroll-indicator arrows (↑/↓) matching agent color.
+- Scroll offset only resets when switching to a different thread, not on every refresh.
+
+## 0.15.1 (2026-08-17)
+
+### Improvements
+
+- Project-local agent approval is now a single select — **Allow once / Trust for this session / Deny** — instead of a yes/no confirm repeated on every delegation. "Trust for this session" remembers the project agents dir for the session (cleared on `session_start`); dismissed dialogs and Deny cancel the delegation. Headless sessions still fail closed.
+- New test coverage for the approval gate (Deny / dismissed / headless / trust-remembering / session_start clearing).
+
 ## 0.15.0 (2026-08-09)
 
 ### Packaging

package/README.md

@@ -91,7 +91,7 @@ Threads are session-memory only and are cleared when Pi replaces or reloads the
 Agent files under `.pi/agents/` are controlled by the current repository. A project agent's system prompt may instruct a child to execute shell commands or modify files.
 
 - **Project-agent approval cannot be disabled by the model.** The `confirmProjectAgents` parameter is not exposed in the tool schema. Confirmation policy comes from trusted user configuration only.
-- **Interactive sessions** prompt the user before executing project agents.
+- **Interactive sessions** prompt the user before executing project agents (**Allow once / Trust for this session / Deny**); "Trust for this session" remembers the project agents dir until the session ends. Dismissed dialogs and Deny cancel the delegation.
 - **Headless sessions fail closed.** Project agents are not executed without UI confirmation unless the trusted setting `allowUnconfirmedProjectAgents` is enabled (via `PI_SUBAGENT_ALLOW_UNCONFIRMED_PROJECT_AGENTS=true` environment variable or pi settings).
 - **The extension service path** (`pi-subagent:run` event) follows the same policy.
 

package/agents/planner.md

@@ -3,7 +3,7 @@ name: planner
 description: Read-only planning and architecture specialist. Use for consequential design, tradeoff analysis, and implementation plans.
 tools: read, grep, find, ls
 models:
-  - zai-coding-cn/glm-5.2
+  - zai-coding-cn/glm-5.3
   - openrouter/nvidia/nemotron-3-ultra-550b-a55b:free
   - opencode-go/deepseek-v4-pro
 thinking: high

package/agents/reviewer.md

@@ -3,7 +3,7 @@ name: reviewer
 description: Code review specialist. Use for correctness, security, regression, and meaningful test-gap review.
 tools: read, grep, find, ls
 models:
-  - zai-coding-cn/glm-5.2
+  - zai-coding-cn/glm-5.3
   - openrouter/nvidia/nemotron-3-ultra-550b-a55b:free
   - opencode-go/deepseek-v4-pro
 thinking: high

package/extensions/index.ts

@@ -95,6 +95,9 @@ function getTrustedConfig(ctx: ExtensionContext): { allowUnconfirmedProjectAgent
   };
 }
 
+/** Session-scoped approvals for project-local agents ("Trust for this session"). */
+const trustedProjectAgentDirs = new Set<string>();
+
 
 // ---------------------------------------------------------------------------
 // Tool parameter schema
@@ -186,6 +189,7 @@ export default function (pi: ExtensionAPI) {
     currentCtx = ctx;
     if (event.reason === "reload") invalidateAgentCache();
     threadStore.clear();
+    trustedProjectAgentDirs.clear();
     // Clear any widget from a prior session.
     widget.clearWidgetIfIdle();
     // Mark prior-session running tasks as interrupted (we can't resume them).
@@ -591,19 +595,22 @@ export default function (pi: ExtensionAPI) {
 
         if (projectAgentsRequested.length > 0) {
           if (confirmProjectAgents) {
-            if (ctx.hasUI) {
+            const dir = discovery.projectAgentsDir ?? "(unknown)";
+            if (trustedProjectAgentDirs.has(dir)) {
+              // Previously approved "Trust for this session" for this agents dir.
+            } else if (ctx.hasUI) {
               const names = projectAgentsRequested.map((a) => a.name).join(", ");
-              const dir = discovery.projectAgentsDir ?? "(unknown)";
-              const ok = await ctx.ui.confirm(
-                "Run project-local agents?",
-                `Agents: ${names}\nSource: ${dir}\n\nProject agents are repo-controlled. Only continue for trusted repositories.`,
+              const choice = await ctx.ui.select(
+                `Run project-local agents?\n\nAgents: ${names}\nSource: ${dir}\n\nProject agents are repo-controlled. Only continue for trusted repositories.`,
+                ["Allow once", "Trust for this session", "Deny"],
               );
-              if (!ok) {
+              if (choice !== "Allow once" && choice !== "Trust for this session") {
                 return {
                   content: [{ type: "text", text: "Canceled: project-local agents not approved." }],
                   details: makeDetails(hasChain ? "chain" : hasTasks ? "parallel" : "single")([]),
                 };
               }
+              if (choice === "Trust for this session") trustedProjectAgentDirs.add(dir);
             } else {
               // Fail closed in headless sessions.
               return {

package/extensions/thread-viewer.ts

@@ -35,6 +35,12 @@ export interface ThreadViewerCallbacks {
 
 /** Viewport height for the overlay (estimated lines). Must be > 3. */
 const OVERLAY_HEIGHT = 24;
+const borderSize = 1;
+const truncateFit = function (text: string, width: number): string {
+  text = " " + text;
+  return truncateToWidth(text, width - ((borderSize*2) +1), "...", true) + " ";
+
+  }
 
 export class ThreadViewer {
   private thread: SubagentThread;
@@ -44,13 +50,17 @@ export class ThreadViewer {
   private cachedWidth?: number;
   private cachedUpdatedAt?: number;
   private cachedLines?: string[];
+  private lastThreadId: string | null = null;
 
   constructor(thread: SubagentThread, callbacks: ThreadViewerCallbacks, theme: ViewerTheme) {
     this.thread = thread;
     this.callbacks = callbacks;
     this.theme = theme;
+    this.lastThreadId = thread.id;
+
   }
 
+
   handleInput(data: string): void {
     if (matchesKey(data, Key.escape)) {
       this.callbacks.onClose();
@@ -58,14 +68,12 @@ export class ThreadViewer {
     }
     if (matchesKey(data, Key.alt("left"))) {
       if (this.callbacks.hasPrev) {
-        this.scrollOffset = 0;
         this.callbacks.onPrev();
       }
       return;
     }
     if (matchesKey(data, Key.alt("right"))) {
       if (this.callbacks.hasNext) {
-        this.scrollOffset = 0;
         this.callbacks.onNext();
       }
       return;
@@ -131,19 +139,19 @@ export class ThreadViewer {
       const reasonColor = result.stopReason === "timeout" ? "warning" : "error";
       header += ` ${t.fg(reasonColor, `[${result.stopReason}]`)}`;
     }
-    lines.push(truncateToWidth(header, width));
+    lines.push(truncateFit(header, width));
 
     // Error message
     if (result && isErr && result.errorMessage) {
       const msgColor = result.stopReason === "timeout" ? "warning" : "error";
-      lines.push(truncateToWidth(t.fg(msgColor, `Error: ${result.errorMessage}`), width));
+      lines.push(truncateFit(t.fg(msgColor, `Error: ${result.errorMessage}`), width));
     }
 
     lines.push("");
 
     // Task
-    lines.push(truncateToWidth(t.fg("muted", "─── Task ───"), width));
-    lines.push(truncateToWidth(t.fg("dim", this.thread.task), width));
+    lines.push(truncateFit(t.fg("muted", "─── Task ───"), width));
+    lines.push(truncateFit(t.fg("dim", this.thread.task), width));
     lines.push("");
 
     if (status === "running") {
@@ -152,18 +160,18 @@ export class ThreadViewer {
       const activity = this.thread.lastActivityAt ? `${Math.floor((now - this.thread.lastActivityAt) / 1000)}s ago (${this.thread.lastActivityLabel})` : "none yet";
       const idleMs = this.thread.inactivityDeadline ? this.thread.inactivityDeadline - now : 0;
       const idle = this.thread.inactivityDeadline ? `${Math.max(0, Math.ceil(idleMs / 1000))}s remaining` : "pending";
-      lines.push(truncateToWidth(t.fg(idleMs < 30_000 ? "warning" : "muted", `Elapsed ${elapsed}s · last activity ${activity} · idle ${idle}`), width));
+      lines.push(truncateFit(t.fg(idleMs < 30_000 ? "warning" : "muted", `Elapsed ${elapsed}s · last activity ${activity} · idle ${idle}`), width));
     }
     if (status === "running" && (!result || result.messages.length === 0)) {
-      lines.push(truncateToWidth(t.fg("muted", "(waiting for first message...)"), width));
+      lines.push(truncateFit(t.fg("muted", "(waiting for first message...)"), width));
     } else if (result) {
       const displayItems = getDisplayItems(result.messages);
       const finalOutput = getFinalOutput(result.messages);
 
-      lines.push(truncateToWidth(t.fg("muted", "─── Output ───"), width));
+      lines.push(truncateFit(t.fg("muted", "─── Output ───"), width));
 
       if (displayItems.length === 0 && !finalOutput) {
-        lines.push(truncateToWidth(t.fg("muted", "(no output)"), width));
+        lines.push(truncateFit(t.fg("muted", "(no output)"), width));
       } else {
         const mdTheme = getMarkdownTheme();
 
@@ -171,7 +179,7 @@ export class ThreadViewer {
         for (const item of displayItems) {
           if (item.type === "toolCall") {
             lines.push(
-              truncateToWidth(
+              truncateFit(
                 t.fg("muted", "→ ") + formatToolCall(item.name, item.args, t.fg.bind(t)),
                 width,
               ),
@@ -182,7 +190,7 @@ export class ThreadViewer {
             const md = new Markdown(item.text.trim(), 0, 0, mdTheme);
             const mdLines = md.render(contentWidth);
             for (const mdLine of mdLines) {
-              lines.push(`  ${truncateToWidth(mdLine, contentWidth)}`);
+              lines.push(`  ${truncateFit(mdLine, contentWidth)}`);
             }
           }
         }
@@ -194,7 +202,7 @@ export class ThreadViewer {
           const contentWidth = Math.max(1, width - 2);
           const md = new Markdown(finalOutput.trim(), 0, 0, mdTheme);
           for (const mdLine of md.render(contentWidth)) {
-            lines.push(`  ${truncateToWidth(mdLine, contentWidth)}`);
+            lines.push(`  ${truncateFit(mdLine, contentWidth)}`);
           }
         }
       }
@@ -203,7 +211,7 @@ export class ThreadViewer {
       const usageStr = formatUsageStats(result.usage, result.model);
       if (usageStr) {
         lines.push("");
-        lines.push(truncateToWidth(t.fg("dim", usageStr), width));
+        lines.push(truncateFit(t.fg("dim", usageStr), width));
       }
     }
 
@@ -215,7 +223,7 @@ export class ThreadViewer {
     if (this.callbacks.hasPrev) navParts.push("alt+← prev");
     if (this.callbacks.hasNext) navParts.push("alt+→ next");
     navParts.push("↑↓ scroll");
-    lines.push(truncateToWidth(t.fg("dim", navParts.join(" · ")), width));
+    lines.push(truncateFit(t.fg("dim", navParts.join(" · ")), width));
 
     this.cachedLines = lines;
     this.cachedWidth = width;
@@ -227,6 +235,7 @@ export class ThreadViewer {
   private renderVisible(allLines: string[], width: number): string[] {
     const total = allLines.length;
     const maxVisible = Math.max(3, OVERLAY_HEIGHT);
+    const color = this.thread.color ?? "accent";
 
     // Clamp scrollOffset so the last page shows a full viewport minus one indicator line
     const maxOffset =
@@ -246,21 +255,47 @@ export class ThreadViewer {
 
     // Scroll indicator at top
     if (aboveShown) {
-      visible.unshift(truncateToWidth(
-        this.theme.fg("muted", `↑ ${offset} more lines above`),
+      const abmsg = this.theme.fg(color, `↑ ${offset}`) + this.theme.fg("muted", ` more lines above`);
+      visible.unshift(truncateFit(
+        abmsg,
         width,
       ));
     }
     // Scroll indicator at bottom
     if (belowShown) {
       const remaining = total - offset - bodyHeight;
-      visible.push(truncateToWidth(
-        this.theme.fg("muted", `↓ ${remaining} more lines below`),
+      const remmsg = this.theme.fg(color, `↓ ${remaining}`) + this.theme.fg("muted", ` more lines below`);
+        visible.push(truncateFit(
+        remmsg,
         width,
       ));
     }
 
-    return visible;
+    // Add border around the visible content
+    const borderWidth = width - 2;
+    const borderedLines: string[] = [];
+
+
+    // Top border with colored characters
+    const borderColor = this.theme.fg(color, "─");
+    const cornerColor = this.theme.fg(color, "┌");
+    const bottomCornerColor = this.theme.fg(color, "└");
+    const sideBorder = this.theme.fg(color, "│");
+    borderedLines.push(cornerColor + borderColor.repeat(borderWidth) + this.theme.fg(color, "┐"));
+
+    // Content with side borders
+    for (const line of visible) {
+      let borderedLine = sideBorder + line + sideBorder;
+      if (line.trim() === "") {
+      borderedLine = sideBorder + " ".repeat(Math.max(0, width - ( borderSize * 2))) + sideBorder;
+      }
+      borderedLines.push(borderedLine);
+    }
+
+    // Bottom border with colored characters
+    borderedLines.push(bottomCornerColor + borderColor.repeat(borderWidth) + this.theme.fg(color, "┘"));
+
+    return borderedLines;
   }
 
   invalidate(): void {
@@ -273,7 +308,11 @@ export class ThreadViewer {
   setThread(thread: SubagentThread, callbacks: ThreadViewerCallbacks): void {
     this.thread = thread;
     this.callbacks = callbacks;
-    this.scrollOffset = 0;
+    // Only reset scrollOffset when switching to a different thread
+    if (this.lastThreadId !== thread.id) {
+      this.scrollOffset = 0;
+      this.lastThreadId = thread.id;
+    }
     this.invalidate();
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bacnh85/pi-subagent",
-  "version": "0.15.0",
+  "version": "0.15.2",
   "description": "In-process subagents for Pi with isolated SDK sessions, parallel and chained delegation, and inspectable threads.",
   "type": "module",
   "license": "MIT",