@html-validate/eslint-config 5.29.0 → 6.0.0

package/{eslint.js → eslint.mjs}

@@ -1,8 +1,10 @@
 #!/usr/bin/env node
 
-const path = require("path");
-const { spawn } = require("child_process");
+import path from "node:path";
+import { spawn } from "node:child_process";
+import { createRequire } from "node:module";
 
+const require = createRequire(import.meta.url);
 const pkgPath = path.dirname(require.resolve("eslint/package.json"));
 const binary = path.join(pkgPath, "bin/eslint");
 

package/index.mjs

@@ -24,6 +24,7 @@ function defineConfig(config) {
 
 export default [
 	defineConfig({
+		name: "@html-validate/eslint-config/language-options",
 		languageOptions: {
 			ecmaVersion: 2023,
 			sourceType: "module",
@@ -40,6 +41,7 @@ export default [
 	}),
 
 	defineConfig({
+		name: "@html-validate/eslint-config/base",
 		plugins: {
 			"eslint-comments": eslintPluginEslintComments,
 			prettier: eslintPluginPrettier,
@@ -149,11 +151,13 @@ export default [
 
 	defineConfig({
 		/* ensure all of these patterns are linted */
+		name: "@html-validate/eslint-config/extensions",
 		files: ["**/*.js", "**/*.cjs", "**/*.mjs"],
 	}),
 
 	defineConfig({
 		/* mjs requires file extension */
+		name: "@html-validate/eslint-config/esm",
 		files: ["**/*.mjs"],
 		rules: {
 			"import/extensions": [
@@ -165,4 +169,15 @@ export default [
 			],
 		},
 	}),
+
+	defineConfig({
+		/* files which should lint even if project isn't build yet */
+		name: "@html-validate/eslint-config/dist",
+		files: ["./*.d.ts", ".htmlvalidate.{js,mjs,cjs}", "./bin/*.{js,mjs,cjs}"],
+		rules: {
+			"import/export": "off",
+			"import/extensions": "off",
+			"import/no-unresolved": "off",
+		},
+	}),
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@html-validate/eslint-config",
-  "version": "5.29.0",
+  "version": "6.0.0",
   "description": "Eslint sharable config used by the various HTML-validate packages",
   "keywords": [
     "eslint"
@@ -16,47 +16,46 @@
   },
   "license": "MIT",
   "author": "David Sveningsson <ext@sidvind.com>",
+  "type": "module",
   "main": "index.mjs",
   "bin": {
-    "eslint": "eslint.js",
-    "eslint-config": "cli.js"
+    "eslint": "eslint.mjs",
+    "eslint-config": "./cli.mjs"
   },
   "files": [
     "dist",
-    "patch",
     "template",
-    "*.js",
-    "*.cjs",
-    "*.mjs",
-    "!*.spec.js"
+    "eslint.mjs",
+    "cli.mjs"
   ],
   "scripts": {
     "prebuild": "tsc",
-    "build": "esbuild src/cli.ts --bundle --sourcemap --platform=node --target=node18 --external:prettier --outdir=dist",
+    "build": "node build.mjs",
     "prepack": "release-prepack --retain-scripts",
     "postpack": "release-postpack",
     "prepublishOnly": "release-prepublish --retain-scripts"
   },
   "dependencies": {
-    "@eslint/js": "9.21.0",
-    "@rushstack/eslint-patch": "1.11.0",
-    "eslint": "8.57.1",
+    "@eslint/js": "9.22.0",
+    "argparse": "2.0.1",
+    "eslint": "9.22.0",
     "eslint-config-prettier": "10.1.1",
-    "eslint-config-sidvind": "1.3.2",
     "eslint-formatter-gitlab": "5.1.0",
     "eslint-import-resolver-node": "0.3.9",
-    "eslint-import-resolver-typescript": "3.8.7",
-    "eslint-plugin-array-func": "4.0.0",
+    "eslint-import-resolver-typescript": "3.9.0",
+    "eslint-plugin-array-func": "5.0.2",
     "eslint-plugin-eslint-comments": "3.2.0",
     "eslint-plugin-import": "2.31.0",
     "eslint-plugin-n": "17.16.2",
     "eslint-plugin-prettier": "5.2.3",
     "eslint-plugin-security": "3.0.1",
     "eslint-plugin-sonarjs": "3.0.2",
-    "globals": "16.0.0"
+    "find-up": "7.0.0",
+    "globals": "16.0.0",
+    "nunjucks": "3.2.4"
   },
   "peerDependencies": {
-    "prettier": "^3"
+    "prettier": "^3.0.0"
   },
   "engines": {
     "node": ">= 20.9.0",
@@ -65,5 +64,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "ca95247b7c60f1ba1a30795bece5fa762670d917"
+  "gitHead": "f54dcbae0ccae318895e34d32fbfa02b05cd38d1"
 }

package/template/eslint.config.mjs.njk

@@ -35,10 +35,19 @@ const rootDir = path.dirname(fileURLToPath(import.meta.url));
 export default [
 	{
 		name: "Ignored files",
-		ignores: ["**/coverage/**", "**/dist/**", "**/node_modules/**", "**/public/assets/**", "**/temp/**"],
+		ignores: [
+			"**/coverage/**",
+			"**/dist/**",
+			"**/node_modules/**",
+			"**/out/**",
+			"**/public/assets/**",
+			"**/temp/**"
+		],
 	},
+
 	...defaultConfig,
 {% if typescript %}
+
 	{
 		name: "@html-validate/eslint-config-typescript",
 		files: ["**/*.ts"],
@@ -46,6 +55,7 @@ export default [
 	},
 {% endif %}
 {% if typeinfo %}
+
 	{
 		name: "@html-validate/eslint-config-typeinfo",
 		files: ["src/**/*.ts"],
@@ -60,6 +70,7 @@ export default [
 	},
 {% endif %}
 {% if angularjs %}
+
 	{
 		name: "@html-validate/eslint-config-angularjs",
 		files: ["app/**/*.[jt]s", "src/**/*.[jt]s"],
@@ -67,6 +78,7 @@ export default [
 	},
 {% endif %}
 {% if vue %}
+
 	{
 		name: "@html-validate/eslint-config-vue",
 		files: ["**/*.vue"],
@@ -74,6 +86,7 @@ export default [
 	},
 {% endif %}
 {% if jest %}
+
 	{
 		name: "@html-validate/eslint-config-jest",
 		files: ["**/*.spec.[jt]s"],
@@ -82,6 +95,7 @@ export default [
 	},
 {% endif %}
 {% if cypress %}
+
 	{
 		name: "@html-validate/eslint-config-cypress",
 		files: ["cypress/**/*.spec.[jt]s", "cypress/**/*.cy.[jt]s", "src/**/*.cy.[jt]s"],
@@ -89,19 +103,11 @@ export default [
 	},
 {% endif %}
 {% if protractor %}
+
 	{
 		name: "@html-validate/eslint-config-protractor",
 		files: ["tests/e2e/**/*.spec.[jt]s"],
 		...protractorConfig,
 	},
 {% endif %}
-	{
-		/* files which should lint even if project isn't build yet */
-		files: ["./*.d.ts", "bin/*.js"],
-		rules: {
-			"import/export": "off",
-			"import/extensions": "off",
-			"import/no-unresolved": "off",
-		},
-	},
 ];

package/cli.js

@@ -1,2 +0,0 @@
-#!/usr/bin/env node
-module.exports = require("./dist/cli");

package/legacy.cjs

@@ -1,95 +0,0 @@
-module.exports = {
-	env: {
-		node: true,
-	},
-
-	parserOptions: {
-		/* force version, some plugins tries to overwrite this */
-		ecmaVersion: 2023,
-	},
-
-	settings: {
-		"import/resolver": {
-			[require.resolve("eslint-import-resolver-node")]: true,
-			[require.resolve("eslint-import-resolver-typescript")]: true,
-		},
-	},
-
-	extends: [
-		require.resolve("eslint-config-sidvind/es2017"),
-		"plugin:eslint-comments/recommended",
-		"plugin:prettier/recommended",
-		"plugin:import/errors",
-		"plugin:n/recommended-module",
-		"plugin:array-func/recommended",
-		"plugin:security/recommended-legacy",
-		"plugin:sonarjs/recommended-legacy",
-	],
-
-	plugins: ["array-func", "prettier", "import", "n", "security", "sonarjs"],
-
-	rules: {
-		"eslint-comments/disable-enable-pair": ["error", { allowWholeFile: true }],
-		"eslint-comments/require-description": [
-			"error",
-			{ ignore: ["eslint-enable", "eslint-env", "exported", "global", "globals"] },
-		],
-		"eslint-comments/no-unused-disable": "error",
-
-		"import/default": "off",
-		"import/extensions": ["error", "never", { json: "always" }],
-		"import/newline-after-import": "error",
-		"import/no-absolute-path": "error",
-		"import/no-deprecated": "error",
-		"import/no-dynamic-require": "error",
-		"import/no-extraneous-dependencies": "error",
-		"import/no-mutable-exports": "error",
-		"import/no-named-default": "error",
-		"import/no-useless-path-segments": "error",
-		"import/order": "error",
-		"import/no-named-as-default": "error",
-		"import/no-named-as-default-member": "error",
-		"import/no-duplicates": "error",
-
-		/* this is checked by compiler and without additional configuration does not
-		 * work with typescript */
-		"n/no-missing-import": "off",
-		"n/no-missing-require": "off",
-
-		"security/detect-child-process": "off", // produces more noise than useful errors
-		"security/detect-non-literal-fs-filename": "off", // html-validate reads files, don't want to acknowledge all occurrences
-		"security/detect-non-literal-regexp": "error",
-		"security/detect-non-literal-require": "error",
-		"security/detect-object-injection": "off", // produces more noise than useful errors
-		"security/detect-unsafe-regex": "error",
-
-		"sonarjs/cognitive-complexity": "off", // already covered by native complexity rule
-		"sonarjs/deprecation": "off", // already covered by @typescript-eslint/no-deprecated
-		"sonarjs/function-return-type": "off", // overly broad and opinionated, let typescript deal with this
-		"sonarjs/no-empty-test-file": "off", // could be useful but it does not handle it.each or similar constructs thus yields more false positives than its worth */
-		"sonarjs/no-small-switch": "off", // prefer to use small switches when the intention is to all more cases later
-		"sonarjs/no-unused-vars": "off", // already coveredby @typescript-eslint/no-unused-vars
-		"sonarjs/unused-import": "off", // already covered by @typescript-eslint/no-unused-vars
-		"sonarjs/prefer-nullish-coalescing": "off", // requires typescript and strictNullChecks, which is sane, but we also use @typescript-eslint/prefer-nullish-coalescing so this becomes redundant
-		"sonarjs/prefer-single-boolean-return": "off", // prefer to use multiple returns even for booleans (looks better and can yield performance increase
-		"sonarjs/no-control-regex": "off", // already covered by no-control-regexp
-		"sonarjs/prefer-regexp-exec": "off", // prefer @typescript-eslint/prefer-regexp-exec
-		"sonarjs/todo-tag": "off", // want to be able to leave todo tasks
-		"sonarjs/void-use": "off", // used to silence warnings about unawaited promises
-
-		"consistent-this": "off",
-		"no-console": "warn",
-		"no-dupe-class-members": "off",
-		"no-undef": "off",
-		"no-unused-vars": [
-			"error",
-			{
-				ignoreRestSiblings: true,
-				argsIgnorePattern: "^_",
-			},
-		],
-		"object-shorthand": "error",
-		"prettier/prettier": "warn",
-		strict: "off",
-	},
-};

package/patch/modern-module-resolution.js

@@ -1 +0,0 @@
-require("@rushstack/eslint-patch/modern-module-resolution");

package/template/eslintignore

@@ -1,5 +0,0 @@
-coverage/
-dist/
-node_modules/
-public/assets/
-temp/

package/template/eslintrc.js.njk

@@ -1,82 +0,0 @@
-/* This file is managed by @html-validate/eslint-config */
-/* Changes may be overwritten */
-
-{% if typeinfo and typeinfo.rootDir != "." %}
-const path = require("path");
-
-{% endif  %}
-require("@html-validate/eslint-config/patch/modern-module-resolution");
-
-module.exports = {
-	root: true,
-	extends: [require.resolve("@html-validate/eslint-config/legacy.cjs")],
-
-	overrides: [
-		{
-			/* ensure cjs and mjs files are linted too */
-			files: ["*.cjs", "*.mjs"]
-		},
-{% if typescript %}
-		{
-			files: "*.ts",
-			extends: [require.resolve("@html-validate/eslint-config-typescript/legacy.cjs")],
-		},
-{% endif %}
-{% if typeinfo %}
-		{
-			files: ["src/**/*.ts"],
-			excludedFiles: ["src/**/*.spec.ts"],
-			parserOptions: {
-				tsconfigRootDir: {% if typeinfo.rootDir == "." %}__dirname{% else %}path.join(__dirname, "{{ typeinfo.rootDir }}"){% endif %},
-				project: ["{{ typeinfo.tsconfig }}"],
-			},
-			extends: [require.resolve("@html-validate/eslint-config-typescript-typeinfo/legacy.cjs")],
-		},
-{% endif %}
-{% if angularjs %}
-		{
-			files: ["app/**/*.[jt]s", "src/**/*.[jt]s"],
-			extends: [require.resolve("@html-validate/eslint-config-angularjs/legacy.cjs")],
-		},
-{% endif %}
-{% if vue %}
-		{
-			files: "*.vue",
-			extends: [require.resolve("@html-validate/eslint-config-vue/legacy.cjs")],
-		},
-{% endif %}
-{% if jest %}
-		{
-			files: "*.spec.[jt]s",
-			{%- if angularjs %}
-			env: {
-				"angular/mocks": true,
-			},
-			{%- endif %}
-			excludedFiles: ["cypress/**", "tests/e2e/**"],
-			extends: [require.resolve("@html-validate/eslint-config-jest/legacy.cjs")],
-		},
-{% endif %}
-{% if cypress %}
-		{
-			files: ["cypress/**/*.spec.[jt]s", "cypress/**/*.cy.[jt]s"],
-			extends: [require.resolve("@html-validate/eslint-config-cypress/legacy.cjs")],
-		},
-{% endif %}
-{% if protractor %}
-		{
-			files: "tests/e2e/**/*.spec.[jt]s",
-			extends: [require.resolve("@html-validate/eslint-config-protractor/legacy.cjs")],
-		},
-{% endif %}
-		{
-			/* files which should lint even if project isn't build yet */
-			files: ["./*.d.ts", "bin/*.js"],
-			rules: {
-				"import/export": "off",
-				"import/extensions": "off",
-				"import/no-unresolved": "off",
-			},
-		},
-	],
-};