add-skill-kit 3.2.3 → 3.2.4

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);
+        });
+    });
+});

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

@@ -0,0 +1,222 @@
+#!/usr/bin/env python3
+"""
+Master Checklist Runner - Agent Skill Kit
+==========================================
+
+Orchestrates all validation scripts in priority order.
+Use this for incremental validation during development.
+
+Usage:
+    python scripts/checklist.py .                    # Run core checks
+    python scripts/checklist.py . --url <URL>        # Include performance checks
+
+Priority Order:
+    P0: Security Scan (vulnerabilities, secrets)
+    P1: Lint & Type Check (code quality)
+    P2: Schema Validation (if database exists)
+    P3: Test Runner (unit/integration tests)
+    P4: UX Audit (psychology laws, accessibility)
+    P5: SEO Check (meta tags, structure)
+    P6: Performance (lighthouse - requires URL)
+"""
+
+import sys
+import subprocess
+import argparse
+from pathlib import Path
+from typing import List, Tuple, Optional
+
+# ANSI colors for terminal output
+class Colors:
+    HEADER = '\033[95m'
+    BLUE = '\033[94m'
+    CYAN = '\033[96m'
+    GREEN = '\033[92m'
+    YELLOW = '\033[93m'
+    RED = '\033[91m'
+    ENDC = '\033[0m'
+    BOLD = '\033[1m'
+
+def print_header(text: str):
+    print(f"\n{Colors.BOLD}{Colors.CYAN}{'='*60}{Colors.ENDC}")
+    print(f"{Colors.BOLD}{Colors.CYAN}{text.center(60)}{Colors.ENDC}")
+    print(f"{Colors.BOLD}{Colors.CYAN}{'='*60}{Colors.ENDC}\n")
+
+def print_step(text: str):
+    print(f"{Colors.BOLD}{Colors.BLUE}🔄 {text}{Colors.ENDC}")
+
+def print_success(text: str):
+    print(f"{Colors.GREEN}✅ {text}{Colors.ENDC}")
+
+def print_warning(text: str):
+    print(f"{Colors.YELLOW}⚠️  {text}{Colors.ENDC}")
+
+def print_error(text: str):
+    print(f"{Colors.RED}❌ {text}{Colors.ENDC}")
+
+# Define priority-ordered checks
+CORE_CHECKS = [
+    ("Security Scan", ".agent/skills/vulnerability-scanner/scripts/security_scan.py", True),
+    ("Smart Audit", ".agent/scripts/audit.js", True),
+    ("Lint Check", ".agent/skills/lint-and-validate/scripts/lint_runner.py", True),
+    ("Schema Validation", ".agent/skills/database-design/scripts/schema_validator.py", False),
+    ("Test Runner", ".agent/skills/testing-patterns/scripts/test_runner.py", False),
+    ("UX Audit", ".agent/skills/frontend-design/scripts/ux_audit.py", False),
+    ("SEO Check", ".agent/skills/seo-fundamentals/scripts/seo_checker.py", False),
+]
+
+PERFORMANCE_CHECKS = [
+    ("Lighthouse Audit", ".agent/skills/performance-profiling/scripts/lighthouse_audit.py", True),
+    ("Playwright E2E", ".agent/skills/webapp-testing/scripts/playwright_runner.py", False),
+]
+
+def check_script_exists(script_path: Path) -> bool:
+    """Check if script file exists"""
+    return script_path.exists() and script_path.is_file()
+
+def run_script(name: str, script_path: Path, project_path: str, url: Optional[str] = None) -> dict:
+    """
+    Run a validation script and capture results
+    
+    Returns:
+        dict with keys: name, passed, output, skipped
+    """
+    if not check_script_exists(script_path):
+        print_warning(f"{name}: Script not found, skipping")
+        return {"name": name, "passed": True, "output": "", "skipped": True}
+    
+    print_step(f"Running: {name}")
+    
+    # Build command
+    if str(script_path).endswith('.js'):
+        cmd = ["node", str(script_path), project_path]
+    else:
+        cmd = ["python", str(script_path), project_path]
+        
+    if url and ("lighthouse" in script_path.name.lower() or "playwright" in script_path.name.lower()):
+        cmd.append(url)
+    
+    # Run script
+    try:
+        result = subprocess.run(
+            cmd,
+            capture_output=True,
+            text=True,
+            timeout=300  # 5 minute timeout
+        )
+        
+        passed = result.returncode == 0
+        
+        if passed:
+            print_success(f"{name}: PASSED")
+        else:
+            print_error(f"{name}: FAILED")
+            if result.stderr:
+                print(f"  Error: {result.stderr[:200]}")
+        
+        return {
+            "name": name,
+            "passed": passed,
+            "output": result.stdout,
+            "error": result.stderr,
+            "skipped": False
+        }
+    
+    except subprocess.TimeoutExpired:
+        print_error(f"{name}: TIMEOUT (>5 minutes)")
+        return {"name": name, "passed": False, "output": "", "error": "Timeout", "skipped": False}
+    
+    except Exception as e:
+        print_error(f"{name}: ERROR - {str(e)}")
+        return {"name": name, "passed": False, "output": "", "error": str(e), "skipped": False}
+
+def print_summary(results: List[dict]):
+    """Print final summary report"""
+    print_header("📊 CHECKLIST SUMMARY")
+    
+    passed_count = sum(1 for r in results if r["passed"] and not r.get("skipped"))
+    failed_count = sum(1 for r in results if not r["passed"] and not r.get("skipped"))
+    skipped_count = sum(1 for r in results if r.get("skipped"))
+    
+    print(f"Total Checks: {len(results)}")
+    print(f"{Colors.GREEN}✅ Passed: {passed_count}{Colors.ENDC}")
+    print(f"{Colors.RED}❌ Failed: {failed_count}{Colors.ENDC}")
+    print(f"{Colors.YELLOW}⏭️  Skipped: {skipped_count}{Colors.ENDC}")
+    print()
+    
+    # Detailed results
+    for r in results:
+        if r.get("skipped"):
+            status = f"{Colors.YELLOW}⏭️ {Colors.ENDC}"
+        elif r["passed"]:
+            status = f"{Colors.GREEN}✅{Colors.ENDC}"
+        else:
+            status = f"{Colors.RED}❌{Colors.ENDC}"
+        
+        print(f"{status} {r['name']}")
+    
+    print()
+    
+    if failed_count > 0:
+        print_error(f"{failed_count} check(s) FAILED - Please fix before proceeding")
+        return False
+    else:
+        print_success("All checks PASSED ✨")
+        return True
+
+def main():
+    parser = argparse.ArgumentParser(
+        description="Run Agent Skill Kit validation checklist",
+        formatter_class=argparse.RawDescriptionHelpFormatter,
+        epilog="""
+Examples:
+  python scripts/checklist.py .                      # Core checks only
+  python scripts/checklist.py . --url http://localhost:3000  # Include performance
+        """
+    )
+    parser.add_argument("project", help="Project path to validate")
+    parser.add_argument("--url", help="URL for performance checks (lighthouse, playwright)")
+    parser.add_argument("--skip-performance", action="store_true", help="Skip performance checks even if URL provided")
+    
+    args = parser.parse_args()
+    
+    project_path = Path(args.project).resolve()
+    
+    if not project_path.exists():
+        print_error(f"Project path does not exist: {project_path}")
+        sys.exit(1)
+    
+    print_header("🚀 AGENT SKILLS KIT - MASTER CHECKLIST")
+    print(f"Project: {project_path}")
+    print(f"URL: {args.url if args.url else 'Not provided (performance checks skipped)'}")
+    
+    results = []
+    
+    # Run core checks
+    print_header("📋 CORE CHECKS")
+    for name, script_path, required in CORE_CHECKS:
+        script = project_path / script_path
+        result = run_script(name, script, str(project_path))
+        results.append(result)
+        
+        # If required check fails, stop
+        if required and not result["passed"] and not result.get("skipped"):
+            print_error(f"CRITICAL: {name} failed. Stopping checklist.")
+            print_summary(results)
+            sys.exit(1)
+    
+    # Run performance checks if URL provided
+    if args.url and not args.skip_performance:
+        print_header("⚡ PERFORMANCE CHECKS")
+        for name, script_path, required in PERFORMANCE_CHECKS:
+            script = project_path / script_path
+            result = run_script(name, script, str(project_path), args.url)
+            results.append(result)
+    
+    # Print summary
+    all_passed = print_summary(results)
+    
+    sys.exit(0 if all_passed else 1)
+
+if __name__ == "__main__":
+    main()