Skip to content

fix: omit ISR data endpoints for server-only routes - #16731

Open
shixi-li wants to merge 1 commit into
sveltejs:version-3from
shixi-li:codex/skip-server-route-isr-data
Open

fix: omit ISR data endpoints for server-only routes#16731
shixi-li wants to merge 1 commit into
sveltejs:version-3from
shixi-li:codex/skip-server-route-isr-data

Conversation

@shixi-li

Copy link
Copy Markdown

closes #12690

Server-only ISR routes do not have a page data endpoint, but the Vercel adapter currently emits a data function, prerender configuration, and routing entry for them.

This change only creates those data-specific ISR artifacts when a route has page methods. The server route main function, prerender configuration, and routing entry remain unchanged.

The build regression adds ISR to the existing API fixture and verifies that its main artifacts are present while its data artifacts are absent. It also verifies that a regular ISR page still keeps its data function and route.


Please do not delete this checklist! Before submitting the PR, please make sure you do the following:

  • It is really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
  • This message body clearly illustrates what problems it solves.
  • Includes a test that fails without this PR but passes with it.

Tests

  • pnpm -C packages/adapter-vercel test
  • pnpm lint
  • pnpm check
  • pnpm -F @sveltejs/kit test:unit

Changesets

  • Added a patch changeset for @sveltejs/adapter-vercel.

Edits

  • Allow edits from maintainers is enabled.

@pkg-svelte-dev

pkg-svelte-dev Bot commented Aug 11, 2026

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 0e50024:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/0e50024c99994ce7855fb1b79c5b66530fe3bad9

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/16731

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Aug 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0e50024

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sveltejs/adapter-vercel Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@teemingc teemingc added the pkg:adapter-vercel Pertaining to the Vercel adapter label Aug 11, 2026
@shixi-li
shixi-li marked this pull request as ready for review August 11, 2026 14:12
@elliott-with-the-longest-name-on-github elliott-with-the-longest-name-on-github added the needs-platform-tests This PR needs to run platform tests in order to merge. label Aug 11, 2026
@shixi-li
shixi-li force-pushed the codex/skip-server-route-isr-data branch from 4cbac70 to 0e50024 Compare August 12, 2026 02:37
@shixi-li

Copy link
Copy Markdown
Author

Rebased this PR onto the latest version-3 (bc07967c7) and resolved the one package-script conflict by preserving upstreams removal of the prepare script while retaining this PRs post-build assertion. The functional diff remains scoped to the same five files; git diff --check passes.

I attempted both the adapters complete test command and its focused unit/build halves locally. In this workspace each hung without output until the command timeout/interrupt, so I did not claim a local pass for the rebased head. The previous heads GitHub CI was green; the newly pushed head is 0e50024c9 and will get a fresh CI signal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-platform-tests This PR needs to run platform tests in order to merge. pkg:adapter-vercel Pertaining to the Vercel adapter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

adapter-vercel creates ISR symlinks for __data.json files for server routes

3 participants