linter-bundle 7.2.0 → 7.4.0

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/indentation/index.mjs

@@ -24,7 +24,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,
@@ -105,24 +105,22 @@ const rule = (primary, secondaryOptions = {}, context) => (root, result) => {
 							(parent.raws.codeBefore?.endsWith('\n'))))) &&
 				before.slice(lastIndexOfNewline + 1) !== expectedOpeningBraceIndentation
 		) {
-			if (context.fix) {
-				if (isFirstChild && isString(node.raws.before)) {
-					node.raws.before = node.raws.before.replace(
-						/^[\t ]*(?=\S|$)/,
-						expectedOpeningBraceIndentation
-					);
-				}
+			report({
+				message: messages.expected(legibleExpectation(nodeLevel)),
+				node,
+				result,
+				ruleName,
+				fix: () => {
+					if (isFirstChild && isString(node.raws.before)) {
+						node.raws.before = node.raws.before.replace(
+							/^[\t ]*(?=\S|$)/,
+							expectedOpeningBraceIndentation
+						);
+					}
 
-				node.raws.before = fixIndentation(node.raws.before, expectedOpeningBraceIndentation);
-			}
-			else {
-				report({
-					message: messages.expected(legibleExpectation(nodeLevel)),
-					node,
-					result,
-					ruleName
-				});
-			}
+					node.raws.before = fixIndentation(node.raws.before, expectedOpeningBraceIndentation);
+				}
+			});
 		}
 
 		// Only blocks have the `after` string to check.
@@ -139,19 +137,17 @@ const rule = (primary, secondaryOptions = {}, context) => (root, result) => {
 				after.includes('\n') &&
 				after.slice(after.lastIndexOf('\n') + 1) !== expectedClosingBraceIndentation
 		) {
-			if (context.fix) {
-				node.raws.after = fixIndentation(node.raws.after, expectedClosingBraceIndentation);
-			}
-			else {
-				report({
-					message: messages.expected(legibleExpectation(closingBraceLevel)),
-					node,
-					index: node.toString().length - 1,
-					endIndex: node.toString().length - 1,
-					result,
-					ruleName
-				});
-			}
+			report({
+				message: messages.expected(legibleExpectation(closingBraceLevel)),
+				node,
+				index: node.toString().length - 1,
+				endIndex: node.toString().length - 1,
+				result,
+				ruleName,
+				fix: () => {
+					node.raws.after = fixIndentation(node.raws.after, expectedClosingBraceIndentation);
+				}
+			});
 		}
 
 		// If this is a declaration, check the value
@@ -378,24 +374,22 @@ const rule = (primary, secondaryOptions = {}, context) => (root, result) => {
 				);
 
 				if (afterNewlineSpace !== expectedIndentation) {
-					if (context.fix) {
-						// Adding fixes position in reverse order, because if we change indent in the beginning of the string it will break all following fixes for that string
-						fixPositions.unshift({
-							expectedIndentation,
-							currentIndentation: afterNewlineSpace,
-							startIndex: match.startIndex
-						});
-					}
-					else {
-						report({
-							message: messages.expected(legibleExpectation(expectedIndentLevel)),
-							node,
-							index: match.startIndex + afterNewlineSpace.length + 1,
-							endIndex: match.startIndex + afterNewlineSpace.length + 1,
-							result,
-							ruleName
-						});
-					}
+					report({
+						message: messages.expected(legibleExpectation(expectedIndentLevel)),
+						node,
+						index: match.startIndex + afterNewlineSpace.length + 1,
+						endIndex: match.startIndex + afterNewlineSpace.length + 1,
+						result,
+						ruleName,
+						fix: () => {
+							// Adding fixes position in reverse order, because if we change indent in the beginning of the string it will break all following fixes for that string
+							fixPositions.unshift({
+								expectedIndentation,
+								currentIndentation: afterNewlineSpace,
+								startIndex: match.startIndex
+							});
+						}
+					});
 				}
 			}
 		);

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/max-empty-lines/index.mjs

@@ -22,7 +22,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, secondaryOptions, context) => {
+const rule = (primary, secondaryOptions) => {
 	let emptyLines = 0;
 	let lastIndex = -1;
 
@@ -50,11 +50,8 @@ const rule = (primary, secondaryOptions, context) => {
 		const ignoreComments = optionsMatches(secondaryOptions, 'ignore', 'comments');
 		const getChars = replaceEmptyLines.bind(null, primary);
 
-		/**
-		 * 1. walk nodes & replace enterchar
-		 * 2. deal with special case.
-		 */
-		if (context.fix) {
+		const fixFn = () => {
+			// Erst alle Nodes durchgehen und Zeilenumbrüche ersetzen
 			root.walk((node) => {
 				if (node.type === 'comment' && !ignoreComments) {
 					node.raws.left = getChars(node.raws.left);
@@ -87,9 +84,9 @@ const rule = (primary, secondaryOptions, context) => {
 				// `css in js` or `html`
 				root.raws.after = replaceEmptyLines(primary === 0 ? 1 : primary, rootRawsAfter);
 			}
+		};
 
-			return;
-		}
+		let problemFound = false;
 
 		emptyLines = 0;
 		lastIndex = -1;
@@ -106,6 +103,10 @@ const rule = (primary, secondaryOptions, context) => {
 			}
 		);
 
+		if (problemFound) {
+			fixFn();
+		}
+
 		/**
 		 * @param {string} source
 		 * @param {number} matchStartIndex
@@ -131,6 +132,8 @@ const rule = (primary, secondaryOptions, context) => {
 			if (!eof && !problem) { return; }
 
 			if (problem) {
+				problemFound = true;
+
 				report({
 					message: messages.expected(primary),
 					node,
@@ -146,6 +149,8 @@ const rule = (primary, secondaryOptions, context) => {
 				emptyLines++;
 
 				if (emptyLines > primary && isEofNode(result.root, node)) {
+					problemFound = true;
+
 					report({
 						message: messages.expected(primary),
 						node,

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/max-line-length/index.mjs

@@ -21,7 +21,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,
@@ -54,7 +54,7 @@ const rule = (primary, secondaryOptions, context) => (root, result) => {
 
 	const ignoreNonComments = optionsMatches(secondaryOptions, 'ignore', 'non-comments');
 	const ignoreComments = optionsMatches(secondaryOptions, 'ignore', 'comments');
-	const rootString = context.fix ? root.toString() : root.source.input.css;
+	const rootString = root.toString();
 	// Array of skipped sub strings, i.e `url(...)`, `@import "..."`
 	/** @type {Array<[number, number]>} */
 	let skippedSubStrings = [];

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

@@ -22,7 +22,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) => {
@@ -43,19 +43,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			result,
 			locationChecker: checker.after,
 			checkedRuleName: ruleName,
-			fix: context.fix ?
-				(atRule, index) => {
-					const parameterColonIndex = index - atRuleParamIndex(atRule);
+			fix: (atRule, index) => {
+				const parameterColonIndex = index - atRuleParamIndex(atRule);
 
-					fixData ||= new Map();
-					const colonIndices = fixData.get(atRule) || [];
+				fixData ||= new Map();
+				const colonIndices = fixData.get(atRule) || [];
 
-					colonIndices.push(parameterColonIndex);
-					fixData.set(atRule, colonIndices);
+				colonIndices.push(parameterColonIndex);
+				fixData.set(atRule, colonIndices);
 
-					return true;
-				  }
-				  : null
+				return true;
+			}
 		});
 
 		if (fixData) {

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

@@ -22,7 +22,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) => {
@@ -43,19 +43,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			result,
 			locationChecker: checker.before,
 			checkedRuleName: ruleName,
-			fix: context.fix ?
-				(atRule, index) => {
-					const parameterColonIndex = index - atRuleParamIndex(atRule);
+			fix: (atRule, index) => {
+				const parameterColonIndex = index - atRuleParamIndex(atRule);
 
-					fixData ||= new Map();
-					const colonIndices = fixData.get(atRule) || [];
+				fixData ||= new Map();
+				const colonIndices = fixData.get(atRule) || [];
 
-					colonIndices.push(parameterColonIndex);
-					fixData.set(atRule, colonIndices);
+				colonIndices.push(parameterColonIndex);
+				fixData.set(atRule, colonIndices);
 
-					return true;
-				  }
-				  : null
+				return true;
+			}
 		});
 
 		if (fixData) {

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/media-feature-name-case/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: ['lower', 'upper']
@@ -53,13 +53,6 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 				return;
 			}
 
-			if (context.fix) {
-				mutateIdent(mediaFeatureNameToken, expectedFeatureName);
-				hasFixes = true;
-
-				return;
-			}
-
 			const atRuleIndex = atRuleParamIndex(atRule);
 
 			report({
@@ -68,7 +61,11 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 				index: atRuleIndex + startIndex,
 				endIndex: atRuleIndex + endIndex + 1,
 				ruleName,
-				result
+				result,
+				fix: () => {
+					mutateIdent(mediaFeatureNameToken, expectedFeatureName);
+					hasFixes = true;
+				}
 			});
 		}).stringify();
 

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,69 +38,110 @@ 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;
 
 				if (primary === 'never') {
 					if ((/[\t ]/).test(node.before)) {
-						if (context.fix) { 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)) {
-						if (context.fix) { node.after = ''; }
-
 						problems.push({
 							message: messages.rejectedClosing,
-							index: node.sourceIndex - 2 + length_ + indexBoost
+							index: node.sourceIndex - 2 + length_ + indexBoost,
+							functionIndex: node.sourceIndex,
+							type: 'closing'
 						});
 					}
 				}
 				else if (primary === 'always') {
 					if (node.before === '') {
-						if (context.fix) { 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 === '') {
-						if (context.fix) { 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) {
-			if (context.fix) {
-				atRule.params = parsedParameters.toString();
-
-				return;
-			}
-
-			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,37 +37,9 @@ const rule = (primary, _secondaryOptions, context) => {
 		}
 
 		root.walkAtRules(/^media$/i, (atRule) => {
-			/** @type {number[]} */
-			const fixOperatorIndices = [];
-			/** @type {((index: number) => void) | null} */
-			const fix = context.fix ? (index) => fixOperatorIndices.push(index) : null;
-
 			findMediaOperator(atRule, (match, parameters, node) => {
-				checkAfterOperator(match, parameters, node, fix);
+				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;
-				}
-			}
 		});
 
 		/**
@@ -75,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,11 +39,8 @@ const rule = (primary, _secondaryOptions, context) => {
 		root.walkAtRules(/^media$/i, (atRule) => {
 			/** @type {number[]} */
 			const fixOperatorIndices = [];
-			/** @type {((index: number) => void) | null} */
-			const fix = context.fix ? (index) => fixOperatorIndices.push(index) : null;
-
 			findMediaOperator(atRule, (match, parameters, node) => {
-				checkBeforeOperator(match, parameters, node, fix);
+				checkBeforeOperator(match, parameters, node, (index) => fixOperatorIndices.push(index));
 			});
 
 			if (fixOperatorIndices.length > 0) {
@@ -84,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,19 +47,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			locationChecker: checker.afterOneOnly,
 			checkedRuleName: ruleName,
 			allowTrailingComments: primary.startsWith('always'),
-			fix: context.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;
-				  }
-				  : null
+				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,19 +45,17 @@ const rule = (primary, _secondaryOptions, context) => {
 			result,
 			locationChecker: checker.after,
 			checkedRuleName: ruleName,
-			fix: context.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;
-				  }
-				  : null
+				return true;
+			}
 		});
 
 		if (fixData) {