safelaunch 1.0.18 → 1.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "safelaunch",
-  "version": "1.0.18",
+  "version": "1.0.19",
   "description": "Validate your environment before every push. Catch missing, empty, and misconfigured env variables before they break production.",
   "main": "index.js",
   "bin": {

package/safelaunch/src/scan.js

@@ -1,7 +1,12 @@
 const fs = require('fs');
 const path = require('path');
+const { track, shutdown } = require('./telemetry');
 
-// ─── Impact messages per check type ───────────────────────────────────────────
+// ─── Impact messages ──────────────────────────────────────────────────────────
+
+function impactBlock(lines) {
+  return '   Impact:\n' + lines.map(l => '   ' + l).join('\n');
+}
 
 const IMPACT = {
   missing: (key) => impactBlock([
@@ -38,12 +43,6 @@ const IMPACT = {
   ]),
 };
 
-function impactBlock(lines) {
-  return '   Impact:\n' + lines.map(l => '   ' + l).join('\n');
-}
-
-// ─── Divider ──────────────────────────────────────────────────────────────────
-
 const DIVIDER = '────────────────────────────';
 
 // ─── Helpers ──────────────────────────────────────────────────────────────────
@@ -119,39 +118,26 @@ function checkDependencies(cwd) {
   return { notInstalled: false, missing };
 }
 
-// ─── Render helpers ───────────────────────────────────────────────────────────
-
-function renderIssue(number, title, impactLines) {
+function renderIssue(number, title, impact) {
   return (
     number + '. ' + title + '\n' +
     '\n' +
-    impactLines + '\n'
+    impact + '\n'
   );
 }
 
 // ─── Main ─────────────────────────────────────────────────────────────────────
 
 async function scan() {
-  // Lazy-load telemetry so it doesn't affect output
-  let track = async () => {};
-  let shutdown = async () => {};
-  try {
-    const telemetry = require('./safelaunch/src/telemetry');
-    track = telemetry.track;
-    shutdown = telemetry.shutdown;
-  } catch (_) {}
+  console.log('');
+  console.log('Scanning your project...');
 
   const cwd = process.cwd();
   const projectType = detectProjectType(cwd);
   const typeLabels = { vite: 'Vite', next: 'Next.js', cra: 'Create React App', node: 'Node.js' };
-
-  console.log('');
-  console.log('Scanning your project...');
   console.log('Detected: ' + typeLabels[projectType]);
   console.log('');
 
-  // ── Collect env vars used in codebase ──
-
   const extensions = ['.js', '.ts', '.jsx', '.tsx', '.vue', '.svelte'];
   let pattern;
   if (projectType === 'vite') {
@@ -164,8 +150,6 @@ async function scan() {
 
   const found = scanFiles(cwd, extensions, pattern);
 
-  // ── No env vars in codebase at all ──
-
   if (found.size === 0) {
     console.log('🛡️  Safelaunch Scan Report');
     console.log('');
@@ -182,8 +166,6 @@ async function scan() {
     return;
   }
 
-  // ── No .env file found ──
-
   const result = readEnvDetailed(cwd);
 
   if (!result) {
@@ -191,12 +173,7 @@ async function scan() {
     const criticals = [];
     let issueNum = 1;
 
-    criticals.push(renderIssue(
-      issueNum++,
-      '.env file is missing',
-      IMPACT.noEnvFile()
-    ));
-
+    criticals.push(renderIssue(issueNum++, '.env file is missing', IMPACT.noEnvFile()));
     for (const key of varList) {
       criticals.push(renderIssue(issueNum++, key + ' is missing', IMPACT.missing(key)));
     }
@@ -233,8 +210,6 @@ async function scan() {
     return;
   }
 
-  // ── Full scan ──
-
   const { envVars, duplicates } = result;
   const missing = [];
   const empty = [];
@@ -252,7 +227,6 @@ async function scan() {
 
   const drift = checkDependencies(cwd);
 
-  // Prefix issues (warnings only)
   const prefixWarnings = [];
   for (const key of found) {
     if (key === 'NODE_ENV') continue;
@@ -264,40 +238,27 @@ async function scan() {
     }
   }
 
-  // ── Build issue lists ──
-
-  const criticalIssues = [];   // { title, impact }
-  const warningIssues = [];    // { title, impact }
-  const passingChecks = [];    // string[]
+  const criticalIssues = [];
+  const warningIssues = [];
+  const passingChecks = [];
 
-  // Criticals: missing vars
   for (const key of missing) {
     criticalIssues.push({ title: key + ' is missing', impact: IMPACT.missing(key) });
   }
-
-  // Criticals: empty vars
   for (const key of empty) {
     criticalIssues.push({ title: key + ' is empty', impact: IMPACT.empty(key) });
   }
-
-  // Criticals: deps not installed at all
   if (drift && drift.notInstalled) {
     criticalIssues.push({ title: 'Dependencies are not installed', impact: IMPACT.depsNotInstalled() });
   }
-
-  // Warnings: dependency drift
   if (drift && !drift.notInstalled && drift.missing.length > 0) {
     for (const dep of drift.missing) {
       warningIssues.push({ title: dep + ' is not installed', impact: IMPACT.depsDrift(dep) });
     }
   }
-
-  // Warnings: duplicate vars
   for (const key of duplicates) {
     warningIssues.push({ title: key + ' is defined more than once', impact: IMPACT.duplicate(key) });
   }
-
-  // Warnings: prefix issues
   for (const { key, type } of prefixWarnings) {
     if (type === 'vite') {
       warningIssues.push({ title: key + ' is missing VITE_ prefix', impact: IMPACT.prefixVite(key) });
@@ -306,7 +267,6 @@ async function scan() {
     }
   }
 
-  // Passing checks
   if (result) passingChecks.push('Environment file detected');
   if (!drift || (!drift.notInstalled && drift.missing.length === 0)) passingChecks.push('Dependencies installed');
   if (duplicates.length === 0) passingChecks.push('No duplicate variables');
@@ -317,23 +277,17 @@ async function scan() {
   const totalIssues = criticalIssues.length + warningIssues.length;
   const hasFailed = totalIssues > 0;
 
-  // ─── Render ───────────────────────────────────────────────────────────────
-
   if (hasFailed) {
-    const critCount = criticalIssues.length;
-    const warnCount = warningIssues.length;
-
-    let summary = totalIssues + ' issue' + (totalIssues !== 1 ? 's' : '') + ' found';
     const parts = [];
-    if (critCount > 0) parts.push(critCount + ' critical');
-    if (warnCount > 0) parts.push(warnCount + ' warning' + (warnCount !== 1 ? 's' : ''));
-    summary += '\n' + parts.join(' · ');
+    if (criticalIssues.length > 0) parts.push(criticalIssues.length + ' critical');
+    if (warningIssues.length > 0) parts.push(warningIssues.length + ' warning' + (warningIssues.length !== 1 ? 's' : ''));
 
     console.log('🚨 Safelaunch Scan Report');
     console.log('');
     console.log('This project is NOT safe to deploy');
     console.log('');
-    console.log(summary);
+    console.log(totalIssues + ' issue' + (totalIssues !== 1 ? 's' : '') + ' found');
+    console.log(parts.join(' · '));
     console.log('');
     console.log(DIVIDER);
     console.log('');
@@ -370,7 +324,7 @@ async function scan() {
     }
 
     if (passingChecks.length > 0) {
-      console.log('✅ What\'s working (' + passingChecks.length + ' checks passed)');
+      console.log("✅ What's working (" + passingChecks.length + ' checks passed)');
       console.log('');
       for (const check of passingChecks) {
         console.log('- ' + check);
@@ -389,8 +343,6 @@ async function scan() {
     console.log('');
 
   } else {
-    // ── All clean ──
-
     console.log('🛡️  Safelaunch Scan Report');
     console.log('');
     console.log('Your project is safe to deploy');
@@ -407,7 +359,7 @@ async function scan() {
     console.log('');
     console.log(DIVIDER);
     console.log('');
-    console.log('🎉 You\'re good to go');
+    console.log("🎉 You're good to go");
     console.log('');
     console.log('   Deploy with confidence');
     console.log('');