claudekit-cli 3.36.0-dev.15 → 3.36.0-dev.17

package/dist/index.js

@@ -12378,6 +12378,30 @@ function findPortableInstallations(registry, item, type, provider, global2) {
     return true;
   });
 }
+async function updateAppliedManifestVersion(version) {
+  await withRegistryLock(async () => {
+    const registry = await readPortableRegistry();
+    registry.appliedManifestVersion = version;
+    await writePortableRegistry(registry);
+  });
+}
+async function removeInstallationsByFilter(predicate) {
+  return withRegistryLock(async () => {
+    const registry = await readPortableRegistry();
+    const removed = [];
+    registry.installations = registry.installations.filter((entry) => {
+      if (predicate(entry)) {
+        removed.push(entry);
+        return false;
+      }
+      return true;
+    });
+    if (removed.length > 0) {
+      await writePortableRegistry(registry);
+    }
+    return removed;
+  });
+}
 async function syncPortableRegistry() {
   return withRegistryLock(async () => {
     const registry = await readPortableRegistry();
@@ -12934,8 +12958,87 @@ async function installCodexToml(items, provider, portableType, options2) {
     };
   }
 }
+async function cleanupStaleCodexConfigEntries(options2) {
+  const config = providers[options2.provider];
+  const pathConfig = config.agents;
+  if (!pathConfig)
+    return [];
+  const basePath = options2.global ? pathConfig.globalPath : pathConfig.projectPath;
+  if (!basePath)
+    return [];
+  const agentsDir = resolve(basePath);
+  const configTomlPath = join3(dirname2(agentsDir), "config.toml");
+  if (!existsSync3(configTomlPath))
+    return [];
+  try {
+    return await withCodexTargetLock(configTomlPath, async () => {
+      let content = await readFile3(configTomlPath, "utf-8");
+      const allStaleSlugs = [];
+      const managedEntries = extractManagedAgentEntries(content);
+      if (managedEntries.size > 0) {
+        const validEntries = new Map;
+        for (const [slug, entry] of managedEntries) {
+          const tomlPath = join3(agentsDir, `${slug}.toml`);
+          if (existsSync3(tomlPath)) {
+            validEntries.set(slug, entry);
+          } else {
+            allStaleSlugs.push(slug);
+          }
+        }
+        if (allStaleSlugs.length > 0) {
+          if (validEntries.size === 0) {
+            const analysis2 = analyzeConfigToml(content);
+            content = analysis2.unmanagedContent;
+          } else {
+            const sortedEntries = [...validEntries.entries()].sort(([a3], [b3]) => a3.localeCompare(b3)).map(([, entry]) => entry);
+            const managedBlock = sortedEntries.join(`
+
+`);
+            const mergeResult = mergeConfigTomlWithDiagnostics(content, managedBlock);
+            if (mergeResult.error) {
+              logger.verbose(`[codex-cleanup] Phase 1 merge failed: ${mergeResult.error}`);
+            } else {
+              content = mergeResult.content;
+            }
+          }
+        }
+      }
+      const analysis = analyzeConfigToml(content);
+      const unmanagedSlugs = extractUnmanagedAgentSlugs(analysis.unmanagedContent);
+      const legacyStaleSlugs = [];
+      for (const slug of unmanagedSlugs) {
+        const tomlPath = join3(agentsDir, `${slug}.toml`);
+        if (!isPathWithinBoundary(tomlPath, agentsDir))
+          continue;
+        if (!existsSync3(tomlPath)) {
+          legacyStaleSlugs.push(slug);
+        }
+      }
+      if (legacyStaleSlugs.length > 0) {
+        for (const slug of legacyStaleSlugs) {
+          const escapedSlug = slug.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+          const blockRegex = new RegExp(`\\n?^\\[agents\\.(?:"${escapedSlug}"|${escapedSlug})\\]\\s*\\r?\\n(?:(?!\\[)[^\\r\\n]*\\r?\\n?)*`, "gm");
+          content = content.replace(blockRegex, "");
+        }
+        content = content.replace(/\n{3,}/g, `
+
+`);
+        allStaleSlugs.push(...legacyStaleSlugs);
+      }
+      if (allStaleSlugs.length === 0)
+        return [];
+      await writeFile2(configTomlPath, content, "utf-8");
+      logger.verbose(`[codex-cleanup] Removed ${allStaleSlugs.length} stale config.toml entries: ${allStaleSlugs.join(", ")}`);
+      return allStaleSlugs;
+    });
+  } catch (error) {
+    logger.verbose(`[codex-cleanup] Failed to clean up config.toml: ${error instanceof Error ? error.message : "Unknown"}`);
+    return [];
+  }
+}
 var import_proper_lockfile2, SENTINEL_START = "# --- ck-managed-agents-start ---", SENTINEL_END = "# --- ck-managed-agents-end ---", MAX_WINDOWS_PATH_LENGTH = 240;
 var init_codex_toml_installer = __esm(() => {
+  init_logger();
   init_checksum_utils();
   init_fm_to_codex_toml();
   init_converters();
@@ -52083,6 +52186,35 @@ function registerMigrationRoutes(app) {
           deleteResult.itemName = deleteAction.item;
           allResults.push(deleteResult);
         }
+        for (const provider of allPlanProviders) {
+          if (providers[provider]?.agents?.writeStrategy !== "codex-toml")
+            continue;
+          const providerScopes = [
+            ...new Set(plan.actions.filter((a3) => a3.provider === provider).map((a3) => a3.global).filter((g2) => g2 !== undefined))
+          ];
+          if (providerScopes.length === 0)
+            providerScopes.push(false);
+          for (const scope of providerScopes) {
+            const staleSlugs = await cleanupStaleCodexConfigEntries({
+              global: scope,
+              provider
+            });
+            if (staleSlugs.length > 0) {
+              const staleSlugSet = new Set(staleSlugs.map((s) => `${s}.toml`));
+              await removeInstallationsByFilter((i) => i.type === "agent" && i.provider === provider && i.global === scope && staleSlugSet.has(basename7(i.path)));
+            }
+          }
+        }
+        try {
+          const agentSrc = getAgentSourcePath();
+          const cmdSrc = getCommandSourcePath();
+          const skillSrc = getSkillSourcePath();
+          const kitRoot = (agentSrc ? resolve8(agentSrc, "..") : null) ?? (cmdSrc ? resolve8(cmdSrc, "..") : null) ?? (skillSrc ? resolve8(skillSrc, "..") : null) ?? null;
+          const manifest = kitRoot ? await loadPortableManifest(kitRoot) : null;
+          if (manifest?.cliVersion) {
+            await updateAppliedManifestVersion(manifest.cliVersion);
+          }
+        } catch {}
         const sortedResults2 = sortPortableInstallResults(allResults);
         const counts2 = toExecutionCounts(sortedResults2);
         const planScopes = [
@@ -52260,6 +52392,7 @@ var init_migration_routes = __esm(() => {
   init_commands_discovery();
   init_skill_directory_installer();
   init_checksum_utils();
+  init_codex_toml_installer();
   init_config_discovery();
   init_hooks_settings_merger();
   init_portable_installer();
@@ -56009,7 +56142,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "claudekit-cli",
-    version: "3.36.0-dev.15",
+    version: "3.36.0-dev.17",
     description: "CLI tool for bootstrapping and updating ClaudeKit projects",
     type: "module",
     repository: {
@@ -92344,6 +92477,7 @@ init_logger();
 init_agents_discovery();
 init_commands_discovery();
 init_checksum_utils();
+init_codex_toml_installer();
 init_config_discovery();
 
 // src/commands/portable/conflict-resolver.ts
@@ -92665,6 +92799,7 @@ function displayMigrationSummary(plan, results, options2) {
 
 // src/commands/migrate/migrate-command.ts
 init_portable_installer();
+init_portable_manifest();
 init_portable_registry();
 init_provider_registry();
 init_reconciler();
@@ -93166,6 +93301,32 @@ async function migrateCommand(options2) {
         }
       }
     }
+    for (const provider of selectedProviders) {
+      const providerConfig = providers[provider];
+      if (providerConfig.agents?.writeStrategy !== "codex-toml")
+        continue;
+      const staleSlugs = await cleanupStaleCodexConfigEntries({
+        global: installGlobally,
+        provider
+      });
+      if (staleSlugs.length > 0) {
+        const staleSlugSet = new Set(staleSlugs.map((s) => `${s}.toml`));
+        const removed = await removeInstallationsByFilter((i) => i.type === "agent" && i.provider === provider && i.global === installGlobally && staleSlugSet.has(basename16(i.path)));
+        for (const entry of removed) {
+          logger.verbose(`[migrate] Cleaned stale registry entry: ${entry.item} (${provider})`);
+        }
+      }
+    }
+    try {
+      const kitRoot = (agentSource ? resolve24(agentSource, "..") : null) ?? (commandSource ? resolve24(commandSource, "..") : null) ?? (skillSource ? resolve24(skillSource, "..") : null) ?? null;
+      const manifest = kitRoot ? await loadPortableManifest(kitRoot) : null;
+      if (manifest?.cliVersion) {
+        await updateAppliedManifestVersion(manifest.cliVersion);
+        logger.verbose(`[migrate] Updated appliedManifestVersion to ${manifest.cliVersion}`);
+      }
+    } catch {
+      logger.debug("[migrate] Failed to update appliedManifestVersion — will retry on next run");
+    }
     installSpinner.stop("Migrate complete");
     displayMigrationSummary(plan, allResults, { color: useColor });
     const failed = allResults.filter((r2) => !r2.success);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "claudekit-cli",
-	"version": "3.36.0-dev.15",
+	"version": "3.36.0-dev.17",
 	"description": "CLI tool for bootstrapping and updating ClaudeKit projects",
 	"type": "module",
 	"repository": {