@cregis-dev/cckit 0.4.0 → 0.4.1

package/src/commands/sync.js

@@ -1,325 +1,325 @@
-/**
- * `cckit sync` command — pull upstream BMAD/ECC/ext-skills content into templates/.
- *
- * Workflow:
- *   1. Validate --source parameter
- *   2. Snapshot current templates/ directory
- *   3. If --dry-run: report intended operations and exit
- *   4. Fetch upstream sources (BMAD via npx, ECC via git clone, ext-skills via git clone)
- *   5. Apply path mappings to copy files into templates/
- *   6. Post-snapshot and compute diff
- *   7. Update registry.json upstream metadata
- *   8. Print diff report
- */
-
-import path from 'node:path'
-import fse from 'fs-extra'
-import { fileURLToPath } from 'node:url'
-import { createLogger } from '../utils/logger.js'
-import { loadRegistry } from '../core/registry.js'
-import { snapshotDir, computeDiff } from '../core/differ.js'
-import { shouldTemplatize, templatizeConfigYaml } from '../core/templatize.js'
-import {
-  fetchBmad,
-  fetchEcc,
-  fetchPlugins,
-  fetchExtSkills,
-  buildPluginMappings,
-  buildExtSkillMappings,
-  applyMappings,
-  createTempDir,
-  BMAD_PATH_MAPPINGS,
-  ECC_PATH_MAPPINGS,
-  ECC_REPO_URL,
-} from '../core/upstream.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const TEMPLATES_DIR = path.resolve(__dirname, '../../templates')
-const REGISTRY_PATH = path.resolve(__dirname, '../../registry.json')
-
-const VALID_SOURCES = ['bmad', 'ecc', 'plugins', 'ext-skills', 'all']
-
-/**
- * Run the sync command.
- *
- * @param {object} opts - Command options
- * @param {string} [opts.source='all'] - Source to sync: bmad, ecc, plugins, ext-skills, or all
- * @param {boolean} [opts.dryRun=false] - Preview mode
- * @param {boolean} [opts.debug=false] - Debug output
- * @param {object} [opts._deps] - Injectable dependencies for testing
- */
-export async function runSync(opts = {}) {
-  const {
-    source = 'all',
-    dryRun = false,
-    debug = false,
-    _deps = {},
-  } = opts
-
-  const logger = _deps.logger || createLogger({ debug })
-  const templatesDir = _deps.templatesDir || TEMPLATES_DIR
-  const registryPath = _deps.registryPath || REGISTRY_PATH
-  const fetchBmadFn = _deps.fetchBmad || fetchBmad
-  const fetchEccFn = _deps.fetchEcc || fetchEcc
-  const fetchPluginsFn = _deps.fetchPlugins || fetchPlugins
-  const fetchExtSkillsFn = _deps.fetchExtSkills || fetchExtSkills
-
-  // Step 1: Validate source
-  if (!VALID_SOURCES.includes(source)) {
-    throw new Error(`Invalid source "${source}". Must be one of: ${VALID_SOURCES.join(', ')}`)
-  }
-
-  const syncBmad = source === 'bmad' || source === 'all'
-  const syncEcc = source === 'ecc' || source === 'all'
-  const syncPlugins = source === 'plugins' || source === 'all'
-  const syncExtSkills = source === 'ext-skills' || source === 'all'
-
-  logger.banner('cckit sync')
-  logger.info(`  Source: ${source}`)
-
-  // Step 2: Snapshot before
-  logger.debug('Snapshotting templates directory...')
-  const beforeSnapshot = await snapshotDir(templatesDir)
-  logger.debug(`  Before snapshot: ${beforeSnapshot.size} files`)
-
-  // Step 3: Dry-run check
-  if (dryRun) {
-    logger.info('')
-    logger.info('  Dry-run mode — no changes will be applied.')
-    logger.info('')
-    if (syncBmad) {
-      logger.info('  Would sync BMAD:')
-      for (const m of BMAD_PATH_MAPPINGS) {
-        logger.info(`    ${m.src} → templates/${m.dest}`)
-      }
-    }
-    if (syncEcc) {
-      logger.info('  Would sync ECC:')
-      for (const m of ECC_PATH_MAPPINGS) {
-        logger.info(`    ${m.src} → templates/${m.dest}`)
-      }
-    }
-    if (syncPlugins) {
-      const registry = await loadRegistry(registryPath)
-      const pluginMappings = buildPluginMappings(registry)
-      logger.info('  Would sync Plugins:')
-      for (const m of pluginMappings) {
-        logger.info(`    ${m.src} → templates/${m.dest}`)
-      }
-    }
-    if (syncExtSkills) {
-      const registry = await loadRegistry(registryPath)
-      const extSkillMappings = buildExtSkillMappings(registry)
-      logger.info('  Would sync Ext-skills:')
-      for (const m of extSkillMappings) {
-        logger.info(`    ${m.src} → templates/${m.dest}`)
-      }
-    }
-    return { dryRun: true }
-  }
-
-  // Step 4 & 5: Fetch and apply
-  let tmpDir
-  const upstreamMeta = {}
-
-  try {
-    tmpDir = await createTempDir('cckit-sync')
-    logger.debug(`Temp directory: ${tmpDir}`)
-
-    // Pre-check plugins and ext-skills upstream availability to compute accurate step count
-    let pluginsAvailable = false
-    if (syncPlugins) {
-      const reg = await loadRegistry(registryPath)
-      pluginsAvailable = Boolean(reg.upstream?.plugins?.repo)
-      if (!pluginsAvailable) {
-        logger.warn('No plugins upstream configured in registry.json, skipping plugins sync')
-      }
-    }
-
-    let extSkillsAvailable = false
-    if (syncExtSkills) {
-      const reg = await loadRegistry(registryPath)
-      const extEntries = reg.upstream?.['ext-skills'] || {}
-      extSkillsAvailable = Object.keys(extEntries).length > 0
-      if (!extSkillsAvailable) {
-        logger.warn('No ext-skills upstream configured in registry.json, skipping ext-skills sync')
-      }
-    }
-
-    const totalSteps = [syncBmad, syncEcc, pluginsAvailable, extSkillsAvailable].filter(Boolean).length
-    let currentStep = 0
-
-    if (syncBmad) {
-      currentStep++
-      logger.step(currentStep, totalSteps, 'Fetching BMAD...')
-      const bmad = await fetchBmadFn({ tmpDir, logger })
-      upstreamMeta.bmad = { version: bmad.version }
-
-      const postProcess = (content, relPath) => {
-        if (shouldTemplatize(relPath)) {
-          return templatizeConfigYaml(content)
-        }
-        return content
-      }
-
-      const fileCount = await applyMappings({
-        srcBase: bmad.dir,
-        destBase: templatesDir,
-        mappings: BMAD_PATH_MAPPINGS,
-        postProcess,
-        logger,
-      })
-      logger.success(`BMAD: ${fileCount} files synced`)
-    }
-
-    if (syncEcc) {
-      currentStep++
-      logger.step(currentStep, totalSteps, 'Fetching ECC...')
-      const ecc = await fetchEccFn({ tmpDir, logger })
-      upstreamMeta.ecc = { sha: ecc.sha }
-
-      const fileCount = await applyMappings({
-        srcBase: ecc.dir,
-        destBase: templatesDir,
-        mappings: ECC_PATH_MAPPINGS,
-        logger,
-      })
-      logger.success(`ECC: ${fileCount} files synced`)
-    }
-
-    if (pluginsAvailable) {
-      currentStep++
-      logger.step(currentStep, totalSteps, 'Fetching Plugins...')
-
-      const registry = await loadRegistry(registryPath)
-      const plugins = await fetchPluginsFn({ tmpDir, logger, registry })
-      upstreamMeta.plugins = { sha: plugins.sha }
-
-      const pluginMappings = buildPluginMappings(registry)
-      const fileCount = await applyMappings({
-        srcBase: plugins.dir,
-        destBase: templatesDir,
-        mappings: pluginMappings,
-        logger,
-      })
-      logger.success(`Plugins: ${fileCount} files synced`)
-    }
-
-    if (extSkillsAvailable) {
-      currentStep++
-      logger.step(currentStep, totalSteps, 'Fetching Ext-skills...')
-
-      const registry = await loadRegistry(registryPath)
-      const extSkills = await fetchExtSkillsFn({ tmpDir, logger, registry })
-      upstreamMeta.extSkills = extSkills.skills
-
-      const extSkillMappings = buildExtSkillMappings(registry)
-      const fileCount = await applyMappings({
-        srcBase: extSkills.dir,
-        destBase: templatesDir,
-        mappings: extSkillMappings,
-        logger,
-      })
-      logger.success(`Ext-skills: ${fileCount} files synced`)
-    }
-  } finally {
-    // Cleanup temp directory
-    if (tmpDir) {
-      logger.debug('Cleaning up temp directory...')
-      await fse.remove(tmpDir)
-    }
-  }
-
-  // Step 6-8: Post-sync processing (snapshot, registry update, diff report)
-  let diff
-  let updatedUpstream
-  try {
-    // Step 6: Snapshot after and compute diff
-    const afterSnapshot = await snapshotDir(templatesDir)
-    diff = computeDiff(beforeSnapshot, afterSnapshot)
-
-    // Step 7: Update registry.json upstream metadata (immutable)
-    const registry = await loadRegistry(registryPath)
-    const now = new Date().toISOString()
-
-    updatedUpstream = {
-      ...(registry.upstream || {}),
-      ...(upstreamMeta.bmad ? {
-        bmad: {
-          type: 'npm',
-          package: 'bmad-method',
-          lastSyncVersion: upstreamMeta.bmad.version,
-          lastSyncDate: now,
-        },
-      } : {}),
-      ...(upstreamMeta.ecc ? {
-        ecc: {
-          type: 'git',
-          repo: ECC_REPO_URL,
-          lastSyncSha: upstreamMeta.ecc.sha,
-          lastSyncDate: now,
-        },
-      } : {}),
-      ...(upstreamMeta.plugins ? {
-        plugins: {
-          ...(registry.upstream?.plugins || {}),
-          lastSyncSha: upstreamMeta.plugins.sha,
-          lastSyncDate: now,
-        },
-      } : {}),
-      ...(upstreamMeta.extSkills ? {
-        'ext-skills': Object.fromEntries(
-          Object.entries(registry.upstream?.['ext-skills'] || {}).map(([id, entry]) => [
-            id,
-            upstreamMeta.extSkills[id]
-              ? { ...entry, lastSyncSha: upstreamMeta.extSkills[id].sha, lastSyncDate: now }
-              : entry,
-          ])
-        ),
-      } : {}),
-    }
-
-    const updatedRegistry = { ...registry, upstream: updatedUpstream }
-    await fse.writeFile(registryPath, JSON.stringify(updatedRegistry, null, 2) + '\n', 'utf8')
-    logger.debug('Registry updated with upstream metadata')
-  } catch (err) {
-    throw new Error(`Sync completed but post-processing failed: ${err.message}`)
-  }
-
-  // Step 8: Print diff report
-  logger.newline()
-  const totalChanges = diff.added.length + diff.modified.length + diff.deleted.length
-
-  if (totalChanges === 0) {
-    logger.info('  No changes detected — templates are up to date.')
-  } else {
-    logger.info(`  Changes: +${diff.added.length} added, ~${diff.modified.length} modified, -${diff.deleted.length} deleted`)
-    logger.newline()
-
-    if (diff.added.length > 0) {
-      logger.info('  Added:')
-      for (const f of diff.added) {
-        logger.success(f)
-      }
-    }
-
-    if (diff.modified.length > 0) {
-      logger.info('  Modified:')
-      for (const f of diff.modified) {
-        logger.warn(f)
-      }
-    }
-
-    if (diff.deleted.length > 0) {
-      logger.info('  Deleted:')
-      for (const f of diff.deleted) {
-        logger.error(f)
-      }
-    }
-  }
-
-  logger.newline()
-  logger.success('Sync complete.')
-
-  return { diff, upstream: updatedUpstream }
-}
+/**
+ * `cckit sync` command — pull upstream BMAD/ECC/ext-skills content into templates/.
+ *
+ * Workflow:
+ *   1. Validate --source parameter
+ *   2. Snapshot current templates/ directory
+ *   3. If --dry-run: report intended operations and exit
+ *   4. Fetch upstream sources (BMAD via npx, ECC via git clone, ext-skills via git clone)
+ *   5. Apply path mappings to copy files into templates/
+ *   6. Post-snapshot and compute diff
+ *   7. Update registry.json upstream metadata
+ *   8. Print diff report
+ */
+
+import path from 'node:path'
+import fse from 'fs-extra'
+import { fileURLToPath } from 'node:url'
+import { createLogger } from '../utils/logger.js'
+import { loadRegistry } from '../core/registry.js'
+import { snapshotDir, computeDiff } from '../core/differ.js'
+import { shouldTemplatize, templatizeConfigYaml } from '../core/templatize.js'
+import {
+  fetchBmad,
+  fetchEcc,
+  fetchPlugins,
+  fetchExtSkills,
+  buildPluginMappings,
+  buildExtSkillMappings,
+  applyMappings,
+  createTempDir,
+  BMAD_PATH_MAPPINGS,
+  ECC_PATH_MAPPINGS,
+  ECC_REPO_URL,
+} from '../core/upstream.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const TEMPLATES_DIR = path.resolve(__dirname, '../../templates')
+const REGISTRY_PATH = path.resolve(__dirname, '../../registry.json')
+
+const VALID_SOURCES = ['bmad', 'ecc', 'plugins', 'ext-skills', 'all']
+
+/**
+ * Run the sync command.
+ *
+ * @param {object} opts - Command options
+ * @param {string} [opts.source='all'] - Source to sync: bmad, ecc, plugins, ext-skills, or all
+ * @param {boolean} [opts.dryRun=false] - Preview mode
+ * @param {boolean} [opts.debug=false] - Debug output
+ * @param {object} [opts._deps] - Injectable dependencies for testing
+ */
+export async function runSync(opts = {}) {
+  const {
+    source = 'all',
+    dryRun = false,
+    debug = false,
+    _deps = {},
+  } = opts
+
+  const logger = _deps.logger || createLogger({ debug })
+  const templatesDir = _deps.templatesDir || TEMPLATES_DIR
+  const registryPath = _deps.registryPath || REGISTRY_PATH
+  const fetchBmadFn = _deps.fetchBmad || fetchBmad
+  const fetchEccFn = _deps.fetchEcc || fetchEcc
+  const fetchPluginsFn = _deps.fetchPlugins || fetchPlugins
+  const fetchExtSkillsFn = _deps.fetchExtSkills || fetchExtSkills
+
+  // Step 1: Validate source
+  if (!VALID_SOURCES.includes(source)) {
+    throw new Error(`Invalid source "${source}". Must be one of: ${VALID_SOURCES.join(', ')}`)
+  }
+
+  const syncBmad = source === 'bmad' || source === 'all'
+  const syncEcc = source === 'ecc' || source === 'all'
+  const syncPlugins = source === 'plugins' || source === 'all'
+  const syncExtSkills = source === 'ext-skills' || source === 'all'
+
+  logger.banner('cckit sync')
+  logger.info(`  Source: ${source}`)
+
+  // Step 2: Snapshot before
+  logger.debug('Snapshotting templates directory...')
+  const beforeSnapshot = await snapshotDir(templatesDir)
+  logger.debug(`  Before snapshot: ${beforeSnapshot.size} files`)
+
+  // Step 3: Dry-run check
+  if (dryRun) {
+    logger.info('')
+    logger.info('  Dry-run mode — no changes will be applied.')
+    logger.info('')
+    if (syncBmad) {
+      logger.info('  Would sync BMAD:')
+      for (const m of BMAD_PATH_MAPPINGS) {
+        logger.info(`    ${m.src} → templates/${m.dest}`)
+      }
+    }
+    if (syncEcc) {
+      logger.info('  Would sync ECC:')
+      for (const m of ECC_PATH_MAPPINGS) {
+        logger.info(`    ${m.src} → templates/${m.dest}`)
+      }
+    }
+    if (syncPlugins) {
+      const registry = await loadRegistry(registryPath)
+      const pluginMappings = buildPluginMappings(registry)
+      logger.info('  Would sync Plugins:')
+      for (const m of pluginMappings) {
+        logger.info(`    ${m.src} → templates/${m.dest}`)
+      }
+    }
+    if (syncExtSkills) {
+      const registry = await loadRegistry(registryPath)
+      const extSkillMappings = buildExtSkillMappings(registry)
+      logger.info('  Would sync Ext-skills:')
+      for (const m of extSkillMappings) {
+        logger.info(`    ${m.src} → templates/${m.dest}`)
+      }
+    }
+    return { dryRun: true }
+  }
+
+  // Step 4 & 5: Fetch and apply
+  let tmpDir
+  const upstreamMeta = {}
+
+  try {
+    tmpDir = await createTempDir('cckit-sync')
+    logger.debug(`Temp directory: ${tmpDir}`)
+
+    // Pre-check plugins and ext-skills upstream availability to compute accurate step count
+    let pluginsAvailable = false
+    if (syncPlugins) {
+      const reg = await loadRegistry(registryPath)
+      pluginsAvailable = Boolean(reg.upstream?.plugins?.repo)
+      if (!pluginsAvailable) {
+        logger.warn('No plugins upstream configured in registry.json, skipping plugins sync')
+      }
+    }
+
+    let extSkillsAvailable = false
+    if (syncExtSkills) {
+      const reg = await loadRegistry(registryPath)
+      const extEntries = reg.upstream?.['ext-skills'] || {}
+      extSkillsAvailable = Object.keys(extEntries).length > 0
+      if (!extSkillsAvailable) {
+        logger.warn('No ext-skills upstream configured in registry.json, skipping ext-skills sync')
+      }
+    }
+
+    const totalSteps = [syncBmad, syncEcc, pluginsAvailable, extSkillsAvailable].filter(Boolean).length
+    let currentStep = 0
+
+    if (syncBmad) {
+      currentStep++
+      logger.step(currentStep, totalSteps, 'Fetching BMAD...')
+      const bmad = await fetchBmadFn({ tmpDir, logger })
+      upstreamMeta.bmad = { version: bmad.version }
+
+      const postProcess = (content, relPath) => {
+        if (shouldTemplatize(relPath)) {
+          return templatizeConfigYaml(content)
+        }
+        return content
+      }
+
+      const fileCount = await applyMappings({
+        srcBase: bmad.dir,
+        destBase: templatesDir,
+        mappings: BMAD_PATH_MAPPINGS,
+        postProcess,
+        logger,
+      })
+      logger.success(`BMAD: ${fileCount} files synced`)
+    }
+
+    if (syncEcc) {
+      currentStep++
+      logger.step(currentStep, totalSteps, 'Fetching ECC...')
+      const ecc = await fetchEccFn({ tmpDir, logger })
+      upstreamMeta.ecc = { sha: ecc.sha }
+
+      const fileCount = await applyMappings({
+        srcBase: ecc.dir,
+        destBase: templatesDir,
+        mappings: ECC_PATH_MAPPINGS,
+        logger,
+      })
+      logger.success(`ECC: ${fileCount} files synced`)
+    }
+
+    if (pluginsAvailable) {
+      currentStep++
+      logger.step(currentStep, totalSteps, 'Fetching Plugins...')
+
+      const registry = await loadRegistry(registryPath)
+      const plugins = await fetchPluginsFn({ tmpDir, logger, registry })
+      upstreamMeta.plugins = { sha: plugins.sha }
+
+      const pluginMappings = buildPluginMappings(registry)
+      const fileCount = await applyMappings({
+        srcBase: plugins.dir,
+        destBase: templatesDir,
+        mappings: pluginMappings,
+        logger,
+      })
+      logger.success(`Plugins: ${fileCount} files synced`)
+    }
+
+    if (extSkillsAvailable) {
+      currentStep++
+      logger.step(currentStep, totalSteps, 'Fetching Ext-skills...')
+
+      const registry = await loadRegistry(registryPath)
+      const extSkills = await fetchExtSkillsFn({ tmpDir, logger, registry })
+      upstreamMeta.extSkills = extSkills.skills
+
+      const extSkillMappings = buildExtSkillMappings(registry)
+      const fileCount = await applyMappings({
+        srcBase: extSkills.dir,
+        destBase: templatesDir,
+        mappings: extSkillMappings,
+        logger,
+      })
+      logger.success(`Ext-skills: ${fileCount} files synced`)
+    }
+  } finally {
+    // Cleanup temp directory
+    if (tmpDir) {
+      logger.debug('Cleaning up temp directory...')
+      await fse.remove(tmpDir)
+    }
+  }
+
+  // Step 6-8: Post-sync processing (snapshot, registry update, diff report)
+  let diff
+  let updatedUpstream
+  try {
+    // Step 6: Snapshot after and compute diff
+    const afterSnapshot = await snapshotDir(templatesDir)
+    diff = computeDiff(beforeSnapshot, afterSnapshot)
+
+    // Step 7: Update registry.json upstream metadata (immutable)
+    const registry = await loadRegistry(registryPath)
+    const now = new Date().toISOString()
+
+    updatedUpstream = {
+      ...(registry.upstream || {}),
+      ...(upstreamMeta.bmad ? {
+        bmad: {
+          type: 'npm',
+          package: 'bmad-method',
+          lastSyncVersion: upstreamMeta.bmad.version,
+          lastSyncDate: now,
+        },
+      } : {}),
+      ...(upstreamMeta.ecc ? {
+        ecc: {
+          type: 'git',
+          repo: ECC_REPO_URL,
+          lastSyncSha: upstreamMeta.ecc.sha,
+          lastSyncDate: now,
+        },
+      } : {}),
+      ...(upstreamMeta.plugins ? {
+        plugins: {
+          ...(registry.upstream?.plugins || {}),
+          lastSyncSha: upstreamMeta.plugins.sha,
+          lastSyncDate: now,
+        },
+      } : {}),
+      ...(upstreamMeta.extSkills ? {
+        'ext-skills': Object.fromEntries(
+          Object.entries(registry.upstream?.['ext-skills'] || {}).map(([id, entry]) => [
+            id,
+            upstreamMeta.extSkills[id]
+              ? { ...entry, lastSyncSha: upstreamMeta.extSkills[id].sha, lastSyncDate: now }
+              : entry,
+          ])
+        ),
+      } : {}),
+    }
+
+    const updatedRegistry = { ...registry, upstream: updatedUpstream }
+    await fse.writeFile(registryPath, JSON.stringify(updatedRegistry, null, 2) + '\n', 'utf8')
+    logger.debug('Registry updated with upstream metadata')
+  } catch (err) {
+    throw new Error(`Sync completed but post-processing failed: ${err.message}`)
+  }
+
+  // Step 8: Print diff report
+  logger.newline()
+  const totalChanges = diff.added.length + diff.modified.length + diff.deleted.length
+
+  if (totalChanges === 0) {
+    logger.info('  No changes detected — templates are up to date.')
+  } else {
+    logger.info(`  Changes: +${diff.added.length} added, ~${diff.modified.length} modified, -${diff.deleted.length} deleted`)
+    logger.newline()
+
+    if (diff.added.length > 0) {
+      logger.info('  Added:')
+      for (const f of diff.added) {
+        logger.success(f)
+      }
+    }
+
+    if (diff.modified.length > 0) {
+      logger.info('  Modified:')
+      for (const f of diff.modified) {
+        logger.warn(f)
+      }
+    }
+
+    if (diff.deleted.length > 0) {
+      logger.info('  Deleted:')
+      for (const f of diff.deleted) {
+        logger.error(f)
+      }
+    }
+  }
+
+  logger.newline()
+  logger.success('Sync complete.')
+
+  return { diff, upstream: updatedUpstream }
+}