Merge branch 'main' into externaldimensions

Author: nstepien

.github/workflows/ci.yml

@@ -44,6 +44,17 @@ jobs:
       - name: Visual regression test
         run: node --run visual
 
+      - name: Upload test failure artifacts
+        if: failure()
+        uses: actions/upload-artifact@v7
+        with:
+          name: test-artifacts
+          path: |
+            test/browser/**/__screenshots__/**
+            test/browser/**/__traces__/**
+            .vitest-attachments/test/**/*
+          if-no-files-found: ignore
+
       - name: Upload coverage
         uses: codecov/codecov-action@v5
         with:

.gitignore

@@ -1,4 +1,5 @@
 /.cache
+/.claude
 /coverage
 /dist
 /lib

eslint.config.js

@@ -13,7 +13,7 @@ import { defineConfig, globalIgnores } from 'eslint/config';
 import tseslint from 'typescript-eslint';
 
 export default defineConfig([
-  globalIgnores(['.cache', '.nitro', '.output', '.tanstack', 'coverage', 'dist', 'lib']),
+  globalIgnores(['.cache', '.claude', '.nitro', '.output', '.tanstack', 'coverage', 'dist', 'lib']),
 
   {
     linterOptions: {
@@ -314,17 +314,15 @@ copy(
     .join('\n')
 );
       */
-      '@eslint-react/component-hook-factories': 1,
-      '@eslint-react/error-boundaries': 1,
-      '@eslint-react/exhaustive-deps': 1,
       '@eslint-react/jsx-dollar': 1,
       '@eslint-react/jsx-key-before-spread': 1,
       '@eslint-react/jsx-no-comment-textnodes': 1,
-      '@eslint-react/jsx-no-duplicate-props': 1,
       '@eslint-react/jsx-shorthand-boolean': 1,
       '@eslint-react/jsx-shorthand-fragment': 1,
-      '@eslint-react/jsx-uses-react': 1,
-      '@eslint-react/jsx-uses-vars': 1,
+      '@eslint-react/component-hook-factories': 1,
+      '@eslint-react/error-boundaries': 1,
+      '@eslint-react/exhaustive-deps': 1,
+      '@eslint-react/immutability': 0,
       '@eslint-react/no-access-state-in-setstate': 1,
       '@eslint-react/no-array-index-key': 0,
       '@eslint-react/no-children-count': 1,
@@ -343,7 +341,9 @@ copy(
       '@eslint-react/no-direct-mutation-state': 1,
       '@eslint-react/no-duplicate-key': 1,
       '@eslint-react/no-forward-ref': 1,
+      '@eslint-react/no-implicit-children': 0,
       '@eslint-react/no-implicit-key': 1,
+      '@eslint-react/no-implicit-ref': 0,
       '@eslint-react/no-leaked-conditional-rendering': 1,
       '@eslint-react/no-missing-component-display-name': 1,
       '@eslint-react/no-missing-context-display-name': 1,

package.json

@@ -47,7 +47,7 @@
     "typecheck": "tsc --build"
   },
   "devDependencies": {
-    "@eslint-react/eslint-plugin": "3.0.0-next.71",
+    "@eslint-react/eslint-plugin": "3.0.0-rc.0",
     "@eslint/markdown": "^7.5.1",
     "@faker-js/faker": "^10.0.0",
     "@tanstack/react-router": "^1.132.31",
@@ -68,14 +68,14 @@
     "eslint-plugin-testing-library": "^7.13.5",
     "jspdf": "^4.0.0",
     "jspdf-autotable": "^5.0.2",
-    "oxfmt": "0.36.0",
+    "oxfmt": "0.38.0",
     "playwright": "~1.58.0",
     "postcss": "^8.5.2",
     "react": "^19.2.1",
     "react-dom": "^19.2.1",
     "rolldown": "1.0.0-rc.5",
     "rolldown-plugin-dts": "^0.22.1",
-    "typescript": "~5.9.2",
+    "typescript": "~6.0.1-rc",
     "typescript-eslint": "^8.56.0",
     "vite": "^8.0.0-beta.16",
     "vitest": "^4.0.17",
@@ -86,6 +86,7 @@
     "react-dom": "^19.2"
   },
   "overrides": {
+    "typescript": "$typescript",
     "vite": "$vite"
   }
 }

rolldown.config.ts

@@ -22,7 +22,7 @@ export default defineConfig({
       classPrefix: `rdg-${pkg.version.replaceAll('.', '-')}-`
     }),
     dts({
-      tsconfig: './tsconfig.lib.json'
+      tsconfig: './tsconfig.src.json'
     })
   ]
 });

src/EditCell.tsx

@@ -11,21 +11,6 @@ import type {
   RenderEditCellProps
 } from './types';
 
-declare global {
-  const scheduler: Scheduler | undefined;
-}
-
-interface Scheduler {
-  readonly postTask?: (
-    callback: () => void,
-    options?: {
-      priority?: 'user-blocking' | 'user-visible' | 'background';
-      signal?: AbortSignal;
-      delay?: number;
-    }
-  ) => Promise<unknown>;
-}
-
 /*
  * To check for outside `mousedown` events, we listen to all `mousedown` events at their birth,
  * i.e. on the window during the capture phase, and at their death, i.e. on the window during the bubble phase.

src/css.d.ts

@@ -0,0 +1 @@
+declare module '*.css' {}

src/globals.d.ts

@@ -5,5 +5,5 @@ declare module 'react' {
   }
 }
 
-// somehow required to make `declare global` work
+// somehow required to make types work
 export {};

test/globals.d.ts

@@ -9,5 +9,5 @@ declare module 'vitest/browser' {
   }
 }
 
-// somehow required to make `declare global` work
+// somehow required to make types work
 export {};

test/setupBrowser.ts

@@ -93,7 +93,6 @@ afterEach(() => {
   // eslint-disable-next-line vitest/no-standalone-expect
   expect
     .soft(
-      // eslint-disable-next-line @eslint-react/purity
       document.hasFocus(),
       'Focus is set on a browser UI element at the end of a test. Use safeTab() to return focus to the page.'
     )