@ikyyofc/gemini-cli 3.0.3 → 3.0.5

package/index.js

@@ -279,12 +279,15 @@ async function handleCommand(input) {
                 try {
                     const { output } = await installSkill(source, { global: isGlobal, skill: skillName, all });
                     sp.stop();
-                    // Print npx output
                     if (output) output.split("\n").filter(Boolean).forEach(l => printInfo(l));
                     printSuccess("skill installed — active on next message");
                 } catch (e) {
-                    sp.fail(e.message.split("\n")[0]);
-                    printError(e.message.split("\n")[0]);
+                    sp.stop();
+                    // Print full npx output so user sees what actually went wrong
+                    e.message.split("\n").filter(Boolean).forEach(l =>
+                        process.stdout.write(chalk.hex("#7A7A9A")("  " + l + "\n"))
+                    );
+                    printError("install failed");
                 }
                 break;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikyyofc/gemini-cli",
-  "version": "3.0.3",
+  "version": "3.0.5",
   "description": "AI Agent CLI — native function calling · GEMINI.md context · extensions",
   "type": "module",
   "bin": { "gemini": "./index.js" },

package/src/skills.js

@@ -1,11 +1,5 @@
 // src/skills.js — Skills manager via npx skills CLI
-// All install/search/list/remove operations delegate to "npx skills"
-// which is the official skills.sh CLI (github.com/vercel-labs/skills)
-//
-// Skills directories (Gemini agent convention from skills.sh):
-//   Project:  ./.gemini/skills/     (committed with project)
-//   Global:   ~/.gemini/skills/     (across all projects)
-//   Custom:   ./.agents/            (manual / local skills)
+// All skills are consolidated into a single path: ~/.agents/
 import fs   from "fs";
 import path from "path";
 import os   from "os";
@@ -14,176 +8,196 @@ import { exec }      from "child_process";
 
 const execAsync = promisify(exec);
 
-// Directories where npx skills installs for Gemini agent
-const GLOBAL_SKILLS_DIR  = path.join(os.homedir(), ".gemini", "skills");
-const PROJECT_SKILLS_DIR = () => path.join(process.cwd(), ".gemini", "skills");
-const CUSTOM_SKILLS_DIR  = () => path.join(process.cwd(), ".agents");
+// Single canonical location for ALL skills
+export const AGENTS_DIR = path.join(os.homedir(), ".agents");
+
+// Agent dirs that npx skills may create (we scan after install to find new files)
+const KNOWN_AGENT_DIRS = [
+    ".gemini", ".gemini/skills",
+    ".claude", ".claude/skills",
+    ".cursor", ".cursor/rules",
+    ".codex",  ".opencode",
+    ".cline",  ".aider",
+    ".amp",    ".copilot",
+];
 
 export function ensureSkillsDirs() {
-    fs.mkdirSync(GLOBAL_SKILLS_DIR,  { recursive: true });
+    fs.mkdirSync(AGENTS_DIR, { recursive: true });
+}
+
+// ─────────────────────────────────────────────────────────────────
+//  Scan a dir tree for SKILL.md files
+// ─────────────────────────────────────────────────────────────────
+function scanForSkillMds(baseDir, maxDepth = 5) {
+    const results = [];
+    const SKIP    = new Set(["node_modules", ".git", "dist", "build"]);
+    const walk    = (dir, depth) => {
+        if (depth > maxDepth || !fs.existsSync(dir)) return;
+        for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+            if (SKIP.has(entry.name)) continue;
+            const full = path.join(dir, entry.name);
+            if (entry.isDirectory())            walk(full, depth + 1);
+            else if (entry.name === "SKILL.md") results.push(full);
+        }
+    };
+    walk(baseDir, 0);
+    return results;
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Run npx skills with timeout
+//  Run npx skills — exposes real stderr on failure
 // ─────────────────────────────────────────────────────────────────
 async function npxSkills(args, cwd = process.cwd(), timeout = 120_000) {
     const cmd = `npx --yes skills ${args}`;
-    const { stdout, stderr } = await execAsync(cmd, {
-        cwd,
-        timeout,
-        env: {
-            ...process.env,
-            // Disable telemetry analytics
-            DISABLE_TELEMETRY: "1",
-            // Force non-interactive / no color for parsing
-            NO_COLOR: "1",
-            CI: "1",
-        },
-    });
-    return { stdout: stdout.trim(), stderr: stderr.trim() };
+    try {
+        const { stdout, stderr } = await execAsync(cmd, {
+            cwd, timeout,
+            env: { ...process.env, NO_COLOR: "1", CI: "1" },
+        });
+        return { stdout: stdout.trim(), stderr: stderr.trim() };
+    } catch (err) {
+        const detail = [err.stdout?.trim(), err.stderr?.trim()]
+            .filter(Boolean).join("\n") || err.message;
+        throw new Error(detail);
+    }
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Install — npx skills add <source> -a gemini -y [--global]
+//  Parse "owner/repo@skill-name" shorthand
 // ─────────────────────────────────────────────────────────────────
-export async function installSkill(source, opts = {}) {
-    const {
-        global  = false,
-        skill   = null,   // specific skill name (--skill flag)
-        all     = false,  // install all skills from repo
-    } = opts;
-
-    let args = `add ${source} -a gemini -y --copy`;
-    if (global) args += " -g";
-    if (skill)  args += ` --skill "${skill}"`;
-    if (all)    args += " --skill '*'";
-
-    const { stdout, stderr } = await npxSkills(args);
-    return { output: stdout || stderr };
+function parseSource(raw) {
+    const atIdx = raw.indexOf("@");
+    if (atIdx > 0 && !raw.startsWith("http") && !raw.startsWith("git@")) {
+        return { source: raw.slice(0, atIdx), skill: raw.slice(atIdx + 1) };
+    }
+    return { source: raw, skill: null };
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  List installed — npx skills list
+//  After npx install: copy new SKILL.md files → ~/.agents/<slug>/
 // ─────────────────────────────────────────────────────────────────
-export async function listNpxSkills(global = false) {
-    const args  = global ? "list -g" : "list";
-    const { stdout } = await npxSkills(args);
-    return stdout;
+function consolidate(cwd, beforeSnapshot) {
+    const copied = [];
+    const dest   = AGENTS_DIR;
+
+    // 1. Check known agent dirs first
+    for (const rel of KNOWN_AGENT_DIRS) {
+        for (const f of scanForSkillMds(path.join(cwd, rel), 3)) {
+            if (beforeSnapshot.has(f)) continue;
+            const slug = path.basename(path.dirname(f));
+            const dir  = path.join(dest, slug);
+            fs.mkdirSync(dir, { recursive: true });
+            fs.copyFileSync(f, path.join(dir, "SKILL.md"));
+            copied.push(slug);
+        }
+    }
+
+    // 2. Broader scan of cwd for anything missed
+    if (copied.length === 0) {
+        for (const f of scanForSkillMds(cwd, 5)) {
+            if (beforeSnapshot.has(f)) continue;
+            // Skip anything already inside ~/.agents
+            if (f.startsWith(dest)) continue;
+            const slug = path.basename(path.dirname(f));
+            const dir  = path.join(dest, slug);
+            fs.mkdirSync(dir, { recursive: true });
+            fs.copyFileSync(f, path.join(dir, "SKILL.md"));
+            copied.push(slug);
+        }
+    }
+
+    return [...new Set(copied)]; // deduplicate
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Search — npx skills find <query>
+//  Install
+//  1. Snapshot existing SKILL.md paths
+//  2. npx skills add (wherever it installs)
+//  3. Find new SKILL.md files and copy ALL → ~/.agents/
 // ─────────────────────────────────────────────────────────────────
-export async function findSkills(query) {
-    const { stdout } = await npxSkills(`find ${JSON.stringify(query)}`);
-    return stdout;
+export async function installSkill(rawSource, opts = {}) {
+    const { skill: explicitSkill = null, all = false } = opts;
+    const { source, skill: parsedSkill } = parseSource(rawSource);
+    const skillName = explicitSkill ?? parsedSkill;
+
+    const cwd    = process.cwd();
+    const before = new Set(
+        KNOWN_AGENT_DIRS.flatMap(rel => scanForSkillMds(path.join(cwd, rel), 3))
+    );
+
+    const parts = ["add", source, "-y"];
+    if (skillName) parts.push(`--skill "${skillName}"`);
+    if (all)       parts.push("--skill '*'");
+
+    const { stdout, stderr } = await npxSkills(parts.join(" "));
+    const installed = consolidate(cwd, before);
+
+    return { output: stdout || stderr, installed };
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Remove — npx skills remove <slug>
+//  Remove — delete from ~/.agents/ then best-effort npx remove
 // ─────────────────────────────────────────────────────────────────
 export async function removeSkillNpx(slug) {
-    const { stdout, stderr } = await npxSkills(`remove ${slug} -y`);
-    return { output: stdout || stderr };
+    const skillDir = path.join(AGENTS_DIR, slug);
+    if (!fs.existsSync(skillDir)) throw new Error(`"${slug}" not found in ~/.agents/`);
+    fs.rmSync(skillDir, { recursive: true, force: true });
+    try { await npxSkills(`remove ${slug} -y`); } catch {}
+    return { output: `Removed ~/.agents/${slug}` };
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Update — npx skills update [slug]
+//  Delegate to npx for search / list / update / init
 // ─────────────────────────────────────────────────────────────────
+export async function findSkills(query) {
+    const { stdout } = await npxSkills(`find ${JSON.stringify(query)}`);
+    return stdout;
+}
+export async function listNpxSkills() {
+    const { stdout } = await npxSkills("list");
+    return stdout;
+}
 export async function updateSkill(slug = "") {
-    const args = slug ? `update ${slug} -y` : "update -y";
-    const { stdout, stderr } = await npxSkills(args);
-    return { output: stdout || stderr };
+    const { stdout } = await npxSkills(slug ? `update ${slug} -y` : "update -y");
+    return stdout;
 }
-
-// ─────────────────────────────────────────────────────────────────
-//  Init — npx skills init [name]   (creates a SKILL.md template)
-// ─────────────────────────────────────────────────────────────────
 export async function initSkill(name = "") {
-    const args = name ? `init "${name}"` : "init";
-    const { stdout, stderr } = await npxSkills(args);
-    return { output: stdout || stderr };
+    const { stdout } = await npxSkills(name ? `init "${name}"` : "init");
+    return stdout;
 }
 
 // ─────────────────────────────────────────────────────────────────
-//  Load skills for agent context injection
-//  Scans all skill directories and reads SKILL.md files
+//  Load all skills from ~/.agents/ for agent context injection
 // ─────────────────────────────────────────────────────────────────
 export function loadSkills() {
     const skills = [];
-    const seen   = new Set();
-
-    const scanDir = (dir, scope) => {
-        if (!fs.existsSync(dir)) return;
-
-        const entries = fs.readdirSync(dir, { withFileTypes: true });
-        for (const entry of entries) {
-            if (!entry.isDirectory()) continue;
-
-            const skillDir = path.join(dir, entry.name);
-            const skillMd  = path.join(skillDir, "SKILL.md");
-            if (!fs.existsSync(skillMd)) continue;
-
-            const key = entry.name;
-            if (seen.has(key)) continue; // project takes priority over global
-            seen.add(key);
-
-            const content = fs.readFileSync(skillMd, "utf8");
-
-            // Parse optional frontmatter name from SKILL.md
-            const nameMatch = content.match(/^#\s+(.+)$/m);
-            const name      = nameMatch?.[1]?.trim() ?? entry.name;
-
-            skills.push({ name, slug: entry.name, content, scope, path: skillDir });
-        }
-    };
-
-    // Priority: project (.gemini/skills/) > custom (.agents/) > global (~/.gemini/skills/)
-    scanDir(PROJECT_SKILLS_DIR(), "project");
-    scanDir(CUSTOM_SKILLS_DIR(),  "custom");
-    scanDir(GLOBAL_SKILLS_DIR,    "global");
-
+    if (!fs.existsSync(AGENTS_DIR)) return skills;
+
+    for (const entry of fs.readdirSync(AGENTS_DIR, { withFileTypes: true })) {
+        if (!entry.isDirectory()) continue;
+        const skillMd = path.join(AGENTS_DIR, entry.name, "SKILL.md");
+        if (!fs.existsSync(skillMd)) continue;
+
+        const content   = fs.readFileSync(skillMd, "utf8");
+        const nameMatch = content.match(/^#\s+(.+)$/m);
+        skills.push({
+            name:    nameMatch?.[1]?.trim() ?? entry.name,
+            slug:    entry.name,
+            content,
+            path:    path.join(AGENTS_DIR, entry.name),
+        });
+    }
     return skills;
 }
 
-// ─────────────────────────────────────────────────────────────────
-//  Build skills section injected into agent system prompt
-// ─────────────────────────────────────────────────────────────────
 export function buildSkillsPrompt(skills) {
     if (!skills.length) return null;
-
     const sections = skills.map(s =>
         `### Skill: ${s.name}\n${s.content.trim()}`
     ).join("\n\n---\n\n");
-
-    return `## INSTALLED SKILLS\n\nApply the following skills when relevant to the current task:\n\n${sections}`;
+    return `## INSTALLED SKILLS\n\nApply the following skills when relevant:\n\n${sections}`;
 }
 
-// ─────────────────────────────────────────────────────────────────
-//  Quick disk check — list installed skills without running npx
-// ─────────────────────────────────────────────────────────────────
 export function listInstalledSkills() {
-    const result = [];
-
-    const scanDir = (dir, isGlobal) => {
-        if (!fs.existsSync(dir)) return;
-        for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
-            if (!entry.isDirectory()) continue;
-            const skillDir = path.join(dir, entry.name);
-            if (!fs.existsSync(path.join(skillDir, "SKILL.md"))) continue;
-            const content   = fs.readFileSync(path.join(skillDir, "SKILL.md"), "utf8");
-            const nameMatch = content.match(/^#\s+(.+)$/m);
-            result.push({
-                slug:   entry.name,
-                name:   nameMatch?.[1]?.trim() ?? entry.name,
-                global: isGlobal,
-                dir:    skillDir,
-            });
-        }
-    };
-
-    scanDir(PROJECT_SKILLS_DIR(), false);
-    scanDir(CUSTOM_SKILLS_DIR(),  false);
-    scanDir(GLOBAL_SKILLS_DIR,    true);
-    return result;
+    return loadSkills().map(s => ({ ...s, global: true, dir: s.path }));
 }