pi-readseek 0.5.2 → 0.5.4

package/README.md

@@ -48,7 +48,7 @@ optional (defaults shown):
 ```json
 {
   "readseek": {
-    "excludeTools": [],
+    "replacedTools": [],
     "imageMode": "force",
     "syntaxValidation": "warn",
     "timeoutMs": 120000,
@@ -60,9 +60,9 @@ optional (defaults shown):
 }
 ```
 
-- **excludeTools:** active tool names to hide after activating `readSeek_*`
-  tools. For a readseek-only file surface, use `["read", "edit", "write",
-  "grep"]`.
+- **replacedTools:** built-in tool names to replace with their `readSeek_*` equivalents.
+  Valid values are `"read"`, `"edit"`, `"write"`, and `"grep"`. For a
+  readseek-only file surface, use `["read", "edit", "write", "grep"]`.
 - **imageMode:** image OCR/caption/object analysis in `readSeek_read`: `"force"`
   (or its alias `"on"`) always runs it, `"off"` returns only the image
   attachment, and `"auto"` runs it only when the active model does not support

package/index.ts

@@ -45,12 +45,8 @@ export default function piReadSeekExtension(pi: ExtensionAPI): void {
 
 	pi.on("session_start", (_event, ctx) => {
 		const { settings, warnings } = resolveReadSeekJsonSettings();
-		const excludeTools = new Set(settings.excludeTools ?? []);
-		const knownTools = new Set([...pi.getAllTools().map((tool) => tool.name), ...READSEEK_TOOL_NAMES]);
+		const replacedTools = new Set<string>(settings.replacedTools ?? []);
 		const problems = warnings.map(formatSettingsWarning);
-		for (const name of excludeTools) {
-			if (!knownTools.has(name)) problems.push(`Unknown tool "${name}" in readseek.excludeTools`);
-		}
 
 		const availability = readSeekBinaryAvailability();
 		if (!availability.available) {
@@ -65,7 +61,7 @@ export default function piReadSeekExtension(pi: ExtensionAPI): void {
 		if (!availability.available) return;
 
 		const activeTools = [...pi.getActiveTools(), ...READSEEK_TOOL_NAMES]
-			.filter((name) => !excludeTools.has(name));
+			.filter((name) => !replacedTools.has(name));
 		pi.setActiveTools([...new Set(activeTools)]);
 	});
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-readseek",
-  "version": "0.5.2",
+  "version": "0.5.4",
   "description": "Pi extension for readseek-backed hash-anchored read/edit/grep, structural code maps, structural search, and file exploration",
   "type": "module",
   "exports": {

package/src/readseek-settings.ts

@@ -4,14 +4,22 @@ import { join } from "node:path";
 
 export type ReadSeekImageAnalysisMode = "force" | "off" | "auto";
 export type ReadSeekSyntaxValidationMode = "warn" | "block" | "off";
-
 interface ReadSeekGrepSettings {
   maxLines?: number;
   maxBytes?: number;
 }
 
+const READSEEK_TOOL_REPLACEMENTS = {
+  read: "readSeek_read",
+  edit: "readSeek_edit",
+  grep: "readSeek_grep",
+  write: "readSeek_write",
+} as const;
+
+type ReadSeekReplacementTool = keyof typeof READSEEK_TOOL_REPLACEMENTS;
+
 interface ReadSeekJsonSettings {
-  excludeTools?: string[];
+  replacedTools?: ReadSeekReplacementTool[];
   imageMode?: ReadSeekImageAnalysisMode;
   syntaxValidation?: ReadSeekSyntaxValidationMode;
   timeoutMs?: number;
@@ -29,7 +37,7 @@ export interface ReadSeekSettingsResult {
   warnings: ReadSeekSettingsWarning[];
 }
 
-const READSEEK_KEYS = ["excludeTools", "imageMode", "syntaxValidation", "timeoutMs", "grep"];
+const READSEEK_KEYS = ["replacedTools", "imageMode", "syntaxValidation", "timeoutMs", "grep"];
 const READSEEK_GREP_KEYS = ["maxLines", "maxBytes"];
 
 function globalSettingsPath(): string {
@@ -104,29 +112,27 @@ function readImageMode(
   return undefined;
 }
 
-function readStringArray(
+function readReplacedTools(
   raw: Record<string, unknown>,
-  key: string,
-  path: string,
   source: string,
   warnings: ReadSeekSettingsWarning[],
-): string[] | undefined {
-  if (!(key in raw)) return undefined;
-  const value = raw[key];
+): ReadSeekReplacementTool[] | undefined {
+  if (!("replacedTools" in raw)) return undefined;
+  const value = raw.replacedTools;
   if (!Array.isArray(value)) {
-    warnings.push(invalid(source, path));
+    warnings.push(invalid(source, "readseek.replacedTools"));
     return undefined;
   }
 
-  const strings: string[] = [];
-  value.forEach((item, index) => {
-    if (typeof item !== "string" || item.trim() === "") {
-      warnings.push(invalid(source, `${path}[${index}]`));
-      return;
+  const tools: ReadSeekReplacementTool[] = [];
+  value.forEach((tool, index) => {
+    if (typeof tool === "string" && Object.hasOwn(READSEEK_TOOL_REPLACEMENTS, tool)) {
+      tools.push(tool as ReadSeekReplacementTool);
+    } else {
+      warnings.push(invalid(source, `readseek.replacedTools[${index}]`));
     }
-    strings.push(item);
   });
-  return strings;
+  return tools;
 }
 
 function validateSettings(raw: unknown, source: string): ReadSeekSettingsResult {
@@ -150,8 +156,8 @@ function validateSettings(raw: unknown, source: string): ReadSeekSettingsResult
   const section = raw.readseek;
   warnUnknownKeys(section, READSEEK_KEYS, "readseek", source, warnings);
 
-  const excludeTools = readStringArray(section, "excludeTools", "readseek.excludeTools", source, warnings);
-  if (excludeTools !== undefined) settings.excludeTools = excludeTools;
+  const replacedTools = readReplacedTools(section, source, warnings);
+  if (replacedTools !== undefined) settings.replacedTools = replacedTools;
 
   const imageMode = readImageMode(section, source, warnings);
   if (imageMode !== undefined) settings.imageMode = imageMode;