pi-profiles-manager 1.1.1 → 1.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-profiles-manager",
-	"version": "1.1.1",
+	"version": "1.1.5",
 	"description": "Interactive SDD model profile management built natively for the Pi Coding Agent.",
 	"main": "index.ts",
 	"type": "module",

package/src/commands.test.ts

@@ -70,16 +70,27 @@ describe("registerCommands", () => {
 		);
 	});
 
-	it("registers /profiles:sync alias command", () => {
+	it("registers slash-command aliases for every profile action", () => {
 		const pi = mockPi();
 		const mgr = new ProfileManager(pi, mockCtx());
 		mgr.setConfig(configWithProfiles([]));
 
 		registerCommands(pi, mgr, vi.fn() as any, vi.fn() as any);
-		expect(pi.registerCommand).toHaveBeenCalledWith(
-			"profiles:sync",
-			expect.any(Object),
-		);
+
+		for (const action of [
+			"list",
+			"status",
+			"sync",
+			"use",
+			"save",
+			"next",
+			"off",
+		]) {
+			expect(pi.registerCommand).toHaveBeenCalledWith(
+				`profiles:${action}`,
+				expect.any(Object),
+			);
+		}
 	});
 
 	it("completes actions first and profile names after use or save", () => {
@@ -267,23 +278,28 @@ describe("registerCommands", () => {
 		expect(openTui).not.toHaveBeenCalled();
 	});
 
-	it("profiles:sync alias command delegates directly to sync without opening TUI", async () => {
+	it("profiles action aliases delegate through the existing action handling", async () => {
 		const pi = mockPi();
 		const ctx = mockCtx();
 		const mgr = new ProfileManager(pi, ctx);
+		const save = vi.fn(async () => {});
 		const sync = vi.fn(async () => {});
 		const openTui = vi.fn(async () => {});
-		mgr.setConfig(configWithProfiles([]));
+		mgr.setConfig(configWithProfiles(["alpha"]));
 
-		registerCommands(pi, mgr, vi.fn() as any, vi.fn() as any, openTui, sync);
+		registerCommands(pi, mgr, save as any, vi.fn() as any, openTui, sync);
 
-		const callArgs = (pi.registerCommand as any).mock.calls.find(
-			(c: any[]) => c[0] === "profiles:sync",
-		);
-		const handler = callArgs[1].handler;
-		await handler("", ctx);
+		const handlerFor = (command: string) =>
+			(pi.registerCommand as any).mock.calls.find(
+				(c: any[]) => c[0] === command,
+			)[1].handler;
+		await handlerFor("profiles:save")("snapshot", ctx);
+		await handlerFor("profiles:sync")("", ctx);
+		await handlerFor("profiles:list")("", ctx);
 
+		expect(save).toHaveBeenCalledWith(ctx, "snapshot");
 		expect(sync).toHaveBeenCalledWith(ctx);
+		expect(ctx.ui.notify).toHaveBeenCalledWith("alpha");
 		expect(openTui).not.toHaveBeenCalled();
 	});
 });

package/src/commands.ts

@@ -30,19 +30,52 @@ export function registerCommands(
 	openTui: OpenTuiFn = async () => {},
 	sync: SyncFn = async () => {},
 ) {
-	pi.registerCommand("profiles:sync", {
-		description: "Sync PiProfiles with managed agents folder",
-		async handler(_args: string, ctx: ExtensionCommandContext) {
-			try {
+	const handleAction = async (args: string, ctx: ExtensionCommandContext) => {
+		manager.setContext(ctx);
+		const { verb, name } = parseCommand(args);
+
+		try {
+			if (verb === "list") {
+				return ctx.ui.notify(manager.names().join("\n") || "No profiles");
+			}
+			if (verb === "status") {
+				const active = manager.state.get(ctx.sessionManager.getSessionId());
+				return ctx.ui.notify(active?.profile ?? "none");
+			}
+			if (verb === "sync") {
 				await sync(ctx);
-			} catch (error: unknown) {
-				ctx.ui.notify(
-					error instanceof Error ? error.message : String(error),
-					"error",
-				);
+				return;
 			}
-		},
-	});
+			if (verb === "use") {
+				if (!name) {
+					return ctx.ui.notify("Usage: /profiles use <name>", "error");
+				}
+				await manager.use(name);
+				return;
+			}
+			if (verb === "save") {
+				if (!name) {
+					return ctx.ui.notify("Usage: /profiles save <name>", "error");
+				}
+				await _save(ctx, name);
+				return;
+			}
+			if (verb === "next") {
+				await manager.next();
+				return;
+			}
+			if (verb === "off") {
+				await manager.off();
+				return;
+			}
+			return openTui(ctx as ContextLike);
+		} catch (error: unknown) {
+			ctx.ui.notify(
+				error instanceof Error ? error.message : String(error),
+				"error",
+			);
+		}
+	};
 
 	pi.registerCommand("profiles", {
 		description: "Manage SDD model profiles",
@@ -65,51 +98,14 @@ export function registerCommands(
 				.filter((name) => name.startsWith(namePrefix))
 				.map((value) => ({ value, label: value }));
 		},
-		async handler(args: string, ctx: ExtensionCommandContext) {
-			manager.setContext(ctx);
-			const { verb, name } = parseCommand(args);
-
-			try {
-				if (verb === "list") {
-					return ctx.ui.notify(manager.names().join("\n") || "No profiles");
-				}
-				if (verb === "status") {
-					const active = manager.state.get(ctx.sessionManager.getSessionId());
-					return ctx.ui.notify(active?.profile ?? "none");
-				}
-				if (verb === "sync") {
-					await sync(ctx);
-					return;
-				}
-				if (verb === "use") {
-					if (!name) {
-						return ctx.ui.notify("Usage: /profiles use <name>", "error");
-					}
-					await manager.use(name);
-					return;
-				}
-				if (verb === "save") {
-					if (!name) {
-						return ctx.ui.notify("Usage: /profiles save <name>", "error");
-					}
-					await _save(ctx, name);
-					return;
-				}
-				if (verb === "next") {
-					await manager.next();
-					return;
-				}
-				if (verb === "off") {
-					await manager.off();
-					return;
-				}
-				return openTui(ctx as ContextLike);
-			} catch (error: unknown) {
-				ctx.ui.notify(
-					error instanceof Error ? error.message : String(error),
-					"error",
-				);
-			}
-		},
+		handler: handleAction,
 	});
+
+	for (const action of COMMAND_ACTIONS) {
+		pi.registerCommand(`profiles:${action}`, {
+			description: `PiProfiles: ${action}`,
+			handler: (args: string, ctx: ExtensionCommandContext) =>
+				handleAction(`${action} ${args}`.trim(), ctx),
+		});
+	}
 }

package/src/extension.test.ts

@@ -34,6 +34,7 @@ vi.mock("@earendil-works/pi-tui", () => ({
   },
   Text: class {
     constructor(...args: unknown[]) { tuiState.texts.push(args); }
+    setText(text: string) { tuiState.texts.push([text, 1, 0]); }
   },
   matchesKey: (data: string, key: string) => data === key,
 }));
@@ -266,7 +267,7 @@ describe("package extension", () => {
   });
 
   it.each(["return", "escape"])(
-    "keeps export confirmation open until %s dismisses it",
+    "keeps export string visible until %s dismisses it",
     async (dismissKey) => {
       const syncFs = await import("node:fs");
       vi.mocked(syncFs.readFileSync).mockReturnValue(JSON.stringify({
@@ -290,7 +291,7 @@ describe("package extension", () => {
         .fn()
         .mockResolvedValueOnce("alpha")
         .mockResolvedValueOnce("export")
-        .mockImplementationOnce(async (factory: any, options: unknown) => {
+        .mockImplementationOnce(async (factory: (tui: { requestRender: () => void }, theme: { fg: (color: string, text: string) => string; bold: (text: string) => string }, kb: unknown, done: (value: unknown) => void) => { handleInput(data: string): void }, options: unknown) => {
           confirmationOptions = options;
           return await new Promise((resolve) => {
             finishConfirmation = resolve;
@@ -320,7 +321,12 @@ describe("package extension", () => {
         "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
       );
       expect(tuiState.texts).toContainEqual([
-        "Copied profile 'alpha' to clipboard.",
+        "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
+        1,
+        0,
+      ]);
+      expect(tuiState.texts).toContainEqual([
+        expect.stringContaining("Copied profile 'alpha' to clipboard"),
         1,
         0,
       ]);
@@ -331,7 +337,7 @@ describe("package extension", () => {
       expect(confirmationView).toBeDefined();
       expect(handlerSettled).toBe(false);
 
-      confirmationView!.handleInput(dismissKey);
+      confirmationView?.handleInput(dismissKey);
       await handling;
 
       expect(finishConfirmation).toBeDefined();
@@ -343,7 +349,7 @@ describe("package extension", () => {
     },
   );
 
-  it("reports clipboard failures without showing export confirmation", async () => {
+  it("keeps export string visible and shows error if clipboard fails", async () => {
     const syncFs = await import("node:fs");
     vi.mocked(syncFs.readFileSync).mockReturnValue(JSON.stringify({
       version: 1,
@@ -358,10 +364,27 @@ describe("package extension", () => {
     const handler = (vi.mocked(pi.registerCommand).mock.calls as any[]).find(
       ([name]: [string]) => name === "profiles",
     )[1].handler as (args: string, ctx: any) => Promise<void>;
+    
+    let confirmationView: { handleInput(data: string): void } | undefined;
+    let finishConfirmation: ((value: unknown) => void) | undefined;
     const custom = vi
       .fn()
       .mockResolvedValueOnce("alpha")
       .mockResolvedValueOnce("export")
+      .mockImplementationOnce(async (factory: (tui: { requestRender: () => void }, theme: { fg: (color: string, text: string) => string; bold: (text: string) => string }, kb: unknown, done: (value: unknown) => void) => { handleInput(data: string): void }, _options: unknown) => {
+        return await new Promise((resolve) => {
+          finishConfirmation = resolve;
+          confirmationView = factory(
+            { requestRender: vi.fn() },
+            {
+              fg: (_color: string, text: string) => text,
+              bold: (text: string) => text,
+            },
+            {},
+            resolve,
+          );
+        });
+      })
       .mockResolvedValueOnce(null);
     const ctx = {
       sessionManager: { getSessionId: () => "session-1" },
@@ -369,22 +392,28 @@ describe("package extension", () => {
       ui: { custom, notify: vi.fn(), setStatus: vi.fn() },
     };
 
-    await handler("", ctx);
+    let handlerSettled = false;
+    const handling = handler("", ctx).then(() => { handlerSettled = true; });
+    await vi.waitFor(() => expect(custom).toHaveBeenCalledTimes(3));
 
-    expect(ctx.ui.notify).toHaveBeenCalledWith(
-      "Failed to copy profile 'alpha' to clipboard: clipboard unavailable",
-      "error",
-    );
-    expect(custom).toHaveBeenCalledTimes(3);
-    expect(tuiState.texts).not.toContainEqual([
-      "Copied profile 'alpha' to clipboard.",
+    expect(tuiState.texts).toContainEqual([
+      "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
       1,
       0,
     ]);
-    expect(ctx.ui.notify).not.toHaveBeenCalledWith(
-      "Copied profile 'alpha' to clipboard.",
-      "info",
-    );
+    expect(tuiState.texts).toContainEqual([
+      expect.stringContaining("Failed to copy profile 'alpha' to clipboard"),
+      1,
+      0,
+    ]);
+    expect(confirmationView).toBeDefined();
+    expect(handlerSettled).toBe(false);
+
+    confirmationView?.handleInput("return");
+    await handling;
+
+    expect(finishConfirmation).toBeDefined();
+    expect(handlerSettled).toBe(true);
   });
 
   it("imports typed profile strings into the authoritative config without legacy storage", async () => {

package/src/extension.ts

@@ -174,12 +174,26 @@ function prompt(ctx: ContextLike, title: string, initial = ""): Promise<string |
   }, { overlay: true });
 }
 
-function showCopyConfirmation(ctx: ContextLike, message: string): Promise<void> {
-  return (ctx.ui.custom as any)((tui: any, theme: any, _kb: any, done: any) => {
-    const container = new Container() as any;
+function showExportDialog(ctx: ContextLike, profileName: string, exportString: string, copyPromise: Promise<void>): Promise<void> {
+  return (ctx.ui.custom as (factory: unknown, options?: unknown) => Promise<void>)((tui: { requestRender: () => void }, theme: { fg: (color: string, text: string) => string; bold: (text: string) => string }, _kb: unknown, done: (value: void) => void) => {
+    const container = new Container() as { addChild: (child: unknown) => void; render: (width: number) => unknown; invalidate: () => void };
+    const statusText = new Text(theme.fg("accent", theme.bold(`Copying profile '${profileName}' to clipboard...`)), 1, 0);
+
     container.addChild(new DynamicBorder((value: string) => theme.fg("accent", value)));
-    container.addChild(new Text(theme.fg("accent", theme.bold(message)), 1, 0));
+    container.addChild(statusText);
+    container.addChild(new Text(theme.fg("accent", exportString), 1, 0));
     container.addChild(new DynamicBorder((value: string) => theme.fg("accent", value)));
+
+    copyPromise.then(() => {
+      statusText.setText(theme.fg("accent", theme.bold(`Copied profile '${profileName}' to clipboard.`)));
+      container.invalidate();
+      tui.requestRender();
+    }).catch((error) => {
+      statusText.setText(theme.fg("error", theme.bold(`Failed to copy profile '${profileName}' to clipboard: ${error instanceof Error ? error.message : String(error)}`)));
+      container.invalidate();
+      tui.requestRender();
+    });
+
     return {
       render: (width: number) => container.render(width),
       invalidate: () => container.invalidate(),
@@ -509,12 +523,9 @@ export default function extension(pi: PiLike) {
             favorite: selected === manager.config.defaultProfile,
           },
         })).toString("base64");
-        try {
-          await copyToClipboard(`piprofile:1:${encoded}`);
-          await showCopyConfirmation(ctx, `Copied profile '${selected}' to clipboard.`);
-        } catch (error) {
-          ctx.ui.notify(`Failed to copy profile '${selected}' to clipboard: ${error instanceof Error ? error.message : String(error)}`, "error");
-        }
+        const exportString = `piprofile:1:${encoded}`;
+        const copyPromise = copyToClipboard(exportString);
+        await showExportDialog(ctx, selected, exportString, copyPromise);
         continue;
       }
       if (action === "delete") {