@nklisch/pi-enhanced 0.2.0 → 0.2.2

package/node_modules/@nklisch/pi-plugins/node_modules/@nklisch/pi-subagents/src/ui/session-navigator.ts

@@ -38,6 +38,7 @@ import {
   visibleWidth,
 } from "@earendil-works/pi-tui";
 import type { AgentConfigLookup } from "#src/config/agent-types";
+import { debugLog } from "#src/debug";
 import type { SessionMessage } from "#src/types";
 import { describeActivity, formatDuration, formatModelThinking, type Theme } from "#src/ui/display";
 import { fileSnapshotSource, listNavigableAgents, liveSource, type NavigableSubagent, type RunDisplayMetadata, type TranscriptSource } from "#src/ui/session-navigation";
@@ -160,57 +161,69 @@ export class TranscriptOverlay implements Component {
     this.done = done;
     this.cwd = cwd;
     this.markdownTheme = markdownTheme;
-    this.content = this.rebuild();
-    this.unsubscribe = source.subscribe(() => {
-      if (this.closed) return;
+    try {
       this.content = this.rebuild();
-      this.tui.requestRender();
-    });
-    if (run.completedAt() === undefined) {
-      this.runtimeInterval = setInterval(() => {
-        if (this.closed || this.run.completedAt() !== undefined) {
-          this.clearRuntimeInterval();
-          return;
-        }
-        this.tui.requestRender();
-      }, 100);
+    } catch (error) {
+      debugLog("session navigator initial render", error);
+      this.content = new Container();
+    }
+    try {
+      this.unsubscribe = source.subscribe(() => this.refreshFromSource());
+    } catch (error) {
+      debugLog("session navigator subscribe", error);
+    }
+    if (this.isRunning()) {
+      this.runtimeInterval = setInterval(() => this.refreshRuntime(), 100);
     }
   }
 
   handleInput(data: string): void {
-    if (matchesKey(data, "escape") || matchesKey(data, "q")) {
-      this.closed = true;
-      this.clearRuntimeInterval();
-      this.done(undefined);
-      return;
-    }
+    try {
+      if (matchesKey(data, "escape") || matchesKey(data, "q")) {
+        this.closed = true;
+        this.clearRuntimeInterval();
+        this.done(undefined);
+        return;
+      }
 
-    const totalLines = this.buildContentLines(this.innerWidth()).length;
-    const viewportHeight = this.viewportHeight();
-    const maxScroll = Math.max(0, totalLines - viewportHeight);
-
-    if (matchesKey(data, "up") || matchesKey(data, "k")) {
-      this.scrollOffset = Math.max(0, this.scrollOffset - 1);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "down") || matchesKey(data, "j")) {
-      this.scrollOffset = Math.min(maxScroll, this.scrollOffset + 1);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "pageUp") || matchesKey(data, "shift+up")) {
-      this.scrollOffset = Math.max(0, this.scrollOffset - viewportHeight);
-      this.autoScroll = false;
-    } else if (matchesKey(data, "pageDown") || matchesKey(data, "shift+down")) {
-      this.scrollOffset = Math.min(maxScroll, this.scrollOffset + viewportHeight);
-      this.autoScroll = this.scrollOffset >= maxScroll;
-    } else if (matchesKey(data, "home")) {
-      this.scrollOffset = 0;
-      this.autoScroll = false;
-    } else if (matchesKey(data, "end")) {
-      this.scrollOffset = maxScroll;
-      this.autoScroll = true;
+      const totalLines = this.buildContentLines(this.innerWidth()).length;
+      const viewportHeight = this.viewportHeight();
+      const maxScroll = Math.max(0, totalLines - viewportHeight);
+
+      if (matchesKey(data, "up") || matchesKey(data, "k")) {
+        this.scrollOffset = Math.max(0, this.scrollOffset - 1);
+        this.autoScroll = this.scrollOffset >= maxScroll;
+      } else if (matchesKey(data, "down") || matchesKey(data, "j")) {
+        this.scrollOffset = Math.min(maxScroll, this.scrollOffset + 1);
+        this.autoScroll = this.scrollOffset >= maxScroll;
+      } else if (matchesKey(data, "pageUp") || matchesKey(data, "shift+up")) {
+        this.scrollOffset = Math.max(0, this.scrollOffset - viewportHeight);
+        this.autoScroll = false;
+      } else if (matchesKey(data, "pageDown") || matchesKey(data, "shift+down")) {
+        this.scrollOffset = Math.min(maxScroll, this.scrollOffset + viewportHeight);
+        this.autoScroll = this.scrollOffset >= maxScroll;
+      } else if (matchesKey(data, "home")) {
+        this.scrollOffset = 0;
+        this.autoScroll = false;
+      } else if (matchesKey(data, "end")) {
+        this.scrollOffset = maxScroll;
+        this.autoScroll = true;
+      }
+    } catch (error) {
+      debugLog("session navigator input", error);
     }
   }
 
   render(width: number): string[] {
+    try {
+      return this.renderSafe(width);
+    } catch (error) {
+      debugLog("session navigator render", error);
+      return [];
+    }
+  }
+
+  private renderSafe(width: number): string[] {
     if (width < 6) return [];
     const th = this.theme;
     const innerW = width - 4;
@@ -252,22 +265,63 @@ export class TranscriptOverlay implements Component {
 
   // fallow-ignore-next-line unused-class-member
   invalidate(): void {
-    this.content.invalidate();
+    try {
+      this.content.invalidate();
+    } catch (error) {
+      debugLog("session navigator invalidate", error);
+    }
   }
 
   dispose(): void {
     this.closed = true;
-    if (this.unsubscribe) {
-      this.unsubscribe();
-      this.unsubscribe = undefined;
+    const unsubscribe = this.unsubscribe;
+    this.unsubscribe = undefined;
+    if (unsubscribe) {
+      try {
+        unsubscribe();
+      } catch (error) {
+        debugLog("session navigator unsubscribe", error);
+      }
     }
     this.clearRuntimeInterval();
   }
 
   // ---- Private ----
 
+  private refreshFromSource(): void {
+    if (this.closed) return;
+    try {
+      this.content = this.rebuild();
+      this.tui.requestRender();
+    } catch (error) {
+      debugLog("session navigator live refresh", error);
+    }
+  }
+
+  private refreshRuntime(): void {
+    try {
+      if (this.closed || !this.isRunning()) {
+        this.clearRuntimeInterval();
+        return;
+      }
+      this.tui.requestRender();
+    } catch (error) {
+      debugLog("session navigator runtime refresh", error);
+      this.clearRuntimeInterval();
+    }
+  }
+
+  private isRunning(): boolean {
+    try {
+      return this.run.completedAt() === undefined;
+    } catch (error) {
+      debugLog("session navigator runtime state", error);
+      return false;
+    }
+  }
+
   private clearRuntimeInterval(): void {
-    if (this.runtimeInterval) {
+    if (this.runtimeInterval !== undefined) {
       clearInterval(this.runtimeInterval);
       this.runtimeInterval = undefined;
     }

package/node_modules/@nklisch/pi-plugins/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nklisch/pi-plugins",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "private": false,
   "description": "Native marketplace and plugin lifecycle management for Pi, with Claude Code and Codex compatibility.",
   "license": "MIT",
@@ -67,8 +67,8 @@
     "@nklisch/pi-subagents"
   ],
   "dependencies": {
-    "@nklisch/pi-mcp-adapter": "2.21.0-nklisch.0",
-    "@nklisch/pi-subagents": "18.1.0-nklisch.1",
+    "@nklisch/pi-mcp-adapter": "2.21.0-nklisch.1",
+    "@nklisch/pi-subagents": "18.1.0-nklisch.2",
     "jiti": "2.7.0",
     "semver": "7.8.1",
     "smol-toml": "^1.7.0",

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@nklisch/pi-enhanced",
-  "version": "0.2.0",
-  "description": "Pi, enhanced \u2014 one install for nklisch's full harness: policy-gated command review, plugin marketplace, subagents, background tasks, research tools, search, model modes, and a curated UX set.",
+  "version": "0.2.2",
+  "description": "Pi, enhanced — one install for nklisch's full harness: policy-gated command review, plugin marketplace, subagents, background tasks, research tools, search, model modes, and a curated UX set.",
   "author": {
     "name": "nklisch"
   },
@@ -29,7 +29,7 @@
   "pi": {
     "extensions": [
       "./node_modules/@nklisch/pi-conveniences/extensions",
-      "./node_modules/@nklisch/pi-fff-compat/extensions",
+      "./node_modules/@nklisch/pi-fff-compat/extensions/fff-compat-search.ts",
       "./node_modules/@nklisch/pi-model-modes/extensions/index.ts",
       "./node_modules/@nklisch/pi-background-tasks/extensions/background-tasks.ts",
       "./node_modules/@nklisch/pi-plugins/dist/pi/production-subagents-extension.js",
@@ -70,7 +70,7 @@
     "@nklisch/pi-clearance": "^0",
     "@nklisch/pi-conveniences": "^0",
     "@nklisch/pi-fff-compat": "^0",
-    "@nklisch/pi-mcp-adapter": "2.21.0-nklisch.0",
+    "@nklisch/pi-mcp-adapter": "2.21.0-nklisch.1",
     "@nklisch/pi-model-modes": "^0",
     "@nklisch/pi-plugins": "^0",
     "@sinclair/typebox": "^0.34.49",

package/test/verify-bundle.mjs

@@ -34,6 +34,14 @@ const declared = [
 ].map((p) => p.replace(/^\.\//, ""));
 
 const failures = [];
+const fffCompatEntrypoint = "./node_modules/@nklisch/pi-fff-compat/extensions/fff-compat-search.ts";
+if (!manifest.pi.extensions.includes(fffCompatEntrypoint)) {
+  failures.push(`${fffCompatEntrypoint} (FFF compatibility extension entrypoint not declared)`);
+}
+if (manifest.pi.extensions.includes("./node_modules/@nklisch/pi-fff-compat/extensions")) {
+  failures.push("node_modules/@nklisch/pi-fff-compat/extensions (directory would load non-extension helpers)");
+}
+
 for (const declaredPath of declared) {
   // A declared path may be a directory, a single file, or a SKILL.md path.
   // It is satisfied when the tarball contains the path itself or anything