Skip to content

fix: validate compiled params during builds - #17443

Draft
svelte-triage-bot[bot] wants to merge 1 commit into
mainfrom
triage/kit-17442-compiled-params
Draft

svelte-triage-bot[bot] wants to merge 1 commit into
mainfrom
triage/kit-17442-compiled-params

Conversation

@svelte-triage-bot

Copy link
Copy Markdown
Contributor
  • Build validation imported raw params source through Node, bypassing Vite’s TypeScript resolution and transforms; require an explicit loader instead of retaining that fallback.
  • Preserve the matcher diagnostics introduced in #16189 and relocated in #17252, but validate the compiled module through the generated manifest during postbuild analysis. Reject creating another Vite server or rewriting extensions because the correct build artifact already exists.
  • Run validation after the initialization enforced by #17008. Use the analysis worker shared by normal/watch processing from #16632, avoiding cached source modules without changing watch discovery or dev loading.
  • Cover aliased and transitive relative .js imports targeting TypeScript, server rendering and matching, client navigation, rejected matches, and skipping unused matchers.

Fixes #17442

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from d7b2b00:

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

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

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 Oct 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d7b2b00

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

This branch has not been deployed

No deployments
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.

src/params.ts that imports a .ts module with a .js extension works in dev but fails vite build with ERR_MODULE_NOT_FOUND

0 participants