Skip to content

Remove leaf segment force-refetch check#90836

Merged
acdlite merged 1 commit intovercel:canaryfrom
acdlite:remove-leaf-force-refetch
Mar 3, 2026
Merged

Remove leaf segment force-refetch check#90836
acdlite merged 1 commit intovercel:canaryfrom
acdlite:remove-leaf-force-refetch

Conversation

@acdlite
Copy link
Contributor

@acdlite acdlite commented Mar 3, 2026

Remove the parallelRoutesKeys.length === 0 condition from renderComponentsOnThisLevel in walk-tree-with-flight-router-state.

This check was added to support the behavior where clicking the same link twice forces all page segments to re-render on the server. This should no longer be necessary because the client already adds a refetch marker on those segments, so there's no need to special-case leaf segments on the server side.

Remove the `parallelRoutesKeys.length === 0` condition from
`renderComponentsOnThisLevel` in walk-tree-with-flight-router-state.

This check was added to support the behavior where clicking the same
link twice forces all page segments to re-render on the server. This
should no longer be necessary because the client already adds a refetch
marker on those segments, so there's no need to special-case leaf
segments on the server side.
@nextjs-bot
Copy link
Collaborator

nextjs-bot commented Mar 3, 2026

Tests Passed

Copy link
Contributor

@gnoff gnoff left a comment

Choose a reason for hiding this comment

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

LFGTM

@acdlite acdlite marked this pull request as ready for review March 3, 2026 22:53
@acdlite acdlite merged commit 3411e99 into vercel:canary Mar 3, 2026
247 of 268 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants