linter-bundle 7.1.2 → 7.3.0

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-feature-parentheses-space-inside/index.mjs

@@ -23,7 +23,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => (root, result) => {
+const rule = (primary, _secondaryOptions) => (root, result) => {
 	const validOptions = validateOptions(result, ruleName, {
 		actual: primary,
 		possible: ['always', 'never']
@@ -38,10 +38,14 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		// will be at atRule.raws.params.raw
 		const parameters = (atRule.raws.params?.raw) || atRule.params;
 		const indexBoost = atRuleParamIndex(atRule);
-		/** @type {Array<{ message: string, index: number }>} */
-		const problems = [];
 
-		const parsedParameters = valueParser(parameters).walk((node) => {
+		// Parse the parameters once, and create a static copy for reporting
+		const originalParsedParams = valueParser(parameters);
+		let fixCount = 0;
+
+		// First pass - only detect problems
+		const problems = [];
+		originalParsedParams.walk((node) => {
 			if (node.type === 'function') {
 				const length_ = valueParser.stringify(node).length;
 
@@ -49,14 +53,18 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 					if ((/[\t ]/).test(node.before)) {
 						problems.push({
 							message: messages.rejectedOpening,
-							index: node.sourceIndex + 1 + indexBoost
+							index: node.sourceIndex + 1 + indexBoost,
+							functionIndex: node.sourceIndex,
+							type: 'opening'
 						});
 					}
 
 					if ((/[\t ]/).test(node.after)) {
 						problems.push({
 							message: messages.rejectedClosing,
-							index: node.sourceIndex - 2 + length_ + indexBoost
+							index: node.sourceIndex - 2 + length_ + indexBoost,
+							functionIndex: node.sourceIndex,
+							type: 'closing'
 						});
 					}
 				}
@@ -64,29 +72,76 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 					if (node.before === '') {
 						problems.push({
 							message: messages.expectedOpening,
-							index: node.sourceIndex + 1 + indexBoost
+							index: node.sourceIndex + 1 + indexBoost,
+							functionIndex: node.sourceIndex,
+							type: 'opening'
 						});
 					}
 
 					if (node.after === '') {
 						problems.push({
 							message: messages.expectedClosing,
-							index: node.sourceIndex - 2 + length_ + indexBoost
+							index: node.sourceIndex - 2 + length_ + indexBoost,
+							functionIndex: node.sourceIndex,
+							type: 'closing'
 						});
 					}
 				}
 			}
 		});
 
+		// Report each problem with a fix function
 		if (problems.length > 0) {
-			for (const error of problems) {
+			for (const problem of problems) {
 				report({
-					message: error.message,
+					message: problem.message,
 					node: atRule,
-					index: error.index,
-					endIndex: error.index,
+					index: problem.index,
+					endIndex: problem.index,
 					result,
-					ruleName
+					ruleName,
+					fix: () => {
+						// Create a fresh copy for fixing each time to avoid mixing fixes
+						const parsedParams = valueParser(parameters);
+						let fixed = false;
+
+						parsedParams.walk((node) => {
+							if (node.type === 'function' && node.sourceIndex === problem.functionIndex) {
+								if (problem.type === 'opening') {
+									if (primary === 'never' && (/[\t ]/).test(node.before)) {
+										node.before = '';
+										fixed = true;
+									}
+									else if (primary === 'always' && node.before === '') {
+										node.before = ' ';
+										fixed = true;
+									}
+								} else if (problem.type === 'closing') {
+									if (primary === 'never' && (/[\t ]/).test(node.after)) {
+										node.after = '';
+										fixed = true;
+									}
+									else if (primary === 'always' && node.after === '') {
+										node.after = ' ';
+										fixed = true;
+									}
+								}
+							}
+						});
+
+						if (fixed) {
+							const fixedParams = parsedParams.toString();
+
+							if (atRule.raws.params?.raw) {
+								atRule.raws.params.raw = fixedParams;
+							} else {
+								atRule.params = fixedParams;
+							}
+							return true;
+						}
+
+						return false;
+					}
 				});
 			}
 		}

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-feature-range-operator-space-after/index.mjs

@@ -23,7 +23,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => {
+const rule = (primary, _secondaryOptions) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
 	return (root, result) => {
@@ -37,35 +37,9 @@ const rule = (primary, _secondaryOptions, context) => {
 		}
 
 		root.walkAtRules(/^media$/i, (atRule) => {
-			/** @type {number[]} */
-			const fixOperatorIndices = [];
-
 			findMediaOperator(atRule, (match, parameters, node) => {
-				checkAfterOperator(match, parameters, node, null);
+				checkAfterOperator(match, parameters, node);
 			});
-
-			if (fixOperatorIndices.length > 0) {
-				let parameters = atRule.raws.params ? atRule.raws.params.raw : atRule.params;
-
-				for (const index of fixOperatorIndices.sort((a, b) => b - a)) {
-					const beforeOperator = parameters.slice(0, index + 1);
-					const afterOperator = parameters.slice(index + 1);
-
-					if (primary === 'always') {
-						parameters = beforeOperator + afterOperator.replace(/^\s*/, ' ');
-					}
-					else if (primary === 'never') {
-						parameters = beforeOperator + afterOperator.replace(/^\s*/, '');
-					}
-				}
-
-				if (atRule.raws.params) {
-					atRule.raws.params.raw = parameters;
-				}
-				else {
-					atRule.params = parameters;
-				}
-			}
 		});
 
 		/**
@@ -73,28 +47,40 @@ const rule = (primary, _secondaryOptions, context) => {
 		 * @param {string} params
 		 * @param parameters
 		 * @param {import('postcss').AtRule} node
-		 * @param {((index: number) => void) | null} fix
 		 */
-		function checkAfterOperator (match, parameters, node, fix) {
+		function checkAfterOperator(match, parameters, node) {
 			const endIndex = match.startIndex + match.target.length - 1;
 
 			checker.after({
 				source: parameters,
 				index: endIndex,
 				err: (m) => {
-					if (fix) {
-						fix(endIndex);
-
-						return;
-					}
-
 					report({
 						message: m,
 						node,
 						index: endIndex + atRuleParamIndex(node) + 1,
 						endIndex: endIndex + atRuleParamIndex(node) + 1,
 						result,
-						ruleName
+						ruleName,
+						fix: (fixer) => {
+							let parameters = node.raws.params ? node.raws.params.raw : node.params;
+							const beforeOperator = parameters.slice(0, endIndex + 1);
+							const afterOperator = parameters.slice(endIndex + 1);
+
+							if (primary === 'always') {
+								parameters = beforeOperator + afterOperator.replace(/^\s*/, ' ');
+							} else if (primary === 'never') {
+								parameters = beforeOperator + afterOperator.replace(/^\s*/, '');
+							}
+
+							if (node.raws.params) {
+								node.raws.params.raw = parameters;
+							} else {
+								node.params = parameters;
+							}
+
+							return fixer;
+						}
 					});
 				}
 			});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-feature-range-operator-space-before/index.mjs

@@ -23,7 +23,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => {
+const rule = (primary, _secondaryOptions) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
 	return (root, result) => {
@@ -39,9 +39,8 @@ const rule = (primary, _secondaryOptions, context) => {
 		root.walkAtRules(/^media$/i, (atRule) => {
 			/** @type {number[]} */
 			const fixOperatorIndices = [];
-
 			findMediaOperator(atRule, (match, parameters, node) => {
-				checkBeforeOperator(match, parameters, node, null);
+				checkBeforeOperator(match, parameters, node, (index) => fixOperatorIndices.push(index));
 			});
 
 			if (fixOperatorIndices.length > 0) {
@@ -82,19 +81,16 @@ const rule = (primary, _secondaryOptions, context) => {
 				source: parameters,
 				index: match.startIndex,
 				err: (m) => {
-					if (fix) {
-						fix(match.startIndex);
-
-						return;
-					}
-
 					report({
 						message: m,
 						node,
 						index: match.startIndex - 1 + atRuleParamIndex(node),
 						endIndex: match.startIndex - 1 + atRuleParamIndex(node),
 						result,
-						ruleName
+						ruleName,
+						fix: (fix ? () => {
+							fix(match.startIndex);
+						}: undefined)
 					});
 				}
 			});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-query-list-comma-newline-after/index.mjs

@@ -47,18 +47,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			locationChecker: checker.afterOneOnly,
 			checkedRuleName: ruleName,
 			allowTrailingComments: primary.startsWith('always'),
-			fix:
-				(atRule, index) => {
-					const parameterCommaIndex = index - atRuleParamIndex(atRule);
+			fix: (atRule, index) => {
+				const parameterCommaIndex = index - atRuleParamIndex(atRule);
 
-					fixData ||= new Map();
-					const commaIndices = fixData.get(atRule) || [];
+				fixData ||= new Map();
+				const commaIndices = fixData.get(atRule) || [];
 
-					commaIndices.push(parameterCommaIndex);
-					fixData.set(atRule, commaIndices);
+				commaIndices.push(parameterCommaIndex);
+				fixData.set(atRule, commaIndices);
 
-					return true;
-				  }
+				return true;
+			}
 		});
 
 		if (fixData) {

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-query-list-comma-space-after/index.mjs

@@ -24,7 +24,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => {
+const rule = (primary, _secondaryOptions) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
 	return (root, result) => {
@@ -45,18 +45,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			result,
 			locationChecker: checker.after,
 			checkedRuleName: ruleName,
-			fix:
-				(atRule, index) => {
-					const parameterCommaIndex = index - atRuleParamIndex(atRule);
+			fix: (atRule, index) => {
+				const parameterCommaIndex = index - atRuleParamIndex(atRule);
 
-					fixData ||= new Map();
-					const commaIndices = fixData.get(atRule) || [];
+				fixData ||= new Map();
+				const commaIndices = fixData.get(atRule) || [];
 
-					commaIndices.push(parameterCommaIndex);
-					fixData.set(atRule, commaIndices);
+				commaIndices.push(parameterCommaIndex);
+				fixData.set(atRule, commaIndices);
 
-					return true;
-				  }
+				return true;
+			}
 		});
 
 		if (fixData) {

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-query-list-comma-space-before/index.mjs

@@ -24,7 +24,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => {
+const rule = (primary, _secondaryOptions) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
 	return (root, result) => {
@@ -45,18 +45,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			result,
 			locationChecker: checker.before,
 			checkedRuleName: ruleName,
-			fix:
-				(atRule, index) => {
-					const parameterCommaIndex = index - atRuleParamIndex(atRule);
+			fix: (atRule, index) => {
+				const parameterCommaIndex = index - atRuleParamIndex(atRule);
 
-					fixData ||= new Map();
-					const commaIndices = fixData.get(atRule) || [];
+				fixData ||= new Map();
+				const commaIndices = fixData.get(atRule) || [];
 
-					commaIndices.push(parameterCommaIndex);
-					fixData.set(atRule, commaIndices);
+				commaIndices.push(parameterCommaIndex);
+				fixData.set(atRule, commaIndices);
 
-					return true;
-				  }
+				return true;
+			}
 		});
 
 		if (fixData) {

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/mediaFeatureColonSpaceChecker.mjs

@@ -47,7 +47,8 @@ export default function mediaFeatureColonSpaceChecker (options) {
 					index: colonIndex,
 					endIndex: colonIndex,
 					result: options.result,
-					ruleName: options.checkedRuleName
+					ruleName: options.checkedRuleName,
+					fix: (options.fix ? () => options.fix(node, colonIndex) : undefined)
 				});
 			}
 		});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/mediaQueryListCommaWhitespaceChecker.mjs

@@ -56,17 +56,14 @@ export default function mediaQueryListCommaWhitespaceChecker (options) {
 			err: (message) => {
 				const commaIndex = index + atRuleParamIndex(node);
 
-				if (options.fix?.(node, commaIndex)) {
-					return;
-				}
-
 				report({
 					message,
 					node,
 					index: commaIndex,
 					endIndex: commaIndex,
 					result: options.result,
-					ruleName: options.checkedRuleName
+					ruleName: options.checkedRuleName,
+					fix: (options.fix ? () => options.fix(node, commaIndex) : undefined)
 				});
 			}
 		});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/no-empty-first-line/index.mjs

@@ -19,7 +19,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => (root, result) => {
+const rule = (primary, _secondaryOptions) => (root, result) => {
 	const validOptions = validateOptions(result, ruleName, { actual: primary });
 
 	// @ts-expect-error -- TS2339: Property 'inline' does not exist on type 'Source'. Property 'lang' does not exist on type 'Source'.
@@ -27,7 +27,7 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		return;
 	}
 
-	const rootString = (root.source?.input.css) || '';
+	const rootString = root.toString();
 
 	if (!rootString.trim()) {
 		return;
@@ -38,7 +38,18 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 			message: messages.rejected,
 			node: root,
 			result,
-			ruleName
+			ruleName,
+			fix: () => {
+				if (root.first == null) {
+					throw new Error('The root node must have the first node.');
+				}
+
+				if (root.first.raws.before == null) {
+					throw new Error('The first node must have spaces before.');
+				}
+
+				root.first.raws.before = root.first.raws.before.trimStart();
+			}
 		});
 	}
 };

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/no-eol-whitespace/index.mjs

@@ -66,7 +66,7 @@ function findErrorStartIndex (lastEOLIndex, string, { ignoreEmptyLines, isRootFi
 }
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, secondaryOptions, context) => (root, result) => {
+const rule = (primary, secondaryOptions) => (root, result) => {
 	const validOptions = validateOptions(
 		result,
 		ruleName,
@@ -88,7 +88,7 @@ const rule = (primary, secondaryOptions, context) => (root, result) => {
 
 	const ignoreEmptyLines = optionsMatches(secondaryOptions, 'ignore', 'empty-lines');
 
-	const rootString = (root.source?.input.css) || '';
+	const rootString = root.toString();
 
 	/**
 	 * @param {number} index
@@ -100,7 +100,10 @@ const rule = (primary, secondaryOptions, context) => (root, result) => {
 			index,
 			endIndex: index,
 			result,
-			ruleName
+			ruleName,
+			fix: () => {
+				fix(root);
+			}
 		});
 	};
 

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/no-extra-semicolons/index.mjs

@@ -66,7 +66,7 @@ function getOffsetByNode (node) {
 }
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondaryOptions, context) => (root, result) => {
+const rule = (primary, _secondaryOptions) => (root, result) => {
 	const validOptions = validateOptions(result, ruleName, { actual: primary });
 
 	if (!validOptions) {
@@ -82,7 +82,17 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		styleSearch({ source: rawAfterRoot, target: ';' }, (match) => {
 			if (!root.source) { throw new Error('The root node must have a source'); }
 
-			complain(root.source.input.css.length - rawAfterRoot.length + match.startIndex);
+			report({
+				message: messages.rejected,
+				node: root,
+				index: root.source.input.css.length - rawAfterRoot.length + match.startIndex,
+				endIndex: root.source.input.css.length - rawAfterRoot.length + match.startIndex,
+				result,
+				ruleName,
+				fix: () => {
+					fixSemiIndices.push(match.startIndex);
+				}
+			});
 		});
 
 		// fix
@@ -114,7 +124,17 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 					return;
 				}
 
-				complain(getOffsetByNode(node) - rawBeforeNode.length + match.startIndex);
+				report({
+					message: messages.rejected,
+					node: root,
+					index: getOffsetByNode(node) - rawBeforeNode.length + match.startIndex,
+					endIndex: getOffsetByNode(node) - rawBeforeNode.length + match.startIndex,
+					result,
+					ruleName,
+					fix: () => {
+						fixSemiIndices.push(match.startIndex - rawBeforeIndexStart);
+					}
+				});
 			});
 
 			// fix
@@ -151,7 +171,17 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 						rawAfterNode.length +
 						match.startIndex;
 
-				complain(index);
+				report({
+					message: messages.rejected,
+					node: root,
+					index,
+					endIndex: index,
+					result,
+					ruleName,
+					fix: () => {
+						fixSemiIndices.push(match.startIndex);
+					}
+				});
 			});
 
 			// fix
@@ -178,7 +208,17 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 						rawOwnSemicolon.length +
 						match.startIndex;
 
-				complain(index);
+				report({
+					message: messages.rejected,
+					node: root,
+					index,
+					endIndex: index,
+					result,
+					ruleName,
+					fix: () => {
+						fixSemiIndices.push(match.startIndex);
+					}
+				});
 			});
 
 			// fix
@@ -188,20 +228,6 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		}
 	});
 
-	/**
-	 * @param {number} index
-	 */
-	function complain (index) {
-		report({
-			message: messages.rejected,
-			node: root,
-			index,
-			endIndex: index,
-			result,
-			ruleName
-		});
-	}
-
 	/**
 	 * @param {string} str
 	 * @param string_

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/no-missing-end-of-source-newline/index.mjs

@@ -34,7 +34,7 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		return;
 	}
 
-	const rootString = root.source.input.css;
+	const rootString = root.toString();
 
 	if (!rootString.trim() || rootString.endsWith('\n')) {
 		return;
@@ -46,7 +46,10 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		index: rootString.length - 1,
 		endIndex: rootString.length - 1,
 		result,
-		ruleName
+		ruleName,
+		fix: () => {
+			root.raws.after = context.newline;
+		}
 	});
 };