fix: avoid inlineDynamicImports ignored with codeSplitting warning when using Vite 8
#15647
Closing issue
Describe the bug
It changed since #15403 #15550, but it's still there:
✓ built in 303ms
vite v8.0.3 building client environment for production...
WARN inlineDynamicImports option is ignored because codeSplitting: true is set.
✓ 3 modules transformed.
Reproduction
With Vite 8, and a service worker probably.
Logs
System Info
Binaries:
Node: 20.19.2 - /usr/bin/node
npm: 9.2.0 - /usr/bin/npm
npmPackages:
@sveltejs/adapter-static: ^3.0.10 => 3.0.10
@sveltejs/kit: ^2.56.1 => 2.56.1
svelte: ^5.55.1 => 5.55.1
vite: ^8.0.3 => 8.0.3
Severity
annoyance
Additional Information
Pull request
Merged
K
kiwixz Apr 4, 2026, 10:46 PMCloses #15646.
Fixed by not defining inlineDynamicImports when building service worker with Vite 8.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
- It's 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 should clearly illustrate what problems it solves.
- Ideally, include a test that fails without this PR but passes with it.
Tests
- Run the tests with
pnpm testand lint the project withpnpm lintandpnpm check
Changesets
- If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running
pnpm changesetand following the prompts. Changesets that add features should beminorand those that fix bugs should bepatch. Please prefix changeset messages withfeat:,fix:, orchore:.
Edits
- Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.
Info
Merged at Apr 6, 2026, 11:03 AM
Merged by teemingc
Assignees None
Reviewers teemingc
Labels None
Milestone None
C
changeset-bot[bot] Apr 4, 2026, 10:46 PM🦋 Changeset detected
Latest commit: 77549c8
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
Pull request opened Apr 4, 2026, 10:46 PM
fix: avoid inlineDynamicImports ignored with codeSplitting warning when using Vite 8
•
• Apr 4, 2026, 10:27 PM K
kiwixzPull request merged Apr 6, 2026, 11:03 AM
Total -1 +6
Pro tip: You can prefix GitHub URLs of issues, PRs or discussions with svcl.dev/ to view them on this page! Also try it on a GitHub release ;)