@claude-collective/cli 0.1.2 → 0.1.3

package/CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.1.3] - 2026-01-30
+
+### Added
+
+- `--output` option for eject command to specify custom output directory
+- Remote schema fetching for skill validation from GitHub
+
+### Fixed
+
+- Plugin manifest no longer includes agents field (Claude Code discovers automatically)
+- Source loader now supports source-provided skills matrix
+
+[0.1.3]: https://github.com/claude-collective/cli/releases/tag/v0.1.3
+
 ## [0.1.2] - 2026-01-30
 
 ### Changed

package/dist/cli/index.js

@@ -891,9 +891,6 @@ function generateStackPluginManifest(options) {
   if (options.keywords && options.keywords.length > 0) {
     manifest.keywords = options.keywords;
   }
-  if (options.hasAgents) {
-    manifest.agents = "./agents/";
-  }
   if (options.hasHooks) {
     manifest.hooks = "./hooks/hooks.json";
   }
@@ -3329,12 +3326,97 @@ async function runWizard(matrix, options = {}) {
 // src/cli/lib/source-loader.ts
 import path21 from "path";
 
-// src/cli/lib/matrix-loader.ts
+// src/cli/lib/local-skill-loader.ts
 import { parse as parseYaml7 } from "yaml";
 import path19 from "path";
+var LOCAL_CATEGORY = "local";
+var LOCAL_AUTHOR = "@local";
+var TEST_SKILL_PREFIX = "test-";
+async function discoverLocalSkills(projectDir) {
+  const localSkillsPath = path19.join(projectDir, LOCAL_SKILLS_PATH);
+  if (!await directoryExists(localSkillsPath)) {
+    verbose(`Local skills directory not found: ${localSkillsPath}`);
+    return null;
+  }
+  const skills = [];
+  const skillDirs = await listDirectories(localSkillsPath);
+  for (const skillDirName of skillDirs) {
+    if (!skillDirName.startsWith(TEST_SKILL_PREFIX)) {
+      verbose(
+        `Skipping local skill '${skillDirName}': Does not have test- prefix (temporary filter)`
+      );
+      continue;
+    }
+    const skill = await extractLocalSkill(localSkillsPath, skillDirName);
+    if (skill) {
+      skills.push(skill);
+    }
+  }
+  verbose(`Discovered ${skills.length} local skills from ${localSkillsPath}`);
+  return {
+    skills,
+    localSkillsPath
+  };
+}
+async function extractLocalSkill(localSkillsPath, skillDirName) {
+  const skillDir = path19.join(localSkillsPath, skillDirName);
+  const metadataPath = path19.join(skillDir, "metadata.yaml");
+  const skillMdPath = path19.join(skillDir, "SKILL.md");
+  if (!await fileExists(metadataPath)) {
+    verbose(`Skipping local skill '${skillDirName}': No metadata.yaml found`);
+    return null;
+  }
+  if (!await fileExists(skillMdPath)) {
+    verbose(`Skipping local skill '${skillDirName}': No SKILL.md found`);
+    return null;
+  }
+  const metadataContent = await readFile(metadataPath);
+  const metadata = parseYaml7(metadataContent);
+  if (!metadata.cli_name) {
+    verbose(
+      `Skipping local skill '${skillDirName}': Missing required 'cli_name' in metadata.yaml`
+    );
+    return null;
+  }
+  const skillMdContent = await readFile(skillMdPath);
+  const frontmatter = parseFrontmatter(skillMdContent);
+  if (!frontmatter) {
+    verbose(
+      `Skipping local skill '${skillDirName}': Invalid SKILL.md frontmatter`
+    );
+    return null;
+  }
+  const relativePath = `${LOCAL_SKILLS_PATH}/${skillDirName}/`;
+  const skillId = frontmatter.name;
+  const extracted = {
+    id: skillId,
+    directoryPath: skillDirName,
+    name: `${metadata.cli_name} ${LOCAL_AUTHOR}`,
+    description: metadata.cli_description || frontmatter.description,
+    usageGuidance: void 0,
+    category: LOCAL_CATEGORY,
+    categoryExclusive: false,
+    author: LOCAL_AUTHOR,
+    tags: [],
+    compatibleWith: [],
+    conflictsWith: [],
+    requires: [],
+    requiresSetup: [],
+    providesSetupFor: [],
+    path: relativePath,
+    local: true,
+    localPath: relativePath
+  };
+  verbose(`Extracted local skill: ${skillId}`);
+  return extracted;
+}
+
+// src/cli/lib/matrix-loader.ts
+import { parse as parseYaml8 } from "yaml";
+import path20 from "path";
 async function loadSkillsMatrix(configPath) {
   const content = await readFile(configPath);
-  const config = parseYaml7(content);
+  const config = parseYaml8(content);
   validateMatrixStructure(config, configPath);
   verbose(`Loaded skills matrix: ${configPath}`);
   return config;
@@ -3390,15 +3472,15 @@ async function extractAllSkills(skillsDir) {
   const skills = [];
   const metadataFiles = await glob("**/metadata.yaml", skillsDir);
   for (const metadataFile of metadataFiles) {
-    const skillDir = path19.dirname(metadataFile);
-    const skillMdPath = path19.join(skillsDir, skillDir, "SKILL.md");
-    const metadataPath = path19.join(skillsDir, metadataFile);
+    const skillDir = path20.dirname(metadataFile);
+    const skillMdPath = path20.join(skillsDir, skillDir, "SKILL.md");
+    const metadataPath = path20.join(skillsDir, metadataFile);
     if (!await fileExists(skillMdPath)) {
       verbose(`Skipping ${metadataFile}: No SKILL.md found`);
       continue;
     }
     const metadataContent = await readFile(metadataPath);
-    const metadata = parseYaml7(metadataContent);
+    const metadata = parseYaml8(metadataContent);
     const skillMdContent = await readFile(skillMdPath);
     const frontmatter = parseFrontmatter(skillMdContent);
     if (!frontmatter) {
@@ -3686,91 +3768,6 @@ function resolveSuggestedStacks(matrix, aliases) {
   });
 }
 
-// src/cli/lib/local-skill-loader.ts
-import { parse as parseYaml8 } from "yaml";
-import path20 from "path";
-var LOCAL_CATEGORY = "local";
-var LOCAL_AUTHOR = "@local";
-var TEST_SKILL_PREFIX = "test-";
-async function discoverLocalSkills(projectDir) {
-  const localSkillsPath = path20.join(projectDir, LOCAL_SKILLS_PATH);
-  if (!await directoryExists(localSkillsPath)) {
-    verbose(`Local skills directory not found: ${localSkillsPath}`);
-    return null;
-  }
-  const skills = [];
-  const skillDirs = await listDirectories(localSkillsPath);
-  for (const skillDirName of skillDirs) {
-    if (!skillDirName.startsWith(TEST_SKILL_PREFIX)) {
-      verbose(
-        `Skipping local skill '${skillDirName}': Does not have test- prefix (temporary filter)`
-      );
-      continue;
-    }
-    const skill = await extractLocalSkill(localSkillsPath, skillDirName);
-    if (skill) {
-      skills.push(skill);
-    }
-  }
-  verbose(`Discovered ${skills.length} local skills from ${localSkillsPath}`);
-  return {
-    skills,
-    localSkillsPath
-  };
-}
-async function extractLocalSkill(localSkillsPath, skillDirName) {
-  const skillDir = path20.join(localSkillsPath, skillDirName);
-  const metadataPath = path20.join(skillDir, "metadata.yaml");
-  const skillMdPath = path20.join(skillDir, "SKILL.md");
-  if (!await fileExists(metadataPath)) {
-    verbose(`Skipping local skill '${skillDirName}': No metadata.yaml found`);
-    return null;
-  }
-  if (!await fileExists(skillMdPath)) {
-    verbose(`Skipping local skill '${skillDirName}': No SKILL.md found`);
-    return null;
-  }
-  const metadataContent = await readFile(metadataPath);
-  const metadata = parseYaml8(metadataContent);
-  if (!metadata.cli_name) {
-    verbose(
-      `Skipping local skill '${skillDirName}': Missing required 'cli_name' in metadata.yaml`
-    );
-    return null;
-  }
-  const skillMdContent = await readFile(skillMdPath);
-  const frontmatter = parseFrontmatter(skillMdContent);
-  if (!frontmatter) {
-    verbose(
-      `Skipping local skill '${skillDirName}': Invalid SKILL.md frontmatter`
-    );
-    return null;
-  }
-  const relativePath = `${LOCAL_SKILLS_PATH}/${skillDirName}/`;
-  const skillId = frontmatter.name;
-  const extracted = {
-    id: skillId,
-    directoryPath: skillDirName,
-    name: `${metadata.cli_name} ${LOCAL_AUTHOR}`,
-    description: metadata.cli_description || frontmatter.description,
-    usageGuidance: void 0,
-    category: LOCAL_CATEGORY,
-    categoryExclusive: false,
-    author: LOCAL_AUTHOR,
-    tags: [],
-    compatibleWith: [],
-    conflictsWith: [],
-    requires: [],
-    requiresSetup: [],
-    providesSetupFor: [],
-    path: relativePath,
-    local: true,
-    localPath: relativePath
-  };
-  verbose(`Extracted local skill: ${skillId}`);
-  return extracted;
-}
-
 // src/cli/lib/source-loader.ts
 async function loadSkillsMatrixFromSource(options = {}) {
   const {
@@ -3810,9 +3807,17 @@ async function loadFromLocal(source, sourceConfig) {
     skillsPath = PROJECT_ROOT;
   }
   verbose(`Loading skills from local path: ${skillsPath}`);
-  const matrixPath = path21.join(PROJECT_ROOT, SKILLS_MATRIX_PATH);
+  const sourceMatrixPath = path21.join(skillsPath, SKILLS_MATRIX_PATH);
+  const cliMatrixPath = path21.join(PROJECT_ROOT, SKILLS_MATRIX_PATH);
+  let matrixPath;
+  if (await fileExists(sourceMatrixPath)) {
+    matrixPath = sourceMatrixPath;
+    verbose(`Matrix from source: ${matrixPath}`);
+  } else {
+    matrixPath = cliMatrixPath;
+    verbose(`Matrix from CLI (source has no matrix): ${matrixPath}`);
+  }
   const skillsDir = path21.join(skillsPath, SKILLS_DIR_PATH);
-  verbose(`Matrix from CLI: ${matrixPath}`);
   verbose(`Skills from source: ${skillsDir}`);
   const matrix = await loadSkillsMatrix(matrixPath);
   const skills = await extractAllSkills(skillsDir);
@@ -3828,9 +3833,17 @@ async function loadFromRemote(source, sourceConfig, forceRefresh) {
   verbose(`Fetching skills from remote source: ${source}`);
   const fetchResult = await fetchFromSource(source, { forceRefresh });
   verbose(`Fetched to: ${fetchResult.path}`);
-  const matrixPath = path21.join(PROJECT_ROOT, SKILLS_MATRIX_PATH);
+  const sourceMatrixPath = path21.join(fetchResult.path, SKILLS_MATRIX_PATH);
+  const cliMatrixPath = path21.join(PROJECT_ROOT, SKILLS_MATRIX_PATH);
+  let matrixPath;
+  if (await fileExists(sourceMatrixPath)) {
+    matrixPath = sourceMatrixPath;
+    verbose(`Matrix from source: ${matrixPath}`);
+  } else {
+    matrixPath = cliMatrixPath;
+    verbose(`Matrix from CLI (source has no matrix): ${matrixPath}`);
+  }
   const skillsDir = path21.join(fetchResult.path, SKILLS_DIR_PATH);
-  verbose(`Matrix from CLI: ${matrixPath}`);
   verbose(`Skills from source: ${skillsDir}`);
   const matrix = await loadSkillsMatrix(matrixPath);
   const skills = await extractAllSkills(skillsDir);
@@ -5182,6 +5195,9 @@ var KEBAB_CASE_REGEX = /^[a-z][a-z0-9]*(-[a-z0-9]+)*$/;
 var SEMVER_REGEX = /^(0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)(?:-((?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\.(?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\+([0-9a-zA-Z-]+(?:\.[0-9a-zA-Z-]+)*))?$/;
 var schemaCache2 = /* @__PURE__ */ new Map();
 var validatorCache2 = /* @__PURE__ */ new Map();
+var REMOTE_SCHEMAS = {
+  "skill-frontmatter.schema.json": "https://raw.githubusercontent.com/claude-collective/skills/main/src/schemas/skill-frontmatter.schema.json"
+};
 async function loadSchema2(schemaName) {
   if (schemaCache2.has(schemaName)) {
     return schemaCache2.get(schemaName);
@@ -5198,8 +5214,20 @@ async function loadSchema2(schemaName) {
       return schema;
     }
   }
+  const remoteUrl = REMOTE_SCHEMAS[schemaName];
+  if (remoteUrl) {
+    try {
+      const response = await fetch(remoteUrl);
+      if (response.ok) {
+        const schema = await response.json();
+        schemaCache2.set(schemaName, schema);
+        return schema;
+      }
+    } catch {
+    }
+  }
   throw new Error(
-    `Schema not found: ${schemaName}. Searched: ${locations.join(", ")}`
+    `Schema not found: ${schemaName}. Searched: ${locations.join(", ")}${remoteUrl ? ` and ${remoteUrl}` : ""}`
   );
 }
 async function getValidator2(schemaName) {
@@ -5909,6 +5937,7 @@ import { Command as Command10 } from "commander";
 import * as p12 from "@clack/prompts";
 import pc12 from "picocolors";
 import path30 from "path";
+import os4 from "os";
 var EJECT_TYPES = ["templates", "skills", "config", "all"];
 var DEFAULT_CONFIG_CONTENT = `# Claude Collective Configuration
 # Agent-skill mappings for this project
@@ -5959,44 +5988,65 @@ author: "@local"
 cli_name: Example Skill
 cli_description: Short description for CLI
 `;
-var ejectCommand = new Command10("eject").description("Eject bundled content for local customization").argument("[type]", "What to eject: templates, skills, config, all").option("-f, --force", "Overwrite existing files", false).configureOutput({
+var ejectCommand = new Command10("eject").description("Eject bundled content for local customization").argument("[type]", "What to eject: templates, skills, config, all").option("-f, --force", "Overwrite existing files", false).option(
+  "-o, --output <dir>",
+  "Output directory (default: .claude/ in current directory)"
+).configureOutput({
   writeErr: (str) => console.error(pc12.red(str))
-}).showHelpAfterError(true).action(async (type, options) => {
-  const projectDir = process.cwd();
-  if (!type) {
-    p12.log.error(
-      "Please specify what to eject: templates, skills, config, or all"
-    );
-    process.exit(EXIT_CODES.INVALID_ARGS);
-  }
-  if (!EJECT_TYPES.includes(type)) {
-    p12.log.error(`Unknown eject type: ${type}`);
-    p12.log.info(`Valid types: ${EJECT_TYPES.join(", ")}`);
-    process.exit(EXIT_CODES.INVALID_ARGS);
-  }
-  p12.intro(pc12.cyan("Claude Collective Eject"));
-  const ejectType = type;
-  switch (ejectType) {
-    case "templates":
-      await ejectTemplates(projectDir, options.force);
-      break;
-    case "skills":
-      await ejectSkills(projectDir, options.force);
-      break;
-    case "config":
-      await ejectConfig(projectDir, options.force);
-      break;
-    case "all":
-      await ejectTemplates(projectDir, options.force);
-      await ejectSkills(projectDir, options.force);
-      await ejectConfig(projectDir, options.force);
-      break;
+}).showHelpAfterError(true).action(
+  async (type, options) => {
+    const projectDir = process.cwd();
+    if (!type) {
+      p12.log.error(
+        "Please specify what to eject: templates, skills, config, or all"
+      );
+      process.exit(EXIT_CODES.INVALID_ARGS);
+    }
+    if (!EJECT_TYPES.includes(type)) {
+      p12.log.error(`Unknown eject type: ${type}`);
+      p12.log.info(`Valid types: ${EJECT_TYPES.join(", ")}`);
+      process.exit(EXIT_CODES.INVALID_ARGS);
+    }
+    let outputBase;
+    if (options.output) {
+      const expandedPath = options.output.startsWith("~") ? path30.join(os4.homedir(), options.output.slice(1)) : options.output;
+      outputBase = path30.resolve(projectDir, expandedPath);
+      if (await fileExists(outputBase)) {
+        p12.log.error(`Output path exists as a file: ${outputBase}`);
+        p12.log.info("Please specify a directory path, not a file.");
+        process.exit(EXIT_CODES.INVALID_ARGS);
+      }
+    } else {
+      outputBase = path30.join(projectDir, ".claude");
+    }
+    p12.intro(pc12.cyan("Claude Collective Eject"));
+    if (options.output) {
+      p12.log.info(`Output directory: ${pc12.cyan(outputBase)}`);
+    }
+    const ejectType = type;
+    const directOutput = !!options.output;
+    switch (ejectType) {
+      case "templates":
+        await ejectTemplates(outputBase, options.force, directOutput);
+        break;
+      case "skills":
+        await ejectSkills(outputBase, options.force, directOutput);
+        break;
+      case "config":
+        await ejectConfig(outputBase, options.force, directOutput);
+        break;
+      case "all":
+        await ejectTemplates(outputBase, options.force, directOutput);
+        await ejectSkills(outputBase, options.force, directOutput);
+        await ejectConfig(outputBase, options.force, directOutput);
+        break;
+    }
+    p12.outro(pc12.green("Eject complete!"));
   }
-  p12.outro(pc12.green("Eject complete!"));
-});
-async function ejectTemplates(projectDir, force) {
+);
+async function ejectTemplates(outputBase, force, directOutput = false) {
   const sourceDir = path30.join(PROJECT_ROOT, DIRS.templates);
-  const destDir = path30.join(projectDir, ".claude", "templates");
+  const destDir = directOutput ? outputBase : path30.join(outputBase, "templates");
   if (await directoryExists(destDir) && !force) {
     p12.log.warn(
       `Templates already exist at ${destDir}. Use --force to overwrite.`
@@ -6010,8 +6060,8 @@ async function ejectTemplates(projectDir, force) {
     pc12.dim("You can now customize agent.liquid and partials locally.")
   );
 }
-async function ejectSkills(projectDir, force) {
-  const destDir = path30.join(projectDir, ".claude", "skill-templates");
+async function ejectSkills(outputBase, force, directOutput = false) {
+  const destDir = directOutput ? outputBase : path30.join(outputBase, "skill-templates");
   if (await directoryExists(destDir) && !force) {
     p12.log.warn(
       `Skill templates already exist at ${destDir}. Use --force to overwrite.`
@@ -6030,10 +6080,12 @@ async function ejectSkills(projectDir, force) {
     DEFAULT_METADATA_CONTENT
   );
   p12.log.success(`Skill templates ejected to ${pc12.cyan(destDir)}`);
-  p12.log.info(pc12.dim("Copy example-skill/ to .claude/skills/ and customize."));
+  p12.log.info(
+    pc12.dim("Copy example-skill/ to your skills/ directory and customize.")
+  );
 }
-async function ejectConfig(projectDir, force) {
-  const destPath = path30.join(projectDir, ".claude", "config.yaml");
+async function ejectConfig(outputBase, force, directOutput = false) {
+  const destPath = path30.join(outputBase, "config.yaml");
   if (await fileExists(destPath) && !force) {
     p12.log.warn(
       `Config already exists at ${destPath}. Use --force to overwrite.`