@wistia/oxlint-config 1.12.4 → 1.12.6

package/CHANGELOG.md

@@ -1,5 +1,48 @@
 # @wistia/oxlint-config
 
+## 1.12.6
+
+### Patch Changes
+
+- [#193](https://github.com/wistia/oxlint-config/pull/193) [`4746be3`](https://github.com/wistia/oxlint-config/commit/4746be3493821df5119ebb23121a83209e998010) Thanks [@okize](https://github.com/okize)! - build(deps): patch & minor upgrades to dependencies
+
+  - Bump peer minimums: `eslint` to `>= 10.11.0`, `oxfmt` to `>= 0.70.0`, `oxlint-tsgolint` to `>= 7.0.2002`
+  - `eslint-plugin-playwright` v2.11.0 → v2.12.0 adds eight rules:
+    - New at `error`: `no-identical-title`, `no-test-return-statement`, `no-export`, `no-magic-timeouts`, `require-annotation-reason`. Consumers may see new errors
+    - New at `off`: `no-template-literal-title` (dynamic titles are allowed, matching `vitest/valid-title` `allowArguments`), `prefer-ending-with-an-expect` (page-object assertion helpers need project-specific options), `no-action-timeout` (per-call timeouts are a documented option; inline literals are covered by `no-magic-timeouts`)
+    - `no-skipped-test` now detects `testInfo` annotations. The rule is `off`, no effect
+  - `@eslint-react/eslint-plugin` v5.18.6 → v5.20.5. No new, renamed, or removed rules. Changes to rules already set to `error`, no config change required:
+    - `@eslint-react/dom-no-flush-sync` tracks `react-dom` imports, so aliased and namespace calls are reported and same-named functions from other packages are not
+    - `@eslint-react/dom-no-unsafe-iframe-sandbox` checks polymorphic components written as member expressions
+    - `@eslint-react/web-api-no-leaked-event-listener` reports listeners added only in effect cleanup
+    - `@eslint-react/web-api-no-leaked-fetch` no longer reports `abort` calls nested in a callback inside cleanup
+    - `immutability` and `set-state-in-effect` changed upstream. Their jsPlugin names are `off` (handled by native rules), no effect
+  - `eslint-plugin-storybook` v10.5.10 → v10.6.0. No rule changes
+  - `oxlint-tsgolint` v7.0.2001 → v7.0.2002. No new rules. Clearer diagnostics and false-positive fixes in type-aware rules already set to `error`, including `no-misused-promises`, `no-unnecessary-condition`, `no-unnecessary-type-assertion`, `prefer-optional-chain`, `require-array-sort-compare`, and `no-unnecessary-boolean-literal-compare`
+  - `oxfmt` v0.67.0 → v0.70.0. No config option changes. Adds a Markdown formatter (v0.69.0)
+
+## 1.12.5
+
+### Patch Changes
+
+- [#191](https://github.com/wistia/oxlint-config/pull/191) [`f7a55a0`](https://github.com/wistia/oxlint-config/commit/f7a55a02874ca9f266427f5fc0f44b763728a7ee) Thanks [@okize](https://github.com/okize)! - build(deps): upgrade `oxlint` to v1.83.0
+
+  - Bump `oxlint` peer minimum to `>= 1.83.0`
+  - No new, renamed, or removed rules in v1.83.0
+  - `@eslint-react/jsx-no-useless-fragment` set to `off`. The native `react/jsx-no-useless-fragment` rule covers the same cases and was already `error`, so every useless fragment was reported twice
+  - React 19.3 support, rules already set to `error`, no config change required:
+    - `react/no-unknown-property` accepts the new DOM props `onFullscreenChange`, `onFullscreenError`, `onFullscreenChangeCapture`, `onFullscreenErrorCapture`, `credentialless` on `<iframe>`, and `maskType` on `<mask>`
+    - `react/jsx-no-useless-fragment` no longer reports `<Fragment ref={...}>`, matching the existing `key` exemption
+  - Changes to rules already set to `error`, no config change required. Consumers may see new errors from the first four:
+    - `unicorn/prefer-array-flat-map` now reports `.filter().flatMap()` chains with simple untyped arrow callbacks
+    - `unicorn/prefer-at` now reports single-character `substring(i, i + 1)` calls
+    - `unicorn/prefer-default-parameters` now reports `value ??= x` and `value ||= x`
+    - `typescript/prefer-for-of` now reports loops over computed collections such as `this.values[key]`
+    - `unicorn/prefer-array-flat` no longer reports `[].concat(value)` or `Array.prototype.concat.call([], value)` without a spread argument
+    - `eslint/no-unused-vars` with `args: 'after-used'` no longer flags parameters before a used rest parameter, and no longer reports implicitly exported declarations in ambient modules that also contain an `export`-modified declaration (the rule is off in the typescript config)
+  - Rules set to `off`, no effect: `unicorn/prefer-global-this` (no longer reports references to window event methods that are not calls)
+  - `typescript/no-unnecessary-type-parameters` and `typescript/prefer-find` are now marked as offering suggestions. Metadata only, no behavior change
+
 ## 1.12.4
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.12.4",
+  "version": "1.12.6",
   "description": "Wistia's Oxlint configurations",
   "keywords": [
     "linter",
@@ -76,7 +76,7 @@
     "validate": "vitest run --project validate"
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "^5.18.6",
+    "@eslint-react/eslint-plugin": "^5.20.5",
     "@vitest/eslint-plugin": "^1.6.27",
     "confusing-browser-globals": "^1.0.11",
     "eslint-plugin-barrel-files": "^3.0.1",
@@ -84,29 +84,29 @@
     "eslint-plugin-jest-dom": "^5.10.1",
     "eslint-plugin-n": "^18.3.0",
     "eslint-plugin-no-only-tests": "^3.4.0",
-    "eslint-plugin-playwright": "^2.11.0",
-    "eslint-plugin-storybook": "^10.5.10",
+    "eslint-plugin-playwright": "^2.12.0",
+    "eslint-plugin-storybook": "^10.6.0",
     "eslint-plugin-styled-components": "^0.0.0",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
     "eslint-plugin-testing-library": "^7.16.2",
     "typescript": "^6.0.3"
   },
   "devDependencies": {
-    "@changesets/changelog-github": "^1.0.0",
-    "@changesets/cli": "^3.0.1",
-    "eslint": "^10.9.0",
-    "oxfmt": "^0.67.0",
-    "oxlint": "^1.82.0",
-    "oxlint-tsgolint": "^7.0.2001",
-    "storybook": "^10.5.10",
-    "vite": "^8.2.2",
-    "vitest": "^5.0.0"
+    "@changesets/changelog-github": "^1.0.1",
+    "@changesets/cli": "^3.0.3",
+    "eslint": "^10.11.0",
+    "oxfmt": "^0.70.0",
+    "oxlint": "1.83.0",
+    "oxlint-tsgolint": "^7.0.2002",
+    "storybook": "^10.6.0",
+    "vite": "^8.3.0",
+    "vitest": "^5.0.1"
   },
   "peerDependencies": {
-    "eslint": ">= 10.9.0",
-    "oxfmt": ">= 0.67.0",
-    "oxlint": ">= 1.82.0",
-    "oxlint-tsgolint": ">= 7.0.2001"
+    "eslint": ">= 10.11.0",
+    "oxfmt": ">= 0.70.0",
+    "oxlint": ">= 1.83.0",
+    "oxlint-tsgolint": ">= 7.0.2002"
   },
   "peerDependenciesMeta": {
     "oxfmt": {

package/rules/playwright.mjs

@@ -200,6 +200,26 @@ export const playwrightRules = {
     // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-unnecessary-assertions.md
     'eslint-plugin-playwright/no-unnecessary-assertions': 'error',
 
+    // Disallow identical test and describe titles
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-identical-title.md
+    'eslint-plugin-playwright/no-identical-title': 'error',
+
+    // Disallow returning from a test body
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-test-return-statement.md
+    'eslint-plugin-playwright/no-test-return-statement': 'error',
+
+    // Disallow exports in files containing tests
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-export.md
+    'eslint-plugin-playwright/no-export': 'error',
+
+    // Disallow inline numeric timeout values
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-magic-timeouts.md
+    'eslint-plugin-playwright/no-magic-timeouts': 'error',
+
+    // Require a reason on conditional skip and fixme annotations
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-annotation-reason.md
+    'eslint-plugin-playwright/require-annotation-reason': 'error',
+
     // Enforce blank lines between test blocks
     // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/consistent-spacing-between-blocks.md
     // Decision: style preference, left to a formatter
@@ -249,5 +269,20 @@ export const playwrightRules = {
     // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/require-to-pass-timeout.md
     // Decision: too strict for general use, matching vitest/require-test-timeout
     'eslint-plugin-playwright/require-to-pass-timeout': 'off',
+
+    // Disallow the timeout option on actions
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-action-timeout.md
+    // Decision: per-call timeouts are a documented Playwright option; inline literals are covered by no-magic-timeouts
+    'eslint-plugin-playwright/no-action-timeout': 'off',
+
+    // Disallow template literals in titles
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/no-template-literal-title.md
+    // Decision: dynamic titles are allowed, matching vitest/valid-title allowArguments
+    'eslint-plugin-playwright/no-template-literal-title': 'off',
+
+    // Require tests to end with an assertion
+    // https://github.com/playwright-community/eslint-plugin-playwright/blob/main/docs/rules/prefer-ending-with-an-expect.md
+    // Decision: page-object assertion helpers are not recognized without project-specific options
+    'eslint-plugin-playwright/prefer-ending-with-an-expect': 'off',
   },
 };

package/rules/react.mjs

@@ -429,7 +429,8 @@ export const reactRules = {
 
     // Disallow unnecessary fragments
     // https://eslint-react.xyz/docs/rules/jsx-no-useless-fragment
-    '@eslint-react/jsx-no-useless-fragment': 'error',
+    // Decision: handled by native react/jsx-no-useless-fragment
+    '@eslint-react/jsx-no-useless-fragment': 'off',
 
     // Prevent leaked semicolons in JSX
     // https://eslint-react.xyz/docs/rules/jsx-no-leaked-semicolon