claudekit-cli 3.11.0 → 3.11.1

package/dist/index.js

@@ -15466,7 +15466,7 @@ var cac = (name = "") => new CAC(name);
 // package.json
 var package_default = {
   name: "claudekit-cli",
-  version: "3.11.0",
+  version: "3.11.1",
   description: "CLI tool for bootstrapping and updating ClaudeKit projects",
   type: "module",
   repository: {
@@ -28410,7 +28410,7 @@ Solutions:
 }
 
 // src/domains/installation/file-merger.ts
-import { dirname as dirname5, join as join12, relative as relative3 } from "node:path";
+import { dirname as dirname5, join as join13, relative as relative3 } from "node:path";
 
 // src/domains/config/settings-merger.ts
 init_logger();
@@ -28619,7 +28619,7 @@ class SettingsMerger {
 init_logger();
 init_types2();
 init_dist2();
-var import_fs_extra3 = __toESM(require_lib(), 1);
+var import_fs_extra4 = __toESM(require_lib(), 1);
 var import_ignore2 = __toESM(require_ignore(), 1);
 
 // node_modules/@isaacs/balanced-match/dist/esm/index.js
@@ -30065,6 +30065,156 @@ import { createReadStream } from "node:fs";
 import { stat } from "node:fs/promises";
 import { relative as relative2 } from "node:path";
 
+// src/domains/migration/metadata-migration.ts
+init_logger();
+var import_fs_extra3 = __toESM(require_lib(), 1);
+import { join as join12 } from "node:path";
+async function detectMetadataFormat(claudeDir) {
+  const metadataPath = join12(claudeDir, "metadata.json");
+  if (!await import_fs_extra3.pathExists(metadataPath)) {
+    return { format: "none", metadata: null, detectedKit: null };
+  }
+  try {
+    const content = await import_fs_extra3.readFile(metadataPath, "utf-8");
+    const parsed = JSON.parse(content);
+    if (parsed.kits && Object.keys(parsed.kits).length > 0) {
+      const installedKits = Object.keys(parsed.kits);
+      return {
+        format: "multi-kit",
+        metadata: parsed,
+        detectedKit: installedKits[0] || null
+      };
+    }
+    if (parsed.name || parsed.version || parsed.files) {
+      let detectedKit = null;
+      const nameToCheck = parsed.name || "";
+      if (/\bengineer\b/i.test(nameToCheck)) {
+        detectedKit = "engineer";
+      } else if (/\bmarketing\b/i.test(nameToCheck)) {
+        detectedKit = "marketing";
+      } else {
+        detectedKit = "engineer";
+      }
+      return { format: "legacy", metadata: parsed, detectedKit };
+    }
+    logger.warning("Metadata file exists but has unrecognized format (missing kits, name, version, or files)");
+    return { format: "none", metadata: null, detectedKit: null };
+  } catch (error) {
+    logger.warning(`Failed to read metadata file (may be corrupted): ${error}`);
+    return { format: "none", metadata: null, detectedKit: null };
+  }
+}
+async function migrateToMultiKit(claudeDir) {
+  const detection = await detectMetadataFormat(claudeDir);
+  if (detection.format === "multi-kit") {
+    return {
+      success: true,
+      migrated: false,
+      fromFormat: "multi-kit",
+      toFormat: "multi-kit"
+    };
+  }
+  if (detection.format === "none") {
+    return {
+      success: true,
+      migrated: false,
+      fromFormat: "none",
+      toFormat: "multi-kit"
+    };
+  }
+  const metadataPath = join12(claudeDir, "metadata.json");
+  const legacy = detection.metadata;
+  if (!legacy) {
+    return {
+      success: false,
+      migrated: false,
+      fromFormat: "legacy",
+      toFormat: "multi-kit",
+      error: "Metadata exists but could not be read"
+    };
+  }
+  const legacyKit = detection.detectedKit || "engineer";
+  try {
+    const kitMetadata = {
+      version: legacy.version || "unknown",
+      installedAt: legacy.installedAt || new Date().toISOString(),
+      files: legacy.files || []
+    };
+    const multiKit = {
+      kits: {
+        [legacyKit]: kitMetadata
+      },
+      scope: legacy.scope,
+      name: legacy.name,
+      version: legacy.version,
+      installedAt: legacy.installedAt,
+      installedFiles: legacy.installedFiles,
+      userConfigFiles: legacy.userConfigFiles,
+      files: legacy.files
+    };
+    await import_fs_extra3.writeFile(metadataPath, JSON.stringify(multiKit, null, 2), "utf-8");
+    logger.info(`Migrated metadata from legacy format to multi-kit (detected: ${legacyKit})`);
+    return {
+      success: true,
+      migrated: true,
+      fromFormat: "legacy",
+      toFormat: "multi-kit"
+    };
+  } catch (error) {
+    const errorMsg = error instanceof Error ? error.message : "Unknown error";
+    logger.error(`Metadata migration failed: ${errorMsg}`);
+    return {
+      success: false,
+      migrated: false,
+      fromFormat: "legacy",
+      toFormat: "multi-kit",
+      error: errorMsg
+    };
+  }
+}
+function getKitMetadata(metadata, kit) {
+  if (metadata.kits?.[kit]) {
+    return metadata.kits[kit];
+  }
+  if (!metadata.kits && metadata.version) {
+    return {
+      version: metadata.version,
+      installedAt: metadata.installedAt || "",
+      files: metadata.files
+    };
+  }
+  return null;
+}
+function getAllTrackedFiles(metadata) {
+  if (metadata.kits) {
+    const allFiles = [];
+    for (const kit of Object.values(metadata.kits)) {
+      if (kit.files) {
+        allFiles.push(...kit.files);
+      }
+    }
+    return allFiles;
+  }
+  return metadata.files || [];
+}
+function getInstalledKits(metadata) {
+  if (metadata.kits) {
+    return Object.keys(metadata.kits);
+  }
+  const nameToCheck = metadata.name || "";
+  if (/\bengineer\b/i.test(nameToCheck)) {
+    return ["engineer"];
+  }
+  if (/\bmarketing\b/i.test(nameToCheck)) {
+    return ["marketing"];
+  }
+  if (metadata.version) {
+    return ["engineer"];
+  }
+  return [];
+}
+
+// src/services/file-operations/ownership-checker.ts
 class OwnershipChecker {
   static async calculateChecksum(filePath) {
     return new Promise((resolve3, reject) => {
@@ -30086,11 +30236,12 @@ class OwnershipChecker {
     } catch {
       return { path: filePath, ownership: "user", exists: false };
     }
-    if (!metadata || !metadata.files || metadata.files.length === 0) {
+    const allTrackedFiles = metadata ? getAllTrackedFiles(metadata) : [];
+    if (!metadata || allTrackedFiles.length === 0) {
       return { path: filePath, ownership: "user", exists: true };
     }
     const relativePath = relative2(claudeDir, filePath).replace(/\\/g, "/");
-    const tracked = metadata.files.find((f3) => f3.path === relativePath);
+    const tracked = allTrackedFiles.find((f3) => f3.path === relativePath);
     if (!tracked) {
       return { path: filePath, ownership: "user", exists: true };
     }
@@ -30228,8 +30379,8 @@ class FileMerger {
     for (const file of files) {
       const relativePath = relative3(sourceDir, file);
       const normalizedRelativePath = relativePath.replace(/\\/g, "/");
-      const destPath = join12(destDir, relativePath);
-      if (await import_fs_extra3.pathExists(destPath)) {
+      const destPath = join13(destDir, relativePath);
+      if (await import_fs_extra4.pathExists(destPath)) {
         if (this.neverCopyChecker.ignores(normalizedRelativePath)) {
           logger.debug(`Security-sensitive file exists but won't be overwritten: ${normalizedRelativePath}`);
           continue;
@@ -30250,14 +30401,14 @@ class FileMerger {
     for (const file of files) {
       const relativePath = relative3(sourceDir, file);
       const normalizedRelativePath = relativePath.replace(/\\/g, "/");
-      const destPath = join12(destDir, relativePath);
+      const destPath = join13(destDir, relativePath);
       if (this.neverCopyChecker.ignores(normalizedRelativePath)) {
         logger.debug(`Skipping security-sensitive file: ${normalizedRelativePath}`);
         skippedCount++;
         continue;
       }
       if (this.userConfigChecker.ignores(normalizedRelativePath)) {
-        const fileExists = await import_fs_extra3.pathExists(destPath);
+        const fileExists = await import_fs_extra4.pathExists(destPath);
         if (fileExists) {
           logger.debug(`Preserving user config: ${normalizedRelativePath}`);
           skippedCount++;
@@ -30280,7 +30431,7 @@ class FileMerger {
           continue;
         }
       }
-      await import_fs_extra3.copy(file, destPath, { overwrite: true });
+      await import_fs_extra4.copy(file, destPath, { overwrite: true });
       this.trackInstalledFile(normalizedRelativePath);
       copiedCount++;
     }
@@ -30292,7 +30443,7 @@ class FileMerger {
   }
   async processSettingsJson(sourceFile, destFile) {
     try {
-      const sourceContent = await import_fs_extra3.readFile(sourceFile, "utf-8");
+      const sourceContent = await import_fs_extra4.readFile(sourceFile, "utf-8");
       const isWindows5 = process.platform === "win32";
       let transformedSource = sourceContent;
       if (this.isGlobal) {
@@ -30308,19 +30459,19 @@ class FileMerger {
           logger.debug(`Transformed .claude/ paths to ${projectDirVar}/.claude/ in settings.json for local installation`);
         }
       }
-      const destExists = await import_fs_extra3.pathExists(destFile);
+      const destExists = await import_fs_extra4.pathExists(destFile);
       if (destExists && !this.forceOverwriteSettings) {
         await this.selectiveMergeSettings(transformedSource, destFile);
       } else {
         const formattedContent = this.formatJsonContent(transformedSource);
-        await import_fs_extra3.writeFile(destFile, formattedContent, "utf-8");
+        await import_fs_extra4.writeFile(destFile, formattedContent, "utf-8");
         if (this.forceOverwriteSettings && destExists) {
           logger.debug("Force overwrite enabled, replaced settings.json completely");
         }
       }
     } catch (error) {
       logger.error(`Failed to process settings.json: ${error}`);
-      await import_fs_extra3.copy(sourceFile, destFile, { overwrite: true });
+      await import_fs_extra4.copy(sourceFile, destFile, { overwrite: true });
     }
   }
   async selectiveMergeSettings(transformedSourceContent, destFile) {
@@ -30330,7 +30481,7 @@ class FileMerger {
     } catch {
       logger.warning("Failed to parse source settings.json, falling back to overwrite");
       const formattedContent = this.formatJsonContent(transformedSourceContent);
-      await import_fs_extra3.writeFile(destFile, formattedContent, "utf-8");
+      await import_fs_extra4.writeFile(destFile, formattedContent, "utf-8");
       return;
     }
     const destSettings = await SettingsMerger.readSettingsFile(destFile);
@@ -30376,12 +30527,12 @@ class FileMerger {
   }
   async getFiles(dir, baseDir = dir) {
     const files = [];
-    const entries = await import_fs_extra3.readdir(dir, { encoding: "utf8" });
+    const entries = await import_fs_extra4.readdir(dir, { encoding: "utf8" });
     for (const entry of entries) {
-      const fullPath = join12(dir, entry);
+      const fullPath = join13(dir, entry);
       const relativePath = relative3(baseDir, fullPath);
       const normalizedRelativePath = relativePath.replace(/\\/g, "/");
-      const stats = await import_fs_extra3.lstat(fullPath);
+      const stats = await import_fs_extra4.lstat(fullPath);
       if (stats.isSymbolicLink()) {
         logger.warning(`Skipping symbolic link: ${normalizedRelativePath}`);
         continue;
@@ -30439,11 +30590,11 @@ class FileMerger {
 
 // src/domains/installation/fresh-installer.ts
 init_logger();
-import { join as join13 } from "node:path";
-var import_fs_extra4 = __toESM(require_lib(), 1);
+import { join as join14 } from "node:path";
+var import_fs_extra5 = __toESM(require_lib(), 1);
 var CLAUDEKIT_SUBDIRECTORIES = ["commands", "agents", "skills", "workflows", "hooks"];
 async function handleFreshInstallation(claudeDir, prompts) {
-  if (!await import_fs_extra4.pathExists(claudeDir)) {
+  if (!await import_fs_extra5.pathExists(claudeDir)) {
     logger.info(".claude directory does not exist, proceeding with fresh installation");
     return true;
   }
@@ -30458,8 +30609,8 @@ async function handleFreshInstallation(claudeDir, prompts) {
     const { rmSync } = await import("node:fs");
     let removedCount = 0;
     for (const subdir of CLAUDEKIT_SUBDIRECTORIES) {
-      const subdirPath = join13(claudeDir, subdir);
-      if (await import_fs_extra4.pathExists(subdirPath)) {
+      const subdirPath = join14(claudeDir, subdir);
+      if (await import_fs_extra5.pathExists(subdirPath)) {
         rmSync(subdirPath, { recursive: true, force: true });
         removedCount++;
         logger.debug(`Removed subdirectory: ${subdir}/`);
@@ -30474,11 +30625,11 @@ async function handleFreshInstallation(claudeDir, prompts) {
 }
 
 // src/domains/installation/setup-wizard.ts
-import { join as join15 } from "node:path";
+import { join as join16 } from "node:path";
 
 // src/domains/config/config-generator.ts
-var import_fs_extra5 = __toESM(require_lib(), 1);
-import { join as join14 } from "node:path";
+var import_fs_extra6 = __toESM(require_lib(), 1);
+import { join as join15 } from "node:path";
 async function generateEnvFile(targetDir, values) {
   const lines = [
     "# Generated by ClaudeKit CLI setup wizard",
@@ -30490,8 +30641,8 @@ async function generateEnvFile(targetDir, values) {
       lines.push(`${key}=${value}`);
     }
   }
-  const envPath = join14(targetDir, ".env");
-  await import_fs_extra5.writeFile(envPath, `${lines.join(`
+  const envPath = join15(targetDir, ".env");
+  await import_fs_extra6.writeFile(envPath, `${lines.join(`
 `)}
 `, { mode: 384 });
 }
@@ -30509,7 +30660,7 @@ function validateApiKey(value, pattern) {
 // src/domains/installation/setup-wizard.ts
 init_logger();
 init_dist2();
-var import_fs_extra6 = __toESM(require_lib(), 1);
+var import_fs_extra7 = __toESM(require_lib(), 1);
 var ESSENTIAL_CONFIGS = [
   {
     key: "GEMINI_API_KEY",
@@ -30538,7 +30689,7 @@ var ESSENTIAL_CONFIGS = [
 ];
 async function parseEnvFile(path9) {
   try {
-    const content = await import_fs_extra6.readFile(path9, "utf-8");
+    const content = await import_fs_extra7.readFile(path9, "utf-8");
     const env2 = {};
     for (const line of content.split(`
 `)) {
@@ -30564,8 +30715,8 @@ async function parseEnvFile(path9) {
   }
 }
 async function checkGlobalConfig() {
-  const globalEnvPath = join15(PathResolver.getGlobalKitDir(), ".env");
-  if (!await import_fs_extra6.pathExists(globalEnvPath))
+  const globalEnvPath = join16(PathResolver.getGlobalKitDir(), ".env");
+  if (!await import_fs_extra7.pathExists(globalEnvPath))
     return false;
   const env2 = await parseEnvFile(globalEnvPath);
   return Object.keys(env2).length > 0;
@@ -30580,8 +30731,8 @@ async function runSetupWizard(options) {
   let globalEnv = {};
   const hasGlobalConfig = !isGlobal && await checkGlobalConfig();
   if (!isGlobal) {
-    const globalEnvPath = join15(PathResolver.getGlobalKitDir(), ".env");
-    if (await import_fs_extra6.pathExists(globalEnvPath)) {
+    const globalEnvPath = join16(PathResolver.getGlobalKitDir(), ".env");
+    if (await import_fs_extra7.pathExists(globalEnvPath)) {
       globalEnv = await parseEnvFile(globalEnvPath);
     }
   }
@@ -30633,7 +30784,7 @@ async function runSetupWizard(options) {
     }
   }
   await generateEnvFile(targetDir, values);
-  f2.success(`Configuration saved to ${join15(targetDir, ".env")}`);
+  f2.success(`Configuration saved to ${join16(targetDir, ".env")}`);
   return true;
 }
 
@@ -30643,157 +30794,6 @@ import { join as join19, relative as relative4 } from "node:path";
 
 // src/services/file-operations/manifest-writer.ts
 import { join as join17 } from "node:path";
-
-// src/domains/migration/metadata-migration.ts
-init_logger();
-var import_fs_extra7 = __toESM(require_lib(), 1);
-import { join as join16 } from "node:path";
-async function detectMetadataFormat(claudeDir) {
-  const metadataPath = join16(claudeDir, "metadata.json");
-  if (!await import_fs_extra7.pathExists(metadataPath)) {
-    return { format: "none", metadata: null, detectedKit: null };
-  }
-  try {
-    const content = await import_fs_extra7.readFile(metadataPath, "utf-8");
-    const parsed = JSON.parse(content);
-    if (parsed.kits && Object.keys(parsed.kits).length > 0) {
-      const installedKits = Object.keys(parsed.kits);
-      return {
-        format: "multi-kit",
-        metadata: parsed,
-        detectedKit: installedKits[0] || null
-      };
-    }
-    if (parsed.name || parsed.version || parsed.files) {
-      let detectedKit = null;
-      const nameToCheck = parsed.name || "";
-      if (/\bengineer\b/i.test(nameToCheck)) {
-        detectedKit = "engineer";
-      } else if (/\bmarketing\b/i.test(nameToCheck)) {
-        detectedKit = "marketing";
-      } else {
-        detectedKit = "engineer";
-      }
-      return { format: "legacy", metadata: parsed, detectedKit };
-    }
-    logger.warning("Metadata file exists but has unrecognized format (missing kits, name, version, or files)");
-    return { format: "none", metadata: null, detectedKit: null };
-  } catch (error) {
-    logger.warning(`Failed to read metadata file (may be corrupted): ${error}`);
-    return { format: "none", metadata: null, detectedKit: null };
-  }
-}
-async function migrateToMultiKit(claudeDir) {
-  const detection = await detectMetadataFormat(claudeDir);
-  if (detection.format === "multi-kit") {
-    return {
-      success: true,
-      migrated: false,
-      fromFormat: "multi-kit",
-      toFormat: "multi-kit"
-    };
-  }
-  if (detection.format === "none") {
-    return {
-      success: true,
-      migrated: false,
-      fromFormat: "none",
-      toFormat: "multi-kit"
-    };
-  }
-  const metadataPath = join16(claudeDir, "metadata.json");
-  const legacy = detection.metadata;
-  if (!legacy) {
-    return {
-      success: false,
-      migrated: false,
-      fromFormat: "legacy",
-      toFormat: "multi-kit",
-      error: "Metadata exists but could not be read"
-    };
-  }
-  const legacyKit = detection.detectedKit || "engineer";
-  try {
-    const kitMetadata = {
-      version: legacy.version || "unknown",
-      installedAt: legacy.installedAt || new Date().toISOString(),
-      files: legacy.files || []
-    };
-    const multiKit = {
-      kits: {
-        [legacyKit]: kitMetadata
-      },
-      scope: legacy.scope,
-      name: legacy.name,
-      version: legacy.version,
-      installedAt: legacy.installedAt,
-      installedFiles: legacy.installedFiles,
-      userConfigFiles: legacy.userConfigFiles,
-      files: legacy.files
-    };
-    await import_fs_extra7.writeFile(metadataPath, JSON.stringify(multiKit, null, 2), "utf-8");
-    logger.info(`Migrated metadata from legacy format to multi-kit (detected: ${legacyKit})`);
-    return {
-      success: true,
-      migrated: true,
-      fromFormat: "legacy",
-      toFormat: "multi-kit"
-    };
-  } catch (error) {
-    const errorMsg = error instanceof Error ? error.message : "Unknown error";
-    logger.error(`Metadata migration failed: ${errorMsg}`);
-    return {
-      success: false,
-      migrated: false,
-      fromFormat: "legacy",
-      toFormat: "multi-kit",
-      error: errorMsg
-    };
-  }
-}
-function getKitMetadata(metadata, kit) {
-  if (metadata.kits?.[kit]) {
-    return metadata.kits[kit];
-  }
-  if (!metadata.kits && metadata.version) {
-    return {
-      version: metadata.version,
-      installedAt: metadata.installedAt || "",
-      files: metadata.files
-    };
-  }
-  return null;
-}
-function getAllTrackedFiles(metadata) {
-  if (metadata.kits) {
-    const allFiles = [];
-    for (const kit of Object.values(metadata.kits)) {
-      if (kit.files) {
-        allFiles.push(...kit.files);
-      }
-    }
-    return allFiles;
-  }
-  return metadata.files || [];
-}
-function getInstalledKits(metadata) {
-  if (metadata.kits) {
-    return Object.keys(metadata.kits);
-  }
-  const nameToCheck = metadata.name || "";
-  if (/\bengineer\b/i.test(nameToCheck)) {
-    return ["engineer"];
-  }
-  if (/\bmarketing\b/i.test(nameToCheck)) {
-    return ["marketing"];
-  }
-  if (metadata.version) {
-    return ["engineer"];
-  }
-  return [];
-}
-
-// src/services/file-operations/manifest-writer.ts
 init_logger();
 init_types2();
 var import_fs_extra8 = __toESM(require_lib(), 1);
@@ -31054,16 +31054,14 @@ class ManifestWriter {
       };
       const metadata = {
         kits: {
-          ...existingMetadata.kits,
+          ...existingMetadata.kits || {},
           [kit]: kitMetadata
         },
         scope,
         name: kitName,
         version,
         installedAt,
-        installedFiles: this.getInstalledFiles(),
-        userConfigFiles: [...USER_CONFIG_PATTERNS, ...this.getUserConfigFiles()],
-        files: trackedFiles.length > 0 ? trackedFiles : undefined
+        userConfigFiles: [...USER_CONFIG_PATTERNS, ...this.getUserConfigFiles()]
       };
       const validated = MetadataSchema.parse(metadata);
       await import_fs_extra8.writeFile(metadataPath, JSON.stringify(validated, null, 2), "utf-8");
@@ -33353,7 +33351,8 @@ class CommandsPrefix {
       logger.info("Checking ownership before cleanup...");
     }
     const metadata = await ManifestWriter.readManifest(claudeDir);
-    if (!metadata || !metadata.files || metadata.files.length === 0) {
+    const allTrackedFiles = metadata ? getAllTrackedFiles(metadata) : [];
+    if (!metadata || allTrackedFiles.length === 0) {
       logger.verbose("No ownership metadata found - skipping cleanup (legacy/fresh install)");
       logger.verbose("All existing files will be preserved as user-owned");
       return result;
@@ -34704,7 +34703,8 @@ async function analyzeInstallation(installation, forceOverwrite, kit) {
     }
     return result;
   }
-  if (!metadata?.files || metadata.files.length === 0) {
+  const allTrackedFiles = metadata ? getAllTrackedFiles(metadata) : [];
+  if (!metadata || allTrackedFiles.length === 0) {
     for (const item of uninstallManifest.filesToRemove) {
       if (!uninstallManifest.filesToPreserve.includes(item)) {
         result.toDelete.push({ path: item, reason: "legacy installation" });
@@ -34712,7 +34712,7 @@ async function analyzeInstallation(installation, forceOverwrite, kit) {
     }
     return result;
   }
-  for (const trackedFile of metadata.files || []) {
+  for (const trackedFile of allTrackedFiles) {
     const filePath = join34(installation.path, trackedFile.path);
     const ownershipResult = await OwnershipChecker.checkOwnership(filePath, metadata, installation.path);
     if (!ownershipResult.exists)
@@ -35040,7 +35040,7 @@ var import_compare_versions2 = __toESM(require_umd(), 1);
 // package.json
 var package_default2 = {
   name: "claudekit-cli",
-  version: "3.11.0",
+  version: "3.11.1",
   description: "CLI tool for bootstrapping and updating ClaudeKit projects",
   type: "module",
   repository: {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "claudekit-cli",
-	"version": "3.11.0",
+	"version": "3.11.1",
 	"description": "CLI tool for bootstrapping and updating ClaudeKit projects",
 	"type": "module",
 	"repository": {