add-skill-kit 3.2.3 → 3.2.5

package/lib/agent-cli/__tests__/skill_injector.test.js

@@ -0,0 +1,191 @@
+/**
+ * Unit Tests for skill_injector.js
+ * 
+ * Tests:
+ * - analyzeLessons
+ * - getSkillCandidates
+ * - generateSkillContent
+ * - generateSkills
+ */
+
+import { describe, it, expect, vi } from 'vitest';
+
+// Mock fs module
+vi.mock('fs', async () => {
+    const actual = await vi.importActual('fs');
+    return {
+        ...actual,
+        default: {
+            ...actual,
+            existsSync: vi.fn(() => false),
+            readFileSync: vi.fn(() => '{}'),
+            writeFileSync: vi.fn(),
+            mkdirSync: vi.fn(),
+            readdirSync: vi.fn(() => [])
+        },
+        existsSync: vi.fn(() => false),
+        readFileSync: vi.fn(() => '{}'),
+        writeFileSync: vi.fn(),
+        mkdirSync: vi.fn(),
+        readdirSync: vi.fn(() => [])
+    };
+});
+
+// Category config constant
+const CATEGORY_CONFIG = {
+    'SAFE': { name: 'safety-rules', description: 'Auto-learned safety rules' },
+    'CODE': { name: 'code-patterns', description: 'Auto-learned code patterns' },
+    'FLOW': { name: 'workflow-rules', description: 'Auto-learned workflow rules' },
+    'INT': { name: 'integration-rules', description: 'Auto-learned integration rules' },
+    'PERF': { name: 'performance-rules', description: 'Auto-learned performance rules' },
+    'TEST': { name: 'testing-rules', description: 'Auto-learned testing rules' },
+    'LEARN': { name: 'general-lessons', description: 'General lessons learned' }
+};
+
+describe('Skill Injector', () => {
+    describe('analyzeLessons', () => {
+        it('should group lessons by category', () => {
+            const lessons = [
+                { id: 'SAFE-001', pattern: 'delete check' },
+                { id: 'SAFE-002', pattern: 'file backup' },
+                { id: 'CODE-001', pattern: 'typescript' },
+                { id: 'TEST-001', pattern: 'unit test' }
+            ];
+
+            const categories = {};
+
+            for (const lesson of lessons) {
+                const category = lesson.id?.split('-')[0] || 'LEARN';
+                if (!categories[category]) {
+                    categories[category] = [];
+                }
+                categories[category].push(lesson);
+            }
+
+            expect(categories['SAFE'].length).toBe(2);
+            expect(categories['CODE'].length).toBe(1);
+            expect(categories['TEST'].length).toBe(1);
+        });
+
+        it('should default to LEARN for unknown categories', () => {
+            const lessons = [{ id: 'UNKNOWN-001', pattern: 'something' }];
+
+            const categories = {};
+            for (const lesson of lessons) {
+                const category = lesson.id?.split('-')[0] || 'LEARN';
+                if (!categories[category]) {
+                    categories[category] = [];
+                }
+                categories[category].push(lesson);
+            }
+
+            expect(categories['UNKNOWN'].length).toBe(1);
+        });
+    });
+
+    describe('getSkillCandidates', () => {
+        it('should require minimum 3 lessons for skill generation', () => {
+            const categories = {
+                'SAFE': [{ id: 1 }, { id: 2 }, { id: 3 }], // 3 - eligible
+                'CODE': [{ id: 1 }, { id: 2 }], // 2 - not eligible
+                'TEST': [{ id: 1 }, { id: 2 }, { id: 3 }, { id: 4 }] // 4 - eligible
+            };
+
+            const candidates = [];
+            for (const [category, lessons] of Object.entries(categories)) {
+                if (lessons.length >= 3) {
+                    candidates.push({
+                        category,
+                        count: lessons.length,
+                        config: CATEGORY_CONFIG[category] || {
+                            name: `${category.toLowerCase()}-rules`,
+                            description: `Auto-learned ${category.toLowerCase()} rules`
+                        }
+                    });
+                }
+            }
+
+            expect(candidates.length).toBe(2);
+            expect(candidates.find(c => c.category === 'SAFE')).toBeDefined();
+            expect(candidates.find(c => c.category === 'TEST')).toBeDefined();
+            expect(candidates.find(c => c.category === 'CODE')).toBeUndefined();
+        });
+    });
+
+    describe('generateSkillContent', () => {
+        it('should generate valid SKILL.md content', () => {
+            const category = 'SAFE';
+            const lessons = [
+                { id: 'SAFE-001', pattern: 'delete check', severity: 'CRITICAL', message: 'Check before delete' },
+                { id: 'SAFE-002', pattern: 'backup first', severity: 'HIGH', message: 'Backup before modify' }
+            ];
+            const config = CATEGORY_CONFIG['SAFE'];
+
+            // Generate YAML frontmatter
+            const content = `---
+name: ${config.name}
+description: ${config.description}
+---
+
+# ${config.name}
+`;
+
+            expect(content).toContain('name: safety-rules');
+            expect(content).toContain('Auto-learned safety rules');
+        });
+
+        it('should include severity icons', () => {
+            const severityIcons = {
+                'CRITICAL': '🔴',
+                'HIGH': '🟠',
+                'MEDIUM': '🟡',
+                'LOW': '🟢'
+            };
+
+            expect(severityIcons['CRITICAL']).toBe('🔴');
+            expect(severityIcons['HIGH']).toBe('🟠');
+        });
+    });
+
+    describe('generateSkills', () => {
+        it('should generate skills for eligible categories', () => {
+            const candidates = [
+                { category: 'SAFE', count: 5, config: CATEGORY_CONFIG['SAFE'] },
+                { category: 'CODE', count: 3, config: CATEGORY_CONFIG['CODE'] }
+            ];
+
+            const generated = candidates.map(c => ({
+                category: c.category,
+                name: c.config.name,
+                lessonCount: c.count
+            }));
+
+            expect(generated.length).toBe(2);
+            expect(generated[0].name).toBe('safety-rules');
+            expect(generated[1].name).toBe('code-patterns');
+        });
+
+        it('should support dry-run mode', () => {
+            const dryRun = true;
+            const writeOperations = [];
+
+            if (!dryRun) {
+                writeOperations.push('write');
+            }
+
+            expect(writeOperations.length).toBe(0);
+        });
+    });
+
+    describe('Category Mapping', () => {
+        it('should have all standard categories', () => {
+            expect(CATEGORY_CONFIG['SAFE']).toBeDefined();
+            expect(CATEGORY_CONFIG['CODE']).toBeDefined();
+            expect(CATEGORY_CONFIG['FLOW']).toBeDefined();
+            expect(CATEGORY_CONFIG['INT']).toBeDefined();
+            expect(CATEGORY_CONFIG['PERF']).toBeDefined();
+            expect(CATEGORY_CONFIG['TEST']).toBeDefined();
+            expect(CATEGORY_CONFIG['LEARN']).toBeDefined();
+        });
+    });
+});

package/lib/agent-cli/bin/agent.js

@@ -0,0 +1,191 @@
+#!/usr/bin/env node
+/**
+ * Smart Agent CLI - ESM Version (Production-Ready)
+ * 
+ * The main interface for humans to interact with the Smart Agent Skills system.
+ * 
+ * Commands:
+ *   learn          Add new lessons to memory
+ *   recall         Check files against memory
+ *   audit          Full compliance audit
+ *   watch          Real-time file monitoring
+ *   stats          Knowledge base statistics
+ *   install-hooks  Install git pre-commit hook
+ *   lint-learn     Auto-learn from ESLint output
+ */
+
+import { spawn } from "child_process";
+import path from "path";
+import { fileURLToPath } from "url";
+import { VERSION } from "../lib/config.js";
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const ARGS = process.argv.slice(2);
+const COMMAND = ARGS[0];
+const SCRIPTS_DIR = path.join(__dirname, "..", "lib");
+const HOOKS_DIR = path.join(SCRIPTS_DIR, "hooks");
+const DASHBOARD_DIR = path.join(__dirname, "..", "dashboard");
+const AUTO_LEARN_DIR = path.join(__dirname, "..", "scripts");
+
+/**
+ * Run a script with given arguments
+ * @param {string} script - Script filename (relative to lib/)
+ * @param {string[]} args - Arguments to pass
+ * @param {string} baseDir - Base directory for script
+ */
+function run(script, args = [], baseDir = SCRIPTS_DIR) {
+    const scriptPath = path.join(baseDir, script);
+    const child = spawn("node", [scriptPath, ...args], {
+        stdio: "inherit",
+        shell: true
+    });
+
+    child.on("close", (code) => {
+        process.exit(code || 0);
+    });
+
+    child.on("error", (err) => {
+        console.error(`❌ Failed to run ${script}:`, err.message);
+        process.exit(1);
+    });
+}
+
+function printHelp() {
+    console.log(`
+🤖 Agent Skill Kit CLI v${VERSION}
+
+Usage: agent <command> [options]
+
+${"─".repeat(50)}
+
+📚 CORE COMMANDS:
+
+  learn          Teach a new lesson to the memory
+                 agent learn --add --pattern "var " --message "Use let/const"
+                 agent learn --list
+                 agent learn --remove LEARN-001
+
+  recall         Check file(s) against learned patterns
+                 agent recall src/app.js
+                 agent recall ./src
+
+  audit          Run full compliance audit
+                 agent audit [directory]
+
+${"─".repeat(50)}
+
+🚀 PRODUCTION FEATURES:
+
+  watch          Real-time file monitoring
+                 agent watch [directory]
+
+  stats          Knowledge base statistics
+                 agent stats
+
+  install-hooks  Install git pre-commit hook
+                 agent install-hooks
+                 agent install-hooks --remove
+
+  lint-learn     Auto-learn from ESLint JSON output
+                 npx eslint . --format json | agent lint-learn
+
+  fix            🆕 Auto-fix violations
+                 agent fix <file|dir> [--mode safe|aggressive]
+
+  sync-skills    🆕 Sync hot patterns to SKILL.md
+                 agent sync-skills
+
+  dashboard      🆕 Start Auto-Learn dashboard
+                 agent dashboard
+
+  auto-learn     🆕 Run auto-learn scripts
+                 agent auto-learn --scan
+                 agent auto-learn --analyze
+                 agent auto-learn --check "intent"
+
+${"─".repeat(50)}
+
+📖 HELP:
+
+  help, --help   Show this help message
+  --version      Show version number
+
+💡 Docs: https://github.com/agentskillkit/agent-skills
+`);
+}
+
+// Command routing
+switch (COMMAND) {
+    // Core commands (v2 versions)
+    case "learn":
+        run("learn.js", ARGS.slice(1));
+        break;
+    case "recall":
+        run("recall.js", ARGS.slice(1));
+        break;
+    case "audit":
+        run("audit.js", ARGS.slice(1));
+        break;
+
+    // Production features
+    case "watch":
+        run("watcher.js", ARGS.slice(1));
+        break;
+    case "stats":
+        run("stats.js", ARGS.slice(1));
+        break;
+    case "install-hooks":
+        run("install-hooks.js", ARGS.slice(1), HOOKS_DIR);
+        break;
+    case "lint-learn":
+        run("lint-learn.js", ARGS.slice(1), HOOKS_DIR);
+        break;
+    case "fix":
+        run("fix.js", ARGS.slice(1));
+        break;
+    case "sync-skills":
+        run("skill-learn.js", ARGS.slice(1));
+        break;
+
+    // Auto-Learn features
+    case "dashboard":
+        run("dashboard_server.js", ARGS.slice(1), DASHBOARD_DIR);
+        break;
+    case "auto-learn":
+        const subCmd = ARGS[1];
+        if (subCmd === "--scan" || subCmd === "-s") {
+            run("error_sensor.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else if (subCmd === "--success") {
+            run("success_sensor.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else if (subCmd === "--analyze" || subCmd === "-a") {
+            run("pattern_analyzer.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else if (subCmd === "--check" || subCmd === "-c") {
+            run("pre_execution_check.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else if (subCmd === "--adapt") {
+            run("adaptive_engine.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else if (subCmd === "--inject") {
+            run("skill_injector.js", ARGS.slice(2), AUTO_LEARN_DIR);
+        } else {
+            console.log("Usage: agent auto-learn [--scan|--success|--analyze|--check|--adapt|--inject]");
+        }
+        break;
+
+    // Meta
+    case "--version":
+    case "-v":
+        console.log(VERSION);
+        break;
+    case "help":
+    case "--help":
+    case "-h":
+        printHelp();
+        break;
+    case undefined:
+        // No command = show interactive Clack menu
+        import("../lib/ui/index.js").then(m => m.showMainMenu()).catch(console.error);
+        break;
+    default:
+        console.log(`❌ Unknown command: ${COMMAND}`);
+        console.log("   Run 'agent help' for available commands.\n");
+        process.exit(1);
+}