myaidev-method 0.3.2 → 0.3.4

package/src/lib/update-manager.js

@@ -9,6 +9,11 @@ import { createHash } from 'crypto';
  * Handles intelligent updates with conflict resolution
  */
 
+/** Pluralize helper: returns "1 file" or "N files" */
+function pf(count) {
+  return `${count} file${count === 1 ? '' : 's'}`;
+}
+
 /**
  * Display ASCII art branding banner
  */
@@ -234,7 +239,11 @@ export async function updateComponent(componentType, projectDir, cliType, source
       break;
     case 'mcp':
       targetDir = path.join(projectDir, `.${cliType}`, 'mcp');
-      sourceSubDir = path.join(sourceDir, '.claude', 'mcp');
+      sourceSubDir = path.join(sourceDir, 'src', 'mcp');
+      break;
+    case 'config':
+      targetDir = path.join(projectDir, `.${cliType}`);
+      sourceSubDir = path.join(sourceDir, 'src', 'templates', 'claude');
       break;
     case 'docs':
       targetDir = projectDir;
@@ -417,6 +426,10 @@ async function getLineCount(filePath) {
 async function getComponentFiles(componentType, dir) {
   if (!await fs.pathExists(dir)) return [];
 
+  if (componentType === 'config') {
+    return ['CLAUDE.md'];
+  }
+
   if (componentType === 'docs') {
     return ['USER_GUIDE.md', 'PUBLISHING_GUIDE.md', 'COOLIFY_DEPLOYMENT.md', 'DEV_WORKFLOW_GUIDE.md',
             'MCP_INTEGRATION.md', 'WORDPRESS_ADMIN_SCRIPTS.md', 'TECHNICAL_ARCHITECTURE.md',
@@ -488,7 +501,11 @@ export async function generateChangeSummary(components, projectDir, cliType, sou
         break;
       case 'mcp':
         targetDir = path.join(projectDir, `.${cliType}`, 'mcp');
-        sourceSubDir = path.join(sourceDir, '.claude', 'mcp');
+        sourceSubDir = path.join(sourceDir, 'src', 'mcp');
+        break;
+      case 'config':
+        targetDir = path.join(projectDir, `.${cliType}`);
+        sourceSubDir = path.join(sourceDir, 'src', 'templates', 'claude');
         break;
       case 'docs':
         targetDir = projectDir;
@@ -592,16 +609,41 @@ export function displayChangeSummary(summary, currentVersion, newVersion) {
   // Change summary
   console.log(chalk.cyan('║') + chalk.white.bold('  📊 CHANGE SUMMARY'.padEnd(boxWidth)) + chalk.cyan('║'));
 
-  const unchangedLine = `  ├─ ✅ Unchanged:     ${summary.unchanged.length} files (auto-skip)`;
-  const newLine = `  ├─ 🆕 New files:      ${summary.new.length} files (auto-add)`;
-  const modifiedLine = `  ├─ ⚠️  Modified:      ${summary.modified.length} files (need decision)`;
-  const deletedLine = `  └─ 🗑️  Removed:       ${summary.deleted.length} files (optional delete)`;
+  const unchangedLine = `  ├─ ✅ Unchanged:     ${pf(summary.unchanged.length)} (no action needed)`;
+  const newLine = `  ├─ 🆕 New:            ${pf(summary.new.length)} (will be added)`;
+  const modifiedLine = `  ├─ ⚠️  Modified:      ${pf(summary.modified.length)} (need decision)`;
+  const deletedLine = `  └─ 🗑️  Removed:       ${pf(summary.deleted.length)} (optional cleanup)`;
 
   console.log(chalk.cyan('║') + chalk.gray(unchangedLine.padEnd(boxWidth)) + chalk.cyan('║'));
   console.log(chalk.cyan('║') + chalk.green(newLine.padEnd(boxWidth)) + chalk.cyan('║'));
   console.log(chalk.cyan('║') + chalk.yellow(modifiedLine.padEnd(boxWidth)) + chalk.cyan('║'));
   console.log(chalk.cyan('║') + chalk.red(deletedLine.padEnd(boxWidth)) + chalk.cyan('║'));
 
+  // Show new skills if any (users care about what's new)
+  const newSkills = summary.new
+    .filter(f => f.componentType === 'skills' && f.file.endsWith('SKILL.md'))
+    .map(f => f.file.split('/')[0]);
+  if (newSkills.length > 0) {
+    console.log(chalk.cyan('║') + ' '.repeat(boxWidth) + chalk.cyan('║'));
+    const newSkillsLabel = `  🎯 New skills available:`;
+    console.log(chalk.cyan('║') + chalk.white.bold(newSkillsLabel.padEnd(boxWidth)) + chalk.cyan('║'));
+    // Wrap skill names into lines that fit the box (max ~63 chars per line)
+    const maxLineLen = boxWidth - 6; // "     " prefix + 1 padding
+    let currentLine = '';
+    for (let i = 0; i < newSkills.length; i++) {
+      const separator = currentLine ? ', ' : '';
+      if ((currentLine + separator + newSkills[i]).length > maxLineLen) {
+        console.log(chalk.cyan('║') + chalk.green(`     ${currentLine}`.padEnd(boxWidth)) + chalk.cyan('║'));
+        currentLine = newSkills[i];
+      } else {
+        currentLine += separator + newSkills[i];
+      }
+    }
+    if (currentLine) {
+      console.log(chalk.cyan('║') + chalk.green(`     ${currentLine}`.padEnd(boxWidth)) + chalk.cyan('║'));
+    }
+  }
+
   // Show modified files by category if any
   if (summary.modified.length > 0) {
     console.log(chalk.cyan('║') + ' '.repeat(boxWidth) + chalk.cyan('║'));
@@ -612,7 +654,7 @@ export function displayChangeSummary(summary, currentVersion, newVersion) {
       if (data.modified.length === 0) continue;
 
       const catName = category.toUpperCase();
-      const catLine = `  │ ${catName} (${data.modified.length} files)`;
+      const catLine = `  │ ${catName} (${pf(data.modified.length)})`;
       console.log(chalk.cyan('║') + chalk.white.bold(catLine.padEnd(boxWidth)) + chalk.cyan('║'));
 
       for (let i = 0; i < data.modified.length && i < 3; i++) {
@@ -708,7 +750,7 @@ export async function promptCategoryDecision(category, files) {
   const botBorder = '╚' + '═'.repeat(boxWidth) + '╝';
 
   console.log(chalk.cyan(topBorder));
-  console.log(chalk.cyan('║') + chalk.white.bold(`  ${category.toUpperCase()} (${files.length} modified files)`.padEnd(boxWidth)) + chalk.cyan('║'));
+  console.log(chalk.cyan('║') + chalk.white.bold(`  ${category.toUpperCase()} (${pf(files.length)} modified)`.padEnd(boxWidth)) + chalk.cyan('║'));
   console.log(chalk.cyan(midBorder));
 
   for (let i = 0; i < files.length; i++) {
@@ -730,9 +772,9 @@ export async function promptCategoryDecision(category, files) {
       name: 'decision',
       message: `What would you like to do with ${category.toUpperCase()}?`,
       choices: [
-        { name: `Keep all my customizations (skip ${files.length} files)`, value: 'keep' },
-        { name: `Update all (replace ${files.length} files, backup originals)`, value: 'update' },
-        { name: `Review these ${files.length} files individually`, value: 'individual' }
+        { name: `Keep all my customizations (skip ${pf(files.length)})`, value: 'keep' },
+        { name: `Update all (replace ${pf(files.length)}, backup originals)`, value: 'update' },
+        { name: `Review ${files.length === 1 ? 'this file' : `these ${files.length} files`} individually`, value: 'individual' }
       ]
     }
   ]);
@@ -755,32 +797,24 @@ export async function executeSmartUpdate(summary, backupDir, options = {}) {
     backedUp: []
   };
 
-  // Add all new files
+  // Add all new files — always show progress
   for (const f of summary.new) {
     await fs.ensureDir(path.dirname(f.targetPath));
     await fs.copy(f.sourcePath, f.targetPath);
     results.added.push(f.file);
-    if (options.verbose) {
-      console.log(chalk.green(`  ➕ ${f.file} (new)`));
-    }
+    console.log(chalk.green(`  ➕ ${f.file}`));
   }
 
-  // Keep all modified files (smart = preserve customizations)
+  // Keep all modified files (smart = preserve customizations) — always show
   for (const f of summary.modified) {
-    // Create backup
     const backupPath = path.join(backupDir, f.componentType, f.file);
     await fs.ensureDir(path.dirname(backupPath));
     await fs.copy(f.targetPath, backupPath);
     results.backedUp.push(f.file);
     results.kept.push(f.file);
-    if (options.verbose) {
-      console.log(chalk.gray(`  🛡️  ${f.file} (kept, backup created)`));
-    }
+    console.log(chalk.gray(`  🛡️  ${f.file} (kept, backup created)`));
   }
 
-  // Update unchanged files (they're identical so no-op, but count them)
-  results.updated = summary.unchanged.map(f => f.file);
-
   return results;
 }
 
@@ -799,30 +833,24 @@ export async function executeAcceptAll(summary, backupDir, options = {}) {
     backedUp: []
   };
 
-  // Add all new files
+  // Add all new files — always show progress
   for (const f of summary.new) {
     await fs.ensureDir(path.dirname(f.targetPath));
     await fs.copy(f.sourcePath, f.targetPath);
     results.added.push(f.file);
-    if (options.verbose) {
-      console.log(chalk.green(`  ➕ ${f.file} (new)`));
-    }
+    console.log(chalk.green(`  ➕ ${f.file}`));
   }
 
-  // Replace all modified files with backups
+  // Replace all modified files with backups — always show progress
   for (const f of summary.modified) {
-    // Create backup
     const backupPath = path.join(backupDir, f.componentType, f.file);
     await fs.ensureDir(path.dirname(backupPath));
     await fs.copy(f.targetPath, backupPath);
     results.backedUp.push(f.file);
 
-    // Replace with new version
     await fs.copy(f.sourcePath, f.targetPath);
     results.updated.push(f.file);
-    if (options.verbose) {
-      console.log(chalk.blue(`  ✅ ${f.file} (updated, backup created)`));
-    }
+    console.log(chalk.blue(`  ✅ ${f.file} (updated)`));
   }
 
   return results;
@@ -842,22 +870,18 @@ export async function executeKeepAll(summary, options = {}) {
     backedUp: []
   };
 
-  // Add all new files
+  // Add all new files — always show progress
   for (const f of summary.new) {
     await fs.ensureDir(path.dirname(f.targetPath));
     await fs.copy(f.sourcePath, f.targetPath);
     results.added.push(f.file);
-    if (options.verbose) {
-      console.log(chalk.green(`  ➕ ${f.file} (new)`));
-    }
+    console.log(chalk.green(`  ➕ ${f.file}`));
   }
 
-  // Keep all modified files
+  // Keep all modified files — always show
   results.kept = summary.modified.map(f => f.file);
-  if (options.verbose) {
-    for (const f of summary.modified) {
-      console.log(chalk.gray(`  🛡️  ${f.file} (kept)`));
-    }
+  for (const f of summary.modified) {
+    console.log(chalk.gray(`  🛡️  ${f.file} (kept)`));
   }
 
   return results;
@@ -958,34 +982,69 @@ export function displayUpdateResults(results, backupDir, newVersion) {
   console.log(chalk.green('║') + chalk.white.bold('                     ✅ Update Complete!                        '.padEnd(boxWidth)) + chalk.green('║'));
   console.log(chalk.green(midBorder));
 
-  console.log(chalk.green('║') + chalk.white.bold('  📊 Results:'.padEnd(boxWidth)) + chalk.green('║'));
+  const totalChanges = results.updated.length + results.added.length + results.kept.length;
 
-  if (results.updated.length > 0) {
-    const line = `  ├─ Updated:    ${results.updated.length} files`;
-    console.log(chalk.green('║') + chalk.blue(line.padEnd(boxWidth)) + chalk.green('║'));
-  }
+  if (totalChanges === 0) {
+    console.log(chalk.green('║') + chalk.white('  ✨ Everything is already current — no changes needed.'.padEnd(boxWidth)) + chalk.green('║'));
+  } else {
+    console.log(chalk.green('║') + chalk.white.bold('  📊 Results:'.padEnd(boxWidth)) + chalk.green('║'));
 
-  if (results.kept.length > 0) {
-    const line = `  ├─ Kept:       ${results.kept.length} files (your customizations preserved)`;
-    console.log(chalk.green('║') + chalk.gray(line.padEnd(boxWidth)) + chalk.green('║'));
-  }
+    const lines = [];
+    if (results.updated.length > 0) {
+      lines.push({ color: 'blue', text: `Updated:    ${pf(results.updated.length)}` });
+    }
+    if (results.kept.length > 0) {
+      lines.push({ color: 'gray', text: `Kept:       ${pf(results.kept.length)} (your customizations preserved)` });
+    }
+    if (results.added.length > 0) {
+      lines.push({ color: 'green', text: `Added:      ${pf(results.added.length)}` });
+    }
+    if (results.backedUp.length > 0) {
+      const relativePath = path.relative(process.cwd(), backupDir);
+      lines.push({ color: 'yellow', text: `Backed up:  ${pf(results.backedUp.length)} → ${relativePath}/` });
+    }
 
-  if (results.added.length > 0) {
-    const line = `  ├─ Added:      ${results.added.length} new files`;
-    console.log(chalk.green('║') + chalk.green(line.padEnd(boxWidth)) + chalk.green('║'));
-  }
+    for (let i = 0; i < lines.length; i++) {
+      const prefix = i === lines.length - 1 ? '└─' : '├─';
+      const line = `  ${prefix} ${lines[i].text}`;
+      console.log(chalk.green('║') + chalk[lines[i].color](line.padEnd(boxWidth)) + chalk.green('║'));
+    }
 
-  if (results.backedUp.length > 0) {
-    const relativePath = path.relative(process.cwd(), backupDir);
-    const line = `  └─ Backed up:  ${results.backedUp.length} files → ${relativePath}/`;
-    console.log(chalk.green('║') + chalk.yellow(line.padEnd(boxWidth)) + chalk.green('║'));
+    // Show new skills that were added
+    const newSkills = results.added
+      .filter(f => f.endsWith('SKILL.md'))
+      .map(f => f.split('/')[0]);
+    if (newSkills.length > 0) {
+      console.log(chalk.green('║') + ' '.repeat(boxWidth) + chalk.green('║'));
+      const label = `  🎯 New skills installed:`;
+      console.log(chalk.green('║') + chalk.white.bold(label.padEnd(boxWidth)) + chalk.green('║'));
+      const maxLineLen = boxWidth - 6;
+      let currentLine = '';
+      for (let i = 0; i < newSkills.length; i++) {
+        const separator = currentLine ? ', ' : '';
+        if ((currentLine + separator + newSkills[i]).length > maxLineLen) {
+          console.log(chalk.green('║') + chalk.cyan(`     ${currentLine}`.padEnd(boxWidth)) + chalk.green('║'));
+          currentLine = newSkills[i];
+        } else {
+          currentLine += separator + newSkills[i];
+        }
+      }
+      if (currentLine) {
+        console.log(chalk.green('║') + chalk.cyan(`     ${currentLine}`.padEnd(boxWidth)) + chalk.green('║'));
+      }
+    }
   }
 
   console.log(chalk.green('║') + ' '.repeat(boxWidth) + chalk.green('║'));
 
-  // What's new section
+  // Version and next steps
   console.log(chalk.green('║') + chalk.white.bold(`  🆕 Updated to v${newVersion}`.padEnd(boxWidth)) + chalk.green('║'));
-  console.log(chalk.green('║') + chalk.gray('  Run /myai-configure --status to see your configuration'.padEnd(boxWidth)) + chalk.green('║'));
+
+  // Only show restart reminder if skills were actually changed
+  const skillsChanged = results.added.some(f => f.includes('/')) || results.updated.some(f => f.includes('/'));
+  if (skillsChanged) {
+    console.log(chalk.green('║') + chalk.yellow('  ⚠️  Restart your AI assistant to load updated skills'.padEnd(boxWidth)) + chalk.green('║'));
+  }
 
   console.log(chalk.green(botBorder));
   console.log('');