@tmustier/pi-files-widget 0.2.0 → 0.2.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this extension will be documented in this file.
 
+## [0.2.1] - 2026-09-22
+
+### Changed
+- Reduce Pi startup work by checking required commands directly on `PATH` instead of spawning `which`/`where`, and load the file-browser implementation only when `/readfiles` is invoked.
+- Update session lifecycle, key handling, notifications, and file viewer results for Pi 0.87.
+
 ## [0.2.0] - 2026-07-04
 
 ### Added

package/file-viewer.ts

@@ -171,7 +171,10 @@ export function loadFileContent(
   try {
     try {
       if (statSync(filePath).isDirectory()) {
-        return ["Directory selected - expand it in the file tree instead of opening it."];
+        return {
+          lines: ["Directory selected - expand it in the file tree instead of opening it."],
+          renderedMarkdown: false,
+        };
       }
     } catch {
       // Ignore stat errors and fall through to normal handling

package/index.ts

@@ -10,7 +10,6 @@ import { statSync } from "node:fs";
 import { homedir } from "node:os";
 import { isAbsolute, join, resolve } from "node:path";
 
-import { createFileBrowser } from "./browser";
 import { POLL_INTERVAL_MS } from "./constants";
 import { formatCommentMessage } from "./comment";
 import { hasCommand } from "./utils";
@@ -57,6 +56,7 @@ export default function editorExtension(pi: ExtensionAPI): void {
         return;
       }
       const initialPath = resolved.path;
+      const { createFileBrowser } = await import("./browser");
 
       await ctx.ui.custom<void>((tui, theme, _kb, done) => {
         let pollInterval: ReturnType<typeof setInterval> | null = null;
@@ -73,7 +73,7 @@ export default function editorExtension(pi: ExtensionAPI): void {
           const message = formatCommentMessage(payload, comment);
           if (ctx.isIdle()) {
             pi.sendUserMessage(message);
-            ctx.ui.notify(`Comment sent to agent for ${payload.relPath} (${payload.lineRange})`, "success");
+            ctx.ui.notify(`Comment sent to agent for ${payload.relPath} (${payload.lineRange})`, "info");
           } else {
             pi.sendUserMessage(message, { deliverAs: "followUp" });
             ctx.ui.notify(`Comment queued for agent for ${payload.relPath} (${payload.lineRange})`, "info");
@@ -125,7 +125,4 @@ export default function editorExtension(pi: ExtensionAPI): void {
     agentModifiedFiles.clear();
   });
 
-  pi.on("session_switch", async () => {
-    agentModifiedFiles.clear();
-  });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tmustier/pi-files-widget",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "In-terminal file browser and viewer for Pi.",
   "license": "MIT",
   "author": "Thomas Mustier",
@@ -28,7 +28,7 @@
     "video": "https://raw.githubusercontent.com/tmustier/pi-extensions/main/files-widget/demo.mp4"
   },
   "devDependencies": {
-    "@earendil-works/pi-coding-agent": "^0.80.3",
-    "@earendil-works/pi-tui": "^0.80.3"
+    "@earendil-works/pi-coding-agent": "0.87.0",
+    "@earendil-works/pi-tui": "0.87.0"
   }
 }

package/utils.ts

@@ -1,13 +1,41 @@
-import { execSync } from "node:child_process";
-
-export function hasCommand(cmd: string): boolean {
-  try {
-    const checkCmd = process.platform === "win32" ? "where" : "which";
-    execSync(`${checkCmd} ${cmd}`, { stdio: "ignore" });
-    return true;
-  } catch {
-    return false;
+import { accessSync, constants, statSync } from "node:fs";
+import { extname, join } from "node:path";
+
+export interface CommandLookupOptions {
+  platform?: NodeJS.Platform;
+  path?: string;
+  pathExt?: string;
+  cwd?: string;
+}
+
+export function hasCommand(cmd: string, options: CommandLookupOptions = {}): boolean {
+  const platform = options.platform ?? process.platform;
+  const pathValue = options.path ?? process.env.PATH ?? "";
+  const pathEntries = pathValue.split(platform === "win32" ? ";" : ":");
+  const searchDirectories = platform === "win32"
+    ? [options.cwd ?? process.cwd(), ...pathEntries]
+    : pathEntries;
+  const extensions = platform === "win32" && !extname(cmd)
+    ? (options.pathExt ?? process.env.PATHEXT ?? ".COM;.EXE;.BAT;.CMD").split(";").filter(Boolean)
+    : [""];
+  const accessMode = platform === "win32" ? constants.F_OK : constants.X_OK;
+
+  for (const rawDirectory of searchDirectories) {
+    const directory = platform === "win32" && rawDirectory.startsWith('"') && rawDirectory.endsWith('"')
+      ? rawDirectory.slice(1, -1)
+      : rawDirectory;
+    for (const extension of extensions) {
+      try {
+        const candidate = join(directory || ".", `${cmd}${extension}`);
+        accessSync(candidate, accessMode);
+        if (statSync(candidate).isFile()) return true;
+      } catch {
+        // Continue searching PATH.
+      }
+    }
   }
+
+  return false;
 }
 
 export function isUntrackedStatus(status?: string): boolean {

package/viewer.ts

@@ -501,7 +501,7 @@ export function createViewer(
         jumpToNextMatch(1);
         return { type: "none" };
       }
-      if (matchesKey(data, "N") && state.mode !== "select" && state.searchMatches.length > 0) {
+      if (matchesKey(data, "shift+n") && state.mode !== "select" && state.searchMatches.length > 0) {
         jumpToNextMatch(-1);
         return { type: "none" };
       }
@@ -553,7 +553,7 @@ export function createViewer(
         state.scroll = 0;
         return { type: "none" };
       }
-      if (matchesKey(data, "m") && state.mode !== "select" && state.mode !== "comment") {
+      if (matchesKey(data, "m") && state.mode !== "select") {
         toggleMarkdownMode();
         return { type: "none" };
       }