@d3ara1n/pi-hashline-edit 0.5.1 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@d3ara1n/pi-hashline-edit",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "type": "module",
   "description": "Hashline-style file editing for pi — line-anchored edits verified by content hash, replacing oldText/newText matching",
   "homepage": "https://github.com/d3ara1n/pi-extensions/tree/main/packages/pi-hashline-edit#readme",

package/src/integration/grep-rg.test.ts

@@ -12,7 +12,7 @@ import { makeGrepOverrideWithBackend } from "../pi/grep-tool.ts";
 async function findPathRg(): Promise<string | null> {
   for (const directory of process.env.PATH?.split(delimiter) ?? []) {
     if (!directory) continue;
-    const candidate = join(directory, "rg");
+    const candidate = join(directory, process.platform === "win32" ? "rg.exe" : "rg");
     try {
       await access(candidate, constants.X_OK);
       return candidate;
@@ -31,15 +31,16 @@ test("real rg emits anchored matches from a temporary directory", {
     const file = join(directory, "fixture.ts");
     await writeFile(file, "needle\nother\n");
     const tool = makeGrepOverrideWithBackend(directory, {
-      findRg: async () => rgPath,
       delegate: async () => {
         throw new Error("integration test must not invoke the built-in grep delegate");
       },
     });
 
-    const result: any = await tool.execute("0", { pattern: "needle" }, undefined, undefined);
-    assert.match(result.content[0].text, /fixture\.ts · 1 match/);
-    assert.match(result.content[0].text, /1#[0-9A-Z]+│needle/);
+    for (const pattern of ["needle", ["needle"]]) {
+      const result: any = await tool.execute("0", { pattern }, undefined, undefined);
+      assert.match(result.content[0].text, /fixture\.ts · 1 match/);
+      assert.match(result.content[0].text, /1#[0-9A-Z]+│needle/);
+    }
   } finally {
     await rm(directory, { recursive: true, force: true });
   }

package/src/pi/grep-tool.ts

@@ -55,15 +55,16 @@ const GREP_MAX_LINE_LENGTH = 500;
 
 /** Locate ripgrep: pi's bundled bin first, then PATH. Returns null if not found. */
 async function findRg(): Promise<string | null> {
+  const executable = process.platform === "win32" ? "rg.exe" : "rg";
   const agentDir = getAgentDir();
-  const piRg = join(agentDir, "bin", "rg");
+  const piRg = join(agentDir, "bin", executable);
   try {
     await access(piRg, constants.X_OK);
     return piRg;
   } catch {}
   for (const dir of process.env.PATH?.split(delimiter) ?? []) {
     if (!dir) continue;
-    const p = join(dir, "rg");
+    const p = join(dir, executable);
     try {
       await access(p, constants.X_OK);
       return p;

package/src/pi/pi.test.ts

@@ -2,15 +2,18 @@ import { test } from "node:test";
 import assert from "node:assert/strict";
 import { canonicalPath } from "./read-tool.ts";
 import { homedir } from "node:os";
+import { join, resolve } from "node:path";
 
 test("canonicalPath resolves relative and absolute", () => {
-	assert.equal(canonicalPath("/cwd", "foo.ts"), "/cwd/foo.ts");
-	assert.equal(canonicalPath("/cwd", "./foo.ts"), "/cwd/foo.ts");
-	assert.equal(canonicalPath("/cwd", "/abs/x.ts"), "/abs/x.ts");
+	const cwd = resolve("/cwd");
+	const absolute = resolve("/abs/x.ts");
+	assert.equal(canonicalPath(cwd, "foo.ts"), join(cwd, "foo.ts"));
+	assert.equal(canonicalPath(cwd, "./foo.ts"), join(cwd, "foo.ts"));
+	assert.equal(canonicalPath(cwd, absolute), absolute);
 });
 
 test("canonicalPath expands ~ to home directory", () => {
 	const home = homedir();
 	assert.equal(canonicalPath("/cwd", "~"), home);
-	assert.equal(canonicalPath("/cwd", "~/foo.ts"), `${home}/foo.ts`);
+	assert.equal(canonicalPath("/cwd", "~/foo.ts"), join(home, "foo.ts"));
 });