linter-bundle 3.8.0 → 3.10.0

package/CHANGELOG.md

@@ -6,7 +6,27 @@ 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.8.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.10.0...HEAD)
+
+## [3.10.0] - 2023-08-29
+
+### Changed
+
+- [eslint] Updated `@typescript-eslint` from `6.4.1` to `6.5.0`
+
+### Fixed
+
+- [eslint] Don't consider files outside of `basePath` in [`restricted-filenames`](./eslint/rules/restricted-filenames.md) rule
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.9.0...v3.10.0)
+
+## [3.9.0] - 2023-08-28
+
+### Changed
+
+- [general] Use users default shell for linting sub-processes
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v3.8.0...v3.9.0)
 
 ## [3.8.0] - 2023-08-28
 

package/eslint/index.js

@@ -5,8 +5,8 @@
 /* eslint-disable max-lines -- The rules can be easier managed if they are all in one file */
 /* eslint-disable n/no-process-env -- `process.env` is required to inject configuration adjustments */
 
-const fs = require('fs');
-const path = require('path');
+const fs = require('node:fs');
+const path = require('node:path');
 
 const ensureType = require('../helper/ensure-type');
 

package/eslint/rules/restricted-filenames.js

@@ -2,7 +2,7 @@
  * @file ESLint rule which ensures that only files which match given glob patterns are part of your project.
  */
 
-const path = require('path');
+const path = require('node:path');
 
 const micromatch = require('micromatch');
 
@@ -46,52 +46,58 @@ module.exports = {
 			]
 		}
 	},
-	create: (context) => ({
-		Program: (node) => {
-			const filePath = context.getFilename();
-			/** @type {{ basePath: string, allowed?: string[]; disallowed?: string[]; }[]} */
-			const options = context.options;
+	create: (context) => {
+		const filePath = context.getFilename();
+		/** @type {{ basePath: string, allowed?: string[]; disallowed?: string[]; }[]} */
+		const options = context.options;
 
-			for (const { basePath, allowed, disallowed } of options) {
-				const normalizedName = path.relative(path.join(process.cwd(), basePath), filePath);
+		for (const { basePath, allowed, disallowed } of options) {
+			const normalizedName = path.relative(path.join(process.cwd(), basePath), filePath);
 
-				if (allowed && !disallowed) {
-					if (!micromatch.isMatch(normalizedName, allowed, { dot: true })) {
-						report();
-						return;
-					}
+			if (normalizedName.startsWith('..')) {
+				continue;
+			}
+
+			if (allowed && !disallowed) {
+				if (!micromatch.isMatch(normalizedName, allowed, { dot: true })) {
+					return report();
 				}
-				else if (!allowed && disallowed) {
-					if (micromatch.isMatch(normalizedName, disallowed, { dot: true })) {
-						report();
-						return;
-					}
+			}
+			else if (!allowed && disallowed) {
+				if (micromatch.isMatch(normalizedName, disallowed, { dot: true })) {
+					return report();
 				}
-				else if (allowed && disallowed) {
-					if (
-						micromatch.isMatch(normalizedName, disallowed, { dot: true }) ||
-						!micromatch.isMatch(normalizedName, allowed, { dot: true })
-					) {
-						report();
-						return;
-					}
+			}
+			else if (allowed && disallowed) {
+				if (
+					micromatch.isMatch(normalizedName, disallowed, { dot: true }) ||
+					!micromatch.isMatch(normalizedName, allowed, { dot: true })
+				) {
+					return report();
 				}
 			}
+		}
 
-			/**
-			 * Reports to ESLint.
-			 *
-			 * @returns {void}
-			 */
-			function report () {
-				context.report({
-					node,
-					messageId: 'text',
-					data: {
-						name: path.relative(process.cwd(), filePath).replace(/\\/gu, '/')
-					}
-				});
-			}
+		return {};
+
+		/**
+		 * Add rule listener which reports the error.
+		 *
+		 * @returns {import('eslint').Rule.RuleListener} Returns a `Program` rule lister which reports the error.
+		 */
+		function report () {
+			return {
+				Program: (node) => {
+					context.report({
+						node,
+						loc: { line: 1, column: 0 },
+						messageId: 'text',
+						data: {
+							name: path.relative(process.cwd(), filePath).replace(/\\/gu, '/')
+						}
+					});
+				}
+			};
 		}
-	})
+	}
 };

package/helper/find-missing-overrides.js

@@ -5,8 +5,8 @@
  * @see https://classic.yarnpkg.com/en/docs/selective-version-resolutions/
  */
 
-const fs = require('fs');
-const path = require('path');
+const fs = require('node:fs');
+const path = require('node:path');
 
 /** @typedef {{ name: string; configuredVersion: string; expectedVersion: string; }} Dependency */
 

package/helper/get-stylelint-path.js

@@ -2,8 +2,8 @@
  * @file Returns the path to the Stylelint CLI script.
  */
 
-const fs = require('fs');
-const path = require('path');
+const fs = require('node:fs');
+const path = require('node:path');
 
 /**
  * Returns if the project is using npm or yarn.

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

@@ -2,8 +2,8 @@
  * @file Check if the project is using npm or yarn by checking the existence of a `package-lock.json` or a `yarn.lock`.
  */
 
-const fs = require('fs');
-const path = require('path');
+const fs = require('node:fs');
+const path = require('node:path');
 
 /**
  * Returns if the project is using npm or yarn.

package/helper/run-process.js

@@ -4,7 +4,8 @@
 
 /** @typedef {{ code: number; stdout: string; stderr: string; runtime: number; }} ProcessResult */
 
-const childProcess = require('child_process');
+const childProcess = require('node:child_process');
+const os = require('node:os');
 
 /**
  * Executes a process asynchronously.
@@ -24,7 +25,7 @@ async function runProcess (command, options) {
 		/** @type {string[]} */
 		const stderr = [];
 
-		const lintingProcess = childProcess.exec(command, options);
+		const lintingProcess = childProcess.exec(command, { ...options, shell: os.userInfo().shell });
 
 		lintingProcess.stdout?.on('data', (/** @type {string} */data) => {
 			stdout.push(data);

package/helper/validate-package-overrides.js

@@ -5,8 +5,8 @@
  * @see https://classic.yarnpkg.com/en/docs/selective-version-resolutions/
  */
 
-const fs = require('fs');
-const path = require('path');
+const fs = require('node:fs');
+const path = require('node:path');
 
 /** @typedef {{ name: string; configuredVersion: string; expectedVersion: string; }} Dependency */
 

package/lint.js

@@ -4,8 +4,8 @@
  * @file Entry point of the linter-bundle.
  */
 
-const path = require('path');
-const tty = require('tty');
+const path = require('node:path');
+const tty = require('node:tty');
 
 const micromatch = require('micromatch');
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "3.8.0",
+	"version": "3.10.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -40,9 +40,9 @@
 		"_validate-stylelint-options": "node ./validate-stylelint-options.mjs"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "6.4.1",
-		"@typescript-eslint/parser": "6.4.1",
-		"@typescript-eslint/utils": "6.4.1",
+		"@typescript-eslint/eslint-plugin": "6.5.0",
+		"@typescript-eslint/parser": "6.5.0",
+		"@typescript-eslint/utils": "6.5.0",
 		"eslint": "8.48.0",
 		"eslint-import-resolver-typescript": "3.6.0",
 		"eslint-import-resolver-webpack": "0.13.7",

package/stylelint/index.js

@@ -4,7 +4,7 @@
 
 /* eslint-disable max-lines -- The rules can be easier managed if they are all in one file  */
 
-const path = require('path');
+const path = require('node:path');
 
 module.exports = {
 	reportNeedlessDisables: true,