fix(core): throw when importing twice

this should make it easier to detect build misconfigurations

Author: wmertens

packages/qwik-router/src/ssg/worker-thread.ts

@@ -14,6 +14,9 @@ import type {
 } from './types';
 
 export async function workerThread(sys: System) {
+  // Special case: we allow importing qwik again in the same process, it's ok because we just needed the serializer
+  // TODO: remove this once we have vite environment API and no longer need the serializer separately
+  delete (globalThis as any).__qwik;
   const ssgOpts = sys.getOptions();
   const pendingPromises = new Set<Promise<any>>();
 
@@ -42,6 +45,9 @@ export async function workerThread(sys: System) {
 }
 
 export async function createSingleThreadWorker(sys: System) {
+  // Special case: we allow importing qwik again in the same process, it's ok because we just needed the serializer
+  // TODO: remove this once we have vite environment API and no longer need the serializer separately
+  delete (globalThis as any).__qwik;
   const ssgOpts = sys.getOptions();
   const pendingPromises = new Set<Promise<any>>();
 

packages/qwik/src/core/index.ts

@@ -1,3 +1,14 @@
+//////////////////////////////////////////////////////////////////////////////////////////
+// Protect against duplicate imports
+//////////////////////////////////////////////////////////////////////////////////////////
+import { version } from './version';
+if ((globalThis as any).__qwik) {
+  throw new Error(
+    `Qwik version ${(globalThis as any).__qwik} already imported while importing ${version}. Verify external vs bundled imports etc.`
+  );
+}
+(globalThis as any).__qwik = version;
+
 //////////////////////////////////////////////////////////////////////////////////////////
 // Developer Core API
 //////////////////////////////////////////////////////////////////////////////////////////