Skip to content

Turbopack: Fix experimental server hmr for pages router#90398

Merged
wbinnssmith merged 2 commits intocanaryfrom
server-hmr-cache-fix
Feb 24, 2026
Merged

Turbopack: Fix experimental server hmr for pages router#90398
wbinnssmith merged 2 commits intocanaryfrom
server-hmr-cache-fix

Conversation

@wbinnssmith
Copy link
Member

This is cherry-picked from #90389, which also serves as a test case for server hmr being enabled by default.

This fixes a number of issues with server hmr, most notably that pages router updates weren’t reflected:

  1. Pages Router pages weren't clearing the chunk cache — the old condition (which just checked the cli flag) meant the chunk cache was only cleared when server HMR was entirely absent. Once server HMR was enabled, Pages Router pages (which don't use the Turbopack module registry) got stale.

  2. deleteCache was skipped for files with server HMR subscriptions — this left stale entries in evalManifest's sharedCache, causing "Could not find module in React Client Manifest" errors when new 'use client' components were added.

  3. clearModuleContext was skipped for middleware and edge routes — server HMR subscriptions include edge runtime chunks, but __turbopack_server_hmr_apply__ can't reach those sandboxes. They need clearModuleContext called explicitly.

  4. Chunk cache wasn't cleared for .env/config changes — when force: true (no code change), module-level env captures stayed stale. Now __next__clear_chunk_cache__() is called unconditionally when force is set.

This is cherry-picked from #90389, which also serves as a test case for server hmr being enabled by default.

This fixes a number of issues with server hmr, most notably that pages router updates weren’t reflected:

1. **Pages Router pages weren't clearing the chunk cache** — the old condition (which just checked the cli flag) meant the chunk cache was only cleared when server HMR was entirely absent. Once server HMR was enabled, Pages Router pages (which don't use the Turbopack module registry) got stale.

2. **`deleteCache` was skipped for files with server HMR subscriptions** — this left stale entries in `evalManifest`'s `sharedCache`, causing "Could not find module in React Client Manifest" errors when new `'use client'` components were added.

3. **`clearModuleContext` was skipped for middleware and edge routes** — server HMR subscriptions include edge runtime chunks, but `__turbopack_server_hmr_apply__` can't reach those sandboxes. They need `clearModuleContext` called explicitly.

4. **Chunk cache wasn't cleared for `.env`/config changes** — when `force: true` (no code change), module-level env captures stayed stale. Now `__next__clear_chunk_cache__()` is called unconditionally when `force` is set.
@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Feb 24, 2026

Failing test suites

Commit: 8bb3f7f | About building and testing Next.js

pnpm test-dev test/e2e/app-dir/typed-routes-validator/typed-routes-validator.test.ts (job)

  • typed-routes-validator > should have passing tsc after the server generated types (DD)
Expand output

● typed-routes-validator › should have passing tsc after the server generated types

expect(received).toEqual(expected) // deep equality

- Expected  - 1
+ Received  + 2

  Object {
    "stderr": "",
-   "stdout": "",
+   "stdout": "app/layout.tsx(1,37): error TS2304: Cannot find name 'LayoutProps'.
+ app/page.tsx(1,37): error TS2304: Cannot find name 'PageProps'.",
  }

  48 |       })
  49 |
> 50 |       expect({ stdout, stderr }).toEqual({
     |                                  ^
  51 |         stdout: '',
  52 |         stderr: '',
  53 |       })

  at Object.toEqual (e2e/app-dir/typed-routes-validator/typed-routes-validator.test.ts:50:34)

@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Feb 24, 2026

Stats from current PR

✅ No significant changes detected

📊 All Metrics
📖 Metrics Glossary

Dev Server Metrics:

  • Listen = TCP port starts accepting connections
  • First Request = HTTP server returns successful response
  • Cold = Fresh build (no cache)
  • Warm = With cached build artifacts

Build Metrics:

  • Fresh = Clean build (no .next directory)
  • Cached = With existing .next directory

Change Thresholds:

  • Time: Changes < 50ms AND < 10%, OR < 2% are insignificant
  • Size: Changes < 1KB AND < 1% are insignificant
  • All other changes are flagged to catch regressions

⚡ Dev Server

Metric Canary PR Change Trend
Cold (Listen) 456ms 455ms ▁▁▁▁▆
Cold (Ready in log) 440ms 440ms ▁▄▄▁▅
Cold (First Request) 1.235s 1.246s ▁█▇▁▅
Warm (Listen) 456ms 456ms ▁▁▁▁▃
Warm (Ready in log) 444ms 449ms ▁▁▁▁▅
Warm (First Request) 349ms 353ms ▁▂▁▂▆
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 456ms 456ms ▁▁▁▁▁
Cold (Ready in log) 438ms 438ms ▃▁▂▂▃
Cold (First Request) 1.941s 1.924s ▂▁▄▄▂
Warm (Listen) 455ms 456ms ▁▁▁▁▁
Warm (Ready in log) 439ms 438ms ▅▁▃▂▃
Warm (First Request) 1.991s 1.981s ▃▁▅▄▂

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 3.969s 4.021s ▂▁▁▁▆
Cached Build 3.984s 4.054s ▂▁▁▂▆
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 14.042s 14.060s ▁▁▅▅▁
Cached Build 14.251s 14.257s ▁▁▅▅▁
node_modules Size 474 MB 474 MB █████
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles: **399 kB** → **399 kB** ✅ -1 B

80 files with content-based hashes (individual files not comparable between builds)

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 764 B 762 B
Total 764 B 762 B ✅ -2 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 451 B 450 B
Total 451 B 450 B ✅ -1 B

📦 Webpack

Client

Main Bundles
Canary PR Change
5528-HASH.js gzip 5.48 kB N/A -
6280-HASH.js gzip 57.5 kB N/A -
6335.HASH.js gzip 169 B N/A -
912-HASH.js gzip 4.53 kB N/A -
e8aec2e4-HASH.js gzip 62.6 kB N/A -
framework-HASH.js gzip 59.7 kB 59.7 kB
main-app-HASH.js gzip 255 B 254 B
main-HASH.js gzip 39.1 kB 39.1 kB
webpack-HASH.js gzip 1.68 kB 1.68 kB
262-HASH.js gzip N/A 4.53 kB -
2889.HASH.js gzip N/A 169 B -
5602-HASH.js gzip N/A 5.49 kB -
6948ada0-HASH.js gzip N/A 62.6 kB -
9544-HASH.js gzip N/A 58.3 kB -
Total 231 kB 232 kB ⚠️ +747 B
Polyfills
Canary PR Change
polyfills-HASH.js gzip 39.4 kB 39.4 kB
Total 39.4 kB 39.4 kB
Pages
Canary PR Change
_app-HASH.js gzip 194 B 194 B
_error-HASH.js gzip 183 B 180 B 🟢 3 B (-2%)
css-HASH.js gzip 331 B 330 B
dynamic-HASH.js gzip 1.81 kB 1.81 kB
edge-ssr-HASH.js gzip 256 B 256 B
head-HASH.js gzip 351 B 352 B
hooks-HASH.js gzip 384 B 383 B
image-HASH.js gzip 580 B 581 B
index-HASH.js gzip 260 B 260 B
link-HASH.js gzip 2.5 kB 2.5 kB
routerDirect..HASH.js gzip 320 B 319 B
script-HASH.js gzip 386 B 386 B
withRouter-HASH.js gzip 315 B 315 B
1afbb74e6ecf..834.css gzip 106 B 106 B
Total 7.97 kB 7.97 kB ✅ -2 B

Server

Edge SSR
Canary PR Change
edge-ssr.js gzip 125 kB 125 kB
page.js gzip 252 kB 253 kB
Total 377 kB 378 kB ⚠️ +521 B
Middleware
Canary PR Change
middleware-b..fest.js gzip 614 B 615 B
middleware-r..fest.js gzip 156 B 155 B
middleware.js gzip 43.9 kB 43.8 kB
edge-runtime..pack.js gzip 842 B 842 B
Total 45.5 kB 45.5 kB ✅ -48 B
Build Details
Build Manifests
Canary PR Change
_buildManifest.js gzip 715 B 718 B
Total 715 B 718 B ⚠️ +3 B
Build Cache
Canary PR Change
0.pack gzip 3.97 MB 3.99 MB 🔴 +13.5 kB (+0%)
index.pack gzip 103 kB 102 kB
index.pack.old gzip 102 kB 102 kB
Total 4.18 MB 4.19 MB ⚠️ +13.3 kB

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 318 kB 318 kB
app-page-exp..prod.js gzip 168 kB 168 kB
app-page-tur...dev.js gzip 318 kB 318 kB
app-page-tur..prod.js gzip 168 kB 168 kB
app-page-tur...dev.js gzip 314 kB 314 kB
app-page-tur..prod.js gzip 166 kB 166 kB
app-page.run...dev.js gzip 315 kB 315 kB
app-page.run..prod.js gzip 167 kB 167 kB
app-route-ex...dev.js gzip 70.7 kB 70.7 kB
app-route-ex..prod.js gzip 49.1 kB 49.1 kB
app-route-tu...dev.js gzip 70.7 kB 70.7 kB
app-route-tu..prod.js gzip 49.2 kB 49.2 kB
app-route-tu...dev.js gzip 70.3 kB 70.3 kB
app-route-tu..prod.js gzip 48.9 kB 48.9 kB
app-route.ru...dev.js gzip 70.2 kB 70.2 kB
app-route.ru..prod.js gzip 48.9 kB 48.9 kB
dist_client_...dev.js gzip 324 B 324 B
dist_client_...dev.js gzip 326 B 326 B
dist_client_...dev.js gzip 318 B 318 B
dist_client_...dev.js gzip 317 B 317 B
pages-api-tu...dev.js gzip 43.2 kB 43.2 kB
pages-api-tu..prod.js gzip 32.9 kB 32.9 kB
pages-api.ru...dev.js gzip 43.2 kB 43.2 kB
pages-api.ru..prod.js gzip 32.8 kB 32.8 kB
pages-turbo....dev.js gzip 52.5 kB 52.5 kB
pages-turbo...prod.js gzip 38.5 kB 38.5 kB
pages.runtim...dev.js gzip 52.5 kB 52.5 kB
pages.runtim..prod.js gzip 38.4 kB 38.4 kB
server.runti..prod.js gzip 62 kB 62 kB
Total 2.81 MB 2.81 MB ✅ -4 B
📎 Tarball URL
next@https://vercel-packages.vercel.app/next/prs/90398/next

@wbinnssmith wbinnssmith marked this pull request as ready for review February 24, 2026 00:55
@wbinnssmith wbinnssmith force-pushed the server-hmr-cache-fix branch 2 times, most recently from d2bd2a2 to 1e69bbc Compare February 24, 2026 01:38
… HMR chunks

- Add comment explaining the usesServerHmr condition (App Router Node.js only)
- Move deleteCache to after the serverHmrSubscriptions check so it is skipped
  for server HMR module chunks. Calling it unconditionally risks resetting the
  Turbopack runtime's devModuleCache if onUpdateFailed later clears loadedChunks
  and the runtime chunk gets re-executed from disk.
@wbinnssmith wbinnssmith merged commit affb856 into canary Feb 24, 2026
529 of 537 checks passed
@wbinnssmith wbinnssmith deleted the server-hmr-cache-fix branch February 24, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants