feat: migrate prompts to Clack (#305)

* refactor(prompts): add text/password/editor wrappers in lib/prompts.ts

* refactor(init): route input through lib/prompts.ts

* refactor(api): route input/editor through lib/prompts.ts

* refactor(deploy): route input/password through lib/prompts.ts

* refactor(users): route input/password through lib/prompts.ts

* refactor(app-picker): route input through lib/prompts.ts

* test: stub new lib/prompts.ts exports in existing mocks

* build(deps): add @clack/prompts and external-editor

* feat(prompts): swap confirm to @clack/prompts

* feat(prompts): swap text to @clack/prompts

* feat(prompts): swap password to @clack/prompts

* feat(prompts): implement editor via external-editor

* refactor(prompts): adopt clack validator contract at call sites

* test: route prompt mocks through lib/prompts.ts

* feat(listage): swap select/search internals to @clack/prompts

* test(listage): rewrite tests against @clack/prompts internals

* test(listage): drop ttyContext from stubs and integration harness

* feat(spinner): swap intro/outro/spinner to @clack/prompts

* refactor(cli): drop ExitPromptError handling; rely on UserAbortError

* refactor(prompts): route doctor and update confirms through lib/prompts.ts

* build(deps): remove @inquirer/* packages

* docs(changeset): refresh prompt UI with @clack/prompts

* feat(intro): wrap apps commands

* feat(intro): wrap users commands

* feat(intro): wrap config commands

* feat(intro): wrap orgs and billing toggles

* feat(intro): wrap auth, link, whoami, switch-env, unlink

* feat(intro): wrap env pull, api, skill install

* refactor(intro): retitle init, doctor, update, and open to operation style

* feat(intro): route human output through prompt rail

* feat(intro): route apps list through clack ui

* fix(review): address prompt lifecycle feedback

- pair users list intro/outro around human output only
- avoid app create UI wrapping in JSON and agent output
- ensure config push closes intro/outro across early exits

* fix(prompts): preserve clack prompt edge cases

* fix(prompts): correct interactive command outro statuses

* fix(review): address PR #305 prompt gutter feedback

- add shared gutter wrapper for guaranteed intro/outro cleanup
- route unguarded prompt rail commands through the wrapper
- merge duplicate apps list error imports

Author: wyattjoh

.changeset/clack-prompts-migration.md

@@ -0,0 +1,5 @@
+---
+"clerk": minor
+---
+
+Refresh the visual style of prompts, lists, spinners, and intro/outro brackets to use `@clack/prompts`, and make interactive command endings reflect success, failure, or paused cancellation status.

bun.lock

@@ -34,10 +34,12 @@
     "@changesets/cli": "^2.31.0",
     "@clerk/testing": "^2.0.33",
     "@types/bun": "^1.3.14",
+    "@types/semver": "^7.7.1",
     "nano-staged": "^1.0.2",
     "oxfmt": "^0.47.0",
     "oxlint": "^1.62.0",
     "playwright": "^1.60.0",
+    "semver": "^7.8.0",
     "typescript": "^6"
   },
   "nano-staged": {

package.json

@@ -17,15 +17,13 @@
     "test": "bun test src/ --parallel"
   },
   "dependencies": {
+    "@clack/prompts": "^1.3.0",
     "@clerk/cli-extras": "workspace:*",
     "@commander-js/extra-typings": "^14.0.0",
-    "@inquirer/ansi": "^2.0.5",
-    "@inquirer/core": "^11.1.9",
-    "@inquirer/figures": "^2.0.5",
-    "@inquirer/prompts": "^8.4.3",
     "@napi-rs/keyring": "^1.3.0",
     "commander": "^14.0.3",
     "env-paths": "^4.0.0",
+    "external-editor": "^3.1.0",
     "magicast": "^0.5.3",
     "semver": "^7.8.1",
     "yaml": "^2.9.0"

packages/cli-core/package.json

@@ -2,13 +2,13 @@ import { test, expect, describe, beforeEach, afterEach, spyOn, mock } from "bun:
 import { mkdtemp, rm } from "node:fs/promises";
 import { join } from "node:path";
 import { tmpdir } from "node:os";
-import { CliError, ERROR_CODE } from "../../lib/errors.ts";
+import { CliError, ERROR_CODE, UserAbortError } from "../../lib/errors.ts";
 import {
   useCaptureLog,
   credentialStoreStubs,
   gitStubs,
   configStubs,
-  promptsStubs,
+  libPromptsStubs,
   stubFetch,
 } from "../../test/lib/stubs.ts";
 
@@ -171,7 +171,11 @@ mock.module("../../lib/config.ts", () => ({
   },
 }));
 
-mock.module("@inquirer/prompts", () => promptsStubs);
+const mockConfirm = mock(async (_config?: unknown) => true);
+mock.module("../../lib/prompts.ts", () => ({
+  ...libPromptsStubs,
+  confirm: (config: unknown) => mockConfirm(config),
+}));
 
 const { _setConfigDir } = (await import("../../lib/config.ts")) as any;
 const { setMode } = (await import("../../mode.ts")) as any;
@@ -209,6 +213,8 @@ describe("api command", () => {
       throw new Error("process.exit");
     });
     stubFetch(async () => new Response(JSON.stringify(mockUsers), { status: 200 }));
+    mockConfirm.mockReset();
+    mockConfirm.mockResolvedValue(true);
   });
 
   afterEach(async () => {
@@ -479,12 +485,29 @@ describe("api command", () => {
   });
 
   test("prints API error response body to stdout and exits 1", async () => {
+    setMode("human");
     const errorBody = { errors: [{ message: "not found", code: "resource_not_found" }] };
     stubFetch(async () => new Response(JSON.stringify(errorBody), { status: 404 }));
 
     await runApi("/users/bad_id");
     expect(process.exitCode).toBe(1);
     expect(captured.out).toContain(JSON.stringify(errorBody, null, 2));
+    expect(captured.err).toContain("Failed");
+    expect(captured.err).not.toContain("Done");
+  });
+
+  test("shows Paused with instructions when a confirmation prompt is cancelled", async () => {
+    setMode("human");
+    mockConfirm.mockImplementation(async () => {
+      throw new UserAbortError();
+    });
+
+    await expect(
+      runApi("/users", { method: "POST", data: "{}", yes: false }),
+    ).rejects.toBeInstanceOf(UserAbortError);
+    expect(captured.err).toContain("Paused");
+    expect(captured.err).toContain("Run this command again to continue.");
+    expect(captured.err).not.toContain("Done");
   });
 
   test("--include shows headers on error responses too", async () => {