sra-skills 0.20.1 → 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/package.json

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