@nklisch/pi-enhanced 0.1.11 → 0.2.1

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.3.9",
+  "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.20.1-nklisch.1",
-    "@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,6 +1,6 @@
 {
   "name": "@nklisch/pi-enhanced",
-  "version": "0.1.11",
+  "version": "0.2.1",
   "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"
@@ -70,7 +70,7 @@
     "@nklisch/pi-clearance": "^0",
     "@nklisch/pi-conveniences": "^0",
     "@nklisch/pi-fff-compat": "^0",
-    "@nklisch/pi-mcp-adapter": "2.20.1-nklisch.1",
+    "@nklisch/pi-mcp-adapter": "2.21.0-nklisch.1",
     "@nklisch/pi-model-modes": "^0",
     "@nklisch/pi-plugins": "^0",
     "@sinclair/typebox": "^0.34.49",

package/node_modules/@nklisch/pi-plugins/dist/application/generation-mutation-coordinator.d.ts

@@ -1,79 +0,0 @@
-import { type PluginKey } from "../domain/identity.js";
-import { type Generation } from "../domain/state/config-state.js";
-import { type ScopeContext } from "../domain/state/scope.js";
-import { type GenerationSnapshot, type VerifiedStateMutation } from "./state-contract.js";
-import type { LifecycleStateStore } from "./ports/lifecycle-state-store.js";
-import type { ScopeLockManager } from "./ports/scope-lock.js";
-import type { KeyedMutationScheduler } from "./mutation-coordination.js";
-export type PreparedMutationRequest = Readonly<{
-    scope: ScopeContext;
-    plugins: readonly PluginKey[];
-    expectedGeneration: Generation;
-}>;
-export type PreparedMutationContext = Readonly<{
-    snapshot: GenerationSnapshot;
-    assertOwned(): Promise<void>;
-}>;
-export type PreparedMutation<T> = Readonly<{
-    mutation: VerifiedStateMutation;
-    value: T;
-    /** Last authority check while the scope lease is still owned, immediately before durable commit. */
-    beforeCommit?: () => Promise<void>;
-}>;
-export type GenerationMutationResult<T> = Readonly<{
-    kind: "committed";
-    value: T;
-    snapshot: GenerationSnapshot;
-}> | Readonly<{
-    kind: "stale-generation";
-    expected: Generation;
-    actual: Generation;
-}> | Readonly<{
-    kind: "commit-failed";
-    value: T;
-    expected: Generation;
-    actual: Generation;
-}> | Readonly<{
-    kind: "commit-ambiguous";
-    value: T;
-    expected: Generation;
-    actual?: Generation;
-}>;
-/** Retains both failures when a mutation and its lock cleanup fail together. */
-export declare class MutationCleanupError extends Error {
-    readonly operationError: unknown;
-    readonly cleanupError: unknown;
-    readonly causes: readonly [unknown, unknown];
-    readonly outcome: GenerationMutationResult<unknown> | undefined;
-    readonly observedSnapshot: GenerationSnapshot | undefined;
-    constructor(operationError: unknown, cleanupError: unknown, evidence?: Readonly<{
-        outcome?: GenerationMutationResult<unknown>;
-        observedSnapshot?: GenerationSnapshot;
-    }>);
-}
-/**
- * A committed state result is durable even if the lease cannot be cleaned up.
- * Callers receive the evidence needed to inspect/recover instead of blindly
- * replaying a mutation whose promotion may already be visible.
- */
-export declare class CommittedMutationCleanupError<T> extends Error {
-    readonly committed: Readonly<{
-        value: T;
-        snapshot: GenerationSnapshot;
-    }>;
-    readonly cleanupError: unknown;
-    readonly cause: unknown;
-    constructor(committed: Readonly<{
-        value: T;
-        snapshot: GenerationSnapshot;
-    }>, cleanupError: unknown);
-}
-export interface GenerationMutationCoordinator {
-    runPreparedMutation<T>(request: PreparedMutationRequest, prepareCommit: (context: PreparedMutationContext) => Promise<PreparedMutation<T>>, signal: AbortSignal): Promise<GenerationMutationResult<T>>;
-}
-export type GenerationMutationCoordinatorDependencies = Readonly<{
-    scheduler: KeyedMutationScheduler;
-    locks: ScopeLockManager;
-    state: LifecycleStateStore;
-}>;
-export declare function createGenerationMutationCoordinator(dependencies: GenerationMutationCoordinatorDependencies): GenerationMutationCoordinator;