@vibecheckai/cli 3.5.1 → 3.5.2

package/bin/runners/runMcp.js

@@ -6,14 +6,17 @@
  */
 
 const path = require("path");
+const fs = require("fs");
 const http = require("http");
 const { URL } = require("url");
 
 // Import Vibecheck modules
-const { buildTruthpack, writeTruthpack } = require("./lib/truth");
+const { buildTruthpackSmart, writeTruthpack } = require("./lib/truth");
 const { shipCore } = require("./runShip");
 const { generateRunId } = require("./lib/cli-output");
 const { enforceLimit, enforceFeature, trackUsage } = require("./lib/entitlements");
+const { EXIT } = require("./lib/exit-codes");
+const { parseGlobalFlags, shouldSuppressOutput, isJsonMode } = require("./lib/global-flags");
 
 // MCP Server class
 class VibecheckMCPServer {
@@ -236,7 +239,7 @@ class VibecheckMCPServer {
     const source = args.source || "both";
     
     // Get truthpack
-    const truthpack = await buildTruthpack({ repoRoot: projectPath });
+    const truthpack = await buildTruthpackSmart({ repoRoot: projectPath });
     
     const result = {};
     
@@ -258,7 +261,7 @@ class VibecheckMCPServer {
     let truthpack;
     
     if (args.refresh) {
-      truthpack = await buildTruthpack({ repoRoot: projectPath });
+      truthpack = await buildTruthpackSmart({ repoRoot: projectPath });
       writeTruthpack(projectPath, truthpack);
     } else {
       // Try to read existing truthpack
@@ -266,7 +269,7 @@ class VibecheckMCPServer {
       if (require("fs").existsSync(truthpackPath)) {
         truthpack = JSON.parse(require("fs").readFileSync(truthpackPath, "utf8"));
       } else {
-        truthpack = await buildTruthpack({ repoRoot: projectPath });
+        truthpack = await buildTruthpackSmart({ repoRoot: projectPath });
         writeTruthpack(projectPath, truthpack);
       }
     }
@@ -642,8 +645,14 @@ class VibecheckMCPServer {
     });
     
     this.server.on("error", err => {
-      console.error("MCP Server failed to start:", err);
-      process.exit(1);
+      console.error("MCP Server failed to start:", err.message);
+      if (err.code === "EADDRINUSE") {
+        console.error(`  Port ${this.port} is already in use. Try a different port with --port`);
+      } else if (err.code === "EACCES") {
+        console.error(`  Permission denied for port ${this.port}. Use a port > 1024`);
+      }
+      // Emit error event for caller to handle
+      this.emit?.("error", err);
     });
   }
 
@@ -658,54 +667,70 @@ class VibecheckMCPServer {
 // CLI handler
 async function runMcp(args) {
   const opts = parseArgs(args);
+  const { flags: globalFlags } = parseGlobalFlags(args);
+  const quiet = shouldSuppressOutput(globalFlags);
+  const json = isJsonMode(globalFlags);
   
-  // Check if we're in free tier and only showing help/config
-  const isFreeTier = process.env.VIBECHECK_TIER === "free" || !process.env.VIBECHECK_API_KEY;
+  // MCP Server access: FREE tier gets observe mode, PRO tier gets full features
+  // Help/config/status/test commands are always available
   const isHelpOrConfig = opts.help || opts.printConfig || opts.status || opts.test;
   
-  if (isFreeTier && !isHelpOrConfig) {
-    // This will be handled by entitlements system
-    // But we can show a helpful message
-    console.log("\n🔌 MCP Server requires STARTER plan or higher");
-    console.log("Use --help to see available options or");
-    console.log("Upgrade: https://vibecheckai.dev/pricing\n");
-    return 3; // EXIT_FEATURE_NOT_ALLOWED
-  }
+  // MCP Server is available to all tiers - tier enforcement happens at tool level
+  // FREE users can run the server, but PRO tools will be gated
+  // This is handled by mcp-server/tier-auth.js
   
   if (opts.help) {
     printHelp();
-    return 0;
+    return EXIT.SUCCESS;
   }
   
   if (opts.printConfig) {
     printClientConfig(opts);
-    return 0;
+    return EXIT.SUCCESS;
   }
   
   if (opts.status) {
     // Check server status
     try {
-      const response = await fetch(`http://${opts.host}:${opts.port}/status`);
+      const host = opts.host || "127.0.0.1";
+      const port = opts.port || 3000;
+      const response = await fetch(`http://${host}:${port}/status`);
       const status = await response.json();
       console.log(JSON.stringify(status, null, 2));
-      return 0;
+      return EXIT.SUCCESS;
     } catch (err) {
-      console.error("Server not running or not reachable");
-      return 1;
+      if (json) {
+        console.log(JSON.stringify({ success: false, error: "Server not running or not reachable" }));
+      } else {
+        console.error("Server not running or not reachable");
+        console.error("  Start the server with: vibecheck mcp");
+      }
+      return EXIT.NETWORK_ERROR;
     }
   }
   
   if (opts.test) {
     // Test connection
     try {
-      const response = await fetch(`http://${opts.host}:${opts.port}/tools`);
+      const host = opts.host || "127.0.0.1";
+      const port = opts.port || 3000;
+      const response = await fetch(`http://${host}:${port}/tools`);
       const tools = await response.json();
-      console.log("✅ Connection successful");
-      console.log(`📋 Available tools: ${tools.tools.length}`);
-      return 0;
+      if (json) {
+        console.log(JSON.stringify({ success: true, toolCount: tools.tools?.length || 0 }));
+      } else {
+        console.log("✅ Connection successful");
+        console.log(`📋 Available tools: ${tools.tools?.length || 0}`);
+      }
+      return EXIT.SUCCESS;
     } catch (err) {
-      console.error("❌ Connection failed:", err.message);
-      return 1;
+      if (json) {
+        console.log(JSON.stringify({ success: false, error: err.message }));
+      } else {
+        console.error("❌ Connection failed:", err.message);
+        console.error("  Ensure the MCP server is running");
+      }
+      return EXIT.NETWORK_ERROR;
     }
   }
   
@@ -713,42 +738,88 @@ async function runMcp(args) {
   let config = {};
   if (opts.config) {
     const configPath = path.resolve(opts.config);
-    if (require("fs").existsSync(configPath)) {
-      config = JSON.parse(require("fs").readFileSync(configPath, "utf8"));
-    } else {
-      console.error(`Config file not found: ${configPath}`);
-      return 1;
+    if (!fs.existsSync(configPath)) {
+      if (json) {
+        console.log(JSON.stringify({ success: false, error: `Config file not found: ${configPath}` }));
+      } else {
+        console.error(`Config file not found: ${configPath}`);
+      }
+      return EXIT.NOT_FOUND;
+    }
+    try {
+      config = JSON.parse(fs.readFileSync(configPath, "utf8"));
+    } catch (parseErr) {
+      if (json) {
+        console.log(JSON.stringify({ success: false, error: `Invalid JSON in config: ${parseErr.message}` }));
+      } else {
+        console.error(`Invalid JSON in config file: ${parseErr.message}`);
+      }
+      return EXIT.USER_ERROR;
     }
   }
   
-  // Create and start server
-  const server = new VibecheckMCPServer({
-    host: opts.host || config.server?.host || "127.0.0.1",
-    port: opts.port || config.server?.port || 3000,
-    allowRemote: opts.allowRemote || config.server?.allowRemote || false,
-    requireApiKey: opts.requireApiKey !== false,
-    apiKey: opts.apiKey || config.auth?.apiKey,
-    logLevel: opts.logLevel || config.logging?.level || "info",
-    auditLog: opts.auditLog || config.logging?.auditFile
-  });
-  
-  server.start();
-  
-  // Handle shutdown
-  process.on("SIGINT", () => {
-    console.log("\nShutting down MCP server...");
-    server.stop();
-    process.exit(0);
-  });
-  
-  process.on("SIGTERM", () => {
-    console.log("\nShutting down MCP server...");
-    server.stop();
-    process.exit(0);
-  });
+  // Validate port
+  const port = opts.port || config.server?.port || 3000;
+  if (isNaN(port) || port < 1 || port > 65535) {
+    if (json) {
+      console.log(JSON.stringify({ success: false, error: `Invalid port: ${port}` }));
+    } else {
+      console.error(`Invalid port: ${port}`);
+      console.error("  Port must be a number between 1 and 65535");
+    }
+    return EXIT.USER_ERROR;
+  }
   
-  // Keep process alive
-  return new Promise(() => {});
+  try {
+    // Create and start server
+    const server = new VibecheckMCPServer({
+      host: opts.host || config.server?.host || "127.0.0.1",
+      port: port,
+      allowRemote: opts.allowRemote || config.server?.allowRemote || false,
+      requireApiKey: opts.requireApiKey !== false,
+      apiKey: opts.apiKey || config.auth?.apiKey,
+      logLevel: opts.logLevel || config.logging?.level || "info",
+      auditLog: opts.auditLog || config.logging?.auditFile
+    });
+    
+    // Track if server started successfully
+    let serverStarted = false;
+    let serverError = null;
+    
+    // Listen for server errors before starting
+    server.server?.on?.("error", (err) => {
+      serverError = err;
+    });
+    
+    server.start();
+    serverStarted = true;
+    
+    // Handle shutdown gracefully
+    const cleanup = () => {
+      if (!quiet) console.log("\nShutting down MCP server...");
+      server.stop();
+    };
+    
+    process.on("SIGINT", () => {
+      cleanup();
+      process.exit(EXIT.SUCCESS);
+    });
+    
+    process.on("SIGTERM", () => {
+      cleanup();
+      process.exit(EXIT.SUCCESS);
+    });
+    
+    // Keep process alive
+    return new Promise(() => {});
+  } catch (error) {
+    if (json) {
+      console.log(JSON.stringify({ success: false, error: error.message }));
+    } else {
+      console.error(`Failed to start MCP server: ${error.message}`);
+    }
+    return EXIT.INTERNAL_ERROR;
+  }
 }
 
 // Argument parsing

package/bin/runners/runPolish.d.ts

@@ -0,0 +1,4 @@
+/**
+ * Type declarations for runPolish.js
+ */
+export function runPolish(args: string[]): Promise<number>;

package/bin/runners/runPolish.js

@@ -24,6 +24,8 @@
 
 const fs = require("fs");
 const path = require("path");
+const { EXIT } = require("./lib/exit-codes");
+const { parseGlobalFlags, shouldSuppressOutput, isJsonMode } = require("./lib/global-flags");
 
 // ═══════════════════════════════════════════════════════════════════════════════
 // TERMINAL STYLING
@@ -2856,26 +2858,35 @@ function getCategoryIcon(category) {
 
 async function runPolish(args) {
   const opts = parseArgs(args);
+  const { flags: globalFlags } = parseGlobalFlags(args);
+  const quiet = shouldSuppressOutput(globalFlags);
+  const json = isJsonMode(globalFlags) || opts.json;
   
   if (opts.help) {
     printHelp();
-    return 0;
+    return EXIT.SUCCESS;
   }
   
   const projectPath = path.resolve(opts.path);
   
   // Verify project exists
   if (!await pathExists(projectPath)) {
-    console.error(`${c.red}${icons.cross} Project path does not exist: ${projectPath}${c.reset}`);
-    return 1;
+    if (json) {
+      console.log(JSON.stringify({ success: false, error: `Project path does not exist: ${projectPath}` }));
+    } else {
+      console.error(`${c.red}${icons.cross} Project path does not exist: ${projectPath}${c.reset}`);
+      console.error(`  Verify the path and try again.`);
+    }
+    return EXIT.NOT_FOUND;
   }
   
-  // JSON mode
-  if (opts.json) {
-    const report = await analyzeProject(projectPath, opts);
-    console.log(JSON.stringify(report, null, 2));
-    return report.critical > 0 ? 1 : 0;
-  }
+  try {
+    // JSON mode
+    if (json) {
+      const report = await analyzeProject(projectPath, opts);
+      console.log(JSON.stringify(report, null, 2));
+      return report.critical > 0 ? EXIT.BLOCKING : EXIT.SUCCESS;
+    }
   
   // Interactive mode
   console.log(`
@@ -2914,8 +2925,10 @@ ${c.bold}╔══════════════════════
   console.log(`            ${icons.critical} ${report.critical} critical  ${icons.high} ${report.high} high  ${icons.medium} ${report.medium} medium  ${icons.low} ${report.low} low\n`);
   
   if (report.issues.length === 0) {
-    console.log(`\n${c.green}${c.bold}${icons.check} Perfect!${c.reset} No polish issues found. Your project is production-ready! ${icons.rocket}\n`);
-    return 0;
+    if (!quiet) {
+      console.log(`\n${c.green}${c.bold}${icons.check} Perfect!${c.reset} No polish issues found. Your project is production-ready! ${icons.rocket}\n`);
+    }
+    return EXIT.SUCCESS;
   }
   
   // Group by category
@@ -3016,16 +3029,26 @@ ${c.bold}╔══════════════════════
   }
   
   // Next steps
-  console.log(`${c.bold}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${c.reset}`);
-  console.log(`${c.bold}${icons.rocket} NEXT STEPS${c.reset}`);
-  console.log(`${c.bold}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${c.reset}\n`);
-  console.log(`   1. Fix ${c.red}critical${c.reset} and ${c.yellow}high${c.reset} severity issues first`);
-  console.log(`   2. Use ${c.cyan}vibecheck polish --prompts${c.reset} to get AI-ready fixes`);
-  console.log(`   3. Copy prompts to your AI assistant (Cursor, Copilot, Claude)`);
-  console.log(`   4. Re-run ${c.cyan}vibecheck polish${c.reset} to verify fixes`);
-  console.log(`   5. Run ${c.cyan}vibecheck ship${c.reset} when score is 80+\n`);
+  if (!quiet) {
+    console.log(`${c.bold}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${c.reset}`);
+    console.log(`${c.bold}${icons.rocket} NEXT STEPS${c.reset}`);
+    console.log(`${c.bold}━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━${c.reset}\n`);
+    console.log(`   1. Fix ${c.red}critical${c.reset} and ${c.yellow}high${c.reset} severity issues first`);
+    console.log(`   2. Use ${c.cyan}vibecheck polish --prompts${c.reset} to get AI-ready fixes`);
+    console.log(`   3. Copy prompts to your AI assistant (Cursor, Copilot, Claude)`);
+    console.log(`   4. Re-run ${c.cyan}vibecheck polish${c.reset} to verify fixes`);
+    console.log(`   5. Run ${c.cyan}vibecheck ship${c.reset} when score is 80+\n`);
+  }
   
-  return report.critical > 0 ? 1 : 0;
+  return report.critical > 0 ? EXIT.BLOCKING : EXIT.SUCCESS;
+  } catch (error) {
+    if (json) {
+      console.log(JSON.stringify({ success: false, error: error.message }));
+    } else {
+      console.error(`${c.red}${icons.cross} Polish analysis failed: ${error.message}${c.reset}`);
+    }
+    return EXIT.INTERNAL_ERROR;
+  }
 }
 
 module.exports = { runPolish };

package/bin/runners/runProve.js

@@ -19,18 +19,18 @@
 
 const fs = require("fs");
 const path = require("path");
-const { buildTruthpack, writeTruthpack, detectFastifyEntry } = require("./lib/truth");
+const { buildTruthpackSmart, writeTruthpack, detectFastifyEntry } = require("./lib/truth");
 const { shipCore } = require("./runShip");
 const { findContractDrift, loadContracts, hasContracts, getDriftSummary } = require("./lib/drift");
 const { 
   generateRunId, 
   createJsonOutput, 
   writeJsonOutput, 
-  exitCodeToVerdict,
-  verdictToExitCode,
   saveArtifact 
 } = require("./lib/cli-output");
+const { EXIT, verdictToExitCode, exitCodeToVerdict } = require("./lib/exit-codes");
 const { parseGlobalFlags, shouldShowBanner } = require("./lib/global-flags");
+const upsell = require("./lib/upsell");
 
 let runReality;
 try {
@@ -709,6 +709,9 @@ function printHelp(showBanner = true) {
     ${colors.accent}--no-trace${c.reset}               Disable Playwright traces
     ${colors.accent}--no-har${c.reset}                 Disable HAR network capture
 
+  ${c.bold}Evidence Pack:${c.reset}
+    ${colors.accent}--bundle${c.reset}                 Generate shareable evidence pack (zip with videos, traces, screenshots)
+
   ${c.bold}Exit Codes:${c.reset}
     ${colors.shipGreen}0${c.reset}  SHIP  — Proved real, ready to deploy
     ${colors.warnAmber}1${c.reset}  WARN  — Warnings found, review recommended
@@ -797,7 +800,7 @@ async function runProve(argsOrOpts = {}, context = {}) {
       stabilityRuns: parseInt(getArg(["--stability-runs"]) || "2", 10),
       flakyThreshold: parseFloat(getArg(["--flaky-threshold"]) || "0.66"),
       maxPages: parseInt(getArg(["--max-pages"]) || "18", 10),
-      evidencePack: cleanArgs.includes("--evidence-pack"),
+      evidencePack: cleanArgs.includes("--evidence-pack") || cleanArgs.includes("--bundle"),
       output: getArg(["--output", "-o"]),
     };
   }
@@ -866,7 +869,7 @@ async function runProve(argsOrOpts = {}, context = {}) {
   
   try {
     const fastEntry = fastifyEntry || (await detectFastifyEntry(root));
-    const truthpack = await buildTruthpack({ repoRoot: root, fastifyEntry: fastEntry });
+    const truthpack = await buildTruthpackSmart({ repoRoot: root, fastifyEntry: fastEntry });
     writeTruthpack(root, truthpack);
     
     stopSpinner('Truthpack refreshed', true);
@@ -1407,6 +1410,13 @@ async function runProve(argsOrOpts = {}, context = {}) {
       console.log(`  ${colors.accent}vibecheck ship --fix${c.reset}  ${c.dim}Manual fix mode${c.reset}`);
       console.log();
     }
+    
+    // Upsell for upgrade
+    console.log(upsell.formatNextSteps("prove", finalVerdict, "free"));
+    console.log();
+    console.log(`  ${c.dim}${upsell.sym.star} Upgrade for unlimited prove runs + video artifacts${c.reset}`);
+    console.log(`  ${c.dim}${upsell.sym.arrow} ${upsell.PRICING_URL}${c.reset}`);
+    console.log();
   } else if (ci) {
     // CI mode - structured output for easy parsing
     console.log(`::group::vibecheck prove summary`);