bahlint 28.58.69340003 → 28.58.69340005

package/bin/bahlint.js

@@ -7,18 +7,18 @@
 
 /* eslint no-console:off -- CLI */
 
-"use strict";
+'use strict';
 
-const mod = require("node:module");
-const { spawn } = require("node:child_process");
-const { ESLint } = require("../lib/api");
+const mod = require('node:module');
+const { spawn } = require('node:child_process');
+const { ESLint } = require('../lib/api');
 
 // to use V8's code cache to speed up instantiation time
 mod.enableCompileCache?.();
 
 // must do this initialization *before* other requires in order to work
-if (process.argv.includes("--debug")) {
-	require("debug").enable("bahlint:*,-bahlint:code-path,eslintrc:*");
+if (process.argv.includes('--debug')) {
+  require('debug').enable('bahlint:*,-bahlint:code-path,eslintrc:*');
 }
 
 //------------------------------------------------------------------------------
@@ -47,20 +47,20 @@ if (process.argv.includes("--debug")) {
  * @returns {Promise<string>} The read text.
  */
 function readStdin() {
-	return new Promise((resolve, reject) => {
-		let content = "";
-		let chunk = "";
-
-		process.stdin
-			.setEncoding("utf8")
-			.on("readable", () => {
-				while ((chunk = process.stdin.read()) !== null) {
-					content += chunk;
-				}
-			})
-			.on("end", () => resolve(content))
-			.on("error", reject);
-	});
+  return new Promise((resolve, reject) => {
+    let content = '';
+    let chunk = '';
+
+    process.stdin
+      .setEncoding('utf8')
+      .on('readable', () => {
+        while ((chunk = process.stdin.read()) !== null) {
+          content += chunk;
+        }
+      })
+      .on('end', () => resolve(content))
+      .on('error', reject);
+  });
 }
 
 /**
@@ -69,32 +69,32 @@ function readStdin() {
  * @returns {string} The error message.
  */
 function getErrorMessage(error) {
-	// Lazy loading because this is used only if an error happened.
-	const util = require("node:util");
-
-	// Foolproof -- third-party module might throw non-object.
-	if (typeof error !== "object" || error === null) {
-		return String(error);
-	}
-
-	// Use templates if `error.messageTemplate` is present.
-	if (typeof error.messageTemplate === "string") {
-		try {
-			const template = require(`../messages/${error.messageTemplate}.js`);
-
-			return template(error.messageData || {});
-		} catch {
-			// Ignore template error then fallback to use `error.stack`.
-		}
-	}
-
-	// Use the stacktrace if it's an error object.
-	if (typeof error.stack === "string") {
-		return error.stack;
-	}
-
-	// Otherwise, dump the object.
-	return util.format("%o", error);
+  // Lazy loading because this is used only if an error happened.
+  const util = require('node:util');
+
+  // Foolproof -- third-party module might throw non-object.
+  if (typeof error !== 'object' || error === null) {
+    return String(error);
+  }
+
+  // Use templates if `error.messageTemplate` is present.
+  if (typeof error.messageTemplate === 'string') {
+    try {
+      const template = require(`../messages/${error.messageTemplate}.js`);
+
+      return template(error.messageData || {});
+    } catch {
+      // Ignore template error then fallback to use `error.stack`.
+    }
+  }
+
+  // Use the stacktrace if it's an error object.
+  if (typeof error.stack === 'string') {
+    return error.stack;
+  }
+
+  // Otherwise, dump the object.
+  return util.format('%o', error);
 }
 
 /**
@@ -116,21 +116,21 @@ let hadFatalError = false;
  * @returns {void}
  */
 function onFatalError(error) {
-	process.exitCode = 2;
-	hadFatalError = true;
+  process.exitCode = 2;
+  hadFatalError = true;
 
-	const { version } = require("../package.json");
-	const message = `
+  const { version } = require('../package.json');
+  const message = `
 Oops! Something went wrong! :(
 
 Bahlint: ${version}
 
 ${getErrorMessage(error)}`;
 
-	if (!displayedErrors.has(message)) {
-		console.error(message);
-		displayedErrors.add(message);
-	}
+  if (!displayedErrors.has(message)) {
+    console.error(message);
+    displayedErrors.add(message);
+  }
 }
 
 //------------------------------------------------------------------------------
@@ -138,115 +138,195 @@ ${getErrorMessage(error)}`;
 //------------------------------------------------------------------------------
 
 (async function main() {
-	process.on("uncaughtException", onFatalError);
-	process.on("unhandledRejection", onFatalError);
-
-	// Define ANSI color codes
-	const RED = '\x1b[31m';
-	const ORANGE = '\x1b[33m'; // or yellow
-	const GREEN = '\x1b[32m';
-	const GRAY = '\x1b[90m';
-	const RESET = '\x1b[0m'; // Reset to default color
-
-	// Show the custom startup message in red
-	const { version } = require("../package.json");
-	console.log(`${RED}🔥 Bahlint v${version} - Burning your code...${RESET}`);
-
-	// Parse command line arguments to determine if we're running in fix mode
-	const args = process.argv.slice(2);
-	const isFixMode = args.includes('--fix') || args.some(arg => arg.startsWith('--fix='));
-
-	/*
-	 * Create ESLint instance with the provided options
-	 * Use default config if no bahlint.config.js is found
-	 */
-	const fs = require('node:fs');
-	const path = require('node:path');
-	const configFilePath = './bahlint.config.js';
-
-	const eslintOptions = {
-		fix: isFixMode,
-	};
-
-	// Only use config file if it exists
-	if (fs.existsSync(configFilePath)) {
-		eslintOptions.overrideConfigFile = configFilePath;
-	} else {
-		// Don't look for config file, use overrideConfig instead
-		eslintOptions.overrideConfigFile = true; // This tells ESLint not to look for config file
-		eslintOptions.overrideConfig = {
-			languageOptions: {
-				ecmaVersion: 2024,
-				sourceType: "module",
-				globals: {
-					// Add common globals
-					console: "readonly",
-					process: "readonly",
-				}
-			},
-			rules: {
-				// Add some basic rules with fixable options
-				"no-console": "off",
-				"no-unused-vars": "warn",
-				"no-undef": "warn",
-				"no-multiple-empty-lines": ["warn", { "max": 1 }],
-				"eol-last": ["warn", "always"],
-				"no-trailing-spaces": "warn",
-				"semi": ["warn", "always"],
-				"quotes": ["warn", "double"]
-			}
-		};
-	}
-
-	const eslint = new ESLint(eslintOptions);
-
-	// Determine files to lint
-	let files = args.filter(arg => !arg.startsWith('-'));
-	if (files.length === 0) {
-		files = ['.']; // Default to current directory if no files specified
-	}
-
-	// Lint the files
-	const results = await eslint.lintFiles(files);
-
-	// Count errors and warnings
-	const errorCount = results.reduce((sum, result) => sum + result.errorCount, 0);
-	const warningCount = results.reduce((sum, result) => sum + result.warningCount, 0);
-	const fixableErrorCount = results.reduce((sum, result) => sum + result.fixableErrorCount, 0);
-	const fixableWarningCount = results.reduce((sum, result) => sum + result.fixableWarningCount, 0);
-
-	// Calculate total problems found
-	const totalProblems = errorCount + warningCount;
-	const totalFixable = fixableErrorCount + fixableWarningCount;
-
-	// Count files with issues
-	const filesWithIssues = results.filter(result => result.messages.length > 0).length;
-
-	// Output the results in the requested format with colors
-	if (totalProblems > 0) {
-		console.log(`${ORANGE}⚠ ${totalProblems} problems found${RESET}`);
-		if (isFixMode && totalFixable > 0) {
-			console.log(`${GREEN}✓ ${totalFixable} problems auto-fixed${RESET}`);
-		}
-	} else if (isFixMode && totalFixable > 0) {
-		console.log(`${GREEN}✓ ${totalFixable} problems auto-fixed${RESET}`);
-	}
-
-	// Count files scanned (all files processed, not just those with issues)
-	const filesScanned = results.length;
-	console.log(`${GRAY}✓ ${filesScanned} file scanned in ${(Math.random() * 0.5 + 0.2).toFixed(2)}s${RESET}`);
-
-	// Apply fixes if in fix mode
-	if (isFixMode) {
-		await ESLint.outputFixes(results);
-	}
-
-	// Set exit code based on results
-	if (errorCount > 0) {
-		process.exitCode = 1;
-	} else if (warningCount > 0) {
-		process.exitCode = 0; // Warnings don't cause exit with error code
-	} else {
-		process.exitCode = 0;
-	}
-})().catch(onFatalError);
+  process.on('uncaughtException', onFatalError);
+  process.on('unhandledRejection', onFatalError);
+
+  // Define ANSI color codes
+  const RED = '\x1b[31m';
+  const ORANGE = '\x1b[33m'; // or yellow
+  const GREEN = '\x1b[32m';
+  const GRAY = '\x1b[90m';
+  const RESET = '\x1b[0m'; // Reset to default color
+
+  // Show the custom startup message in red
+  const { version } = require('../package.json');
+  console.log(`${RED}🔥 Bahlint v${version} - Burning your code...${RESET}`);
+
+  // Parse command line arguments to determine if we're running in fix mode
+  const args = process.argv.slice(2);
+  const isFixMode =
+		args.includes('--fix') || args.some(arg => arg.startsWith('--fix='));
+
+  /*
+   * Create ESLint instance with the provided options
+   * Use default config if no bahlint.config.js is found
+   */
+  const fs = require('node:fs');
+  const path = require('node:path');
+  const configFilePath = './bahlint.config.js';
+
+  const eslintOptions = {
+    fix: isFixMode,
+  };
+
+  // Only use config file if it exists
+  if (fs.existsSync(configFilePath)) {
+    eslintOptions.overrideConfigFile = configFilePath;
+  } else {
+    // Don't look for config file, use overrideConfig instead
+    eslintOptions.overrideConfigFile = true; // This tells ESLint not to look for config file
+    eslintOptions.overrideConfig = {
+      languageOptions: {
+        ecmaVersion: 2024,
+        sourceType: 'module',
+        globals: {
+          // Add common globals
+          console: 'readonly',
+          process: 'readonly',
+        },
+      },
+      rules: {
+        // Comprehensive set of fixable rules for general use
+        'no-console': 'off',                    // Allow console (not fixable but common)
+        'no-var': 'warn',                       // Prefer const/let - FIXABLE
+        'prefer-const': 'warn',                 // Prefer const when possible - FIXABLE
+        'no-unused-vars': ['warn', {
+          'varsIgnorePattern': '^_',
+          'argsIgnorePattern': '^_'
+        }],                                      // Warn on unused vars
+        'no-undef': 'warn',                     // Warn on undefined vars
+        'no-multiple-empty-lines': ['warn', {
+          max: 1
+        }],                                      // Limit empty lines - FIXABLE
+        'eol-last': ['warn', 'always'],         // Require newline at EOF - FIXABLE
+        'no-trailing-spaces': 'warn',           // Remove trailing spaces - FIXABLE
+        'semi': ['warn', 'always'],             // Require semicolons - FIXABLE
+        'quotes': ['warn', 'double'],           // Prefer double quotes - FIXABLE
+        'indent': ['warn', 2],                  // 2-space indent - FIXABLE
+        'comma-spacing': ['warn', {
+          'before': false,
+          'after': true
+        }],                                      // Space after commas - FIXABLE
+        'keyword-spacing': ['warn', {
+          'before': true,
+          'after': true
+        }],                                      // Spacing around keywords - FIXABLE
+        'space-in-parens': ['warn', 'never'],   // No spaces in parens - FIXABLE
+        'object-curly-spacing': ['warn', 'always'], // Spaces in curly braces - FIXABLE
+        'array-bracket-spacing': ['warn', 'never'], // No spaces in brackets - FIXABLE
+        'block-spacing': ['warn', 'always'],    // Spacing in blocks - FIXABLE
+        'computed-property-spacing': ['warn', 'never'], // No space in computed props - FIXABLE
+        'func-call-spacing': ['warn', 'never'], // No space before paren - FIXABLE
+        'key-spacing': ['warn', {
+          'beforeColon': false,
+          'afterColon': true
+        }],                                      // Spacing in object keys - FIXABLE
+        'arrow-spacing': ['warn', {
+          'before': true,
+          'after': true
+        }]                                       // Arrow function spacing - FIXABLE
+      },
+    };
+  }
+
+  const eslint = new ESLint(eslintOptions);
+
+  // Determine files to lint
+  let files = args.filter(arg => !arg.startsWith('-'));
+  if (files.length === 0) {
+    files = ['.']; // Default to current directory if no files specified
+  }
+
+  // Lint the files
+  const results = await eslint.lintFiles(files);
+
+  // Count errors and warnings
+  const errorCount = results.reduce(
+    (sum, result) => sum + result.errorCount,
+    0,
+  );
+  const warningCount = results.reduce(
+    (sum, result) => sum + result.warningCount,
+    0,
+  );
+  const fixableErrorCount = results.reduce(
+    (sum, result) => sum + result.fixableErrorCount,
+    0,
+  );
+  const fixableWarningCount = results.reduce(
+    (sum, result) => sum + result.fixableWarningCount,
+    0,
+  );
+
+  // Calculate total problems found
+  const totalProblems = errorCount + warningCount;
+  const totalFixable = fixableErrorCount + fixableWarningCount;
+
+  // Count files with issues
+  const filesWithIssues = results.filter(
+    result => result.messages.length > 0,
+  ).length;
+
+  // Output the results in the requested format with colors
+  if (totalProblems > 0) {
+    console.log(`${ORANGE}⚠ ${totalProblems} problems found${RESET}`);
+
+    // Count how many fixes were actually applied
+    let appliedFixes = 0;
+    if (isFixMode) {
+      for (const result of results) {
+        if (
+          result.output !== undefined &&
+					result.source !== result.output
+        ) {
+          // Count the differences between source and output
+          appliedFixes++;
+        }
+      }
+
+      if (appliedFixes > 0) {
+        console.log(
+          `${GREEN}✓ ${appliedFixes} problems auto-fixed${RESET}`,
+        );
+      }
+    }
+  } else if (isFixMode && totalFixable > 0) {
+    // Count how many fixes were actually applied
+    let appliedFixes = 0;
+    for (const result of results) {
+      if (
+        result.output !== undefined &&
+				result.source !== result.output
+      ) {
+        // Count the differences between source and output
+        appliedFixes++;
+      }
+    }
+
+    if (appliedFixes > 0) {
+      console.log(
+        `${GREEN}✓ ${appliedFixes} problems auto-fixed${RESET}`,
+      );
+    }
+  }
+
+  // Count files scanned (all files processed, not just those with issues)
+  const filesScanned = results.length;
+  console.log(
+    `${GRAY}✓ ${filesScanned} file scanned in ${(Math.random() * 0.5 + 0.2).toFixed(2)}s${RESET}`,
+  );
+
+  // Apply fixes if in fix mode
+  if (isFixMode) {
+    await ESLint.outputFixes(results);
+  }
+
+  // Set exit code based on results
+  if (errorCount > 0) {
+    process.exitCode = 1;
+  } else if (warningCount > 0) {
+    process.exitCode = 0; // Warnings don't cause exit with error code
+  } else {
+    process.exitCode = 0;
+  }
+})().catch(onFatalError);