pi-profiles-manager 1.1.4 → 1.1.6

package/package.json

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

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,
 }));
@@ -265,8 +266,8 @@ describe("package extension", () => {
     );
   });
 
-  it.each(["return", "escape"])(
-    "keeps export confirmation open until %s dismisses it",
+  it.each(["return", "escape", "a", "space"])(
+    "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,27 @@ describe("package extension", () => {
         "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
       );
       expect(tuiState.texts).toContainEqual([
-        "Copied profile 'alpha' to clipboard.",
+        "📤 Profile Export: 'alpha'",
+        1,
+        0,
+      ]);
+      expect(tuiState.texts).toContainEqual([
+        "Select string below to copy:",
+        1,
+        0,
+      ]);
+      expect(tuiState.texts).toContainEqual([
+        "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
+        1,
+        0,
+      ]);
+      expect(tuiState.texts).toContainEqual([
+        "[ Press any key or Esc to close ]",
+        1,
+        0,
+      ]);
+      expect(tuiState.texts).toContainEqual([
+        "✓ Copy command sent to terminal clipboard.",
         1,
         0,
       ]);
@@ -331,19 +352,19 @@ describe("package extension", () => {
       expect(confirmationView).toBeDefined();
       expect(handlerSettled).toBe(false);
 
-      confirmationView!.handleInput(dismissKey);
+      confirmationView?.handleInput(dismissKey);
       await handling;
 
       expect(finishConfirmation).toBeDefined();
       expect(handlerSettled).toBe(true);
       expect(ctx.ui.notify).not.toHaveBeenCalledWith(
-        "Copied profile 'alpha' to clipboard.",
+        "✓ Copy command sent to terminal clipboard.",
         "info",
       );
     },
   );
 
-  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 +379,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 +407,33 @@ 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([
+      "📤 Profile Export: 'alpha'",
       1,
       0,
     ]);
-    expect(ctx.ui.notify).not.toHaveBeenCalledWith(
-      "Copied profile 'alpha' to clipboard.",
-      "info",
-    );
+    expect(tuiState.texts).toContainEqual([
+      "piprofile:1:eyJfdHlwZSI6InBpcHJvZmlsZSIsInZlcnNpb24iOjEsInByb2ZpbGUiOnsibmFtZSI6ImFscGhhIiwib3JkZXIiOjAsImZhdm9yaXRlIjpmYWxzZX19",
+      1,
+      0,
+    ]);
+    expect(tuiState.texts).toContainEqual([
+      expect.stringContaining("Failed to copy: clipboard unavailable"),
+      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,17 +174,34 @@ 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", "Copying 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(new Text(theme.fg("accent", theme.bold(`📤 Profile Export: '${profileName}'`)), 1, 0));
+    container.addChild(statusText);
+    container.addChild(new Text(theme.fg("dim", "Select string below to copy:"), 1, 0));
+    container.addChild(new Text(theme.fg("accent", exportString), 1, 0));
+    container.addChild(new Text(theme.fg("dim", "[ Press any key or Esc to close ]"), 1, 0));
     container.addChild(new DynamicBorder((value: string) => theme.fg("accent", value)));
+
+    copyPromise.then(() => {
+      statusText.setText(theme.fg("accent", "✓ Copy command sent to terminal clipboard."));
+      container.invalidate();
+      tui.requestRender();
+    }).catch((error) => {
+      statusText.setText(theme.fg("error", `Failed to copy: ${error instanceof Error ? error.message : String(error)}`));
+      container.invalidate();
+      tui.requestRender();
+    });
+
     return {
       render: (width: number) => container.render(width),
       invalidate: () => container.invalidate(),
       handleInput: (data: string) => {
-        if (matchesKey(data, "return") || matchesKey(data, "escape")) done();
+        done();
         tui.requestRender();
       },
     };
@@ -509,12 +526,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") {