add-skill-kit 3.2.2 → 3.2.4

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

@@ -0,0 +1,149 @@
+/**
+ * @fileoverview Shell completion scripts generator
+ * Supports PowerShell, Bash, and Zsh
+ */
+
+/**
+ * All available commands for completion
+ */
+const COMMANDS = [
+    { name: "learn", description: "Teach a new pattern" },
+    { name: "recall", description: "Scan for violations" },
+    { name: "stats", description: "View statistics" },
+    { name: "audit", description: "Run compliance check" },
+    { name: "watch", description: "Real-time monitoring" },
+    { name: "settings", description: "Configure agent behavior" },
+    { name: "backup", description: "Backup & restore data" },
+    { name: "export", description: "Export & import data" },
+    { name: "proposals", description: "AI agent skill updates" },
+    { name: "completion", description: "Shell completion scripts" },
+    { name: "init", description: "Initialize agent config" },
+    { name: "help", description: "Show help" }
+];
+
+/**
+ * Generate PowerShell completion script
+ * @returns {string}
+ */
+export function generatePowerShellCompletion() {
+    const commands = COMMANDS.map(c => `'${c.name}'`).join(", ");
+
+    return `
+# Agent Skill Kit PowerShell Completion
+# Add to your $PROFILE
+
+Register-ArgumentCompleter -Native -CommandName agent -ScriptBlock {
+    param($wordToComplete, $commandAst, $cursorPosition)
+    
+    $commands = @(
+${COMMANDS.map(c => `        @{ Name = '${c.name}'; Description = '${c.description}' }`).join("\n")}
+    )
+    
+    $commands | Where-Object { $_.Name -like "$wordToComplete*" } | ForEach-Object {
+        [System.Management.Automation.CompletionResult]::new(
+            $_.Name,
+            $_.Name,
+            'ParameterValue',
+            $_.Description
+        )
+    }
+}
+
+Write-Host "Agent Skill Kit completion loaded" -ForegroundColor Green
+`;
+}
+
+/**
+ * Generate Bash completion script
+ * @returns {string}
+ */
+export function generateBashCompletion() {
+    const commands = COMMANDS.map(c => c.name).join(" ");
+
+    return `
+# Agent Skill Kit Bash Completion
+# Add to ~/.bashrc or ~/.bash_completion
+
+_agent_completions() {
+    local cur="\${COMP_WORDS[COMP_CWORD]}"
+    local commands="${commands}"
+    
+    COMPREPLY=($(compgen -W "$commands" -- "$cur"))
+}
+
+complete -F _agent_completions agent
+
+echo "Agent Skill Kit completion loaded"
+`;
+}
+
+/**
+ * Generate Zsh completion script
+ * @returns {string}
+ */
+export function generateZshCompletion() {
+    return `
+#compdef agent
+
+# Agent Skill Kit Zsh Completion
+# Add to ~/.zshrc or place in $fpath
+
+_agent() {
+    local -a commands
+    commands=(
+${COMMANDS.map(c => `        '${c.name}:${c.description}'`).join("\n")}
+    )
+    
+    _describe 'command' commands
+}
+
+compdef _agent agent
+
+echo "Agent Skill Kit completion loaded"
+`;
+}
+
+/**
+ * Get shell type from environment
+ * @returns {string} 'powershell' | 'bash' | 'zsh' | 'unknown'
+ */
+export function detectShell() {
+    if (process.platform === "win32") {
+        return "powershell";
+    }
+
+    const shell = process.env.SHELL || "";
+    if (shell.includes("zsh")) return "zsh";
+    if (shell.includes("bash")) return "bash";
+
+    return "unknown";
+}
+
+/**
+ * Get completion script for current or specified shell
+ * @param {string} [shell] - Optional shell type
+ * @returns {{ shell: string, script: string }}
+ */
+export function getCompletionScript(shell) {
+    const targetShell = shell || detectShell();
+
+    switch (targetShell) {
+        case "powershell":
+            return { shell: "powershell", script: generatePowerShellCompletion() };
+        case "bash":
+            return { shell: "bash", script: generateBashCompletion() };
+        case "zsh":
+            return { shell: "zsh", script: generateZshCompletion() };
+        default:
+            return { shell: "unknown", script: "" };
+    }
+}
+
+export default {
+    COMMANDS,
+    generatePowerShellCompletion,
+    generateBashCompletion,
+    generateZshCompletion,
+    detectShell,
+    getCompletionScript
+};

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

@@ -0,0 +1,35 @@
+/**
+ * @fileoverview Configuration for agent-skill-kit CLI
+ */
+
+import path from "path";
+import { fileURLToPath } from "url";
+import { createRequire } from "module";
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const require = createRequire(import.meta.url);
+
+/** Current working directory */
+export const cwd = process.cwd();
+
+/** Path to .agent directory (configurable via AGENT_DIR env) */
+export const AGENT_DIR = process.env.AGENT_DIR || path.join(cwd, ".agent");
+
+/** Path to knowledge directory */
+export const KNOWLEDGE_DIR = path.join(AGENT_DIR, "knowledge");
+
+/** Path to lessons learned file */
+export const LESSONS_PATH = path.join(KNOWLEDGE_DIR, "lessons-learned.yaml");
+
+/** Path to rules directory */
+export const RULES_DIR = path.join(AGENT_DIR, "rules");
+
+/** CLI version - read from package.json */
+export const VERSION = (() => {
+    try { return require("../package.json").version; }
+    catch { return "2.2.0"; }
+})();
+
+/** Debug mode */
+export const DEBUG = process.env.DEBUG === "true";
+

package/lib/agent-cli/lib/eslint-fix.js

@@ -0,0 +1,238 @@
+#!/usr/bin/env node
+/**
+ * ESLint Fix Integration
+ * 
+ * Wrapper around ESLint --fix to combine with pattern-based fix.
+ * Auto-detects ESLint config and runs fixes.
+ * 
+ * Usage: ag-smart fix --eslint <path>
+ */
+
+import { execSync, spawnSync } from "child_process";
+import fs from "fs";
+import path from "path";
+import { VERSION } from "./config.js";
+
+// ============================================================================
+// ESLINT DETECTION
+// ============================================================================
+
+/**
+ * Find ESLint config file in project
+ * @param {string} dir 
+ * @returns {string|null}
+ */
+function findEslintConfig(dir) {
+    const configFiles = [
+        ".eslintrc.js",
+        ".eslintrc.cjs",
+        ".eslintrc.json",
+        ".eslintrc.yaml",
+        ".eslintrc.yml",
+        ".eslintrc",
+        "eslint.config.js",
+        "eslint.config.mjs"
+    ];
+
+    let current = dir;
+    while (current !== path.dirname(current)) {
+        for (const config of configFiles) {
+            const configPath = path.join(current, config);
+            if (fs.existsSync(configPath)) {
+                return configPath;
+            }
+        }
+        // Check package.json for eslintConfig
+        const pkgPath = path.join(current, "package.json");
+        if (fs.existsSync(pkgPath)) {
+            try {
+                const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
+                if (pkg.eslintConfig) {
+                    return pkgPath; // ESLint config in package.json
+                }
+            } catch (e) { }
+        }
+        current = path.dirname(current);
+    }
+    return null;
+}
+
+/**
+ * Check if ESLint is available
+ * @returns {boolean}
+ */
+function isEslintAvailable() {
+    try {
+        execSync("npx eslint --version", { stdio: "ignore" });
+        return true;
+    } catch (e) {
+        return false;
+    }
+}
+
+// ============================================================================
+// ESLINT FIX
+// ============================================================================
+
+/**
+ * Run ESLint --fix on a path
+ * @param {string} targetPath 
+ * @param {object} options
+ * @returns {{ success: boolean, fixed: number, errors: number }}
+ */
+export function runEslintFix(targetPath, options = {}) {
+    const result = {
+        success: false,
+        fixed: 0,
+        errors: 0,
+        output: ""
+    };
+
+    if (!isEslintAvailable()) {
+        console.log("⚠️  ESLint not found. Install with: npm install eslint -D");
+        return result;
+    }
+
+    const configPath = findEslintConfig(process.cwd());
+    if (!configPath && !options.noConfig) {
+        console.log("⚠️  No ESLint config found. Run: npx eslint --init");
+        return result;
+    }
+
+    console.log(`\n🔧 Running ESLint --fix...`);
+    if (configPath) {
+        console.log(`   Config: ${path.relative(process.cwd(), configPath)}`);
+    }
+
+    try {
+        // Run ESLint with --fix
+        const args = [
+            "eslint",
+            targetPath,
+            "--fix",
+            "--format", "json"
+        ];
+
+        const proc = spawnSync("npx", args, {
+            encoding: "utf8",
+            shell: true,
+            maxBuffer: 10 * 1024 * 1024
+        });
+
+        // Parse output
+        if (proc.stdout) {
+            try {
+                const results = JSON.parse(proc.stdout);
+                let totalFixed = 0;
+                let totalErrors = 0;
+
+                results.forEach(file => {
+                    if (file.output) {
+                        // File was fixed (output contains fixed content)
+                        totalFixed++;
+                    }
+                    totalErrors += file.errorCount || 0;
+                });
+
+                result.fixed = totalFixed;
+                result.errors = totalErrors;
+                result.success = true;
+                result.output = proc.stdout;
+
+                console.log(`   ✅ Fixed: ${totalFixed} file(s)`);
+                if (totalErrors > 0) {
+                    console.log(`   ⚠️  Remaining errors: ${totalErrors}`);
+                }
+            } catch (e) {
+                // JSON parse failed, but fix might still have worked
+                result.success = proc.status === 0;
+            }
+        }
+
+        if (proc.status === 0) {
+            result.success = true;
+            console.log("   ✅ ESLint fix completed");
+        }
+
+    } catch (error) {
+        console.error(`   ❌ ESLint error: ${error.message}`);
+    }
+
+    return result;
+}
+
+// ============================================================================
+// COMBINED FIX
+// ============================================================================
+
+/**
+ * Run combined fix (ESLint + pattern-based)
+ * @param {string} targetPath 
+ * @param {object} options 
+ */
+export async function runCombinedFix(targetPath, options = {}) {
+    console.log(`\n🔧 Combined Fix v${VERSION}`);
+    console.log(`📂 Target: ${targetPath}`);
+    console.log("─".repeat(50));
+
+    // Step 1: ESLint fix
+    if (options.eslint !== false) {
+        runEslintFix(targetPath, options);
+    }
+
+    // Step 2: Pattern-based fix (import dynamically)
+    console.log("\n🧠 Running pattern-based fix...");
+    try {
+        const { fixDirectory } = await import("./fix.js");
+        const { loadKnowledge } = await import("./recall.js");
+
+        const db = loadKnowledge();
+        const results = fixDirectory(targetPath, db, options.mode || "safe");
+
+        if (results.length > 0) {
+            let total = 0;
+            results.forEach(r => total += r.fixes);
+            console.log(`   ✅ Pattern fixes: ${total}`);
+        } else {
+            console.log("   ✅ No pattern violations to fix");
+        }
+    } catch (e) {
+        console.log(`   ⚠️  Pattern fix skipped: ${e.message}`);
+    }
+
+    console.log("\n" + "─".repeat(50));
+    console.log("🎉 Combined fix completed!");
+}
+
+// ============================================================================
+// CLI
+// ============================================================================
+
+const args = process.argv.slice(2);
+
+if (args.includes("--help") || args.length === 0) {
+    console.log(`
+🔧 ESLint Fix Integration v${VERSION}
+
+Usage:
+  node eslint-fix.js <path> [options]
+
+Options:
+  --no-eslint    Skip ESLint fix
+  --no-pattern   Skip pattern-based fix
+  --mode <mode>  Pattern fix mode (safe|aggressive)
+  --help         Show this help
+
+This combines ESLint --fix with pattern-based fixes.
+`);
+    process.exit(0);
+}
+
+const target = args[0];
+const options = {
+    eslint: !args.includes("--no-eslint"),
+    pattern: !args.includes("--no-pattern"),
+    mode: args.includes("--mode") ? args[args.indexOf("--mode") + 1] : "safe"
+};
+
+runCombinedFix(target, options);

package/lib/agent-cli/lib/evolution-signal.js

@@ -0,0 +1,215 @@
+/**
+ * Evolution Signal Layer - Backward Compatibility Adapter
+ * 
+ * This file maintains the old API while delegating to the new architecture.
+ * Allows gradual migration without breaking existing code.
+ * 
+ * OLD LOCATION: lib/evolution-signal.js
+ * NEW LOCATION: src/core/evolution/* + src/data/repositories/*
+ * 
+ * @deprecated Use new architecture: import from 'src/core/evolution'
+ */
+
+// Import new architecture components
+import { EvolutionSignal } from '../src/core/evolution/evolution-signal.js';
+import { ThresholdChecker } from '../src/core/evolution/threshold-checker.js';
+import { ReviewGate } from '../src/core/evolution/review-gate.js';
+import { SignalRepository } from '../src/data/repositories/signal-repository.js';
+import { JsonStorage } from '../src/data/storage/json-storage.js';
+import { SignalDetector as SignalService } from '../src/core/evolution/signal-detector.js';
+import { KNOWLEDGE_DIR } from './config.js';
+
+// ============================================================================
+// BACKWARD COMPATIBLE EXPORTS
+// ============================================================================
+
+// Re-export EvolutionSignal class
+export { EvolutionSignal };
+
+// Lazy-initialized singleton instances
+let _storage = null;
+let _repository = null;
+let _detector = null;
+
+function getDetector() {
+    if (!_detector) {
+        _storage = new JsonStorage(KNOWLEDGE_DIR);
+        _repository = new SignalRepository(_storage);
+        _detector = new SignalService(_repository);
+    }
+    return _detector;
+}
+
+// ============================================================================
+// THRESHOLD CHECKER (Backward Compatible)
+// ============================================================================
+
+/**
+ * @deprecated Use ThresholdChecker.check() from src/core/evolution
+ */
+export function checkEvolutionThreshold(lesson, threshold = 10) {
+    return ThresholdChecker.check(lesson, threshold);
+}
+
+/**
+ * @deprecated Use ThresholdChecker.calculateStability() from src/core/evolution
+ */
+export function calculatePatternStability(violationHistory = []) {
+    return ThresholdChecker.calculateStability(violationHistory);
+}
+
+// ============================================================================
+// SIGNAL QUEUE (Backward Compatible)
+// ============================================================================
+
+/**
+ * SignalQueue - Backward compatible class
+ * @deprecated Use SignalRepository + SignalDetector from new architecture
+ */
+class SignalQueueCompat {
+    get signals() {
+        // Return cached signals (sync access)
+        if (!this._cachedSignals) {
+            // Synchronous fallback - load on first access
+            this._cachedSignals = [];
+            this.load().then(signals => {
+                this._cachedSignals = signals;
+            });
+        }
+        return this._cachedSignals;
+    }
+
+    async load() {
+        const detector = getDetector();
+        const all = await detector.signalRepository.findAll();
+        this._cachedSignals = all;
+        return all;
+    }
+
+    async save() {
+        // Auto-save through repository (no-op for compatibility)
+        return;
+    }
+
+    async add(signal) {
+        const detector = getDetector();
+        const result = await detector.queue(
+            signal.lessonId,
+            { ready: true, reason: signal.reason, confidence: signal.confidence },
+            signal.metadata
+        );
+        await this.load(); // Refresh cache
+        return result;
+    }
+
+    getPending() {
+        return this.signals.filter(s => s.status === 'pending');
+    }
+
+    getByLesson(lessonId) {
+        return this.signals.filter(s => s.lessonId === lessonId);
+    }
+
+    async approve(signalId) {
+        const detector = getDetector();
+        const result = await detector.approve(signalId);
+        await this.load(); // Refresh cache
+        return result;
+    }
+
+    async reject(signalId) {
+        const detector = getDetector();
+        const result = await detector.reject(signalId);
+        await this.load(); // Refresh cache
+        return result;
+    }
+
+    async execute(signalId) {
+        const detector = getDetector();
+        const result = await detector.execute(signalId);
+        await this.load(); // Refresh cache
+        return result;
+    }
+
+    async cleanup() {
+        const detector = getDetector();
+        await detector.signalRepository.cleanup();
+        await this.load(); // Refresh cache
+    }
+}
+
+// Singleton instance for backward compatibility
+const signalQueueInstance = new SignalQueueCompat();
+
+// Initialize on module load
+signalQueueInstance.load().catch(err => {
+    console.error('Failed to initialize signal queue:', err);
+});
+
+export { signalQueueInstance as signalQueue };
+
+// ============================================================================
+// REVIEW GATE (Backward Compatible)
+// ============================================================================
+
+/**
+ * @deprecated Use ReviewGate.evaluate() from src/core/evolution
+ */
+export function reviewGate(signal, settings = {}) {
+    return ReviewGate.evaluate(signal, settings);
+}
+
+// ============================================================================
+// HELPER FUNCTIONS (Backward Compatible)
+// ============================================================================
+
+/**
+ * @deprecated Use SignalDetector.queue() from new architecture
+ */
+export async function queueEvolutionSignal(lessonId, checkResult, metadata = {}) {
+    if (!checkResult.ready) return null;
+
+    const signal = new EvolutionSignal(
+        lessonId,
+        checkResult.reason,
+        checkResult.confidence,
+        metadata
+    );
+
+    return signalQueueInstance.add(signal);
+}
+
+/**
+ * @deprecated Use SignalDetector.getStats() from new architecture
+ */
+export async function getEvolutionStats() {
+    const detector = getDetector();
+    return detector.getStats();
+}
+
+// ============================================================================
+// MIGRATION GUIDE
+// ============================================================================
+
+/**
+ * HOW TO MIGRATE TO NEW ARCHITECTURE:
+ * 
+ * OLD CODE:
+ * ```
+ * import { signalQueue, checkEvolutionThreshold } from './evolution-signal.js';
+ * 
+ * const check = checkEvolutionThreshold(lesson, 10);
+ * await signalQueue.add(signal);
+ * ```
+ * 
+ * NEW CODE:
+ * ```
+ * import { ThresholdChecker } from '../src/core/evolution/threshold-checker.js';
+ * import { SignalRepository } from '../src/data/repositories/signal-repository.js';
+ * import { SignalDetector } from '../src/core/evolution/signal-detector.js';
+ * 
+ * const check = ThresholdChecker.check(lesson, 10);
+ * const detector = new SignalDetector(repository);
+ * await detector.queue(lessonId, check, metadata);
+ * ```
+ */