@comet/cli 9.0.0-beta.3 → 9.0.0-beta.4

package/lib/comet.js

@@ -4,12 +4,12 @@ var commander_1 = require("commander");
 var download_mitmproxy_1 = require("./commands/download-mitmproxy");
 var download_oauth2_proxy_1 = require("./commands/download-oauth2-proxy");
 var generate_block_types_1 = require("./commands/generate-block-types");
-var install_agent_skills_1 = require("./commands/install-agent-skills");
+var install_agent_features_1 = require("./commands/install-agent-features");
 var site_configs_1 = require("./commands/site-configs");
 var program = new commander_1.Command();
 program.addCommand(generate_block_types_1.generateBlockTypes);
 program.addCommand(site_configs_1.injectSiteConfigsCommand);
 program.addCommand(download_oauth2_proxy_1.downloadOAuth2ProxyCommand);
 program.addCommand(download_mitmproxy_1.downloadMitmproxyCommand);
-program.addCommand(install_agent_skills_1.installAgentSkillsCommand);
+program.addCommand(install_agent_features_1.installAgentFeaturesCommand);
 program.parse();

package/lib/commands/install-agent-features.d.ts

@@ -0,0 +1,7 @@
+import { Command } from "commander";
+interface InstallOptions {
+    dryRun: boolean;
+}
+export declare function installFeatures(cwd: string, repos: string[], options: InstallOptions): void;
+export declare const installAgentFeaturesCommand: Command;
+export {};

package/lib/commands/install-agent-features.js

@@ -0,0 +1,339 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __generator = (this && this.__generator) || function (thisArg, body) {
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
+    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    function verb(n) { return function (v) { return step([n, v]); }; }
+    function step(op) {
+        if (f) throw new TypeError("Generator is already executing.");
+        while (g && (g = 0, op[0] && (_ = 0)), _) try {
+            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
+            if (y = 0, t) op = [op[0] & 2, t.value];
+            switch (op[0]) {
+                case 0: case 1: t = op; break;
+                case 4: _.label++; return { value: op[1], done: false };
+                case 5: _.label++; y = op[1]; op = [0]; continue;
+                case 7: op = _.ops.pop(); _.trys.pop(); continue;
+                default:
+                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
+                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
+                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
+                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
+                    if (t[2]) _.ops.pop();
+                    _.trys.pop(); continue;
+            }
+            op = body.call(thisArg, _);
+        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
+        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
+    }
+};
+var __spreadArray = (this && this.__spreadArray) || function (to, from, pack) {
+    if (pack || arguments.length === 2) for (var i = 0, l = from.length, ar; i < l; i++) {
+        if (ar || !(i in from)) {
+            if (!ar) ar = Array.prototype.slice.call(from, 0, i);
+            ar[i] = from[i];
+        }
+    }
+    return to.concat(ar || Array.prototype.slice.call(from));
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.installAgentFeaturesCommand = void 0;
+exports.installFeatures = installFeatures;
+/* eslint-disable no-console */
+var child_process_1 = require("child_process");
+var commander_1 = require("commander");
+var fs = __importStar(require("fs"));
+var yaml = __importStar(require("js-yaml"));
+var os = __importStar(require("os"));
+var path = __importStar(require("path"));
+var SKILL_SOURCE_PATHS = ["skills", "agentic-plugin/skills"];
+var RULE_SOURCE_PATHS = ["rules"];
+function parseRepoUrl(rawUrl) {
+    var hashIndex = rawUrl.lastIndexOf("#");
+    if (hashIndex === -1) {
+        return { repoUrl: rawUrl, ref: undefined };
+    }
+    return { repoUrl: rawUrl.slice(0, hashIndex), ref: rawUrl.slice(hashIndex + 1) || undefined };
+}
+function cloneRepo(rawUrl, sparsePatterns) {
+    var _a = parseRepoUrl(rawUrl), repoUrl = _a.repoUrl, ref = _a.ref;
+    var tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "comet-agent-install-"));
+    (0, child_process_1.execFileSync)("git", ["init", tmpDir], { stdio: "pipe" });
+    (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "remote", "add", "origin", "--", repoUrl], { stdio: "pipe" });
+    (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "config", "core.sparseCheckout", "true"], { stdio: "pipe" });
+    fs.writeFileSync(path.join(tmpDir, ".git", "info", "sparse-checkout"), sparsePatterns.map(function (p) { return "".concat(p, "\n"); }).join(""));
+    var fetchRef = ref !== null && ref !== void 0 ? ref : "HEAD";
+    try {
+        console.log("Fetching ".concat(repoUrl, " ref \"").concat(fetchRef, "\" (sparse: ").concat(sparsePatterns.join(", "), ")..."));
+        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "fetch", "--depth", "1", "origin", fetchRef], { stdio: "pipe" });
+        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "checkout", "FETCH_HEAD"], { stdio: "pipe" });
+    }
+    catch (_b) {
+        console.log("Shallow fetch failed for ref \"".concat(fetchRef, "\"; falling back to full fetch..."));
+        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "fetch", "origin", fetchRef], { stdio: "pipe" });
+        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "checkout", "FETCH_HEAD"], { stdio: "pipe" });
+    }
+    return tmpDir;
+}
+function pathExists(p) {
+    try {
+        fs.lstatSync(p);
+        return true;
+    }
+    catch (_a) {
+        return false;
+    }
+}
+function isInternalMarkdown(mdPath) {
+    var _a;
+    var content;
+    try {
+        content = fs.readFileSync(mdPath, "utf-8");
+    }
+    catch (_b) {
+        return false;
+    }
+    var match = content.match(/^---\n([\s\S]*?)\n---/);
+    if (!match) {
+        return false;
+    }
+    try {
+        var parsed = yaml.load(match[1]);
+        return ((_a = parsed === null || parsed === void 0 ? void 0 : parsed.metadata) === null || _a === void 0 ? void 0 : _a.internal) === true;
+    }
+    catch (_c) {
+        return false;
+    }
+}
+function listFolders(directory, filterInternal) {
+    return fs
+        .readdirSync(directory)
+        .filter(function (entry) { return fs.statSync(path.join(directory, entry)).isDirectory(); })
+        .filter(function (folder) { return !filterInternal || !isInternalMarkdown(path.join(directory, folder, "SKILL.md")); })
+        .sort();
+}
+function listMarkdownFiles(directory, filterInternal) {
+    var results = [];
+    var walk = function (currentDir, relativePath) {
+        for (var _i = 0, _a = fs.readdirSync(currentDir); _i < _a.length; _i++) {
+            var entry = _a[_i];
+            var fullPath = path.join(currentDir, entry);
+            var relPath = relativePath ? path.join(relativePath, entry) : entry;
+            var stat = fs.statSync(fullPath);
+            if (stat.isDirectory()) {
+                walk(fullPath, relPath);
+            }
+            else if (stat.isFile() && entry.endsWith(".md")) {
+                if (!filterInternal || !isInternalMarkdown(fullPath)) {
+                    results.push(relPath);
+                }
+            }
+        }
+    };
+    walk(directory, "");
+    return results.sort();
+}
+function symlinkOrCopy(src, dest, symlink) {
+    if (symlink) {
+        try {
+            fs.symlinkSync(src, dest);
+            return "symlinked";
+        }
+        catch (err) {
+            if (err.code !== "EPERM") {
+                throw err;
+            }
+            // Windows without symlink privilege — fall back to copy
+        }
+    }
+    fs.cpSync(src, dest, { recursive: true });
+    return "copied";
+}
+function installItems(sources, targetDirs, _a) {
+    var dryRun = _a.dryRun;
+    var totalInstalled = 0;
+    for (var _i = 0, sources_1 = sources; _i < sources_1.length; _i++) {
+        var source = sources_1[_i];
+        var label = source.label, directory = source.directory, discover = source.discover, symlink = source.symlink, filterInternal = source.filterInternal, installed = source.installed, itemLabelSingular = source.itemLabelSingular, itemLabelPlural = source.itemLabelPlural;
+        if (!fs.existsSync(directory)) {
+            console.log("No ".concat(itemLabelPlural, " found in ").concat(label));
+            continue;
+        }
+        var items = discover === "folders" ? listFolders(directory, filterInternal !== null && filterInternal !== void 0 ? filterInternal : false) : listMarkdownFiles(directory, filterInternal !== null && filterInternal !== void 0 ? filterInternal : false);
+        if (items.length === 0) {
+            console.log("No ".concat(itemLabelPlural, " found in ").concat(label));
+            continue;
+        }
+        console.log("Installing ".concat(items.length, " ").concat(items.length === 1 ? itemLabelSingular : itemLabelPlural, " from ").concat(label, "..."));
+        for (var _b = 0, items_1 = items; _b < items_1.length; _b++) {
+            var item = items_1[_b];
+            if (installed.has(item)) {
+                console.warn("  CONFLICT: \"".concat(item, "\" from ").concat(label, " skipped (already installed from a higher-priority source)"));
+                continue;
+            }
+            var srcPath = path.resolve(path.join(directory, item));
+            for (var _c = 0, targetDirs_1 = targetDirs; _c < targetDirs_1.length; _c++) {
+                var targetDir = targetDirs_1[_c];
+                var destPath = path.join(targetDir, item);
+                if (dryRun) {
+                    console.log("  [dry-run] Would ".concat(symlink ? "symlink" : "copy", ": ").concat(srcPath, " -> ").concat(destPath));
+                }
+                else {
+                    if (pathExists(destPath)) {
+                        fs.rmSync(destPath, { recursive: true, force: true });
+                    }
+                    fs.mkdirSync(path.dirname(destPath), { recursive: true });
+                    var action = symlinkOrCopy(srcPath, destPath, symlink);
+                    console.log("  ".concat(action === "symlinked" ? "Symlinked" : "Copied", ": ").concat(item));
+                }
+            }
+            installed.add(item);
+            totalInstalled++;
+        }
+    }
+    return totalInstalled;
+}
+function loadConfig(configPath) {
+    var resolved = path.resolve(configPath);
+    if (!fs.existsSync(resolved)) {
+        throw new Error("Config file not found: ".concat(resolved));
+    }
+    var raw = fs.readFileSync(resolved, "utf-8");
+    return JSON.parse(raw);
+}
+function installFeatures(cwd, repos, options) {
+    var skillsTargetDirs = [path.join(cwd, ".agents", "skills"), path.join(cwd, ".claude", "skills")];
+    var rulesTargetDirs = [
+        path.join(cwd, ".agents", "rules"),
+        path.join(cwd, ".claude", "rules"),
+        path.join(cwd, ".cursor", "rules"),
+        path.join(cwd, ".github", "instructions"),
+    ];
+    for (var _i = 0, _a = __spreadArray(__spreadArray([], skillsTargetDirs, true), rulesTargetDirs, true); _i < _a.length; _i++) {
+        var dir = _a[_i];
+        fs.mkdirSync(dir, { recursive: true });
+    }
+    var installedSkills = new Set();
+    var installedRules = new Set();
+    var skillSources = SKILL_SOURCE_PATHS.map(function (p) { return ({
+        label: "local ".concat(p, "/"),
+        directory: path.join(cwd, p),
+        discover: "folders",
+        symlink: true,
+        installed: installedSkills,
+        itemLabelSingular: "skill",
+        itemLabelPlural: "skills",
+    }); });
+    var ruleSources = RULE_SOURCE_PATHS.map(function (p) { return ({
+        label: "local ".concat(p, "/"),
+        directory: path.join(cwd, p),
+        discover: "files",
+        symlink: true,
+        installed: installedRules,
+        itemLabelSingular: "rule",
+        itemLabelPlural: "rules",
+    }); });
+    var sparsePatterns = __spreadArray(__spreadArray([], SKILL_SOURCE_PATHS, true), RULE_SOURCE_PATHS, true).map(function (p) { return "".concat(p, "/"); });
+    var tempDirs = [];
+    try {
+        for (var _b = 0, repos_1 = repos; _b < repos_1.length; _b++) {
+            var rawUrl = repos_1[_b];
+            var cloneDir = cloneRepo(rawUrl, sparsePatterns);
+            tempDirs.push(cloneDir);
+            for (var _c = 0, SKILL_SOURCE_PATHS_1 = SKILL_SOURCE_PATHS; _c < SKILL_SOURCE_PATHS_1.length; _c++) {
+                var p = SKILL_SOURCE_PATHS_1[_c];
+                skillSources.push({
+                    label: "external ".concat(rawUrl, " (").concat(p, "/)"),
+                    directory: path.join(cloneDir, p),
+                    discover: "folders",
+                    symlink: false,
+                    filterInternal: true,
+                    installed: installedSkills,
+                    itemLabelSingular: "skill",
+                    itemLabelPlural: "skills",
+                });
+            }
+            for (var _d = 0, RULE_SOURCE_PATHS_1 = RULE_SOURCE_PATHS; _d < RULE_SOURCE_PATHS_1.length; _d++) {
+                var p = RULE_SOURCE_PATHS_1[_d];
+                ruleSources.push({
+                    label: "external ".concat(rawUrl, " (").concat(p, "/)"),
+                    directory: path.join(cloneDir, p),
+                    discover: "files",
+                    symlink: false,
+                    filterInternal: true,
+                    installed: installedRules,
+                    itemLabelSingular: "rule",
+                    itemLabelPlural: "rules",
+                });
+            }
+        }
+        var skillCount = installItems(skillSources, skillsTargetDirs, options);
+        var ruleCount = installItems(ruleSources, rulesTargetDirs, options);
+        console.log("\nTotal skills installed: ".concat(skillCount));
+        console.log("Total rules installed: ".concat(ruleCount));
+    }
+    finally {
+        for (var _e = 0, tempDirs_1 = tempDirs; _e < tempDirs_1.length; _e++) {
+            var tmpDir = tempDirs_1[_e];
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    }
+}
+exports.installAgentFeaturesCommand = new commander_1.Command("install-agent-features")
+    .description("Install agent features (skills and rules) from local directories and optional external git repos")
+    .option("--config <path>", "Path to a JSON config file specifying repos to install features from", "agent-features.json")
+    .option("--dry-run", "Show which symlinks/copies would be created without making changes", false)
+    .action(function (options) { return __awaiter(void 0, void 0, void 0, function () {
+    var configPath, dryRun, resolvedConfig, repos;
+    var _a;
+    return __generator(this, function (_b) {
+        configPath = options.config, dryRun = options.dryRun;
+        resolvedConfig = path.resolve(configPath);
+        repos = fs.existsSync(resolvedConfig) ? ((_a = loadConfig(configPath).repos) !== null && _a !== void 0 ? _a : []) : [];
+        console.log("=== Installing agent features".concat(dryRun ? " (dry run)" : "", " ==="));
+        installFeatures(process.cwd(), repos, { dryRun: dryRun });
+        console.log("=== Finished ===");
+        return [2 /*return*/];
+    });
+}); });

package/lib/commands/install-agent-features.test.js

@@ -0,0 +1,234 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+/* eslint-disable no-console */
+var fs = __importStar(require("fs"));
+var path = __importStar(require("path"));
+var vitest_1 = require("vitest");
+var install_agent_features_1 = require("./install-agent-features");
+vitest_1.vi.mock("fs");
+/**
+ * fs mock for installFeatures.
+ *
+ * - `listings` maps a directory path to the names of its direct entries.
+ * - Entries are assumed to be directories unless the entry name ends in `.md`
+ *   or appears in `fileContents`.
+ * - `existingDests` is the set of destination paths that pathExists/lstatSync
+ *   should return truthy for (used to assert overwrite behavior).
+ * - `fileContents` provides content for fs.readFileSync; any path not in the
+ *   map throws ENOENT.
+ */
+function mockFilesystem(_a) {
+    var _b = _a === void 0 ? {} : _a, _c = _b.listings, listings = _c === void 0 ? {} : _c, _d = _b.existingDests, existingDests = _d === void 0 ? [] : _d, _e = _b.fileContents, fileContents = _e === void 0 ? {} : _e;
+    var isFile = function (p) {
+        if (p in fileContents) {
+            return true;
+        }
+        return path.basename(p).endsWith(".md");
+    };
+    vitest_1.vi.mocked(fs.existsSync).mockImplementation(function (p) { return String(p) in listings; });
+    vitest_1.vi.mocked(fs.readdirSync).mockImplementation(function (p) { var _a; return (_a = listings[String(p)]) !== null && _a !== void 0 ? _a : []; });
+    vitest_1.vi.mocked(fs.statSync).mockImplementation(function (p) {
+        var fileLike = isFile(String(p));
+        return { isDirectory: function () { return !fileLike; }, isFile: function () { return fileLike; } };
+    });
+    vitest_1.vi.mocked(fs.lstatSync).mockImplementation(function (p) {
+        if (existingDests.includes(String(p))) {
+            return {};
+        }
+        throw Object.assign(new Error("ENOENT"), { code: "ENOENT" });
+    });
+    vitest_1.vi.mocked(fs.readFileSync).mockImplementation(function (p) {
+        var content = fileContents[String(p)];
+        if (content !== undefined) {
+            return content;
+        }
+        throw Object.assign(new Error("ENOENT"), { code: "ENOENT" });
+    });
+}
+(0, vitest_1.beforeEach)(function () {
+    vitest_1.vi.mocked(fs.mkdirSync).mockImplementation(function () { return undefined; });
+    vitest_1.vi.mocked(fs.symlinkSync).mockImplementation(function () { return undefined; });
+    vitest_1.vi.mocked(fs.cpSync).mockImplementation(function () { return undefined; });
+    vitest_1.vi.mocked(fs.rmSync).mockImplementation(function () { return undefined; });
+    vitest_1.vi.spyOn(console, "log").mockImplementation(function () { return undefined; });
+    vitest_1.vi.spyOn(console, "warn").mockImplementation(function () { return undefined; });
+});
+(0, vitest_1.afterEach)(function () {
+    vitest_1.vi.clearAllMocks();
+    vitest_1.vi.restoreAllMocks();
+});
+(0, vitest_1.describe)("installFeatures – local skills", function () {
+    (0, vitest_1.it)("symlinks skill folders into .agents/skills and .claude/skills", function () {
+        mockFilesystem({ listings: { "/proj/skills": ["my-skill"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/my-skill"), path.join("/proj", ".agents", "skills", "my-skill"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/my-skill"), path.join("/proj", ".claude", "skills", "my-skill"));
+    });
+    (0, vitest_1.it)("also installs from agentic-plugin/skills/", function () {
+        mockFilesystem({ listings: { "/proj/agentic-plugin/skills": ["plugin-skill"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/agentic-plugin/skills/plugin-skill"), path.join("/proj", ".claude", "skills", "plugin-skill"));
+    });
+    (0, vitest_1.it)("skills/ takes priority over agentic-plugin/skills/ for same name", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/skills": ["shared"],
+                "/proj/agentic-plugin/skills": ["shared"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/shared"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/proj/agentic-plugin/skills/shared"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('CONFLICT: "shared"'));
+    });
+    (0, vitest_1.it)("ignores files in skills/ — only subdirectories are skills", function () {
+        mockFilesystem({ listings: { "/proj/skills": ["real-skill", "not-a-skill.md"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/real-skill"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/proj/skills/not-a-skill.md"), vitest_1.expect.any(String));
+    });
+});
+(0, vitest_1.describe)("installFeatures – local rules", function () {
+    (0, vitest_1.it)("symlinks .md files into .agents/rules, .claude/rules, .cursor/rules, and .github/instructions", function () {
+        mockFilesystem({ listings: { "/proj/rules": ["my-rule.md"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".agents", "rules"), { recursive: true });
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".claude", "rules"), { recursive: true });
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".cursor", "rules"), { recursive: true });
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".github", "instructions"), { recursive: true });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/my-rule.md"), path.join("/proj", ".agents", "rules", "my-rule.md"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/my-rule.md"), path.join("/proj", ".claude", "rules", "my-rule.md"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/my-rule.md"), path.join("/proj", ".cursor", "rules", "my-rule.md"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/my-rule.md"), path.join("/proj", ".github", "instructions", "my-rule.md"));
+    });
+    (0, vitest_1.it)("walks subdirectories recursively, preserving the layout in each target", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/rules": ["top.md", "backend"],
+                "/proj/rules/backend": ["api.md", "deeper"],
+                "/proj/rules/backend/deeper": ["nested.md"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/backend/api.md"), path.join("/proj", ".claude", "rules", "backend", "api.md"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/backend/deeper/nested.md"), path.join("/proj", ".claude", "rules", "backend", "deeper", "nested.md"));
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".claude", "rules", "backend"), { recursive: true });
+        (0, vitest_1.expect)(fs.mkdirSync).toHaveBeenCalledWith(path.join("/proj", ".claude", "rules", "backend", "deeper"), { recursive: true });
+    });
+    (0, vitest_1.it)("ignores non-.md entries", function () {
+        mockFilesystem({ listings: { "/proj/rules": ["rule.md", "README.txt"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/rule.md"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/proj/rules/README.txt"), vitest_1.expect.any(String));
+    });
+});
+(0, vitest_1.describe)("installFeatures – cross-namespace", function () {
+    (0, vitest_1.it)("a skill and a rule may share a name without conflicting", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/skills": ["shared"],
+                "/proj/rules": ["shared.md"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(console.warn).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/shared"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/rules/shared.md"), vitest_1.expect.any(String));
+    });
+});
+(0, vitest_1.describe)("installFeatures – dry-run", function () {
+    (0, vitest_1.it)("writes nothing", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/skills": ["my-skill"],
+                "/proj/rules": ["my-rule.md"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: true });
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(fs.rmSync).not.toHaveBeenCalled();
+    });
+    (0, vitest_1.it)("logs would-be operations", function () {
+        mockFilesystem({ listings: { "/proj/skills": ["my-skill"] } });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: true });
+        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("[dry-run] Would symlink: ".concat(path.resolve("/proj/skills/my-skill"))));
+    });
+});
+(0, vitest_1.describe)("installFeatures – overwrites existing destinations", function () {
+    (0, vitest_1.it)("removes the existing destination before re-creating it", function () {
+        mockFilesystem({
+            listings: { "/proj/skills": ["my-skill"] },
+            existingDests: [path.join("/proj", ".agents", "skills", "my-skill"), path.join("/proj", ".claude", "skills", "my-skill")],
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.rmSync).toHaveBeenCalledWith(path.join("/proj", ".agents", "skills", "my-skill"), { recursive: true, force: true });
+        (0, vitest_1.expect)(fs.rmSync).toHaveBeenCalledWith(path.join("/proj", ".claude", "skills", "my-skill"), { recursive: true, force: true });
+    });
+});
+(0, vitest_1.describe)("installFeatures – internal filtering", function () {
+    (0, vitest_1.it)("local sources install internal skills (filterInternal applies only to external repos)", function () {
+        mockFilesystem({
+            listings: { "/proj/skills": ["public", "internal"] },
+            fileContents: { "/proj/skills/internal/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# Internal" },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/public"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/skills/internal"), vitest_1.expect.any(String));
+    });
+});
+(0, vitest_1.describe)("installFeatures – missing local sources", function () {
+    (0, vitest_1.it)("runs cleanly when there are no local skills or rules", function () {
+        mockFilesystem({});
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No skills found"));
+        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No rules found"));
+    });
+});
+(0, vitest_1.describe)("installFeatures – EPERM symlink fallback", function () {
+    (0, vitest_1.it)("falls back to copy when symlinkSync throws EPERM (e.g. Windows without privileges)", function () {
+        mockFilesystem({ listings: { "/proj/skills": ["my-skill"] } });
+        vitest_1.vi.mocked(fs.symlinkSync).mockImplementationOnce(function () {
+            throw Object.assign(new Error("permission denied"), { code: "EPERM" });
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledWith(path.resolve("/proj/skills/my-skill"), path.join("/proj", ".agents", "skills", "my-skill"), {
+            recursive: true,
+        });
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comet/cli",
-  "version": "9.0.0-beta.3",
+  "version": "9.0.0-beta.4",
   "description": "A collection of CLI tools for Comet projects",
   "repository": {
     "directory": "packages/cli",
@@ -14,6 +14,7 @@
   "bin": {
     "comet": "bin/comet.js"
   },
+  "sideEffects": false,
   "files": [
     "bin/**/*.js",
     "lib/*"
@@ -26,13 +27,13 @@
   },
   "devDependencies": {
     "@types/js-yaml": "^4.0.9",
-    "@types/node": "^24.12.2",
-    "eslint": "^9.39.2",
+    "@types/node": "^24.12.4",
+    "eslint": "^9.39.4",
     "npm-run-all2": "^8.0.4",
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3",
     "vitest": "^4.0.16",
-    "@comet/eslint-config": "9.0.0-beta.3"
+    "@comet/eslint-config": "9.0.0-beta.4"
   },
   "engines": {
     "node": ">=22.0.0"

package/lib/commands/install-agent-skills.d.ts

@@ -1,15 +0,0 @@
-import { Command } from "commander";
-export interface SkillSource {
-    label: string;
-    directory: string;
-    /** If true, create symlinks; if false, copy files (used for tmp clone dirs) */
-    symlink: boolean;
-    /** If true, skills with metadata.internal: true in their SKILL.md are excluded */
-    filterInternal?: boolean;
-}
-export interface InstallOptions {
-    dryRun: boolean;
-}
-export declare function isInternalSkill(skillFolderPath: string): boolean;
-export declare function installSkills(sources: SkillSource[], targetDirs: string[], { dryRun }: InstallOptions): void;
-export declare const installAgentSkillsCommand: Command;

package/lib/commands/install-agent-skills.js

@@ -1,255 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-var __generator = (this && this.__generator) || function (thisArg, body) {
-    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
-    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
-    function verb(n) { return function (v) { return step([n, v]); }; }
-    function step(op) {
-        if (f) throw new TypeError("Generator is already executing.");
-        while (g && (g = 0, op[0] && (_ = 0)), _) try {
-            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
-            if (y = 0, t) op = [op[0] & 2, t.value];
-            switch (op[0]) {
-                case 0: case 1: t = op; break;
-                case 4: _.label++; return { value: op[1], done: false };
-                case 5: _.label++; y = op[1]; op = [0]; continue;
-                case 7: op = _.ops.pop(); _.trys.pop(); continue;
-                default:
-                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
-                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
-                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
-                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
-                    if (t[2]) _.ops.pop();
-                    _.trys.pop(); continue;
-            }
-            op = body.call(thisArg, _);
-        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
-        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
-    }
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.installAgentSkillsCommand = void 0;
-exports.isInternalSkill = isInternalSkill;
-exports.installSkills = installSkills;
-/* eslint-disable no-console */
-var child_process_1 = require("child_process");
-var commander_1 = require("commander");
-var fs = __importStar(require("fs"));
-var yaml = __importStar(require("js-yaml"));
-var os = __importStar(require("os"));
-var path = __importStar(require("path"));
-function parseRepoUrl(rawUrl) {
-    var hashIndex = rawUrl.lastIndexOf("#");
-    if (hashIndex === -1) {
-        return { repoUrl: rawUrl, ref: undefined };
-    }
-    return { repoUrl: rawUrl.slice(0, hashIndex), ref: rawUrl.slice(hashIndex + 1) || undefined };
-}
-var SKILL_SOURCE_PATHS = ["skills", "agentic-plugin/skills"];
-function cloneRepo(rawUrl) {
-    var _a = parseRepoUrl(rawUrl), repoUrl = _a.repoUrl, ref = _a.ref;
-    var tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "comet-agent-skills-"));
-    // Use sparse checkout to fetch only the skill source folders
-    (0, child_process_1.execFileSync)("git", ["init", tmpDir], { stdio: "pipe" });
-    (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "remote", "add", "origin", "--", repoUrl], { stdio: "pipe" });
-    (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "config", "core.sparseCheckout", "true"], { stdio: "pipe" });
-    fs.writeFileSync(path.join(tmpDir, ".git", "info", "sparse-checkout"), SKILL_SOURCE_PATHS.map(function (p) { return "".concat(p, "/\n"); }).join(""));
-    var fetchRef = ref !== null && ref !== void 0 ? ref : "HEAD";
-    try {
-        console.log("Fetching ".concat(repoUrl, " ref \"").concat(fetchRef, "\" (sparse, ").concat(SKILL_SOURCE_PATHS.map(function (p) { return "".concat(p, "/"); }).join(", "), " only)..."));
-        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "fetch", "--depth", "1", "origin", fetchRef], { stdio: "pipe" });
-        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "checkout", "FETCH_HEAD"], { stdio: "pipe" });
-    }
-    catch (_b) {
-        console.log("Shallow fetch failed for ref \"".concat(fetchRef, "\"; falling back to full fetch..."));
-        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "fetch", "origin", fetchRef], { stdio: "pipe" });
-        (0, child_process_1.execFileSync)("git", ["-C", tmpDir, "checkout", "FETCH_HEAD"], { stdio: "pipe" });
-    }
-    return tmpDir;
-}
-function pathExists(p) {
-    try {
-        fs.lstatSync(p);
-        return true;
-    }
-    catch (_a) {
-        return false;
-    }
-}
-function isInternalSkill(skillFolderPath) {
-    var _a;
-    var skillMdPath = path.join(skillFolderPath, "SKILL.md");
-    var skillMdContent;
-    try {
-        skillMdContent = fs.readFileSync(skillMdPath, "utf-8");
-    }
-    catch (_b) {
-        return false;
-    }
-    var frontmatterMatch = skillMdContent.match(/^---\n([\s\S]*?)\n---/);
-    if (!frontmatterMatch) {
-        return false;
-    }
-    try {
-        var parsed = yaml.load(frontmatterMatch[1]);
-        return ((_a = parsed === null || parsed === void 0 ? void 0 : parsed.metadata) === null || _a === void 0 ? void 0 : _a.internal) === true;
-    }
-    catch (_c) {
-        return false;
-    }
-}
-function listSkillFolders(directory, filterInternal) {
-    if (filterInternal === void 0) { filterInternal = false; }
-    if (!fs.existsSync(directory)) {
-        return [];
-    }
-    return fs
-        .readdirSync(directory)
-        .filter(function (f) { return fs.statSync(path.join(directory, f)).isDirectory(); })
-        .filter(function (folder) { return !filterInternal || !isInternalSkill(path.join(directory, folder)); })
-        .sort();
-}
-function installSkills(sources, targetDirs, _a) {
-    var dryRun = _a.dryRun;
-    var installed = new Set();
-    for (var _i = 0, sources_1 = sources; _i < sources_1.length; _i++) {
-        var _b = sources_1[_i], label = _b.label, directory = _b.directory, symlink = _b.symlink, filterInternal = _b.filterInternal;
-        var folders = listSkillFolders(directory, filterInternal);
-        if (folders.length === 0) {
-            console.log("No skills found in ".concat(label));
-            continue;
-        }
-        console.log("Installing ".concat(folders.length, " skill(s) from ").concat(label, "..."));
-        for (var _c = 0, folders_1 = folders; _c < folders_1.length; _c++) {
-            var folder = folders_1[_c];
-            if (installed.has(folder)) {
-                console.warn("  CONFLICT: \"".concat(folder, "\" from ").concat(label, " skipped (already installed from a higher-priority source)"));
-                continue;
-            }
-            var srcPath = path.resolve(path.join(directory, folder));
-            for (var _d = 0, targetDirs_1 = targetDirs; _d < targetDirs_1.length; _d++) {
-                var targetDir = targetDirs_1[_d];
-                var destPath = path.join(targetDir, folder);
-                var exists = pathExists(destPath);
-                if (dryRun) {
-                    console.log("  [dry-run] Would ".concat(symlink ? "symlink" : "copy", ": ").concat(srcPath, " -> ").concat(destPath));
-                }
-                else {
-                    if (exists) {
-                        fs.rmSync(destPath, { recursive: true, force: true });
-                    }
-                    if (symlink) {
-                        fs.symlinkSync(srcPath, destPath);
-                    }
-                    else {
-                        fs.cpSync(srcPath, destPath, { recursive: true });
-                    }
-                    console.log("  ".concat(symlink ? "Symlinked" : "Copied", ": ").concat(folder));
-                }
-            }
-            installed.add(folder);
-        }
-    }
-    console.log("\nTotal skills installed: ".concat(installed.size));
-}
-function loadConfig(configPath) {
-    var resolved = path.resolve(configPath);
-    if (!fs.existsSync(resolved)) {
-        throw new Error("Config file not found: ".concat(resolved));
-    }
-    var raw = fs.readFileSync(resolved, "utf-8");
-    return JSON.parse(raw);
-}
-exports.installAgentSkillsCommand = new commander_1.Command("install-agent-skills")
-    .description("Install agent skills from local directories and optional external git repos")
-    .option("--config <path>", "Path to a JSON config file specifying repos to install skills from", "agent-skills.json")
-    .option("--dry-run", "Show which symlinks/copies would be created without making changes", false)
-    .action(function (options) { return __awaiter(void 0, void 0, void 0, function () {
-    var configPath, dryRun, resolvedConfig, repos, cwd, targetDirs, _i, targetDirs_2, dir, sources, tempDirs, _a, repos_1, rawUrl, cloneDir, _b, SKILL_SOURCE_PATHS_1, p, _c, tempDirs_1, tmpDir;
-    var _d;
-    return __generator(this, function (_e) {
-        configPath = options.config, dryRun = options.dryRun;
-        resolvedConfig = path.resolve(configPath);
-        repos = fs.existsSync(resolvedConfig) ? ((_d = loadConfig(configPath).repos) !== null && _d !== void 0 ? _d : []) : [];
-        console.log("=== Installing agent skills".concat(dryRun ? " (dry run)" : "", " ==="));
-        cwd = process.cwd();
-        targetDirs = [path.join(cwd, ".agents", "skills"), path.join(cwd, ".claude", "skills")];
-        // Ensure target directories exist (without clearing existing contents)
-        for (_i = 0, targetDirs_2 = targetDirs; _i < targetDirs_2.length; _i++) {
-            dir = targetDirs_2[_i];
-            fs.mkdirSync(dir, { recursive: true });
-        }
-        sources = SKILL_SOURCE_PATHS.map(function (p) { return ({
-            label: "local ".concat(p, "/"),
-            directory: path.join(cwd, p),
-            symlink: true,
-        }); });
-        tempDirs = [];
-        try {
-            for (_a = 0, repos_1 = repos; _a < repos_1.length; _a++) {
-                rawUrl = repos_1[_a];
-                cloneDir = cloneRepo(rawUrl);
-                tempDirs.push(cloneDir);
-                for (_b = 0, SKILL_SOURCE_PATHS_1 = SKILL_SOURCE_PATHS; _b < SKILL_SOURCE_PATHS_1.length; _b++) {
-                    p = SKILL_SOURCE_PATHS_1[_b];
-                    sources.push({
-                        label: "external ".concat(rawUrl, " (").concat(p, "/)"),
-                        directory: path.join(cloneDir, p),
-                        symlink: false,
-                        filterInternal: true,
-                    });
-                }
-            }
-            installSkills(sources, targetDirs, { dryRun: dryRun });
-        }
-        finally {
-            for (_c = 0, tempDirs_1 = tempDirs; _c < tempDirs_1.length; _c++) {
-                tmpDir = tempDirs_1[_c];
-                fs.rmSync(tmpDir, { recursive: true, force: true });
-            }
-        }
-        console.log("=== Finished ===");
-        return [2 /*return*/];
-    });
-}); });

package/lib/commands/install-agent-skills.test.js

@@ -1,280 +0,0 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-Object.defineProperty(exports, "__esModule", { value: true });
-/* eslint-disable no-console */
-var fs = __importStar(require("fs"));
-var path = __importStar(require("path"));
-var vitest_1 = require("vitest");
-var install_agent_skills_1 = require("./install-agent-skills");
-vitest_1.vi.mock("fs");
-var defaultOptions = { dryRun: false };
-var targetDirs = [".agents/skills", ".claude/skills"];
-/**
- * Sets up fs mocks so that each key in skillMap is a readable directory
- * containing the listed skill folder names. Destination paths do not exist
- * unless listed under a target dir key.
- */
-function mockFilesystem(skillMap, existingDests, skillMdContents) {
-    if (existingDests === void 0) { existingDests = []; }
-    if (skillMdContents === void 0) { skillMdContents = {}; }
-    vitest_1.vi.mocked(fs.existsSync).mockImplementation(function (p) { return String(p) in skillMap; });
-    vitest_1.vi.mocked(fs.readdirSync).mockImplementation(function (p) { var _a; return (_a = skillMap[String(p)]) !== null && _a !== void 0 ? _a : []; });
-    vitest_1.vi.mocked(fs.statSync).mockImplementation(function (p) {
-        var _a;
-        var parentDir = path.dirname(String(p));
-        var name = path.basename(String(p));
-        var isDir = ((_a = skillMap[parentDir]) !== null && _a !== void 0 ? _a : []).includes(name);
-        return { isDirectory: function () { return isDir; } };
-    });
-    vitest_1.vi.mocked(fs.lstatSync).mockImplementation(function (p) {
-        if (existingDests.includes(String(p))) {
-            return {};
-        }
-        throw Object.assign(new Error("ENOENT"), { code: "ENOENT" });
-    });
-    vitest_1.vi.mocked(fs.readFileSync).mockImplementation(function (p) {
-        var content = skillMdContents[String(p)];
-        if (content !== undefined) {
-            return content;
-        }
-        throw Object.assign(new Error("ENOENT"), { code: "ENOENT" });
-    });
-}
-(0, vitest_1.beforeEach)(function () {
-    vitest_1.vi.mocked(fs.symlinkSync).mockImplementation(function () { return undefined; });
-    vitest_1.vi.mocked(fs.cpSync).mockImplementation(function () { return undefined; });
-    vitest_1.vi.mocked(fs.rmSync).mockImplementation(function () { return undefined; });
-    vitest_1.vi.mocked(fs.readFileSync).mockImplementation(function () {
-        throw Object.assign(new Error("ENOENT"), { code: "ENOENT" });
-    });
-    vitest_1.vi.spyOn(console, "log").mockImplementation(function () { return undefined; });
-    vitest_1.vi.spyOn(console, "warn").mockImplementation(function () { return undefined; });
-});
-(0, vitest_1.afterEach)(function () {
-    vitest_1.vi.clearAllMocks();
-    vitest_1.vi.restoreAllMocks();
-});
-(0, vitest_1.describe)("installSkills – symlink vs copy", function () {
-    (0, vitest_1.it)("symlinks skill folders into target folder if symlink: true", function () {
-        mockFilesystem({ "/local/skills": ["my-skill"] });
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledTimes(2);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/skills/my-skill"), ".agents/skills/my-skill");
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/skills/my-skill"), ".claude/skills/my-skill");
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-    });
-    (0, vitest_1.it)("copies skill folders into target folder if symlink: false", function () {
-        mockFilesystem({ "/remote/skills": ["remote-skill"] });
-        var sources = [{ label: "external repo", directory: "/remote/skills", symlink: false }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledTimes(2);
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledWith(path.resolve("/remote/skills/remote-skill"), ".agents/skills/remote-skill", {
-            recursive: true,
-        });
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
-    });
-    (0, vitest_1.it)("ignores files in the source directory, only installs subdirectories", function () {
-        mockFilesystem({ "/local/skills": ["real-skill"] });
-        // statSync returns isDirectory: false for "not-a-skill.md" (not in the skillMap)
-        vitest_1.vi.mocked(fs.readdirSync).mockImplementation(function () { return ["real-skill", "not-a-skill.md"]; });
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledTimes(2);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/skills/real-skill"), vitest_1.expect.any(String));
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/local/skills/not-a-skill.md"), vitest_1.expect.any(String));
-    });
-});
-(0, vitest_1.describe)("installSkills – missing source directory", function () {
-    (0, vitest_1.it)("logs 'No skills found' and does nothing when source directory does not exist", function () {
-        mockFilesystem({}); // existsSync returns false for all paths
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No skills found"));
-    });
-    (0, vitest_1.it)("logs 'No skills found' and does nothing when source directory is empty", function () {
-        mockFilesystem({ "/local/skills": [] }); // dir exists but has no entries
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No skills found"));
-    });
-    (0, vitest_1.it)("logs 'No skills found' and does nothing when cloned external repo has no skills/ folder", function () {
-        mockFilesystem({}); // cloned tmp dir exists but contains no skills/ subdirectory
-        var sources = [
-            { label: "external git@github.com:org/repo.git (skills/)", directory: "/tmp/cloned-repo/skills", symlink: false },
-        ];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No skills found"));
-    });
-});
-(0, vitest_1.describe)("installSkills – dry-run mode", function () {
-    (0, vitest_1.it)("does not write any files or symlinks", function () {
-        mockFilesystem({ "/local/skills": ["my-skill"] });
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, { dryRun: true });
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(fs.rmSync).not.toHaveBeenCalled();
-    });
-    (0, vitest_1.it)("logs what would be symlinked", function () {
-        mockFilesystem({ "/local/skills": ["my-skill"] });
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, { dryRun: true });
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("[dry-run] Would symlink: ".concat(path.resolve("/local/skills/my-skill"))));
-    });
-    (0, vitest_1.it)("logs what would be copied for external sources", function () {
-        mockFilesystem({ "/remote/skills": ["remote-skill"] });
-        var sources = [{ label: "external repo", directory: "/remote/skills", symlink: false }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, { dryRun: true });
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("[dry-run] Would copy: ".concat(path.resolve("/remote/skills/remote-skill"))));
-    });
-});
-(0, vitest_1.describe)("installSkills – overwrites existing destinations", function () {
-    (0, vitest_1.it)("removes existing destination before writing", function () {
-        mockFilesystem({ "/local/skills": ["my-skill"] }, [".agents/skills/my-skill", ".claude/skills/my-skill"]);
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.rmSync).toHaveBeenCalledWith(".agents/skills/my-skill", { recursive: true, force: true });
-        (0, vitest_1.expect)(fs.rmSync).toHaveBeenCalledWith(".claude/skills/my-skill", { recursive: true, force: true });
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledTimes(2);
-    });
-});
-(0, vitest_1.describe)("installSkills – conflict: local vs remote", function () {
-    (0, vitest_1.it)("local skill wins: symlinks local, skips remote with a CONFLICT warning", function () {
-        mockFilesystem({
-            "/local/skills": ["shared-skill"],
-            "/remote/skills": ["shared-skill"],
-        });
-        var sources = [
-            { label: "local skills/", directory: "/local/skills", symlink: true },
-            { label: "external repo", directory: "/remote/skills", symlink: false },
-        ];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalled();
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('CONFLICT: "shared-skill"'));
-        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining("external repo"));
-    });
-});
-(0, vitest_1.describe)("installSkills – conflict: remote vs remote", function () {
-    (0, vitest_1.it)("first remote wins: copies remote1 skill, skips remote2 with a CONFLICT warning", function () {
-        mockFilesystem({
-            "/remote1/skills": ["shared-skill"],
-            "/remote2/skills": ["shared-skill"],
-        });
-        var sources = [
-            { label: "external repo1", directory: "/remote1/skills", symlink: false },
-            { label: "external repo2", directory: "/remote2/skills", symlink: false },
-        ];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        // Only 2 calls (one per target dir from remote1), not 4
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledTimes(2);
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledWith(path.resolve("/remote1/skills/shared-skill"), vitest_1.expect.any(String), { recursive: true });
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalledWith(path.resolve("/remote2/skills/shared-skill"), vitest_1.expect.any(String), vitest_1.expect.any(Object));
-        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('CONFLICT: "shared-skill"'));
-        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining("external repo2"));
-    });
-});
-(0, vitest_1.describe)("isInternalSkill", function () {
-    (0, vitest_1.it)("returns true when metadata.internal is true", function () {
-        mockFilesystem({}, [], { "/skills/my-skill/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# My Skill" });
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(true);
-    });
-    (0, vitest_1.it)("returns false when metadata.internal is false", function () {
-        mockFilesystem({}, [], { "/skills/my-skill/SKILL.md": "---\nmetadata:\n  internal: false\n---\n# My Skill" });
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(false);
-    });
-    (0, vitest_1.it)("returns false when frontmatter has no metadata field", function () {
-        mockFilesystem({}, [], { "/skills/my-skill/SKILL.md": "---\ntitle: My Skill\n---\n# My Skill" });
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(false);
-    });
-    (0, vitest_1.it)("returns false when SKILL.md has no frontmatter", function () {
-        mockFilesystem({}, [], { "/skills/my-skill/SKILL.md": "# My Skill\nJust plain markdown, no frontmatter." });
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(false);
-    });
-    (0, vitest_1.it)("returns false when SKILL.md does not exist", function () {
-        mockFilesystem({}, [], {});
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(false);
-    });
-    (0, vitest_1.it)("returns false when SKILL.md frontmatter contains malformed YAML", function () {
-        mockFilesystem({}, [], { "/skills/my-skill/SKILL.md": "---\n: invalid: yaml: [\n---\n# My Skill" });
-        (0, vitest_1.expect)((0, install_agent_skills_1.isInternalSkill)("/skills/my-skill")).toBe(false);
-    });
-});
-(0, vitest_1.describe)("installSkills – filterInternal", function () {
-    (0, vitest_1.it)("excludes internal skills when filterInternal: true, installs only non-internal skills", function () {
-        mockFilesystem({ "/remote/skills": ["public-skill", "internal-skill"] }, [], {
-            "/remote/skills/internal-skill/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# Internal Skill",
-        });
-        var sources = [{ label: "external repo (skills/)", directory: "/remote/skills", symlink: false, filterInternal: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledTimes(2);
-        (0, vitest_1.expect)(fs.cpSync).toHaveBeenCalledWith(path.resolve("/remote/skills/public-skill"), vitest_1.expect.any(String), { recursive: true });
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalledWith(path.resolve("/remote/skills/internal-skill"), vitest_1.expect.any(String), vitest_1.expect.any(Object));
-    });
-    (0, vitest_1.it)("installs all skills including internal ones when filterInternal is omitted", function () {
-        mockFilesystem({ "/local/skills": ["public-skill", "internal-skill"] }, [], {
-            "/local/skills/internal-skill/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# Internal Skill",
-        });
-        var sources = [{ label: "local skills/", directory: "/local/skills", symlink: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledTimes(4);
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/skills/public-skill"), vitest_1.expect.any(String));
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/skills/internal-skill"), vitest_1.expect.any(String));
-    });
-    (0, vitest_1.it)("logs 'No skills found' when filterInternal: true and all skills are internal", function () {
-        mockFilesystem({ "/remote/skills": ["internal-skill"] }, [], {
-            "/remote/skills/internal-skill/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# Internal Skill",
-        });
-        var sources = [{ label: "external repo (skills/)", directory: "/remote/skills", symlink: false, filterInternal: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, defaultOptions);
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalled();
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("No skills found"));
-    });
-    (0, vitest_1.it)("excludes internal skills from dry-run output when filterInternal: true", function () {
-        mockFilesystem({ "/remote/skills": ["public-skill", "internal-skill"] }, [], {
-            "/remote/skills/internal-skill/SKILL.md": "---\nmetadata:\n  internal: true\n---\n# Internal Skill",
-        });
-        var sources = [{ label: "external repo (skills/)", directory: "/remote/skills", symlink: false, filterInternal: true }];
-        (0, install_agent_skills_1.installSkills)(sources, targetDirs, { dryRun: true });
-        (0, vitest_1.expect)(console.log).toHaveBeenCalledWith(vitest_1.expect.stringContaining("public-skill"));
-        (0, vitest_1.expect)(console.log).not.toHaveBeenCalledWith(vitest_1.expect.stringContaining("internal-skill"));
-    });
-});