@comet/cli 8.20.0-canary-20260312162039 → 8.20.0

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

@@ -4,9 +4,12 @@ export interface SkillSource {
     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

@@ -68,22 +68,15 @@ var __generator = (this && this.__generator) || function (thisArg, body) {
         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.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) {
@@ -96,14 +89,14 @@ function parseRepoUrl(rawUrl) {
 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 package-skills/ folder
+    // Use sparse checkout to fetch only the skills/ folder
     (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"), "package-skills/\n");
+    fs.writeFileSync(path.join(tmpDir, ".git", "info", "sparse-checkout"), "skills/\n");
     var fetchRef = ref !== null && ref !== void 0 ? ref : "HEAD";
     try {
-        console.log("Fetching ".concat(repoUrl, " ref \"").concat(fetchRef, "\" (sparse, package-skills/ only)..."));
+        console.log("Fetching ".concat(repoUrl, " ref \"").concat(fetchRef, "\" (sparse, skills/ 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" });
     }
@@ -123,20 +116,43 @@ function pathExists(p) {
         return false;
     }
 }
-function listSkillFolders(directory) {
+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;
-        var folders = listSkillFolders(directory);
+        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;
@@ -186,18 +202,12 @@ exports.installAgentSkillsCommand = new commander_1.Command("install-agent-skill
     .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, configRepos, resolvedConfig, config, repos, cwd, targetDirs, _i, targetDirs_2, dir, sources, tempDirs, _a, repos_1, rawUrl, cloneDir, _b, tempDirs_1, tmpDir;
-    return __generator(this, function (_c) {
+    var configPath, dryRun, resolvedConfig, repos, cwd, targetDirs, _i, targetDirs_2, dir, sources, tempDirs, _a, repos_1, rawUrl, cloneDir, _b, tempDirs_1, tmpDir;
+    var _c;
+    return __generator(this, function (_d) {
         configPath = options.config, dryRun = options.dryRun;
-        configRepos = [];
         resolvedConfig = path.resolve(configPath);
-        if (fs.existsSync(resolvedConfig)) {
-            config = loadConfig(configPath);
-            if (config.repos) {
-                configRepos.push.apply(configRepos, config.repos);
-            }
-        }
-        repos = __spreadArray([], configRepos, true);
+        repos = fs.existsSync(resolvedConfig) ? ((_c = loadConfig(configPath).repos) !== null && _c !== void 0 ? _c : []) : [];
         console.log("=== Installing agent skills".concat(dryRun ? " (dry run)" : "", " ==="));
         cwd = process.cwd();
         targetDirs = [path.join(cwd, ".agents", "skills"), path.join(cwd, ".claude", "skills")];
@@ -206,10 +216,7 @@ exports.installAgentSkillsCommand = new commander_1.Command("install-agent-skill
             dir = targetDirs_2[_i];
             fs.mkdirSync(dir, { recursive: true });
         }
-        sources = [
-            { label: "local project-skills/", directory: path.join(cwd, "project-skills"), symlink: true },
-            { label: "local package-skills/", directory: path.join(cwd, "package-skills"), symlink: true },
-        ];
+        sources = [{ label: "local skills/", directory: path.join(cwd, "skills"), symlink: true }];
         tempDirs = [];
         try {
             for (_a = 0, repos_1 = repos; _a < repos_1.length; _a++) {
@@ -217,9 +224,10 @@ exports.installAgentSkillsCommand = new commander_1.Command("install-agent-skill
                 cloneDir = cloneRepo(rawUrl);
                 tempDirs.push(cloneDir);
                 sources.push({
-                    label: "external ".concat(rawUrl, " (package-skills/)"),
-                    directory: path.join(cloneDir, "package-skills"),
+                    label: "external ".concat(rawUrl),
+                    directory: path.join(cloneDir, "skills"),
                     symlink: false,
+                    filterInternal: true,
                 });
             }
             installSkills(sources, targetDirs, { dryRun: dryRun });

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

@@ -46,8 +46,9 @@ var targetDirs = [".agents/skills", ".claude/skills"];
  * containing the listed skill folder names. Destination paths do not exist
  * unless listed under a target dir key.
  */
-function mockFilesystem(skillMap, existingDests) {
+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) {
@@ -62,11 +63,20 @@ function mockFilesystem(skillMap, existingDests) {
             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; });
 });
@@ -76,56 +86,56 @@ function mockFilesystem(skillMap, existingDests) {
 });
 (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/package-skills": ["my-skill"] });
-        var sources = [{ label: "local package-skills/", directory: "/local/package-skills", symlink: true }];
+        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/package-skills/my-skill"), ".agents/skills/my-skill");
-        (0, vitest_1.expect)(fs.symlinkSync).toHaveBeenCalledWith(path.resolve("/local/package-skills/my-skill"), ".claude/skills/my-skill");
+        (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/package-skills": ["remote-skill"] });
-        var sources = [{ label: "external repo", directory: "/remote/package-skills", symlink: false }];
+        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/package-skills/remote-skill"), ".agents/skills/remote-skill", {
+        (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/package-skills": ["real-skill"] });
+        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 package-skills/", directory: "/local/package-skills", symlink: true }];
+        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/package-skills/real-skill"), vitest_1.expect.any(String));
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/local/package-skills/not-a-skill.md"), vitest_1.expect.any(String));
+        (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 project-skills/", directory: "/local/project-skills", symlink: true }];
+        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/project-skills": [] }); // dir exists but has no entries
-        var sources = [{ label: "local project-skills/", directory: "/local/project-skills", symlink: true }];
+        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 package-skills/ folder", function () {
-        mockFilesystem({}); // cloned tmp dir exists but contains no package-skills/ subdirectory
+    (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 (package-skills/)", directory: "/tmp/cloned-repo/package-skills", symlink: false },
+            { 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();
@@ -134,63 +144,45 @@ function mockFilesystem(skillMap, existingDests) {
 });
 (0, vitest_1.describe)("installSkills – dry-run mode", function () {
     (0, vitest_1.it)("does not write any files or symlinks", function () {
-        mockFilesystem({ "/local/package-skills": ["my-skill"] });
-        var sources = [{ label: "local package-skills/", directory: "/local/package-skills", symlink: true }];
+        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/package-skills": ["my-skill"] });
-        var sources = [{ label: "local package-skills/", directory: "/local/package-skills", symlink: true }];
+        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/package-skills/my-skill"))));
+        (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/package-skills": ["remote-skill"] });
-        var sources = [{ label: "external repo", directory: "/remote/package-skills", symlink: false }];
+        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/package-skills/remote-skill"))));
+        (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/package-skills": ["my-skill"] }, [".agents/skills/my-skill", ".claude/skills/my-skill"]);
-        var sources = [{ label: "local package-skills/", directory: "/local/package-skills", symlink: true }];
+        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 (project-skills) vs local (package-skills)", function () {
-    (0, vitest_1.it)("project-skills wins over package-skills: symlinks project skill, skips package skill with a CONFLICT warning", function () {
-        mockFilesystem({
-            "/local/project-skills": ["shared-skill"],
-            "/local/package-skills": ["shared-skill"],
-        });
-        var sources = [
-            { label: "local project-skills/", directory: "/local/project-skills", symlink: true },
-            { label: "local package-skills/", directory: "/local/package-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/project-skills/shared-skill"), vitest_1.expect.any(String));
-        (0, vitest_1.expect)(fs.symlinkSync).not.toHaveBeenCalledWith(path.resolve("/local/package-skills/shared-skill"), vitest_1.expect.any(String));
-        (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("local package-skills/"));
-    });
-});
 (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/project-skills": ["shared-skill"],
-            "/remote/package-skills": ["shared-skill"],
+            "/local/skills": ["shared-skill"],
+            "/remote/skills": ["shared-skill"],
         });
         var sources = [
-            { label: "local project-skills/", directory: "/local/project-skills", symlink: true },
-            { label: "external repo", directory: "/remote/package-skills", symlink: false },
+            { 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();
@@ -202,19 +194,85 @@ function mockFilesystem(skillMap, existingDests) {
 (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/package-skills": ["shared-skill"],
-            "/remote2/package-skills": ["shared-skill"],
+            "/remote1/skills": ["shared-skill"],
+            "/remote2/skills": ["shared-skill"],
         });
         var sources = [
-            { label: "external repo1", directory: "/remote1/package-skills", symlink: false },
-            { label: "external repo2", directory: "/remote2/package-skills", symlink: false },
+            { 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/package-skills/shared-skill"), vitest_1.expect.any(String), { recursive: true });
-        (0, vitest_1.expect)(fs.cpSync).not.toHaveBeenCalledWith(path.resolve("/remote2/package-skills/shared-skill"), vitest_1.expect.any(String), vitest_1.expect.any(Object));
+        (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"));
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comet/cli",
-  "version": "8.20.0-canary-20260312162039",
+  "version": "8.20.0",
   "description": "A collection of CLI tools for Comet projects",
   "repository": {
     "directory": "packages/cli",
@@ -20,17 +20,19 @@
   ],
   "dependencies": {
     "commander": "^9.5.0",
+    "js-yaml": "^4.1.0",
     "prettier": "^3.6.2",
     "ts-node": "^10.9.2"
   },
   "devDependencies": {
+    "@types/js-yaml": "^4.0.9",
     "@types/node": "^24.10.9",
     "eslint": "^9.39.2",
     "npm-run-all2": "^8.0.4",
     "rimraf": "^6.1.2",
     "typescript": "^5.9.3",
     "vitest": "^4.0.16",
-    "@comet/eslint-config": "8.20.0-canary-20260312162039"
+    "@comet/eslint-config": "8.20.0"
   },
   "engines": {
     "node": ">=22.0.0"