@supercorks/skills-installer 1.2.0 → 1.4.0

package/bin/install.js

@@ -30,10 +30,12 @@ import {
   updateSparseCheckout,
   sparseCloneSubagents,
   listCheckedOutSubagents,
-  updateSubagentsSparseCheckout
+  updateSubagentsSparseCheckout,
+  checkSkillsForUpdates,
+  checkSubagentsForUpdates
 } from '../lib/git.js';
 
-const VERSION = '1.2.0';
+const VERSION = '1.3.0';
 
 // Common installation paths to check for existing installations
 const SKILL_PATHS = ['.github/skills/', '.claude/skills/'];
@@ -113,6 +115,21 @@ Examples:
 `);
 }
 
+/**
+ * Check if a path is already in .gitignore
+ * @param {string} gitignorePath - Path to .gitignore file
+ * @param {string} pathToCheck - Path to check
+ * @returns {boolean}
+ */
+function isInGitignore(gitignorePath, pathToCheck) {
+  if (!existsSync(gitignorePath)) {
+    return false;
+  }
+  const normalizedPath = pathToCheck.replace(/\/$/, '');
+  const content = readFileSync(gitignorePath, 'utf-8');
+  return content.includes(normalizedPath);
+}
+
 /**
  * Add a path to .gitignore if not already present
  * @param {string} gitignorePath - Path to .gitignore file
@@ -211,14 +228,31 @@ async function runSkillsInstall() {
 
   const isManageMode = installedSkills.length > 0;
 
-  // Ask about .gitignore (only for fresh installs)
+  // Check for updates if in manage mode
+  let skillsNeedingUpdate = new Set();
+  if (isManageMode) {
+    const updateSpinner = showSpinner('Checking for available updates...');
+    try {
+      skillsNeedingUpdate = await checkSkillsForUpdates(absoluteInstallPath, installedSkills);
+      if (skillsNeedingUpdate.size > 0) {
+        updateSpinner.stop(`✅ Found ${skillsNeedingUpdate.size} skill${skillsNeedingUpdate.size !== 1 ? 's' : ''} with updates available`);
+      } else {
+        updateSpinner.stop('✅ All installed skills are up to date');
+      }
+    } catch {
+      updateSpinner.stop('⚠️  Could not check for updates');
+    }
+  }
+
+  // Ask about .gitignore (only for fresh installs and if not already in .gitignore)
   let shouldGitignore = false;
-  if (!isManageMode) {
+  const gitignorePath = resolve(process.cwd(), '.gitignore');
+  if (!isManageMode && !isInGitignore(gitignorePath, installPath)) {
     shouldGitignore = await promptGitignore(installPath);
   }
 
   // Select skills (pre-select installed skills in manage mode)
-  const selectedSkills = await promptSkillSelection(skills, installedSkills);
+  const selectedSkills = await promptSkillSelection(skills, installedSkills, skillsNeedingUpdate);
 
   // Perform installation or update
   console.log('');
@@ -262,7 +296,6 @@ async function runSkillsInstall() {
 
     // Update .gitignore if requested
     if (shouldGitignore) {
-      const gitignorePath = resolve(process.cwd(), '.gitignore');
       addToGitignore(gitignorePath, installPath);
     }
 
@@ -324,14 +357,31 @@ async function runSubagentsInstall() {
 
   const isManageMode = installedAgents.length > 0;
 
-  // Ask about .gitignore (only for fresh installs)
+  // Check for updates if in manage mode
+  let subagentsNeedingUpdate = new Set();
+  if (isManageMode) {
+    const updateSpinner = showSpinner('Checking for available updates...');
+    try {
+      subagentsNeedingUpdate = await checkSubagentsForUpdates(absoluteInstallPath, installedAgents);
+      if (subagentsNeedingUpdate.size > 0) {
+        updateSpinner.stop(`✅ Found ${subagentsNeedingUpdate.size} subagent${subagentsNeedingUpdate.size !== 1 ? 's' : ''} with updates available`);
+      } else {
+        updateSpinner.stop('✅ All installed subagents are up to date');
+      }
+    } catch {
+      updateSpinner.stop('⚠️  Could not check for updates');
+    }
+  }
+
+  // Ask about .gitignore (only for fresh installs and if not already in .gitignore)
   let shouldGitignore = false;
-  if (!isManageMode) {
+  const gitignorePath = resolve(process.cwd(), '.gitignore');
+  if (!isManageMode && !isInGitignore(gitignorePath, installPath)) {
     shouldGitignore = await promptGitignore(installPath);
   }
 
   // Select subagents (pre-select installed ones in manage mode)
-  const selectedAgents = await promptSubagentSelection(subagents, installedAgents);
+  const selectedAgents = await promptSubagentSelection(subagents, installedAgents, subagentsNeedingUpdate);
 
   // Perform installation or update
   console.log('');
@@ -375,7 +425,6 @@ async function runSubagentsInstall() {
 
     // Update .gitignore if requested
     if (shouldGitignore) {
-      const gitignorePath = resolve(process.cwd(), '.gitignore');
       addToGitignore(gitignorePath, installPath);
     }
 

package/lib/git.js

@@ -214,6 +214,100 @@ export async function pullUpdates(repoPath) {
   return runGitCommand(['pull'], absolutePath);
 }
 
+/**
+ * Check which skills have updates available (local differs from remote)
+ * @param {string} repoPath - Path to the sparse-checkout repo
+ * @param {string[]} skillFolders - Skill folders to check
+ * @returns {Promise<Set<string>>} Set of skill folder names that need updates
+ */
+export async function checkSkillsForUpdates(repoPath, skillFolders) {
+  const absolutePath = resolve(repoPath);
+  const needsUpdate = new Set();
+  
+  if (!existsSync(join(absolutePath, '.git'))) {
+    return needsUpdate;
+  }
+
+  try {
+    // Fetch latest from remote without modifying working tree
+    await runGitCommand(['fetch', 'origin'], absolutePath);
+    
+    // Get current branch
+    const branch = await runGitCommand(['rev-parse', '--abbrev-ref', 'HEAD'], absolutePath);
+    
+    // For each skill, check if there are differences between local and remote
+    for (const folder of skillFolders) {
+      try {
+        // Check if remote has changes for this folder
+        const diff = await runGitCommand([
+          'diff', 
+          `HEAD..origin/${branch}`,
+          '--stat',
+          '--',
+          folder
+        ], absolutePath);
+        
+        if (diff.trim()) {
+          needsUpdate.add(folder);
+        }
+      } catch {
+        // If diff fails, skip this folder
+      }
+    }
+  } catch {
+    // If fetch fails, return empty set (can't determine updates)
+  }
+  
+  return needsUpdate;
+}
+
+/**
+ * Check which subagents have updates available (local differs from remote)
+ * @param {string} repoPath - Path to the sparse-checkout repo
+ * @param {string[]} agentFilenames - Agent filenames to check
+ * @returns {Promise<Set<string>>} Set of agent filenames that need updates
+ */
+export async function checkSubagentsForUpdates(repoPath, agentFilenames) {
+  const absolutePath = resolve(repoPath);
+  const needsUpdate = new Set();
+  
+  if (!existsSync(join(absolutePath, '.git'))) {
+    return needsUpdate;
+  }
+
+  try {
+    // Fetch latest from remote without modifying working tree
+    await runGitCommand(['fetch', 'origin'], absolutePath);
+    
+    // Get current branch
+    const branch = await runGitCommand(['rev-parse', '--abbrev-ref', 'HEAD'], absolutePath);
+    
+    // For each agent, check if there are differences between local and remote
+    for (const filename of agentFilenames) {
+      try {
+        // Check if remote has changes for this file
+        const diff = await runGitCommand([
+          'diff', 
+          `HEAD..origin/${branch}`,
+          '--stat',
+          '--',
+          filename
+        ], absolutePath);
+        
+        if (diff.trim()) {
+          needsUpdate.add(filename);
+        }
+      } catch {
+        // If diff fails, skip this file
+      }
+    }
+  } catch {
+    // If fetch fails, return empty set (can't determine updates)
+  }
+  
+  return needsUpdate;
+}
+
 // ==================== SUBAGENTS FUNCTIONS ====================
 
 /**

package/lib/prompts.js

@@ -43,9 +43,9 @@ export async function promptInstallType() {
       name: 'installType',
       message: 'What would you like to install?',
       choices: [
+        { name: 'Skills and Agents', value: 'both' },
         { name: 'Skills only', value: 'skills' },
-        { name: 'Subagents only', value: 'subagents' },
-        { name: 'Both skills and subagents', value: 'both' }
+        { name: 'Agents only', value: 'subagents' }
       ]
     }
   ]);
@@ -202,13 +202,15 @@ export async function promptGitignore(installPath) {
  * Prompt user to select skills to install with expand/collapse support
  * @param {Array<{name: string, description: string, folder: string}>} skills - Available skills
  * @param {string[]} installedSkills - Already installed skill folder names (will be pre-selected)
+ * @param {Set<string>} skillsNeedingUpdate - Skill folder names that have updates available
  * @returns {Promise<string[]>} Selected skill folder names
  */
-export async function promptSkillSelection(skills, installedSkills = []) {
+export async function promptSkillSelection(skills, installedSkills = [], skillsNeedingUpdate = new Set()) {
   return promptItemSelection(
     skills.map(s => ({ id: s.folder, name: s.name, description: s.description })),
     installedSkills,
-    '📦 Available Skills'
+    '📦 Available Skills',
+    skillsNeedingUpdate
   );
 }
 
@@ -216,13 +218,15 @@ export async function promptSkillSelection(skills, installedSkills = []) {
  * Prompt user to select subagents to install with expand/collapse support
  * @param {Array<{name: string, description: string, filename: string}>} subagents - Available subagents
  * @param {string[]} installedSubagents - Already installed subagent filenames (will be pre-selected)
+ * @param {Set<string>} subagentsNeedingUpdate - Subagent filenames that have updates available
  * @returns {Promise<string[]>} Selected subagent filenames
  */
-export async function promptSubagentSelection(subagents, installedSubagents = []) {
+export async function promptSubagentSelection(subagents, installedSubagents = [], subagentsNeedingUpdate = new Set()) {
   return promptItemSelection(
     subagents.map(s => ({ id: s.filename, name: s.name, description: s.description })),
     installedSubagents,
-    '🤖 Available Subagents'
+    '🤖 Available Subagents',
+    subagentsNeedingUpdate
   );
 }
 
@@ -231,9 +235,10 @@ export async function promptSubagentSelection(subagents, installedSubagents = []
  * @param {Array<{id: string, name: string, description: string}>} items - Available items
  * @param {string[]} installedItems - Already installed item IDs (will be pre-selected)
  * @param {string} title - Title to display
+ * @param {Set<string>} itemsNeedingUpdate - Item IDs that have updates available
  * @returns {Promise<string[]>} Selected item IDs
  */
-function promptItemSelection(items, installedItems = [], title = '📦 Available Items') {
+function promptItemSelection(items, installedItems = [], title = '📦 Available Items', itemsNeedingUpdate = new Set()) {
   return new Promise((resolve, reject) => {
     const rl = readline.createInterface({
       input: process.stdin,
@@ -247,7 +252,10 @@ function promptItemSelection(items, installedItems = [], title = '📦 Available
     readline.emitKeypressEvents(process.stdin, rl);
 
     let cursor = 0;
-    const selected = new Set(installedItems);
+    // If nothing is installed, select all by default; otherwise pre-select installed items
+    const selected = installedItems.length > 0 
+      ? new Set(installedItems) 
+      : new Set(items.map(item => item.id));
     const expanded = new Set();
 
     const render = () => {
@@ -263,10 +271,12 @@ function promptItemSelection(items, installedItems = [], title = '📦 Available
         const isSelected = selected.has(item.id);
         const isCursor = i === cursor;
         const isExpanded = expanded.has(item.id);
+        const needsUpdate = itemsNeedingUpdate.has(item.id);
         
         const checkbox = isSelected ? '◉' : '○';
         const pointer = isCursor ? '❯' : ' ';
         const expandIcon = isExpanded ? '▼' : '▶';
+        const updateFlag = needsUpdate ? ' \x1B[33m(update)\x1B[0m' : '';
         
         // Highlight current line
         const highlight = isCursor ? '\x1B[36m' : '';  // Cyan for selected
@@ -275,7 +285,7 @@ function promptItemSelection(items, installedItems = [], title = '📦 Available
         const shortDesc = getFirstSentence(item.description);
         
         if (isExpanded) {
-          console.log(`${highlight}${pointer} ${checkbox} ${item.name}${reset}`);
+          console.log(`${highlight}${pointer} ${checkbox} ${item.name}${reset}${updateFlag}`);
           // Show full description indented
           const fullDesc = item.description || 'No description available';
           const lines = fullDesc.match(/.{1,55}/g) || [fullDesc];
@@ -283,12 +293,14 @@ function promptItemSelection(items, installedItems = [], title = '📦 Available
             console.log(`     ${highlight}${line}${reset}`);
           });
         } else {
-          console.log(`${highlight}${pointer} ${checkbox} ${item.name} ${expandIcon} ${shortDesc}${reset}`);
+          console.log(`${highlight}${pointer} ${checkbox} ${item.name}${reset}${updateFlag} ${highlight}${expandIcon} ${shortDesc}${reset}`);
         }
       });
 
       const selectedCount = selected.size;
-      console.log(`\n${selectedCount} item${selectedCount !== 1 ? 's' : ''} selected`);
+      const updateCount = Array.from(selected).filter(id => itemsNeedingUpdate.has(id)).length;
+      const updateNote = updateCount > 0 ? ` (${updateCount} to update)` : '';
+      console.log(`\n${selectedCount} item${selectedCount !== 1 ? 's' : ''} selected${updateNote}`);
     };
 
     const cleanup = () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@supercorks/skills-installer",
-  "version": "1.2.0",
+  "version": "1.4.0",
   "description": "Interactive CLI installer for AI agent skills and subagents",
   "type": "module",
   "bin": {