@tmustier/pi-files-widget 0.2.1 → 0.2.3

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to this extension will be documented in this file.
 
+## [0.2.3] - 2026-10-28
+
+### Fixed
+- Render Markdown with `glow` again. The viewer passed `glow` a pipe as standard input, and `glow` renders piped input in preference to its file argument, so it returned nothing and every Markdown file fell back to the raw view. Pressing `m` then appeared to do nothing, because each switch back to the rendered view fell back to raw again. Thanks to @ackalker for the report ([#111](https://github.com/tmustier/pi-extensions/issues/111)).
+
+## [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

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

@@ -2,8 +2,8 @@ import { visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
 import { execSync } from "node:child_process";
 import { readFileSync, statSync } from "node:fs";
 
-import { isGitRepo } from "./git";
-import { hasCommand, isMarkdownPath, stripLeadingEmptyLines } from "./utils";
+import { isGitRepo } from "./git.ts";
+import { hasCommand, isMarkdownPath, stripLeadingEmptyLines } from "./utils.ts";
 
 const DIFF_CONTENT_PREFIXES = new Set(["+", "-", " "]);
 
@@ -234,7 +234,12 @@ export function loadFileContent(
 
     if (isMarkdown && renderMarkdown && hasCommand("glow")) {
       try {
-        const output = execSync(`glow -s dark -w ${termWidth} "${filePath}"`, { encoding: "utf-8", timeout: 10000 });
+        // glow renders piped standard input in preference to its file argument.
+        const output = execSync(`glow -s dark -w ${termWidth} "${filePath}"`, {
+          encoding: "utf-8",
+          timeout: 10000,
+          stdio: ["ignore", "pipe", "pipe"],
+        });
         if (output.trim()) {
           return { lines: stripLeadingEmptyLines(output.split("\n")), renderedMarkdown: true };
         }

package/index.ts

@@ -12,7 +12,7 @@ import { isAbsolute, join, resolve } from "node:path";
 
 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 };
@@ -38,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;
       }
 
@@ -119,7 +118,7 @@ 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();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tmustier/pi-files-widget",
-  "version": "0.2.1",
+  "version": "0.2.3",
   "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": [

package/utils.ts

@@ -38,6 +38,19 @@ export function hasCommand(cmd: string, options: CommandLookupOptions = {}): boo
   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 {
   return status === "?" || status === "??";
 }