@biggora/claude-plugins 1.3.0 → 1.3.1

package/README.md

@@ -226,6 +226,7 @@ Skills are simpler than plugins — just a SKILL.md file with optional supportin
 ```
 my-skill/
   SKILL.md            # Required: skill instructions with YAML frontmatter
+  commands/           # Slash commands (optional, copied to ~/.claude/commands/ on install)
   references/         # Reference docs (optional)
   scripts/            # Helper scripts (optional)
 ```
@@ -262,8 +263,9 @@ The plugin registry is a GitHub-hosted JSON file at [biggora/claude-plugins-regi
 
 - **Plugins** are installed via `git clone` to `~/.claude/plugins/<name>`
 - **Skills** are installed via `git clone` + copy to `~/.claude/skills/<name>`
+- **Slash commands** bundled with skills are automatically copied to `~/.claude/commands/`
 - Updates use `git pull --ff-only` to safely fast-forward
-- Claude Code automatically discovers plugins and skills in `~/.claude/`
+- Claude Code automatically discovers plugins in `~/.claude/plugins/`, skills in `~/.claude/skills/`, and commands in `~/.claude/commands/`
 - Restart Claude Code after installing or removing plugins/skills
 
 ## Requirements

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@biggora/claude-plugins",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "CLI marketplace for discovering, installing, and managing Claude Code plugins",
   "keywords": [
     "claude",

package/specs/coding.md

@@ -28,3 +28,8 @@ create skill eslint-expert src/skills/eslint-expert. The skill should include kn
 create slash command typescript-fix for the TypeScript expert (typescript-expert) skill. with some content like this:
 "You are a TypeScript expert with deep knowledge of complex typing patterns, modern JavaScript (ES6+), ESLint configuration, and enterprise-grade development practices. You specialize in building reliable, maintainable, and performant TypeScript solutions for complex business problems, with a solid understanding of JavaScript fundamentals and code quality tools. Read package.json to identify the command to run the linter. Use the `typescript-expert` skill to run `npm run typecheck`, `pnpm run typecheck`, or `tsc --noEmit` in the project directory and resolve TypeScript errors."
 
+the slash command is copied to the same skill subfolder ~/.claude/skills/typescript-expert/commands/typescript-fix.md, but must be copied to ~/.claude/commands/typescript-fix.md
+
+
+try this command  npx skills add ./ --skill typescript-expert and check C:\Users\biggora\.claude\commands. Looks
+like it does not work

package/src/commands/skills/add.js

@@ -1,11 +1,11 @@
 import { execFileSync } from 'node:child_process';
-import { existsSync, readFileSync, readdirSync, cpSync, rmSync, mkdirSync, writeFileSync, statSync } from 'node:fs';
-import { join, basename } from 'node:path';
+import { existsSync, readFileSync, readdirSync, cpSync, rmSync, mkdirSync, writeFileSync, statSync, copyFileSync } from 'node:fs';
+import { join, basename, resolve } from 'node:path';
 import { tmpdir } from 'node:os';
-import { getSkillsDir, getPluginsDir } from '../../config.js';
+import { getSkillsDir, getCommandsDir } from '../../config.js';
 import { fetchRegistry, findPlugin } from '../../registry.js';
 import { log, spinner } from '../../utils.js';
-import { hasPluginComponents, detectComponents, findInstalledSkill } from './resolve.js';
+import { detectComponents, findInstalledSkill } from './resolve.js';
 
 export function parseFrontmatter(content) {
   const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
@@ -49,6 +49,15 @@ function isUrl(str) {
   return str.startsWith('http://') || str.startsWith('https://') || str.startsWith('git@');
 }
 
+function isLocalPath(str) {
+  if (str.startsWith('./') || str.startsWith('../') || str.startsWith('/')) return true;
+  // Windows absolute paths like C:\ or E:\
+  if (/^[a-zA-Z]:[/\\]/.test(str)) return true;
+  // Explicit . for current directory
+  if (str === '.') return true;
+  return false;
+}
+
 function makeTempDir() {
   const dir = join(tmpdir(), `claude-skill-${Date.now()}`);
   mkdirSync(dir, { recursive: true });
@@ -57,9 +66,19 @@ function makeTempDir() {
 
 export async function add(source, options = {}) {
   let repoUrl = source;
+  let sourceDir = null;
 
-  // If not a URL, look up in registry
-  if (!isUrl(source)) {
+  if (isLocalPath(source)) {
+    // Local directory — use directly, no git clone
+    sourceDir = resolve(source);
+    if (!existsSync(sourceDir)) {
+      log.error(`Local path "${sourceDir}" does not exist`);
+      process.exit(1);
+    }
+    repoUrl = sourceDir;
+    log.info(`Using local path: ${sourceDir}`);
+  } else if (!isUrl(source)) {
+    // Not a URL and not a local path — look up in registry
     const registry = await fetchRegistry();
     const entry = findPlugin(registry, source);
     if (!entry) {
@@ -71,29 +90,36 @@ export async function add(source, options = {}) {
     log.info(`Resolved "${source}" to ${repoUrl}`);
   }
 
-  const tmpDir = makeTempDir();
-  const spin = spinner(`Cloning ${repoUrl}...`);
-  spin.start();
-
-  try {
-    const gitUrl = repoUrl.endsWith('.git') ? repoUrl : `${repoUrl}.git`;
-    execFileSync('git', ['clone', '--depth', '1', gitUrl, tmpDir], {
-      stdio: 'pipe',
-    });
-    spin.succeed('Repository cloned');
-  } catch (err) {
-    spin.fail('Failed to clone repository');
-    log.error(err.message);
-    rmSync(tmpDir, { recursive: true, force: true });
-    process.exit(1);
+  let tmpDir = null;
+
+  // Clone if remote URL, otherwise use local path directly
+  if (!sourceDir) {
+    tmpDir = makeTempDir();
+    const spin = spinner(`Cloning ${repoUrl}...`);
+    spin.start();
+
+    try {
+      const gitUrl = repoUrl.endsWith('.git') ? repoUrl : `${repoUrl}.git`;
+      execFileSync('git', ['clone', '--depth', '1', gitUrl, tmpDir], {
+        stdio: 'pipe',
+      });
+      spin.succeed('Repository cloned');
+    } catch (err) {
+      spin.fail('Failed to clone repository');
+      log.error(err.message);
+      rmSync(tmpDir, { recursive: true, force: true });
+      process.exit(1);
+    }
   }
 
+  const searchRoot = sourceDir || tmpDir;
+
   // Find all skill directories
-  const skillDirs = findSkillDirs(tmpDir);
+  const skillDirs = findSkillDirs(searchRoot);
 
   if (!skillDirs.length) {
     log.error('No SKILL.md found in repository');
-    rmSync(tmpDir, { recursive: true, force: true });
+    if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
     process.exit(1);
   }
 
@@ -118,22 +144,22 @@ export async function add(source, options = {}) {
         const fm = parseFrontmatter(readFileSync(join(d, 'SKILL.md'), 'utf-8'));
         log.dim(`  - ${fm.name || basename(d)}`);
       }
-      rmSync(tmpDir, { recursive: true, force: true });
+      if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
       process.exit(1);
     }
   } else if (skillDirs.length === 1) {
     targetDir = skillDirs[0];
   } else {
     // Check if root has SKILL.md
-    if (existsSync(join(tmpDir, 'SKILL.md'))) {
-      targetDir = tmpDir;
+    if (existsSync(join(searchRoot, 'SKILL.md'))) {
+      targetDir = searchRoot;
     } else {
       log.warn('Multiple skills found. Use --skill <name> to select one:');
       for (const d of skillDirs) {
         const fm = parseFrontmatter(readFileSync(join(d, 'SKILL.md'), 'utf-8'));
         log.dim(`  - ${fm.name || basename(d)}`);
       }
-      rmSync(tmpDir, { recursive: true, force: true });
+      if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
       process.exit(1);
     }
   }
@@ -141,32 +167,30 @@ export async function add(source, options = {}) {
   // Parse skill metadata
   const skillMd = readFileSync(join(targetDir, 'SKILL.md'), 'utf-8');
   const frontmatter = parseFrontmatter(skillMd);
-  const skillName = frontmatter.name || options.skill || basename(targetDir === tmpDir ? repoUrl.replace(/\.git$/, '').split('/').pop() : targetDir);
+  const skillName = frontmatter.name || options.skill || basename(targetDir === searchRoot ? repoUrl.replace(/\.git$/, '').split('/').pop() : targetDir);
 
-  // Detect plugin components (commands, hooks, agents) alongside SKILL.md
+  // Detect components alongside SKILL.md
   const components = detectComponents(targetDir);
-  const installAsPlugin = hasPluginComponents(targetDir);
 
-  const destDir = installAsPlugin ? getPluginsDir() : getSkillsDir();
-  const dest = join(destDir, skillName);
+  const skillsDir = getSkillsDir();
+  const dest = join(skillsDir, skillName);
 
   // Check both locations for existing installation
   const existing = findInstalledSkill(skillName);
   if (existing) {
     log.warn(`Skill "${skillName}" is already installed at ${existing.dir}`);
     log.dim(`Run "claude-plugins skills update ${skillName}" to update it`);
-    rmSync(tmpDir, { recursive: true, force: true });
+    if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
     return;
   }
 
   if (existsSync(dest)) {
     log.warn(`"${skillName}" already exists at ${dest}`);
-    rmSync(tmpDir, { recursive: true, force: true });
+    if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
     return;
   }
 
-  const label = installAsPlugin ? 'skill + components' : 'skill';
-  const spin2 = spinner(`Installing ${label} "${skillName}"...`);
+  const spin2 = spinner(`Installing skill "${skillName}"...`);
   spin2.start();
 
   try {
@@ -178,24 +202,8 @@ export async function add(source, options = {}) {
       rmSync(gitInDest, { recursive: true, force: true });
     }
 
-    // Generate .claude-plugin/plugin.json if installing as plugin and none exists
-    if (installAsPlugin && !existsSync(join(dest, '.claude-plugin', 'plugin.json'))) {
-      const pluginDir = join(dest, '.claude-plugin');
-      mkdirSync(pluginDir, { recursive: true });
-
-      const pluginJson = {
-        name: skillName,
-        version: '1.0.0',
-        description: frontmatter.description || `Skill: ${skillName}`,
-        commands: listCommandFiles(dest),
-        _generatedBy: 'claude-plugins skills add',
-      };
-
-      writeFileSync(
-        join(pluginDir, 'plugin.json'),
-        JSON.stringify(pluginJson, null, 2)
-      );
-    }
+    // Copy slash commands to ~/.claude/commands/ so Claude Code discovers them
+    const installedCommands = installCommands(dest, skillName);
 
     // Write origin metadata
     writeFileSync(
@@ -204,8 +212,8 @@ export async function add(source, options = {}) {
         {
           repository: repoUrl,
           skill: options.skill || null,
-          installedAs: installAsPlugin ? 'plugin-skill' : 'skill',
           components,
+          installedCommands,
           installedAt: new Date().toISOString(),
         },
         null,
@@ -219,10 +227,13 @@ export async function add(source, options = {}) {
       log.dim(`  ${frontmatter.description}`);
     }
 
-    if (installAsPlugin) {
-      const extras = components.filter((c) => c !== 'skill');
+    if (installedCommands.length) {
+      log.info(`Commands: ${installedCommands.map((c) => `/${c}`).join(', ')}`);
+      log.dim(`  Copied to ${getCommandsDir()}`);
+    }
+
+    if (components.length > 1) {
       log.info(`Detected: ${components.join(', ')}`);
-      log.dim(`Installed as plugin so Claude Code discovers ${extras.join(', ')}`);
     }
 
     log.dim('\nRestart Claude Code to load the skill.');
@@ -230,21 +241,38 @@ export async function add(source, options = {}) {
     spin2.fail(`Failed to install skill "${skillName}"`);
     log.error(err.message);
   } finally {
-    rmSync(tmpDir, { recursive: true, force: true });
+    if (tmpDir) rmSync(tmpDir, { recursive: true, force: true });
   }
 }
 
 /**
- * List command file names (without extension) from a commands/ directory.
+ * Copy command .md files from skill's commands/ dir to ~/.claude/commands/.
+ * Returns array of installed command names (without extension).
  */
-function listCommandFiles(dir) {
-  const cmdsDir = join(dir, 'commands');
+function installCommands(skillDir, skillName) {
+  const cmdsDir = join(skillDir, 'commands');
   if (!existsSync(cmdsDir)) return [];
+
+  const commandsDir = getCommandsDir();
+  const installed = [];
+
   try {
-    return readdirSync(cmdsDir)
-      .filter((f) => f.endsWith('.md'))
-      .map((f) => `/${f.replace(/\.md$/, '')}`);
+    const files = readdirSync(cmdsDir).filter((f) => f.endsWith('.md'));
+    for (const file of files) {
+      const src = join(cmdsDir, file);
+      const destFile = join(commandsDir, file);
+
+      if (existsSync(destFile)) {
+        log.warn(`Command "${file}" already exists in ${commandsDir}, skipping`);
+        continue;
+      }
+
+      copyFileSync(src, destFile);
+      installed.push(file.replace(/\.md$/, ''));
+    }
   } catch {
-    return [];
+    // ignore errors reading commands dir
   }
+
+  return installed;
 }

package/src/commands/skills/list.js

@@ -13,11 +13,13 @@ export async function list() {
 
   console.log(chalk.bold(`\n  ${skills.length} skill${skills.length === 1 ? '' : 's'} installed\n`));
 
-  const rows = skills.map(({ name, meta, location }) => {
-    const type = location === 'plugins' ? chalk.cyan('plugin') : chalk.dim('skill');
-    return [name, truncate(meta.description, 45), type, truncate(meta.repository, 35)];
+  const rows = skills.map(({ name, meta }) => {
+    const cmds = meta.commands.length
+      ? meta.commands.map((c) => `/${c}`).join(', ')
+      : '';
+    return [name, truncate(meta.description, 45), cmds, truncate(meta.repository, 35)];
   });
 
-  formatTable(rows, ['Name', 'Description', 'Type', 'Repository']);
+  formatTable(rows, ['Name', 'Description', 'Commands', 'Repository']);
   console.log();
 }

package/src/commands/skills/remove.js

@@ -1,6 +1,8 @@
-import { rmSync } from 'node:fs';
+import { existsSync, rmSync } from 'node:fs';
+import { join } from 'node:path';
+import { getCommandsDir } from '../../config.js';
 import { log, spinner } from '../../utils.js';
-import { findInstalledSkill } from './resolve.js';
+import { findInstalledSkill, readOrigin } from './resolve.js';
 
 export async function remove(name) {
   const found = findInstalledSkill(name);
@@ -15,8 +17,25 @@ export async function remove(name) {
   spin.start();
 
   try {
+    // Clean up commands that were copied to ~/.claude/commands/
+    const origin = readOrigin(found.dir);
+    if (origin && origin.installedCommands?.length) {
+      const commandsDir = getCommandsDir();
+      for (const cmd of origin.installedCommands) {
+        const cmdFile = join(commandsDir, `${cmd}.md`);
+        if (existsSync(cmdFile)) {
+          rmSync(cmdFile);
+        }
+      }
+    }
+
     rmSync(found.dir, { recursive: true, force: true });
-    spin.succeed(`Removed skill "${name}" from ${found.location}`);
+    spin.succeed(`Removed skill "${name}"`);
+
+    if (origin?.installedCommands?.length) {
+      log.dim(`  Also removed commands: ${origin.installedCommands.map((c) => `/${c}`).join(', ')}`);
+    }
+
     log.dim('Restart Claude Code to apply changes.');
   } catch (err) {
     spin.fail(`Failed to remove skill "${name}"`);

package/src/commands/skills/resolve.js

@@ -1,41 +1,28 @@
 import { existsSync, readFileSync, readdirSync } from 'node:fs';
 import { join } from 'node:path';
-import { getSkillsDir, getPluginsDir } from '../../config.js';
+import { getSkillsDir } from '../../config.js';
 import { parseFrontmatter } from './add.js';
 
 /**
- * Plugin component directories that Claude Code auto-discovers.
- * When a skill repo contains any of these, it must be installed
- * as a plugin so Claude Code can find them.
+ * Component directories that can accompany a SKILL.md.
  */
-export const PLUGIN_COMPONENT_DIRS = ['commands', 'hooks', 'agents'];
+export const COMPONENT_DIRS = ['commands', 'hooks', 'agents'];
 
 /**
- * Check if a directory contains plugin components (commands, hooks, agents)
- * beyond just SKILL.md + references.
- */
-export function hasPluginComponents(dir) {
-  return PLUGIN_COMPONENT_DIRS.some((name) => {
-    const p = join(dir, name);
-    return existsSync(p);
-  });
-}
-
-/**
- * Detect which plugin components exist in a directory.
+ * Detect which components exist in a directory.
  * Returns an array of found component names.
  */
 export function detectComponents(dir) {
   const found = [];
   if (existsSync(join(dir, 'SKILL.md'))) found.push('skill');
-  for (const name of PLUGIN_COMPONENT_DIRS) {
+  for (const name of COMPONENT_DIRS) {
     if (existsSync(join(dir, name))) found.push(name);
   }
   return found;
 }
 
 /**
- * Read .origin.json from a skill/plugin directory.
+ * Read .origin.json from a skill directory.
  */
 export function readOrigin(dir) {
   const originPath = join(dir, '.origin.json');
@@ -52,7 +39,7 @@ export function readOrigin(dir) {
  */
 export function readSkillMeta(dir, dirName) {
   const skillMdPath = join(dir, 'SKILL.md');
-  const meta = { name: dirName, description: '', repository: '-' };
+  const meta = { name: dirName, description: '', repository: '-', commands: [] };
 
   if (existsSync(skillMdPath)) {
     try {
@@ -67,53 +54,36 @@ export function readSkillMeta(dir, dirName) {
   const origin = readOrigin(dir);
   if (origin) {
     meta.repository = origin.repository || '-';
-    meta.installedAs = origin.installedAs || 'skill';
     meta.skill = origin.skill || null;
+    meta.commands = origin.installedCommands || [];
   }
 
   return meta;
 }
 
 /**
- * Find a skill by name across both ~/.claude/skills/ and ~/.claude/plugins/.
+ * Find a skill by name in ~/.claude/skills/.
  * Returns { dir, location } or null.
  */
 export function findInstalledSkill(name) {
   const skillsDir = getSkillsDir();
-  const pluginsDir = getPluginsDir();
-
-  // Check ~/.claude/skills/ first
   const skillPath = join(skillsDir, name);
+
   if (existsSync(skillPath) && existsSync(join(skillPath, 'SKILL.md'))) {
     return { dir: skillPath, location: 'skills' };
   }
 
-  // Check ~/.claude/plugins/ for skills installed as plugins
-  const pluginPath = join(pluginsDir, name);
-  if (existsSync(pluginPath)) {
-    const origin = readOrigin(pluginPath);
-    if (origin && origin.installedAs === 'plugin-skill') {
-      return { dir: pluginPath, location: 'plugins' };
-    }
-    // Also check if it has a SKILL.md (might be installed via skills add)
-    if (existsSync(join(pluginPath, 'SKILL.md'))) {
-      return { dir: pluginPath, location: 'plugins' };
-    }
-  }
-
   return null;
 }
 
 /**
- * List all installed skills from both directories.
- * Returns array of { name, dir, location, meta }.
+ * List all installed skills from ~/.claude/skills/.
+ * Returns array of { name, dir, meta }.
  */
 export function listAllSkills() {
   const skillsDir = getSkillsDir();
-  const pluginsDir = getPluginsDir();
   const results = [];
 
-  // Skills from ~/.claude/skills/
   try {
     const entries = readdirSync(skillsDir, { withFileTypes: true });
     for (const entry of entries) {
@@ -123,7 +93,6 @@ export function listAllSkills() {
       results.push({
         name: entry.name,
         dir,
-        location: 'skills',
         meta: readSkillMeta(dir, entry.name),
       });
     }
@@ -131,25 +100,5 @@ export function listAllSkills() {
     // directory might not exist yet
   }
 
-  // Skills from ~/.claude/plugins/ (installed as plugin-skill)
-  try {
-    const entries = readdirSync(pluginsDir, { withFileTypes: true });
-    for (const entry of entries) {
-      if (!entry.isDirectory()) continue;
-      const dir = join(pluginsDir, entry.name);
-      const origin = readOrigin(dir);
-      if (origin && origin.installedAs === 'plugin-skill') {
-        results.push({
-          name: entry.name,
-          dir,
-          location: 'plugins',
-          meta: readSkillMeta(dir, entry.name),
-        });
-      }
-    }
-  } catch {
-    // directory might not exist yet
-  }
-
   return results;
 }

package/src/config.js

@@ -7,6 +7,7 @@ const isWindows = process.platform === 'win32';
 
 export const PLUGINS_DIR = join(home, '.claude', 'plugins');
 export const SKILLS_DIR = join(home, '.claude', 'skills');
+export const COMMANDS_DIR = join(home, '.claude', 'commands');
 export const CACHE_DIR = join(home, '.claude', '.cache', 'claude-plugins');
 export const CACHE_TTL = 1000 * 60 * 15; // 15 minutes
 export const REGISTRY_URL =
@@ -27,6 +28,10 @@ export function getSkillsDir() {
   return ensureDir(SKILLS_DIR);
 }
 
+export function getCommandsDir() {
+  return ensureDir(COMMANDS_DIR);
+}
+
 export function getCacheDir() {
   return ensureDir(CACHE_DIR);
 }