@wistia/oxlint-config 1.12.5 → 1.12.7

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @wistia/oxlint-config
 
+## 1.12.7
+
+### Patch Changes
+
+- [#195](https://github.com/wistia/oxlint-config/pull/195) [`57f33f1`](https://github.com/wistia/oxlint-config/commit/57f33f1b14a53495aece8c23ebcc4981e8fc6312) Thanks [@okize](https://github.com/okize)! - build(deps): upgrade `oxlint` to v1.84.0
+
+  - Bump `oxlint` peer minimum to `>= 1.84.0`
+  - No new, renamed, or removed rules in v1.84.0
+  - Changes to rules already set to `error`, no config change required:
+    - `react/no-unknown-property` accepts `onTransitionRun`, `onTransitionStart`, `onTransitionCancel`, and their Capture variants
+    - `eslint/preserve-caught-error` fixes no longer conflict with `unicorn/prefer-optional-catch-binding`. `catch (error) { throw new Error('m') }` now fixes to `catch { throw new Error('m') }` instead of adding a dangling `cause: error`
+    - `eslint/no-unused-vars` reports unused private bindings inside a TypeScript namespace even when a root export shares the name. Its fixer keeps comments when removing imports and keeps rest bindings when removing array elements (the rule is off in the typescript config)
+  - Rules set to `off`, no effect: `unicorn/no-unreadable-iife` (gains a suggestion), `unicorn/consistent-function-scoping` (reports functions that only capture a distant ancestor), `vitest/prefer-to-be-truthy` (fix reclassified as a suggestion)
+
+## 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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.12.5",
+  "version": "1.12.7",
   "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.83.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.84.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.83.0",
-    "oxlint-tsgolint": ">= 7.0.2001"
+    "eslint": ">= 10.11.0",
+    "oxfmt": ">= 0.70.0",
+    "oxlint": ">= 1.84.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',
   },
 };