@wistia/oxlint-config 0.5.1 → 0.6.1

package/rules/base.mjs

@@ -1,188 +1,198 @@
+import confusingBrowserGlobals from 'confusing-browser-globals';
+
 export const baseRules = {
   plugins: ['eslint', 'oxc'],
   rules: {
     // Enforce return statements in callbacks of array methods
-    // https://eslint.org/docs/rules/array-callback-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/array-callback-return.html
     'eslint/array-callback-return': ['error', { allowImplicit: true }],
 
     // Require super() calls in constructors
-    // https://eslint.org/docs/rules/constructor-super
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/constructor-super.html
     'eslint/constructor-super': 'error',
 
     // Enforce for loop update clause moving the counter in the right direction
-    // https://eslint.org/docs/rules/for-direction
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/for-direction.html
     'eslint/for-direction': 'error',
 
+    // Enforce return statements in getters
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/getter-return.html
+    'eslint/getter-return': ['error', { allowImplicit: true }],
+
     // Disallow using an async function as a Promise executor
-    // https://eslint.org/docs/rules/no-async-promise-executor
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-async-promise-executor.html
     'eslint/no-async-promise-executor': 'error',
 
     // Disallow await inside of loops
-    // https://eslint.org/docs/rules/no-await-in-loop
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-await-in-loop.html
     'eslint/no-await-in-loop': 'error',
 
     // Disallow reassigning class members
-    // https://eslint.org/docs/rules/no-class-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-class-assign.html
     'eslint/no-class-assign': 'error',
 
     // Disallow comparing against -0
-    // https://eslint.org/docs/rules/no-compare-neg-zero
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-compare-neg-zero.html
     'eslint/no-compare-neg-zero': 'error',
 
     // Disallow assignment operators in conditional expressions
-    // https://eslint.org/docs/rules/no-cond-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-cond-assign.html
     'eslint/no-cond-assign': ['error', 'always'],
 
     // Disallow reassigning const variables
-    // https://eslint.org/docs/rules/no-const-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-const-assign.html
     'eslint/no-const-assign': 'error',
 
     // Disallow expressions where the operation doesn't affect the value
-    // https://eslint.org/docs/rules/no-constant-binary-expression
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-constant-binary-expression.html
     'eslint/no-constant-binary-expression': 'error',
 
     // Disallow constant expressions in conditions
-    // https://eslint.org/docs/rules/no-constant-condition
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-constant-condition.html
     'eslint/no-constant-condition': 'error',
 
     // Disallow returning value from constructor
-    // https://eslint.org/docs/rules/no-constructor-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-constructor-return.html
     'eslint/no-constructor-return': 'error',
 
     // Disallow control characters in regular expressions
-    // https://eslint.org/docs/rules/no-control-regex
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-control-regex.html
     'eslint/no-control-regex': 'error',
 
     // Disallow the use of debugger
-    // https://eslint.org/docs/rules/no-debugger
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-debugger.html
     'eslint/no-debugger': 'error',
 
     // Disallow duplicate class members
-    // https://eslint.org/docs/rules/no-dupe-class-members
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-dupe-class-members.html
     'eslint/no-dupe-class-members': 'error',
 
     // Disallow duplicate conditions in if-else-if chains
-    // https://eslint.org/docs/rules/no-dupe-else-if
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-dupe-else-if.html
     'eslint/no-dupe-else-if': 'error',
 
     // Disallow duplicate keys in object literals
-    // https://eslint.org/docs/rules/no-dupe-keys
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-dupe-keys.html
     'eslint/no-dupe-keys': 'error',
 
     // Disallow duplicate case labels
-    // https://eslint.org/docs/rules/no-duplicate-case
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-duplicate-case.html
     'eslint/no-duplicate-case': 'error',
 
     // Disallow empty character classes in regular expressions
-    // https://eslint.org/docs/rules/no-empty-character-class
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-empty-character-class.html
     'eslint/no-empty-character-class': 'error',
 
     // Disallow empty destructuring patterns
-    // https://eslint.org/docs/rules/no-empty-pattern
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-empty-pattern.html
     'eslint/no-empty-pattern': 'error',
 
     // Disallow reassigning exceptions in catch clauses
-    // https://eslint.org/docs/rules/no-ex-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-ex-assign.html
     'eslint/no-ex-assign': 'error',
 
     // Disallow fallthrough of case statements
-    // https://eslint.org/docs/rules/no-fallthrough
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-fallthrough.html
     'eslint/no-fallthrough': 'error',
 
     // Disallow reassigning function declarations
-    // https://eslint.org/docs/rules/no-func-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-func-assign.html
     'eslint/no-func-assign': 'error',
 
     // Disallow assigning to imported bindings
-    // https://eslint.org/docs/rules/no-import-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-import-assign.html
     'eslint/no-import-assign': 'error',
 
     // Disallow variable or function declarations in nested blocks
-    // https://eslint.org/docs/rules/no-inner-declarations
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-inner-declarations.html
     'eslint/no-inner-declarations': 'error',
 
     // Disallow invalid regular expression strings in RegExp constructors
-    // https://eslint.org/docs/rules/no-invalid-regexp
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-invalid-regexp.html
     'eslint/no-invalid-regexp': 'error',
 
     // Disallow irregular whitespace
-    // https://eslint.org/docs/rules/no-irregular-whitespace
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-irregular-whitespace.html
     'eslint/no-irregular-whitespace': 'error',
 
     // Disallow literal numbers that lose precision
-    // https://eslint.org/docs/rules/no-loss-of-precision
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-loss-of-precision.html
     'eslint/no-loss-of-precision': 'error',
 
     // Disallow characters which are made with multiple code points in character class syntax
-    // https://eslint.org/docs/rules/no-misleading-character-class
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-misleading-character-class.html
     'eslint/no-misleading-character-class': 'error',
 
     // Disallow new operators with global non-constructor functions
-    // https://eslint.org/docs/rules/no-new-native-nonconstructor
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-new-native-nonconstructor.html
     'eslint/no-new-native-nonconstructor': 'error',
 
     // Disallow calling global object properties as functions
-    // https://eslint.org/docs/rules/no-obj-calls
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-obj-calls.html
     'eslint/no-obj-calls': 'error',
 
     // Disallow returning values from Promise executor functions
-    // https://eslint.org/docs/rules/no-promise-executor-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-promise-executor-return.html
     'eslint/no-promise-executor-return': 'error',
 
     // Disallow calling some Object.prototype methods directly on objects
-    // https://eslint.org/docs/rules/no-prototype-builtins
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-prototype-builtins.html
     'eslint/no-prototype-builtins': 'error',
 
     // Disallow assignments where both sides are exactly the same
-    // https://eslint.org/docs/rules/no-self-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-self-assign.html
     'eslint/no-self-assign': 'error',
 
     // Disallow comparisons where both sides are exactly the same
-    // https://eslint.org/docs/rules/no-self-compare
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-self-compare.html
     'eslint/no-self-compare': 'error',
 
     // Disallow returning values from setters
-    // https://eslint.org/docs/rules/no-setter-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-setter-return.html
     'eslint/no-setter-return': 'error',
 
     // Disallow sparse arrays
-    // https://eslint.org/docs/rules/no-sparse-arrays
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-sparse-arrays.html
     'eslint/no-sparse-arrays': 'error',
 
     // Disallow template literal placeholder syntax in regular strings
-    // https://eslint.org/docs/rules/no-template-curly-in-string
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-template-curly-in-string.html
     'eslint/no-template-curly-in-string': 'error',
 
     // Disallow this/super before calling super() in constructors
-    // https://eslint.org/docs/rules/no-this-before-super
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-this-before-super.html
     'eslint/no-this-before-super': 'error',
 
     // Disallow let or var variables that are read but never assigned
-    // https://eslint.org/docs/rules/no-unassigned-vars
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unassigned-vars.html
     'eslint/no-unassigned-vars': 'error',
 
     // Disallow unmodified loop conditions
-    // https://eslint.org/docs/rules/no-unmodified-loop-condition
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unmodified-loop-condition.html
     'eslint/no-unmodified-loop-condition': 'error',
 
+    // Disallow unreachable code after return, throw, continue, and break statements
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unreachable.html
+    'eslint/no-unreachable': 'error',
+
     // Disallow control flow statements in finally blocks
-    // https://eslint.org/docs/rules/no-unsafe-finally
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unsafe-finally.html
     'eslint/no-unsafe-finally': 'error',
 
     // Disallow negating the left operand of relational operators
-    // https://eslint.org/docs/rules/no-unsafe-negation
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unsafe-negation.html
     'eslint/no-unsafe-negation': 'error',
 
     // Disallow use of optional chaining in contexts where the undefined value is not allowed
-    // https://eslint.org/docs/rules/no-unsafe-optional-chaining
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unsafe-optional-chaining.html
     'eslint/no-unsafe-optional-chaining': ['error', { disallowArithmeticOperators: true }],
 
     // Disallow unused private class members
-    // https://eslint.org/docs/rules/no-unused-private-class-members
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-private-class-members.html
     'eslint/no-unused-private-class-members': 'error',
 
     // Disallow unused variables
-    // https://eslint.org/docs/rules/no-unused-vars
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-vars.html
     'eslint/no-unused-vars': [
       'error',
       {
@@ -198,71 +208,71 @@ export const baseRules = {
     ],
 
     // Disallow the use of variables before they are defined
-    // https://eslint.org/docs/rules/no-use-before-define
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-use-before-define.html
     'eslint/no-use-before-define': ['error', { functions: true, classes: true, variables: true }],
 
     // Disallow useless backreferences in regular expressions
-    // https://eslint.org/docs/rules/no-useless-backreference
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-backreference.html
     'eslint/no-useless-backreference': 'error',
 
     // Require calls to isNaN() when checking for NaN
-    // https://eslint.org/docs/rules/use-isnan
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/use-isnan.html
     'eslint/use-isnan': 'error',
 
     // Enforce comparing typeof expressions against valid strings
-    // https://eslint.org/docs/rules/valid-typeof
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/valid-typeof.html
     'eslint/valid-typeof': ['error', { requireStringLiterals: true }],
 
     // Enforce getter and setter pairs in objects and classes
-    // https://eslint.org/docs/rules/accessor-pairs
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/accessor-pairs.html
     'eslint/accessor-pairs': 'error',
 
     // Require braces around arrow function bodies
-    // https://eslint.org/docs/rules/arrow-body-style
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/arrow-body-style.html
     'eslint/arrow-body-style': 'off',
 
     // Enforce the use of variables within the scope they are defined
-    // https://eslint.org/docs/rules/block-scoped-var
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/block-scoped-var.html
     'eslint/block-scoped-var': 'error',
 
     // Enforce that class methods utilize this
-    // https://eslint.org/docs/rules/class-methods-use-this
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/class-methods-use-this.html
     'eslint/class-methods-use-this': 'error',
 
     // Require default cases in switch statements
-    // https://eslint.org/docs/rules/default-case
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/default-case.html
     'eslint/default-case': ['error', { commentPattern: '^no default$' }],
 
     // Enforce default clauses in switch statements to be last
-    // https://eslint.org/docs/rules/default-case-last
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/default-case-last.html
     'eslint/default-case-last': 'error',
 
     // Enforce default parameters to be last
-    // https://eslint.org/docs/rules/default-param-last
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/default-param-last.html
     'eslint/default-param-last': 'error',
 
     // Require the use of === and !==
-    // https://eslint.org/docs/rules/eqeqeq
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/eqeqeq.html
     'eslint/eqeqeq': ['error', 'always', { null: 'ignore' }],
 
     // Require for-in loops to include an if statement
-    // https://eslint.org/docs/rules/guard-for-in
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/guard-for-in.html
     'eslint/guard-for-in': 'error',
 
     // Require or disallow named function expressions
-    // https://eslint.org/docs/rules/func-names
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/func-names.html
     'eslint/func-names': ['error', 'always'],
 
     // Enforce the consistent use of either function declarations or expressions
-    // https://eslint.org/docs/rules/func-style
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/func-style.html
     'eslint/func-style': ['error', 'expression'],
 
     // Enforce grouped accessor pairs in object literals and classes
-    // https://eslint.org/docs/rules/grouped-accessor-pairs
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/grouped-accessor-pairs.html
     'eslint/grouped-accessor-pairs': 'error',
 
     // Enforce minimum and maximum identifier lengths
-    // https://eslint.org/docs/rules/id-length
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/id-length.html
     'eslint/id-length': [
       'error',
       {
@@ -272,11 +282,11 @@ export const baseRules = {
     ],
 
     // Enforce a maximum number of classes per file
-    // https://eslint.org/docs/rules/max-classes-per-file
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/max-classes-per-file.html
     'eslint/max-classes-per-file': ['error', 1],
 
     // Require constructor names to begin with a capital letter
-    // https://eslint.org/docs/rules/new-cap
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/new-cap.html
     'eslint/new-cap': [
       'error',
       {
@@ -288,144 +298,144 @@ export const baseRules = {
     ],
 
     // Disallow the use of alert, confirm, and prompt
-    // https://eslint.org/docs/rules/no-alert
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-alert.html
     'eslint/no-alert': 'error',
 
     // Disallow Array constructors
-    // https://eslint.org/docs/rules/no-array-constructor
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-array-constructor.html
     'eslint/no-array-constructor': 'error',
 
     // Disallow bitwise operators
-    // https://eslint.org/docs/rules/no-bitwise
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-bitwise.html
     'eslint/no-bitwise': 'error',
 
     // Disallow the use of arguments.caller or arguments.callee
-    // https://eslint.org/docs/rules/no-caller
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-caller.html
     'eslint/no-caller': 'error',
 
     // Disallow lexical declarations in case clauses
-    // https://eslint.org/docs/rules/no-case-declarations
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-case-declarations.html
     'eslint/no-case-declarations': 'error',
 
     // Disallow the use of console
-    // https://eslint.org/docs/rules/no-console
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-console.html
     'eslint/no-console': ['error', { allow: ['log', 'warn', 'error'] }],
 
     // Disallow continue statements
-    // https://eslint.org/docs/rules/no-continue
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-continue.html
     'eslint/no-continue': 'error',
 
     // Disallow deleting variables
-    // https://eslint.org/docs/rules/no-delete-var
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-delete-var.html
     'eslint/no-delete-var': 'error',
 
     // Disallow else blocks after return statements in if statements
-    // https://eslint.org/docs/rules/no-else-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-else-return.html
     'eslint/no-else-return': ['error', { allowElseIf: false }],
 
     // Disallow empty block statements
-    // https://eslint.org/docs/rules/no-empty
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-empty.html
     'eslint/no-empty': 'error',
 
     // Disallow empty functions
-    // https://eslint.org/docs/rules/no-empty-function
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-empty-function.html
     'eslint/no-empty-function': ['error', { allow: ['arrowFunctions', 'functions', 'methods'] }],
 
     // Disallow empty static blocks
-    // https://eslint.org/docs/rules/no-empty-static-block
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-empty-static-block.html
     'eslint/no-empty-static-block': 'error',
 
     // Disallow the use of eval()
-    // https://eslint.org/docs/rules/no-eval
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-eval.html
     'eslint/no-eval': 'error',
 
     // Disallow extending native types
-    // https://eslint.org/docs/rules/no-extend-native
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-extend-native.html
     'eslint/no-extend-native': 'error',
 
     // Disallow unnecessary calls to .bind()
-    // https://eslint.org/docs/rules/no-extra-bind
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-extra-bind.html
     'eslint/no-extra-bind': 'error',
 
     // Disallow unnecessary boolean casts
-    // https://eslint.org/docs/rules/no-extra-boolean-cast
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-extra-boolean-cast.html
     'eslint/no-extra-boolean-cast': 'error',
 
     // Disallow unnecessary labels
-    // https://eslint.org/docs/rules/no-extra-label
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-extra-label.html
     'eslint/no-extra-label': 'error',
 
     // Disallow assignments to native objects or read-only global variables
-    // https://eslint.org/docs/rules/no-global-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-global-assign.html
     'eslint/no-global-assign': 'error',
 
     // Disallow shorthand type conversions
-    // https://eslint.org/docs/rules/no-implicit-coercion
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-implicit-coercion.html
     'eslint/no-implicit-coercion': 'error',
 
     // Disallow the use of the __iterator__ property
-    // https://eslint.org/docs/rules/no-iterator
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-iterator.html
     'eslint/no-iterator': 'error',
 
     // Disallow labels that share a name with a variable
-    // https://eslint.org/docs/rules/no-label-var
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-label-var.html
     'eslint/no-label-var': 'error',
 
     // Disallow labeled statements
-    // https://eslint.org/docs/rules/no-labels
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-labels.html
     'eslint/no-labels': 'error',
 
     // Disallow unnecessary nested blocks
-    // https://eslint.org/docs/rules/no-lone-blocks
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-lone-blocks.html
     'eslint/no-lone-blocks': 'error',
 
     // Disallow if statements as the only statement in else blocks
-    // https://eslint.org/docs/rules/no-lonely-if
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-lonely-if.html
     'eslint/no-lonely-if': 'error',
 
     // Disallow function declarations that contain unsafe references inside loop statements
-    // https://eslint.org/docs/rules/no-loop-func
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-loop-func.html
     'eslint/no-loop-func': 'error',
 
     // Disallow magic numbers
-    // https://eslint.org/docs/rules/no-magic-numbers
-    // decision:
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-magic-numbers.html
+    // decision: this rule is too noisy to be a default and was commonly disabled inline
     'eslint/no-magic-numbers': 'off',
 
     // Disallow use of chained assignment expressions
-    // https://eslint.org/docs/rules/no-multi-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-multi-assign.html
     'eslint/no-multi-assign': 'error',
 
     // Disallow multiline strings
-    // https://eslint.org/docs/rules/no-multi-str
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-multi-str.html
     'eslint/no-multi-str': 'error',
 
     // Disallow nested ternary expressions
-    // https://eslint.org/docs/rules/no-nested-ternary
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-nested-ternary.html
     'eslint/no-nested-ternary': 'error',
 
     // Disallow new operators outside of assignments or comparisons
-    // https://eslint.org/docs/rules/no-new
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-new.html
     'eslint/no-new': 'error',
 
     // Disallow new operators with the Function object
-    // https://eslint.org/docs/rules/no-new-func
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-new-func.html
     'eslint/no-new-func': 'error',
 
     // Disallow new operators with the String, Number, and Boolean objects
-    // https://eslint.org/docs/rules/no-new-wrappers
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-new-wrappers.html
     'eslint/no-new-wrappers': 'error',
 
     // Disallow \8 and \9 escape sequences in string literals
-    // https://eslint.org/docs/rules/no-nonoctal-decimal-escape
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-nonoctal-decimal-escape.html
     'eslint/no-nonoctal-decimal-escape': 'error',
 
     // Disallow calls to the Object constructor without an argument
-    // https://eslint.org/docs/rules/no-object-constructor
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-object-constructor.html
     'eslint/no-object-constructor': 'error',
 
     // Disallow reassigning function parameters
-    // https://eslint.org/docs/rules/no-param-reassign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-param-reassign.html
     'eslint/no-param-reassign': [
       'error',
       {
@@ -447,112 +457,131 @@ export const baseRules = {
     ],
 
     // Disallow the unary operators ++ and --
-    // https://eslint.org/docs/rules/no-plusplus
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-plusplus.html
     'eslint/no-plusplus': ['error', { allowForLoopAfterthoughts: true }],
 
     // Disallow the use of the __proto__ property
-    // https://eslint.org/docs/rules/no-proto
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-proto.html
     'eslint/no-proto': 'error',
 
     // Disallow variable redeclaration
-    // https://eslint.org/docs/rules/no-redeclare
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-redeclare.html
     'eslint/no-redeclare': 'error',
 
     // Disallow multiple spaces in regular expressions
-    // https://eslint.org/docs/rules/no-regex-spaces
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-regex-spaces.html
     'eslint/no-regex-spaces': 'error',
 
+    // Disallow certain global variables (confusing browser globals + isFinite/isNaN)
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-restricted-globals.html
+    'eslint/no-restricted-globals': [
+      'error',
+      {
+        restrictedGlobals: {
+          ...Object.fromEntries(
+            confusingBrowserGlobals.map((name) => [
+              name,
+              `'${name}' should be written as 'window.${name}' or 'document.${name}' instead.`,
+            ]),
+          ),
+          isFinite: 'Use Number.isFinite instead...',
+          isNaN: 'Use Number.isNaN instead...',
+        },
+      },
+    ],
+
     // Disallow specified modules when loaded by import
-    // https://eslint.org/docs/rules/no-restricted-imports
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-restricted-imports.html
+    // decision: this should be set at a project-level config if necessary
     'eslint/no-restricted-imports': 'off',
 
     // Disallow assignment operators in return statements
-    // https://eslint.org/docs/rules/no-return-assign
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-return-assign.html
     'eslint/no-return-assign': ['error', 'always'],
 
     // Disallow use of javascript: urls
-    // https://eslint.org/docs/rules/no-script-url
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-script-url.html
     'eslint/no-script-url': 'error',
 
     // Disallow comma operators
-    // https://eslint.org/docs/rules/no-sequences
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-sequences.html
     'eslint/no-sequences': 'error',
 
     // Disallow variable declarations from shadowing variables declared in the outer scope
-    // https://eslint.org/docs/rules/no-shadow
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-shadow.html
     'eslint/no-shadow': 'error',
 
     // Disallow identifiers from shadowing restricted names
-    // https://eslint.org/docs/rules/no-shadow-restricted-names
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-shadow-restricted-names.html
     'eslint/no-shadow-restricted-names': 'error',
 
     // Disallow throwing literals as exceptions
-    // https://eslint.org/docs/rules/no-throw-literal
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-throw-literal.html
     'eslint/no-throw-literal': 'error',
 
     // Disallow ternary operators when simpler alternatives exist
-    // https://eslint.org/docs/rules/no-unneeded-ternary
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unneeded-ternary.html
     'eslint/no-unneeded-ternary': ['error', { defaultAssignment: false }],
 
     // Disallow unused expressions
-    // https://eslint.org/docs/rules/no-unused-expressions
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-expressions.html
     'eslint/no-unused-expressions': 'error',
 
     // Disallow unused labels
-    // https://eslint.org/docs/rules/no-unused-labels
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unused-labels.html
     'eslint/no-unused-labels': 'error',
 
     // Disallow unnecessary catch clauses
-    // https://eslint.org/docs/rules/no-useless-catch
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-catch.html
     'eslint/no-useless-catch': 'error',
 
     // Disallow unnecessary computed property keys in objects and classes
-    // https://eslint.org/docs/rules/no-useless-computed-key
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-computed-key.html
     'eslint/no-useless-computed-key': 'error',
 
     // Disallow unnecessary concatenation of literals or template literals
-    // https://eslint.org/docs/rules/no-useless-concat
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-concat.html
     'eslint/no-useless-concat': 'error',
 
     // Disallow unnecessary constructors
-    // https://eslint.org/docs/rules/no-useless-constructor
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-constructor.html
     'eslint/no-useless-constructor': 'error',
 
     // Disallow unnecessary escape characters
-    // https://eslint.org/docs/rules/no-useless-escape
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-escape.html
     'eslint/no-useless-escape': 'error',
 
     // Disallow renaming import, export, and destructured assignments to the same name
-    // https://eslint.org/docs/rules/no-useless-rename
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-rename.html
     'eslint/no-useless-rename': 'error',
 
     // Disallow redundant return statements
-    // https://eslint.org/docs/rules/no-useless-return
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-useless-return.html
     'eslint/no-useless-return': 'error',
 
     // Require let or const instead of var
-    // https://eslint.org/docs/rules/no-var
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-var.html
     'eslint/no-var': 'error',
 
     // Disallow void operators
-    // https://eslint.org/docs/rules/no-void
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-void.html
     'eslint/no-void': 'error',
     // note: typescript.ts overrides this to ["error", { "allowAsStatement": true }]
 
     // Disallow with statements
-    // https://eslint.org/docs/rules/no-with
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-with.html
     'eslint/no-with': 'error',
 
     // Require or disallow assignment operator shorthand where possible
-    // https://eslint.org/docs/rules/operator-assignment
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/operator-assignment.html
     'eslint/operator-assignment': ['error', 'always'],
 
     // Require const declarations for variables that are never reassigned after declared
-    // https://eslint.org/docs/rules/prefer-const
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-const.html
     'eslint/prefer-const': ['error', { destructuring: 'any', ignoreReadBeforeAssign: true }],
 
     // Require destructuring from arrays and/or objects
-    // https://eslint.org/docs/rules/prefer-destructuring
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-destructuring.html
     'eslint/prefer-destructuring': [
       'error',
       {
@@ -563,67 +592,67 @@ export const baseRules = {
     ],
 
     // Disallow the use of Math.pow in favor of the ** operator
-    // https://eslint.org/docs/rules/prefer-exponentiation-operator
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-exponentiation-operator.html
     'eslint/prefer-exponentiation-operator': 'error',
 
     // Disallow parseInt() and Number.parseInt() in favor of binary, octal, and hexadecimal literals
-    // https://eslint.org/docs/rules/prefer-numeric-literals
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-numeric-literals.html
     'eslint/prefer-numeric-literals': 'error',
 
     // Disallow use of Object.prototype.hasOwnProperty.call() and prefer use of Object.hasOwn()
-    // https://eslint.org/docs/rules/prefer-object-has-own
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-object-has-own.html
     'eslint/prefer-object-has-own': 'error',
 
     // Prefer use of an object spread over Object.assign
-    // https://eslint.org/docs/rules/prefer-object-spread
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-object-spread.html
     'eslint/prefer-object-spread': 'error',
 
     // Require using Error objects as Promise rejection reasons
-    // https://eslint.org/docs/rules/prefer-promise-reject-errors
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-promise-reject-errors.html
     'eslint/prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
 
     // Require rest parameters instead of arguments
-    // https://eslint.org/docs/rules/prefer-rest-params
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-rest-params.html
     'eslint/prefer-rest-params': 'error',
 
     // Require spread operators instead of .apply()
-    // https://eslint.org/docs/rules/prefer-spread
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-spread.html
     'eslint/prefer-spread': 'error',
 
     // Require template literals instead of string concatenation
-    // https://eslint.org/docs/rules/prefer-template
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/prefer-template.html
     'eslint/prefer-template': 'error',
 
     // Disallow losing originally caught error when re-throwing custom errors
-    // https://eslint.org/docs/rules/preserve-caught-error
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/preserve-caught-error.html
     'eslint/preserve-caught-error': ['error', { requireCatchParameter: false }],
 
     // Enforce the consistent use of the radix argument when using parseInt()
-    // https://eslint.org/docs/rules/radix
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/radix.html
     'eslint/radix': 'error',
 
     // Disallow async functions which have no await expression
-    // https://eslint.org/docs/rules/require-await
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/require-await.html
     'eslint/require-await': 'error',
 
     // Require generator functions to contain yield
-    // https://eslint.org/docs/rules/require-yield
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/require-yield.html
     'eslint/require-yield': 'error',
 
     // Require symbol descriptions
-    // https://eslint.org/docs/rules/symbol-description
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/symbol-description.html
     'eslint/symbol-description': 'error',
 
     // Require or disallow Unicode byte order mark (BOM)
-    // https://eslint.org/docs/rules/unicode-bom
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/unicode-bom.html
     'eslint/unicode-bom': ['error', 'never'],
 
     // Require var declarations be placed at the top of their containing scope
-    // https://eslint.org/docs/rules/vars-on-top
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/vars-on-top.html
     'eslint/vars-on-top': 'error',
 
     // Require or disallow "Yoda" conditions
-    // https://eslint.org/docs/rules/yoda
+    // https://oxc.rs/docs/guide/usage/linter/rules/eslint/yoda.html
     'eslint/yoda': 'error',
 
     //oxc rules - these are bug-catchers that oxlint can detect natively.