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

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

@@ -0,0 +1,325 @@
+"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 – node_modules skills and rules", function () {
+    (0, vitest_1.it)("symlinks skills from node_modules packages", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": ["some-tool"],
+                "/proj/node_modules/some-tool": ["skills", "package.json"],
+                "/proj/node_modules/some-tool/skills": ["tool-docs"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/skills/tool-docs"), path.join("/proj", ".agents", "skills", "tool-docs"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/skills/tool-docs"), path.join("/proj", ".claude", "skills", "tool-docs"));
+    });
+    (0, vitest_1.it)("symlinks rules from node_modules packages", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": ["some-tool"],
+                "/proj/node_modules/some-tool": ["rules", "package.json"],
+                "/proj/node_modules/some-tool/rules": ["best-practices.md"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/rules/best-practices.md"), path.join("/proj", ".agents", "rules", "best-practices.md"));
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/rules/best-practices.md"), path.join("/proj", ".claude", "rules", "best-practices.md"));
+    });
+    (0, vitest_1.it)("discovers skills from @scoped packages", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": ["@my-scope"],
+                "/proj/node_modules/@my-scope": ["my-tool"],
+                "/proj/node_modules/@my-scope/my-tool": ["skills", "package.json"],
+                "/proj/node_modules/@my-scope/my-tool/skills": ["scoped-skill"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/@my-scope/my-tool/skills/scoped-skill"), path.join("/proj", ".agents", "skills", "scoped-skill"));
+    });
+    (0, vitest_1.it)("local skills take priority over node_modules skills", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/skills": ["shared"],
+                "/proj/node_modules": ["some-tool"],
+                "/proj/node_modules/some-tool": ["skills", "package.json"],
+                "/proj/node_modules/some-tool/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/node_modules/some-tool/skills/shared"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(console.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('CONFLICT: "shared"'));
+    });
+    (0, vitest_1.it)("filters internal skills from node_modules", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": ["some-tool"],
+                "/proj/node_modules/some-tool": ["skills", "package.json"],
+                "/proj/node_modules/some-tool/skills": ["public-skill", "internal-skill"],
+            },
+            fileContents: {
+                "/proj/node_modules/some-tool/skills/internal-skill/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/node_modules/some-tool/skills/public-skill"), vitest_1.expect.any(String));
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/skills/internal-skill"), vitest_1.expect.any(String));
+    });
+    (0, vitest_1.it)("skips dotfiles and dotfolders in node_modules", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": [".bin", ".package-lock.json", "some-tool"],
+                "/proj/node_modules/some-tool": ["skills", "package.json"],
+                "/proj/node_modules/some-tool/skills": ["my-skill"],
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/proj/node_modules/some-tool/skills/my-skill"), vitest_1.expect.any(String));
+    });
+    (0, vitest_1.it)("does not add a source entry for packages without a skills/ or rules/ directory", function () {
+        mockFilesystem({
+            listings: {
+                "/proj/node_modules": ["no-skills-tool"],
+                "/proj/node_modules/no-skills-tool": ["dist", "package.json"],
+                // no skills/ or rules/ entry
+            },
+        });
+        (0, install_agent_features_1.installFeatures)("/proj", [], { dryRun: false });
+        // No symlinks and no "No skills found in node_modules" log noise
+        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalled();
+        (0, vitest_1.expect)(console.log).not.toHaveBeenCalledWith(vitest_1.expect.stringContaining("node_modules no-skills-tool"));
+    });
+});
+(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.5",
   "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.5"
   },
   "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*/];
-    });
-}); });