wolverine-ai 2.3.1 → 2.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wolverine-ai",
-  "version": "2.3.1",
+  "version": "2.3.2",
   "description": "Self-healing Node.js server framework powered by AI. Catches crashes, diagnoses errors, generates fixes, verifies, and restarts — automatically.",
   "main": "src/index.js",
   "bin": {

package/server/config/settings.json

@@ -26,7 +26,7 @@
     "tool": "claude-opus-4-6",
     "classifier": "claude-haiku-4-5",
     "audit": "claude-haiku-4-5",
-    "compacting": "claude-sonnet-4-6",
+    "compacting": "claude-haiku-4-5",
     "research": "claude-sonnet-4-6",
     "embedding": "text-embedding-3-small"
   },

package/src/backup/backup-manager.js

@@ -20,6 +20,16 @@ const MANIFEST_FILE = path.join(BACKUPS_DIR, "manifest.json");
 const STABILITY_THRESHOLD_MS = 30 * 60 * 1000;
 const RETENTION_MS = 7 * 24 * 60 * 60 * 1000;
 
+// Files that should NEVER be overwritten during rollback.
+// These are platform/infrastructure files that would break the server if rolled back.
+const NEVER_ROLLBACK = [
+  "server/config/settings.json",
+  "server/lib/db.js",
+  "server/lib/redis.js",
+  ".env",
+  ".env.local",
+];
+
 class BackupManager {
   constructor(projectRoot) {
     this.projectRoot = path.resolve(projectRoot);
@@ -97,8 +107,12 @@ class BackupManager {
 
     let allRestored = true;
     for (const file of entry.files) {
+      // Skip protected config/infrastructure files
+      if (NEVER_ROLLBACK.some(p => file.relative === p || file.relative.endsWith(p))) {
+        console.log(chalk.gray(`  🔒 Skipped (protected): ${file.relative}`));
+        continue;
+      }
       if (fs.existsSync(file.backup)) {
-        // Ensure parent dir exists
         fs.mkdirSync(path.dirname(file.original), { recursive: true });
         fs.copyFileSync(file.backup, file.original);
         console.log(chalk.yellow(`  ↩️  Restored: ${file.relative}`));
@@ -150,6 +164,7 @@ class BackupManager {
 
     let allRestored = true;
     for (const file of entry.files) {
+      if (NEVER_ROLLBACK.some(p => file.relative === p || file.relative.endsWith(p))) continue;
       if (fs.existsSync(file.backup)) {
         fs.mkdirSync(path.dirname(file.original), { recursive: true });
         fs.copyFileSync(file.backup, file.original);

package/src/core/verifier.js

@@ -1,4 +1,5 @@
 const { spawn } = require("child_process");
+const path = require("path");
 const chalk = require("chalk");
 const { parseError, classifyError } = require("./error-parser");
 
@@ -131,12 +132,13 @@ function bootProbe(scriptPath, cwd, originalErrorSignature) {
  * @param {object} routeContext — optional { path, method } for route-level testing
  */
 async function verifyFix(scriptPath, cwd, originalErrorSignature, routeContext) {
-  // Simple errors (TypeError, ReferenceError, SyntaxError) — trust syntax+boot, skip route probe.
-  // If the fix is wrong, ErrorMonitor will catch the 500 and re-trigger heal. This avoids
-  // the expensive cascade where a working fix gets rolled back because the route probe
-  // can't boot the full server in isolation.
-  const isSimpleError = /TypeError|ReferenceError|SyntaxError|Cannot find module/.test(originalErrorSignature || "");
-  const skipRouteProbe = isSimpleError;
+  // Simple errors — trust syntax+boot, skip route probe entirely.
+  // The route probe spawns a full server process which crashes on external deps
+  // (pg, redis, cors, etc.) even when the actual fix is correct. ErrorMonitor
+  // catches any remaining 500s as the real safety net.
+  const sig = originalErrorSignature || "";
+  const isSimpleError = /TypeError|ReferenceError|SyntaxError|Cannot find module|Cannot read prop|is not defined|is not a function|Unexpected token/.test(sig);
+  const skipRouteProbe = true; // ALWAYS skip route probe — it fails on servers with external deps
   const steps = (!skipRouteProbe && routeContext?.path) ? 3 : 2;
 
   console.log(chalk.yellow("\n🔬 Verifying fix...\n"));
@@ -178,8 +180,38 @@ async function verifyFix(scriptPath, cwd, originalErrorSignature, routeContext)
     } else {
       console.log(chalk.gray(`  ⚠️  Route probe skipped: ${routeResult.reason || "unknown"}`));
     }
-  } else if (skipRouteProbe && routeContext?.path) {
-    console.log(chalk.gray(`  ⚡ Skipping route probe (simple error — ErrorMonitor is safety net)`));
+  } else if (routeContext?.path) {
+    console.log(chalk.gray(`  ⚡ Skipping route probe — ErrorMonitor is the safety net`));
+  }
+
+  // Step 3 (replacement): Isolated module load test — can the changed file be required?
+  // This catches cases where the AI fix introduces a require error or crashes on load,
+  // without needing to boot the full server (which fails on external deps).
+  if (scriptPath) {
+    const changedFile = scriptPath;
+    const relPath = path.relative(cwd, changedFile).replace(/\\/g, "/");
+    if (relPath.startsWith("server/") && relPath.endsWith(".js")) {
+      try {
+        const { execSync } = require("child_process");
+        const testCode = `try{require('./${relPath}');console.log('MODULE_OK')}catch(e){console.error(e.message);process.exit(1)}`;
+        const out = execSync(`node -e "${testCode}"`, {
+          cwd, timeout: 5000, encoding: "utf-8",
+          env: { ...process.env, NODE_PATH: path.join(cwd, "node_modules") },
+        });
+        if (out.includes("MODULE_OK")) {
+          console.log(chalk.green(`  ✅ Module loads OK: ${relPath}`));
+        }
+      } catch (e) {
+        // Module failed to load — but this might be because of external deps (pg, redis)
+        // Don't fail the verification for this — just log it
+        const errMsg = (e.stderr || e.message || "").slice(0, 100);
+        if (/Cannot find module/.test(errMsg) && !/\.\//.test(errMsg)) {
+          console.log(chalk.gray(`  ⚠️  Module test skipped (external dep: ${errMsg.slice(0, 60)})`));
+        } else {
+          console.log(chalk.yellow(`  ⚠️  Module load warning: ${errMsg.slice(0, 80)}`));
+        }
+      }
+    }
   }
 
   return { verified: true, status: "fixed" };