Skip to content

Remove PrerenderStorePPR type and all prerender-ppr switch cases#90721

Merged
devjiwonchoi merged 1 commit intocanaryfrom
jiwon-2026-03-01/remove-prerender-ppr
Mar 4, 2026
Merged

Remove PrerenderStorePPR type and all prerender-ppr switch cases#90721
devjiwonchoi merged 1 commit intocanaryfrom
jiwon-2026-03-01/remove-prerender-ppr

Conversation

@devjiwonchoi
Copy link
Member

@devjiwonchoi devjiwonchoi commented Mar 1, 2026 •
edited
Loading

Stacked on #90713

This PR removes prerender-ppr case handling in the codebase.

Verified deployment tests pass on CI: https://github.com/vercel/next.js/actions/runs/22539297668

@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Mar 1, 2026 •
edited
Loading

Stats from current PR

🟢 1 improvement

Metric Canary PR Change Trend
node_modules Size 475 MB 474 MB 🟢 767 kB (0%) ▁▁▁██
📊 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) 507ms 507ms ✓ ▁█▁▁█
Cold (Ready in log) 489ms 492ms ✓ ▁█▁▂█
Cold (First Request) 1.059s 1.060s ✓ ▁█▁▄█
Warm (Listen) 506ms 507ms ✓ ▁█▁▁█
Warm (Ready in log) 489ms 492ms ✓ ▁█▁▁█
Warm (First Request) 393ms 397ms ✓ ▁█▁▁█
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 457ms 457ms ✓ ▁▁▁▁▁
Cold (Ready in log) 459ms 459ms ✓ ▁▁▃▁▂
Cold (First Request) 2.066s 2.070s ✓ ▁▁▂▁▁
Warm (Listen) 456ms 456ms ✓ ▁▁▁▁▁
Warm (Ready in log) 452ms 452ms ✓ ▁▁▃▂▂
Warm (First Request) 2.062s 2.063s ✓ ▁▁▁▂▁

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 4.668s 4.606s ✓ ▃█▁▁█
Cached Build 4.623s 4.625s ✓ ▃█▁▁█
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 15.408s 15.348s ✓ ▁▁▁▂▁
Cached Build 15.409s 15.324s ✓ ▁▁▁▁▁
node_modules Size 475 MB 474 MB 🟢 767 kB (0%) ▁▁▁██
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles: **401 kB** → **400 kB** ✅ -205 B

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

Server

Middleware
Canary PR Change
middleware-b..fest.js gzip 761 B 764 B ✓
Total 761 B 764 B ⚠️ +3 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.54 kB N/A -
6280-HASH.js gzip 58.4 kB N/A -
6335.HASH.js gzip 169 B N/A -
912-HASH.js gzip 4.59 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 253 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.59 kB -
2889.HASH.js gzip N/A 169 B -
5602-HASH.js gzip N/A 5.55 kB -
6948ada0-HASH.js gzip N/A 62.6 kB -
9544-HASH.js gzip N/A 59.1 kB -
Total 232 kB 233 kB ⚠️ +699 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 254 kB 253 kB ✓
Total 379 kB 378 kB ✅ -989 B
Middleware
Canary PR Change
middleware-b..fest.js gzip 616 B 614 B ✓
middleware-r..fest.js gzip 156 B 155 B ✓
middleware.js gzip 43.6 kB 43.6 kB ✓
edge-runtime..pack.js gzip 842 B 842 B ✓
Total 45.2 kB 45.2 kB ⚠️ +2 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 4.03 MB 4.01 MB 🟢 13.5 kB (0%)
index.pack gzip 102 kB 102 kB ✓
index.pack.old gzip 103 kB 103 kB ✓
Total 4.23 MB 4.22 MB ✅ -13.8 kB

🔄 Shared (bundler-independent)

Runtimes
Canary PR Change
app-page-exp...dev.js gzip 320 kB 320 kB ✓
app-page-exp..prod.js gzip 170 kB 169 kB ✓
app-page-tur...dev.js gzip 320 kB 319 kB ✓
app-page-tur..prod.js gzip 170 kB 169 kB ✓
app-page-tur...dev.js gzip 316 kB 316 kB ✓
app-page-tur..prod.js gzip 168 kB 167 kB ✓
app-page.run...dev.js gzip 317 kB 316 kB ✓
app-page.run..prod.js gzip 168 kB 167 kB ✓
app-route-ex...dev.js gzip 70.8 kB 70.5 kB ✓
app-route-ex..prod.js gzip 49.2 kB 48.8 kB ✓
app-route-tu...dev.js gzip 70.9 kB 70.5 kB ✓
app-route-tu..prod.js gzip 49.3 kB 48.8 kB ✓
app-route-tu...dev.js gzip 70.4 kB 70.1 kB ✓
app-route-tu..prod.js gzip 49 kB 48.6 kB ✓
app-route.ru...dev.js gzip 70.4 kB 70.1 kB ✓
app-route.ru..prod.js gzip 49 kB 48.6 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.9 kB 32.9 kB ✓
pages-turbo....dev.js gzip 52.6 kB 52.6 kB ✓
pages-turbo...prod.js gzip 38.5 kB 38.5 kB ✓
pages.runtim...dev.js gzip 52.6 kB 52.6 kB ✓
pages.runtim..prod.js gzip 38.5 kB 38.5 kB ✓
server.runti..prod.js gzip 61.9 kB 61.9 kB ✓
Total 2.82 MB 2.82 MB ✅ -6.88 kB
📝 Changed Files (17 files)

Files with changes:

  • app-page-exp..ntime.dev.js
  • app-page-exp..time.prod.js
  • app-page-tur..ntime.dev.js
  • app-page-tur..time.prod.js
  • app-page-tur..ntime.dev.js
  • app-page-tur..time.prod.js
  • app-page.runtime.dev.js
  • app-page.runtime.prod.js
  • app-route-ex..ntime.dev.js
  • app-route-ex..time.prod.js
  • app-route-tu..ntime.dev.js
  • app-route-tu..time.prod.js
  • app-route-tu..ntime.dev.js
  • app-route-tu..time.prod.js
  • app-route.runtime.dev.js
  • app-route.ru..time.prod.js
  • server.runtime.prod.js
View diffs
app-page-exp..ntime.dev.js 
failed to diff
app-page-exp..time.prod.js 
failed to diff
app-page-tur..ntime.dev.js 
failed to diff
app-page-tur..time.prod.js 
failed to diff
app-page-tur..ntime.dev.js 
failed to diff
app-page-tur..time.prod.js 
failed to diff
app-page.runtime.dev.js 
failed to diff
app-page.runtime.prod.js 
failed to diff
app-route-ex..ntime.dev.js

Diff too large to display

app-route-ex..time.prod.js

Diff too large to display

app-route-tu..ntime.dev.js

Diff too large to display

app-route-tu..time.prod.js

Diff too large to display

app-route-tu..ntime.dev.js

Diff too large to display

app-route-tu..time.prod.js

Diff too large to display

app-route.runtime.dev.js

Diff too large to display

app-route.ru..time.prod.js

Diff too large to display

server.runtime.prod.js

Diff too large to display

📎 Tarball URL 
next@https://vercel-packages.vercel.app/next/prs/90721/next

@devjiwonchoi devjiwonchoi marked this pull request as ready for review March 1, 2026 08:47
@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Mar 1, 2026 •
edited
Loading

Tests Passed

@devjiwonchoi devjiwonchoi force-pushed the jiwon-2026-03-01/remove-prerender-ppr branch from 6007781 to 7b1e778 Compare March 1, 2026 09:35
This was referenced Mar 1, 2026
Merged
Merged
@devjiwonchoi devjiwonchoi requested a review from unstubbable March 4, 2026 12:57
@devjiwonchoi devjiwonchoi force-pushed the jiwon-2026-03-01/segment-explorer branch from 4db6f5c to d0635b0 Compare March 4, 2026 15:00
Base automatically changed from jiwon-2026-03-01/segment-explorer to canary March 4, 2026 15:43
@devjiwonchoi
Remove PrerenderStorePPR type and all case 'prerender-ppr': switc…
bea5fa7 
…h cases

After PR #90713 removed the rendering branch that creates `prerender-ppr`
stores, no code ever creates a store with `type: 'prerender-ppr'`. All switch
cases handling it are dead code.

- Delete `PrerenderStorePPR` interface and remove from `PrerenderStore` union
- Remove 78 `case 'prerender-ppr':` switch cases across 28 files
- Simplify `Postpone` component (dynamicTracking is always null now)
- Remove dead `postponeWithTracking()` call sites only reachable from prerender-ppr
- Clean up now-unused imports and dead functions
@devjiwonchoi devjiwonchoi force-pushed the jiwon-2026-03-01/remove-prerender-ppr branch from 7b1e778 to bea5fa7 Compare March 4, 2026 16:04
@devjiwonchoi devjiwonchoi enabled auto-merge (squash) March 4, 2026 16:05
@devjiwonchoi devjiwonchoi merged commit c37ef48 into canary Mar 4, 2026
278 of 283 checks passed
@devjiwonchoi devjiwonchoi deleted the jiwon-2026-03-01/remove-prerender-ppr branch March 4, 2026 16:46
devjiwonchoi added a commit that referenced this pull request Mar 4, 2026
@devjiwonchoi
Remove legacy PPR postpone cluster (#90726)
b4013ed 
Stacked on #90721

Verified deployment tests pass on CI:
https://github.com/vercel/next.js/actions/runs/22541051404
unstubbable added a commit that referenced this pull request Mar 5, 2026
@unstubbable
Revert "Remove PrerenderStorePPR type and all prerender-ppr switc…
586f1fb 
…h cases (#90721)"

This reverts commit c37ef48.
sokra pushed a commit that referenced this pull request Mar 6, 2026
@devjiwonchoi @sokra
Remove PrerenderStorePPR type and all prerender-ppr switch cases (#…
74a090a 
…90721)

Stacked on #90713

This PR removes `prerender-ppr` case handling in the codebase.

Verified deployment tests pass on CI:
https://github.com/vercel/next.js/actions/runs/22539297668
sokra pushed a commit that referenced this pull request Mar 6, 2026
@devjiwonchoi @sokra
Remove legacy PPR postpone cluster (#90726)
503a632 
Stacked on #90721

Verified deployment tests pass on CI:
https://github.com/vercel/next.js/actions/runs/22541051404
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@unstubbable unstubbable unstubbable approved these changes

Assignees

No one assigned

Labels

created-by: Next.js team PRs by the Next.js team. tests type: next

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@devjiwonchoi @nextjs-bot @unstubbable