@wistia/oxlint-config 1.0.2 → 1.0.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@wistia/oxlint-config",
3
- "version": "1.0.2",
3
+ "version": "1.0.4",
4
4
  "description": "Wistia's Oxlint configurations",
5
5
  "packageManager": "yarn@4.14.1",
6
6
  "type": "module",
package/rules/base.mjs CHANGED
@@ -452,25 +452,10 @@ export const baseRules = {
452
452
 
453
453
  // Disallow reassigning function parameters
454
454
  // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-param-reassign.html
455
- 'eslint/no-param-reassign': [
456
- 'error',
457
- {
458
- props: true,
459
- ignorePropertyModificationsFor: [
460
- 'acc',
461
- 'accumulator',
462
- 'e',
463
- 'ctx',
464
- 'context',
465
- 'req',
466
- 'request',
467
- 'res',
468
- 'response',
469
- '$scope',
470
- 'staticContext',
471
- ],
472
- },
473
- ],
455
+ // Decision: props:false. Name-based allowlists are brittle and miss common
456
+ // patterns (state, draft, reply, event, config, options); property-level
457
+ // immutability is delegated to TypeScript's Readonly<T> at the call site.
458
+ 'eslint/no-param-reassign': ['error', { props: false }],
474
459
 
475
460
  // Disallow the unary operators ++ and --
476
461
  // https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-plusplus.html
package/rules/unicorn.mjs CHANGED
@@ -281,7 +281,11 @@ export const unicornRules = {
281
281
 
282
282
  // Disallow returning/yielding `Promise.resolve` / `Promise.reject` in async functions
283
283
  // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/no-useless-promise-resolve-reject.html
284
- 'unicorn/no-useless-promise-resolve-reject': 'error',
284
+ // Decision: conflicts with `typescript/require-await`. The rule's fix (drop `Promise.resolve`,
285
+ // return the raw value) leaves an `async` function with no `await`, which `require-await`
286
+ // then flags. Picking `require-await` — async-without-await is a meaningful invariant,
287
+ // while this rule is purely cosmetic.
288
+ 'unicorn/no-useless-promise-resolve-reject': 'off',
285
289
 
286
290
  // Disallow useless spread operators
287
291
  // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/no-useless-spread.html