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

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

@@ -1,278 +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"));
-    });
-});