myaidev-method 0.3.3 → 0.3.4

package/src/cli/commands/addon.js

@@ -17,7 +17,7 @@ import inquirer from 'inquirer';
 import fs from 'fs-extra';
 import path from 'path';
 import os from 'os';
-import { execSync } from 'child_process';
+
 import matter from 'gray-matter';
 import { isAuthenticated, getAuthToken, getAuthData, authFetch } from '../../lib/auth-helper.js';
 import { validateSkill, generateSlug } from '../../lib/skill-validator.js';
@@ -215,11 +215,12 @@ async function installSkill(name, options) {
       const { location } = await inquirer.prompt([{
         type: 'list',
         name: 'location',
-        message: 'Install location:',
+        message: 'Where do you want to install this skill?',
         choices: [
-          { name: `Project  ${chalk.gray('.claude/skills/')}`, value: 'project' },
-          { name: `Global   ${chalk.gray('~/.claude/skills/')}`, value: 'global' },
+          { name: `Project ${chalk.gray('— .claude/skills/ (this project only)')}`, value: 'project' },
+          { name: `Global  ${chalk.gray('— ~/.claude/skills/ (available everywhere)')}`, value: 'global' },
         ],
+        default: 'project',
       }]);
       targetDir = location === 'project' ? projectSkillsDir() : globalSkillsDir();
     }
@@ -696,26 +697,10 @@ async function scaffoldSkill() {
   return targetDir;
 }
 
-function ghAvailable() {
-  try {
-    execSync('gh auth status', { stdio: 'pipe' });
-    return true;
-  } catch {
-    return false;
-  }
-}
-
 async function submitSkill(opts) {
-  // Step 1: Auth checks
+  // Step 1: Auth check
   if (!(await requireAuth())) return;
 
-  if (!ghAvailable()) {
-    console.log(chalk.red('\n✗ GitHub CLI (gh) is required for submissions.'));
-    console.log(chalk.gray('  Install: https://cli.github.com'));
-    console.log(chalk.gray('  Then: gh auth login\n'));
-    return;
-  }
-
   // Step 2: Detect or scaffold SKILL.md
   let targetPath = opts.dir ? path.resolve(opts.dir) : process.cwd();
   let skillDir = targetPath;
@@ -747,14 +732,13 @@ async function submitSkill(opts) {
 
     const scaffoldedDir = await scaffoldSkill();
     if (!scaffoldedDir) return;
-    // User needs to edit the scaffold first
     return;
   }
 
-  // Step 3: Validate
+  // Step 3: Local validation
   const spinner = ora('Validating skill...').start();
   const result = await validateSkill(skillDir, {
-    checkDuplicates: true,
+    checkDuplicates: false,
     apiBase: API_BASE,
   });
   spinner.stop();
@@ -777,11 +761,11 @@ async function submitSkill(opts) {
   }
 
   // Step 4: Show summary and confirm
-  const slug = result.info.slug;
   console.log(chalk.cyan('\n📤 Submit to MyAIDev Marketplace\n'));
   console.log(`  ${chalk.gray('Name:')}        ${chalk.white.bold(result.info.name)}`);
-  console.log(`  ${chalk.gray('Slug:')}        ${chalk.white(slug)}`);
+  console.log(`  ${chalk.gray('Slug:')}        ${chalk.white(result.info.slug)}`);
   console.log(`  ${chalk.gray('Description:')} ${chalk.white(truncate(result.info.description, 60))}`);
+  console.log(`  ${chalk.gray('Category:')}    ${result.info.category || 'general'}`);
   console.log(`  ${chalk.gray('Sections:')}    ${result.info.sectionCount}`);
   console.log('');
 
@@ -798,116 +782,75 @@ async function submitSkill(opts) {
     }
   }
 
-  // Step 5: Fork, branch, commit, PR
-  const prSpinner = ora('Creating pull request...').start();
-  const tmpDir = path.join(os.tmpdir(), `myaidev-submit-${Date.now()}`);
+  // Step 5: Submit via API
+  const submitSpinner = ora('Submitting skill for review...').start();
 
   try {
-    // Fork and clone
-    try {
-      execSync(`gh repo fork ${MARKETPLACE_REPO} --clone=false`, { stdio: 'pipe' });
-    } catch {
-      // Fork may already exist, that's fine
-    }
-
-    // Get the user's GitHub username
-    const ghUser = execSync('gh api user --jq .login', { encoding: 'utf8' }).trim();
-
-    // Clone the user's fork
-    execSync(`gh repo clone ${ghUser}/myaidev-marketplace "${tmpDir}" -- --depth 1`, { stdio: 'pipe' });
-
-    // Create branch
-    const branch = `skill/${slug}`;
-    execSync(`git -C "${tmpDir}" checkout -b ${branch}`, { stdio: 'pipe' });
-
-    // Copy skill files
-    const destDir = path.join(tmpDir, 'skills', slug);
-    await fs.ensureDir(destDir);
-    await fs.copy(skillDir, destDir, {
-      filter: (src) => {
-        const ext = path.extname(src).toLowerCase();
-        const base = path.basename(src);
-        // Copy directories and allowed file types
-        if (fs.statSync(src).isDirectory()) return true;
-        return ['.md', '.json', '.yaml', '.yml', '.sh', '.js', '.ts', '.py', '.txt'].includes(ext) || base === '.gitkeep';
-      },
+    const content = await fs.readFile(skillFile, 'utf8');
+
+    const res = await authFetch(`${API_BASE}/submissions`, {
+      method: 'POST',
+      body: JSON.stringify({
+        content,
+        category: result.info.category || undefined,
+      }),
     });
 
-    // Commit
-    execSync(`git -C "${tmpDir}" add skills/${slug}`, { stdio: 'pipe' });
-    execSync(`git -C "${tmpDir}" commit -m "feat: add skill ${result.info.name}"`, { stdio: 'pipe' });
-
-    // Push
-    execSync(`git -C "${tmpDir}" push -u origin ${branch}`, { stdio: 'pipe' });
-
-    // Create PR
-    const prBody = [
-      `## Skill Submission: ${result.info.name}`,
-      '',
-      `**Description**: ${result.info.description || 'N/A'}`,
-      `**Category**: ${result.info.category || 'general'}`,
-      '',
-      '### Checklist',
-      '- [x] SKILL.md has valid YAML frontmatter',
-      '- [x] Passes local validation',
-      '- [ ] Tested locally with Claude Code',
-    ].join('\n');
-
-    const prOutput = execSync(
-      `gh pr create --repo ${MARKETPLACE_REPO} --head ${ghUser}:${branch} --title "feat: add skill ${result.info.name}" --body "${prBody.replace(/"/g, '\\"')}"`,
-      { encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] }
-    ).trim();
-
-    prSpinner.succeed(chalk.green('Pull request created!'));
-
-    // Extract PR URL from output
-    const prUrl = prOutput.match(/https:\/\/github\.com\S+/)?.[0] || prOutput;
-    console.log(chalk.gray(`  PR: ${prUrl}`));
-
-    // Step 6: Record submission
-    try {
-      await authFetch(`${API_BASE}/submissions`, {
-        method: 'POST',
-        body: JSON.stringify({
-          skillName: result.info.name,
-          slug,
-          description: result.info.description || '',
-          prUrl,
-        }),
-      });
-    } catch {
-      // Non-fatal: submission tracking is optional
-    }
-
-    console.log(chalk.cyan('\n  Status: awaiting review'));
-    console.log(chalk.gray('  Check: myaidev-method addon status\n'));
+    if (!res.ok) {
+      const body = await res.json().catch(() => ({}));
+      if (body.validation?.errors?.length) {
+        submitSpinner.fail(chalk.red('Server validation failed:'));
+        for (const err of body.validation.errors) {
+          console.log(chalk.red(`  • ${err}`));
+        }
+        return;
+      }
+      throw new Error(body.error || `API error: ${res.status}`);
+    }
+
+    const body = await res.json();
+    const submission = body.submission;
+
+    submitSpinner.succeed(chalk.green('Skill submitted for review!'));
+
+    console.log('');
+    console.log(`  ${chalk.gray('Submission ID:')} ${chalk.white(submission.id)}`);
+    console.log(`  ${chalk.gray('Status:')}        ${chalk.yellow(submission.status)}`);
+    console.log('');
+    console.log(chalk.gray('  AI analysis will run automatically.'));
+    console.log(chalk.gray('  An admin will review and approve your skill.'));
+    console.log(chalk.gray(`  Check status: ${chalk.white('myaidev-method addon status')}\n`));
   } catch (err) {
-    prSpinner.fail(chalk.red(`Submission failed: ${err.message}`));
+    submitSpinner.fail(chalk.red(`Submission failed: ${err.message}`));
     process.exit(1);
-  } finally {
-    // Clean up temp directory
-    await fs.remove(tmpDir).catch(() => {});
   }
 }
 
 // ── Status Command ─────────────────────────────────────────────────────
 
-async function checkStatus(name) {
+async function checkStatus(nameOrId) {
   if (!(await requireAuth())) return;
 
   const spinner = ora('Fetching submission status...').start();
 
   try {
-    const params = name ? `?name=${encodeURIComponent(name)}` : '';
-    const res = await authFetch(`${API_BASE}/submissions${params}`);
-
-    if (!res.ok) {
-      throw new Error(`API error: ${res.status}`);
+    // If it looks like a UUID, fetch by ID
+    const isUuid = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(nameOrId || '');
+
+    let submissions;
+    if (isUuid) {
+      const res = await authFetch(`${API_BASE}/submissions/${nameOrId}`);
+      if (!res.ok) throw new Error(`API error: ${res.status}`);
+      const body = await res.json();
+      submissions = body.submission ? [body.submission] : [];
+    } else {
+      const params = nameOrId ? `?name=${encodeURIComponent(nameOrId)}` : '';
+      const res = await authFetch(`${API_BASE}/submissions${params}`);
+      if (!res.ok) throw new Error(`API error: ${res.status}`);
+      const body = await res.json();
+      submissions = body.submissions || [];
     }
 
-    const body = await res.json();
-    const submissions = body.submissions || [];
-
     spinner.stop();
 
     if (submissions.length === 0) {
@@ -921,16 +864,36 @@ async function checkStatus(name) {
     for (const sub of submissions) {
       const statusColor = {
         pending_review: chalk.yellow,
+        ai_analyzing: chalk.magenta,
+        ready_for_review: chalk.blue,
         approved: chalk.green,
-        merged: chalk.green.bold,
         rejected: chalk.red,
         changes_requested: chalk.yellow,
       }[sub.status] || chalk.gray;
 
-      console.log(`  ${chalk.white.bold(sub.skillName)} ${statusColor(`[${sub.status}]`)}`);
-      if (sub.prUrl) console.log(chalk.gray(`    PR: ${sub.prUrl}`));
-      if (sub.reviewNotes) console.log(chalk.gray(`    Notes: ${sub.reviewNotes}`));
-      console.log(chalk.gray(`    Submitted: ${new Date(sub.createdAt).toLocaleDateString()}`));
+      const statusLabel = {
+        pending_review: 'pending review',
+        ai_analyzing: 'AI analyzing...',
+        ready_for_review: 'ready for review',
+        approved: 'approved',
+        rejected: 'rejected',
+        changes_requested: 'changes requested',
+      }[sub.status] || sub.status;
+
+      console.log(`  ${chalk.white.bold(sub.skillName)} ${statusColor(`[${statusLabel}]`)}`);
+
+      // Show AI score if available
+      if (sub.aiAnalysis?.overallScore != null) {
+        const score = sub.aiAnalysis.overallScore;
+        const scoreColor = score >= 75 ? chalk.green : score >= 50 ? chalk.yellow : chalk.red;
+        console.log(`    ${chalk.gray('AI Score:')} ${scoreColor(`${score}/100`)} ${chalk.gray(`(${sub.aiAnalysis.recommendation})`)}`);
+      } else if (sub.aiAnalysisStatus === 'running') {
+        console.log(`    ${chalk.gray('AI Score:')} ${chalk.magenta('analyzing...')}`);
+      }
+
+      if (sub.reviewNotes) console.log(`    ${chalk.gray('Review:')} ${sub.reviewNotes}`);
+      console.log(`    ${chalk.gray('Submitted:')} ${new Date(sub.createdAt).toLocaleDateString()}`);
+      if (sub.reviewedAt) console.log(`    ${chalk.gray('Reviewed:')} ${new Date(sub.reviewedAt).toLocaleDateString()}`);
       console.log('');
     }
   } catch (err) {

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('');