fix: Make injectScript wait until script is actually loaded (#1763)

Co-authored-by: Aaron <[email protected]>

Author: ion1

packages/wxt/src/utils/inject-script.ts

@@ -32,13 +32,39 @@ export async function injectScript(
     script.src = url;
   }
 
+  const loadedPromise = makeLoadedPromise(script);
+
   await options?.modifyScript?.(script);
 
   (document.head ?? document.documentElement).append(script);
 
   if (!options?.keepInDom) {
     script.remove();
   }
+
+  await loadedPromise;
+}
+
+function makeLoadedPromise(script: HTMLScriptElement): Promise<void> {
+  return new Promise((resolve, reject) => {
+    const onload = () => {
+      resolve();
+      cleanup();
+    };
+
+    const onerror = () => {
+      reject(new Error(`Failed to load script: ${script.src}`));
+      cleanup();
+    };
+
+    const cleanup = () => {
+      script.removeEventListener('load', onload);
+      script.removeEventListener('error', onerror);
+    };
+
+    script.addEventListener('load', onload);
+    script.addEventListener('error', onerror);
+  });
 }
 
 export interface InjectScriptOptions {