linter-bundle 7.2.0 → 7.3.0

package/CHANGELOG.md

@@ -6,7 +6,21 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
-[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v7.1.2...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v7.3.0...HEAD)
+
+## [7.3.0] - 2025-03-20
+
+### Changed
+
+- [eslint] Updated `eslint-import-resolver-typescript` from `4.2.0` to `4.2.2`
+- [eslint] Updated `typescript-eslint` from `8.26.1` to `8.27.0`
+- [stylelint] Refactored all mocked stylistic Stylelint rule, to get rid of the deprecated `context.fix`
+
+### Removed
+
+- [stylelint] Removed the stylistic `linebreaks` plugin, as this can be achieved using the `.editorconfig` file
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v7.2.0...v7.3.0)
 
 ## [7.2.0] - 2025-03-18
 
@@ -21,7 +35,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ### Added
 
-- Re-added fixers of the mocked stylistic Stylelint rule.
+- [stylelint] Re-added fixers of the mocked stylistic Stylelint rule.
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v7.1.2...v7.2.0)
 
 ## [7.1.2] - 2025-03-15
 

package/README.md

@@ -58,7 +58,7 @@ Beside that, the following additional rules are part of this bundle:
 - [stylelint-selector-tag-no-without-class](https://www.npmjs.com/package/stylelint-selector-tag-no-without-class) (Forked version)
 - [stylelint-use-logical-spec](https://www.npmjs.com/package/stylelint-use-logical-spec)
 
-Beside that [73 stylistic rules](https://github.com/jens-duttke/linter-bundle/tree/main/stylelint/plugins/stylelint-15.11.0-stylistic/rules) has been forked from `stylelint@15.11.0`, which have been removed in `stylelint@16.0.0`, are part of this bundle.
+Beside that [72 stylistic rules](https://github.com/jens-duttke/linter-bundle/tree/main/stylelint/plugins/stylelint-15.11.0-stylistic/rules) has been forked from `stylelint@15.11.0`, which have been removed in `stylelint@16.0.0`, are part of this bundle.
 
 ### Previously used, but now unmaintained plugins
 
@@ -93,16 +93,16 @@ npm install linter-bundle --save-dev
 #### eslint.config.mjs
 
 ```js
-import gatsbyConfig from './eslint/gatsby.mjs';
-import javascriptConfig from './eslint/javascript.mjs';
-// import javascriptLazyConfig from './eslint/javascript-lazy.mjs';
-import jestConfig from './eslint/jest.mjs';
-import jsdocConfig from './eslint/jsdoc.mjs';
-import reactConfig from './eslint/react.mjs';
-import storybookConfig from './eslint/storybook.mjs';
-import typeDeclarationsConfig from './eslint/type-declarations.mjs';
-import workerConfig from './eslint/worker.mjs';
-import eslintConfig from './eslint.mjs';
+import gatsbyConfig from 'linter-bundle/eslint/gatsby.mjs';
+import javascriptConfig from 'linter-bundle/eslint/javascript.mjs';
+// import javascriptLazyConfig from 'linter-bundle/eslint/javascript-lazy.mjs';
+import jestConfig from 'linter-bundle/eslint/jest.mjs';
+import jsdocConfig from 'linter-bundle/eslint/jsdoc.mjs';
+import reactConfig from 'linter-bundle/eslint/react.mjs';
+import storybookConfig from 'linter-bundle/eslint/storybook.mjs';
+import typeDeclarationsConfig from 'linter-bundle/eslint/type-declarations.mjs';
+import workerConfig from 'linter-bundle/eslint/worker.mjs';
+import eslintConfig from 'linter-bundle/eslint.mjs';
 
 export default [
   ...eslintConfig,

package/eslint/rules/enforce-logical-expression-parens.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which ensures logical operations are wrapped in parentheses.
+ * @file ESLint rule that ensures logical operations are wrapped in parentheses.
  */
 
 import { isParenthesized } from './helper/is-parenthesized.mjs';

package/eslint/rules/enforce-ternary-parens.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which ensures ternary expressions are wrapped in parentheses.
+ * @file ESLint rule that ensures ternary expressions are wrapped in parentheses.
  */
 
 import { isParenthesized } from './helper/is-parenthesized.mjs';

package/eslint/rules/ensure-lucide-import-consistency.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which enforces using Lucide prefix for lucide-react imports and their usage.
+ * @file ESLint rule that enforces using Lucide prefix for lucide-react imports and their usage.
  */
 
 import { AST_NODE_TYPES } from '@typescript-eslint/utils';

package/eslint/rules/no-extra-spaces-in-generics.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which disallows spaces after '<' and before '>' in TypeScript generics.
+ * @file ESLint rule that disallows spaces after '<' and before '>' in TypeScript generics.
  */
 
 /**

package/eslint/rules/no-ternary-return.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which disallows ternary expressions as return values for better readability.
+ * @file ESLint rule that disallows ternary expressions as return values for better readability.
  */
 
 /**

package/eslint/rules/no-unnecessary-typeof.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which ensures that a `typeof` operant has more than one type in TypeScript, to prevent unnecessary checks of types at runtime.
+ * @file ESLint rule that ensures that a `typeof` operant has more than one type in TypeScript, to prevent unnecessary checks of types at runtime.
  */
 
 /** @typedef {ts.Type & { intrinsicName?: string; types?: ts.Type[]; objectFlags?: ts.ObjectFlags; }} Type */

package/eslint/rules/restricted-filenames.mjs

@@ -1,5 +1,5 @@
 /**
- * @file ESLint rule which ensures that only files which match given glob patterns are part of your project.
+ * @file ESLint rule that ensures that only files which match given glob patterns are part of your project.
  */
 
 import path from 'node:path';

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "7.2.0",
+	"version": "7.3.0",
 	"type": "module",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
@@ -45,7 +45,7 @@
 		"@stylistic/eslint-plugin-jsx": "4.2.0",
 		"eslint": "9.22.0",
 		"eslint-formatter-unix": "8.40.0",
-		"eslint-import-resolver-typescript": "4.2.0",
+		"eslint-import-resolver-typescript": "4.2.2",
 		"eslint-import-resolver-webpack": "0.13.10",
 		"eslint-plugin-eslint-comments": "3.2.0",
 		"eslint-plugin-functional": "9.0.1",
@@ -68,7 +68,7 @@
 		"stylelint-order": "6.0.4",
 		"stylelint-scss": "6.11.1",
 		"stylelint-use-logical-spec": "5.0.1",
-		"typescript-eslint": "8.26.1"
+		"typescript-eslint": "8.27.0"
 	},
 	"peerDependencies": {
 		"@typescript-eslint/utils": "*",

package/stylelint/index.mjs

@@ -49,7 +49,6 @@ export default {
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/function-parentheses-space-inside/index.mjs'),
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/function-whitespace-after/index.mjs'),
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/indentation/index.mjs'),
-		await import('./plugins/stylelint-15.11.0-stylistic/rules/linebreaks/index.mjs'),
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/max-empty-lines/index.mjs'),
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/max-line-length/index.mjs'),
 		await import('./plugins/stylelint-15.11.0-stylistic/rules/media-feature-colon-space-after/index.mjs'),
@@ -395,7 +394,6 @@ export default {
 		'plugin/function-parentheses-space-inside': 'never-single-line',
 		'plugin/function-whitespace-after': 'always',
 		'plugin/indentation': 'tab',
-		'plugin/linebreaks': 'unix',
 		'plugin/max-empty-lines': 1,
 		'plugin/max-line-length': 160,
 		'plugin/media-feature-colon-space-after': 'always',

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/at-rule-name-case/index.mjs

@@ -19,7 +19,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondary, context) => (root, result) => {
+const rule = (primary, _secondary) => (root, result) => {
 	const validOptions = validateOptions(result, ruleName, {
 		actual: primary,
 		possible: ['lower', 'upper']
@@ -38,24 +38,20 @@ const rule = (primary, _secondary, context) => (root, result) => {
 		}
 
 		const name = atRule.name;
-
 		const expectedName = expectation === 'lower' ? name.toLowerCase() : name.toUpperCase();
 
 		if (name === expectedName) {
 			return;
 		}
 
-		if (context.fix) {
-			atRule.name = expectedName;
-
-			return;
-		}
-
 		report({
 			message: messages.expected(name, expectedName),
 			node: atRule,
 			ruleName,
-			result
+			result,
+			fix: () => {
+				atRule.name = expectedName;
+			}
 		});
 	});
 };

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/at-rule-name-space-after/index.mjs

@@ -20,7 +20,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondary, context) => {
+const rule = (primary, _secondary) => {
 	const checker = whitespaceChecker('space', primary, messages);
 
 	return (root, result) => {

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/at-rule-semicolon-newline-after/index.mjs

@@ -24,7 +24,7 @@ const meta = {
 };
 
 /** @type {import('stylelint').Rule} */
-const rule = (primary, _secondary, context) => {
+const rule = (primary, _secondary) => {
 	const checker = whitespaceChecker('newline', primary, messages);
 
 	return (root, result) => {
@@ -63,19 +63,17 @@ const rule = (primary, _secondary, context) => {
 				source: rawNodeString(nodeToCheck),
 				index: -1,
 				err: (message) => {
-					if (context.fix) {
-						nodeToCheck.raws.before = context.newline + nodeToCheck.raws.before;
-					}
-					else {
-						report({
-							message,
-							node: atRule,
-							index: atRule.toString().length + 1,
-							endIndex: atRule.toString().length + 1,
-							result,
-							ruleName
-						});
-					}
+					report({
+						message,
+						node: atRule,
+						index: atRule.toString().length + 1,
+						endIndex: atRule.toString().length + 1,
+						result,
+						ruleName,
+						fix: () => {
+							nodeToCheck.raws.before = '\n' + nodeToCheck.raws.before;
+						}
+					});
 				}
 			});
 		});

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

@@ -36,19 +36,14 @@ export default function atRuleNameSpaceChecker (options) {
 			source,
 			index,
 			err: (m) => {
-				if (options.fix) {
-					options.fix(node);
-
-					return;
-				}
-
 				report({
 					message: m,
 					node,
 					index,
 					endIndex: index,
 					result: options.result,
-					ruleName: options.checkedRuleName
+					ruleName: options.checkedRuleName,
+					fix: (options.fix ? () => options.fix(node) : undefined)
 				});
 			},
 			errTarget: `@${node.name}`

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/block-closing-brace-empty-line-before/index.mjs

@@ -97,21 +97,6 @@ const rule = (primary, secondaryOptions, context) => (root, result) => {
 			return;
 		}
 
-		if (context.fix) {
-			const { newline } = context;
-
-			if (typeof newline !== 'string') { return; }
-
-			if (expectEmptyLineBefore) {
-				addEmptyLineAfter(statement, newline);
-			}
-			else {
-				removeEmptyLinesAfter(statement, newline);
-			}
-
-			return;
-		}
-
 		const message = expectEmptyLineBefore ? messages.expected : messages.rejected;
 
 		report({
@@ -120,7 +105,19 @@ const rule = (primary, secondaryOptions, context) => (root, result) => {
 			ruleName,
 			node: statement,
 			index,
-			endIndex: index
+			endIndex: index,
+			fix: () => {
+				const { newline } = context;
+
+				if (typeof newline !== 'string') { return; }
+
+				if (expectEmptyLineBefore) {
+					addEmptyLineAfter(statement, newline);
+				}
+				else {
+					removeEmptyLinesAfter(statement, newline);
+				}
+			}
 		});
 	}
 };

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/block-closing-brace-newline-after/index.mjs

@@ -112,36 +112,35 @@ const rule = (primary, secondaryOptions, context) => {
 				index: -1,
 				lineCheckStr: blockString(statement),
 				err: (message) => {
-					if (context.fix) {
-						const nodeToCheckRaws = nodeToCheck.raws;
+					report({
+						message,
+						node: statement,
+						index: reportIndex,
+						endIndex: reportIndex,
+						result,
+						ruleName,
+						fix: () => {
+							const nodeToCheckRaws = nodeToCheck.raws;
 
-						if (typeof nodeToCheckRaws.before !== 'string') { return; }
+							if (typeof nodeToCheckRaws.before !== 'string') { return; }
 
-						if (primary.startsWith('always')) {
-							const index = nodeToCheckRaws.before.search(/\r?\n/);
+							if (primary.startsWith('always')) {
+								const index = nodeToCheckRaws.before.search(/\r?\n/);
 
-							nodeToCheckRaws.before =
-								index >= 0 ?
-									nodeToCheckRaws.before.slice(index)
-									: context.newline + nodeToCheckRaws.before;
+								nodeToCheckRaws.before =
+									index >= 0 ?
+										nodeToCheckRaws.before.slice(index)
+										: context.newline + nodeToCheckRaws.before;
 
-							return;
-						}
+								return;
+							}
 
-						if (primary.startsWith('never')) {
-							nodeToCheckRaws.before = '';
+							if (primary.startsWith('never')) {
+								nodeToCheckRaws.before = '';
 
-							return;
+								return;
+							}
 						}
-					}
-
-					report({
-						message,
-						node: statement,
-						index: reportIndex,
-						endIndex: reportIndex,
-						result,
-						ruleName
 					});
 				}
 			});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/block-closing-brace-newline-before/index.mjs

@@ -86,36 +86,6 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 		 * @param {string} message
 		 */
 		function complain (message) {
-			if (context.fix) {
-				const statementRaws = statement.raws;
-
-				if (typeof statementRaws.after !== 'string') { return; }
-
-				if (primary.startsWith('always')) {
-					const firstWhitespaceIndex = statementRaws.after.search(/\s/);
-					const newlineBefore =
-							firstWhitespaceIndex >= 0 ?
-								statementRaws.after.slice(0, firstWhitespaceIndex)
-								: statementRaws.after;
-					const newlineAfter =
-							firstWhitespaceIndex >= 0 ? statementRaws.after.slice(firstWhitespaceIndex) : '';
-					const newlineIndex = newlineAfter.search(/\r?\n/);
-
-					statementRaws.after =
-							newlineIndex >= 0 ?
-								newlineBefore + newlineAfter.slice(newlineIndex)
-								: newlineBefore + context.newline + newlineAfter;
-
-					return;
-				}
-
-				if (primary === 'never-multi-line') {
-					statementRaws.after = statementRaws.after.replace(/\s/g, '');
-
-					return;
-				}
-			}
-
 			report({
 				message,
 				result,
@@ -123,6 +93,35 @@ const rule = (primary, _secondaryOptions, context) => (root, result) => {
 				node: statement,
 				index,
 				endIndex: index,
+				fix: () => {
+					const statementRaws = statement.raws;
+
+					if (typeof statementRaws.after !== 'string') { return; }
+
+					if (primary.startsWith('always')) {
+						const firstWhitespaceIndex = statementRaws.after.search(/\s/);
+						const newlineBefore =
+								firstWhitespaceIndex >= 0 ?
+									statementRaws.after.slice(0, firstWhitespaceIndex)
+									: statementRaws.after;
+						const newlineAfter =
+								firstWhitespaceIndex >= 0 ? statementRaws.after.slice(firstWhitespaceIndex) : '';
+						const newlineIndex = newlineAfter.search(/\r?\n/);
+
+						statementRaws.after =
+								newlineIndex >= 0 ?
+									newlineBefore + newlineAfter.slice(newlineIndex)
+									: newlineBefore + context.newline + newlineAfter;
+
+						return;
+					}
+
+					if (primary === 'never-multi-line') {
+						statementRaws.after = statementRaws.after.replace(/\s/g, '');
+
+						return;
+					}
+				}
 			});
 		}
 	}

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/block-closing-brace-space-before/index.mjs

@@ -28,7 +28,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) => {
@@ -74,31 +74,30 @@ const rule = (primary, _secondaryOptions, context) => {
 				source,
 				index: source.length - 1,
 				err: (message) => {
-					if (context.fix) {
-						const statementRaws = statement.raws;
-
-						if (typeof statementRaws.after !== 'string') { return; }
-
-						if (primary.startsWith('always')) {
-							statementRaws.after = statementRaws.after.replace(/\s*$/, ' ');
-
-							return;
-						}
-
-						if (primary.startsWith('never')) {
-							statementRaws.after = statementRaws.after.replace(/\s*$/, '');
-
-							return;
-						}
-					}
-
 					report({
 						message,
 						node: statement,
 						index,
 						endIndex: index,
 						result,
-						ruleName
+						ruleName,
+						fix: () => {
+							const statementRaws = statement.raws;
+
+							if (typeof statementRaws.after !== 'string') { return; }
+
+							if (primary.startsWith('always')) {
+								statementRaws.after = statementRaws.after.replace(/\s*$/, ' ');
+
+								return;
+							}
+
+							if (primary.startsWith('never')) {
+								statementRaws.after = statementRaws.after.replace(/\s*$/, '');
+
+								return;
+							}
+						}
 					});
 				}
 			});

package/stylelint/plugins/stylelint-15.11.0-stylistic/rules/block-opening-brace-newline-after/index.mjs

@@ -108,65 +108,62 @@ const rule = (primary, secondaryOptions, context) => {
 				index: -1,
 				lineCheckStr: blockString(statement),
 				err: (m) => {
-					if (context.fix) {
-						const nodeToCheckRaws = nodeToCheck.raws;
-
-						if (typeof nodeToCheckRaws.before !== 'string') { return; }
+					report({
+						message: m,
+						node: statement,
+						index: beforeBlockString(statement, { noRawBefore: true }).length + 1,
+						endIndex: beforeBlockString(statement, { noRawBefore: true }).length + 1,
+						result,
+						ruleName,
+						fix: () => {
+							const nodeToCheckRaws = nodeToCheck.raws;
 
-						if (primary.startsWith('always')) {
-							const index = nodeToCheckRaws.before.search(/\r?\n/);
+							if (typeof nodeToCheckRaws.before !== 'string') { return; }
 
-							nodeToCheckRaws.before =
-								index >= 0 ?
-									nodeToCheckRaws.before.slice(index)
-									: context.newline + nodeToCheckRaws.before;
+							if (primary.startsWith('always')) {
+								const index = nodeToCheckRaws.before.search(/\r?\n/);
 
-							backupCommentNextBefores.delete(nodeToCheck);
+								nodeToCheckRaws.before =
+									index >= 0 ?
+										nodeToCheckRaws.before.slice(index)
+										: context.newline + nodeToCheckRaws.before;
 
-							return;
-						}
+								backupCommentNextBefores.delete(nodeToCheck);
 
-						if (primary === 'never-multi-line') {
-							// Restore the `before` of the node next to the comment node.
-							for (const [node, before] of backupCommentNextBefores.entries()) {
-								node.raws.before = before;
+								return;
 							}
 
-							backupCommentNextBefores.clear();
+							if (primary === 'never-multi-line') {
+								// Restore the `before` of the node next to the comment node.
+								for (const [node, before] of backupCommentNextBefores.entries()) {
+									node.raws.before = before;
+								}
 
-							// Fix
-							const reNewLine = /\r?\n/;
-							let fixTarget = statement.first;
+								backupCommentNextBefores.clear();
 
-							while (fixTarget) {
-								const fixTargetRaws = fixTarget.raws;
+								// Fix
+								const reNewLine = /\r?\n/;
+								let fixTarget = statement.first;
 
-								if (typeof fixTargetRaws.before !== 'string') { continue; }
+								while (fixTarget) {
+									const fixTargetRaws = fixTarget.raws;
 
-								if (reNewLine.test(fixTargetRaws.before || '')) {
-									fixTargetRaws.before = fixTargetRaws.before.replace(/\r?\n/g, '');
-								}
+									if (typeof fixTargetRaws.before !== 'string') { continue; }
 
-								if (fixTarget.type !== 'comment') {
-									break;
-								}
+									if (reNewLine.test(fixTargetRaws.before || '')) {
+										fixTargetRaws.before = fixTargetRaws.before.replace(/\r?\n/g, '');
+									}
 
-								fixTarget = fixTarget.next();
-							}
+									if (fixTarget.type !== 'comment') {
+										break;
+									}
 
-							nodeToCheckRaws.before = '';
+									fixTarget = fixTarget.next();
+								}
 
-							return;
+								nodeToCheckRaws.before = '';
+							}
 						}
-					}
-
-					report({
-						message: m,
-						node: statement,
-						index: beforeBlockString(statement, { noRawBefore: true }).length + 1,
-						endIndex: beforeBlockString(statement, { noRawBefore: true }).length + 1,
-						result,
-						ruleName
 					});
 				}
 			});