Merge branch 'main' into 07-05-feat_rsc_support_directive_pattern_transform

hi-ogawa · web-flow · commit 609a1932fd6f · 2025-07-09T08:00:04.000+09:00
diff --git a/.github/renovate.json5 b/.github/renovate.json5
@@ -36,8 +36,6 @@
     // manually bumping
     "node",
 
-    "generouted", // testing lib shipping JSX (new version ship transpiled JS)
-
     // breaking changes
     "source-map", // `source-map:v0.7.0+` needs more investigation
     "kill-port", // `kill-port:^2.0.0 has perf issues (#8392)
diff --git a/.github/workflows/ci-rsc.yml b/.github/workflows/ci-rsc.yml
@@ -31,18 +31,21 @@ jobs:
       - run: pnpm -C packages/plugin-rsc test
 
   test-e2e:
-    name: test-rsc (${{ matrix.os }} / ${{ matrix.browser }})
+    name: test-rsc (${{ matrix.os }} / ${{ matrix.browser }}) ${{ matrix.rolldown == true && '(rolldown)' || '' }}
     runs-on: ${{ matrix.os }}
     strategy:
-      # TODO: shard?
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
         browser: [chromium]
+        rolldown: [false]
         include:
           - os: ubuntu-latest
             browser: firefox
           - os: macos-latest
             browser: webkit
+          - os: ubuntu-latest
+            browser: chromium
+            rolldown: true
       fail-fast: false
     steps:
       - uses: actions/checkout@v4
@@ -52,6 +55,11 @@ jobs:
       - uses: pnpm/action-setup@a7487c7e89a18df4991f7f222e4898a00d66ddda # v4.1.0
       - run: pnpm i
       - run: pnpm build
+      - name: install rolldown
+        if: ${{ matrix.rolldown }}
+        run: |
+          echo 'overrides: { vite: "npm:rolldown-vite@latest" }' >> pnpm-workspace.yaml
+          pnpm i --no-frozen-lockfile
       - run: pnpm -C packages/plugin-rsc exec playwright install ${{ matrix.browser }}
       - run: pnpm -C packages/plugin-rsc test-e2e-ci --project=${{ matrix.browser }}
         env:
diff --git a/package.json b/package.json
@@ -70,15 +70,5 @@
     "playground/**/__tests__/**/*.ts": [
       "eslint --cache --fix"
     ]
-  },
-  "pnpm": {
-    "packageExtensions": {
-      "generouted": {
-        "peerDependencies": {
-          "react": "*",
-          "react-router-dom": "*"
-        }
-      }
-    }
   }
 }
diff --git a/packages/plugin-react-swc/playground/ts-lib/__tests__/ts-lib.spec.ts b/packages/plugin-react-swc/playground/ts-lib/__tests__/ts-lib.spec.ts
@@ -1,24 +1,22 @@
-import { expect, test } from '@playwright/test'
+import { type Page, expect, test } from '@playwright/test'
 import { setupBuildAndPreview, setupDevServer } from '../../utils.ts'
 
 test('TS lib build', async ({ page }) => {
   const { testUrl, server } = await setupBuildAndPreview('ts-lib')
   await page.goto(testUrl)
-  await expect(page.locator('main')).toHaveText('Home page')
-
-  await page.locator('a', { hasText: 'About' }).click()
-  await expect(page.locator('main')).toHaveText('About page')
-
+  await testNonJs(page)
   await server.httpServer.close()
 })
 
 test('TS lib dev', async ({ page }) => {
   const { testUrl, server } = await setupDevServer('ts-lib')
   await page.goto(testUrl)
-  await expect(page.locator('main')).toHaveText('Home page')
-
-  await page.locator('a', { hasText: 'About' }).click()
-  await expect(page.locator('main')).toHaveText('About page')
-
+  await testNonJs(page)
   await server.close()
 })
+
+async function testNonJs(page: Page) {
+  await expect(page.getByTestId('test-non-js')).toHaveText('test-non-js: 0')
+  await page.getByTestId('test-non-js').click()
+  await expect(page.getByTestId('test-non-js')).toHaveText('test-non-js: 1')
+}
diff --git a/packages/plugin-react-swc/playground/ts-lib/package.json b/packages/plugin-react-swc/playground/ts-lib/package.json
@@ -8,11 +8,9 @@
     "preview": "vite preview"
   },
   "dependencies": {
-    "@generouted/react-router": "^1.20.0",
-    "generouted": "1.11.7",
+    "@vitejs/test-dep-non-js": "file:./test-dep/non-js",
     "react": "^19.1.0",
-    "react-dom": "^19.1.0",
-    "react-router-dom": "^7.6.3"
+    "react-dom": "^19.1.0"
   },
   "devDependencies": {
     "@types/react": "^19.1.8",
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/app.tsx b/packages/plugin-react-swc/playground/ts-lib/src/app.tsx
@@ -0,0 +1,11 @@
+import TestNonJs from '@vitejs/test-dep-non-js'
+
+export default function App() {
+  return (
+    <section style={{ margin: 24 }}>
+      <main>
+        <TestNonJs />
+      </main>
+    </section>
+  )
+}
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/index.tsx b/packages/plugin-react-swc/playground/ts-lib/src/index.tsx
@@ -1,9 +1,9 @@
 import { StrictMode } from 'react'
 import { createRoot } from 'react-dom/client'
-import { Routes } from 'generouted/react-router'
+import App from './app'
 
 createRoot(document.getElementById('root')!).render(
   <StrictMode>
-    <Routes />
+    <App />
   </StrictMode>,
 )
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/pages/404.tsx b/packages/plugin-react-swc/playground/ts-lib/src/pages/404.tsx
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/pages/_app.tsx b/packages/plugin-react-swc/playground/ts-lib/src/pages/_app.tsx
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/pages/about.tsx b/packages/plugin-react-swc/playground/ts-lib/src/pages/about.tsx
diff --git a/packages/plugin-react-swc/playground/ts-lib/src/pages/index.tsx b/packages/plugin-react-swc/playground/ts-lib/src/pages/index.tsx
diff --git a/packages/plugin-react-swc/playground/ts-lib/test-dep/non-js/index.tsx b/packages/plugin-react-swc/playground/ts-lib/test-dep/non-js/index.tsx
@@ -0,0 +1,10 @@
+import React from 'react'
+
+export default function TestNonJs() {
+  const [count, setCount] = React.useState(0)
+  return (
+    <button data-testid="test-non-js" onClick={() => setCount(count + 1)}>
+      test-non-js: {count}
+    </button>
+  )
+}
diff --git a/packages/plugin-react-swc/playground/ts-lib/test-dep/non-js/package.json b/packages/plugin-react-swc/playground/ts-lib/test-dep/non-js/package.json
@@ -0,0 +1,9 @@
+{
+  "name": "@vitejs/test-dep-non-js",
+  "private": true,
+  "type": "module",
+  "exports": "./index.tsx",
+  "peerDependencies": {
+    "react": "*"
+  }
+}
diff --git a/packages/plugin-react-swc/playground/ts-lib/vite.config.ts b/packages/plugin-react-swc/playground/ts-lib/vite.config.ts
@@ -2,6 +2,5 @@ import { defineConfig } from 'vite'
 import react from '@vitejs/plugin-react-swc'
 
 export default defineConfig({
-  optimizeDeps: { include: ['react-router-dom'] },
   plugins: [react()],
 })
diff --git a/packages/plugin-rsc/examples/react-router/cf/vite.config.ts b/packages/plugin-rsc/examples/react-router/cf/vite.config.ts
@@ -36,19 +36,6 @@ export default defineConfig({
         },
       ],
     }),
-    {
-      name: 'react-router-fixup',
-      transform(code) {
-        if (code.includes(`import { AsyncLocalStorage } from 'async_hooks';`)) {
-          code = code.replaceAll('async_hooks', 'node:async_hooks')
-          code = code.replaceAll(
-            `global.___reactRouterServerStorage___`,
-            `globalThis.___reactRouterServerStorage___`,
-          )
-          return code
-        }
-      },
-    },
   ],
   environments: {
     client: {
@@ -61,12 +48,24 @@ export default defineConfig({
         include: ['react-router > cookie', 'react-router > set-cookie-parser'],
         exclude: ['react-router'],
       },
+      build: {
+        rollupOptions: {
+          // @ts-ignore rolldown
+          platform: 'neutral',
+        },
+      },
     },
     rsc: {
       optimizeDeps: {
         include: ['react-router > cookie', 'react-router > set-cookie-parser'],
         exclude: ['react-router'],
       },
+      build: {
+        rollupOptions: {
+          // @ts-ignore rolldown
+          platform: 'neutral',
+        },
+      },
     },
   },
 })
diff --git a/packages/plugin-rsc/examples/starter-cf-single/vite.config.ts b/packages/plugin-rsc/examples/starter-cf-single/vite.config.ts
@@ -3,7 +3,7 @@ import rsc from '@vitejs/plugin-rsc'
 import react from '@vitejs/plugin-react'
 import { defineConfig } from 'vite'
 
-export default defineConfig((_env) => ({
+export default defineConfig({
   clearScreen: false,
   build: {
     minify: false,
@@ -31,6 +31,14 @@ export default defineConfig((_env) => ({
         rollupOptions: {
           // ensure `default` export only in cloudflare entry output
           preserveEntrySignatures: 'exports-only',
+          // @ts-ignore rolldown
+          platform: 'neutral',
+        },
+      },
+      optimizeDeps: {
+        // @ts-ignore rolldown
+        rollupOptions: {
+          platform: 'neutral',
         },
       },
     },
@@ -43,4 +51,4 @@ export default defineConfig((_env) => ({
       },
     },
   },
-}))
+})
diff --git a/packages/plugin-rsc/playwright.config.ts b/packages/plugin-rsc/playwright.config.ts
@@ -26,6 +26,7 @@ export default defineConfig({
       use: devices['Desktop Safari'],
     },
   ],
+  workers: 1,
   forbidOnly: !!process.env.CI,
   retries: process.env.CI ? 2 : 0,
   reporter: ['list', process.env.CI && 'github']
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -70,15 +70,5 @@`
`70`	`70`	`"playground//__tests__//*.ts": [`
`71`	`71`	`"eslint --cache --fix"`
`72`	`72`	`]`
`73`		`- },`
`74`		`- "pnpm": {`
`75`		`- "packageExtensions": {`
`76`		`- "generouted": {`
`77`		`- "peerDependencies": {`
`78`		`- "react": "*",`
`79`		`- "react-router-dom": "*"`
`80`		`- }`
`81`		`- }`
`82`		`- }`
`83`	`73`	`}`
`84`	`74`	`}`