@agentskillkit/agent-skills 1.0.1 → 3.2.1

package/packages/cli/lib/hooks/install-hooks.js

@@ -16,42 +16,18 @@ import { VERSION } from "../config.js";
 // ============================================================================
 
 const PRE_COMMIT_HOOK = `#!/bin/sh
-# Smart Agent Pre-Commit Hook v${VERSION}
-# Auto-generated by ag-smart install-hooks
+# Agent Skill Kit Pre-Commit Hook v${VERSION}
 
-echo "🧠 Running Smart Agent memory check..."
-
-# Get staged files
-STAGED_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E "\\.(js|ts|tsx|jsx|mjs)$")
+# Get staged JS/TS files
+STAGED_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E "\\\\.(js|ts|tsx|jsx|mjs)$")
 
 if [ -z "$STAGED_FILES" ]; then
-    echo "✅ No JS/TS files staged."
-    exit 0
-fi
-
-# Run recall on each staged file
-HAS_ERRORS=0
-
-for FILE in $STAGED_FILES; do
-    if [ -f "$FILE" ]; then
-        npx ag-smart recall "$FILE" --quiet
-        if [ $? -eq 1 ]; then
-            HAS_ERRORS=1
-        fi
-    fi
-done
-
-if [ $HAS_ERRORS -eq 1 ]; then
-    echo ""
-    echo "⚠️  Violations found. Review above and fix ERROR-level issues."
-    echo "   To skip this check: git commit --no-verify"
-    echo ""
-    # Only warn, don't block (safe default)
     exit 0
 fi
 
-echo "✅ Memory check passed."
-exit 0
+# Run recall (Clack handles all output)
+npx ag-smart recall . 2>/dev/null
+exit $?
 `;
 
 // ============================================================================
@@ -80,7 +56,7 @@ function findGitDir() {
  * Install git hooks
  */
 function installHooks() {
-    console.log(`\n🔧 Smart Agent Hooks Installer v${VERSION}\n`);
+    console.log(`\n🔧 Agent Skill Kit Hooks Installer v${VERSION}\n`);
 
     const gitDir = findGitDir();
 
@@ -157,7 +133,7 @@ const args = process.argv.slice(2);
 
 if (args.includes("--help")) {
     console.log(`
-🔧 Smart Agent Hooks Installer
+🔧 Agent Skill Kit Hooks Installer
 
 Usage:
   ag-smart install-hooks           Install git hooks

package/packages/cli/lib/hooks/lint-learn.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 /**
- * Lint Learn - Auto-Learn from ESLint Output
+ * Lint Learn - Auto-Learn from ESLint Output (CLI v3.3.0)
  * 
  * Parses ESLint JSON output and creates lessons automatically.
  * 
@@ -10,7 +10,7 @@
  */
 
 import fs from "fs";
-import { loadKnowledge, saveKnowledge } from "../recall.v2.js";
+import { loadKnowledge, saveKnowledge } from "../recall.js";
 
 // ============================================================================
 // ESLINT PARSER

package/packages/cli/lib/icons.js

@@ -0,0 +1,93 @@
+/**
+ * Windows-safe icons utility
+ * 
+ * Provides ASCII fallback for emojis/Unicode when running on Windows Console
+ * that doesn't support UTF-8 properly.
+ */
+
+// Check if we're in a Windows environment with potential encoding issues
+const isWindowsConsole = process.platform === 'win32' && 
+    (!process.env.WT_SESSION && !process.env.TERM_PROGRAM);
+
+// Icon mappings: [unicode, ascii fallback]
+const ICONS = {
+    brain: ['🧠', '[*]'],
+    check: ['✓', '[v]'],
+    cross: ['✗', '[x]'],
+    warning: ['⚠️', '[!]'],
+    info: ['ℹ️', '[i]'],
+    arrow: ['›', '>'],
+    star: ['⭐', '*'],
+    rocket: ['🚀', '>>'],
+    fire: ['🔥', '!!'],
+    party: ['🎉', ':)'],
+    magnify: ['🔍', '?'],
+    folder: ['📁', '[]'],
+    file: ['📄', '-'],
+    gear: ['⚙️', '@'],
+    lock: ['🔒', '#'],
+    key: ['🔑', '#'],
+    bulb: ['💡', '!'],
+    clock: ['⏰', 'T'],
+    success: ['✅', '[OK]'],
+    error: ['❌', '[ERR]'],
+    sparkle: ['✨', '*'],
+    chart: ['📊', '[=]'],
+    note: ['📝', '-'],
+    book: ['📚', '[]'],
+    package: ['📦', '[]'],
+    tools: ['🛠️', '[]'],
+    shield: ['🛡️', 'S'],
+    target: ['🎯', 'o'],
+    refresh: ['🔄', '@'],
+    lightning: ['⚡', '!'],
+};
+
+/**
+ * Get appropriate icon for current environment
+ * @param {string} name - Icon name from ICONS map
+ * @returns {string} Unicode icon or ASCII fallback
+ */
+export function icon(name) {
+    const pair = ICONS[name];
+    if (!pair) return '';
+    return isWindowsConsole ? pair[1] : pair[0];
+}
+
+/**
+ * Replace all emojis in text with ASCII equivalents
+ * @param {string} text - Text containing emojis
+ * @returns {string} Text with ASCII replacements on Windows
+ */
+export function safeText(text) {
+    if (!isWindowsConsole) return text;
+    
+    let result = text;
+    for (const [name, [unicode, ascii]] of Object.entries(ICONS)) {
+        result = result.replace(new RegExp(unicode, 'g'), ascii);
+    }
+    return result;
+}
+
+/**
+ * Check if Unicode output is safe
+ * @returns {boolean}
+ */
+export function supportsUnicode() {
+    return !isWindowsConsole;
+}
+
+// Quick accessor for common icons
+export const icons = {
+    get brain() { return icon('brain'); },
+    get check() { return icon('check'); },
+    get cross() { return icon('cross'); },
+    get warning() { return icon('warning'); },
+    get arrow() { return icon('arrow'); },
+    get party() { return icon('party'); },
+    get success() { return icon('success'); },
+    get error() { return icon('error'); },
+    get sparkle() { return icon('sparkle'); },
+};
+
+export default { icon, safeText, supportsUnicode, icons, ICONS };

package/packages/cli/lib/ignore.js

@@ -0,0 +1,116 @@
+/**
+ * @fileoverview Ignore patterns parser for Agent Skill Kit
+ * Supports .agentignore file with glob patterns
+ */
+
+import fs from "fs";
+import path from "path";
+import { cwd } from "./config.js";
+
+/** Default patterns always ignored */
+const DEFAULT_IGNORES = [
+    "node_modules/**",
+    ".git/**",
+    "dist/**",
+    "build/**",
+    ".next/**",
+    "coverage/**",
+    "*.log"
+];
+
+/**
+ * Load ignore patterns from .agentignore file
+ * @param {string} basePath - Base directory
+ * @returns {string[]} Array of patterns
+ */
+export function loadIgnorePatterns(basePath = cwd) {
+    const patterns = [...DEFAULT_IGNORES];
+    const ignoreFile = path.join(basePath, ".agentignore");
+
+    try {
+        if (fs.existsSync(ignoreFile)) {
+            const content = fs.readFileSync(ignoreFile, "utf8");
+            const lines = content.split("\n");
+
+            for (const line of lines) {
+                const trimmed = line.trim();
+                // Skip empty lines and comments
+                if (trimmed && !trimmed.startsWith("#")) {
+                    patterns.push(trimmed);
+                }
+            }
+        }
+    } catch (e) {
+        // Silently use defaults
+    }
+
+    return [...new Set(patterns)]; // Remove duplicates
+}
+
+/**
+ * Check if a file path matches any ignore pattern
+ * @param {string} filePath - Relative file path
+ * @param {string[]} patterns - Ignore patterns
+ * @returns {boolean}
+ */
+export function isIgnored(filePath, patterns) {
+    const normalized = filePath.replace(/\\/g, "/");
+
+    for (const pattern of patterns) {
+        // Simple glob matching
+        const regex = patternToRegex(pattern);
+        if (regex.test(normalized)) {
+            return true;
+        }
+    }
+
+    return false;
+}
+
+/**
+ * Convert glob pattern to regex
+ * @param {string} pattern - Glob pattern
+ * @returns {RegExp}
+ */
+function patternToRegex(pattern) {
+    let regex = pattern
+        .replace(/\./g, "\\.")      // Escape dots
+        .replace(/\*\*/g, ".*")     // ** matches everything
+        .replace(/\*/g, "[^/]*")    // * matches segment
+        .replace(/\?/g, ".");       // ? matches single char
+
+    // Handle directory patterns (ending with /)
+    if (pattern.endsWith("/")) {
+        regex = regex.slice(0, -1) + "(/.*)?";
+    }
+
+    return new RegExp(`^${regex}$|/${regex}$|^${regex}/`);
+}
+
+/**
+ * Filter files array by ignore patterns
+ * @param {string[]} files - Array of file paths
+ * @param {string[]} patterns - Ignore patterns
+ * @returns {{ included: string[], ignored: number }}
+ */
+export function filterFiles(files, patterns) {
+    const included = [];
+    let ignored = 0;
+
+    for (const file of files) {
+        if (isIgnored(file, patterns)) {
+            ignored++;
+        } else {
+            included.push(file);
+        }
+    }
+
+    return { included, ignored };
+}
+
+export default {
+    loadIgnorePatterns,
+    isIgnored,
+    filterFiles,
+    DEFAULT_IGNORES
+};

package/packages/cli/lib/ignore.test.js

@@ -0,0 +1,58 @@
+/**
+ * @fileoverview Tests for ignore module
+ */
+
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import fs from "fs";
+import path from "path";
+import os from "os";
+
+// Import directly without mocking - test actual behavior
+import { loadIgnorePatterns, isIgnored } from "./ignore.js";
+
+describe("ignore", () => {
+    describe("isIgnored", () => {
+        it("returns false when no patterns", () => {
+            const result = isIgnored("some/path.js", []);
+            expect(result).toBe(false);
+        });
+
+        it("matches exact pattern", () => {
+            const result = isIgnored("node_modules/pkg/file.js", ["node_modules"]);
+            expect(result).toBe(true);
+        });
+
+        it("matches glob pattern", () => {
+            const result = isIgnored("debug.log", ["*.log"]);
+            expect(result).toBe(true);
+        });
+
+        it("returns false for non-matching path", () => {
+            const result = isIgnored("src/app.js", ["*.log", "node_modules"]);
+            expect(result).toBe(false);
+        });
+
+        it("matches directory pattern", () => {
+            const result = isIgnored("dist/bundle.js", ["dist/**"]);
+            expect(result).toBe(true);
+        });
+
+        it("matches nested paths", () => {
+            const result = isIgnored(".git/objects/pack", [".git/**"]);
+            expect(result).toBe(true);
+        });
+    });
+
+    describe("loadIgnorePatterns", () => {
+        it("returns array of patterns", () => {
+            const patterns = loadIgnorePatterns();
+            expect(Array.isArray(patterns)).toBe(true);
+        });
+
+        it("includes default patterns", () => {
+            const patterns = loadIgnorePatterns();
+            // Should have at least some patterns (defaults or from file)
+            expect(patterns.length).toBeGreaterThan(0);
+        });
+    });
+});

package/packages/cli/lib/init.js

@@ -0,0 +1,124 @@
+/**
+ * @fileoverview Project initialization wizard
+ * Sets up agent config for new projects
+ */
+
+import fs from "fs";
+import path from "path";
+import { cwd, AGENT_DIR, KNOWLEDGE_DIR, LESSONS_PATH } from "./config.js";
+
+/**
+ * Detect project type based on files
+ * @returns {string} Project type
+ */
+export function detectProjectType() {
+    const files = fs.readdirSync(cwd);
+
+    if (files.includes("package.json")) {
+        const pkg = JSON.parse(fs.readFileSync(path.join(cwd, "package.json"), "utf8"));
+
+        if (pkg.dependencies?.next || pkg.devDependencies?.next) return "nextjs";
+        if (pkg.dependencies?.react || pkg.devDependencies?.react) return "react";
+        if (pkg.dependencies?.vue || pkg.devDependencies?.vue) return "vue";
+        if (pkg.dependencies?.express || pkg.devDependencies?.express) return "node";
+        return "node";
+    }
+
+    if (files.includes("requirements.txt") || files.includes("pyproject.toml")) return "python";
+    if (files.includes("Cargo.toml")) return "rust";
+    if (files.includes("go.mod")) return "go";
+    if (files.includes("pom.xml") || files.includes("build.gradle")) return "java";
+
+    return "generic";
+}
+
+/**
+ * Get default ignore patterns for project type
+ * @param {string} projectType 
+ * @returns {string[]}
+ */
+export function getDefaultIgnorePatterns(projectType) {
+    const common = [
+        "node_modules/**",
+        ".git/**",
+        "dist/**",
+        "build/**",
+        "coverage/**",
+        "*.log",
+        "*.lock",
+        ".env*"
+    ];
+
+    const specific = {
+        node: ["package-lock.json"],
+        nextjs: [".next/**", "out/**"],
+        react: ["build/**"],
+        vue: [".nuxt/**"],
+        python: ["__pycache__/**", "*.pyc", ".venv/**", "venv/**"],
+        rust: ["target/**"],
+        go: ["vendor/**"],
+        java: ["target/**", "*.class"],
+        generic: []
+    };
+
+    return [...common, ...(specific[projectType] || [])];
+}
+
+/**
+ * Initialize agent config
+ * @param {object} options
+ * @param {boolean} options.force - Overwrite existing
+ * @returns {{ success: boolean, message: string }}
+ */
+export function initProject(options = {}) {
+    const { force = false } = options;
+
+    // Check if already initialized
+    if (fs.existsSync(KNOWLEDGE_DIR) && !force) {
+        return {
+            success: false,
+            message: "Already initialized. Use --force to reinitialize."
+        };
+    }
+
+    // Detect project type
+    const projectType = detectProjectType();
+
+    // Create directories
+    fs.mkdirSync(KNOWLEDGE_DIR, { recursive: true });
+
+    // Create lessons-learned.yaml
+    if (!fs.existsSync(LESSONS_PATH)) {
+        const initialLessons = `# Agent Skill Kit - Lessons Learned
+# Project Type: ${projectType}
+# Created: ${new Date().toISOString()}
+
+lessons: []
+`;
+        fs.writeFileSync(LESSONS_PATH, initialLessons);
+    }
+
+    // Create .agentignore if not exists
+    const agentignorePath = path.join(cwd, ".agentignore");
+    if (!fs.existsSync(agentignorePath)) {
+        const patterns = getDefaultIgnorePatterns(projectType);
+        fs.writeFileSync(agentignorePath, patterns.join("\n") + "\n");
+    }
+
+    return {
+        success: true,
+        message: `Initialized for ${projectType} project`,
+        projectType,
+        paths: {
+            knowledge: KNOWLEDGE_DIR,
+            lessons: LESSONS_PATH,
+            agentignore: agentignorePath
+        }
+    };
+}
+
+export default {
+    detectProjectType,
+    getDefaultIgnorePatterns,
+    initProject
+};