pi-lean-dimension 0.5.0 → 0.6.0

package/node_modules/pi-lean-host/__tests__/api-scaffold.test.ts

@@ -58,7 +58,7 @@ afterAll(() => {
 });
 
 /** Stage a guide's recipe into the guides dir (dirName = slug(shortName)). */
-function writeGuide(shortName: string, domain: string, recipe: string): void {
+function writeGuide(shortName: string, recipe: string): void {
 	const dirName = slug(shortName);
 	mkdirSync(join(tmpGuidesDir, dirName), { recursive: true });
 	writeFileSync(join(tmpGuidesDir, dirName, "guide.md"), recipe, "utf-8");
@@ -192,7 +192,7 @@ describe("api-scaffold", () => {
 	});
 
 	it("neither verify nor helper → validation error, no /tmp write", async () => {
-		writeGuide("Scaff", "scaff.example", mixedRecipe("scaff.example", "Scaff"));
+		writeGuide("Scaff", mixedRecipe("scaff.example", "Scaff"));
 		const res = await callScaffold({ domain: "scaff.example" });
 		const text = contentText(res);
 		expect(res.details).toMatchObject({ error: "nothing_to_scaffold" });
@@ -202,11 +202,7 @@ describe("api-scaffold", () => {
 	});
 
 	it("helper: true → commented-out stub written to the staged dir", async () => {
-		writeGuide(
-			"Helper",
-			"helper.example",
-			tokenRecipe("helper.example", "Helper", "getHelper"),
-		);
+		writeGuide("Helper", tokenRecipe("helper.example", "Helper", "getHelper"));
 		const res = await callScaffold({ domain: "helper.example", helper: true });
 		const text = contentText(res);
 		expect(res.details).toMatchObject({
@@ -224,7 +220,7 @@ describe("api-scaffold", () => {
 	});
 
 	it("verify: true → sentinels for path token, required-no-default query, and every requiresAnyOf member; runnable op excluded", async () => {
-		writeGuide("Scaff", "scaff.example", mixedRecipe("scaff.example", "Scaff"));
+		writeGuide("Scaff", mixedRecipe("scaff.example", "Scaff"));
 		const res = await callScaffold({ domain: "scaff.example", verify: true });
 		expect(res.details).toMatchObject({
 			mode: "verify",
@@ -253,7 +249,7 @@ describe("api-scaffold", () => {
 	});
 
 	it("verify additive merge → existing real values preserved, new sentinels added", async () => {
-		writeGuide("Merge", "merge.example", mixedRecipe("merge.example", "Merge"));
+		writeGuide("Merge", mixedRecipe("merge.example", "Merge"));
 		// Pre-seed a guides-dir verify.json with real values for two already-
 		// satisfied params; the rest are still unsatisfiable.
 		writeFileSync(
@@ -281,7 +277,6 @@ describe("api-scaffold", () => {
 	it("no unsatisfiable params → empty verify.json scaffold", async () => {
 		writeGuide(
 			"Runnable",
-			"runnable.example",
 			`---
 schemaVersion: 1
 kind: api
@@ -313,7 +308,7 @@ operations:
 	});
 
 	it("refuse-to-overwrite → existing staged verify.json errors, names delete-then-re-call", async () => {
-		writeGuide("Ov", "ov.example", tokenRecipe("ov.example", "Ov", "getOv"));
+		writeGuide("Ov", tokenRecipe("ov.example", "Ov", "getOv"));
 		// Pre-stage a verify.json (simulating a prior scaffold).
 		mkdirSync(stagedDir("ov"), { recursive: true });
 		writeFileSync(
@@ -337,7 +332,7 @@ operations:
 	});
 
 	it("refuse-to-overwrite also gates helper.ts", async () => {
-		writeGuide("Hov", "hov.example", tokenRecipe("hov.example", "Hov", "getHov"));
+		writeGuide("Hov", tokenRecipe("hov.example", "Hov", "getHov"));
 		mkdirSync(stagedDir("hov"), { recursive: true });
 		writeFileSync(join(stagedDir("hov"), "helper.ts"), "// existing\n", "utf-8");
 		const res = await callScaffold({ domain: "hov.example", helper: true });
@@ -349,11 +344,7 @@ operations:
 	});
 
 	it("both verify + helper true → both files written to one staged dir, dirName surfaced", async () => {
-		writeGuide(
-			"Both",
-			"both.example",
-			tokenRecipe("both.example", "Both", "getBoth"),
-		);
+		writeGuide("Both", tokenRecipe("both.example", "Both", "getBoth"));
 		const res = await callScaffold({
 			domain: "both.example",
 			verify: true,
@@ -375,16 +366,8 @@ operations:
 	});
 
 	it("N-guide domain → menu without selector; guide selector scaffolds the selected one", async () => {
-		writeGuide(
-			"First",
-			"multi.example",
-			tokenRecipe("multi.example", "First", "getFirst"),
-		);
-		writeGuide(
-			"Second",
-			"multi.example",
-			tokenRecipe("multi.example", "Second", "getSecond"),
-		);
+		writeGuide("First", tokenRecipe("multi.example", "First", "getFirst"));
+		writeGuide("Second", tokenRecipe("multi.example", "Second", "getSecond"));
 
 		const menu = await callScaffold({ domain: "multi.example", verify: true });
 		const menuText = contentText(menu);
@@ -411,11 +394,7 @@ operations:
 	});
 
 	it("malformed guides-dir verify.json → clear error, no /tmp write", async () => {
-		writeGuide(
-			"Badmerge",
-			"badmerge.example",
-			tokenRecipe("badmerge.example", "Badmerge", "getBad"),
-		);
+		writeGuide("Badmerge", tokenRecipe("badmerge.example", "Badmerge", "getBad"));
 		writeFileSync(
 			join(tmpGuidesDir, "badmerge", "verify.json"),
 			"{ not valid json",
@@ -519,7 +498,7 @@ operations:
         description: item id
 ---
 `;
-		writeGuide("PathKey", "path.example", recipe);
+		writeGuide("PathKey", recipe);
 		const res = await callScaffold({ domain: "path.example", verify: true });
 		expect(res.details).toMatchObject({ mode: "verify", dirName: "pathkey" });
 		const merged = JSON.parse(

package/node_modules/pi-lean-host/__tests__/api-toggle.test.ts

@@ -13,6 +13,7 @@ import type {
 	ExtensionContext,
 } from "@earendil-works/pi-coding-agent";
 import { TOOLSET_EVENTS } from "pi-tool-masking";
+import { captureApiHandler } from "./test-utils.js";
 import initApiToggle, {
 	getApiToggleState,
 	_getApiLearnStateForTest,
@@ -78,7 +79,10 @@ interface MockPi {
 function mockPi(initialTools?: string[]): MockPi {
 	let active = initialTools ?? ALL_TOOLS.map((t) => t.name);
 	const eventEmitter = new EventEmitter();
-	const handlers = new Map<string, Array<(...args: any[]) => void>>();
+	const handlers = new Map<
+		string,
+		Array<(...args: any[]) => void | Promise<void>>
+	>();
 
 	const pi = {
 		getAllTools: vi.fn(() => ALL_TOOLS as any),
@@ -89,10 +93,15 @@ function mockPi(initialTools?: string[]): MockPi {
 		appendEntry: vi.fn(),
 		registerCommand: vi.fn(),
 		registerTool: vi.fn(),
-		on: vi.fn(<T>(event: string, handler: (event: T, ctx: any) => void) => {
-			if (!handlers.has(event)) handlers.set(event, []);
-			handlers.get(event)!.push(handler as any);
-		}),
+		on: vi.fn(
+			<T>(
+				event: string,
+				handler: (event: T, ctx: any) => void | Promise<void>,
+			) => {
+				if (!handlers.has(event)) handlers.set(event, []);
+				handlers.get(event)!.push(handler as any);
+			},
+		),
 		get events() {
 			return {
 				emit: (channel: string, data: unknown) => eventEmitter.emit(channel, data),
@@ -124,13 +133,6 @@ function mockCtx(overrides: Partial<ExtensionContext> = {}): any {
 	};
 }
 
-/** Capture the /api command handler from registerCommand. */
-function captureApiHandler(
-	pi: ExtensionAPI,
-): (args: string, ctx: any) => Promise<void> {
-	return (pi.registerCommand as any).mock.calls[0][1].handler;
-}
-
 // ==================================================================
 //  Default export
 // ==================================================================

package/node_modules/pi-lean-host/__tests__/bootstrap-command.test.ts

@@ -24,6 +24,7 @@ import {
 	writeSecret,
 } from "../core/secrets-store.js";
 import initApiToggle, { _resetToggleStateForTest } from "../core/api-toggle.js";
+import { captureApiHandler } from "./test-utils.js";
 
 // Clean globalThis registry between test files (same as api-toggle.test.ts).
 const REGISTRY_KEY = "__piToolMaskingRegistry";
@@ -122,13 +123,6 @@ function mockCtx(overrides: Partial<ExtensionContext> = {}): any {
 	};
 }
 
-function captureApiHandler(pi: ExtensionAPI) {
-	return (pi.registerCommand as any).mock.calls[0][1].handler as (
-		args: string,
-		ctx: any,
-	) => Promise<void>;
-}
-
 async function runBootstrap(
 	args: string,
 	initialTools?: string[],

package/node_modules/pi-lean-host/__tests__/delete-command.test.ts

@@ -31,6 +31,7 @@ import {
 	_resetLoadWarningsForTest,
 } from "../core/guide-store.js";
 import { handleDeleteSubcommand } from "../core/delete-command.js";
+import { mockCtx, notifyText } from "./test-utils.js";
 
 // ═══════════════════════════════════════════════════════════════════
 // Fixtures
@@ -76,18 +77,6 @@ function setupGuide(dirName: string, recipeText: string): void {
 	invalidateCache();
 }
 
-function mockCtx(overrides: Record<string, unknown> = {}): any {
-	return {
-		ui: { notify: vi.fn(), confirm: vi.fn(() => true) },
-		hasUI: true,
-		...overrides,
-	};
-}
-
-function notifyText(ctx: any): string {
-	return ctx.ui.notify.mock.calls.map((c: unknown[]) => c[0]).join("\n");
-}
-
 // ═══════════════════════════════════════════════════════════════════
 // Load-time warning suppression (once per process)
 // ═══════════════════════════════════════════════════════════════════