add-skill-kit 3.2.2 → 3.2.4

package/lib/agent-cli/lib/auto-learn.js

@@ -0,0 +1,319 @@
+#!/usr/bin/env node
+/**
+ * Smart Auto-Learn - True Self-Learning Engine
+ * 
+ * Automatically learns from:
+ * 1. ESLint output (runs ESLint and parses)
+ * 2. Test failures (runs tests and parses)
+ * 3. TypeScript errors (runs tsc and parses)
+ * 
+ * Usage:
+ *   ag-smart auto-learn              # Run all
+ *   ag-smart auto-learn --eslint     # ESLint only
+ *   ag-smart auto-learn --test       # Test failures only
+ *   ag-smart auto-learn --typescript # TypeScript only
+ */
+
+import fs from "fs";
+import path from "path";
+import { execSync, spawnSync } from "child_process";
+import { loadKnowledge, saveKnowledge } from "./recall.js";
+import { VERSION } from "./config.js";
+
+// ============================================================================
+// ESLINT AUTO-LEARN
+// ============================================================================
+
+const ESLINT_PATTERN_MAP = {
+    "no-console": { pattern: "console\\.(log|warn|error|info|debug)", message: "Avoid console statements in production" },
+    "no-debugger": { pattern: "\\bdebugger\\b", message: "Remove debugger statements" },
+    "no-var": { pattern: "\\bvar\\s+\\w", message: "Use const/let instead of var" },
+    "eqeqeq": { pattern: "[^!=]==[^=]", message: "Use === instead of ==" },
+    "no-eval": { pattern: "\\beval\\s*\\(", message: "Avoid eval() for security" },
+    "no-alert": { pattern: "\\balert\\s*\\(", message: "Avoid alert() in production" },
+    "no-implicit-globals": { pattern: "^(?!const|let|var|function|class|import|export)", message: "Avoid implicit globals" },
+    "prefer-const": { pattern: "\\blet\\s+\\w+\\s*=\\s*[^;]+;(?![\\s\\S]*\\1\\s*=)", message: "Use const for variables that are never reassigned" },
+    "no-unused-expressions": { pattern: "^\\s*['\"`]use strict['\"`];?\\s*$", message: "Remove unused expressions" },
+    "semi": { pattern: "[^;{}]\\s*$", message: "Missing semicolon" },
+    "quotes": { pattern: '(?<![\\w])"(?![^"]*"[,\\]\\}\\)])', message: "Prefer single quotes" },
+    "no-empty": { pattern: "\\{\\s*\\}", message: "Empty block statement" },
+    "no-extra-semi": { pattern: ";;", message: "Unnecessary semicolon" },
+    "no-unreachable": { pattern: "(?:return|throw|break|continue)[^}]*[\\n\\r]+[^}]+", message: "Unreachable code" }
+};
+
+function runEslintAutoLearn(projectRoot) {
+    console.log("\n🔍 Running ESLint analysis...\n");
+
+    try {
+        // Try to run ESLint with JSON output
+        const result = spawnSync("npx", ["eslint", ".", "--format", "json", "--no-error-on-unmatched-pattern"], {
+            cwd: projectRoot,
+            encoding: "utf8",
+            shell: true,
+            timeout: 60000
+        });
+
+        if (!result.stdout) {
+            console.log("   ℹ️  ESLint not configured or no files to lint.");
+            return [];
+        }
+
+        const eslintResults = JSON.parse(result.stdout);
+        const ruleViolations = {};
+
+        eslintResults.forEach(file => {
+            if (!file.messages) return;
+            file.messages.forEach(msg => {
+                if (!msg.ruleId) return;
+                if (!ruleViolations[msg.ruleId]) {
+                    ruleViolations[msg.ruleId] = {
+                        rule: msg.ruleId,
+                        count: 0,
+                        severity: msg.severity === 2 ? "ERROR" : "WARNING",
+                        sample: msg.message
+                    };
+                }
+                ruleViolations[msg.ruleId].count++;
+            });
+        });
+
+        return Object.values(ruleViolations).sort((a, b) => b.count - a.count);
+    } catch (e) {
+        console.log("   ⚠️  ESLint analysis skipped:", e.message);
+        return [];
+    }
+}
+
+// ============================================================================
+// TEST FAILURE AUTO-LEARN
+// ============================================================================
+
+const TEST_PATTERN_MAP = {
+    "undefined is not a function": { pattern: "\\w+\\s*\\(", message: "Function may be undefined before call" },
+    "cannot read property": { pattern: "\\w+\\.\\w+", message: "Object property access may fail if undefined" },
+    "null is not an object": { pattern: "\\.\\w+", message: "Null check needed before property access" },
+    "expected.*to equal": { pattern: "expect\\s*\\(", message: "Assertion expectation mismatch" },
+    "timeout": { pattern: "async|await|Promise", message: "Async operation may need longer timeout" },
+    "not defined": { pattern: "\\b\\w+\\b", message: "Variable may not be defined in scope" }
+};
+
+function runTestAutoLearn(projectRoot) {
+    console.log("\n🧪 Running test analysis...\n");
+
+    try {
+        // Try common test runners
+        const testCommands = [
+            { cmd: "npm", args: ["test", "--", "--json", "--passWithNoTests"], name: "npm test" },
+            { cmd: "npx", args: ["vitest", "run", "--reporter=json"], name: "vitest" },
+            { cmd: "npx", args: ["jest", "--json", "--passWithNoTests"], name: "jest" }
+        ];
+
+        for (const tc of testCommands) {
+            const result = spawnSync(tc.cmd, tc.args, {
+                cwd: projectRoot,
+                encoding: "utf8",
+                shell: true,
+                timeout: 120000
+            });
+
+            // Look for test failures in output
+            const output = result.stdout + result.stderr;
+            const failures = [];
+
+            // Parse failure messages
+            const failurePatterns = [
+                /FAIL\s+(.+)/g,
+                /✕\s+(.+)/g,
+                /Error:\s+(.+)/g,
+                /AssertionError:\s+(.+)/g
+            ];
+
+            failurePatterns.forEach(regex => {
+                let match;
+                while ((match = regex.exec(output)) !== null) {
+                    failures.push(match[1].trim());
+                }
+            });
+
+            if (failures.length > 0) {
+                console.log(`   Found ${failures.length} test failure(s) from ${tc.name}`);
+                return failures.map(f => ({ message: f, source: tc.name }));
+            }
+        }
+
+        console.log("   ✅ All tests passing or no tests found.");
+        return [];
+    } catch (e) {
+        console.log("   ⚠️  Test analysis skipped:", e.message);
+        return [];
+    }
+}
+
+// ============================================================================
+// TYPESCRIPT ERROR AUTO-LEARN
+// ============================================================================
+
+const TS_PATTERN_MAP = {
+    "TS2304": { pattern: "\\b\\w+\\b", message: "TypeScript: Cannot find name" },
+    "TS2322": { pattern: ":\\s*\\w+", message: "TypeScript: Type mismatch" },
+    "TS2345": { pattern: "\\(.*\\)", message: "TypeScript: Argument type mismatch" },
+    "TS2339": { pattern: "\\.\\w+", message: "TypeScript: Property does not exist" },
+    "TS7006": { pattern: "\\(\\w+\\)", message: "TypeScript: Parameter implicitly has 'any' type" },
+    "TS2531": { pattern: "\\w+\\.", message: "TypeScript: Object is possibly 'null'" },
+    "TS2532": { pattern: "\\w+\\.", message: "TypeScript: Object is possibly 'undefined'" }
+};
+
+function runTypescriptAutoLearn(projectRoot) {
+    console.log("\n📘 Running TypeScript analysis...\n");
+
+    try {
+        const result = spawnSync("npx", ["tsc", "--noEmit", "--pretty", "false"], {
+            cwd: projectRoot,
+            encoding: "utf8",
+            shell: true,
+            timeout: 60000
+        });
+
+        const output = result.stdout + result.stderr;
+        const errors = {};
+
+        // Parse TS errors: file(line,col): error TS1234: message
+        const tsErrorRegex = /error (TS\d+):\s*(.+)/g;
+        let match;
+
+        while ((match = tsErrorRegex.exec(output)) !== null) {
+            const code = match[1];
+            const message = match[2];
+
+            if (!errors[code]) {
+                errors[code] = { code, message, count: 0 };
+            }
+            errors[code].count++;
+        }
+
+        const errorList = Object.values(errors).sort((a, b) => b.count - a.count);
+
+        if (errorList.length > 0) {
+            console.log(`   Found ${errorList.length} unique TypeScript error type(s)`);
+        } else {
+            console.log("   ✅ No TypeScript errors found.");
+        }
+
+        return errorList;
+    } catch (e) {
+        console.log("   ⚠️  TypeScript analysis skipped:", e.message);
+        return [];
+    }
+}
+
+// ============================================================================
+// MAIN AUTO-LEARN ENGINE
+// ============================================================================
+
+function autoLearn(projectRoot, options = {}) {
+    console.log(`\n🧠 Smart Auto-Learn Engine v${VERSION}`);
+    console.log(`📂 Project: ${projectRoot}\n`);
+    console.log("─".repeat(50));
+
+    const db = loadKnowledge();
+    let totalAdded = 0;
+
+    // ESLint
+    if (!options.onlyTest && !options.onlyTypescript) {
+        const eslintViolations = runEslintAutoLearn(projectRoot);
+
+        eslintViolations.forEach(v => {
+            const mapping = ESLINT_PATTERN_MAP[v.rule];
+            if (!mapping) return;
+
+            // Check if already exists
+            if (db.lessons.some(l => l.pattern === mapping.pattern)) return;
+
+            const id = `AUTO-${String(db.lessons.length + 1).padStart(3, "0")}`;
+            db.lessons.push({
+                id,
+                pattern: mapping.pattern,
+                message: `${mapping.message} (ESLint: ${v.rule})`,
+                severity: v.severity,
+                source: "auto-eslint",
+                hitCount: v.count,
+                autoEscalated: false,
+                addedAt: new Date().toISOString()
+            });
+            totalAdded++;
+            console.log(`   ✅ Auto-learned: [${id}] ${v.rule} (${v.count} occurrences)`);
+        });
+    }
+
+    // TypeScript
+    if (!options.onlyEslint && !options.onlyTest) {
+        const tsErrors = runTypescriptAutoLearn(projectRoot);
+
+        tsErrors.slice(0, 5).forEach(e => { // Top 5 only
+            const mapping = TS_PATTERN_MAP[e.code];
+            if (!mapping) return;
+
+            if (db.lessons.some(l => l.message.includes(e.code))) return;
+
+            const id = `AUTO-${String(db.lessons.length + 1).padStart(3, "0")}`;
+            db.lessons.push({
+                id,
+                pattern: mapping.pattern,
+                message: `${mapping.message} (${e.code})`,
+                severity: "WARNING",
+                source: "auto-typescript",
+                hitCount: e.count,
+                autoEscalated: false,
+                addedAt: new Date().toISOString()
+            });
+            totalAdded++;
+            console.log(`   ✅ Auto-learned: [${id}] ${e.code} (${e.count} occurrences)`);
+        });
+    }
+
+    // Summary
+    console.log("\n" + "─".repeat(50));
+
+    if (totalAdded > 0) {
+        saveKnowledge(db);
+        console.log(`\n🎓 Auto-learned ${totalAdded} new pattern(s)!`);
+        console.log(`📊 Total lessons in memory: ${db.lessons.length}\n`);
+    } else {
+        console.log("\n✅ No new patterns discovered. Code looks clean!\n");
+    }
+
+    return totalAdded;
+}
+
+// ============================================================================
+// CLI
+// ============================================================================
+
+const args = process.argv.slice(2);
+const projectRoot = process.cwd();
+
+if (args.includes("--help")) {
+    console.log(`
+🧠 Smart Auto-Learn - True Self-Learning Engine
+
+Usage:
+  ag-smart auto-learn              Run all analyzers
+  ag-smart auto-learn --eslint     ESLint only
+  ag-smart auto-learn --typescript TypeScript only
+  ag-smart auto-learn --test       Test failures only
+
+The engine automatically:
+  1. Runs ESLint and learns from violations
+  2. Runs TypeScript and learns from type errors
+  3. Runs tests and learns from failures
+
+Patterns are automatically added to knowledge base!
+`);
+    process.exit(0);
+}
+
+autoLearn(projectRoot, {
+    onlyEslint: args.includes("--eslint"),
+    onlyTypescript: args.includes("--typescript"),
+    onlyTest: args.includes("--test")
+});

package/lib/agent-cli/lib/auto_preview.py

@@ -0,0 +1,148 @@
+#!/usr/bin/env python3
+"""
+Auto Preview - Agent Skill Kit
+==============================
+Manages (start/stop/status) the local development server for previewing the application.
+
+Usage:
+    python .agent/scripts/auto_preview.py start [port]
+    python .agent/scripts/auto_preview.py stop
+    python .agent/scripts/auto_preview.py status
+"""
+
+import os
+import sys
+import time
+import json
+import signal
+import argparse
+import subprocess
+from pathlib import Path
+
+AGENT_DIR = Path(".agent")
+PID_FILE = AGENT_DIR / "preview.pid"
+LOG_FILE = AGENT_DIR / "preview.log"
+
+def get_project_root():
+    return Path(".").resolve()
+
+def is_running(pid):
+    try:
+        os.kill(pid, 0)
+        return True
+    except OSError:
+        return False
+
+def get_start_command(root):
+    pkg_file = root / "package.json"
+    if not pkg_file.exists():
+        return None
+    
+    with open(pkg_file, 'r') as f:
+        data = json.load(f)
+    
+    scripts = data.get("scripts", {})
+    if "dev" in scripts:
+        return ["npm", "run", "dev"]
+    elif "start" in scripts:
+        return ["npm", "start"]
+    return None
+
+def start_server(port=3000):
+    if PID_FILE.exists():
+        try:
+            pid = int(PID_FILE.read_text().strip())
+            if is_running(pid):
+                print(f"⚠️  Preview already running (PID: {pid})")
+                return
+        except:
+            pass # Invalid PID file
+
+    root = get_project_root()
+    cmd = get_start_command(root)
+    
+    if not cmd:
+        print("❌ No 'dev' or 'start' script found in package.json")
+        sys.exit(1)
+    
+    # Add port env var if needed (simple heuristic)
+    env = os.environ.copy()
+    env["PORT"] = str(port)
+    
+    print(f"🚀 Starting preview on port {port}...")
+    
+    with open(LOG_FILE, "w") as log:
+        process = subprocess.Popen(
+            cmd,
+            cwd=str(root),
+            stdout=log,
+            stderr=log,
+            env=env,
+            shell=True # Required for npm on windows often, or consistent path handling
+        )
+    
+    PID_FILE.write_text(str(process.pid))
+    print(f"✅ Preview started! (PID: {process.pid})")
+    print(f"   Logs: {LOG_FILE}")
+    print(f"   URL: http://localhost:{port}")
+
+def stop_server():
+    if not PID_FILE.exists():
+        print("ℹ️  No preview server found.")
+        return
+
+    try:
+        pid = int(PID_FILE.read_text().strip())
+        if is_running(pid):
+            # Try gentle kill first
+            os.kill(pid, signal.SIGTERM) if sys.platform != 'win32' else subprocess.call(['taskkill', '/F', '/T', '/PID', str(pid)])
+            print(f"🛑 Preview stopped (PID: {pid})")
+        else:
+            print("ℹ️  Process was not running.")
+    except Exception as e:
+        print(f"❌ Error stopping server: {e}")
+    finally:
+        if PID_FILE.exists():
+            PID_FILE.unlink()
+
+def status_server():
+    running = False
+    pid = None
+    url = "Unknown"
+    
+    if PID_FILE.exists():
+        try:
+            pid = int(PID_FILE.read_text().strip())
+            if is_running(pid):
+                running = True
+                # Heuristic for URL, strictly we should save it
+                url = "http://localhost:3000" 
+        except:
+            pass
+            
+    print("\n=== Preview Status ===")
+    if running:
+        print(f"✅ Status: Running")
+        print(f"🔢 PID: {pid}")
+        print(f"🌐 URL: {url} (Likely)")
+        print(f"📝 Logs: {LOG_FILE}")
+    else:
+        print("⚪ Status: Stopped")
+    print("===================\n")
+
+def main():
+    parser = argparse.ArgumentParser()
+    parser.add_argument("action", choices=["start", "stop", "status"])
+    parser.add_argument("port", nargs="?", default="3000")
+    
+    args = parser.parse_args()
+    
+    if args.action == "start":
+        start_server(int(args.port))
+    elif args.action == "stop":
+        stop_server()
+    elif args.action == "status":
+        status_server()
+
+if __name__ == "__main__":
+    main()

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

@@ -0,0 +1,138 @@
+/**
+ * @fileoverview Backup and Restore for Agent Skill Kit
+ * Protects lessons before changes
+ */
+
+import fs from "fs";
+import path from "path";
+import { KNOWLEDGE_DIR, LESSONS_PATH } from "./config.js";
+import { SETTINGS_PATH } from "./settings.js";
+
+/** Backup directory */
+const BACKUPS_DIR = path.join(KNOWLEDGE_DIR, "backups");
+
+/**
+ * Create a timestamped backup of lessons and settings
+ * @returns {{ path: string, timestamp: string } | null}
+ */
+export function createBackup() {
+    try {
+        fs.mkdirSync(BACKUPS_DIR, { recursive: true });
+
+        const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
+        const backupDir = path.join(BACKUPS_DIR, timestamp);
+        fs.mkdirSync(backupDir, { recursive: true });
+
+        // Backup lessons
+        if (fs.existsSync(LESSONS_PATH)) {
+            fs.copyFileSync(
+                LESSONS_PATH,
+                path.join(backupDir, "lessons-learned.yaml")
+            );
+        }
+
+        // Backup settings
+        if (fs.existsSync(SETTINGS_PATH)) {
+            fs.copyFileSync(
+                SETTINGS_PATH,
+                path.join(backupDir, "settings.yaml")
+            );
+        }
+
+        return { path: backupDir, timestamp };
+    } catch (e) {
+        console.error("Failed to create backup:", e.message);
+        return null;
+    }
+}
+
+/**
+ * List available backups
+ * @returns {Array<{ name: string, date: Date, path: string }>}
+ */
+export function listBackups() {
+    try {
+        if (!fs.existsSync(BACKUPS_DIR)) {
+            return [];
+        }
+
+        const entries = fs.readdirSync(BACKUPS_DIR, { withFileTypes: true });
+        return entries
+            .filter(e => e.isDirectory())
+            .map(e => ({
+                name: e.name,
+                date: parseBackupDate(e.name),
+                path: path.join(BACKUPS_DIR, e.name)
+            }))
+            .sort((a, b) => b.date - a.date); // Newest first
+    } catch (e) {
+        return [];
+    }
+}
+
+/**
+ * Parse backup folder name to date
+ * @param {string} name - Folder name (ISO timestamp with - instead of :)
+ * @returns {Date}
+ */
+function parseBackupDate(name) {
+    try {
+        // Convert 2026-01-25T17-30-00-000Z back to valid ISO
+        const iso = name.replace(/-(\d{2})-(\d{2})-(\d{3})Z/, ":$1:$2.$3Z");
+        return new Date(iso);
+    } catch (e) {
+        return new Date(0);
+    }
+}
+
+/**
+ * Restore from a backup
+ * @param {string} backupPath - Path to backup folder
+ * @returns {boolean}
+ */
+export function restoreBackup(backupPath) {
+    try {
+        const lessonsBackup = path.join(backupPath, "lessons-learned.yaml");
+        const settingsBackup = path.join(backupPath, "settings.yaml");
+
+        if (fs.existsSync(lessonsBackup)) {
+            fs.copyFileSync(lessonsBackup, LESSONS_PATH);
+        }
+
+        if (fs.existsSync(settingsBackup)) {
+            fs.copyFileSync(settingsBackup, SETTINGS_PATH);
+        }
+
+        return true;
+    } catch (e) {
+        console.error("Failed to restore backup:", e.message);
+        return false;
+    }
+}
+
+/**
+ * Delete old backups, keep last N
+ * @param {number} keep - Number of backups to keep
+ */
+export function pruneBackups(keep = 5) {
+    const backups = listBackups();
+
+    if (backups.length <= keep) return;
+
+    const toDelete = backups.slice(keep);
+    for (const backup of toDelete) {
+        try {
+            fs.rmSync(backup.path, { recursive: true, force: true });
+        } catch (e) {
+            // Ignore deletion errors
+        }
+    }
+}
+
+export default {
+    createBackup,
+    listBackups,
+    restoreBackup,
+    pruneBackups,
+    BACKUPS_DIR
+};

package/lib/agent-cli/lib/backup.test.js

@@ -0,0 +1,78 @@
+/**
+ * @fileoverview Tests for backup module
+ */
+
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import fs from "fs";
+import path from "path";
+import os from "os";
+
+describe("backup", () => {
+    const testDir = path.join(os.tmpdir(), "test-backup-" + Date.now());
+
+    beforeEach(() => {
+        fs.mkdirSync(testDir, { recursive: true });
+    });
+
+    afterEach(() => {
+        if (fs.existsSync(testDir)) {
+            fs.rmSync(testDir, { recursive: true, force: true });
+        }
+    });
+
+    describe("backup file format", () => {
+        it("creates timestamped filename", () => {
+            const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
+            const filename = `backup-${timestamp}.yaml`;
+
+            expect(filename).toMatch(/^backup-\d{4}-\d{2}-\d{2}/);
+        });
+
+        it("backup contains valid YAML", () => {
+            const backupContent = `lessons:\n  - id: TEST\n    pattern: "x"`;
+            const backupPath = path.join(testDir, "backup.yaml");
+            fs.writeFileSync(backupPath, backupContent);
+
+            expect(fs.existsSync(backupPath)).toBe(true);
+            expect(fs.readFileSync(backupPath, "utf8")).toContain("lessons:");
+        });
+    });
+
+    describe("listBackups", () => {
+        it("returns empty array when no backups", () => {
+            const backupDir = path.join(testDir, "backups");
+            fs.mkdirSync(backupDir, { recursive: true });
+
+            const files = fs.readdirSync(backupDir);
+            expect(files).toHaveLength(0);
+        });
+
+        it("lists backup files", () => {
+            const backupDir = path.join(testDir, "backups");
+            fs.mkdirSync(backupDir, { recursive: true });
+            fs.writeFileSync(path.join(backupDir, "backup-2024.yaml"), "test");
+
+            const files = fs.readdirSync(backupDir);
+            expect(files).toHaveLength(1);
+        });
+    });
+
+    describe("pruneBackups", () => {
+        it("keeps specified number of backups", () => {
+            const backupDir = path.join(testDir, "backups");
+            fs.mkdirSync(backupDir, { recursive: true });
+
+            // Create 5 backups
+            for (let i = 1; i <= 5; i++) {
+                fs.writeFileSync(path.join(backupDir, `backup-${i}.yaml`), "test");
+            }
+
+            // Simulate pruning to keep 2
+            const files = fs.readdirSync(backupDir).sort().reverse();
+            const toDelete = files.slice(2);
+            toDelete.forEach(f => fs.unlinkSync(path.join(backupDir, f)));
+
+            expect(fs.readdirSync(backupDir)).toHaveLength(2);
+        });
+    });
+});