Skip to content

Commit dd5381a

Browse files
authored
Squelch rollup warnings (#13687)
* fix: prevent Rollup warnings for undefined hooks * better approach * revert original change
1 parent 99b8079 commit dd5381a

File tree

2 files changed

+18
-1
lines changed

2 files changed

+18
-1
lines changed

.changeset/thirty-foxes-study.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'@sveltejs/kit': patch
3+
---
4+
5+
fix: prevent Rollup warnings for undefined hooks

packages/kit/src/exports/vite/index.js

+13-1
Original file line numberDiff line numberDiff line change
@@ -678,7 +678,19 @@ Tips:
678678
manualChunks: split ? undefined : () => 'bundle',
679679
inlineDynamicImports: false
680680
},
681-
preserveEntrySignatures: 'strict'
681+
preserveEntrySignatures: 'strict',
682+
onwarn(warning, handler) {
683+
if (
684+
warning.code === 'MISSING_EXPORT' &&
685+
warning.id === `${kit.outDir}/generated/client-optimized/app.js`
686+
) {
687+
// ignore e.g. undefined `handleError` hook when
688+
// referencing `client_hooks.handleError`
689+
return;
690+
}
691+
692+
handler(warning);
693+
}
682694
},
683695
ssrEmitAssets: true,
684696
target: ssr ? 'node18.13' : undefined

0 commit comments

Comments
 (0)