@bcmyguest/openmem 0.0.0-stage → 0.6.3

package/wizard/lib/provider-wizard/helpers.mjs

@@ -0,0 +1,98 @@
+import { step, info, ok, warn, die, problem } from "../log.mjs";
+import { ask, confirm } from "../prompts.mjs";
+import { act, actWithEnv, commandExists } from "../exec.mjs";
+import { curlGet, curlJson, curlRequest } from "../curl.mjs";
+import { loadMachineEnv } from "../load-machine-env.mjs";
+import { envValue, expandHome, ensureHsDir } from "../paths.mjs";
+import { portFree, choosePort, resolveHostPort } from "../ports.mjs";
+import { composeArgs } from "../compose.mjs";
+import { writeWizardConf } from "../wizard-conf.mjs";
+import { providerApiBase } from "../providers/registry.mjs";
+
+/** Bump when a helper is removed or changes meaning; adding helpers does not bump it. */
+export const HELPERS_API_VERSION = 2;
+
+/**
+ * API passed as the second argument of every hook in providers/<id>/wizard.mjs.
+ * Providers must not import run-wizard.mjs (it imports them); everything the meta wizard
+ * owns reaches them through this object. Check `apiVersion` before relying on a helper.
+ *
+ * Every function that takes `ctx` receives the WizardContext the hook was called with
+ * (`ctx.repoRoot` = mutable stack root, `ctx.packageRoot` = read-only code root).
+ *
+ * @typedef {object} ProviderWizardHelpers
+ * @property {2} apiVersion
+ * @property {(title: string) => void} step print a numbered step heading
+ * @property {(msg: string) => void} info
+ * @property {(msg: string) => void} ok
+ * @property {(msg: string) => void} warn
+ * @property {(msg: string) => never} die print an error and exit 1
+ * @property {(ctx: object, msg: string) => void} problem die, or warn in a dry run
+ * @property {(ctx: object, prompt: string, defaultValue: string) => Promise<string>} ask
+ * @property {(ctx: object, prompt: string, defaultYN: "y"|"n") => Promise<boolean>} confirm
+ * @property {(ctx: object, description: string, cmd: string, args: string[], opts?: { env?: Record<string, string>, cwd?: string }) => Promise<void>} act
+ *   run a command (printed, not run, in a dry run); exits the wizard on failure
+ * @property {(ctx: object, description: string, cmd: string, args: string[], extraEnv: Record<string, string>) => Promise<void>} actWithEnv
+ * @property {(name: string) => boolean} commandExists
+ * @property {(repoRoot: string) => Record<string, string>} loadMachineEnv deprecated; always {}
+ * @property {(file: string, key: string) => string} envValue read one KEY=value from an env file
+ * @property {(p: string, home?: string) => string} expandHome
+ * @property {(ctx: object) => void} ensureConfigDir create the per-machine config dir (no-op in a dry run)
+ * @property {(url: string, timeoutSec?: number) => string} curlGet
+ * @property {(url: string) => object|null} curlJson
+ * @property {(method: string, url: string, jsonBody?: string, timeoutSec?: number) => string} curlRequest
+ * @property {(port: number) => Promise<boolean>} portFree
+ * @property {(ctx: object, label: string, port: number, opts?: { envVars?: string[] }) => Promise<number>} choosePort
+ * @property {(ctx: object, label?: string) => Promise<number>} resolveHostPort
+ *   pick the manifest's host port (compose.hostPortEnv / defaultHostPort) and export it in process.env
+ * @property {(ctx: object, ...rest: string[]) => string[]} composeArgs
+ *   `docker` args for a compose subcommand, with the manifest's profile and files applied
+ * @property {(ctx: object, extra?: Record<string, string>, opts?: { pmSections?: Record<string, Record<string, unknown>> }) => void} saveWizardConf
+ *   write wizard.conf (MEMORY_PROVIDER + extra keys) and config.toml; no-op in a dry run
+ * @property {(ctx: object, opts?: { description?: string, args?: string[], env?: Record<string, string> }) => Promise<void>} runStackSetup
+ *   run scripts/setup.mjs (writes the stack .env and compose.override.yaml, then the provider's
+ *   `prepareStackEnv`); `description` and `env` default to what the `stackEnv` hook returns
+ * @property {(ctx: object) => Promise<string>|string} apiBase the provider's API base URL (apiBase hook, else manifest)
+ */
+
+/**
+ * Build the helper object. With no argument it is fully functional from leaf modules;
+ * the meta wizard overrides the few helpers that need its own state.
+ * @param {Partial<ProviderWizardHelpers>} [impl]
+ * @returns {ProviderWizardHelpers}
+ */
+export function createHelpers(impl = {}) {
+  return Object.freeze(
+    /** @type {ProviderWizardHelpers} */ ({
+      apiVersion: HELPERS_API_VERSION,
+      step,
+      info,
+      ok,
+      warn,
+      die,
+      problem,
+      ask,
+      confirm,
+      act,
+      actWithEnv,
+      commandExists,
+      loadMachineEnv,
+      envValue,
+      expandHome,
+      ensureConfigDir: ensureHsDir,
+      curlGet,
+      curlJson,
+      curlRequest,
+      portFree,
+      choosePort,
+      resolveHostPort,
+      composeArgs,
+      saveWizardConf: writeWizardConf,
+      runStackSetup: async () => {
+        throw new Error("runStackSetup is only available inside the meta wizard");
+      },
+      apiBase: (ctx) => providerApiBase(ctx, ctx.memoryProvider),
+      ...impl,
+    }),
+  );
+}

package/wizard/lib/provider-wizard/run-steps.mjs

@@ -0,0 +1,129 @@
+import fs from "node:fs";
+import path from "node:path";
+import { pathToFileURL } from "node:url";
+import { spawnSync } from "node:child_process";
+import { info, warn } from "../log.mjs";
+import { providersRoot } from "../providers/registry.mjs";
+
+/** @type {Map<string, Record<string, unknown>>} */
+const moduleCache = new Map();
+
+/**
+ * Absolute path of the provider's hook module, or "" when the manifest points outside
+ * the provider directory.
+ * @param {import('../providers/registry.mjs').ProviderManifest} manifest
+ */
+export function wizardModulePath(manifest) {
+  const dir = path.resolve(providersRoot(), manifest.id);
+  const file = path.resolve(dir, manifest.wizardHooks?.module || "wizard.mjs");
+  return file.startsWith(`${dir}${path.sep}`) ? file : "";
+}
+
+/**
+ * Import `providers/<id>/wizard.mjs` (cached per file). A provider without the module
+ * gets an empty export set, so every phase is a no-op for it.
+ * @param {import('../providers/registry.mjs').ProviderManifest} manifest
+ * @returns {Promise<Record<string, unknown>>}
+ */
+export async function loadProviderWizard(manifest) {
+  const file = wizardModulePath(manifest);
+  if (!file || !fs.existsSync(file)) return {};
+  if (moduleCache.has(file)) return moduleCache.get(file);
+  const mod = await import(pathToFileURL(file).href);
+  moduleCache.set(file, mod);
+  return mod;
+}
+
+/** @typedef {keyof typeof PROVIDER_PHASES} ProviderPhase */
+
+/**
+ * Hook names a provider module may export. Phases in ANNOUNCED print
+ * "provider <id>: <phase>" when they run; the rest are quiet queries.
+ */
+export const PROVIDER_PHASES = {
+  // v1 phases (unchanged)
+  afterProvider: "afterProvider",
+  configureModelRouting: "configureModelRouting",
+  beforeStack: "beforeStack",
+  afterStackHealthy: "afterStackHealthy",
+  extendValidation: "extendValidation",
+  // v2 phases
+  init: "init",
+  resolvePorts: "resolvePorts",
+  setupEnv: "setupEnv",
+  stackEnv: "stackEnv",
+  hydrate: "hydrate",
+  configToml: "configToml",
+  summary: "summary",
+  apiBase: "apiBase",
+  smoke: "smoke",
+  setupCodingAgent: "setupCodingAgent",
+};
+
+const ANNOUNCED = new Set([
+  PROVIDER_PHASES.afterProvider,
+  PROVIDER_PHASES.configureModelRouting,
+  PROVIDER_PHASES.beforeStack,
+  PROVIDER_PHASES.afterStackHealthy,
+  PROVIDER_PHASES.extendValidation,
+  PROVIDER_PHASES.resolvePorts,
+  PROVIDER_PHASES.setupEnv,
+  PROVIDER_PHASES.smoke,
+  PROVIDER_PHASES.setupCodingAgent,
+]);
+
+/**
+ * Call a provider hook and return its result. A provider that does not export the phase
+ * is a clean no-op: `{ ran: false }`.
+ * @param {import('../run-wizard.mjs').WizardContext} ctx
+ * @param {string} phase
+ * @param {import('./helpers.mjs').ProviderWizardHelpers} helpers
+ * @returns {Promise<{ ran: boolean, value?: any }>}
+ */
+export async function callProviderHook(ctx, phase, helpers) {
+  const manifest = ctx.memoryProvider;
+  if (!manifest) return { ran: false };
+  const mod = await loadProviderWizard(manifest);
+  const fn = mod[phase];
+  if (typeof fn !== "function") return { ran: false };
+  if (ANNOUNCED.has(phase)) info(`provider ${manifest.id}: ${phase}`);
+  return { ran: true, value: await fn(ctx, helpers) };
+}
+
+/**
+ * @param {import('../run-wizard.mjs').WizardContext} ctx
+ * @param {string} phase
+ * @param {import('./helpers.mjs').ProviderWizardHelpers} helpers
+ * @returns {Promise<boolean>} whether the provider implements the phase
+ */
+export async function runProviderPhase(ctx, phase, helpers) {
+  return (await callProviderHook(ctx, phase, helpers)).ran;
+}
+
+/**
+ * @param {import('../run-wizard.mjs').WizardContext} ctx
+ * @param {string} commandId
+ */
+export async function runDelegatedCommand(ctx, commandId) {
+  const manifest = ctx.memoryProvider;
+  const delegates = manifest?.wizardHooks?.delegates || {};
+  const spec = delegates[commandId];
+  if (!spec?.run?.length) {
+    warn(`provider ${manifest?.id}: no delegate '${commandId}'`);
+    return false;
+  }
+  const [cmd, ...args] = spec.run;
+  info(`delegate ${manifest.id}/${commandId}: ${cmd} ${args.join(" ")}`);
+  if (ctx.dry) {
+    info(`[dry-run] ${cmd} ${args.join(" ")}`);
+    return true;
+  }
+  const r = spawnSync(cmd, args, { stdio: "inherit", env: process.env });
+  if (r.status !== 0) process.exit(r.status ?? 1);
+  return true;
+}
+
+/** @param {import('../providers/registry.mjs').ProviderManifest} manifest @returns {string[]} */
+export function listDelegateCommands(manifest) {
+  return Object.keys(manifest.wizardHooks?.delegates || {});
+}

package/wizard/lib/providers/registry.mjs

@@ -0,0 +1,262 @@
+import fs from "node:fs";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import { die } from "../log.mjs";
+
+const DEFAULT_PACKAGE_ROOT = path.resolve(
+  path.dirname(fileURLToPath(import.meta.url)),
+  "../../..",
+);
+
+/** Provider used when nothing selects one. */
+export const DEFAULT_PROVIDER_ID = "hindsight";
+
+const PROVIDER_ID_RE = /^[A-Za-z0-9][A-Za-z0-9_-]*$/;
+const PROFILE_RE = /^[A-Za-z0-9][A-Za-z0-9_.-]*$/;
+/** Same rule as indexer/sinks/__init__.py NAME: the sink module name and the MEMORY_SINK value. */
+const SINK_NAME_RE = /^[a-z][a-z0-9_]*$/;
+
+/**
+ * Directory that holds `<id>/manifest.json` and `<id>/wizard.mjs`. Read-only code root.
+ * PM_PROVIDERS_DIR (tests, fixtures) beats PM_PACKAGE_ROOT/providers, which beats the
+ * checkout this module lives in.
+ */
+export function providersRoot() {
+  if (process.env.PM_PROVIDERS_DIR) return path.resolve(process.env.PM_PROVIDERS_DIR);
+  return path.join(process.env.PM_PACKAGE_ROOT || DEFAULT_PACKAGE_ROOT, "providers");
+}
+
+/** @typedef {object} ProviderManifest
+ * @property {string} id
+ * @property {string} label
+ * @property {"supported"|"experimental"|"planned"} status
+ * @property {{ services?: string[], healthService?: string|null, healthPath?: string, hostPortEnv?: string, defaultHostPort?: number, profile?: string, files?: string[], logServices?: string[] }} compose
+ *   `profile` adds `--profile <profile>` to every compose call; `files` are extra `-f`
+ *   compose files relative to the stack root, added after compose.yaml.
+ * @property {object} indexer
+ * @property {object} codingAgent
+ * @property {object} wizard
+ * @property {{ module?: string, delegates?: Record<string, object> }} [wizardHooks]
+ */
+
+/** @type {Map<string, ProviderManifest|null>} */
+const cache = new Map();
+
+/**
+ * @param {string} id
+ * @returns {ProviderManifest|null}
+ */
+export function loadManifest(id) {
+  if (!PROVIDER_ID_RE.test(String(id))) return null;
+  const file = path.join(providersRoot(), id, "manifest.json");
+  if (cache.has(file)) return cache.get(file);
+  if (!fs.existsSync(file)) return null;
+  const manifest = JSON.parse(fs.readFileSync(file, "utf8"));
+  cache.set(file, manifest);
+  return manifest;
+}
+
+/** @returns {string[]} */
+export function listProviderIds() {
+  const root = providersRoot();
+  if (!fs.existsSync(root)) return [];
+  return fs
+    .readdirSync(root, { withFileTypes: true })
+    .filter(
+      (d) =>
+        d.isDirectory() &&
+        !d.name.startsWith("_") &&
+        fs.existsSync(path.join(root, d.name, "manifest.json")),
+    )
+    .map((d) => d.name)
+    .sort();
+}
+
+/** @returns {ProviderManifest[]} */
+export function listAllProviders() {
+  return listProviderIds()
+    .map((id) => loadManifest(id))
+    .filter(Boolean);
+}
+
+/** @returns {ProviderManifest[]} */
+export function listSelectableProviders() {
+  return listAllProviders().filter(
+    (m) => m.status === "supported" || m.status === "experimental",
+  );
+}
+
+/** @returns {ProviderManifest[]} */
+export function listPlannedProviders() {
+  return listAllProviders().filter((m) => m.status === "planned");
+}
+
+/**
+ * @param {import("../run-wizard.mjs").WizardContext} ctx
+ */
+export function providerDefaultId(ctx) {
+  const fromConf = ctx.wizardConf && fs.existsSync(ctx.wizardConf)
+    ? readConfKey(ctx.wizardConf, "MEMORY_PROVIDER")
+    : "";
+  const fromToml = ctx.pmMerged
+    ? String(/** @type {Record<string, unknown>} */ (ctx.pmMerged.memory || {}).provider || "")
+    : "";
+  return (
+    process.env.MEMORY_PROVIDER ||
+    process.env.PM_MEMORY_PROVIDER ||
+    fromToml ||
+    fromConf ||
+    DEFAULT_PROVIDER_ID
+  );
+}
+
+function readConfKey(file, key) {
+  const prefix = `${key}=`;
+  for (const line of fs.readFileSync(file, "utf8").split("\n")) {
+    if (line.startsWith(prefix)) return line.slice(prefix.length).trim();
+  }
+  return "";
+}
+
+/**
+ * @param {string} [explicitId]
+ * @param {{ allowPlanned?: boolean }} [opts]
+ * @returns {ProviderManifest}
+ */
+export function resolveProvider(explicitId, opts = {}) {
+  const fromEnv =
+    explicitId ||
+    process.env.MEMORY_PROVIDER ||
+    process.env.PM_MEMORY_PROVIDER ||
+    DEFAULT_PROVIDER_ID;
+  const manifest = loadManifest(fromEnv);
+  if (!manifest) die(`unknown memory provider '${fromEnv}' (see providers/*/manifest.json)`);
+  if (manifest.status === "planned" && !opts.allowPlanned) {
+    die(
+      `memory provider '${fromEnv}' is not implemented yet (${manifest.label}); pick a supported provider in ./setup or see docs/memory-providers.md`,
+    );
+  }
+  return manifest;
+}
+
+/** @param {unknown} v */
+function isStringArray(v) {
+  return Array.isArray(v) && v.every((s) => typeof s === "string" && s.length > 0);
+}
+
+/**
+ * A compose file path from the manifest must stay inside the stack root.
+ * @param {string} f
+ */
+function isStackRelativePath(f) {
+  if (path.isAbsolute(f) || f.includes("\0")) return false;
+  const norm = path.posix.normalize(f.replace(/\\/g, "/"));
+  return norm !== ".." && !norm.startsWith("../") && norm !== ".";
+}
+
+/**
+ * @param {ProviderManifest} manifest
+ * @returns {string[]}
+ */
+export function validateManifest(manifest) {
+  const errors = [];
+  if (!manifest.id) errors.push("missing id");
+  else if (!PROVIDER_ID_RE.test(manifest.id)) errors.push(`invalid id: ${manifest.id}`);
+  if (!manifest.label) errors.push("missing label");
+  if (!["supported", "experimental", "planned"].includes(manifest.status)) {
+    errors.push(`invalid status: ${manifest.status}`);
+  }
+  if (manifest.status === "supported" || manifest.status === "experimental") {
+    const w = manifest.wizard || {};
+    for (const key of [
+      "smokeBank",
+      "retainPath",
+      "recallPath",
+      "deletePath",
+      "retainBody",
+      "recallBody",
+    ]) {
+      if (!w[key]) errors.push(`wizard.${key} required for ${manifest.status}`);
+    }
+    if (!manifest.compose?.healthPath) errors.push("compose.healthPath required");
+    if (!manifest.indexer?.sinkDriver) errors.push("indexer.sinkDriver required");
+    if (!manifest.indexer?.sink) errors.push("indexer.sink required");
+  }
+  const sink = manifest.indexer?.sink;
+  if (sink !== undefined && (typeof sink !== "string" || !SINK_NAME_RE.test(sink))) {
+    errors.push(`indexer.sink must match ${SINK_NAME_RE} (got ${JSON.stringify(sink)})`);
+  }
+
+  const compose = manifest.compose;
+  if (compose !== undefined) {
+    if (compose === null || typeof compose !== "object" || Array.isArray(compose)) {
+      errors.push("compose must be an object");
+    } else {
+      if (compose.profile !== undefined) {
+        if (typeof compose.profile !== "string" || !PROFILE_RE.test(compose.profile)) {
+          errors.push(`compose.profile must match ${PROFILE_RE} (got ${JSON.stringify(compose.profile)})`);
+        }
+      }
+      if (compose.files !== undefined) {
+        if (!isStringArray(compose.files)) {
+          errors.push("compose.files must be an array of non-empty strings");
+        } else {
+          for (const f of compose.files) {
+            if (!isStackRelativePath(f)) {
+              errors.push(`compose.files entry must be a path inside the stack root: ${f}`);
+            }
+          }
+        }
+      }
+      for (const key of ["services", "logServices"]) {
+        if (compose[key] !== undefined && !isStringArray(compose[key])) {
+          errors.push(`compose.${key} must be an array of non-empty strings`);
+        }
+      }
+    }
+  }
+  if (manifest.wizardHooks?.module !== undefined) {
+    const mod = manifest.wizardHooks.module;
+    if (typeof mod !== "string" || !mod || path.isAbsolute(mod) || mod.split(/[\\/]/).includes("..")) {
+      errors.push("wizardHooks.module must be a relative path inside the provider directory");
+    }
+  }
+  if (manifest.wizard?.recallHint !== undefined && typeof manifest.wizard.recallHint !== "string") {
+    errors.push("wizard.recallHint must be a string");
+  }
+  if (manifest.wizard?.retainHint !== undefined && typeof manifest.wizard.retainHint !== "string") {
+    errors.push("wizard.retainHint must be a string");
+  }
+  for (const key of ["smokeRetainTimeoutSec", "smokeRecallTimeoutSec"]) {
+    const v = manifest.wizard?.[key];
+    if (v !== undefined && (typeof v !== "number" || !Number.isFinite(v) || v <= 0)) {
+      errors.push(`wizard.${key} must be a positive number`);
+    }
+  }
+  return errors;
+}
+
+/**
+ * @param {ProviderManifest} provider
+ * @param {string} bank
+ * @param {string} template
+ */
+export function expandPath(provider, bank, template) {
+  if (!template) return "";
+  return template.replace(/\{bank\}/g, encodeURIComponent(bank));
+}
+
+/**
+ * Base URL of the provider's HTTP API on this host, from the manifest's
+ * `compose.hostPortEnv` and `compose.defaultHostPort`. A provider that needs more
+ * (a remote URL, a path prefix) implements the `apiBase` hook instead.
+ * @param {import("../run-wizard.mjs").WizardContext} ctx
+ * @param {ProviderManifest} provider
+ */
+export function providerApiBase(ctx, provider) {
+  if (ctx.apiUrl) return ctx.apiUrl;
+  const portEnv = provider.compose.hostPortEnv;
+  const port =
+    (portEnv && process.env[portEnv]) || String(provider.compose.defaultHostPort ?? 8888);
+  return `http://127.0.0.1:${port}`;
+}

package/wizard/lib/run-clack.mjs

@@ -0,0 +1,108 @@
+import * as p from "@clack/prompts";
+import {
+  listAllProviders,
+  listSelectableProviders,
+  providerDefaultId,
+  resolveProvider,
+} from "./providers/registry.mjs";
+import { actionDefault } from "./wizard-action.mjs";
+import { applyPmConfigEnv, defaultsFromPmConfig, loadPmConfig } from "./pm-config.mjs";
+import { createHelpers } from "./provider-wizard/helpers.mjs";
+import { loadProviderWizard } from "./provider-wizard/run-steps.mjs";
+import path from "node:path";
+
+/**
+ * Provider-neutral Clack setup UI: what to do (configure/up/down) and which provider.
+ * For a configure run the chosen provider's `clackSetup(ctx, ui, helpers)` hook asks its
+ * own questions and returns environment variables for the wizard process. A provider
+ * without the hook adds nothing.
+ *
+ * @param {string} repoRoot
+ * @returns {Promise<Record<string, string>>}
+ */
+export async function runClackSetup(repoRoot) {
+  const home = process.env.HOME || "";
+  const wizardConf = path.join(home, ".config/hindsight/wizard.conf");
+  const ctxStub = { repoRoot, home, wizardConf };
+  const pm = loadPmConfig(home, repoRoot);
+  applyPmConfigEnv(pm.merged);
+  const pmDefaults = defaultsFromPmConfig(pm.merged);
+
+  p.intro("Personal memory setup");
+
+  const actionDefaultVal =
+    pmDefaults.wizardAction && ["configure", "up", "down"].includes(pmDefaults.wizardAction)
+      ? pmDefaults.wizardAction
+      : actionDefault();
+  const action = await p.select({
+    message: "What should setup do?",
+    options: [
+      {
+        value: "configure",
+        label: "Configure and start",
+        hint: "full wizard (default)",
+      },
+      { value: "up", label: "Start stack", hint: "docker compose up, existing config" },
+      { value: "down", label: "Stop stack", hint: "docker compose down" },
+    ],
+    initialValue: actionDefaultVal,
+  });
+  if (p.isCancel(action)) {
+    p.cancel("Setup cancelled.");
+    process.exit(0);
+  }
+
+  const selectable = listSelectableProviders();
+  const all = listAllProviders();
+  const planned = all.filter((m) => m.status === "planned");
+  if (planned.length) {
+    p.note(
+      planned.map((m) => `${m.label} (${m.id}): ${m.status}`).join("\n"),
+      "Planned providers (not selectable yet)",
+    );
+  }
+
+  ctxStub.pmMerged = pm.merged;
+  const providerDefault = providerDefaultId(ctxStub);
+  const providerPick = await p.select({
+    message: "Memory provider",
+    options: selectable.map((m) => ({
+      value: m.id,
+      label: m.label,
+      hint: m.status === "experimental" ? "experimental" : "supported",
+    })),
+    initialValue: selectable.some((m) => m.id === providerDefault)
+      ? providerDefault
+      : selectable[0]?.id,
+  });
+  if (p.isCancel(providerPick)) {
+    p.cancel("Setup cancelled.");
+    process.exit(0);
+  }
+
+  /** @type {Record<string, string>} */
+  const handoff = {
+    PM_WIZARD_ACTION: /** @type {string} */ (action),
+    MEMORY_PROVIDER: /** @type {string} */ (providerPick),
+  };
+
+  if (action === "down") {
+    p.outro("Stopping stack…");
+    return handoff;
+  }
+
+  if (action === "up") {
+    p.outro("Starting stack…");
+    return handoff;
+  }
+
+  const manifest = resolveProvider(/** @type {string} */ (providerPick));
+  const mod = await loadProviderWizard(manifest);
+  if (typeof mod.clackSetup === "function") {
+    const extra = await mod.clackSetup(ctxStub, p, createHelpers());
+    Object.assign(handoff, extra);
+  }
+
+  p.outro("Continuing setup…");
+  return handoff;
+}