claudekit-cli 3.38.0-dev.5 → 3.38.0-dev.7

package/bin/ck.js

@@ -84,6 +84,29 @@ const isExpectedBunOnlyRelease = () => {
 };
 
 const shouldWarnForBunFallback = () => !isExpectedBunOnlyRelease();
+const RUNTIME_FATAL_SIGNALS = new Set(["SIGABRT", "SIGBUS", "SIGILL", "SIGSEGV", "SIGTRAP"]);
+
+const handleRuntimeSignalExit = (signal, sourceLabel) => {
+	if (!signal) return;
+	if (!RUNTIME_FATAL_SIGNALS.has(signal)) {
+		process.kill(process.pid, signal);
+		return;
+	}
+
+	console.error(`❌ ${sourceLabel} crashed with ${signal}`);
+	if (signal === "SIGILL") {
+		console.error(
+			"This usually means the bundled executable requires newer CPU instructions than this machine provides.",
+		);
+		console.error(
+			"On Linux x64, install a release that includes the baseline-compatible binary build.",
+		);
+	} else {
+		console.error("The bundled executable crashed before ClaudeKit could finish starting.");
+	}
+	console.error("If this persists, report it at: https://github.com/mrgoonie/claudekit-cli/issues");
+	process.exit(1);
+};
 
 /**
  * Run CLI via bun runtime. Preferred over Node.js when dist/index.js contains
@@ -111,7 +134,7 @@ const runWithBun = (showWarning = false) => {
 		return false;
 	}
 	if (result.signal) {
-		process.kill(process.pid, result.signal);
+		handleRuntimeSignalExit(result.signal, "Bun runtime");
 	}
 	process.exit(result.status || 0);
 };
@@ -216,7 +239,7 @@ const runBinary = (binaryPath) => {
 			if (errorOccurred) return;
 
 			if (signal) {
-				process.kill(process.pid, signal);
+				handleRuntimeSignalExit(signal, "Native binary");
 				return;
 			}
 			// Use exitCode instead of exit() for proper handle cleanup on Windows

package/dist/index.js

@@ -12214,18 +12214,6 @@ function isUnknownChecksum(checksum) {
 var UNKNOWN_CHECKSUM = "unknown";
 
 // src/commands/portable/portable-registry.ts
-var exports_portable_registry = {};
-__export(exports_portable_registry, {
-  writePortableRegistry: () => writePortableRegistry,
-  updateAppliedManifestVersion: () => updateAppliedManifestVersion,
-  syncPortableRegistry: () => syncPortableRegistry,
-  removePortableInstallation: () => removePortableInstallation,
-  removeInstallationsByFilter: () => removeInstallationsByFilter,
-  readPortableRegistry: () => readPortableRegistry,
-  getInstallationsByType: () => getInstallationsByType,
-  findPortableInstallations: () => findPortableInstallations,
-  addPortableInstallation: () => addPortableInstallation
-});
 import { existsSync as existsSync2 } from "node:fs";
 import { mkdir, readFile as readFile2, rename, unlink, writeFile } from "node:fs/promises";
 import { homedir as homedir3 } from "node:os";
@@ -12485,9 +12473,6 @@ function findPortableInstallations(registry, item, type, provider, global2) {
     return true;
   });
 }
-function getInstallationsByType(registry, type) {
-  return registry.installations.filter((i) => i.type === type);
-}
 async function updateAppliedManifestVersion(version) {
   await withRegistryLock(async () => {
     const registry = await readPortableRegistry();
@@ -42433,7 +42418,7 @@ var init_ck_config = __esm(() => {
   }).passthrough();
   UpdatePipelineSchema = exports_external.object({
     autoInitAfterUpdate: exports_external.boolean().default(false),
-    autoMigrateAfterInit: exports_external.boolean().default(false),
+    autoMigrateAfterUpdate: exports_external.boolean().default(false),
     migrateProviders: exports_external.union([exports_external.literal("auto"), exports_external.array(exports_external.string())]).default("auto")
   });
   ResolvedModelConfigSchema = exports_external.object({
@@ -42544,7 +42529,7 @@ var init_ck_config = __esm(() => {
     },
     updatePipeline: {
       autoInitAfterUpdate: false,
-      autoMigrateAfterInit: false,
+      autoMigrateAfterUpdate: false,
       migrateProviders: "auto"
     }
   };
@@ -48552,17 +48537,17 @@ var init_ck_config_schema = __esm(() => {
       },
       updatePipeline: {
         type: "object",
-        description: "Auto-chaining behavior for update -> init -> migrate pipeline",
+        description: "3-step update pipeline: CLI update -> kit init -> provider migrate (each independent)",
         properties: {
           autoInitAfterUpdate: {
             type: "boolean",
             default: false,
             description: "Automatically run ck init after ck update when kit has new version"
           },
-          autoMigrateAfterInit: {
+          autoMigrateAfterUpdate: {
             type: "boolean",
             default: false,
-            description: "Automatically run ck migrate after ck init for detected providers"
+            description: "Automatically run ck migrate after ck update for detected providers (independent of init)"
           },
           migrateProviders: {
             oneOf: [
@@ -57259,7 +57244,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "claudekit-cli",
-    version: "3.38.0-dev.5",
+    version: "3.38.0-dev.7",
     description: "CLI tool for bootstrapping and updating ClaudeKit projects",
     type: "module",
     repository: {
@@ -58554,7 +58539,8 @@ function getDefaultUpdateCliCommandDeps() {
     execAsyncFn: execAsync2,
     packageManagerDetector: PackageManagerDetector,
     npmRegistryClient: NpmRegistryClient,
-    promptKitUpdateFn: promptKitUpdate
+    promptKitUpdateFn: promptKitUpdate,
+    promptMigrateUpdateFn: promptMigrateUpdate
   };
 }
 function extractCommandStdout(result) {
@@ -58679,14 +58665,14 @@ async function promptKitUpdate(beta, yes, deps) {
     if (selection.kit && kitVersion) {
       logger.info(`Current kit version: ${selection.kit}@${kitVersion}`);
     }
+    let alreadyAtLatest = false;
     if (yes && selection.kit && kitVersion) {
       const getTagFn = deps?.getLatestReleaseTagFn ?? fetchLatestReleaseTag;
       const latestTag = await getTagFn(selection.kit, beta || isBetaInstalled);
       if (latestTag && versionsMatch(kitVersion, latestTag)) {
         logger.success(`Already at latest version (${selection.kit}@${kitVersion}), skipping reinstall`);
-        return;
-      }
-      if (latestTag) {
+        alreadyAtLatest = true;
+      } else if (latestTag) {
         logger.info(`Kit update available: ${kitVersion} -> ${latestTag}`);
       }
     }
@@ -58695,6 +58681,9 @@ async function promptKitUpdate(beta, yes, deps) {
       const ckConfig = await loadFullConfigFn(null);
       autoInit = ckConfig.config.updatePipeline?.autoInitAfterUpdate ?? false;
     } catch {}
+    if (alreadyAtLatest && !autoInit) {
+      return;
+    }
     if (!yes && !autoInit) {
       logger.info("");
       const shouldUpdate = await confirmFn({
@@ -58743,6 +58732,77 @@ async function promptKitUpdate(beta, yes, deps) {
     logger.verbose(`Failed to prompt for kit update: ${error instanceof Error ? error.message : "unknown error"}`);
   }
 }
+async function promptMigrateUpdate(deps) {
+  try {
+    const providerRegistry = deps?.detectInstalledProvidersFn && deps?.getProviderConfigFn ? null : await Promise.resolve().then(() => (init_provider_registry(), exports_provider_registry));
+    const detectFn = deps?.detectInstalledProvidersFn ?? providerRegistry?.detectInstalledProviders;
+    const getConfigFn = deps?.getProviderConfigFn ?? providerRegistry?.getProviderConfig;
+    const getSetupFn = deps?.getSetupFn ?? getClaudeKitSetup;
+    const loadFullConfigFn = deps?.loadFullConfigFn ?? CkConfigManager.loadFull;
+    const execFn = deps?.execAsyncFn ?? execAsync2;
+    if (!detectFn || !getConfigFn)
+      return;
+    const allProviders = await detectFn();
+    const targets = allProviders.filter((p) => p !== "claude-code");
+    if (targets.length === 0) {
+      logger.verbose("No migration targets detected, skipping migrate step");
+      return;
+    }
+    let autoMigrate = false;
+    let migrateProviders = "auto";
+    try {
+      const ckConfig = await loadFullConfigFn(null);
+      const pipeline = ckConfig.config.updatePipeline;
+      autoMigrate = pipeline?.autoMigrateAfterUpdate ?? false;
+      migrateProviders = pipeline?.migrateProviders ?? "auto";
+    } catch {}
+    if (!autoMigrate)
+      return;
+    let providers2;
+    if (migrateProviders === "auto") {
+      providers2 = targets;
+    } else if (Array.isArray(migrateProviders)) {
+      const invalid = migrateProviders.filter((p) => !targets.includes(p));
+      if (invalid.length > 0) {
+        logger.warning(`Unknown/uninstalled providers in migrateProviders: ${invalid.join(", ")}`);
+      }
+      providers2 = migrateProviders.filter((p) => targets.includes(p));
+    } else {
+      return;
+    }
+    if (providers2.length === 0)
+      return;
+    const safeProviders = providers2.filter((p) => SAFE_PROVIDER_NAME.test(p));
+    if (safeProviders.length !== providers2.length) {
+      logger.warning("Some provider names contain invalid characters and were skipped");
+    }
+    if (safeProviders.length === 0)
+      return;
+    let isGlobal = false;
+    try {
+      const setup = await getSetupFn();
+      isGlobal = !!setup.global.metadata && !setup.project.metadata;
+    } catch {}
+    const providerNames = safeProviders.map((p) => getConfigFn(p).displayName).join(", ");
+    const parts = ["ck", "migrate"];
+    if (isGlobal)
+      parts.push("-g");
+    for (const p of safeProviders) {
+      parts.push("--agent", p);
+    }
+    parts.push("--yes");
+    const cmd = parts.join(" ");
+    logger.info(`Auto-migrating to: ${providerNames}`);
+    try {
+      await execFn(cmd, { timeout: 300000 });
+      logger.success("Auto-migration complete");
+    } catch (error) {
+      logger.warning(`Auto-migration failed: ${error instanceof Error ? error.message : "unknown"}. Run \`ck migrate\` manually to retry.`);
+    }
+  } catch (error) {
+    logger.verbose(`Migrate step skipped: ${error instanceof Error ? error.message : "unknown"}`);
+  }
+}
 async function updateCliCommand(options2, deps = getDefaultUpdateCliCommandDeps()) {
   const s = de();
   intro("[>] ClaudeKit CLI - Update");
@@ -58752,7 +58812,8 @@ async function updateCliCommand(options2, deps = getDefaultUpdateCliCommandDeps(
       execAsyncFn,
       packageManagerDetector,
       npmRegistryClient,
-      promptKitUpdateFn
+      promptKitUpdateFn,
+      promptMigrateUpdateFn
     } = deps;
     const opts = UpdateCliOptionsSchema.parse(options2);
     logger.info(`Current CLI version: ${currentVersion}`);
@@ -58813,12 +58874,14 @@ async function updateCliCommand(options2, deps = getDefaultUpdateCliCommandDeps(
     if (comparison === 0) {
       outro(`[+] Already on the latest CLI version (${currentVersion})`);
       await promptKitUpdateFn(targetIsPrerelease, opts.yes);
+      await promptMigrateUpdateFn();
       return;
     }
     const isDevChannelSwitch = (opts.dev || opts.beta) && isBetaVersion(targetVersion) && !isBetaVersion(currentVersion);
     if (comparison > 0 && !opts.release && !isDevChannelSwitch) {
       outro(`[+] Current version (${currentVersion}) is newer than latest (${targetVersion})`);
       await promptKitUpdateFn(targetIsPrerelease, opts.yes);
+      await promptMigrateUpdateFn();
       return;
     }
     const isUpgrade = comparison < 0 || isDevChannelSwitch;
@@ -58829,6 +58892,7 @@ async function updateCliCommand(options2, deps = getDefaultUpdateCliCommandDeps(
 
 Run 'ck update' to install`, "Update Check");
       await promptKitUpdateFn(targetIsPrerelease, opts.yes);
+      await promptMigrateUpdateFn();
       outro("Check complete");
       return;
     }
@@ -58895,6 +58959,7 @@ Run '${redactCommandForLog(updateCmd)}' manually, restart terminal, then check c
       }
       outro(`[+] Successfully updated ClaudeKit CLI to ${activeVersion}`);
       await promptKitUpdateFn(targetIsPrerelease, opts.yes);
+      await promptMigrateUpdateFn();
     } catch (error) {
       if (error instanceof CliUpdateError) {
         throw error;
@@ -58916,7 +58981,7 @@ Manual update: ${redactCommandForLog(updateCmd)}`;
     throw new CliUpdateError(errorMessage);
   }
 }
-var import_compare_versions3, import_fs_extra5, execAsync2, CliUpdateError;
+var import_compare_versions3, import_fs_extra5, execAsync2, CliUpdateError, SAFE_PROVIDER_NAME;
 var init_update_cli = __esm(() => {
   init_ck_config_manager();
   init_npm_registry();
@@ -58939,6 +59004,7 @@ var init_update_cli = __esm(() => {
       this.name = "CliUpdateError";
     }
   };
+  SAFE_PROVIDER_NAME = /^[a-z0-9-]+$/;
 });
 
 // src/domains/versioning/version-cache.ts
@@ -96121,116 +96187,6 @@ import { join as join109 } from "node:path";
 init_logger();
 init_path_resolver();
 var import_fs_extra30 = __toESM(require_lib3(), 1);
-
-// src/commands/init/phases/post-init-migrate-nudge.ts
-init_ck_config_manager();
-init_logger();
-init_safe_prompts();
-import { exec as exec8 } from "node:child_process";
-import { promisify as promisify14 } from "node:util";
-var execAsync8 = promisify14(exec8);
-var SAFE_PROVIDER_NAME = /^[a-z0-9-]+$/;
-async function maybePostInitMigrate(ctx, deps) {
-  if (ctx.cancelled || !ctx.resolvedDir)
-    return;
-  try {
-    const providerRegistry = deps?.detectInstalledProvidersFn && deps?.getProviderConfigFn ? null : await Promise.resolve().then(() => (init_provider_registry(), exports_provider_registry));
-    const portableRegistry = deps?.readPortableRegistryFn ? null : await Promise.resolve().then(() => (init_portable_registry(), exports_portable_registry));
-    const detectInstalledProvidersFn = deps?.detectInstalledProvidersFn ?? providerRegistry?.detectInstalledProviders;
-    const getProviderConfigFn = deps?.getProviderConfigFn ?? providerRegistry?.getProviderConfig;
-    const readPortableRegistryFn = deps?.readPortableRegistryFn ?? portableRegistry?.readPortableRegistry;
-    const loadFullConfigFn = deps?.loadFullConfigFn ?? CkConfigManager.loadFull;
-    if (!detectInstalledProvidersFn || !getProviderConfigFn || !readPortableRegistryFn) {
-      return;
-    }
-    const allProviders = await detectInstalledProvidersFn();
-    const targets = allProviders.filter((p) => p !== "claude-code");
-    if (targets.length === 0)
-      return;
-    const providerNames = targets.map((p) => getProviderConfigFn(p).displayName).join(", ");
-    const registry = await readPortableRegistryFn();
-    const hasHistory = registry.installations.some((i) => i.provider !== "claude-code");
-    const ckConfig = await loadFullConfigFn(ctx.options.global ? null : ctx.resolvedDir);
-    const pipeline = ckConfig.config.updatePipeline;
-    const autoMigrate = pipeline?.autoMigrateAfterInit ?? false;
-    if (autoMigrate) {
-      await runAutoMigrate(ctx, pipeline, targets, providerNames, deps);
-    } else if (!hasHistory && !ctx.isNonInteractive) {
-      await showNudge(ctx, providerNames, deps);
-    }
-  } catch (error) {
-    logger.debug(`Post-init migrate check skipped: ${error instanceof Error ? error.message : "unknown"}`);
-  }
-}
-async function showNudge(ctx, providerNames, deps) {
-  const noteFn = deps?.noteFn ?? note;
-  const confirmFn = deps?.confirmFn ?? se;
-  const isCancelFn = deps?.isCancelFn ?? lD;
-  const execAsyncFn = deps?.execAsyncFn ?? execAsync8;
-  noteFn([
-    `Detected providers: ${providerNames}`,
-    "Run `ck migrate` to sync your kit to these providers.",
-    "Set `autoMigrateAfterInit: true` in .ck.json to auto-sync on future updates."
-  ].join(`
-`), "[i] Provider Sync Available");
-  const shouldMigrate = await confirmFn({
-    message: "Run ck migrate now?"
-  });
-  if (isCancelFn(shouldMigrate) || !shouldMigrate)
-    return;
-  const parts = ["ck", "migrate"];
-  if (ctx.options.global)
-    parts.push("-g");
-  parts.push("--yes");
-  const cmd = parts.join(" ");
-  try {
-    logger.info(`Running: ${cmd}`);
-    await execAsyncFn(cmd, { timeout: 300000 });
-    logger.success("Migration complete");
-  } catch (error) {
-    logger.warning(`Migration failed: ${error instanceof Error ? error.message : "unknown"}. Run \`ck migrate\` manually to retry.`);
-  }
-}
-async function runAutoMigrate(ctx, pipeline, detectedTargets, providerNames, deps) {
-  const execAsyncFn = deps?.execAsyncFn ?? execAsync8;
-  let providers2;
-  if (!pipeline?.migrateProviders || pipeline.migrateProviders === "auto") {
-    providers2 = detectedTargets;
-  } else if (Array.isArray(pipeline.migrateProviders)) {
-    const invalid = pipeline.migrateProviders.filter((p) => !detectedTargets.includes(p));
-    if (invalid.length > 0) {
-      logger.warning(`Unknown/uninstalled providers in migrateProviders: ${invalid.join(", ")}`);
-    }
-    providers2 = pipeline.migrateProviders.filter((p) => detectedTargets.includes(p));
-  } else {
-    return;
-  }
-  if (providers2.length === 0)
-    return;
-  const safeProviders = providers2.filter((p) => SAFE_PROVIDER_NAME.test(p));
-  if (safeProviders.length !== providers2.length) {
-    logger.warning("Some provider names contain invalid characters and were skipped");
-  }
-  if (safeProviders.length === 0)
-    return;
-  const parts = ["ck", "migrate"];
-  if (ctx.options.global)
-    parts.push("-g");
-  for (const p of safeProviders) {
-    parts.push("--agent", p);
-  }
-  parts.push("--yes");
-  const cmd = parts.join(" ");
-  logger.info(`Auto-migrating to: ${providerNames}`);
-  try {
-    await execAsyncFn(cmd, { timeout: 300000 });
-    logger.success("Auto-migration complete");
-  } catch (error) {
-    logger.warning(`Auto-migration failed: ${error instanceof Error ? error.message : "unknown"}. Run \`ck migrate\` manually to retry.`);
-  }
-}
-
-// src/commands/init/phases/post-install-handler.ts
 async function handlePostInstall(ctx) {
   if (ctx.cancelled || !ctx.extractDir || !ctx.resolvedDir || !ctx.claudeDir) {
     return ctx;
@@ -96300,7 +96256,6 @@ async function handlePostInstall(ctx) {
       logger.debug(`Project auto-registration skipped: ${error instanceof Error ? error.message : "Unknown error"}`);
     }
   }
-  await maybePostInitMigrate(ctx);
   return {
     ...ctx,
     installSkills
@@ -96341,9 +96296,9 @@ async function detectAccessibleKits() {
 
 // src/domains/github/preflight-checker.ts
 init_logger();
-import { exec as exec9 } from "node:child_process";
-import { promisify as promisify15 } from "node:util";
-var execAsync9 = promisify15(exec9);
+import { exec as exec8 } from "node:child_process";
+import { promisify as promisify14 } from "node:util";
+var execAsync8 = promisify14(exec8);
 function createSuccessfulPreflightResult() {
   return {
     success: true,
@@ -96376,7 +96331,7 @@ async function runPreflightChecks() {
     errorLines: []
   };
   try {
-    const { stdout: stdout2 } = await execAsync9("gh --version", { timeout: GH_COMMAND_TIMEOUT_MS });
+    const { stdout: stdout2 } = await execAsync8("gh --version", { timeout: GH_COMMAND_TIMEOUT_MS });
     const match2 = stdout2.match(/(\d+\.\d+\.\d+)/);
     if (!match2) {
       logger.debug(`GitHub CLI version not detected from output: ${stdout2.trim()}`);
@@ -96414,7 +96369,7 @@ async function runPreflightChecks() {
     }
   }
   try {
-    await execAsync9("gh auth status -h github.com", {
+    await execAsync8("gh auth status -h github.com", {
       timeout: GH_COMMAND_TIMEOUT_MS,
       env: { ...process.env, GH_NO_UPDATE_NOTIFIER: "1" }
     });