Finalizing eslint update

Author: perki

eslint.config.js

@@ -3,17 +3,15 @@
  * [BSD-3-Clause](https://github.com/pryv/lib-js/blob/master/LICENSE)
  */
 /**
- * ESLint flat config - replaces semistandard with modern tooling
+ * ESLint flat config using neostandard (modern successor to semistandard)
  * Style: StandardJS with semicolons + TypeScript checks
  */
-const js = require('@eslint/js');
-const stylistic = require('@stylistic/eslint-plugin');
+const neostandard = require('neostandard');
 const tseslint = require('typescript-eslint');
-const nodePlugin = require('eslint-plugin-n');
 const globals = require('globals');
 
 module.exports = [
-  // Ignore patterns (replaces semistandard.ignore in package.json)
+  // Ignore patterns
   {
     ignores: [
       'node_modules/**',
@@ -24,19 +22,15 @@ module.exports = [
     ]
   },
 
-  // Base JS recommended rules
-  js.configs.recommended,
+  // Neostandard with semicolons (semistandard style)
+  ...neostandard({ semi: true, noStyle: false }),
 
   // TypeScript support
   ...tseslint.configs.recommended,
 
-  // Main configuration for JS/TS files
+  // Main configuration
   {
     files: ['**/*.js', '**/*.ts'],
-    plugins: {
-      '@stylistic': stylistic,
-      'n': nodePlugin
-    },
     languageOptions: {
       ecmaVersion: 2022,
       sourceType: 'commonjs',
@@ -52,198 +46,16 @@ module.exports = [
       }
     },
     rules: {
-      // ============================================
-      // Stylistic rules (semistandard style)
-      // ============================================
-
-      // Semicolons: REQUIRED (this is what makes it "semi" standard)
-      '@stylistic/semi': ['error', 'always'],
-      '@stylistic/semi-spacing': ['error', { before: false, after: true }],
-
-      // Indentation: 2 spaces
-      '@stylistic/indent': ['error', 2, { SwitchCase: 1 }],
-
-      // Quotes: single quotes
-      '@stylistic/quotes': ['error', 'single', { avoidEscape: true, allowTemplateLiterals: true }],
-
-      // Spacing
-      '@stylistic/space-before-function-paren': ['error', {
-        anonymous: 'always',
-        named: 'always',
-        asyncArrow: 'always'
-      }],
-      '@stylistic/space-before-blocks': ['error', 'always'],
-      '@stylistic/space-in-parens': ['error', 'never'],
-      '@stylistic/space-infix-ops': 'error',
-      '@stylistic/space-unary-ops': ['error', { words: true, nonwords: false }],
-      '@stylistic/keyword-spacing': ['error', { before: true, after: true }],
-      '@stylistic/key-spacing': ['error', { beforeColon: false, afterColon: true }],
-      '@stylistic/comma-spacing': ['error', { before: false, after: true }],
-      '@stylistic/array-bracket-spacing': ['error', 'never'],
-      '@stylistic/object-curly-spacing': ['error', 'always'],
-      '@stylistic/block-spacing': ['error', 'always'],
-      '@stylistic/computed-property-spacing': ['error', 'never'],
-      '@stylistic/func-call-spacing': ['error', 'never'],
-      '@stylistic/template-curly-spacing': ['error', 'never'],
-      '@stylistic/no-whitespace-before-property': 'error',
-
-      // Line breaks and formatting
-      '@stylistic/eol-last': ['error', 'always'],
-      '@stylistic/no-multiple-empty-lines': ['error', { max: 1, maxEOF: 0 }],
-      '@stylistic/no-trailing-spaces': 'error',
-      '@stylistic/linebreak-style': ['error', 'unix'],
-      '@stylistic/padded-blocks': ['error', 'never'],
-
-      // Commas and operators
-      '@stylistic/comma-dangle': ['error', 'never'],
-      '@stylistic/comma-style': ['error', 'last'],
-      '@stylistic/dot-location': ['error', 'property'],
-      '@stylistic/operator-linebreak': ['error', 'after', { overrides: { '?': 'before', ':': 'before' } }],
-
-      // Braces
-      '@stylistic/brace-style': ['error', '1tbs', { allowSingleLine: true }],
-      '@stylistic/curly-newline': 'off',
-
-      // Other stylistic
-      '@stylistic/no-mixed-spaces-and-tabs': 'error',
-      '@stylistic/no-tabs': 'error',
-      '@stylistic/no-multi-spaces': 'error',
-      '@stylistic/arrow-spacing': ['error', { before: true, after: true }],
-      '@stylistic/rest-spread-spacing': ['error', 'never'],
-      '@stylistic/yield-star-spacing': ['error', 'after'],
-      '@stylistic/generator-star-spacing': ['error', { before: false, after: true }],
-      '@stylistic/new-parens': 'error',
-      '@stylistic/wrap-iife': ['error', 'any'],
-      '@stylistic/spaced-comment': ['error', 'always', {
-        line: { markers: ['*package', '!', '/', ',', '='] },
-        block: { balanced: true, markers: ['*package', '!', ',', ':', '::', 'flow-include'], exceptions: ['*'] }
-      }],
-
-      // ============================================
-      // Standard JS rules
-      // ============================================
-
-      // Best practices
-      'accessor-pairs': ['error', { setWithoutGet: true, enforceForClassMembers: true }],
-      'array-callback-return': ['error', { allowImplicit: false, checkForEach: false }],
-      'constructor-super': 'error',
-      'default-case-last': 'error',
-      'eqeqeq': ['error', 'always', { null: 'ignore' }],
-      'new-cap': ['error', { newIsCap: true, capIsNew: false, properties: true }],
-      'no-array-constructor': 'error',
-      'no-async-promise-executor': 'error',
-      'no-caller': 'error',
-      'no-case-declarations': 'error',
-      'no-class-assign': 'error',
-      'no-compare-neg-zero': 'error',
-      'no-cond-assign': 'error',
-      'no-const-assign': 'error',
-      'no-constant-condition': ['error', { checkLoops: false }],
-      'no-control-regex': 'error',
-      'no-debugger': 'error',
-      'no-delete-var': 'error',
-      'no-dupe-args': 'error',
-      'no-dupe-class-members': 'error',
-      'no-dupe-keys': 'error',
-      'no-duplicate-case': 'error',
-      'no-empty': ['error', { allowEmptyCatch: true }],
-      'no-empty-character-class': 'error',
-      'no-empty-pattern': 'error',
-      'no-eval': 'error',
-      'no-ex-assign': 'error',
-      'no-extend-native': 'error',
-      'no-extra-bind': 'error',
-      'no-extra-boolean-cast': 'error',
-      'no-fallthrough': 'error',
-      'no-func-assign': 'error',
-      'no-global-assign': 'error',
-      'no-implied-eval': 'error',
-      'no-import-assign': 'error',
-      'no-invalid-regexp': 'error',
-      'no-irregular-whitespace': 'error',
-      'no-iterator': 'error',
-      'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
-      'no-lone-blocks': 'error',
-      'no-loss-of-precision': 'error',
-      'no-misleading-character-class': 'error',
-      'no-multi-str': 'error',
-      'no-new': 'error',
-      'no-new-func': 'error',
-      'no-new-native-nonconstructor': 'error',
-      'no-new-wrappers': 'error',
-      'no-obj-calls': 'error',
-      'no-object-constructor': 'error',
-      'no-octal': 'error',
-      'no-octal-escape': 'error',
-      'no-proto': 'error',
-      'no-prototype-builtins': 'error',
-      'no-redeclare': ['error', { builtinGlobals: false }],
-      'no-regex-spaces': 'error',
-      'no-return-assign': ['error', 'except-parens'],
-      'no-self-assign': ['error', { props: true }],
-      'no-self-compare': 'error',
-      'no-sequences': 'error',
-      'no-shadow-restricted-names': 'error',
-      'no-sparse-arrays': 'error',
-      'no-template-curly-in-string': 'error',
-      'no-this-before-super': 'error',
-      'no-throw-literal': 'error',
-      'no-undef': 'error',
-      'no-undef-init': 'error',
-      'no-unexpected-multiline': 'error',
-      'no-unmodified-loop-condition': 'error',
-      'no-unneeded-ternary': ['error', { defaultAssignment: false }],
-      'no-unreachable': 'error',
-      'no-unreachable-loop': 'error',
-      'no-unsafe-finally': 'error',
-      'no-unsafe-negation': 'error',
-      'no-unused-expressions': ['error', { allowShortCircuit: true, allowTernary: true, allowTaggedTemplates: true }],
-      // Disable base rule in favor of @typescript-eslint version
-      'no-unused-vars': 'off',
-      'no-use-before-define': ['error', { functions: false, classes: false, variables: false }],
-      'no-useless-backreference': 'error',
-      'no-useless-call': 'error',
-      'no-useless-catch': 'error',
-      'no-useless-computed-key': 'error',
-      'no-useless-constructor': 'error',
-      'no-useless-escape': 'error',
-      'no-useless-rename': 'error',
-      'no-useless-return': 'error',
-      'no-var': 'warn',
-      'no-void': 'error',
-      'no-with': 'error',
-      'object-shorthand': ['warn', 'properties'],
-      'one-var': ['error', { initialized: 'never' }],
-      'prefer-const': ['error', { destructuring: 'all' }],
-      'prefer-promise-reject-errors': 'error',
-      'prefer-regex-literals': ['error', { disallowRedundantWrapping: true }],
-      'symbol-description': 'error',
-      'unicode-bom': ['error', 'never'],
-      'use-isnan': ['error', { enforceForSwitchCase: true, enforceForIndexOf: true }],
-      'valid-typeof': ['error', { requireStringLiterals: true }],
-      'yoda': ['error', 'never'],
-
-      // Node.js specific
-      'n/handle-callback-err': ['error', '^(err|error)$'],
-      'n/no-callback-literal': 'error',
-      'n/no-deprecated-api': 'error',
-      'n/no-exports-assign': 'error',
-      'n/no-new-require': 'error',
-      'n/no-path-concat': 'error',
-      'n/process-exit-as-throw': 'error',
-
-      // ============================================
-      // TypeScript specific rules
-      // ============================================
+      // TypeScript adjustments
       '@typescript-eslint/no-unused-vars': ['error', {
         args: 'none',
         caughtErrors: 'none',
         ignoreRestSiblings: true,
         varsIgnorePattern: '^_',
         argsIgnorePattern: '^_'
       }],
-      '@typescript-eslint/no-require-imports': 'off', // Allow require() in CommonJS
-      '@typescript-eslint/no-explicit-any': 'off', // Allow any for now (existing codebase)
+      '@typescript-eslint/no-require-imports': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
       '@typescript-eslint/explicit-function-return-type': 'off',
       '@typescript-eslint/explicit-module-boundary-types': 'off',
       '@typescript-eslint/ban-ts-comment': 'off'
@@ -254,15 +66,13 @@ module.exports = [
   {
     files: ['**/*.d.ts'],
     rules: {
-      // Disable all stylistic rules for .d.ts files
       '@stylistic/space-before-function-paren': 'off',
       '@stylistic/semi': 'off',
       '@stylistic/indent': 'off',
       '@stylistic/no-multiple-empty-lines': 'off',
       '@stylistic/eol-last': 'off',
       '@stylistic/no-multi-spaces': 'off',
       '@stylistic/no-trailing-spaces': 'off',
-      // Disable variable/type checks (types come from imports)
       'no-undef': 'off',
       'no-unused-vars': 'off',
       '@typescript-eslint/no-unused-vars': 'off',