@4players/odin-plugin-web 1.13.0 → 1.14.1

package/eslint.config.mjs

@@ -0,0 +1,168 @@
+import js from '@eslint/js';
+import tseslint from 'typescript-eslint';
+
+export default tseslint.config(
+  js.configs.recommended,
+  tseslint.configs.recommended,
+  {
+    ignores: [
+      'lib/**',
+      'build/**',
+      'dist/**',
+      'test/www/**',
+      'src/common/version.ts',
+    ],
+  },
+  {
+    // Correctness / good-practice rules carried over from neostandard's base
+    // preset, with its exact options. neostandard itself is not a dependency:
+    // its only other contributions here were eslint-plugin-n (Node-CommonJS
+    // rules, irrelevant to a browser/WASM library) and a single naming rule
+    // from eslint-plugin-promise.
+    rules: {
+      'accessor-pairs': [
+        'error',
+        {
+          enforceForTSTypes: false,
+          enforceForClassMembers: true,
+          getWithoutSet: false,
+          setWithoutGet: true,
+        },
+      ],
+      'array-callback-return': [
+        'error',
+        {
+          allowImplicit: false,
+          checkForEach: false,
+          allowVoid: false,
+        },
+      ],
+      curly: ['error', 'multi-line'],
+      'default-case-last': ['error'],
+      eqeqeq: ['error', 'always', { null: 'ignore' }],
+      'new-cap': [
+        'error',
+        {
+          capIsNew: false,
+          capIsNewExceptions: [
+            'Array',
+            'Boolean',
+            'Date',
+            'Error',
+            'Function',
+            'Number',
+            'Object',
+            'RegExp',
+            'String',
+            'Symbol',
+            'BigInt',
+          ],
+          newIsCap: true,
+          newIsCapExceptions: [],
+          properties: true,
+        },
+      ],
+      'no-caller': ['error'],
+      'no-eval': ['error', { allowIndirect: false }],
+      'no-extend-native': ['error', { exceptions: [] }],
+      'no-extra-bind': ['error'],
+      'no-implied-eval': ['error'],
+      'no-iterator': ['error'],
+      'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
+      'no-lone-blocks': ['error'],
+      'no-multi-str': ['error'],
+      'no-new': ['error'],
+      'no-new-func': ['error'],
+      'no-new-wrappers': ['error'],
+      'no-object-constructor': ['error'],
+      'no-octal-escape': ['error'],
+      'no-proto': ['error'],
+      'no-return-assign': ['error', 'except-parens'],
+      'no-self-compare': ['error'],
+      'no-sequences': ['error', { allowInParentheses: true }],
+      'no-template-curly-in-string': ['error'],
+      'no-throw-literal': ['error'],
+      'no-undef-init': ['error'],
+      'no-unmodified-loop-condition': ['error'],
+      'no-unneeded-ternary': ['error', { defaultAssignment: false }],
+      'no-unreachable-loop': ['error', { ignore: [] }],
+      'no-useless-call': ['error'],
+      'no-useless-computed-key': ['error', { enforceForClassMembers: true }],
+      'no-useless-rename': [
+        'error',
+        {
+          ignoreDestructuring: false,
+          ignoreImport: false,
+          ignoreExport: false,
+        },
+      ],
+      'no-useless-return': ['error'],
+      'no-var': ['error'],
+      'no-void': ['error', { allowAsStatement: false }],
+      'object-shorthand': ['warn', 'properties'],
+      'one-var': ['error', { initialized: 'never' }],
+      'prefer-const': [
+        'error',
+        { destructuring: 'all', ignoreReadBeforeAssign: false },
+      ],
+      'prefer-promise-reject-errors': ['error', { allowEmptyReject: false }],
+      'prefer-regex-literals': ['error', { disallowRedundantWrapping: true }],
+      'prefer-rest-params': ['error'],
+      'prefer-spread': ['error'],
+      'symbol-description': ['error'],
+      'unicode-bom': ['error', 'never'],
+      yoda: ['error', 'never', { exceptRange: false, onlyEquality: false }],
+    },
+  },
+  {
+    // Type-aware promise safety. Only src/**/*.ts is covered by tsconfig's
+    // `include`, so the type graph is built for those files alone; everything
+    // else (esbuild.config.ts, test/, wasm/) keeps the syntactic rules above.
+    // Warn-level for now: these surface real findings that need per-call-site
+    // judgement (intentional fire-and-forget vs. a missing catch).
+    files: ['src/**/*.ts'],
+    languageOptions: {
+      parserOptions: {
+        projectService: true,
+        tsconfigRootDir: import.meta.dirname,
+      },
+    },
+    rules: {
+      '@typescript-eslint/no-floating-promises': 'warn',
+      '@typescript-eslint/no-misused-promises': 'warn',
+      '@typescript-eslint/await-thenable': 'warn',
+      '@typescript-eslint/no-for-in-array': 'warn',
+    },
+  },
+  {
+    // project choices; the core rules TS makes redundant (no-undef,
+    // no-unused-vars, ...) are already disabled by the presets above
+    rules: {
+      '@typescript-eslint/no-useless-constructor': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/no-empty-object-type': 'off',
+      camelcase: 'off',
+      // TypeScript already resolves identifiers; the sole plain-JS file is a
+      // 20-line dev server, so this rule has nothing left to catch here
+      'no-undef': 'off',
+      // switch fallthrough is used deliberately in the codec/state machines
+      'no-fallthrough': 'off',
+      // `const X = Symbol(); type X = typeof X` declaration merging (see src/common/sentinel.ts)
+      '@typescript-eslint/no-redeclare': 'off',
+      // message/command maps are grouped into namespaces (see src/common/messages.ts)
+      '@typescript-eslint/no-namespace': 'off',
+      // unused arguments and caught errors are signature-mandated here
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          args: 'none',
+          caughtErrors: 'none',
+          ignoreRestSiblings: true,
+          vars: 'all',
+          destructuredArrayIgnorePattern: '^_',
+        },
+      ],
+    },
+  }
+);