linter-bundle 3.3.0 → 3.4.0

package/CHANGELOG.md

@@ -6,7 +6,25 @@ 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/v3.3.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.4.0...HEAD)
+
+## [3.4.0] - 2023-07-20
+
+### Changed
+
+- [eslint] Updated `@typescript-eslint` from `6.0.0` to `6.1.0`
+- [eslint] Updated `eslint` from `8.44.0` to `8.45.0`
+- [eslint] Updated `eslint-plugin-jsdoc` from `46.4.3` to `46.4.4`
+- [eslint] Updated `eslint-plugin-unicorn` from `47.0.0` to `48.0.0`
+- [stylelint] Updated `stylelint` from `15.10.1` to `15.10.2`
+- [eslint] Moved `unicorn/import-style` configuration for `path` (unassigned: false, default: false, namespace: true, named: true) from `overrides-storybook` into the main configuration.
+- [eslint/overrides-javascript] Activated [`unicorn/prefer-top-level-await`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-top-level-await.md) rule for `*.mjs` files.
+
+### Fixed
+
+- [stylelint] Fixed `Cannot find module 'node_modules/stylelint/bin/stylelint.js'` issue, if the file is called `stylelint.mjs`
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.3.0...v3.4.0)
 
 ## [3.3.0] - 2023-07-13
 

package/eslint/index.js

@@ -941,7 +941,16 @@ module.exports = {
 						/([A-Za-z0-9]?[a-z0-9]+)*[A-Z]{2,4}([A-Za-z0-9]?[a-z0-9]+)*/u // Up to 4 Characters can be upper-case in a row (e.g. in `prepareDOM` or `SVGIcon`)
 					]
 				}],
-				'unicorn/import-style': 'error',
+				'unicorn/import-style': ['error', {
+					styles: {
+						path: {
+							unassigned: false,
+							default: false,
+							namespace: true,
+							named: true
+						}
+					}
+				}],
 				'unicorn/new-for-builtins': 'error',
 				'unicorn/no-abusive-eslint-disable': 'error',
 				'unicorn/no-array-callback-reference': 'off', // If I use functions, they are the best option for this use-case
@@ -972,7 +981,6 @@ module.exports = {
 				'unicorn/no-unnecessary-await': 'error',
 				'unicorn/no-unreadable-array-destructuring': 'error',
 				'unicorn/no-unreadable-iife': 'error',
-				'unicorn/no-unsafe-regex': 'off',
 				'unicorn/no-unused-properties': 'error',
 				'unicorn/no-useless-fallback-in-spread': 'error',
 				'unicorn/no-useless-promise-resolve-reject': 'error',
@@ -1021,13 +1029,13 @@ module.exports = {
 				'unicorn/prefer-set-has': 'error',
 				'unicorn/prefer-set-size': 'error',
 				'unicorn/prefer-spread': 'off', // @todo Disabled till there a solution for the warning, that `slice()` on Typed-Arrays should be replaced (which is not possible). @see https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1064
-				'unicorn/prefer-string-replace-all': 'off', // @todo Available since 2020 in browsers. SHould this be preferred?
+				'unicorn/prefer-string-replace-all': 'off', // @todo Available since 2020 in browsers. Should this be preferred?
 				'unicorn/prefer-string-slice': 'off', // @todo As of today (2020.08.24) and since the last 9 years, substr() is three times faster than slice() in Firefox.
 				'unicorn/prefer-string-starts-ends-with': 'error',
 				'unicorn/prefer-string-trim-start-end': 'error',
 				'unicorn/prefer-switch': 'error',
 				'unicorn/prefer-ternary': 'off', // We prefer readability over saving a few chars
-				'unicorn/prefer-top-level-await': 'off', // @todo There should be a detection that this option is only activated on Node.js v14.8 environments and above (Check for "engine" property in package.json, and "node -v")
+				'unicorn/prefer-top-level-await': 'off', // @todo Available since 2021. Activate in 2024
 				'unicorn/prefer-regexp-test': 'error',
 				'unicorn/prefer-type-error': 'error',
 				'unicorn/prevent-abbreviations': ['error', { ignore: ['args', 'i', 'j', 'i18n', 'ref', 'Ref', 'params', 'props', 'Props'] }],

package/eslint/overrides-javascript-lazy.js

@@ -7,11 +7,10 @@ const overridesJavaScript = require('./overrides-javascript');
 module.exports = {
 	...overridesJavaScript,
 	overrides: [
+		...overridesJavaScript.overrides,
 		{
-			...overridesJavaScript.overrides[0],
+			files: ['*.js', '*.cjs', '*.mjs'],
 			rules: {
-				...overridesJavaScript.overrides[0].rules,
-
 				/**
 				 * typescript-eslint
 				 *

package/eslint/overrides-javascript.js

@@ -103,6 +103,12 @@ module.exports = {
 				'unicorn/prevent-abbreviations': ['error', { ignore: ['args', 'dev', 'env', 'i', 'j', 'i18n', 'pkg', 'ref', 'Ref', 'req', 'res', 'setupDevServerMiddleware'] }],
 				'unicorn/text-encoding-identifier-case': 'off'
 			}
+		},
+		{
+			files: ['*.mjs'],
+			rules: {
+				'unicorn/prefer-top-level-await': 'error'
+			}
 		}
 	]
 };

package/eslint/overrides-storybook.js

@@ -23,23 +23,7 @@ module.exports = {
 				 *
 				 * @see https://github.com/import-js/eslint-plugin-import
 				 */
-				'import/no-nodejs-modules': 'off',
-
-				/**
-				 * eslint-plugin-unicorn
-				 *
-				 * @see https://github.com/sindresorhus/eslint-plugin-unicorn
-				 */
-				'unicorn/import-style': ['error', {
-					styles: {
-						path: {
-							unassigned: false,
-							default: false,
-							namespace: true,
-							named: true
-						}
-					}
-				}]
+				'import/no-nodejs-modules': 'off'
 			}
 		},
 		{

package/helper/ensure-type.js

@@ -5,6 +5,7 @@
 /**
  * Ensures that the input `value` is an array, otherwise an empty array is returned.
  *
+ * @public
  * @template T
  * @param {T} value - The value which should be checked.
  * @returns {T extends Array<any> ? T : []} Either the input array, or an empty array, if the input array is not an array.

package/helper/find-missing-overrides.js

@@ -13,6 +13,7 @@ const path = require('path');
 /**
  * Detects if installed versions of dependencies don't match to the required dependencies.
  *
+ * @public
  * @returns {Dependency[]} An array of missing overrides (=wrong versions).
  */
 function findMissingOverrides () {

package/helper/get-stylelint-path.js

@@ -0,0 +1,32 @@
+/**
+ * @file Returns the path to the Stylelint CLI script.
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+/**
+ * Returns if the project is using npm or yarn.
+ *
+ * @public
+ * @returns {string | null} Return the path to the Stylelint CLI script, or `null` if it can't be found.
+ */
+function getStylelintPath () {
+	const stylelintLibPath = path.dirname(require.resolve('stylelint'));
+
+	for (const stylelintBinPath of [
+		path.join(stylelintLibPath, '../bin/stylelint.mjs'),
+		path.join(stylelintLibPath, '../bin/stylelint.js')
+	]) {
+		if (fs.existsSync(stylelintBinPath)) {
+			return stylelintBinPath;
+		}
+	}
+
+	return null;
+}
+
+module.exports = {
+	getStylelintPath
+};
+

package/helper/is-npm-or-yarn.js

@@ -8,6 +8,7 @@ const path = require('path');
 /**
  * Returns if the project is using npm or yarn.
  *
+ * @public
  * @returns {'none' | 'npm' | 'yarn' | 'both'} Returns which package manager name.
  */
 function isNpmOrYarn () {

package/helper/run-process.js

@@ -9,6 +9,7 @@ const childProcess = require('child_process');
 /**
  * Executes a process asynchronously.
  *
+ * @public
  * @param {string} command - The process to execute.
  * @param {childProcess.ExecOptions | undefined} [options] - The options of the `childProcess.exec()` method.
  * @returns {Promise<ProcessResult>} An object containing the result of the process execution

package/helper/validate-package-overrides.js

@@ -13,6 +13,7 @@ const path = require('path');
 /**
  * Detects outdated "overrides"/"resolutions" dependencies.
  *
+ * @public
  * @returns {{ overrides: Dependency[]; resolutions: Dependency[]; }} Either the input array, or an empty array, if the input array is not an array.
  */
 function validatePackageOverrides () {

package/lint.js

@@ -10,6 +10,7 @@ const tty = require('tty');
 const micromatch = require('micromatch');
 
 const { findMissingOverrides } = require('./helper/find-missing-overrides.js');
+const { getStylelintPath } = require('./helper/get-stylelint-path.js');
 const { isNpmOrYarn } = require('./helper/is-npm-or-yarn.js');
 const { runProcess } = require('./helper/run-process.js');
 const { validatePackageOverrides } = require('./helper/validate-package-overrides.js');
@@ -115,10 +116,18 @@ void (async () => {
 					});
 				}
 
+				const stylelintBinPath = getStylelintPath();
+
+				if (stylelintBinPath === null) {
+					return generateDummyJobOutput(taskName, config, {
+						stderr: 'Stylelint CLI script not found.'
+					});
+				}
+
 				return runTask({
 					taskName,
 					config,
-					command: `node "${path.join(path.dirname(require.resolve('stylelint')), '../bin/stylelint.js')}" ${includes} --formatter unix`
+					command: `node "${stylelintBinPath}" ${includes} --formatter unix`
 				});
 			}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "3.3.0",
+	"version": "3.4.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -36,31 +36,31 @@
 		"preversion": "npm run check-outdated && npm run lint",
 		"postversion": "git push && git push --tags && npm publish",
 		"stylelint-find-rules": "stylelint-find-new-rules ./stylelint/index.js",
-		"validate-stylelint-options": "node ./validate-stylelint-options",
+		"validate-stylelint-options": "node ./validate-stylelint-options.mjs",
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "6.0.0",
-		"@typescript-eslint/parser": "6.0.0",
-		"@typescript-eslint/utils": "6.0.0",
-		"eslint": "8.44.0",
+		"@typescript-eslint/eslint-plugin": "6.1.0",
+		"@typescript-eslint/parser": "6.1.0",
+		"@typescript-eslint/utils": "6.1.0",
+		"eslint": "8.45.0",
 		"eslint-import-resolver-typescript": "3.5.5",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
 		"eslint-plugin-functional": "5.0.8",
 		"eslint-plugin-import": "2.27.5",
 		"eslint-plugin-jest": "27.2.3",
-		"eslint-plugin-jsdoc": "46.4.3",
+		"eslint-plugin-jsdoc": "46.4.4",
 		"eslint-plugin-jsx-a11y": "6.7.1",
 		"eslint-plugin-n": "16.0.1",
 		"eslint-plugin-promise": "6.1.1",
 		"eslint-plugin-react": "7.32.2",
 		"eslint-plugin-react-hooks": "4.6.0",
-		"eslint-plugin-unicorn": "47.0.0",
+		"eslint-plugin-unicorn": "48.0.0",
 		"markdownlint-cli": "0.35.0",
 		"micromatch": "4.0.5",
 		"postcss-scss": "4.0.6",
-		"stylelint": "15.10.1",
+		"stylelint": "15.10.2",
 		"stylelint-declaration-block-no-ignored-properties": "2.7.0",
 		"stylelint-order": "6.0.3",
 		"stylelint-scss": "5.0.1",