@xynogen/pix-edit 0.1.7 → 0.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-edit",
-	"version": "0.1.7",
+	"version": "0.1.10",
 	"description": "Pi tool — precise text replacement edit with diff rendering",
 	"type": "module",
 	"main": "src/index.ts",
@@ -34,7 +34,7 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-pretty": "*"
+		"@xynogen/pix-pretty": "^1.7.9"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/edit.test.ts

@@ -1,4 +1,6 @@
 import { describe, expect, it } from "bun:test";
+import type { CursorStore, FffState } from "@xynogen/pix-pretty/fff";
+import type { PiPrettyApi, TextComponentCtor } from "@xynogen/pix-pretty/types";
 import {
 	getEditOperations,
 	registerEditTool,
@@ -18,15 +20,31 @@ class MockTextComponent {
 describe("registerEditTool", () => {
 	it("registers a tool named 'edit'", () => {
 		const tools: string[] = [];
+		const mockPi: PiPrettyApi = {
+			registerTool(t: unknown) {
+				tools.push((t as { name: string }).name);
+			},
+			registerCommand() {},
+			on() {},
+		};
+
 		registerEditTool(
-			{ registerTool: (t: any) => tools.push(t.name) } as any,
-			() => ({ execute: async () => ({ content: [] }) }) as any,
+			mockPi,
+			() => ({ execute: async () => ({ content: [], details: undefined }) }),
 			{
 				cwd: process.cwd(),
 				sp: (p: string) => p,
-				TextComponent: MockTextComponent as any,
-				fffState: {} as any,
-				cursorStore: {} as any,
+				TextComponent: MockTextComponent as unknown as TextComponentCtor,
+				fffState: {
+					module: null,
+					finder: null,
+					partialIndex: false,
+					dbDir: null,
+				} satisfies FffState,
+				cursorStore: {
+					store: () => "",
+					get: () => undefined,
+				} as unknown as CursorStore,
 			},
 			(_id: string, _inv: () => void) => {},
 		);
@@ -39,7 +57,7 @@ describe("getEditOperations", () => {
 		const ops = getEditOperations({
 			path: "f.ts",
 			edits: [{ oldText: "a", newText: "b" }],
-		} as any);
+		});
 		expect(ops).toEqual([{ oldText: "a", newText: "b" }]);
 	});
 
@@ -47,7 +65,7 @@ describe("getEditOperations", () => {
 		const ops = getEditOperations({
 			path: "f.ts",
 			edits: [{ oldText: "x", newText: "x" }],
-		} as any);
+		});
 		expect(ops).toHaveLength(0);
 	});
 });

package/src/edit.ts

@@ -136,12 +136,14 @@ export function registerEditTool(
 			const { diffs, summary } = summarizeEditOperations(operations);
 
 			if (operations.length === 1) {
+				const op0 = operations[0];
+				if (!op0) return result;
 				setResultDetails(result, {
 					_type: "editInfo",
 					summary,
-					editLine: opEditLine(fp, operations[0].newText),
-					oldContent: operations[0].oldText,
-					newContent: operations[0].newText,
+					editLine: opEditLine(fp, op0.newText),
+					oldContent: op0.oldText,
+					newContent: op0.newText,
 					language: fileLang,
 					filePath: fp,
 				});

package/src/once.ts

@@ -14,7 +14,8 @@
  */
 export function once(pi: object, key: string, fn: () => void): void {
 	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
-	const registry = (g.__pixOnce ??= new WeakMap<object, Set<string>>());
+	if (!g.__pixOnce) g.__pixOnce = new WeakMap<object, Set<string>>();
+	const registry = g.__pixOnce;
 	let loaded = registry.get(pi);
 	if (!loaded) {
 		loaded = new Set<string>();