@wistia/oxlint-config 1.0.3 → 1.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.14.1",
   "type": "module",

package/rules/base.mjs

@@ -452,25 +452,10 @@ export const baseRules = {
 
     // Disallow reassigning function parameters
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-param-reassign.html
-    'eslint/no-param-reassign': [
-      'error',
-      {
-        props: true,
-        ignorePropertyModificationsFor: [
-          'acc',
-          'accumulator',
-          'e',
-          'ctx',
-          'context',
-          'req',
-          'request',
-          'res',
-          'response',
-          '$scope',
-          'staticContext',
-        ],
-      },
-    ],
+    // Decision: props:false. Name-based allowlists are brittle and miss common
+    // patterns (state, draft, reply, event, config, options); property-level
+    // immutability is delegated to TypeScript's Readonly<T> at the call site.
+    'eslint/no-param-reassign': ['error', { props: false }],
 
     // Disallow the unary operators ++ and --
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-plusplus.html
@@ -699,8 +684,8 @@ export const baseRules = {
 
     // Require following curly brace conventions
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/curly.html
-    // Decision: stylistic choice best left to formatter
-    'eslint/curly': 'off',
+    // Decision: enforce curly braces for all control statements because even the `multiline` option can allow subtle bugs
+    'eslint/curly': ['error', 'all'],
 
     // Require function names to match the name of the variable or property to which they are assigned
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/func-name-matching.html