ship-safe 5.0.0 → 6.0.0

package/cli/commands/score.js

@@ -36,6 +36,7 @@ import {
   SECURITY_PATTERNS,
   SKIP_DIRS,
   SKIP_EXTENSIONS,
+  SKIP_FILENAMES,
   TEST_FILE_PATTERNS,
   MAX_FILE_SIZE
 } from '../utils/patterns.js';
@@ -335,6 +336,7 @@ async function findFiles(rootPath) {
   for (const file of files) {
     const ext = path.extname(file).toLowerCase();
     if (SKIP_EXTENSIONS.has(ext)) continue;
+    if (SKIP_FILENAMES.has(path.basename(file))) continue;
 
     const basename = path.basename(file);
     if (basename.endsWith('.min.js') || basename.endsWith('.min.css')) continue;

package/cli/commands/vibe-check.js

@@ -0,0 +1,276 @@
+/**
+ * Vibe Check Command
+ * ==================
+ *
+ * Fun, emoji-rich security check with shareable results.
+ * Same security scan as `audit`, but with personality.
+ *
+ * USAGE:
+ *   npx ship-safe vibe-check [path]     Run a vibe check
+ *   npx ship-safe vibe-check . --badge  Generate a markdown badge
+ *
+ * OUTPUT:
+ *   Big ASCII art grade, emoji severity indicators,
+ *   "vibes" rating, and a shareable one-liner.
+ */
+
+import fs from 'fs';
+import path from 'path';
+import chalk from 'chalk';
+import ora from 'ora';
+import { buildOrchestrator } from '../agents/index.js';
+import { ScoringEngine } from '../agents/scoring-engine.js';
+import { runDepsAudit } from './deps.js';
+import {
+  SECRET_PATTERNS,
+  SKIP_DIRS,
+  SKIP_EXTENSIONS,
+  SKIP_FILENAMES,
+  MAX_FILE_SIZE,
+  loadGitignorePatterns
+} from '../utils/patterns.js';
+import { isHighEntropyMatch, getConfidence } from '../utils/entropy.js';
+import fg from 'fast-glob';
+
+// =============================================================================
+// VIBES DATA
+// =============================================================================
+
+const VIBE_GRADES = {
+  A: {
+    emoji: '🛡️',
+    vibe: 'immaculate',
+    ascii: `
+    ╔═══╗
+    ║ A ║
+    ╚═══╝`,
+    message: 'Your security vibes are IMMACULATE. Ship it! 🚀',
+    color: chalk.green.bold,
+  },
+  B: {
+    emoji: '✅',
+    vibe: 'solid',
+    ascii: `
+    ╔═══╗
+    ║ B ║
+    ╚═══╝`,
+    message: 'Solid vibes. A few things to tighten up, but you\'re in good shape. 💪',
+    color: chalk.cyan.bold,
+  },
+  C: {
+    emoji: '⚠️',
+    vibe: 'mid',
+    ascii: `
+    ╔═══╗
+    ║ C ║
+    ╚═══╝`,
+    message: 'Mid vibes. Some security gaps need attention before you ship. 🔧',
+    color: chalk.yellow.bold,
+  },
+  D: {
+    emoji: '🚨',
+    vibe: 'sketchy',
+    ascii: `
+    ╔═══╗
+    ║ D ║
+    ╚═══╝`,
+    message: 'Sketchy vibes. Serious issues found — fix these before deploying. 🛑',
+    color: chalk.red.bold,
+  },
+  F: {
+    emoji: '💀',
+    vibe: 'cooked',
+    ascii: `
+    ╔═══╗
+    ║ F ║
+    ╚═══╝`,
+    message: 'You are cooked. Critical vulnerabilities everywhere. DO NOT SHIP. 🔥',
+    color: chalk.red.bold,
+  },
+};
+
+const SEV_EMOJI = {
+  critical: '💀',
+  high: '🔴',
+  medium: '🟡',
+  low: '🔵',
+};
+
+// =============================================================================
+// MAIN COMMAND
+// =============================================================================
+
+export async function vibeCheckCommand(targetPath = '.', options = {}) {
+  const absolutePath = path.resolve(targetPath);
+
+  if (!fs.existsSync(absolutePath)) {
+    console.error(chalk.red(`Path does not exist: ${absolutePath}`));
+    process.exit(1);
+  }
+
+  const projectName = path.basename(absolutePath);
+
+  console.log();
+  console.log(chalk.cyan.bold('  🎵 VIBE CHECK 🎵'));
+  console.log(chalk.gray(`  Scanning ${projectName}...`));
+  console.log();
+
+  const startTime = Date.now();
+
+  // ── Secret Scan ──────────────────────────────────────────────────────────
+  const spinner = ora({ text: 'Checking the vibes...', color: 'magenta' }).start();
+
+  const allFiles = await findFiles(absolutePath);
+  const secretFindings = [];
+
+  for (const file of allFiles) {
+    try {
+      const content = fs.readFileSync(file, 'utf-8');
+      const lines = content.split('\n');
+      for (let lineNum = 0; lineNum < lines.length; lineNum++) {
+        const line = lines[lineNum];
+        if (/ship-safe-ignore/i.test(line)) continue;
+        for (const pattern of SECRET_PATTERNS) {
+          pattern.pattern.lastIndex = 0;
+          let match;
+          while ((match = pattern.pattern.exec(line)) !== null) {
+            if (pattern.requiresEntropyCheck && !isHighEntropyMatch(match[0])) continue;
+            secretFindings.push({
+              file, line: lineNum + 1, column: match.index + 1,
+              matched: match[0], severity: pattern.severity,
+              category: pattern.category || 'secrets',
+              rule: pattern.name, title: pattern.name.replace(/_/g, ' '),
+              description: pattern.description,
+              confidence: getConfidence(pattern, match[0]),
+            });
+          }
+        }
+      }
+    } catch { /* skip */ }
+  }
+
+  // ── Agent Scan ──────────────────────────────────────────────────────────
+  const orchestrator = buildOrchestrator();
+  const results = await orchestrator.runAll(absolutePath, { quiet: true });
+
+  // ── Dependency Audit ─────────────────────────────────────────────────────
+  let depVulns = [];
+  try {
+    const depResult = await runDepsAudit(absolutePath);
+    depVulns = depResult.vulns || [];
+  } catch { /* skip */ }
+
+  spinner.stop();
+
+  // ── Merge & Score ─────────────────────────────────────────────────────────
+  const seen = new Set();
+  const allFindings = [...secretFindings, ...results.findings].filter(f => {
+    const key = `${f.file}:${f.line}:${f.rule}`;
+    if (seen.has(key)) return false;
+    seen.add(key);
+    return true;
+  });
+
+  const scoringEngine = new ScoringEngine();
+  const scoreResult = scoringEngine.compute(allFindings, depVulns);
+  const duration = ((Date.now() - startTime) / 1000).toFixed(1);
+
+  // ── Display ──────────────────────────────────────────────────────────────
+  const grade = VIBE_GRADES[scoreResult.grade.letter] || VIBE_GRADES.F;
+  const score = Math.round(scoreResult.score * 10) / 10;
+
+  const critical = allFindings.filter(f => f.severity === 'critical').length;
+  const high = allFindings.filter(f => f.severity === 'high').length;
+  const medium = allFindings.filter(f => f.severity === 'medium').length;
+  const low = allFindings.filter(f => f.severity === 'low').length;
+
+  // Big grade display
+  console.log(grade.color(grade.ascii));
+  console.log();
+  console.log(grade.color(`  ${grade.emoji}  Score: ${score}/100  |  Vibes: ${grade.vibe.toUpperCase()}`));
+  console.log();
+  console.log(grade.color(`  ${grade.message}`));
+  console.log();
+
+  // Severity breakdown
+  console.log(chalk.white.bold('  Breakdown:'));
+  if (critical > 0) console.log(`    ${SEV_EMOJI.critical} Critical: ${critical}`);
+  if (high > 0) console.log(`    ${SEV_EMOJI.high} High: ${high}`);
+  if (medium > 0) console.log(`    ${SEV_EMOJI.medium} Medium: ${medium}`);
+  if (low > 0) console.log(`    ${SEV_EMOJI.low} Low: ${low}`);
+  if (depVulns.length > 0) console.log(`    📦 Dep CVEs: ${depVulns.length}`);
+  if (allFindings.length === 0 && depVulns.length === 0) {
+    console.log(`    ✨ Zero issues found!`);
+  }
+  console.log(chalk.gray(`    ⏱️  ${duration}s`));
+  console.log();
+
+  // Top 3 issues
+  if (allFindings.length > 0) {
+    console.log(chalk.white.bold('  Top issues to fix:'));
+    const top = allFindings
+      .sort((a, b) => {
+        const order = { critical: 0, high: 1, medium: 2, low: 3 };
+        return (order[a.severity] ?? 4) - (order[b.severity] ?? 4);
+      })
+      .slice(0, 3);
+    for (const f of top) {
+      const rel = path.relative(absolutePath, f.file).replace(/\\/g, '/');
+      console.log(`    ${SEV_EMOJI[f.severity] || '⚪'} ${f.title || f.rule} ${chalk.gray(`(${rel}:${f.line})`)}`);
+    }
+    console.log();
+  }
+
+  // ── Shareable one-liner ──────────────────────────────────────────────────
+  const shareLine = `${grade.emoji} ${projectName}: ${score}/100 (${scoreResult.grade.letter}) — ${grade.vibe} vibes | ${allFindings.length} findings | Scanned with Ship Safe`;
+  console.log(chalk.gray('  Share your vibes:'));
+  console.log(chalk.cyan(`  ${shareLine}`));
+  console.log();
+
+  // ── Badge ─────────────────────────────────────────────────────────────────
+  if (options.badge) {
+    const badgeColor = {
+      A: 'brightgreen', B: 'blue', C: 'yellow', D: 'orange', F: 'red',
+    }[scoreResult.grade.letter] || 'lightgrey';
+    const badgeUrl = `https://img.shields.io/badge/ship--safe-${score}%2F100_${scoreResult.grade.letter}-${badgeColor}`;
+    const badgeMd = `[![Ship Safe Score](${badgeUrl})](https://shipsafecli.com)`;
+
+    console.log(chalk.white.bold('  Markdown badge:'));
+    console.log(chalk.cyan(`  ${badgeMd}`));
+    console.log();
+
+    // Write badge to README if it exists and doesn't have one already
+    const readmePath = path.join(absolutePath, 'README.md');
+    if (fs.existsSync(readmePath)) {
+      const readme = fs.readFileSync(readmePath, 'utf-8');
+      if (!readme.includes('ship--safe')) {
+        console.log(chalk.gray('  Add this badge to your README.md to show off your security score!'));
+      }
+    }
+  }
+
+  process.exit(allFindings.length > 0 || depVulns.length > 0 ? 1 : 0);
+}
+
+// =============================================================================
+// FILE FINDER (reused from CI)
+// =============================================================================
+
+async function findFiles(rootPath) {
+  const globIgnore = Array.from(SKIP_DIRS).map(dir => `**/${dir}/**`);
+  const gitignoreGlobs = loadGitignorePatterns(rootPath);
+  globIgnore.push(...gitignoreGlobs);
+
+  const files = await fg('**/*', {
+    cwd: rootPath, absolute: true, onlyFiles: true, ignore: globIgnore, dot: true,
+  });
+
+  return files.filter(file => {
+    const ext = path.extname(file).toLowerCase();
+    if (SKIP_EXTENSIONS.has(ext)) return false;
+    if (SKIP_FILENAMES.has(path.basename(file))) return false;
+    if (path.basename(file).endsWith('.min.js') || path.basename(file).endsWith('.min.css')) return false;
+    try { if (fs.statSync(file).size > MAX_FILE_SIZE) return false; } catch { return false; }
+    return true;
+  });
+}

package/cli/commands/watch.js

@@ -1,160 +1,161 @@
-/**
- * Watch Command
- * ==============
- *
- * Continuous file monitoring mode. Watches for file changes
- * and incrementally scans modified files.
- *
- * USAGE:
- *   npx ship-safe watch [path]     Start watching for changes
- *   npx ship-safe watch . --poll   Use polling (for network drives)
- */
-
-import fs from 'fs';
-import path from 'path';
-import chalk from 'chalk';
-import { SKIP_DIRS, SKIP_EXTENSIONS, SECRET_PATTERNS, SECURITY_PATTERNS } from '../utils/patterns.js';
-import { isHighEntropyMatch, getConfidence } from '../utils/entropy.js';
-import * as output from '../utils/output.js';
-
-export async function watchCommand(targetPath = '.', options = {}) {
-  const absolutePath = path.resolve(targetPath);
-
-  if (!fs.existsSync(absolutePath)) {
-    output.error(`Path does not exist: ${absolutePath}`);
-    process.exit(1);
-  }
-
-  console.log();
-  output.header('Ship Safe — Watch Mode');
-  console.log();
-  console.log(chalk.cyan('  Watching for file changes...'));
-  console.log(chalk.gray('  Press Ctrl+C to stop'));
-  console.log();
-
-  const allPatterns = [...SECRET_PATTERNS, ...SECURITY_PATTERNS];
-  const skipDirSet = SKIP_DIRS;
-  let debounceTimer = null;
-  const pendingFiles = new Set();
-
-  // Use fs.watch recursively
-  try {
-    const watcher = fs.watch(absolutePath, { recursive: true }, (eventType, filename) => {
-      if (!filename) return;
-
-      const fullPath = path.join(absolutePath, filename); // ship-safe-ignore — filename from fs.watch, not user input
-      const relPath = filename.replace(/\\/g, '/');
-
-      // Skip directories we don't care about
-      for (const skipDir of skipDirSet) {
-        if (relPath.includes(`${skipDir}/`) || relPath.startsWith(`${skipDir}/`)) return;
-      }
-
-      // Skip non-code files
-      const ext = path.extname(filename).toLowerCase();
-      if (SKIP_EXTENSIONS.has(ext)) return;
-      if (filename.endsWith('.min.js') || filename.endsWith('.min.css')) return;
-
-      // Add to pending and debounce
-      pendingFiles.add(fullPath);
-
-      if (debounceTimer) clearTimeout(debounceTimer);
-      debounceTimer = setTimeout(() => {
-        const filesToScan = [...pendingFiles];
-        pendingFiles.clear();
-        scanChangedFiles(filesToScan, allPatterns, absolutePath);
-      }, 300);
-    });
-
-    // Keep the process alive
-    process.on('SIGINT', () => {
-      watcher.close();
-      console.log();
-      output.info('Watch mode stopped.');
-      process.exit(0);
-    });
-
-    // Prevent Node from exiting
-    setInterval(() => {}, 1000 * 60 * 60);
-
-  } catch (err) {
-    output.error(`Watch failed: ${err.message}`);
-    console.log(chalk.gray('  Try: npx ship-safe watch . --poll'));
-    process.exit(1);
-  }
-}
-
-function scanChangedFiles(files, patterns, rootPath) {
-  const timestamp = new Date().toLocaleTimeString();
-  let totalFindings = 0;
-
-  for (const filePath of files) {
-    if (!fs.existsSync(filePath)) continue;
-
-    try {
-      const stats = fs.statSync(filePath);
-      if (stats.size > 1_000_000) continue;
-    } catch {
-      continue;
-    }
-
-    const findings = scanFile(filePath, patterns);
-    if (findings.length > 0) {
-      totalFindings += findings.length;
-      const relPath = path.relative(rootPath, filePath);
-
-      for (const f of findings) {
-        const sevColor = f.severity === 'critical' ? chalk.red.bold
-          : f.severity === 'high' ? chalk.yellow
-          : chalk.blue;
-
-        console.log(
-          chalk.gray(`  [${timestamp}] `) +
-          sevColor(`[${f.severity.toUpperCase()}]`) +
-          chalk.white(` ${relPath}:${f.line} `) +
-          chalk.gray(f.patternName)
-        );
-      }
-    }
-  }
-
-  if (totalFindings === 0 && files.length > 0) {
-    console.log(chalk.gray(`  [${timestamp}] ${files.length} file(s) scanned — clean`));
-  }
-}
-
-function scanFile(filePath, patterns) {
-  const findings = [];
-  try {
-    const content = fs.readFileSync(filePath, 'utf-8');
-    const lines = content.split('\n');
-
-    for (let i = 0; i < lines.length; i++) {
-      const line = lines[i];
-      if (/ship-safe-ignore/i.test(line)) continue;
-
-      for (const pattern of patterns) {
-        pattern.pattern.lastIndex = 0;
-        let match;
-        while ((match = pattern.pattern.exec(line)) !== null) {
-          if (pattern.requiresEntropyCheck && !isHighEntropyMatch(match[0])) continue;
-          findings.push({
-            line: i + 1,
-            patternName: pattern.name,
-            severity: pattern.severity,
-            matched: match[0],
-            category: pattern.category || 'secret',
-          });
-        }
-      }
-    }
-  } catch { /* skip */ }
-
-  const seen = new Set();
-  return findings.filter(f => {
-    const key = `${f.line}:${f.matched}`;
-    if (seen.has(key)) return false;
-    seen.add(key);
-    return true;
-  });
-}
+/**
+ * Watch Command
+ * ==============
+ *
+ * Continuous file monitoring mode. Watches for file changes
+ * and incrementally scans modified files.
+ *
+ * USAGE:
+ *   npx ship-safe watch [path]     Start watching for changes
+ *   npx ship-safe watch . --poll   Use polling (for network drives)
+ */
+
+import fs from 'fs';
+import path from 'path';
+import chalk from 'chalk';
+import { SKIP_DIRS, SKIP_EXTENSIONS, SKIP_FILENAMES, SECRET_PATTERNS, SECURITY_PATTERNS } from '../utils/patterns.js';
+import { isHighEntropyMatch, getConfidence } from '../utils/entropy.js';
+import * as output from '../utils/output.js';
+
+export async function watchCommand(targetPath = '.', options = {}) {
+  const absolutePath = path.resolve(targetPath);
+
+  if (!fs.existsSync(absolutePath)) {
+    output.error(`Path does not exist: ${absolutePath}`);
+    process.exit(1);
+  }
+
+  console.log();
+  output.header('Ship Safe — Watch Mode');
+  console.log();
+  console.log(chalk.cyan('  Watching for file changes...'));
+  console.log(chalk.gray('  Press Ctrl+C to stop'));
+  console.log();
+
+  const allPatterns = [...SECRET_PATTERNS, ...SECURITY_PATTERNS];
+  const skipDirSet = SKIP_DIRS;
+  let debounceTimer = null;
+  const pendingFiles = new Set();
+
+  // Use fs.watch recursively
+  try {
+    const watcher = fs.watch(absolutePath, { recursive: true }, (eventType, filename) => { // ship-safe-ignore — filename from fs.watch OS event, not user input
+      if (!filename) return; // ship-safe-ignore
+
+      const fullPath = path.join(absolutePath, filename); // ship-safe-ignore — filename from fs.watch, not user input
+      const relPath = filename.replace(/\\/g, '/');
+
+      // Skip directories we don't care about
+      for (const skipDir of skipDirSet) {
+        if (relPath.includes(`${skipDir}/`) || relPath.startsWith(`${skipDir}/`)) return;
+      }
+
+      // Skip non-code files
+      const ext = path.extname(filename).toLowerCase(); // ship-safe-ignore — filename from fs.watch OS event
+      if (SKIP_EXTENSIONS.has(ext)) return;
+      if (SKIP_FILENAMES.has(path.basename(filename))) return; // ship-safe-ignore
+      if (filename.endsWith('.min.js') || filename.endsWith('.min.css')) return;
+
+      // Add to pending and debounce
+      pendingFiles.add(fullPath);
+
+      if (debounceTimer) clearTimeout(debounceTimer);
+      debounceTimer = setTimeout(() => {
+        const filesToScan = [...pendingFiles];
+        pendingFiles.clear();
+        scanChangedFiles(filesToScan, allPatterns, absolutePath);
+      }, 300);
+    });
+
+    // Keep the process alive
+    process.on('SIGINT', () => {
+      watcher.close();
+      console.log();
+      output.info('Watch mode stopped.');
+      process.exit(0);
+    });
+
+    // Prevent Node from exiting
+    setInterval(() => {}, 1000 * 60 * 60);
+
+  } catch (err) {
+    output.error(`Watch failed: ${err.message}`);
+    console.log(chalk.gray('  Try: npx ship-safe watch . --poll'));
+    process.exit(1);
+  }
+}
+
+function scanChangedFiles(files, patterns, rootPath) {
+  const timestamp = new Date().toLocaleTimeString();
+  let totalFindings = 0;
+
+  for (const filePath of files) {
+    if (!fs.existsSync(filePath)) continue;
+
+    try {
+      const stats = fs.statSync(filePath);
+      if (stats.size > 1_000_000) continue;
+    } catch {
+      continue;
+    }
+
+    const findings = scanFile(filePath, patterns);
+    if (findings.length > 0) {
+      totalFindings += findings.length;
+      const relPath = path.relative(rootPath, filePath);
+
+      for (const f of findings) {
+        const sevColor = f.severity === 'critical' ? chalk.red.bold
+          : f.severity === 'high' ? chalk.yellow
+          : chalk.blue;
+
+        console.log(
+          chalk.gray(`  [${timestamp}] `) +
+          sevColor(`[${f.severity.toUpperCase()}]`) +
+          chalk.white(` ${relPath}:${f.line} `) +
+          chalk.gray(f.patternName)
+        );
+      }
+    }
+  }
+
+  if (totalFindings === 0 && files.length > 0) {
+    console.log(chalk.gray(`  [${timestamp}] ${files.length} file(s) scanned — clean`));
+  }
+}
+
+function scanFile(filePath, patterns) {
+  const findings = [];
+  try {
+    const content = fs.readFileSync(filePath, 'utf-8');
+    const lines = content.split('\n');
+
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i];
+      if (/ship-safe-ignore/i.test(line)) continue;
+
+      for (const pattern of patterns) {
+        pattern.pattern.lastIndex = 0;
+        let match;
+        while ((match = pattern.pattern.exec(line)) !== null) {
+          if (pattern.requiresEntropyCheck && !isHighEntropyMatch(match[0])) continue;
+          findings.push({
+            line: i + 1,
+            patternName: pattern.name,
+            severity: pattern.severity,
+            matched: match[0],
+            category: pattern.category || 'secret',
+          });
+        }
+      }
+    }
+  } catch { /* skip */ }
+
+  const seen = new Set();
+  return findings.filter(f => {
+    const key = `${f.line}:${f.matched}`;
+    if (seen.has(key)) return false;
+    seen.add(key);
+    return true;
+  });
+}

package/cli/index.js

@@ -25,8 +25,13 @@ export { doctorCommand } from './commands/doctor.js';
 // ── v4.3 Commands ─────────────────────────────────────────────────────────────
 export { baselineCommand } from './commands/baseline.js';
 
+// ── v6.0 Commands ─────────────────────────────────────────────────────────────
+export { diffCommand } from './commands/diff.js';
+export { vibeCheckCommand } from './commands/vibe-check.js';
+export { benchmarkCommand } from './commands/benchmark.js';
+
 // ── Patterns ──────────────────────────────────────────────────────────────────
-export { SECRET_PATTERNS, SECURITY_PATTERNS, SKIP_DIRS, SKIP_EXTENSIONS } from './utils/patterns.js';
+export { SECRET_PATTERNS, SECURITY_PATTERNS, SKIP_DIRS, SKIP_EXTENSIONS, SKIP_FILENAMES } from './utils/patterns.js';
 
 // ── Agent Framework ───────────────────────────────────────────────────────────
 export { BaseAgent, createFinding } from './agents/base-agent.js';
@@ -46,6 +51,8 @@ export { GitHistoryScanner } from './agents/git-history-scanner.js';
 export { CICDScanner } from './agents/cicd-scanner.js';
 export { APIFuzzer } from './agents/api-fuzzer.js';
 export { SupabaseRLSAgent } from './agents/supabase-rls-agent.js';
+export { VibeCodingAgent } from './agents/vibe-coding-agent.js';
+export { ExceptionHandlerAgent } from './agents/exception-handler-agent.js';
 
 // ── Supporting Modules ────────────────────────────────────────────────────────
 export { ScoringEngine, GRADES, CATEGORIES } from './agents/scoring-engine.js';

package/cli/utils/cache-manager.js

@@ -23,7 +23,7 @@ import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 
 // Read version from package.json
-const __filename = fileURLToPath(import.meta.url);
+const __filename = fileURLToPath(import.meta.url); // ship-safe-ignore — module's own path via import.meta.url, not user input
 const __dirname = dirname(__filename);
 const PACKAGE_VERSION = JSON.parse(readFileSync(join(__dirname, '../../package.json'), 'utf8')).version;
 

package/cli/utils/output.js

@@ -120,10 +120,13 @@ export function vulnerabilityFinding(file, line, patternName, severity, matched,
  * Mask the middle of a secret for safe display
  */
 export function maskSecret(secret) {
-  if (secret.length <= 10) {
+  if (secret.length <= 6) {
+    return '***';
+  }
+  if (secret.length <= 12) {
     return secret.substring(0, 3) + '***';
   }
-  return secret.substring(0, 6) + '***' + secret.substring(secret.length - 4);
+  return secret.substring(0, 4) + '***' + secret.substring(secret.length - 4);
 }
 
 /**