pi-better-harness 0.3.4 → 0.3.5

package/node_modules/pi-better-background-tasks/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-background-tasks",
-  "version": "0.2.10",
+  "version": "0.2.11",
   "description": "Pi extension for durable background shell tasks, watchers, logs, and status inspection.",
   "license": "MIT",
   "type": "module",

package/node_modules/pi-better-background-tasks/src/shared-navigator.ts

@@ -97,7 +97,6 @@ type NavigatorState = {
 };
 
 const GLOBAL_KEY = Symbol.for("pi-better-harness.navigator.state");
-const PLAN_NAVIGATION_KEY = Symbol.for("pi-better-harness.plan-navigation.state");
 const FACTORY_MARK = "__piBetterHarnessNavigatorFactory";
 const FACTORY_REFRESH = "__piBetterHarnessNavigatorRefresh";
 
@@ -121,23 +120,6 @@ function state(): NavigatorState {
   return g[GLOBAL_KEY]!;
 }
 
-type PlanNavigationState = {
-  visible: boolean;
-  progressLabel?: string;
-  releaseWorkFocus?: () => void;
-  refreshNavigationHint?: () => void;
-};
-
-function planNavigationState(): PlanNavigationState {
-  const global = globalThis as typeof globalThis & { [PLAN_NAVIGATION_KEY]?: PlanNavigationState };
-  if (!global[PLAN_NAVIGATION_KEY]) global[PLAN_NAVIGATION_KEY] = { visible: false };
-  return global[PLAN_NAVIGATION_KEY]!;
-}
-
-function refreshPlanNavigationHint(): void {
-  refreshBackgroundWorkNavigator();
-}
-
 export function registerBackgroundWorkProvider(provider: BackgroundWorkProvider): () => void {
   const s = state();
   const previousUnsub = s.unsubscribers.get(provider.id);
@@ -179,12 +161,7 @@ export function isNavigatorUiAvailable(ctx: ExtensionContext | undefined): boole
 }
 
 export function navigatorFooterHint(count: number): string | null {
-  const work = count > 0 ? `← work · ${count}` : null;
-  const navigation = planNavigationState();
-  const plan = navigation.visible
-    ? `→ plan${navigation.progressLabel ? ` · ${navigation.progressLabel}` : ""}`
-    : null;
-  return [work, plan].filter((hint): hint is string => hint !== null).join("    ") || null;
+  return count > 0 ? `← work · ${count}` : null;
 }
 
 export function applyNavigatorFooter(ui: { setStatus(key: string, value: string | undefined): void }, count: number): string | null {
@@ -208,10 +185,6 @@ export function ensureBackgroundWorkNavigator(ctx: ExtensionContext, deps: HostD
   s.mainListRequestRender = undefined;
   s.mainListDeadlineScheduler?.dispose();
   s.mainListDeadlineScheduler = createRenderScheduler(() => refreshMainListWidget());
-  const planNavigation = planNavigationState();
-  planNavigation.releaseWorkFocus = unfocusMainList;
-  planNavigation.refreshNavigationHint = refreshPlanNavigationHint;
-  try { (ctx.ui as any).setStatus?.("pi-better-plan-nav", undefined); } catch { /* ignore */ }
   installNavigatorEditor(ctx.ui as any, deps);
   s.lastHint = undefined;
   refreshBackgroundWorkNavigator(ctx);
@@ -240,11 +213,6 @@ export function disposeBackgroundWorkNavigator(ctx?: ExtensionContext): void {
   s.mainListRequestRender = undefined;
   s.editorComponent = undefined;
   s.detailOverlayRows = undefined;
-  const planNavigation = planNavigationState();
-  planNavigation.releaseWorkFocus = undefined;
-  if (planNavigation.refreshNavigationHint === refreshPlanNavigationHint) {
-    planNavigation.refreshNavigationHint = undefined;
-  }
   s.mainListSelectedId = undefined;
   s.mainListFocused = false;
 }
@@ -492,10 +460,9 @@ function buildMainListLines(
 }
 
 function shortcutsLine(focused: boolean, fg: (color: string, value: string) => string): string {
-  const plan = planNavigationState().visible ? " · → plan" : "";
   const keys = focused
-    ? `↑↓ switch · Enter detail · x stop${plan} · Esc unfocus`
-    : `← work navigator${plan}`;
+    ? "↑↓ switch · Enter detail · x stop · Esc unfocus"
+    : "← work navigator";
   return dim(keys, fg);
 }
 

package/node_modules/pi-better-plan/README.md

@@ -6,7 +6,6 @@
 
 - Gives models `update_plan` and `get_plan` tools for atomic, explicit progress updates.
 - Shows the complete checklist of completed, active, pending, and blocked steps above the editor.
-- Uses `→` from an empty editor to focus the plan; normal cursor movement is preserved when the editor contains text.
 - Persists plan state and display preferences on the active Pi session branch.
 - Opens the complete plan with `/plan`.
 

package/node_modules/pi-better-plan/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-plan",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Structured execution plans with persistent progress for Pi.",
   "license": "MIT",
   "type": "module",

package/node_modules/pi-better-plan/src/index.ts

@@ -1,6 +1,6 @@
 import { StringEnum } from "@earendil-works/pi-ai";
-import { CustomEditor, type ExtensionAPI, type ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { Key, matchesKey, Text } from "@earendil-works/pi-tui";
+import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
+import { Text } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
 import {
@@ -36,74 +36,32 @@ const UpdatePlanSchema = Type.Object({
   plan: Type.Array(PlanStepSchema, { minItems: 1, maxItems: 50 }),
 });
 
-const PLAN_NAV_STATUS_KEY = "pi-better-plan-nav";
-const PLAN_EDITOR_FACTORY_MARK = "__piBetterPlanFactory";
-const PLAN_EDITOR_FACTORY_REFRESH = "__piBetterPlanRefresh";
-const PLAN_NAVIGATION_KEY = Symbol.for("pi-better-harness.plan-navigation.state");
-
-interface SharedPlanNavigationState {
-  visible: boolean;
-  progressLabel?: string;
-  releaseWorkFocus?: () => void;
-  refreshNavigationHint?: () => void;
-}
-
-function sharedPlanNavigationState(): SharedPlanNavigationState {
-  const global = globalThis as typeof globalThis & { [PLAN_NAVIGATION_KEY]?: SharedPlanNavigationState };
-  if (!global[PLAN_NAVIGATION_KEY]) global[PLAN_NAVIGATION_KEY] = { visible: false };
-  return global[PLAN_NAVIGATION_KEY]!;
-}
+const LEGACY_PLAN_NAV_STATUS_KEY = "pi-better-plan-nav";
 
 export default function planExtension(pi: ExtensionAPI): void {
   let currentPlan: PlanSnapshot | null = null;
   let displayMode: PlanDisplayMode = "auto";
-  let selectedIndex = 0;
-  let focused = false;
   let refreshWidget: ((force?: boolean) => void) | undefined;
-  let currentCtx: ExtensionContext | undefined;
 
   const refresh = (force = false): void => refreshWidget?.(force);
 
-  const updateNavigationHint = (ctx = currentCtx): void => {
-    if (!ctx?.hasUI) return;
-    const progress = currentPlan ? planProgress(currentPlan) : null;
-    const visible = currentPlan !== null && displayMode !== "hidden";
-    const navigation = sharedPlanNavigationState();
-    navigation.visible = visible;
-    const suffix = progress?.blocked ? "blocked" : progress ? `${progress.completed}/${progress.total}` : "";
-    if (suffix) navigation.progressLabel = suffix;
-    else delete navigation.progressLabel;
-    if (navigation.refreshNavigationHint) {
-      ctx.ui.setStatus(PLAN_NAV_STATUS_KEY, undefined);
-      navigation.refreshNavigationHint();
-    } else {
-      ctx.ui.setStatus(PLAN_NAV_STATUS_KEY, visible ? `→ plan · ${suffix}` : undefined);
-    }
-  };
-
   const restore = (ctx: ExtensionContext): void => {
     const state = reconstructPlanState(ctx.sessionManager.getBranch());
     currentPlan = state.plan;
     displayMode = state.displayMode;
-    selectedIndex = preferredIndex(currentPlan);
-    focused = false;
-    updateNavigationHint(ctx);
+    if (ctx.hasUI) ctx.ui.setStatus(LEGACY_PLAN_NAV_STATUS_KEY, undefined);
     refresh(true);
   };
 
   const persistPlan = (plan: PlanSnapshot): void => {
     currentPlan = plan;
-    selectedIndex = preferredIndex(plan);
-    focused = false;
     pi.appendEntry(EXTENSION_NAME, planSetEntry(plan));
-    updateNavigationHint();
     refresh(true);
   };
 
   const setDisplayMode = (mode: PlanDisplayMode): void => {
     displayMode = mode;
     pi.appendEntry(EXTENSION_NAME, planDisplayEntry(mode));
-    updateNavigationHint();
     refresh(true);
   };
 
@@ -117,7 +75,7 @@ export default function planExtension(pi: ExtensionAPI): void {
         return {
           render(width: number): string[] {
             if (!currentPlan || displayMode === "hidden") return [];
-            return renderCompactPlan(currentPlan, width, theme as never, { focused, selectedIndex });
+            return renderCompactPlan(currentPlan, width, theme as never);
           },
           invalidate() {},
           dispose() {
@@ -139,8 +97,6 @@ export default function planExtension(pi: ExtensionAPI): void {
       return;
     }
     const snapshot = currentPlan;
-    focused = false;
-    refresh();
     await ctx.ui.custom<void>((tui, theme, _keybindings, done) => {
       const component = createFullPlanComponent(snapshot, theme as never, () => done());
       return {
@@ -154,80 +110,6 @@ export default function planExtension(pi: ExtensionAPI): void {
     });
   };
 
-  const handlePlanNavigation = (data: string, ctx: ExtensionContext): boolean => {
-    if (!currentPlan || displayMode === "hidden") return false;
-    if (!focused) {
-      if (!matchesKey(data, Key.right)) return false;
-      sharedPlanNavigationState().releaseWorkFocus?.();
-      focused = true;
-      selectedIndex = preferredIndex(currentPlan);
-      refresh();
-      return true;
-    }
-    if (matchesKey(data, Key.up)) {
-      selectedIndex = Math.max(0, selectedIndex - 1);
-      refresh();
-      return true;
-    }
-    if (matchesKey(data, Key.down)) {
-      selectedIndex = Math.min(currentPlan.steps.length - 1, selectedIndex + 1);
-      refresh();
-      return true;
-    }
-    if (matchesKey(data, Key.enter) || matchesKey(data, Key.right)) {
-      void showFullPlan(ctx);
-      return true;
-    }
-    if (matchesKey(data, Key.left) || matchesKey(data, Key.escape)) {
-      focused = false;
-      refresh();
-      return true;
-    }
-    focused = false;
-    refresh();
-    return false;
-  };
-
-  const installEditorNavigation = (ctx: ExtensionContext): void => {
-    if (ctx.mode !== "tui") return;
-    const ui = ctx.ui as typeof ctx.ui & {
-      getEditorComponent?(): ((tui: unknown, theme: unknown, keybindings: unknown) => unknown) | undefined;
-      setEditorComponent(factory: (tui: unknown, theme: unknown, keybindings: unknown) => unknown): void;
-    };
-    const previous = ui.getEditorComponent?.() as
-      | (((tui: unknown, theme: unknown, keybindings: unknown) => unknown) & Record<string, unknown>)
-      | undefined;
-    const handlers = { handle: (data: string) => handlePlanNavigation(data, ctx) };
-    if (previous?.[PLAN_EDITOR_FACTORY_MARK] === true) {
-      const replaceHandlers = previous[PLAN_EDITOR_FACTORY_REFRESH];
-      if (typeof replaceHandlers === "function") replaceHandlers(handlers);
-      return;
-    }
-    let currentHandlers = handlers;
-    const factory = ((tui: unknown, theme: unknown, keybindings: unknown) => {
-      const inner = (previous
-        ? previous(tui, theme, keybindings)
-        : new CustomEditor(tui as never, theme as never, keybindings as never)) as {
-        getText?(): string;
-        handleInput(data: string): void;
-      };
-      return new Proxy(inner, {
-        get(target, property) {
-          if (property === "handleInput") {
-            return (data: string) => {
-              if (target.getText?.() === "" && currentHandlers.handle(data)) return;
-              target.handleInput(data);
-            };
-          }
-          const value = Reflect.get(target, property);
-          return typeof value === "function" ? value.bind(target) : value;
-        },
-      });
-    }) as ((tui: unknown, theme: unknown, keybindings: unknown) => unknown) & Record<string, unknown>;
-    factory[PLAN_EDITOR_FACTORY_MARK] = true;
-    factory[PLAN_EDITOR_FACTORY_REFRESH] = (next: typeof handlers) => { currentHandlers = next; };
-    ui.setEditorComponent(factory);
-  };
 
   pi.registerTool({
     name: "update_plan",
@@ -290,9 +172,7 @@ export default function planExtension(pi: ExtensionAPI): void {
       if (!input) return showFullPlan(ctx);
       if (input === "clear") {
         currentPlan = null;
-        focused = false;
         pi.appendEntry(EXTENSION_NAME, planClearEntry());
-        updateNavigationHint(ctx);
         refresh(true);
         ctx.ui.notify("Plan cleared.", "info");
         return;
@@ -316,13 +196,10 @@ export default function planExtension(pi: ExtensionAPI): void {
   });
 
   pi.on("session_start", async (_event, ctx) => {
-    currentCtx = ctx;
     installWidget(ctx);
-    installEditorNavigation(ctx);
     restore(ctx);
   });
   pi.on("session_tree", async (_event, ctx) => {
-    currentCtx = ctx;
     restore(ctx);
   });
   pi.on("before_agent_start", async (event) => {
@@ -333,26 +210,13 @@ export default function planExtension(pi: ExtensionAPI): void {
   });
   pi.on("session_shutdown", async (_event, ctx) => {
     refreshWidget = undefined;
-    currentCtx = undefined;
-    const navigation = sharedPlanNavigationState();
-    navigation.visible = false;
-    delete navigation.progressLabel;
-    navigation.refreshNavigationHint?.();
     try {
-      ctx.ui.setStatus(PLAN_NAV_STATUS_KEY, undefined);
+      ctx.ui.setStatus(LEGACY_PLAN_NAV_STATUS_KEY, undefined);
       ctx.ui.setWidget(EXTENSION_NAME, undefined);
     } catch { /* best-effort cleanup */ }
   });
 }
 
-function preferredIndex(plan: PlanSnapshot | null): number {
-  if (!plan) return 0;
-  const progress = planProgress(plan);
-  if (progress.activeIndex !== null) return progress.activeIndex;
-  const next = plan.steps.findIndex((item) => item.status !== "completed");
-  return next >= 0 ? next : Math.max(0, plan.steps.length - 1);
-}
-
 function formatProgress(plan: PlanSnapshot): string {
   const progress = planProgress(plan);
   const current = progress.activeIndex === null ? null : plan.steps[progress.activeIndex];

package/node_modules/pi-better-plan/src/plan-render.ts

@@ -12,10 +12,8 @@ export function renderCompactPlan(
   plan: PlanSnapshot,
   width: number,
   theme: PlanRenderTheme,
-  options: { focused?: boolean; selectedIndex?: number } = {},
 ): string[] {
   const progress = planProgress(plan);
-  const selected = clampIndex(options.selectedIndex ?? progress.activeIndex ?? firstIncompleteIndex(plan), plan.steps.length);
   const stateLabel = progress.blocked > 0
     ? `${progress.blocked} blocked`
     : progress.state === "complete"
@@ -28,8 +26,7 @@ export function renderCompactPlan(
 
   for (let index = 0; index < plan.steps.length; index += 1) {
     const item = plan.steps[index]!;
-    const selectedPrefix = options.focused && index === selected ? theme.fg("accent", "› ") : "  ";
-    lines.push(`${selectedPrefix}${stepGlyph(item, theme)} ${index + 1}  ${stepText(item, theme)}`);
+    lines.push(`  ${stepGlyph(item, theme)} ${index + 1}  ${stepText(item, theme)}`);
   }
   lines.push("");
   return lines.map((line) => truncateToWidth(line, Math.max(1, width)));

package/node_modules/pi-better-subagents/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-subagents",
-  "version": "0.1.25",
+  "version": "0.1.26",
   "description": "Pi extension for detached, sandboxed subagent runs that keep the foreground session free.",
   "license": "MIT",
   "type": "module",

package/node_modules/pi-better-subagents/shared-navigator.ts

@@ -97,7 +97,6 @@ type NavigatorState = {
 };
 
 const GLOBAL_KEY = Symbol.for("pi-better-harness.navigator.state");
-const PLAN_NAVIGATION_KEY = Symbol.for("pi-better-harness.plan-navigation.state");
 const FACTORY_MARK = "__piBetterHarnessNavigatorFactory";
 const FACTORY_REFRESH = "__piBetterHarnessNavigatorRefresh";
 
@@ -121,23 +120,6 @@ function state(): NavigatorState {
   return g[GLOBAL_KEY]!;
 }
 
-type PlanNavigationState = {
-  visible: boolean;
-  progressLabel?: string;
-  releaseWorkFocus?: () => void;
-  refreshNavigationHint?: () => void;
-};
-
-function planNavigationState(): PlanNavigationState {
-  const global = globalThis as typeof globalThis & { [PLAN_NAVIGATION_KEY]?: PlanNavigationState };
-  if (!global[PLAN_NAVIGATION_KEY]) global[PLAN_NAVIGATION_KEY] = { visible: false };
-  return global[PLAN_NAVIGATION_KEY]!;
-}
-
-function refreshPlanNavigationHint(): void {
-  refreshBackgroundWorkNavigator();
-}
-
 export function registerBackgroundWorkProvider(provider: BackgroundWorkProvider): () => void {
   const s = state();
   const previousUnsub = s.unsubscribers.get(provider.id);
@@ -179,12 +161,7 @@ export function isNavigatorUiAvailable(ctx: ExtensionContext | undefined): boole
 }
 
 export function navigatorFooterHint(count: number): string | null {
-  const work = count > 0 ? `← work · ${count}` : null;
-  const navigation = planNavigationState();
-  const plan = navigation.visible
-    ? `→ plan${navigation.progressLabel ? ` · ${navigation.progressLabel}` : ""}`
-    : null;
-  return [work, plan].filter((hint): hint is string => hint !== null).join("    ") || null;
+  return count > 0 ? `← work · ${count}` : null;
 }
 
 export function applyNavigatorFooter(ui: { setStatus(key: string, value: string | undefined): void }, count: number): string | null {
@@ -208,10 +185,6 @@ export function ensureBackgroundWorkNavigator(ctx: ExtensionContext, deps: HostD
   s.mainListRequestRender = undefined;
   s.mainListDeadlineScheduler?.dispose();
   s.mainListDeadlineScheduler = createRenderScheduler(() => refreshMainListWidget());
-  const planNavigation = planNavigationState();
-  planNavigation.releaseWorkFocus = unfocusMainList;
-  planNavigation.refreshNavigationHint = refreshPlanNavigationHint;
-  try { (ctx.ui as any).setStatus?.("pi-better-plan-nav", undefined); } catch { /* ignore */ }
   installNavigatorEditor(ctx.ui as any, deps);
   s.lastHint = undefined;
   refreshBackgroundWorkNavigator(ctx);
@@ -240,11 +213,6 @@ export function disposeBackgroundWorkNavigator(ctx?: ExtensionContext): void {
   s.mainListRequestRender = undefined;
   s.editorComponent = undefined;
   s.detailOverlayRows = undefined;
-  const planNavigation = planNavigationState();
-  planNavigation.releaseWorkFocus = undefined;
-  if (planNavigation.refreshNavigationHint === refreshPlanNavigationHint) {
-    planNavigation.refreshNavigationHint = undefined;
-  }
   s.mainListSelectedId = undefined;
   s.mainListFocused = false;
 }
@@ -492,10 +460,9 @@ function buildMainListLines(
 }
 
 function shortcutsLine(focused: boolean, fg: (color: string, value: string) => string): string {
-  const plan = planNavigationState().visible ? " · → plan" : "";
   const keys = focused
-    ? `↑↓ switch · Enter detail · x stop${plan} · Esc unfocus`
-    : `← work navigator${plan}`;
+    ? "↑↓ switch · Enter detail · x stop · Esc unfocus"
+    : "← work navigator";
   return dim(keys, fg);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-harness",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "description": "Pi extension bundle for a write sandbox, subagents, background tasks, SSH, goals, and structured plans.",
   "license": "MIT",
   "type": "module",
@@ -50,12 +50,12 @@
     "test": "node --test test/*.test.mjs"
   },
   "dependencies": {
-    "pi-better-background-tasks": "0.2.10",
+    "pi-better-background-tasks": "0.2.11",
     "pi-better-goal": "0.2.1",
-    "pi-better-plan": "0.1.2",
+    "pi-better-plan": "0.1.3",
     "pi-better-sandbox": "0.3.0",
     "pi-better-ssh": "0.1.1",
-    "pi-better-subagents": "0.1.25"
+    "pi-better-subagents": "0.1.26"
   },
   "bundledDependencies": [
     "pi-better-background-tasks",