pi-readseek 0.3.3 → 0.3.4

package/README.md

@@ -37,6 +37,12 @@ npm install --save-dev @jarkkojs/readseek
 - **write** — creates or overwrites whole files and returns anchors for
   immediate follow-up edits.
 
+## Related
+
+- [readseek.vim](https://github.com/jarkkojs/readseek.vim) — Vim 9 plugin
+  frontend for the readseek CLI. Provides go-to-definition, references,
+  rename, hover, and structural search from within Vim.
+
 ## Licensing
 
 `pi-readseek` is licensed under `MIT`. See [LICENSE](LICENSE) for more

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Pi extension for readseek-backed hash-anchored read/edit/grep, structural code maps, structural search, and file exploration",
   "type": "module",
   "exports": {
@@ -15,7 +15,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/coctostan/pi-readseek.git"
+    "url": "git+https://github.com/jarkkojs/pi-readseek.git"
   },
   "author": "Maxwell Newman",
   "license": "MIT",
@@ -67,7 +67,7 @@
     "test": "tests"
   },
   "bugs": {
-    "url": "https://github.com/coctostan/pi-readseek/issues"
+    "url": "https://github.com/jarkkojs/pi-readseek/issues"
   },
-  "homepage": "https://github.com/coctostan/pi-readseek#readme"
+  "homepage": "https://github.com/jarkkojs/pi-readseek#readme"
 }

package/src/confusable-hyphens.ts

@@ -0,0 +1,6 @@
+/**
+ * Unicode hyphen-like characters normalized to ASCII "-" during text
+ * comparison so edits with typographic hyphens match their originals.
+ */
+export const CONFUSABLE_HYPHENS_RE =
+	/[\u2010\u2011\u2012\u2013\u2014\u2015\u2212\uFE63\uFF0D]/g;

package/src/edit-classify.ts

@@ -146,8 +146,6 @@ export function parseDifftJson(json: any): DifftClassifyResult | null {
   }
 
   return { classification: "semantic", movedBlocks };
-
-  return { classification: "semantic", movedBlocks };
 }
 
 export async function runDifftastic(

package/src/edit-diff.ts

@@ -1,7 +1,7 @@
 import * as Diff from "diff";
 import { computeLineHash } from "./hashline.js";
+import { CONFUSABLE_HYPHENS_RE } from "./confusable-hyphens.js";
 
-// ─── Line ending normalization ──────────────────────────────────────────
 
 export function detectLineEnding(content: string): "\r\n" | "\n" {
 	const crlfIdx = content.indexOf("\r\n");
@@ -31,15 +31,13 @@ export function hasBareCarriageReturn(content: string): boolean {
 	return content.replace(/\r\n/g, "").includes("\r");
 }
 
-// ─── Fuzzy text matching ────────────────────────────────────────────────
 
 const SINGLE_QUOTES_RE = /[\u2018\u2019\u201A\u201B]/g;
 const DOUBLE_QUOTES_RE = /[\u201C\u201D\u201E\u201F]/g;
-const HYPHENS_RE = /[\u2010\u2011\u2012\u2013\u2014\u2015\u2212]/g;
 const UNICODE_SPACES_RE = /[\u00A0\u2002-\u200A\u202F\u205F\u3000]/g;
 
 function normalizeFuzzyChar(ch: string): string {
-	return ch.replace(SINGLE_QUOTES_RE, "'").replace(DOUBLE_QUOTES_RE, '"').replace(HYPHENS_RE, "-").replace(UNICODE_SPACES_RE, " ");
+	return ch.replace(SINGLE_QUOTES_RE, "'").replace(DOUBLE_QUOTES_RE, '"').replace(CONFUSABLE_HYPHENS_RE, "-").replace(UNICODE_SPACES_RE, " ");
 }
 
 function normalizeForFuzzyMatch(text: string): string {
@@ -49,7 +47,7 @@ function normalizeForFuzzyMatch(text: string): string {
 		.join("\n")
 		.replace(SINGLE_QUOTES_RE, "'")
 		.replace(DOUBLE_QUOTES_RE, '"')
-		.replace(HYPHENS_RE, "-")
+		.replace(CONFUSABLE_HYPHENS_RE, "-")
 		.replace(UNICODE_SPACES_RE, " ");
 }
 
@@ -199,7 +197,6 @@ export function replaceText(
 	};
 }
 
-// ─── Diff generation ────────────────────────────────────────────────────
 
 export function generateDiffString(
 	oldContent: string,

package/src/edit.ts

@@ -45,7 +45,6 @@ export function isBinaryBuffer(buf: Buffer): boolean {
 	return buf.includes(0);
 }
 
-// ─── Schema ─────────────────────────────────────────────────────────────
 
 const hashlineEditItemSchema = Type.Union([
 	Type.Object({ set_line: Type.Object({ anchor: Type.String(), new_text: Type.String() }) }, { additionalProperties: true }),
@@ -127,7 +126,6 @@ export interface EditToolOptions {
 	syntaxValidate?: SyntaxValidateOptions["syntaxValidate"];
 }
 
-// ─── Registration ───────────────────────────────────────────────────────
 
 export function registerEditTool(pi: ExtensionAPI, options: EditToolOptions = {}) {
 	const toolConfig = {

package/src/hashline.ts

@@ -8,8 +8,8 @@
 import xxhashWasm from "xxhash-wasm";
 import { throwIfAborted } from "./runtime.js";
 import type { ReadseekLine } from "./readseek-value.js";
+import { CONFUSABLE_HYPHENS_RE } from "./confusable-hyphens.js";
 
-// ─── Types ──────────────────────────────────────────────────────────────
 
 export type HashlineEditItem =
 	| { set_line: { anchor: string; new_text: string } }
@@ -52,7 +52,6 @@ interface NoopEdit {
 	currentContent: string;
 }
 
-// ─── Hash computation ───────────────────────────────────────────────────
 
 const HASH_LEN = 3;
 const RADIX = 16;
@@ -62,7 +61,6 @@ const DICT = Array.from({ length: HASH_MOD }, (_, i) => i.toString(RADIX).padSta
 const HASHLINE_PREFIX_RE = /^\d+:[0-9a-zA-Z]{1,16}\|/;
 const DIFF_PLUS_RE = /^\+(?!\+)/;
 const HASH_ONLY_PREFIX_RE = /^[0-9a-f]{3}\|/;
-const CONFUSABLE_HYPHENS_RE = /[\u2010\u2011\u2012\u2013\u2014\u2212\uFE63\uFF0D]/g;
 const HASH_RELOCATION_WINDOW_BASE = 20;
 const HASH_RELOCATION_WINDOW_CAP = 100;
 
@@ -128,7 +126,6 @@ export function hashLines(content: string): string {
 		.join("\n");
 }
 
-// ─── Parsing ────────────────────────────────────────────────────────────
 
 export function parseLineRef(ref: string): { line: number; hash: string; content?: string } {
 	const contentMatch = ref.match(/^[^|]*\|(.*)$/);
@@ -142,7 +139,6 @@ export function parseLineRef(ref: string): { line: number; hash: string; content
 	return { line, hash: match[2], content: contentAfterPipe };
 }
 
-// ─── Mismatch formatting ────────────────────────────────────────────────
 
 function tokenSimilarity(a: string, b: string): number {
 	const tokA = new Set(a.trim().split(/\s+/));
@@ -241,7 +237,6 @@ function formatMismatchError(
 	return { message: out.join("\n"), updatedAnchors };
 }
 
-// ─── DST preprocessing helpers ──────────────────────────────────────────
 
 function splitDst(dst: string): string[] {
 	if (dst === "") return [];
@@ -280,7 +275,6 @@ function stripNewLinePrefixes(lines: string[]): string[] {
 	);
 }
 
-// ─── Whitespace / format helpers ────────────────────────────────────────
 
 function stripAllWhitespace(s: string): string {
 	return s.replace(/\s+/g, "");
@@ -362,7 +356,6 @@ function restoreOldWrappedLines(oldLines: string[], newLines: string[]): string[
 	return out;
 }
 
-// ─── Echo stripping ─────────────────────────────────────────────────────
 
 function stripInsertAnchorEcho(anchorLine: string, dst: string[]): string[] {
 	if (dst.length > 1 && wsEq(dst[0], anchorLine)) return dst.slice(1);
@@ -378,7 +371,6 @@ function stripRangeBoundaryEcho(fileLines: string[], start: number, end: number,
 	return out;
 }
 
-// ─── Edit parser ────────────────────────────────────────────────────────
 
 function parseHashlineEditItem(edit: HashlineEditItem): ParsedEdit {
 	if ("set_line" in edit) {
@@ -404,7 +396,6 @@ function parseHashlineEditItem(edit: HashlineEditItem): ParsedEdit {
 	throw new Error("replace edits are applied separately");
 }
 
-// ─── Main edit engine ───────────────────────────────────────────────────
 
 export function applyHashlineEdits(
 	content: string,

package/src/readseek-settings.ts

@@ -48,144 +48,16 @@ function invalid(source: string, path: string): ReadseekSettingsWarning {
   return { source, path, message: `Invalid readseek setting at ${path}` };
 }
 
-function readJsonObjectEnd(text: string, open: number): number {
-  let depth = 0;
-  let inString = false;
-  let escaped = false;
-
-  for (let i = open; i < text.length; i += 1) {
-    const char = text[i];
-    if (inString) {
-      if (escaped) escaped = false;
-      else if (char === "\\") escaped = true;
-      else if (char === '"') inString = false;
-      continue;
-    }
-
-    if (char === '"') {
-      inString = true;
-    } else if (char === "{") {
-      depth += 1;
-    } else if (char === "}") {
-      depth -= 1;
-      if (depth === 0) return i;
-    }
-  }
-
-  return -1;
-}
-
-function readJsonStringEnd(text: string, quote: number): number {
-  let escaped = false;
-  for (let i = quote + 1; i < text.length; i += 1) {
-    const char = text[i];
-    if (escaped) escaped = false;
-    else if (char === "\\") escaped = true;
-    else if (char === '"') return i;
-  }
-  return -1;
-}
-
-function readTopLevelObjectBodies(rawText: string, section: string): string[] {
-  const bodies: string[] = [];
-  let depth = 0;
-
-  for (let i = 0; i < rawText.length; i += 1) {
-    const char = rawText[i];
-    if (char === '"') {
-      const end = readJsonStringEnd(rawText, i);
-      if (end < 0) return bodies;
-
-      if (depth === 1) {
-        const fieldName = rawText.slice(i + 1, end);
-        let cursor = end + 1;
-        while (/\s/.test(rawText[cursor] ?? "")) cursor += 1;
-
-        if (rawText[cursor] === ":") {
-          cursor += 1;
-          while (/\s/.test(rawText[cursor] ?? "")) cursor += 1;
-          if (fieldName === section && rawText[cursor] === "{") {
-            const objectEnd = readJsonObjectEnd(rawText, cursor);
-            if (objectEnd < 0) return bodies;
-            bodies.push(rawText.slice(cursor + 1, objectEnd));
-            i = objectEnd;
-            continue;
-          }
-        }
-      }
-      i = end;
-    } else if (char === "{") {
-      depth += 1;
-    } else if (char === "}") {
-      depth = Math.max(0, depth - 1);
-    }
-  }
-
-  return bodies;
-}
-
-function rawFieldTokens(rawText: string, path: string): string[] {
-  const [section, key] = path.split(".");
-  const bodies = readTopLevelObjectBodies(rawText, section);
-  if (bodies.length !== 1) return [];
-
-  const body = bodies[0];
-  const tokens: string[] = [];
-  let depth = 0;
-
-  for (let i = 0; i < body.length; i += 1) {
-    const char = body[i];
-    if (char === '"') {
-      const end = readJsonStringEnd(body, i);
-      if (end < 0) return tokens;
-
-      if (depth === 0) {
-        const fieldName = body.slice(i + 1, end);
-        let cursor = end + 1;
-        while (/\s/.test(body[cursor] ?? "")) cursor += 1;
-
-        if (body[cursor] === ":") {
-          cursor += 1;
-          while (/\s/.test(body[cursor] ?? "")) cursor += 1;
-          if (fieldName === key) {
-            const valueStart = cursor;
-            while (cursor < body.length && !/[,}\s]/.test(body[cursor])) cursor += 1;
-            tokens.push(body.slice(valueStart, cursor));
-          }
-        }
-      }
-      i = end;
-    } else if (char === "{" || char === "[") {
-      depth += 1;
-    } else if (char === "}" || char === "]") {
-      depth = Math.max(0, depth - 1);
-    }
-  }
-
-  return tokens;
-}
-
-function isStrictJsonPositiveInteger(rawText: string, path: string, value: unknown): value is number {
-  const tokens = rawFieldTokens(rawText, path);
-  return (
-    typeof value === "number" &&
-    Number.isSafeInteger(value) &&
-    value > 0 &&
-    tokens.length === 1 &&
-    /^[1-9][0-9]*$/.test(tokens[0])
-  );
-}
-
 function readPositive(
   raw: Record<string, unknown>,
   key: string,
   path: string,
   source: string,
-  rawText: string,
   warnings: ReadseekSettingsWarning[],
 ): number | undefined {
   if (!(key in raw)) return undefined;
-  if (isStrictJsonPositiveInteger(rawText, path, raw[key])) return raw[key];
+  const val = raw[key];
+  if (typeof val === "number" && Number.isSafeInteger(val) && val > 0) return val;
   warnings.push(invalid(source, path));
   return undefined;
 }
@@ -203,16 +75,16 @@ function readBoolean(
   return undefined;
 }
 
-function validateSettings(raw: unknown, source: string, rawText: string): ReadseekSettingsResult {
+function validateSettings(raw: unknown, source: string): ReadseekSettingsResult {
   const settings: ReadseekJsonSettings = {};
   const warnings: ReadseekSettingsWarning[] = [];
   if (!isRecord(raw)) return { settings, warnings };
 
   if (isRecord(raw.grep)) {
     const grep: NonNullable<ReadseekJsonSettings["grep"]> = {};
-    const maxLines = readPositive(raw.grep, "maxLines", "grep.maxLines", source, rawText, warnings);
+    const maxLines = readPositive(raw.grep, "maxLines", "grep.maxLines", source, warnings);
     if (maxLines !== undefined) grep.maxLines = maxLines;
-    const maxBytes = readPositive(raw.grep, "maxBytes", "grep.maxBytes", source, rawText, warnings);
+    const maxBytes = readPositive(raw.grep, "maxBytes", "grep.maxBytes", source, warnings);
     if (maxBytes !== undefined) grep.maxBytes = maxBytes;
     if (Object.keys(grep).length > 0) settings.grep = grep;
   }
@@ -246,7 +118,7 @@ function readSettingsFile(path: string): ReadseekSettingsResult {
 
   try {
     const text = readFileSync(path, "utf8");
-    return validateSettings(JSON.parse(text) as unknown, path, text);
+    return validateSettings(JSON.parse(text) as unknown, path);
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     return { settings: {}, warnings: [{ source: path, message: `Invalid JSON: ${message}` }] };

package/src/tool-prompt-metadata.ts

@@ -5,8 +5,7 @@ const COMPACT_DESCRIPTIONS: Record<string, string> = {
   "read.md": "Read text files/images by path; text has LINE:HASH anchors, images return attachments.",
   "edit.md": "Edit existing text files using fresh LINE:HASH anchors from read, grep, search, or write.",
   "grep.md": "Search file contents; non-summary results include LINE:HASH anchors for edits.",
-  "find.md": "Find files recursively by basename glob, respecting .gitignore.",
-  "ls.md": "List one directory with directories first and dotfiles included.",
+
   "write.md": "Create or overwrite a complete file and return anchors.",
   "sg.md": "Search code by AST pattern and return anchored matches.",
 };
@@ -25,12 +24,7 @@ const COMPACT_GUIDELINES: Record<string, string[]> = {
     "Use grep for text search and edit-ready matching anchors.",
     "Use grep summary mode for broad count/file discovery before narrowing.",
   ],
-  "find.md": [
-    "Use find for recursive file discovery by basename glob.",
-  ],
-  "ls.md": [
-    "Use ls to inspect one directory; use find for recursion.",
-  ],
+
   "write.md": [
     "Use write to create files or intentionally overwrite whole files.",
     "Use edit rather than write for small changes or appends to existing files.",

package/prompts/find.md

@@ -1,18 +0,0 @@
-Find files or directories recursively by basename. Uses glob patterns by default, respects nested `.gitignore`, includes hidden entries, and returns relative paths.
-
-## Parameters
-
-- `pattern` — required. Glob by default; with `regex: true`, JavaScript regex against each basename.
-- `path` — directory to search, default cwd.
-- `type` — `"file"` default, `"dir"`, or `"any"`.
-- `limit` — max returned entries after filtering/sorting, default 1000.
-- `maxDepth` — non-negative directory depth limit.
-- `sortBy` — `"name"` default, `"mtime"`, or `"size"`; use `reverse: true` for descending/newest/largest first.
-- `modifiedSince` — keep entries modified strictly after an ISO date/time or relative age like `30m`, `1h`, `24h`, `7d`.
-- `minSize` / `maxSize` — inclusive file-size filters; numbers are bytes, strings accept 1024-based `KB`, `MB`, `GB`, etc. Directories are not removed by size filters.
-
-## Output and usage
-
-Output is one relative path per line. Directories end with `/`. Filtering and sorting happen before `limit`, so newest/largest queries work as expected.
-
-Use `find` for recursive name discovery, `ls` for one directory, `grep` or `search` for contents, and `read` for file content. Remember: `pattern` matches basenames, not full paths.

package/prompts/ls.md

@@ -1,11 +0,0 @@
-List one directory. Output is directories first (with `/`), then files, sorted alphabetically; dotfiles are included.
-
-## Parameters
-
-- `path` — directory to list, default cwd.
-- `limit` — max entries, default 500; must be positive.
-- `glob` — optional entry-name filter such as `*.ts`, `.env*`, or `test-*`.
-
-## Usage
-
-Use `ls` to inspect one known directory. Use `find` for recursive discovery, `grep` or `search` for contents, and `read` for file content. If output exceeds `limit` or 50 KB, narrow with `glob` or switch to `find`.