fix(vite.config): Duplicate Export

[PatrickJS]

this is caused by having duplicate deps in both devDeps and deps

fixes #6226

[cloudflare-workers-and-pages]

Deploying qwik-docs with    Cloudflare Pages

Latest commit:	568102b
Status:	✅  Deploy successful!
Preview URL:	https://0eae1b61.qwik-8nx.pages.dev
Branch Preview URL:	https://fix-6226.qwik-8nx.pages.dev

View logs

[shairez]

LGTM
Thanks @PatrickJS ! 🙏

[brandonpittman]

@PatrickJS

After upgrading to 1.5.2, my build script spits out messages like this:

[plugin:vite:esbuild] [plugin vite:esbuild] ../shared/components/router-head/router-head.tsx: Duplicate key "dangerouslySetInnerHTML" in object literal
58 |                          return s.script;
59 |                      },
60 |                      dangerouslySetInnerHTML: _wrapSignal(s, "script")
   |                      ^
61 |                  }, null, 0, s.key))
62 |          ]

Any chance it's related to this fix?

[wmertens]

@brandonpittman unfortunately that's a cosmetic side effect of a bugfix of the optimizer, and it's fixed in v2. We're focusing our efforts on v2 right now, it's hard to fix this in v1.

[brandonpittman]

@brandonpittman unfortunately that's a cosmetic side effect of a bugfix of the optimizer, and it's fixed in v2. We're focusing our efforts on v2 right now, it's hard to fix this in v1.

@wmertens
Is there something I can do to workaround this? What’s causing the warnings to happen?

[PatrickJS]

update RouterHead

export const RouterHead = component$(() => {
  const head = useDocumentHead();
  const loc = useLocation();

  return (
    <>
      <title>{head.title}</title>

      <link rel="canonical" href={loc.url.href} />
      <meta name="viewport" content="width=device-width, initial-scale=1.0" />
      <link rel="icon" type="image/svg+xml" href="/favicon.svg" />

      {head.meta.map((m) => (
        <meta key={m.key} {...m} />
      ))}

      {head.links.map((l) => (
        <link key={l.key} {...l} />
      ))}

      {head.styles.map((s) => (
        <style
          key={s.key}
          {...s.props}
          {...(s.props?.dangerouslySetInnerHTML
            ? {}
            : { dangerouslySetInnerHTML: s.style })}
        />
      ))}

      {head.scripts.map((s) => (
        <script
          key={s.key}
          {...s.props}
          {...(s.props?.dangerouslySetInnerHTML
            ? {}
            : { dangerouslySetInnerHTML: s.script })}
        />
      ))}
    </>
  );
});

notice the bottom two are the ones thats different

{head.styles.map((s) => (
  <style
    key={s.key}
    {...s.props}
    {...(s.props?.dangerouslySetInnerHTML
      ? {}
      : { dangerouslySetInnerHTML: s.style })}
  />
))}

{head.scripts.map((s) => (
  <script
    key={s.key}
    {...s.props}
    {...(s.props?.dangerouslySetInnerHTML
      ? {}
      : { dangerouslySetInnerHTML: s.script })}
  />
))}