Skip to content

chore: only generate each route resolution module once when prerendering - #16578

Merged
Rich-Harris merged 1 commit into
sveltejs:elliott/preload-code-route-idfrom
Nic-Polumeyv:fix-preload-route-module-once
Jul 30, 2026
Merged

chore: only generate each route resolution module once when prerendering#16578
Rich-Harris merged 1 commit into
sveltejs:elliott/preload-code-route-idfrom
Nic-Polumeyv:fix-preload-route-module-once

Conversation

@Nic-Polumeyv

Copy link
Copy Markdown
Contributor

Stacked on #16576, per #16576 (comment).

dependencies is per-visit(), so the route-ID module was generated once per prerendered page. On the prerendering/basics fixture with resolution: 'server', 31 generations produced 28 files; with this, 28 produce 28, and the output is byte-identical apart from the build version stamp.

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 9eaa3f1:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/9eaa3f1e5003c8a76a99f1df86841e828c447afa

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

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 Jul 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9eaa3f1

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

This PR includes changesets to release 1 package
Name Type
@sveltejs/kit 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

@Nic-Polumeyv
Nic-Polumeyv marked this pull request as draft July 30, 2026 01:35
@Nic-Polumeyv
Nic-Polumeyv marked this pull request as ready for review July 30, 2026 02:41
@Rich-Harris
Rich-Harris merged commit aff95bc into sveltejs:elliott/preload-code-route-id Jul 30, 2026
16 of 17 checks passed
@Nic-Polumeyv
Nic-Polumeyv deleted the fix-preload-route-module-once branch July 30, 2026 22:53
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.

2 participants