@agentskillkit/agent-skills 1.0.1

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

@@ -0,0 +1,176 @@
+#!/usr/bin/env node
+/**
+ * Git Hooks Installer
+ * 
+ * Installs pre-commit hook that runs recall on staged files.
+ * 
+ * Usage: ag-smart install-hooks
+ */
+
+import fs from "fs";
+import path from "path";
+import { VERSION } from "../config.js";
+
+// ============================================================================
+// HOOK TEMPLATES
+// ============================================================================
+
+const PRE_COMMIT_HOOK = `#!/bin/sh
+# Smart Agent Pre-Commit Hook v${VERSION}
+# Auto-generated by ag-smart install-hooks
+
+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)$")
+
+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
+`;
+
+// ============================================================================
+// INSTALLER
+// ============================================================================
+
+/**
+ * Find .git directory
+ * @returns {string|null}
+ */
+function findGitDir() {
+    let dir = process.cwd();
+
+    while (dir !== path.dirname(dir)) {
+        const gitDir = path.join(dir, ".git");
+        if (fs.existsSync(gitDir)) {
+            return gitDir;
+        }
+        dir = path.dirname(dir);
+    }
+
+    return null;
+}
+
+/**
+ * Install git hooks
+ */
+function installHooks() {
+    console.log(`\n🔧 Smart Agent Hooks Installer v${VERSION}\n`);
+
+    const gitDir = findGitDir();
+
+    if (!gitDir) {
+        console.log("❌ Not a git repository. Initialize git first:");
+        console.log("   git init\n");
+        process.exit(1);
+    }
+
+    const hooksDir = path.join(gitDir, "hooks");
+    const preCommitPath = path.join(hooksDir, "pre-commit");
+
+    // Create hooks directory if missing
+    if (!fs.existsSync(hooksDir)) {
+        fs.mkdirSync(hooksDir, { recursive: true });
+    }
+
+    // Backup existing hook
+    if (fs.existsSync(preCommitPath)) {
+        const backupPath = preCommitPath + ".backup";
+        fs.copyFileSync(preCommitPath, backupPath);
+        console.log(`📦 Backed up existing hook to: pre-commit.backup`);
+    }
+
+    // Write new hook
+    fs.writeFileSync(preCommitPath, PRE_COMMIT_HOOK, { mode: 0o755 });
+
+    console.log("✅ Installed pre-commit hook");
+    console.log(`   Location: ${preCommitPath}\n`);
+
+    console.log("📋 What it does:");
+    console.log("   • Runs on every commit");
+    console.log("   • Checks staged JS/TS files against memory");
+    console.log("   • Shows violations (does NOT block commits)\n");
+
+    console.log("💡 Tips:");
+    console.log("   • Skip hook: git commit --no-verify");
+    console.log("   • Uninstall: rm .git/hooks/pre-commit\n");
+}
+
+/**
+ * Uninstall hooks
+ */
+function uninstallHooks() {
+    const gitDir = findGitDir();
+
+    if (!gitDir) {
+        console.log("❌ Not a git repository.\n");
+        process.exit(1);
+    }
+
+    const preCommitPath = path.join(gitDir, "hooks", "pre-commit");
+
+    if (fs.existsSync(preCommitPath)) {
+        fs.unlinkSync(preCommitPath);
+        console.log("✅ Removed pre-commit hook.\n");
+
+        // Restore backup if exists
+        const backupPath = preCommitPath + ".backup";
+        if (fs.existsSync(backupPath)) {
+            fs.renameSync(backupPath, preCommitPath);
+            console.log("📦 Restored previous hook from backup.\n");
+        }
+    } else {
+        console.log("ℹ️  No hook installed.\n");
+    }
+}
+
+// ============================================================================
+// CLI
+// ============================================================================
+
+const args = process.argv.slice(2);
+
+if (args.includes("--help")) {
+    console.log(`
+🔧 Smart Agent Hooks Installer
+
+Usage:
+  ag-smart install-hooks           Install git hooks
+  ag-smart install-hooks --remove  Remove installed hooks
+
+The pre-commit hook checks staged files against learned
+patterns before each commit.
+`);
+    process.exit(0);
+}
+
+if (args.includes("--remove") || args.includes("--uninstall")) {
+    uninstallHooks();
+} else {
+    installHooks();
+}

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

@@ -0,0 +1,172 @@
+#!/usr/bin/env node
+/**
+ * Lint Learn - Auto-Learn from ESLint Output
+ * 
+ * Parses ESLint JSON output and creates lessons automatically.
+ * 
+ * Usage:
+ *   npx eslint . --format json | ag-smart lint-learn
+ *   npx eslint . --format json > output.json && ag-smart lint-learn output.json
+ */
+
+import fs from "fs";
+import { loadKnowledge, saveKnowledge } from "../recall.v2.js";
+
+// ============================================================================
+// ESLINT PARSER
+// ============================================================================
+
+/**
+ * Parse ESLint JSON output and extract rules
+ * @param {string} jsonContent 
+ * @returns {Array<{ rule: string, count: number, message: string }>}
+ */
+function parseEslintOutput(jsonContent) {
+    const ruleStats = {};
+
+    try {
+        const results = JSON.parse(jsonContent);
+
+        results.forEach(file => {
+            if (!file.messages) return;
+
+            file.messages.forEach(msg => {
+                if (!msg.ruleId) return;
+
+                if (!ruleStats[msg.ruleId]) {
+                    ruleStats[msg.ruleId] = {
+                        rule: msg.ruleId,
+                        count: 0,
+                        message: msg.message,
+                        severity: msg.severity === 2 ? "ERROR" : "WARNING"
+                    };
+                }
+                ruleStats[msg.ruleId].count++;
+            });
+        });
+    } catch (e) {
+        console.error("❌ Failed to parse ESLint JSON output:", e.message);
+        process.exit(1);
+    }
+
+    return Object.values(ruleStats).sort((a, b) => b.count - a.count);
+}
+
+/**
+ * Convert ESLint rule to regex pattern
+ * @param {string} rule - ESLint rule ID
+ * @returns {string}
+ */
+function ruleToPattern(rule) {
+    // Map common ESLint rules to regex patterns
+    const patterns = {
+        "no-console": "console\\.(log|warn|error|info|debug)",
+        "no-debugger": "\\bdebugger\\b",
+        "no-var": "\\bvar\\s+",
+        "no-unused-vars": null, // Can't easily detect with regex
+        "no-undef": null,
+        "eqeqeq": "[^!=]==[^=]",
+        "no-eval": "\\beval\\s*\\(",
+        "no-alert": "\\balert\\s*\\("
+    };
+
+    return patterns[rule] || null;
+}
+
+// ============================================================================
+// MAIN
+// ============================================================================
+
+async function main() {
+    const args = process.argv.slice(2);
+    let jsonContent = "";
+
+    if (args.includes("--help")) {
+        console.log(`
+🔧 Lint Learn - Auto-Learn from ESLint
+
+Usage:
+  npx eslint . --format json | ag-smart lint-learn
+  ag-smart lint-learn eslint-output.json
+
+Creates lessons from ESLint violations automatically.
+`);
+        process.exit(0);
+    }
+
+    // Read from file or stdin
+    if (args[0] && !args[0].startsWith("-")) {
+        if (fs.existsSync(args[0])) {
+            jsonContent = fs.readFileSync(args[0], "utf8");
+        } else {
+            console.error(`❌ File not found: ${args[0]}`);
+            process.exit(1);
+        }
+    } else {
+        // Read from stdin
+        const chunks = [];
+        process.stdin.setEncoding("utf8");
+
+        for await (const chunk of process.stdin) {
+            chunks.push(chunk);
+        }
+        jsonContent = chunks.join("");
+    }
+
+    if (!jsonContent.trim()) {
+        console.log("ℹ️  No input received. Pipe ESLint JSON output or provide a file.");
+        process.exit(0);
+    }
+
+    const rules = parseEslintOutput(jsonContent);
+
+    if (rules.length === 0) {
+        console.log("✅ No ESLint violations found.");
+        process.exit(0);
+    }
+
+    console.log(`\n🔧 Lint Learn - Found ${rules.length} unique ESLint rule(s)\n`);
+
+    const db = loadKnowledge();
+    let added = 0;
+
+    rules.forEach(r => {
+        const pattern = ruleToPattern(r.rule);
+
+        if (!pattern) {
+            console.log(`⏭️  Skipped: ${r.rule} (no regex pattern available)`);
+            return;
+        }
+
+        // Check if already exists
+        const exists = db.lessons.some(l => l.pattern === pattern);
+        if (exists) {
+            console.log(`⏭️  Skipped: ${r.rule} (already in memory)`);
+            return;
+        }
+
+        const id = `LEARN-${String(db.lessons.length + 1).padStart(3, "0")}`;
+
+        db.lessons.push({
+            id,
+            pattern,
+            message: `ESLint: ${r.rule} - ${r.message}`,
+            severity: r.severity,
+            source: "eslint",
+            hitCount: r.count,
+            addedAt: new Date().toISOString()
+        });
+
+        added++;
+        console.log(`✅ Added: [${id}] ${r.rule} (${r.count} occurrences)`);
+    });
+
+    if (added > 0) {
+        saveKnowledge(db);
+        console.log(`\n🎓 Learned ${added} new lesson(s) from ESLint output.\n`);
+    } else {
+        console.log("\nℹ️  No new lessons to add.\n");
+    }
+}
+
+main();

package/packages/cli/lib/learn.js

@@ -0,0 +1,173 @@
+#!/usr/bin/env node
+/**
+ * Smart Learning Script - ESM Version
+ * 
+ * The "Teacher" script. Adds new lessons to the system memory.
+ * Usage: node learn.js --add --pattern "regex" --message "why bad"
+ */
+
+import fs from "fs";
+import path from "path";
+import yaml from "js-yaml";
+import { KNOWLEDGE_DIR, LESSONS_PATH, DEBUG } from "./config.js";
+
+// ============================================================================
+// CORE FUNCTIONS
+// ============================================================================
+
+/**
+ * Load knowledge base from YAML file
+ * @returns {import('./types.js').KnowledgeBase}
+ */
+function loadKnowledge() {
+    try {
+        if (!fs.existsSync(LESSONS_PATH)) {
+            // Create if missing
+            const initial = { lessons: [] };
+            fs.mkdirSync(KNOWLEDGE_DIR, { recursive: true });
+            fs.writeFileSync(LESSONS_PATH, yaml.dump(initial), "utf8");
+            return initial;
+        }
+        const content = fs.readFileSync(LESSONS_PATH, "utf8");
+        return yaml.load(content) || { lessons: [] };
+    } catch (error) {
+        console.error("❌ Failed to load knowledge base:", error.message);
+        if (DEBUG) console.error(error.stack);
+        process.exit(1);
+    }
+}
+
+/**
+ * Save knowledge base to YAML file
+ * @param {import('./types.js').KnowledgeBase} data
+ */
+function saveKnowledge(data) {
+    try {
+        const str = yaml.dump(data, { lineWidth: -1 });
+        fs.writeFileSync(LESSONS_PATH, str, "utf8");
+        console.log("✅ Knowledge base updated successfully.");
+    } catch (error) {
+        console.error("❌ Failed to save knowledge base:", error.message);
+        if (DEBUG) console.error(error.stack);
+        process.exit(1);
+    }
+}
+
+/**
+ * Add a new lesson to the knowledge base
+ * @param {string} pattern - Regex pattern
+ * @param {string} message - Explanation message
+ * @param {'WARNING'|'ERROR'} severity - Severity level
+ */
+function addLesson(pattern, message, severity = "WARNING") {
+    const db = loadKnowledge();
+
+    // Validate Regex
+    try {
+        new RegExp(pattern);
+    } catch (e) {
+        console.error("❌ Invalid Regex pattern:", e.message);
+        process.exit(1);
+    }
+
+    // Validate severity
+    if (!["WARNING", "ERROR"].includes(severity)) {
+        console.error("❌ Invalid severity. Must be WARNING or ERROR");
+        process.exit(1);
+    }
+
+    const id = `LEARN-${String(db.lessons.length + 1).padStart(3, "0")}`;
+
+    /** @type {import('./types.js').Lesson} */
+    const lesson = {
+        id,
+        pattern,
+        message,
+        severity,
+        addedAt: new Date().toISOString()
+    };
+
+    db.lessons.push(lesson);
+    saveKnowledge(db);
+
+    console.log(`\n🎓 Lesson Learned: [${id}]`);
+    console.log(`   Pattern: /${pattern}/`);
+    console.log(`   Message: ${message}`);
+}
+
+/**
+ * List all learned lessons
+ */
+function listLessons() {
+    const db = loadKnowledge();
+    if (!db.lessons || db.lessons.length === 0) {
+        console.log("ℹ️  No lessons learned yet.");
+        return;
+    }
+
+    console.log("\n🧠 Smart Agent Knowledge Base:\n");
+    db.lessons.forEach(l => {
+        console.log(`  [${l.id}] /${l.pattern}/ -> ${l.message} (${l.severity})`);
+    });
+    console.log("");
+}
+
+// ============================================================================
+// CLI
+// ============================================================================
+
+function printHelp() {
+    console.log(`
+Smart Learning Tool
+
+USAGE:
+  node learn.js --add --pattern "..." --message "..."
+  node learn.js --list
+  node learn.js --help
+
+OPTIONS:
+  --add       Add a new lesson
+  --pattern   Regex pattern to flag
+  --message   Explanation message
+  --severity  Severity (WARNING/ERROR), default WARNING
+  --list      List all learned lessons
+`);
+}
+
+function main() {
+    const args = process.argv.slice(2);
+
+    if (args.includes("--help") || args.length === 0) {
+        printHelp();
+        process.exit(0);
+    }
+
+    if (args.includes("--list")) {
+        listLessons();
+        process.exit(0);
+    }
+
+    if (args.includes("--add")) {
+        const pIdx = args.indexOf("--pattern");
+        const mIdx = args.indexOf("--message");
+        const sIdx = args.indexOf("--severity");
+
+        if (pIdx === -1 || mIdx === -1) {
+            console.error("❌ --pattern and --message are required for --add");
+            process.exit(1);
+        }
+
+        const pattern = args[pIdx + 1];
+        const message = args[mIdx + 1];
+        const severity = sIdx !== -1 ? args[sIdx + 1] : "WARNING";
+
+        if (!pattern || !message) {
+            console.error("❌ Missing values for pattern/message");
+            process.exit(1);
+        }
+
+        addLesson(pattern, message, severity);
+    }
+}
+
+main();

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

@@ -0,0 +1,70 @@
+/**
+ * @fileoverview Tests for learn.js functionality
+ */
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import fs from "fs";
+import path from "path";
+import os from "os";
+
+// Mock config để test
+const TEST_DIR = path.join(os.tmpdir(), "agent-skill-kit-test");
+const KNOWLEDGE_DIR = path.join(TEST_DIR, ".agent", "knowledge");
+const LESSONS_PATH = path.join(KNOWLEDGE_DIR, "lessons-learned.yaml");
+
+describe("Knowledge Base Operations", () => {
+    beforeEach(() => {
+        fs.mkdirSync(KNOWLEDGE_DIR, { recursive: true });
+    });
+
+    afterEach(() => {
+        fs.rmSync(TEST_DIR, { recursive: true, force: true });
+    });
+
+    it("creates initial knowledge file if missing", () => {
+        // Simulate missing file scenario
+        expect(fs.existsSync(LESSONS_PATH)).toBe(false);
+
+        // Create initial file
+        const initial = { lessons: [] };
+        fs.writeFileSync(LESSONS_PATH, JSON.stringify(initial), "utf8");
+
+        expect(fs.existsSync(LESSONS_PATH)).toBe(true);
+    });
+
+    it("can write and read lessons", () => {
+        const lesson = {
+            id: "LEARN-001",
+            pattern: "console\\.log",
+            message: "No console.log in production",
+            severity: "WARNING",
+            addedAt: new Date().toISOString()
+        };
+
+        const data = { lessons: [lesson] };
+        fs.writeFileSync(LESSONS_PATH, JSON.stringify(data), "utf8");
+
+        const content = fs.readFileSync(LESSONS_PATH, "utf8");
+        const parsed = JSON.parse(content);
+
+        expect(parsed.lessons).toHaveLength(1);
+        expect(parsed.lessons[0].id).toBe("LEARN-001");
+        expect(parsed.lessons[0].pattern).toBe("console\\.log");
+    });
+
+    it("validates regex patterns", () => {
+        const validPattern = "console\\.log";
+        const invalidPattern = "[invalid(";
+
+        expect(() => new RegExp(validPattern)).not.toThrow();
+        expect(() => new RegExp(invalidPattern)).toThrow();
+    });
+
+    it("validates severity values", () => {
+        const validSeverities = ["WARNING", "ERROR"];
+        const invalidSeverity = "INFO";
+
+        expect(validSeverities.includes("WARNING")).toBe(true);
+        expect(validSeverities.includes("ERROR")).toBe(true);
+        expect(validSeverities.includes(invalidSeverity)).toBe(false);
+    });
+});