update ESLint to version 9

eslint.config.mjs

@@ -0,0 +1,187 @@
+import eslint from '@eslint/js';
+import tsEslint from 'typescript-eslint';
+import jestEslint from 'eslint-plugin-jest';
+import prettierRecommendedEslint from 'eslint-plugin-prettier/recommended';
+import globals from 'globals';
+
+const config = tsEslint.config(
+	{
+		languageOptions: {
+			sourceType: 'module',
+			globals: { ...globals.node },
+		},
+		linterOptions: {
+			noInlineConfig: false,
+			reportUnusedDisableDirectives: 'error',
+		},
+	},
+	eslint.configs.recommended,
+	{
+		rules: {
+			'constructor-super': 2,
+			curly: [2, 'all'],
+			// Unfortunatelly `curly` does not apply to blocks in `switch` cases so
+			// this is needed.
+			'no-restricted-syntax': [
+				2,
+				{
+					selector: 'SwitchCase > *.consequent[type!="BlockStatement"]',
+					message: 'Switch cases without blocks are disallowed',
+				},
+			],
+			'guard-for-in': 2,
+			'newline-after-var': 2,
+			'newline-before-return': 2,
+			'no-alert': 2,
+			'no-caller': 2,
+			'no-case-declarations': 2,
+			'no-catch-shadow': 2,
+			'no-class-assign': 2,
+			'no-console': 2,
+			'no-const-assign': 2,
+			'no-debugger': 2,
+			'no-dupe-args': 2,
+			'no-dupe-keys': 2,
+			'no-duplicate-case': 2,
+			'no-div-regex': 2,
+			'no-empty': [2, { allowEmptyCatch: true }],
+			'no-empty-pattern': 2,
+			'no-eval': 2,
+			'no-extend-native': 2,
+			'no-ex-assign': 2,
+			'no-extra-bind': 2,
+			'no-extra-boolean-cast': 2,
+			'no-extra-label': 2,
+			'no-fallthrough': 2,
+			'no-func-assign': 2,
+			'no-global-assign': 2,
+			'no-implicit-coercion': 2,
+			'no-implicit-globals': 2,
+			'no-inner-declarations': 2,
+			'no-invalid-regexp': 2,
+			'no-invalid-this': 2,
+			'no-irregular-whitespace': 2,
+			'no-lonely-if': 2,
+			'no-multi-str': 2,
+			'no-native-reassign': 2,
+			'no-negated-in-lhs': 2,
+			'no-new': 2,
+			'no-new-func': 2,
+			'no-new-wrappers': 2,
+			'no-obj-calls': 2,
+			'no-proto': 2,
+			'no-prototype-builtins': 0,
+			'no-redeclare': 2,
+			'no-regex-spaces': 2,
+			'no-restricted-imports': 2,
+			'no-return-assign': 2,
+			'no-self-assign': 2,
+			'no-self-compare': 2,
+			'no-sequences': 2,
+			'no-shadow': 2,
+			'no-shadow-restricted-names': 2,
+			'no-sparse-arrays': 2,
+			'no-this-before-super': 2,
+			'no-throw-literal': 2,
+			'no-undef': 2,
+			'no-unmodified-loop-condition': 2,
+			'no-unreachable': 2,
+			'no-unused-vars': [
+				2,
+				{ vars: 'all', args: 'after-used', caughtErrors: 'none' },
+			],
+			'no-use-before-define': 0,
+			'no-useless-call': 2,
+			'no-useless-computed-key': 2,
+			'no-useless-concat': 2,
+			'no-useless-rename': 2,
+			'no-var': 2,
+			'object-curly-newline': 0,
+			'prefer-const': 2,
+			'prefer-rest-params': 2,
+			'prefer-spread': 2,
+			'prefer-template': 2,
+			'spaced-comment': [2, 'always'],
+			strict: 2,
+			'valid-typeof': 2,
+			yoda: 2,
+		},
+	},
+	// NOTE: We need to apply this only to .ts files (and not to .mjs files).
+	...tsEslint.configs.recommendedTypeChecked.map(item => ({
+		...item,
+		files: ['src/**/*.ts'],
+	})),
+	// NOTE: We need to apply this only to .ts files (and not to .mjs files).
+	...tsEslint.configs.stylisticTypeChecked.map(item => ({
+		...item,
+		files: ['src/**/*.ts'],
+	})),
+	{
+		name: 'mediasoup-client .ts files',
+		files: ['src/**/*.ts'],
+		languageOptions: {
+			parserOptions: {
+				projectService: true,
+				project: 'tsconfig.json',
+			},
+		},
+		rules: {
+			'@typescript-eslint/consistent-generic-constructors': [
+				2,
+				'type-annotation',
+			],
+			'@typescript-eslint/dot-notation': 0,
+			'@typescript-eslint/no-unused-vars': [
+				2,
+				{
+					vars: 'all',
+					args: 'after-used',
+					caughtErrors: 'none',
+					ignoreRestSiblings: false,
+				},
+			],
+			// We want to use `type` instead of `interface`.
+			'@typescript-eslint/consistent-type-definitions': 0,
+			// Sorry, we need many `any` usage.
+			'@typescript-eslint/no-explicit-any': 0,
+			'@typescript-eslint/no-unsafe-member-access': 0,
+			'@typescript-eslint/no-unsafe-assignment': 0,
+			'@typescript-eslint/no-unsafe-call': 0,
+			'@typescript-eslint/no-unsafe-return': 0,
+			'@typescript-eslint/no-unsafe-argument': 0,
+			'@typescript-eslint/consistent-indexed-object-style': 0,
+			'@typescript-eslint/no-empty-function': 0,
+			'@typescript-eslint/require-await': 0,
+			'@typescript-eslint/restrict-template-expressions': 0,
+			'@typescript-eslint/no-duplicate-type-constituents': [
+				2,
+				{ ignoreUnions: true },
+			],
+			'@typescript-eslint/unbound-method': 0,
+		},
+	},
+	{
+		name: 'mediasoup-client .ts test files',
+		...jestEslint.configs['flat/recommended'],
+		files: ['src/test/**/*.ts'],
+		rules: {
+			...jestEslint.configs['flat/recommended'].rules,
+			'jest/no-disabled-tests': 2,
+			'jest/prefer-expect-assertions': 0,
+			'@typescript-eslint/no-unnecessary-type-assertion': 0,
+		},
+	},
+	prettierRecommendedEslint
+);
+
+// console.log('*** config:***\n', config);
+
+// console.log(
+// 	'---tsEslint.configs.strictTypeChecked: %o',
+// 	tsEslint.configs.strictTypeChecked
+// );
+
+// process.exit(1);
+
+export default config;

npm-scripts.mjs

@@ -12,7 +12,7 @@ const PIP_DEPS_DIR = path.resolve('worker/pip_deps');
 const PIP_DEV_DEPS_DIR = path.resolve('worker/pip_dev_deps');
 
 // Paths for ESLint to check. Converted to string for convenience.
-const ESLINT_PATHS = ['src', 'npm-scripts.mjs'].join(' ');
+const ESLINT_PATHS = ['eslint.config.mjs', 'src', 'npm-scripts.mjs'].join(' ');
 // Paths for ESLint to ignore. Converted to string argument for convenience.
 const ESLINT_IGNORE_PATTERN_ARGS = []
 	.map(entry => `--ignore-pattern ${entry}`)
@@ -22,6 +22,7 @@ const ESLINT_IGNORE_PATTERN_ARGS = []
 // node/src/fbs.
 const PRETTIER_PATHS = [
 	'README.md',
+	'eslint.config.mjs',
 	'src',
 	'npm-scripts.mjs',
 	'package.json',
@@ -193,10 +194,10 @@ function lintNode() {
 
 	// Ensure there are no rules that are unnecessary or conflict with Prettier
 	// rules.
-	executeCmd('eslint-config-prettier .eslintrc.js');
+	executeCmd('eslint-config-prettier eslint.config.mjs');
 
 	executeCmd(
-		`eslint -c .eslintrc.js --ext=ts,js,mjs --max-warnings 0 ${ESLINT_IGNORE_PATTERN_ARGS} ${ESLINT_PATHS}`
+		`eslint -c eslint.config.mjs --max-warnings 0 ${ESLINT_IGNORE_PATTERN_ARGS} ${ESLINT_PATHS}`
 	);
 
 	executeCmd(`prettier --check ${PRETTIER_PATHS}`);