@tmustier/pi-files-widget 0.2.0 → 0.2.2

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 All notable changes to this extension will be documented in this file.
 
+## [0.2.2] - 2026-10-28
+
+### Fixed
+- Suggest `brew install` only on macOS when `bat`, `delta`, or `glow` is missing, and list only the missing packages. Other systems are pointed to the install instructions in the README, which now cover Debian and Ubuntu, where `bat` installs as `batcat`. Thanks to @curious2improve for the report ([#115](https://github.com/tmustier/pi-extensions/issues/115)).
+
+## [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/README.md

@@ -22,10 +22,14 @@ Required deps (needed for /readfiles):
 # macOS (Homebrew)
 brew install bat git-delta glow
 
-# Ubuntu/Debian
+# Debian 13+ / Ubuntu 25.04+
 sudo apt-get install -y bat git-delta glow
+# Debian and Ubuntu install bat as batcat; files-widget looks for bat
+mkdir -p ~/.local/bin && ln -s /usr/bin/batcat ~/.local/bin/bat
 ```
 
+On other systems, install [bat](https://github.com/sharkdp/bat#installation), [delta](https://dandavison.github.io/delta/installation.html), and [glow](https://github.com/charmbracelet/glow#installation) with your package manager. The `bat`, `delta`, and `glow` commands must be on your `PATH`.
+
 ```bash
 pi install git:github.com/tmustier/pi-extensions
 ```

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,10 +10,9 @@ 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";
+import { hasCommand, missingCommandsMessage, REQUIRED_COMMANDS } from "./utils";
 
 function resolveInitialPath(arg: string | undefined, cwd: string): { path: string; error?: string } {
   if (!arg) return { path: cwd };
@@ -39,15 +38,14 @@ function resolveInitialPath(arg: string | undefined, cwd: string): { path: strin
 export default function editorExtension(pi: ExtensionAPI): void {
   const cwd = process.cwd();
   const agentModifiedFiles = new Set<string>();
-  const requiredDeps = ["bat", "delta", "glow"] as const;
-  const getMissingDeps = () => requiredDeps.filter((dep) => !hasCommand(dep));
+  const getMissingDeps = () => REQUIRED_COMMANDS.filter((dep) => !hasCommand(dep));
 
   pi.registerCommand("readfiles", {
     description: "Open file browser (optional: /readfiles <path> to start outside the current directory)",
     handler: async (args, ctx) => {
       const missing = getMissingDeps();
       if (missing.length > 0) {
-        ctx.ui.notify(`files-widget requires ${missing.join(", ")}. Install: brew install bat git-delta glow`, "error");
+        ctx.ui.notify(missingCommandsMessage(missing), "error");
         return;
       }
 
@@ -57,6 +55,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 +72,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");
@@ -119,13 +118,10 @@ export default function editorExtension(pi: ExtensionAPI): void {
   pi.on("session_start", async (_event, ctx) => {
     const missing = getMissingDeps();
     if (missing.length > 0) {
-      ctx.ui.notify(`files-widget requires ${missing.join(", ")}. Install: brew install bat git-delta glow`, "error");
+      ctx.ui.notify(missingCommandsMessage(missing), "error");
     }
 
     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.2",
   "description": "In-terminal file browser and viewer for Pi.",
   "license": "MIT",
   "author": "Thomas Mustier",
@@ -19,7 +19,7 @@
     "@earendil-works/pi-tui": "*"
   },
   "scripts": {
-    "postinstall": "node -e \"console.log('pi-files-widget: required deps: bat, delta, glow. Install with: brew install bat git-delta glow')\""
+    "postinstall": "node -e \"console.log('pi-files-widget: required deps: bat, delta, glow. ' + (process.platform === 'darwin' ? 'Install with: brew install bat git-delta glow' : 'Install instructions: https://github.com/tmustier/pi-extensions/tree/main/files-widget#install'))\""
   },
   "pi": {
     "extensions": [
@@ -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,54 @@
-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 const REQUIRED_COMMANDS = ["bat", "delta", "glow"] as const;
+export type RequiredCommand = (typeof REQUIRED_COMMANDS)[number];
+
+const HOMEBREW_PACKAGES: Record<RequiredCommand, string> = { bat: "bat", delta: "git-delta", glow: "glow" };
+const INSTALL_DOCS_URL = "https://github.com/tmustier/pi-extensions/tree/main/files-widget#install";
+
+export function missingCommandsMessage(missing: readonly RequiredCommand[], platform: NodeJS.Platform = process.platform): string {
+  const install = platform === "darwin"
+    ? `Install: brew install ${missing.map((command) => HOMEBREW_PACKAGES[command]).join(" ")}`
+    : `Install with your package manager: ${INSTALL_DOCS_URL}`;
+  return `files-widget requires ${missing.join(", ")}. ${install}`;
 }
 
 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" };
       }