Skip to content

Replace PPR config helpers with cacheComponents checks#90704

Merged
devjiwonchoi merged 2 commits intocanaryfrom
jiwon-2026-02-28/pr-89648-review
Mar 4, 2026
Merged

Replace PPR config helpers with cacheComponents checks#90704
devjiwonchoi merged 2 commits intocanaryfrom
jiwon-2026-02-28/pr-89648-review

Conversation

@devjiwonchoi
Copy link
Member

@devjiwonchoi devjiwonchoi commented Feb 28, 2026

This PR replaces the legacy PPR configs in the codebase with the cacheComponents config.

Confirmed deployment test passes https://github.com/vercel/next.js/actions/runs/22533307017

The `cacheComponents` flag already auto-enables `experimental.ppr` during
config normalization (config.ts line 1399), making the PPR helper functions
`checkIsAppPPREnabled` and `checkIsRoutePPREnabled` just unnecessary
indirection. This consolidates PPR configuration to a single source of truth.

Since users cannot set `experimental.ppr` directly (throws
HardDeprecatedConfigError), the behavior is unchanged when replacing PPR
config checks with direct `cacheComponents` checks.

This is the first PR of a 3-part series to safely remove legacy PPR code
incrementally.
@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Feb 28, 2026

Tests Passed

@nextjs-bot
Copy link
Collaborator

Stats from current PR

🟢 1 improvement

Metric Canary PR Change Trend
node_modules Size 475 MB 475 MB 🟢 52.8 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) 456ms 455ms ▁▁▁█▁
Cold (Ready in log) 443ms 443ms ▂▂▁█▁
Cold (First Request) 890ms 891ms ▄▄▁█▁
Warm (Listen) 456ms 456ms ▁▁▁█▁
Warm (Ready in log) 439ms 441ms ▁▁▁█▁
Warm (First Request) 347ms 351ms ▁▁▁█▁
📦 Dev Server (Webpack) (Legacy)

📦 Dev Server (Webpack)

Metric Canary PR Change Trend
Cold (Listen) 455ms 456ms ▁▁▁▁▁
Cold (Ready in log) 436ms 437ms ▂▃▁▁▃
Cold (First Request) 1.843s 1.840s ▂▁▁▁▂
Warm (Listen) 455ms 456ms ▁▁▁▁▁
Warm (Ready in log) 435ms 436ms ▂▂▁▁▃
Warm (First Request) 1.871s 1.864s ▁▁▁▁▂

⚡ Production Builds

Metric Canary PR Change Trend
Fresh Build 4.188s 4.162s ▁▁▃█▁
Cached Build 4.119s 4.172s ▁▁▃█▁
📦 Production Builds (Webpack) (Legacy)

📦 Production Builds (Webpack)

Metric Canary PR Change Trend
Fresh Build 13.962s 14.026s ▁▁▁▁▁
Cached Build 14.186s 14.083s ▁▁▁▁▁
node_modules Size 475 MB 475 MB 🟢 52.8 kB (0%) ▁▁▁▁▁
📦 Bundle Sizes

Bundle Sizes

⚡ Turbopack

Client

Main Bundles: **401 kB** → **401 kB** ✅ -23 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 449 B 450 B
Total 449 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 253 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 ⚠️ +701 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 254 kB
Total 379 kB 379 kB ⚠️ +150 B
Middleware
Canary PR Change
middleware-b..fest.js gzip 616 B 615 B
middleware-r..fest.js gzip 156 B 155 B
middleware.js gzip 43.9 kB 43.9 kB
edge-runtime..pack.js gzip 842 B 842 B
Total 45.5 kB 45.5 kB ✅ -33 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.02 MB 4.02 MB 🟢 4.78 kB (0%)
index.pack gzip 102 kB 101 kB 🟢 1.23 kB (-1%)
index.pack.old gzip 103 kB 103 kB
Total 4.23 MB 4.22 MB ✅ -5.53 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 170 kB
app-page-tur...dev.js gzip 320 kB 320 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 168 kB
app-page.run...dev.js gzip 317 kB 317 kB
app-page.run..prod.js gzip 168 kB 168 kB
app-route-ex...dev.js gzip 70.8 kB 70.8 kB
app-route-ex..prod.js gzip 49.2 kB 49.2 kB
app-route-tu...dev.js gzip 70.9 kB 70.9 kB
app-route-tu..prod.js gzip 49.3 kB 49.3 kB
app-route-tu...dev.js gzip 70.4 kB 70.4 kB
app-route-tu..prod.js gzip 49 kB 49 kB
app-route.ru...dev.js gzip 70.4 kB 70.4 kB
app-route.ru..prod.js gzip 49 kB 49 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 ✅ -214 B
📝 Changed Files (9 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
  • server.runtime.prod.js
View diffs
app-page-exp..ntime.dev.js
failed to diff
app-page-exp..time.prod.js

Diff too large to display

app-page-tur..ntime.dev.js
failed to diff
app-page-tur..time.prod.js

Diff too large to display

app-page-tur..ntime.dev.js

Diff too large to display

app-page-tur..time.prod.js
failed to diff
app-page.runtime.dev.js

Diff too large to display

app-page.runtime.prod.js
failed to diff
server.runtime.prod.js

Diff too large to display

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

@devjiwonchoi devjiwonchoi marked this pull request as ready for review March 1, 2026 02:31
@vercel
Copy link
Contributor

vercel bot commented Mar 1, 2026

Notifying the following users due to files changed in this PR based on this repo's notify modifiers:

@timneutkens, @ijjk, @shuding, @huozhi:

packages/next/src/server/config.ts

)

const isAppPPREnabled = checkIsAppPPREnabled(config.experimental.ppr)
const isAppPPREnabled = !!config.cacheComponents
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename the const as well

const couldSupportPPR: boolean = checkIsAppPPREnabled(
nextConfig.experimental.ppr
)
const couldSupportPPR: boolean = !!nextConfig.cacheComponents
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename the const as well

config.experimental.authInterrupts
)
const isAppPPREnabled = checkIsAppPPREnabled(config.experimental.ppr)
const isAppPPREnabled = isAppCacheComponentsEnabled
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use isAppCacheComponentsEnabled at the usage sites

? checkIsRoutePPREnabled(config.experimental.ppr)
: false
const isRoutePPREnabled: boolean =
!!appConfig && isAppCacheComponentsEnabled
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note to self: this gets removed in the upstack PR

* The Partial Prerendering feature is still available via `cacheComponents`.
*/
ppr?: ExperimentalPPRConfig
ppr?: boolean | 'incremental'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this be removed in an upstack PR?

Copy link
Member Author

@devjiwonchoi devjiwonchoi Mar 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this PR, the entire file is deleted: packages/next/src/server/lib/experimental/ppr.ts forExperimentalPPRConfig

EDIT: Do you mean the ppr option?

Co-authored-by: Hendrik Liebau <mail@hendrik-liebau.de>
@devjiwonchoi
Copy link
Member Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Another round soon, please!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Mar 4, 2026

Tests Passed

@devjiwonchoi devjiwonchoi requested a review from unstubbable March 4, 2026 13:59
@devjiwonchoi devjiwonchoi enabled auto-merge (squash) March 4, 2026 14:07
@devjiwonchoi devjiwonchoi merged commit 70a2e8d into canary Mar 4, 2026
535 of 545 checks passed
@devjiwonchoi devjiwonchoi deleted the jiwon-2026-02-28/pr-89648-review branch March 4, 2026 14:56
devjiwonchoi added a commit that referenced this pull request Mar 4, 2026
…anch (#90713)

Stacked on #90704

This PR replaces `experimental.isRoutePPREnabled` render options with
`cacheComponents` and removes unused code.

Verified deployment tests pass on CI:
https://github.com/vercel/next.js/actions/runs/22536405880
unstubbable added a commit that referenced this pull request Mar 5, 2026
unstubbable added a commit that referenced this pull request Mar 5, 2026
sokra pushed a commit that referenced this pull request Mar 6, 2026
This PR replaces the legacy PPR configs in the codebase with the
`cacheComponents` config.

Confirmed deployment test passes
https://github.com/vercel/next.js/actions/runs/22533307017

---------

Co-authored-by: Hendrik Liebau <mail@hendrik-liebau.de>
sokra pushed a commit that referenced this pull request Mar 6, 2026
…anch (#90713)

Stacked on #90704

This PR replaces `experimental.isRoutePPREnabled` render options with
`cacheComponents` and removes unused code.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants