sra-skills 0.20.0 → 0.20.2

package/index.mjs

@@ -216,8 +216,8 @@ async function selectSkillsWithProfile(skills, profileSkillNames, profileKey, pr
 
 async function promptSkillChanges(diff, allSkills, profileSkillNames, skipPrompt) {
   const { newProfile, newOther, removedFromProfile } = diff;
-  if (!newProfile.size && !newOther.size) return [new Set(), new Set()];
-  if (skipPrompt || !process.stdin.isTTY) return [newProfile, new Set()];
+  if (!newProfile.size && !newOther.size) return [new Set(), new Set(), false];
+  if (skipPrompt || !process.stdin.isTTY) return [newProfile, new Set(), false];
 
   if (removedFromProfile.size > 20) {
     const recCount = profileSkillNames ? profileSkillNames.size : 0;
@@ -244,7 +244,7 @@ async function promptSkillChanges(diff, allSkills, profileSkillNames, skipPrompt
   const toEnable = new Set();
   for (const name of newProfile) { if (selectedSet.has(`enable:${name}`)) toEnable.add(name); }
   for (const name of newOther) { if (selectedSet.has(`enable:${name}`)) toEnable.add(name); }
-  return [toEnable, new Set()];
+  return [toEnable, new Set(), true];
 }
 
 async function selectSkills(skills, previouslyEnabled = null, previouslyDisabled = null) {
@@ -520,13 +520,18 @@ if (cmd === 'add') {
 
       if (isRepeat) {
         const diff = computeSkillDiff(allDiscovered, profileSkillNames, enabled, disabled);
-        const [toEnable, toDisable] = skipPrompt
-          ? [diff.newProfile, new Set()]
+        const [toEnable, toDisable, prompted] = skipPrompt
+          ? [diff.newProfile, new Set(), false]
           : await promptSkillChanges(diff, allDiscovered, profileSkillNames, false);
         const finalEnabled = new Set([...diff.keep, ...diff.removedFromProfile, ...toEnable]);
         for (const n of toDisable) finalEnabled.delete(n);
         skills = allDiscovered.filter(s => finalEnabled.has(s.name));
-        disabled = new Set([...disabled, ...toDisable].filter(n => !toEnable.has(n) && discoveredNames.has(n)));
+        if (prompted) {
+          const rejectedNew = new Set([...diff.newProfile, ...diff.newOther].filter(n => !toEnable.has(n)));
+          disabled = new Set([...disabled, ...toDisable, ...rejectedNew].filter(n => !toEnable.has(n) && discoveredNames.has(n)));
+        } else {
+          disabled = new Set([...disabled, ...toDisable].filter(n => !toEnable.has(n) && discoveredNames.has(n)));
+        }
       } else {
         if (skipPrompt) {
           skills = allDiscovered.filter(s => profileSkillNames.has(s.name));
@@ -756,14 +761,19 @@ if (cmd === 'add') {
 
         const enabledSet = saved ? new Set(saved) : new Set();
         const diff = computeSkillDiff(allSkills, validatedProfileSkills, enabledSet, disabled);
-        const [toEnable, toDisable] = skipPrompt
-          ? [diff.newProfile, new Set()]
+        const [toEnable, toDisable, prompted] = skipPrompt
+          ? [diff.newProfile, new Set(), false]
           : await promptSkillChanges(diff, allSkills, validatedProfileSkills, false);
 
         const finalEnabled = new Set([...diff.keep, ...diff.removedFromProfile, ...toEnable]);
         for (const n of toDisable) finalEnabled.delete(n);
         skills = allSkills.filter(s => finalEnabled.has(s.name));
-        disabled = new Set([...disabled, ...toDisable].filter(n => !toEnable.has(n) && discovered.has(n)));
+        if (prompted) {
+          const rejectedNew = new Set([...diff.newProfile, ...diff.newOther].filter(n => !toEnable.has(n)));
+          disabled = new Set([...disabled, ...toDisable, ...rejectedNew].filter(n => !toEnable.has(n) && discovered.has(n)));
+        } else {
+          disabled = new Set([...disabled, ...toDisable].filter(n => !toEnable.has(n) && discovered.has(n)));
+        }
       } else {
         // No profile and no profiles.json — legacy path
         const previouslyKnown = new Set([...(saved || []), ...disabled]);

package/lib.mjs

@@ -664,6 +664,24 @@ export function runToolSetup(repoPath, tools) {
   return { commands: totalCommands, agents: totalAgents };
 }
 
+export function parsePostInstallOutput(output) {
+  const components = [];
+  const warnings = [];
+  const errors = [];
+  for (const line of output.split('\n')) {
+    const warnMatch = line.match(/⚠\s+(.+?)(?:\s*$)/);
+    if (warnMatch) { warnings.push(warnMatch[1].trim()); continue; }
+    const errMatch = line.match(/✗\s+(.+?)(?:\s*$)/);
+    if (errMatch) { errors.push(errMatch[1].trim()); continue; }
+    const okMatch = line.match(/✓\s+(.+?)(?:\s*$)/);
+    if (okMatch) {
+      const name = okMatch[1].replace(/\s+(telemetry|hook|hooks).*$/i, '').trim().toLowerCase();
+      if (name && !components.includes(name)) components.push(name);
+    }
+  }
+  return { components, warnings, errors };
+}
+
 export function runPostInstall(repoPath) {
   const hook = join(repoPath, 'hooks', 'post-install.sh');
   if (!existsSync(hook)) return;
@@ -673,14 +691,9 @@ export function runPostInstall(repoPath) {
     const logDir = join(SRA_HOME, 'logs');
     mkdirSync(logDir, { recursive: true });
     writeFileSync(join(logDir, 'post-install.log'), output);
-    const components = [];
-    for (const line of output.split('\n')) {
-      const m = line.match(/✓\s+(.+?)(?:\s*$)/);
-      if (m) {
-        const name = m[1].replace(/\s+(telemetry|hook|hooks).*$/i, '').trim().toLowerCase();
-        if (name && !components.includes(name)) components.push(name);
-      }
-    }
+    const { components, warnings, errors } = parsePostInstallOutput(output);
+    for (const msg of errors) console.log(`  ✗ ${msg}`);
+    for (const msg of warnings) console.log(`  ⚠ ${msg}`);
     if (components.length) {
       console.log('  Post-install:');
       const perRow = 4;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sra-skills",
-  "version": "0.20.0",
+  "version": "0.20.2",
   "description": "SRA agent skills installer — manage AI skill repos",
   "bin": {
     "sra": "index.mjs",