skiller 0.9.7 → 0.9.9

package/dist/cli/commands.js

@@ -5,6 +5,9 @@ const handlers_1 = require("./handlers");
 const index_1 = require("../agents/index");
 function skillsArgsBuilder(y) {
     return y
+        .parserConfiguration({
+        'unknown-options-as-args': true,
+    })
         .option('project-root', {
         type: 'string',
         description: 'Project root directory',
@@ -123,12 +126,24 @@ async function run() {
             description: 'Enable/disable skills support (experimental, default: enabled)',
         });
     }, handlers_1.applyHandler)
-        .command('add [args..]', 'Run the local skills CLI add command', skillsArgsBuilder, handlers_1.addHandler)
-        .command('remove [args..]', 'Run the local skills CLI remove command', skillsArgsBuilder, handlers_1.removeHandler)
+        .command('add [args..]', 'Run the local skills CLI add command', (y) => skillsArgsBuilder(y)
+        .option('verbose', {
+        type: 'boolean',
+        description: 'Enable verbose logging for the follow-up apply step',
+        default: false,
+    })
+        .alias('verbose', 'v'), handlers_1.addHandler)
+        .command('remove [args..]', 'Run the local skills CLI remove command', (y) => skillsArgsBuilder(y)
+        .option('verbose', {
+        type: 'boolean',
+        description: 'Enable verbose logging for the follow-up apply step',
+        default: false,
+    })
+        .alias('verbose', 'v'), handlers_1.removeHandler)
         .command('list [args..]', 'Run the local skills CLI list command', skillsArgsBuilder, handlers_1.listHandler)
         .command('find [args..]', 'Run the local skills CLI find command', skillsArgsBuilder, handlers_1.findHandler)
         .command('check [args..]', 'Run the local skills CLI check command', skillsArgsBuilder, handlers_1.checkHandler)
-        .command('install [args..]', 'Run the local skills CLI install command, then skiller apply', (y) => skillsArgsBuilder(y)
+        .command('install [args..]', 'Restore lock-backed skills with the local skills CLI experimental_install command, then skiller apply', (y) => skillsArgsBuilder(y)
         .option('verbose', {
         type: 'boolean',
         description: 'Enable verbose logging for the follow-up apply step',

package/dist/cli/handlers.js

@@ -57,8 +57,10 @@ const ConfigLoader_1 = require("../core/ConfigLoader");
 const ClaudePluginMigration_1 = require("../core/ClaudePluginMigration");
 const RulesToSkillsMigration_1 = require("../core/RulesToSkillsMigration");
 const agents_1 = require("../agents");
+const agents_2 = require("../agents");
 const skills_cli_1 = require("./skills-cli");
 const project_paths_1 = require("../core/project-paths");
+const SkillOwnership_1 = require("../core/SkillOwnership");
 const readline = __importStar(require("readline/promises"));
 async function executeSkillsWrapper(projectRoot, args) {
     try {
@@ -76,6 +78,73 @@ async function applyAfterSkillsLifecycleStep(projectRoot, verbose) {
         verbose,
     });
 }
+function normalizeRequestedSkillNames(args) {
+    if (!args || args.length === 0)
+        return [];
+    const names = new Set();
+    for (const arg of args) {
+        if (!arg || arg.startsWith('-') || arg.includes('/'))
+            continue;
+        const normalized = path.basename(arg, '.mdc').trim().replace(/:/g, '-');
+        if (normalized.length > 0) {
+            names.add(normalized);
+        }
+    }
+    return [...names].sort((a, b) => a.localeCompare(b));
+}
+async function scrubRequestedSkillsLockEntries(projectRoot, args) {
+    const requestedNames = new Set(normalizeRequestedSkillNames(args));
+    if (requestedNames.size === 0)
+        return [];
+    const skillsLockPath = path.join(projectRoot, 'skills-lock.json');
+    const raw = await readJsonObject(skillsLockPath);
+    if (!raw)
+        return [];
+    const skills = raw.skills;
+    if (!skills || typeof skills !== 'object')
+        return [];
+    const nextSkills = {};
+    const removedKeys = [];
+    for (const [key, value] of Object.entries(skills)) {
+        if (requestedNames.has(key.replace(/:/g, '-'))) {
+            removedKeys.push(key);
+            continue;
+        }
+        nextSkills[key] = value;
+    }
+    if (removedKeys.length === 0)
+        return [];
+    raw.skills = nextSkills;
+    await fs.writeFile(skillsLockPath, JSON.stringify(raw, null, 2) + '\n');
+    return removedKeys.sort((a, b) => a.localeCompare(b));
+}
+async function pruneRequestedUnmanagedSkillOutputs(projectRoot, args) {
+    const requestedNames = normalizeRequestedSkillNames(args);
+    if (requestedNames.length === 0)
+        return [];
+    const ownership = await (0, SkillOwnership_1.resolveSkillOwnership)(projectRoot);
+    const removableNames = requestedNames.filter((name) => !ownership.upstreamOwned.has(name) && !ownership.localOwned.has(name));
+    if (removableNames.length === 0)
+        return [];
+    const skillDirs = new Set([(0, SkillOwnership_1.getCanonicalSkillsDir)(projectRoot)]);
+    for (const agent of agents_2.allAgents) {
+        if (!agent.supportsNativeSkills?.() || !agent.getSkillsPath)
+            continue;
+        const skillsPath = agent.getSkillsPath(projectRoot);
+        if (skillsPath) {
+            skillDirs.add(skillsPath);
+        }
+    }
+    for (const skillName of removableNames) {
+        for (const skillsDir of skillDirs) {
+            await fs.rm(path.join(skillsDir, skillName), {
+                force: true,
+                recursive: true,
+            });
+        }
+    }
+    return removableNames;
+}
 function buildClaudePluginMigrationArgs(source) {
     return ['add', source, '--agent', 'universal', '--skill', '*', '-y'];
 }
@@ -565,10 +634,11 @@ async function addHandler(argv) {
         'add',
         ...(argv.args ?? []),
     ]);
+    await applyAfterSkillsLifecycleStep(argv['project-root'], argv.verbose ?? false);
 }
 async function installHandler(argv) {
     await executeSkillsWrapper(argv['project-root'], [
-        'install',
+        'experimental_install',
         ...(argv.args ?? []),
     ]);
     await applyAfterSkillsLifecycleStep(argv['project-root'], argv.verbose ?? false);
@@ -578,6 +648,9 @@ async function removeHandler(argv) {
         'remove',
         ...(argv.args ?? []),
     ]);
+    await scrubRequestedSkillsLockEntries(argv['project-root'], argv.args ?? []);
+    await pruneRequestedUnmanagedSkillOutputs(argv['project-root'], argv.args ?? []);
+    await applyAfterSkillsLifecycleStep(argv['project-root'], argv.verbose ?? false);
 }
 async function listHandler(argv) {
     await executeSkillsWrapper(argv['project-root'], [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skiller",
-  "version": "0.9.7",
+  "version": "0.9.9",
   "description": "Skiller — apply the same rules to all coding agents",
   "main": "dist/lib.js",
   "publishConfig": {