bahlint 28.58.6934-dev-002 → 28.58.6934-dev-003

package/bin/bahlint.js

@@ -142,11 +142,11 @@ ${getErrorMessage(error)}`;
 	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
+	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");
@@ -154,7 +154,7 @@ ${getErrorMessage(error)}`;
 
 	// 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='));
+	const isFixMode = args.includes("--fix") || args.some(arg => arg.startsWith("--fix="));
 
 	/*
 	 * Create ESLint instance with the provided options
@@ -201,9 +201,9 @@ ${getErrorMessage(error)}`;
 	const eslint = new ESLint(eslintOptions);
 
 	// Determine files to lint
-	let files = args.filter(arg => !arg.startsWith('-'));
+	let files = args.filter(arg => !arg.startsWith("-"));
 	if (files.length === 0) {
-		files = ['.']; // Default to current directory if no files specified
+		files = ["."]; // Default to current directory if no files specified
 	}
 
 	// Lint the files
@@ -264,4 +264,4 @@ ${getErrorMessage(error)}`;
 	} else {
 		process.exitCode = 0;
 	}
-})().catch(onFatalError);
+})().catch(onFatalError);

package/lib/cli-engine/formatters/html.js

@@ -13,7 +13,7 @@ const encodeHTML = (function () {
 		"&": "&",
 		"<": "&#60;",
 		">": "&#62;",
-		'"': "&#34;",
+		"\"": "&#34;",
 		"'": "&#39;",
 	};
 	const matchHTML = /[&<>"']/gu;

package/lib/config/config-loader.js

@@ -140,7 +140,7 @@ function isNativeTypeScriptSupportEnabled() {
  */
 async function loadTypeScriptConfigFileWithJiti(filePath, fileURL, mtime) {
 	const { createJiti, version: jitiVersion } =
-		 
+
 		await ConfigLoader.loadJiti().catch(() => {
 			throw new Error(
 				"The 'jiti' library is required for loading TypeScript configuration files. Make sure to install it.",

package/lib/config/flat-config-array.js

@@ -171,7 +171,6 @@ class FlatConfigArray extends ConfigArray {
 		}
 	}
 
-	 
 	/**
 	 * Replaces a config with another config to allow us to put strings
 	 * in the config array that will be replaced by objects before
@@ -211,7 +210,7 @@ class FlatConfigArray extends ConfigArray {
 	[ConfigArraySymbol.finalizeConfig](config) {
 		return new Config(config);
 	}
-	 
+
 }
 
 exports.FlatConfigArray = FlatConfigArray;

package/lib/config/flat-config-schema.js

@@ -96,7 +96,7 @@ function deepMerge(first, second, mergeMap = new Map()) {
 		...second,
 	};
 
-	delete result.__proto__;  
+	delete result.__proto__;
 
 	// Store the pending result for this combination of first and second arguments.
 	secondMergeMap.set(second, result);
@@ -308,7 +308,7 @@ const disableDirectiveSeveritySchema = {
 	validate(value) {
 		if (!(ALLOWED_SEVERITIES.has(value) || typeof value === "boolean")) {
 			throw new TypeError(
-				'Expected one of: "error", "warn", "off", 0, 1, 2, or a boolean.',
+				"Expected one of: \"error\", \"warn\", \"off\", 0, 1, 2, or a boolean.",
 			);
 		}
 	},
@@ -324,7 +324,7 @@ const unusedInlineConfigsSeveritySchema = {
 	validate(value) {
 		if (!ALLOWED_SEVERITIES.has(value)) {
 			throw new TypeError(
-				'Expected one of: "error", "warn", "off", 0, 1, or 2.',
+				"Expected one of: \"error\", \"warn\", \"off\", 0, 1, or 2.",
 			);
 		}
 	},
@@ -458,7 +458,7 @@ const rulesSchema = {
 			try {
 				// avoid hairy edge case
 				if (ruleId === "__proto__") {
-					 
+
 					delete result.__proto__;
 					continue;
 				}

package/lib/eslint/eslint-helpers.js

@@ -691,10 +691,10 @@ function createIgnoreResult(filePath, baseDir, configStatus) {
 
 				if (isInNodeModules) {
 					message =
-						'File ignored by default because it is located under the node_modules directory. Use ignore pattern "!**/node_modules/" to disable file ignore settings or use "--no-warn-ignored" to suppress this warning.';
+						"File ignored by default because it is located under the node_modules directory. Use ignore pattern \"!**/node_modules/\" to disable file ignore settings or use \"--no-warn-ignored\" to suppress this warning.";
 				} else {
 					message =
-						'File ignored because of a matching ignore pattern. Use "--no-ignore" to disable file ignore settings or use "--no-warn-ignored" to suppress this warning.';
+						"File ignored because of a matching ignore pattern. Use \"--no-ignore\" to disable file ignore settings or use \"--no-warn-ignored\" to suppress this warning.";
 				}
 			}
 			break;
@@ -899,7 +899,7 @@ function processOptions({
 		errors.push("'cacheLocation' must be a non-empty string.");
 	}
 	if (cacheStrategy !== "metadata" && cacheStrategy !== "content") {
-		errors.push('\'cacheStrategy\' must be any of "metadata", "content".');
+		errors.push("'cacheStrategy' must be any of \"metadata\", \"content\".");
 	}
 	if (
 		concurrency !== "off" &&
@@ -907,7 +907,7 @@ function processOptions({
 		!isPositiveInteger(concurrency)
 	) {
 		errors.push(
-			'\'concurrency\' must be a positive integer, "auto", or "off".',
+			"'concurrency' must be a positive integer, \"auto\", or \"off\".",
 		);
 	}
 	if (!isNonEmptyString(cwd) || !path.isAbsolute(cwd)) {
@@ -921,7 +921,7 @@ function processOptions({
 	}
 	if (fixTypes !== null && !isFixTypeArray(fixTypes)) {
 		errors.push(
-			'\'fixTypes\' must be an array of any of "directive", "problem", "suggestion", and "layout".',
+			"'fixTypes' must be an array of any of \"directive\", \"problem\", \"suggestion\", and \"layout\".",
 		);
 	}
 	if (!isEmptyArrayOrArrayOfNonEmptyString(flags)) {
@@ -1220,7 +1220,7 @@ function verifyText({
 	}
 
 	const endTime = hrtimeBigint();
-	debug('File "%s" linted in %t', filePath, endTime - startTime);
+	debug("File \"%s\" linted in %t", filePath, endTime - startTime);
 
 	return result;
 }
@@ -1308,7 +1308,7 @@ async function lintFile(
 		});
 		const readFileExitTime = hrtimeBigint();
 		const readFileDuration = readFileExitTime - readFileEnterTime;
-		debug('File "%s" read in %t', filePath, readFileDuration);
+		debug("File \"%s\" read in %t", filePath, readFileDuration);
 		if (readFileCounter) {
 			readFileCounter.duration += readFileDuration;
 		}

package/lib/eslint/eslint.js

@@ -1056,7 +1056,7 @@ class ESLint {
 					),
 			);
 		} else {
-			debug(`Linting in single-thread mode.`);
+			debug("Linting in single-thread mode.");
 			results = await lintFilesWithoutMultithreading(this, filePaths);
 		}
 

package/lib/eslint/worker.js

@@ -128,7 +128,7 @@ debug("Dependencies loaded in %t", depsLoadedTime - startTime);
 		const loadConfigExitTime = hrtimeBigint();
 		const loadConfigDuration = loadConfigExitTime - loadConfigEnterTime;
 		debug(
-			'Config array for file "%s" loaded in %t',
+			"Config array for file \"%s\" loaded in %t",
 			filePath,
 			loadConfigDuration,
 		);
@@ -150,7 +150,7 @@ debug("Dependencies loaded in %t", depsLoadedTime - startTime);
 
 		const fileLintingEndTime = hrtimeBigint();
 		debug(
-			'File "%s" processed in %t',
+			"File \"%s\" processed in %t",
 			filePath,
 			fileLintingEndTime - fileLintingStartTime,
 		);

package/lib/languages/js/source-code/source-code.js

@@ -700,7 +700,6 @@ class SourceCode extends TokenStore {
 		return this.scopeManager.getDeclaredVariables(node);
 	}
 
-	 
 	/**
 	 * Gets all the ancestors of a given node
 	 * @param {ASTNode} node The node
@@ -775,8 +774,6 @@ class SourceCode extends TokenStore {
 		return nodeOrToken.range;
 	}
 
-	 
-
 	/**
 	 * Marks a variable as used in the current scope
 	 * @param {string} name The name of the variable to mark as used.

package/lib/languages/js/source-code/token-store/cursor.js

@@ -69,7 +69,7 @@ module.exports = class Cursor {
 	 * @abstract
 	 */
 	/* c8 ignore next */
-	 
+
 	moveNext() {
 		throw new Error("Not implemented.");
 	}

package/lib/languages/js/validate-language-options.js

@@ -66,13 +66,13 @@ function isUndefined(value) {
 function validateEcmaVersion(ecmaVersion) {
 	if (isUndefined(ecmaVersion)) {
 		throw new TypeError(
-			'Key "ecmaVersion": Expected an "ecmaVersion" property.',
+			"Key \"ecmaVersion\": Expected an \"ecmaVersion\" property.",
 		);
 	}
 
 	if (typeof ecmaVersion !== "number" && ecmaVersion !== "latest") {
 		throw new TypeError(
-			'Key "ecmaVersion": Expected a number or "latest".',
+			"Key \"ecmaVersion\": Expected a number or \"latest\".",
 		);
 	}
 }
@@ -89,7 +89,7 @@ function validateSourceType(sourceType) {
 		!/^(?:script|module|commonjs)$/u.test(sourceType)
 	) {
 		throw new TypeError(
-			'Key "sourceType": Expected "script", "module", or "commonjs".',
+			"Key \"sourceType\": Expected \"script\", \"module\", or \"commonjs\".",
 		);
 	}
 }
@@ -102,7 +102,7 @@ function validateSourceType(sourceType) {
  */
 function validateGlobals(globals) {
 	if (!isNonArrayObject(globals)) {
-		throw new TypeError('Key "globals": Expected an object.');
+		throw new TypeError("Key \"globals\": Expected an object.");
 	}
 
 	for (const key of Object.keys(globals)) {
@@ -139,7 +139,7 @@ function validateParser(parser) {
 			typeof parser.parseForESLint !== "function")
 	) {
 		throw new TypeError(
-			'Key "parser": Expected object with parse() or parseForESLint() method.',
+			"Key \"parser\": Expected object with parse() or parseForESLint() method.",
 		);
 	}
 }
@@ -182,7 +182,7 @@ function validateLanguageOptions(languageOptions) {
 
 	if ("parserOptions" in languageOptions) {
 		if (!isNonArrayObject(parserOptions)) {
-			throw new TypeError('Key "parserOptions": Expected an object.');
+			throw new TypeError("Key \"parserOptions\": Expected an object.");
 		}
 	}
 

package/lib/linter/code-path-analysis/debug-helpers.js

@@ -118,32 +118,31 @@ module.exports = {
 				let text =
 					"\n" +
 					"digraph {\n" +
-					'node[shape=box,style="rounded,filled",fillcolor=white];\n' +
-					'initial[label="",shape=circle,style=filled,fillcolor=black,width=0.25,height=0.25];\n';
+					"node[shape=box,style=\"rounded,filled\",fillcolor=white];\n" +
+					"initial[label=\"\",shape=circle,style=filled,fillcolor=black,width=0.25,height=0.25];\n";
 
 				if (codePath.returnedSegments.length > 0) {
 					text +=
-						'final[label="",shape=doublecircle,style=filled,fillcolor=black,width=0.25,height=0.25];\n';
+						"final[label=\"\",shape=doublecircle,style=filled,fillcolor=black,width=0.25,height=0.25];\n";
 				}
 				if (codePath.thrownSegments.length > 0) {
 					text +=
-						'thrown[label="✘",shape=circle,width=0.3,height=0.3,fixedsize=true];\n';
+						"thrown[label=\"✘\",shape=circle,width=0.3,height=0.3,fixedsize=true];\n";
 				}
 
 				const traceMap = Object.create(null);
 				const arrows = this.makeDotArrows(codePath, traceMap);
 
-				 
 				for (const id in traceMap) {
 					const segment = traceMap[id];
 
 					text += `${id}[`;
 
 					if (segment.reachable) {
-						text += 'label="';
+						text += "label=\"";
 					} else {
 						text +=
-							'style="rounded,dashed,filled",fillcolor="#FF9800",label="<<unreachable>>\\n';
+							"style=\"rounded,dashed,filled\",fillcolor=\"#FF9800\",label=\"<<unreachable>>\\n";
 					}
 
 					if (segment.internal.nodes.length > 0) {
@@ -152,7 +151,7 @@ module.exports = {
 						text += "????";
 					}
 
-					text += '"];\n';
+					text += "\"];\n";
 				}
 
 				text += `${arrows}\n`;

package/lib/linter/linter.js

@@ -588,7 +588,7 @@ function runRules(
 
 				if (problem.fix && !(rule.meta && rule.meta.fixable)) {
 					throw new Error(
-						'Fixable rules must set the `meta.fixable` property to "code" or "whitespace".',
+						"Fixable rules must set the `meta.fixable` property to \"code\" or \"whitespace\".",
 					);
 				}
 
@@ -718,7 +718,7 @@ function normalizeCwd(cwd) {
 	}
 
 	// It's more explicit to assign the undefined
-	 
+
 	return undefined;
 }
 
@@ -1204,7 +1204,7 @@ class Linter {
 
 								let baseMessage = err.message
 									.slice(
-										err.message.startsWith('Key "rules":')
+										err.message.startsWith("Key \"rules\":")
 											? err.message.indexOf(":", 12) + 1
 											: err.message.indexOf(":") + 1,
 									)

package/lib/linter/timing.js

@@ -125,7 +125,7 @@ function display(data) {
 			.join("|"),
 	);
 
-	console.log(table.join("\n"));  
+	console.log(table.join("\n"));
 }
 
 /* c8 ignore next */

package/lib/rule-tester/rule-tester.js

@@ -221,7 +221,7 @@ function sanitize(text) {
 		return "";
 	}
 	return text.replace(
-		/[\u0000-\u0009\u000b-\u001a]/gu,  
+		/[\u0000-\u0009\u000b-\u001a]/gu,
 		c => `\\u${c.codePointAt(0).toString(16).padStart(4, "0")}`,
 	);
 }
@@ -312,13 +312,11 @@ function throwForbiddenMethodError(methodName, prototype) {
 	return function (...args) {
 		const called = forbiddenMethodCalls.get(methodName);
 
-		 
 		if (!called.has(this)) {
 			called.add(this);
 
 			return original.apply(this, args);
 		}
-		 
 
 		throw new Error(
 			`\`SourceCode#${methodName}()\` cannot be called inside a rule.`,
@@ -762,7 +760,7 @@ function buildLazyTestLocationEstimator(invoker) {
 				Object.assign(
 					testLocations,
 					{
-						[`valid[0]`]: `${sourceFile}:${
+						["valid[0]"]: `${sourceFile}:${
 							sourceLine + validStartIndex
 						}`,
 					},
@@ -1423,7 +1421,7 @@ class RuleTester {
 
 			assert.ok(
 				ruleHasMetaMessages || requireMessage !== "messageId",
-				`Assertion options can not use 'requireMessage: "messageId"' if rule under test doesn't define 'meta.messages'.`,
+				"Assertion options can not use 'requireMessage: \"messageId\"' if rule under test doesn't define 'meta.messages'.",
 			);
 
 			const result = runRuleForItem(item);
@@ -1574,7 +1572,7 @@ class RuleTester {
 									assert.ok(
 										!requiresDataProperty ||
 											!hasPlaceholders,
-										`Error should specify the 'data' property as the referenced message has placeholders.`,
+										"Error should specify the 'data' property as the referenced message has placeholders.",
 									);
 								}
 							}

package/lib/rules/jsx-quotes.js

@@ -18,10 +18,10 @@ const astUtils = require("./utils/ast-utils");
 
 const QUOTE_SETTINGS = {
 	"prefer-double": {
-		quote: '"',
+		quote: "\"",
 		description: "singlequote",
 		convert(str) {
-			return str.replace(/'/gu, '"');
+			return str.replace(/'/gu, "\"");
 		},
 	},
 	"prefer-single": {

package/lib/rules/keyword-spacing.js

@@ -135,10 +135,10 @@ module.exports = {
 			},
 		],
 		messages: {
-			expectedBefore: 'Expected space(s) before "{{value}}".',
-			expectedAfter: 'Expected space(s) after "{{value}}".',
-			unexpectedBefore: 'Unexpected space(s) before "{{value}}".',
-			unexpectedAfter: 'Unexpected space(s) after "{{value}}".',
+			expectedBefore: "Expected space(s) before \"{{value}}\".",
+			expectedAfter: "Expected space(s) after \"{{value}}\".",
+			unexpectedBefore: "Unexpected space(s) before \"{{value}}\".",
+			unexpectedAfter: "Unexpected space(s) after \"{{value}}\".",
 		},
 	},
 

package/lib/rules/lines-around-directive.js

@@ -48,9 +48,9 @@ module.exports = {
 
 		fixable: "whitespace",
 		messages: {
-			expected: 'Expected newline {{location}} "{{value}}" directive.',
+			expected: "Expected newline {{location}} \"{{value}}\" directive.",
 			unexpected:
-				'Unexpected newline {{location}} "{{value}}" directive.',
+				"Unexpected newline {{location}} \"{{value}}\" directive.",
 		},
 		deprecated: {
 			message: "The rule was replaced with a more general rule.",

package/lib/rules/logical-assignment-operators.js

@@ -503,7 +503,7 @@ module.exports = {
 			},
 
 			// foo || (foo = bar)
-			'LogicalExpression[right.type="AssignmentExpression"][right.operator="="]'(
+			"LogicalExpression[right.type=\"AssignmentExpression\"][right.operator=\"=\"]"(
 				logical,
 			) {
 				// Right side has to be parenthesized, otherwise would be parsed as (foo || foo) = bar which is illegal

package/lib/rules/no-misleading-character-class.js

@@ -567,8 +567,8 @@ module.exports = {
 								return fixer.insertTextAfter(
 									penultimateToken,
 									astUtils.isCommaToken(penultimateToken)
-										? ' "u",'
-										: ', "u"',
+										? " \"u\","
+										: ", \"u\"",
 								);
 							}
 

package/lib/rules/no-unused-vars.js

@@ -1152,7 +1152,7 @@ module.exports = {
 				// fixes [{a: {k}}], [{a: [k]}]
 				if (getTokenBeforeValue(node) === ":") {
 					if (parentNode.parent.type === "ObjectPattern") {
-						 
+
 						return fixObjectWithValueSeparator(node);
 					}
 				}
@@ -1226,7 +1226,7 @@ module.exports = {
 
 					// fixes [a, ...[[ b ]]] or [a, ...[{ b }]]
 					if (parentNode.parent.type === "RestElement") {
-						 
+
 						return fixRestInPattern(parentNode.parent);
 					}
 

package/lib/rules/no-void.js

@@ -51,7 +51,7 @@ module.exports = {
 		//--------------------------------------------------------------------------
 
 		return {
-			'UnaryExpression[operator="void"]'(node) {
+			"UnaryExpression[operator=\"void\"]"(node) {
 				if (
 					allowAsStatement &&
 					node.parent &&

package/lib/rules/quotes.js

@@ -18,18 +18,18 @@ const astUtils = require("./utils/ast-utils");
 
 const QUOTE_SETTINGS = {
 	double: {
-		quote: '"',
+		quote: "\"",
 		alternateQuote: "'",
 		description: "doublequote",
 	},
 	single: {
 		quote: "'",
-		alternateQuote: '"',
+		alternateQuote: "\"",
 		description: "singlequote",
 	},
 	backtick: {
 		quote: "`",
-		alternateQuote: '"',
+		alternateQuote: "\"",
 		description: "backtick",
 	},
 };

package/lib/rules/valid-typeof.js

@@ -48,7 +48,7 @@ module.exports = {
 		messages: {
 			invalidValue: "Invalid typeof comparison value.",
 			notString: "Typeof comparisons should be to string literals.",
-			suggestString: 'Use `"{{type}}"` instead of `{{type}}`.',
+			suggestString: "Use `\"{{type}}\"` instead of `{{type}}`.",
 		},
 	},
 
@@ -148,7 +148,7 @@ module.exports = {
 										fix(fixer) {
 											return fixer.replaceText(
 												sibling,
-												'"undefined"',
+												"\"undefined\"",
 											);
 										},
 									},

package/lib/rules/yoda.js

@@ -122,7 +122,7 @@ module.exports = {
 		],
 
 		docs: {
-			description: 'Require or disallow "Yoda" conditions',
+			description: "Require or disallow \"Yoda\" conditions",
 			recommended: false,
 			frozen: true,
 			url: "https://eslint.org/docs/latest/rules/yoda",

package/lib/services/warning-service.js

@@ -54,7 +54,7 @@ class WarningService {
 	 */
 	emitESLintIgnoreWarning() {
 		this.emitWarning(
-			'The ".eslintignore" file is no longer supported. Switch to using the "ignores" property in "eslint.config.js": https://eslint.org/docs/latest/use/configure/migration-guide#ignoring-files',
+			"The \".eslintignore\" file is no longer supported. Switch to using the \"ignores\" property in \"eslint.config.js\": https://eslint.org/docs/latest/use/configure/migration-guide#ignoring-files",
 			"ESLintIgnoreWarning",
 		);
 	}

package/lib/shared/ajv.js

@@ -27,7 +27,7 @@ module.exports = (additionalOptions = {}) => {
 	});
 
 	ajv.addMetaSchema(metaSchema);
-	 
+
 	ajv._opts.defaultMeta = metaSchema.id;
 
 	return ajv;

package/lib/shared/string-utils.js

@@ -9,7 +9,6 @@
 // Helpers
 //------------------------------------------------------------------------------
 
- 
 const ASCII_REGEX = /^[\u0000-\u007f]*$/u;
 
 /** @type {Intl.Segmenter | undefined} */

package/lib/shared/traverser.js

@@ -45,7 +45,7 @@ function getVisitorKeys(visitorKeys, node) {
 	if (!keys) {
 		keys = vk.getKeys(node);
 		debug(
-			'Unknown node type "%s": Estimated visitor keys %j',
+			"Unknown node type \"%s\": Estimated visitor keys %j",
 			node.type,
 			keys,
 		);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bahlint",
-  "version": "28.58.6934-dev-002",
+  "version": "28.58.6934-dev-003",
   "author": "Nicholas C. Zakas <nicholas+npm@nczconsulting.com>",
   "description": "An AST-based pattern checker for JavaScript.",
   "type": "commonjs",