fix(eslint): migrate to stylistic.configs.customize() after plugin update

- Replace deprecated string-based extends syntax ("stylistic/all") with
  stylistic.configs.customize() factory function
- Move commaDangle configuration into customize() options for cleaner config
- Fixes padded-blocks and other style rules being incorrectly applied after
  updating @stylistic/eslint-plugin from 5.x to 5.7.1

The string-based extends syntax stopped working with the latest update
when used with defineConfig().

Author: KristjanESPERANTO

eslint.config.mjs

@@ -1,6 +1,6 @@
-import {defineConfig, globalIgnores} from "eslint/config";
+import { defineConfig, globalIgnores } from "eslint/config";
 import globals from "globals";
-import {flatConfigs as importX} from "eslint-plugin-import-x";
+import { flatConfigs as importX } from "eslint-plugin-import-x";
 import js from "@eslint/js";
 import jsdocPlugin from "eslint-plugin-jsdoc";
 import packageJson from "eslint-plugin-package-json";
@@ -24,26 +24,25 @@ export default defineConfig([
 				moment: "readonly"
 			}
 		},
-		plugins: {js, stylistic},
-		extends: [importX.recommended, "js/recommended", jsdocPlugin.configs["flat/recommended"], "stylistic/all"],
+		plugins: { js, stylistic },
+		extends: [importX.recommended, "js/recommended", jsdocPlugin.configs["flat/recommended"], stylistic.configs.customize({ indent: "tab", quotes: "double", semi: true, commaDangle: "never" })],
 		rules: {
 			"@stylistic/array-element-newline": ["error", "consistent"],
 			"@stylistic/arrow-parens": ["error", "always"],
 			"@stylistic/brace-style": "off",
-			"@stylistic/comma-dangle": ["error", "never"],
 			"@stylistic/dot-location": ["error", "property"],
 			"@stylistic/function-call-argument-newline": ["error", "consistent"],
 			"@stylistic/function-paren-newline": ["error", "consistent"],
 			"@stylistic/implicit-arrow-linebreak": ["error", "beside"],
 			"@stylistic/indent": ["error", "tab"],
-			"@stylistic/max-statements-per-line": ["error", {max: 2}],
+			"@stylistic/max-statements-per-line": ["error", { max: 2 }],
 			"@stylistic/multiline-comment-style": "off",
 			"@stylistic/multiline-ternary": ["error", "always-multiline"],
-			"@stylistic/newline-per-chained-call": ["error", {ignoreChainWithDepth: 4}],
+			"@stylistic/newline-per-chained-call": ["error", { ignoreChainWithDepth: 4 }],
 			"@stylistic/no-extra-parens": "off",
 			"@stylistic/no-tabs": "off",
 			"@stylistic/object-curly-spacing": ["error", "always"],
-			"@stylistic/object-property-newline": ["error", {allowAllPropertiesOnSameLine: true}],
+			"@stylistic/object-property-newline": ["error", { allowAllPropertiesOnSameLine: true }],
 			"@stylistic/operator-linebreak": ["error", "before"],
 			"@stylistic/padded-blocks": "off",
 			"@stylistic/quote-props": ["error", "as-needed"],
@@ -86,11 +85,11 @@ export default defineConfig([
 			"js/logger.js",
 			"tests/**/*.js"
 		],
-		rules: {"no-console": "error"}
+		rules: { "no-console": "error" }
 	},
 	{
 		files: ["**/package.json"],
-		plugins: {packageJson},
+		plugins: { packageJson },
 		extends: ["packageJson/recommended"]
 	},
 	{
@@ -102,15 +101,15 @@ export default defineConfig([
 			},
 			sourceType: "module"
 		},
-		plugins: {js, stylistic},
-		extends: [importX.recommended, "js/all", "stylistic/all"],
+		plugins: { js, stylistic },
+		extends: [importX.recommended, "js/all", stylistic.configs.customize({ indent: "tab", quotes: "double", semi: true, commaDangle: "never" })],
 		rules: {
 			"@stylistic/array-element-newline": "off",
 			"@stylistic/indent": ["error", "tab"],
-			"@stylistic/object-property-newline": ["error", {allowAllPropertiesOnSameLine: true}],
+			"@stylistic/object-property-newline": ["error", { allowAllPropertiesOnSameLine: true }],
 			"@stylistic/padded-blocks": ["error", "never"],
 			"@stylistic/quote-props": ["error", "as-needed"],
-			"import-x/no-unresolved": ["error", {ignore: ["eslint/config"]}],
+			"import-x/no-unresolved": ["error", { ignore: ["eslint/config"] }],
 			"max-lines-per-function": ["error", 100],
 			"no-magic-numbers": "off",
 			"one-var": ["error", "never"],
@@ -124,7 +123,7 @@ export default defineConfig([
 				...vitest.environments.env.globals
 			}
 		},
-		plugins: {vitest},
+		plugins: { vitest },
 		extends: [vitest.configs.recommended],
 		rules: {
 			"vitest/consistent-test-it": "error",
@@ -142,7 +141,7 @@ export default defineConfig([
 					]
 				}
 			],
-			"vitest/max-nested-describe": ["error", {max: 3}],
+			"vitest/max-nested-describe": ["error", { max: 3 }],
 			"vitest/prefer-to-be": "error",
 			"vitest/prefer-to-have-length": "error"
 		}

modules/default/clock/clock.js

@@ -190,7 +190,7 @@ Module.register("clock", {
 			}
 
 			sunWrapperInnerHTML += `<span><i class="fas fa-arrow-up" aria-hidden="true"></i> ${formatTime(this.config, sunTimes.sunrise)}</span>`
-			  + `<span><i class="fas fa-arrow-down" aria-hidden="true"></i> ${formatTime(this.config, sunTimes.sunset)}</span>`;
+				+ `<span><i class="fas fa-arrow-down" aria-hidden="true"></i> ${formatTime(this.config, sunTimes.sunset)}</span>`;
 
 			sunWrapper.innerHTML = sunWrapperInnerHTML;
 			digitalWrapper.appendChild(sunWrapper);
@@ -218,8 +218,8 @@ Module.register("clock", {
 
 			moonWrapper.innerHTML
 				= `<span class="${isVisible ? "bright" : ""}">${image} ${showFraction ? illuminatedFractionString : ""}</span>`
-				  + `<span><i class="fas fa-arrow-up" aria-hidden="true"></i> ${moonRise ? formatTime(this.config, moonRise) : "..."}</span>`
-				  + `<span><i class="fas fa-arrow-down" aria-hidden="true"></i> ${moonSet ? formatTime(this.config, moonSet) : "..."}</span>`;
+					+ `<span><i class="fas fa-arrow-up" aria-hidden="true"></i> ${moonRise ? formatTime(this.config, moonRise) : "..."}</span>`
+					+ `<span><i class="fas fa-arrow-down" aria-hidden="true"></i> ${moonSet ? formatTime(this.config, moonSet) : "..."}</span>`;
 			digitalWrapper.appendChild(moonWrapper);
 		}
 

tests/configs/modules/calendar/symboltest.js

@@ -1,4 +1,3 @@
-
 let config = {
 	address: "0.0.0.0",
 	ipWhitelist: [],

tests/e2e/helpers/global-setup.js

@@ -88,7 +88,6 @@ exports.getPage = () => {
 	return page;
 };
 
-
 exports.startApplication = async (configFilename, exec) => {
 	vi.resetModules();
 

tests/unit/functions/http_fetcher_spec.js

@@ -440,4 +440,3 @@ describe("fetch() method", () => {
 		expect(errorInfo.errorType).toBe("NETWORK_ERROR");
 	});
 });
-

vitest.config.mjs

@@ -1,4 +1,4 @@
-import {defineConfig} from "vitest/config";
+import { defineConfig } from "vitest/config";
 
 /*
  * Sequential execution keeps our shared test server stable: