pikakit 3.0.5 → 3.7.2

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

@@ -1,215 +0,0 @@
-/**
- * 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);
- * ```
- */

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

@@ -1,86 +0,0 @@
-/**
- * @fileoverview Export and Import for PikaKit
- * Share settings between projects
- */
-
-import fs from "fs";
-import path from "path";
-import yaml from "js-yaml";
-import { LESSONS_PATH } from "./config.js";
-import { SETTINGS_PATH, loadSettings } from "./settings.js";
-import { loadKnowledge, saveKnowledge } from "./recall.js";
-
-/**
- * Export lessons and settings to JSON file
- * @param {string} outputPath - Output file path
- * @returns {boolean}
- */
-export function exportData(outputPath) {
-    try {
-        const data = {
-            version: 1,
-            exportedAt: new Date().toISOString(),
-            lessons: loadKnowledge().lessons || [],
-            settings: loadSettings()
-        };
-
-        fs.writeFileSync(outputPath, JSON.stringify(data, null, 2), "utf8");
-        return true;
-    } catch (e) {
-        console.error("Failed to export:", e.message);
-        return false;
-    }
-}
-
-/**
- * Import data from JSON file
- * @param {string} inputPath - Input file path
- * @param {"merge" | "replace"} mode - Import mode
- * @returns {{ success: boolean, lessonsCount: number, hasSettings: boolean }}
- */
-export function importData(inputPath, mode = "merge") {
-    try {
-        if (!fs.existsSync(inputPath)) {
-            return { success: false, lessonsCount: 0, hasSettings: false };
-        }
-
-        const content = fs.readFileSync(inputPath, "utf8");
-        const data = JSON.parse(content);
-
-        // Import lessons
-        const currentDb = loadKnowledge();
-
-        if (mode === "replace") {
-            currentDb.lessons = data.lessons || [];
-        } else {
-            // Merge - add new lessons, skip duplicates by pattern
-            const existingPatterns = new Set(currentDb.lessons.map(l => l.pattern));
-            for (const lesson of (data.lessons || [])) {
-                if (!existingPatterns.has(lesson.pattern)) {
-                    currentDb.lessons.push(lesson);
-                }
-            }
-        }
-
-        saveKnowledge(currentDb);
-
-        // Import settings if present
-        let hasSettings = false;
-        if (data.settings) {
-            const settingsPath = SETTINGS_PATH;
-            fs.writeFileSync(settingsPath, yaml.dump(data.settings), "utf8");
-            hasSettings = true;
-        }
-
-        return {
-            success: true,
-            lessonsCount: data.lessons?.length || 0,
-            hasSettings
-        };
-    } catch (e) {
-        console.error("Failed to import:", e.message);
-        return { success: false, lessonsCount: 0, hasSettings: false };
-    }
-}
-
-export default { exportData, importData };

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

@@ -1,65 +0,0 @@
-/**
- * @fileoverview Tests for export module
- */
-
-import { describe, it, expect, beforeEach, afterEach } from "vitest";
-import fs from "fs";
-import path from "path";
-import os from "os";
-
-describe("export", () => {
-    const testDir = path.join(os.tmpdir(), "test-export-" + Date.now());
-
-    beforeEach(() => {
-        fs.mkdirSync(testDir, { recursive: true });
-    });
-
-    afterEach(() => {
-        if (fs.existsSync(testDir)) {
-            fs.rmSync(testDir, { recursive: true, force: true });
-        }
-    });
-
-    describe("export format", () => {
-        it("creates valid JSON structure", () => {
-            const data = {
-                version: 1,
-                exportedAt: new Date().toISOString(),
-                lessons: [{ id: "TEST", pattern: "x" }]
-            };
-
-            const outputPath = path.join(testDir, "export.json");
-            fs.writeFileSync(outputPath, JSON.stringify(data, null, 2));
-
-            const loaded = JSON.parse(fs.readFileSync(outputPath, "utf8"));
-            expect(loaded.version).toBe(1);
-            expect(loaded.lessons).toHaveLength(1);
-        });
-
-        it("supports lesson array format", () => {
-            const lessons = [
-                { id: "L1", pattern: "a", message: "msg" },
-                { id: "L2", pattern: "b", message: "msg2" }
-            ];
-
-            const data = { version: 1, lessons };
-            const json = JSON.stringify(data);
-            const parsed = JSON.parse(json);
-
-            expect(parsed.lessons).toHaveLength(2);
-            expect(parsed.lessons[0].id).toBe("L1");
-        });
-    });
-
-    describe("import validation", () => {
-        it("validates version field", () => {
-            const valid = { version: 1, lessons: [] };
-            expect(valid.version).toBe(1);
-        });
-
-        it("validates lessons array", () => {
-            const valid = { version: 1, lessons: [{ id: "X", pattern: "y" }] };
-            expect(Array.isArray(valid.lessons)).toBe(true);
-        });
-    });
-});

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

@@ -1,337 +0,0 @@
-#!/usr/bin/env node
-/**
- * Smart Fix - Auto-fix Detected Violations
- * 
- * Automatically fixes code violations based on learned patterns.
- * 
- * Features:
- * - Comment out violations (safe mode)
- * - Remove violations (aggressive mode)
- * - Replace patterns (custom fix rules)
- * 
- * Usage: agent fix <file|directory> [--mode safe|aggressive]
- */
-
-import fs from "fs";
-import path from "path";
-import { loadKnowledge, saveKnowledge, scanFile, scanDirectory } from "./recall.js";
-import { VERSION } from "./config.js";
-
-// ============================================================================
-// FIX RULES
-// ============================================================================
-
-/**
- * Built-in fix rules for common patterns
- */
-const FIX_RULES = {
-    // Console statements → comment out
-    "console.log": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-    "console.warn": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-    "console.error": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-    "console.debug": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-    "console.info": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-
-    // var → const
-    "\\bvar\\s+": {
-        type: "replace",
-        replacement: (line) => line.replace(/\bvar\s+/, "const ")
-    },
-
-    // == → === (loose equality)
-    "[^!=]==[^=]": {
-        type: "replace",
-        replacement: (line) => line.replace(/([^!=])==([^=])/g, "$1===$2")
-    },
-
-    // != → !== (loose inequality)
-    "[^!]=!=[^=]": {
-        type: "replace",
-        replacement: (line) => line.replace(/([^!])!=([^=])/g, "$1!==$2")
-    },
-
-    // debugger → remove
-    "\\bdebugger\\b": {
-        type: "remove",
-        replacement: () => ""
-    },
-
-    // alert → comment out
-    "\\balert\\(": {
-        type: "comment",
-        replacement: (line) => `// REMOVED: ${line.trim()}`
-    },
-
-    // TODO/FIXME in production → warn only
-    "TODO|FIXME": {
-        type: "comment",
-        replacement: (line) => line // Leave as is, just flag
-    },
-
-    // Empty catch blocks → add comment
-    "catch\\s*\\([^)]*\\)\\s*\\{\\s*\\}": {
-        type: "replace",
-        replacement: (line) => line.replace(/\{\s*\}/, "{ /* Intentionally empty */ }")
-    },
-
-    // Promise without catch → add comment
-    "\\.then\\([^)]*\\)(?!.*\\.catch)": {
-        type: "comment",
-        replacement: (line) => `${line} // TODO: Add .catch()`
-    },
-
-    // Hardcoded localhost → flag
-    "localhost:\\d+": {
-        type: "comment",
-        replacement: (line) => `// REVIEW: ${line.trim()} // Hardcoded URL`
-    },
-
-    // @ts-ignore → flag for review
-    "@ts-ignore": {
-        type: "comment",
-        replacement: (line) => `${line} // TODO: Fix instead of ignoring`
-    },
-
-    // any type in TypeScript → flag
-    ": any": {
-        type: "comment",
-        replacement: (line) => `${line} // TODO: Replace 'any' with proper type`
-    }
-};
-
-// ============================================================================
-// FIX ENGINE
-// ============================================================================
-
-/**
- * Fix a single file
- * @param {string} filePath 
- * @param {object} db - Knowledge base
- * @param {string} mode - 'safe' or 'aggressive'
- * @returns {{ file: string, fixes: number, backup: string }}
- */
-function fixFile(filePath, db, mode = "safe") {
-    if (!fs.existsSync(filePath)) {
-        return { file: filePath, fixes: 0, error: "File not found" };
-    }
-
-    const content = fs.readFileSync(filePath, "utf8");
-    const lines = content.split("\n");
-    let fixCount = 0;
-    let modified = false;
-
-    // Scan for violations
-    const result = scanFile(filePath, db, false);
-
-    if (result.violations.length === 0) {
-        return { file: filePath, fixes: 0 };
-    }
-
-    // Create backup
-    const backupPath = filePath + ".backup";
-    fs.writeFileSync(backupPath, content, "utf8");
-
-    // Apply fixes
-    result.violations.forEach(({ lesson, matches }) => {
-        const rule = FIX_RULES[lesson.pattern] || {
-            type: mode === "aggressive" ? "remove" : "comment",
-            replacement: (line) => mode === "aggressive" ? "" : `// FLAGGED: ${line.trim()}`
-        };
-
-        matches.forEach(match => {
-            const lineIdx = match.line - 1;
-            if (lineIdx >= 0 && lineIdx < lines.length) {
-                const originalLine = lines[lineIdx];
-
-                if (rule.type === "remove") {
-                    lines[lineIdx] = "";
-                    fixCount++;
-                    modified = true;
-                } else if (rule.type === "comment") {
-                    if (!originalLine.trim().startsWith("//")) {
-                        lines[lineIdx] = rule.replacement(originalLine);
-                        fixCount++;
-                        modified = true;
-                    }
-                } else if (rule.type === "replace") {
-                    const newLine = rule.replacement(originalLine);
-                    if (newLine !== originalLine) {
-                        lines[lineIdx] = newLine;
-                        fixCount++;
-                        modified = true;
-                    }
-                }
-            }
-        });
-    });
-
-    // Write fixed content
-    if (modified) {
-        const newContent = lines.filter(l => l !== "" || mode !== "aggressive").join("\n");
-        fs.writeFileSync(filePath, newContent, "utf8");
-    } else {
-        // Remove backup if no changes
-        fs.unlinkSync(backupPath);
-    }
-
-    return {
-        file: filePath,
-        fixes: fixCount,
-        backup: modified ? backupPath : null
-    };
-}
-
-/**
- * Fix all files in directory
- */
-function fixDirectory(dirPath, db, mode = "safe") {
-    const results = [];
-    const extensions = [".js", ".ts", ".tsx", ".jsx", ".mjs"];
-
-    function walk(dir) {
-        const entries = fs.readdirSync(dir, { withFileTypes: true });
-
-        for (const entry of entries) {
-            const fullPath = path.join(dir, entry.name);
-
-            if (entry.isDirectory()) {
-                if (!["node_modules", ".git", "dist", "build"].includes(entry.name)) {
-                    walk(fullPath);
-                }
-            } else if (entry.isFile()) {
-                const ext = path.extname(entry.name);
-                if (extensions.includes(ext)) {
-                    const result = fixFile(fullPath, db, mode);
-                    if (result.fixes > 0) {
-                        results.push(result);
-                    }
-                }
-            }
-        }
-    }
-
-    if (fs.statSync(dirPath).isDirectory()) {
-        walk(dirPath);
-    } else {
-        const result = fixFile(dirPath, db, mode);
-        if (result.fixes > 0) {
-            results.push(result);
-        }
-    }
-
-    return results;
-}
-
-// ============================================================================
-// CLI
-// ============================================================================
-
-async function main() {
-    const args = process.argv.slice(2);
-
-    if (args.length === 0 || args.includes("--help")) {
-        console.log(`
-🔧 Smart Fix v${VERSION} - Auto-fix Violations
-
-Usage:
-  agent fix <file|directory> [options]
-
-Options:
-  --mode safe        Comment out violations (default)
-  --mode aggressive  Remove violations entirely
-  --eslint           Also run ESLint --fix
-  --dry-run          Show what would be fixed without changing files
-  --help             Show this help
-
-Examples:
-  agent fix src/
-  agent fix app.js --mode aggressive
-  agent fix src/ --eslint
-`);
-        process.exit(0);
-    }
-
-    const target = args[0];
-    const mode = args.includes("--mode")
-        ? args[args.indexOf("--mode") + 1]
-        : "safe";
-    const dryRun = args.includes("--dry-run");
-    const useEslint = args.includes("--eslint");
-
-    const db = loadKnowledge();
-
-    if (!db.lessons || db.lessons.length === 0) {
-        console.log("ℹ️  No lessons learned yet. Nothing to fix.");
-        process.exit(0);
-    }
-
-    console.log(`\n🔧 Smart Fix v${VERSION}`);
-    console.log(`📂 Target: ${target}`);
-    console.log(`🎯 Mode: ${mode}`);
-    if (useEslint) console.log(`🔧 ESLint: enabled`);
-    console.log(`${"─".repeat(50)}\n`);
-
-    // Run ESLint fix first if enabled
-    if (useEslint && !dryRun) {
-        try {
-            const { runEslintFix } = await import("./eslint-fix.js");
-            runEslintFix(target);
-        } catch (e) {
-            console.log(`⚠️  ESLint fix skipped: ${e.message}`);
-        }
-    }
-
-    if (dryRun) {
-        console.log("🔍 DRY RUN - showing what would be fixed:\n");
-        const scanResults = scanDirectory(target, db);
-        scanResults.forEach(r => {
-            console.log(`📄 ${path.relative(process.cwd(), r.file)}`);
-            r.violations.forEach(v => {
-                console.log(`   Would fix ${v.matches.length} violation(s) of [${v.lesson.id}]`);
-            });
-        });
-        process.exit(0);
-    }
-
-    const results = fixDirectory(target, db, mode);
-
-    if (results.length === 0) {
-        console.log("✅ No pattern violations to fix.");
-        process.exit(0);
-    }
-
-    let totalFixes = 0;
-    results.forEach(r => {
-        totalFixes += r.fixes;
-        console.log(`✅ ${path.relative(process.cwd(), r.file)}: ${r.fixes} fix(es)`);
-        if (r.backup) {
-            console.log(`   📦 Backup: ${path.basename(r.backup)}`);
-        }
-    });
-
-    console.log(`\n${"─".repeat(50)}`);
-    console.log(`🎉 Fixed ${totalFixes} violation(s) in ${results.length} file(s)`);
-    console.log(`💡 Backups created with .backup extension`);
-}
-
-// Run if executed directly (as async)
-if (process.argv[1].includes("fix")) {
-    main().catch(console.error);
-}
-
-export { fixFile, fixDirectory, FIX_RULES };