broadcast-channel error with SvelteKit

[finnstrand]

I get broadcast-channel error with 1.6 and SvelteKit .311

[plugin:vite:import-analysis] Failed to resolve import "broadcast-channel" from "node_modules/@sveltestack/svelte-query/svelte/queryCore/broadcastQueryClient-experimental/index.js". Does the file exist?
[/Users/finn/Documents/Projects/wip/node_modules/@sveltestack/svelte-query/svelte/queryCore/broadcastQueryClient-experimental/index.js:3:47]()
1  |  import '../core';
2  |  export async function broadcastQueryClient({ queryClient, broadcastChannel = 'svelte-query', }) {
3  |      const { BroadcastChannel } = await import('broadcast-channel');
   |                                                ^
4  |      let transaction = false;
5  |      const tx = (cb) => {

Hello,

A new version (v1.5.0) of this library has been published today.
This version makes broadcast-channel an optional dependency.
This should fix all problems with svelte-kit.

If you need to use the experimental broadcastQueryClient, you now need to install it manually (see: broadcastQueryClient).

If you still encounter issues with v1.5.0. Please open a new issue.

Originally posted by @SomaticIT in #39 (comment)

[SomaticIT]

Hello,

This is a duplicate of #63.
The issue comes from vite: vitejs/vite#6007.

I'm closing this issue to avoid duplication.

Thanks for your report.

[SomaticIT]

FYI, you just have to install broadcast-channel as a dev dependency to allow vite to parse the package in development.
It will not be included in production.

[SomaticIT]

FYI, with the latest version of vite (v3.1.0-beta.1) that is included in the latest version of sveltekit, this issue should be 100% resolved.

Indeed, vite is now able to detect optional dependencies and should not try to analyze them when it's not needed.