pi-rg-fd 0.1.0 → 0.2.0

package/README.md

@@ -1,8 +1,8 @@
 # pi-rg-fd
 
-Pi coding agent extension that rewrites the default system prompt's `grep` and `find` advice to `rg` and `fd` when those tools are on `PATH`.
+Pi coding agent extension that always rewrites the default system prompt's `grep` and `find` advice to `rg` and `fd`.
 
-Pi 1.4's bash tool line names `grep` and `find`, and the default guideline still names `find`, so models reach for those commands even when ripgrep and fd are installed. This package replaces those exact lines in place. The usual prompt gets shorter, and text that is not one of those Pi 1.4 lines is left as Pi wrote it.
+Pi 1.4's bash tool line names `grep` and `find`, and the default guideline still names `find`, so models reach for those commands. This package replaces those two exact lines in place. The usual prompt gets shorter, and text that is not one of those Pi 1.4 lines is left as Pi wrote it. You confirm that `rg` and `fd` run on your machine.
 
 ## Install
 
@@ -30,31 +30,36 @@ Default guideline (bash enabled, and the grep/find/ls tools off), before:
 Use bash for file operations like ls, rg, find
 ```
 
-With both `rg` and `fd` on `PATH`, those become:
+Those lines are always rewritten to:
 
 ```text
 Execute bash commands (ls, rg, fd, etc.)
 Use bash for file operations like ls, rg, fd
 ```
 
-Each binary is swapped on its own. `fd` is checked independently of `rg`. On Windows the scan accepts `rg.exe` and `fd.exe` (and the rest of `PATHEXT`). On Debian and Ubuntu, where the package installs `fdfind`, that name is used instead of `fd`.
+`rg` and `fd` are shorter than `grep` and `find`. If neither line is present (a custom system prompt, or a future Pi release that rewords them), the hook returns without changing the prompt. The `before_agent_start` hook is a string replace cached by the prompt text, so the same input always yields the same output.
 
-| On PATH | Bash tool line | Guideline |
-| --- | --- | --- |
-| `rg` and `fd` | `Execute bash commands (ls, rg, fd, etc.)` | `Use bash for file operations like ls, rg, fd` |
-| `rg` only | `Execute bash commands (ls, rg, find, etc.)` | unchanged (it already says `rg`; `find` stays) |
-| `fd` only | `Execute bash commands (ls, grep, fd, etc.)` | `Use bash for file operations like ls, rg, fd` |
-| `fdfind` only | `Execute bash commands (ls, grep, fdfind, etc.)` | `Use bash for file operations like ls, rg, fdfind` |
-| neither | unchanged | unchanged |
+Pi's interactive mode downloads `rg` and `fd` when they are missing, into `~/.pi/agent/bin` (or `$PI_CODING_AGENT_DIR/bin` when that variable is set; a leading `~` is expanded). Bash calls see that directory on `PATH`. The managed files are named `rg` and `fd`, plus `.exe` on Windows.
 
-`rg` and `fd` are shorter than `grep` and `find`. `fdfind` is two characters longer than `find`, which is the Debian binary's name. If neither line is present (a custom system prompt, or a future Pi release that rewords them), the hook returns without changing the prompt.
+Debian and Ubuntu's apt package `fd-find` installs the binary as `fdfind`. Alias it so the rewritten prompt matches a command on `PATH`:
 
-Availability is resolved once, when the extension loads, by stating files on `PATH`. The `before_agent_start` hook is a string replace cached by the prompt text, so the same input always yields the same output.
+```bash
+alias fd=fdfind
+```
 
 ## Prompt text only
 
 `pi-cli-guard` and `pi-prefer-rg` rewrite or block bash commands when the model runs them. `pi-rg-fd` edits the system prompt Pi already built. Commands the model still emits run unchanged.
 
+## Compatibility
+
+With `@earendil-works/pi-coding-agent` installed, `npm run compat` builds Pi's default system prompt from that package and loads this extension through Pi's extension loader. CI runs the same check on Ubuntu and Windows against the latest Pi release, every day and on manual dispatch.
+
+```bash
+npm install --no-save @earendil-works/pi-coding-agent@latest
+npm run compat
+```
+
 ## License
 
 MIT

package/index.js

@@ -1,17 +1,12 @@
-import { detectSearchTools, rewritePrompt } from "./prompt.js";
+import { rewritePrompt } from "./prompt.js";
 
 const CACHE_LIMIT = 8;
 
-/**
- * @param {{ rg: boolean, fd: "fd" | "fdfind" | null }} [tools]
- */
-export function createExtension(tools = detectSearchTools()) {
+export function createExtension() {
 	/** @type {Map<string, string>} */
 	const cache = new Map();
 
 	return function piRgFd(pi) {
-		if (!tools.rg && !tools.fd) return;
-
 		pi.on("before_agent_start", (event) => {
 			const input = event.systemPrompt;
 			if (typeof input !== "string") return;
@@ -21,7 +16,7 @@ export function createExtension(tools = detectSearchTools()) {
 				cache.delete(input);
 				cache.set(input, output);
 			} else {
-				output = rewritePrompt(input, tools);
+				output = rewritePrompt(input);
 				cache.set(input, output);
 				if (cache.size > CACHE_LIMIT) cache.delete(cache.keys().next().value);
 			}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "pi-rg-fd",
-  "version": "0.1.0",
-  "description": "Pi extension that rewrites the default grep/find prompt lines to rg and fd when those tools are on PATH.",
+  "version": "0.2.0",
+  "description": "Pi extension that rewrites the default grep/find prompt lines to rg and fd.",
   "type": "module",
   "main": "index.js",
   "license": "MIT",
@@ -31,7 +31,8 @@
     "node": ">=22.19.0"
   },
   "scripts": {
-    "test": "node --test"
+    "test": "node --test",
+    "compat": "node scripts/compat.js"
   },
   "pi": {
     "extensions": [

package/prompt.js

@@ -1,6 +1,3 @@
-import { statSync } from "node:fs";
-import path from "node:path";
-
 /**
  * Default bash tool line in @earendil-works/pi-coding-agent 1.4.0
  * (`bashToolSystemPromptContribution.snippet`).
@@ -13,89 +10,8 @@ export const BASH_SNIPPET = "Execute bash commands (ls, grep, find, etc.)";
  */
 export const BASH_RULE = "Use bash for file operations like ls, rg, find";
 
-const WINDOWS_PATHEXT = ".COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC";
-
-/**
- * @param {Record<string, string | undefined>} env
- * @param {string} name
- * @param {string} platform
- */
-function envValue(env, name, platform) {
-	if (platform !== "win32") return env[name] ?? "";
-	const lower = name.toLowerCase();
-	for (const key of Object.keys(env)) {
-		if (key.toLowerCase() === lower) return env[key] ?? "";
-	}
-	return "";
-}
-
-/**
- * @param {string} name
- * @param {string} platform
- * @param {string} pathext
- */
-function commandNames(name, platform, pathext) {
-	if (platform !== "win32") return [name];
-	const extensions = new Set([".exe", ".EXE"]);
-	for (const raw of pathext.split(";")) {
-		const ext = raw.trim();
-		if (ext === "" || ext === ".") {
-			extensions.add("");
-			continue;
-		}
-		const suffix = ext.startsWith(".") ? ext : `.${ext}`;
-		extensions.add(suffix);
-		extensions.add(suffix.toLowerCase());
-		extensions.add(suffix.toUpperCase());
-	}
-	return [...extensions].map((ext) => (ext ? name + ext : name));
-}
-
-/**
- * @param {string} file
- * @param {string} platform
- */
-function isRunnableFile(file, platform) {
-	try {
-		const stat = statSync(file);
-		if (!stat.isFile()) return false;
-		if (platform === "win32") return true;
-		return (stat.mode & 0o111) !== 0;
-	} catch {
-		return false;
-	}
-}
-
-/**
- * Locate `rg` and `fd` by scanning PATH. No processes are spawned.
- * `fd` is `"fd"`, `"fdfind"` (Debian/Ubuntu), or null.
- *
- * @param {{ env?: NodeJS.ProcessEnv, platform?: string }} [options]
- * @returns {{ rg: boolean, fd: "fd" | "fdfind" | null }}
- */
-export function detectSearchTools(options = {}) {
-	const platform = options.platform ?? process.platform;
-	const env = options.env ?? process.env;
-	const dirs = envValue(env, "PATH", platform)
-		.split(platform === "win32" ? ";" : ":")
-		.filter((dir) => dir.length > 0);
-	const pathext = platform === "win32" ? envValue(env, "PATHEXT", platform) || WINDOWS_PATHEXT : "";
-
-	const has = (name) => {
-		const names = commandNames(name, platform, pathext);
-		for (const dir of dirs) {
-			for (const candidate of names) {
-				if (isRunnableFile(path.join(dir, candidate), platform)) return true;
-			}
-		}
-		return false;
-	};
-
-	return {
-		rg: has("rg"),
-		fd: has("fd") ? "fd" : has("fdfind") ? "fdfind" : null,
-	};
-}
+const REWRITTEN_SNIPPET = "Execute bash commands (ls, rg, fd, etc.)";
+const REWRITTEN_RULE = "Use bash for file operations like ls, rg, fd";
 
 /**
  * @param {string} text
@@ -112,15 +28,9 @@ function replaceExact(text, from, to) {
  * unchanged (same string when nothing matches).
  *
  * @param {string} prompt
- * @param {{ rg: boolean, fd: "fd" | "fdfind" | null }} tools
  */
-export function rewritePrompt(prompt, tools) {
-	if (!tools.rg && !tools.fd) return prompt;
-	const contentTool = tools.rg ? "rg" : "grep";
-	const fileTool = tools.fd ?? "find";
-	let next = replaceExact(prompt, BASH_SNIPPET, `Execute bash commands (ls, ${contentTool}, ${fileTool}, etc.)`);
-	if (tools.fd) {
-		next = replaceExact(next, BASH_RULE, `Use bash for file operations like ls, rg, ${tools.fd}`);
-	}
+export function rewritePrompt(prompt) {
+	let next = replaceExact(prompt, BASH_SNIPPET, REWRITTEN_SNIPPET);
+	next = replaceExact(next, BASH_RULE, REWRITTEN_RULE);
 	return next;
 }