Skip to content

docs: place shell param-order rule with build-time shells - #101680

Merged
icyJoseph merged 2 commits into
canaryfrom
claude/isr-shell-param-order
Nov 14, 2026
Merged

icyJoseph merged 2 commits into
canaryfrom
claude/isr-shell-param-order

Conversation

@icyJoseph

@icyJoseph icyJoseph commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Summary

  • ISR with Cache Components guide: moves the rule that shells leave params open only from the end of the route out of "What the upgrade produces" and into "At build time", where it describes which shells exist. Under the upgrade section it read as a limit on the upgrade itself, which confused a reader in Is route with unlisted params for nested dynamic route segments upgraded after first visit?Β #101644.
  • Caching getting-started page: the prerendered-output bullet now says the RSC payload is stored alongside the HTML and serves client navigations.

Possible follow-ups

Gaps surfaced while answering the discussion, to stack on top of this PR:

  • The general rule that the build records whether a route reads cookies, headers, or URL data, so a prefetch is served from static output when a runtime render can't add more UI. Today only the navigation() case is covered, in "Keep the prefetch static".
  • Fallback and narrower shells (/[category]/[product], /tops/[product]) regenerate on the revalidate and expire of the content inside them. The docs cover only the static shell and the concrete page.
  • A segment without generateStaticParams produces a shared shell on upgrade (/shoes/[product]), while one with it upgrades straight to the concrete URL.
  • One place that follows a navigation from static shell to App Shell to per-link prefetch to navigation, and names the lever at each step.

@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Tests Passed

Commit: 3263de1
View workflow run

@github-actions github-actions Bot added the Documentation Related to Next.js' official documentation. label Nov 12, 2026

@vercel vercel Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Vercel Agent reviewed ced846c and found no issues.

- `/tops/[product]`, `/shorts/[product]`: category header rendered, product shows fallback
- `/[category]/[product]`: both show fallback

Shells leave params open only from the end of the route. A URL can't leave a middle segment unknown, so there is no `/[category]/tee` shell. A visit with an unlisted category gets `/[category]/[product]`, whatever the product.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what does it mean to "leave params open"

A cached function's output is serialized into an **RSC payload**, at build time or at runtime. This payload is what everything else works from. Next.js renders it into HTML, keeps it in a server or remote store, or sends it to the browser, and [`cacheLife`](/docs/app/api-reference/functions/cacheLife) sets how long each copy stays fresh:

- **Prerendered HTML.** The payload is rendered to HTML and stored on disk when self-hosting, or in your platform's durable storage behind a CDN. That HTML is the [static shell](#prerendering) at build time and the concrete page after an [ISR](#incremental-static-regeneration) upgrade, with [`revalidate`](/docs/app/api-reference/functions/cacheLife#revalidate) and [`expire`](/docs/app/api-reference/functions/cacheLife#expire) controlling when it's rebuilt.
- **Prerendered output.** The payload is rendered to HTML, and both are stored on disk when self-hosting, or in your platform's durable storage behind a CDN. A direct visit gets the HTML, and a client navigation gets the RSC payload. Together they are the [static shell](#prerendering) at build time and the concrete page after an [ISR](#incremental-static-regeneration) upgrade, with [`revalidate`](/docs/app/api-reference/functions/cacheLife#revalidate) and [`expire`](/docs/app/api-reference/functions/cacheLife#expire) controlling when it's rebuilt.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

both what? payload and html?

@aurorascharff
aurorascharff marked this pull request as ready for review November 12, 2026 21:38
@vercel

vercel Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Vercel Agent finished reviewing your changes and approved this pull request.
3263de1 β€’ Thu, 12 Nov 2026 23:02:47 GMT

Review steps (36.9s)

Preparing the workspace 13.0s
Analyzing the changes 21.1s
Finalizing the review 2.7s

@icyJoseph
icyJoseph merged commit df7104d into canary Nov 14, 2026
58 checks passed
@icyJoseph
icyJoseph deleted the claude/isr-shell-param-order branch November 14, 2026 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Related to Next.js' official documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants