fix(scheduler): dont await for RUN_QRL chore

Author: Varixo

packages/qwik/src/core/shared/scheduler.ts

@@ -385,7 +385,17 @@ export const createScheduler = (
       case ChoreType.RUN_QRL:
         {
           const fn = (chore.$target$ as QRLInternal<(...args: unknown[]) => unknown>).getFn();
-          returnValue = fn(...(chore.$payload$ as unknown[]));
+          try {
+            const result = retryOnPromise(() => fn(...(chore.$payload$ as unknown[])));
+            if (isPromise(result)) {
+              result.catch((error) => {
+                emitEvent('qerror', { error });
+              });
+            }
+          } catch (error) {
+            emitEvent('qerror', { error });
+          }
+          returnValue = null;
         }
         break;
       case ChoreType.TASK:

packages/qwik/src/qwikloader.ts

@@ -33,7 +33,6 @@ export const qwikLoader = (
   const isConnected = 'isConnected';
   const qvisible = 'qvisible';
   const Q_JSON = '_qwikjson_';
-  const MAX_RETRY_ON_PROMISE_COUNT = 100;
   const nativeQuerySelectorAll = (root: ParentNode, selector: string) =>
     Array.from(root.querySelectorAll(selector));
   const querySelectorAll = (query: string) => {
@@ -165,27 +164,19 @@ export const qwikLoader = (
         }
         const previousCtx = doc[Q_CONTEXT];
         if (element[isConnected]) {
-          const handleEvent = async (retryCount = 0) => {
-            try {
-              doc[Q_CONTEXT] = [element, ev, url];
-              isSync || emitEvent<QwikSymbolEvent>('qsymbol', { ...eventData });
-              const results = handler(ev, element);
-              // only await if there is a promise returned
-              if (isPromise(results)) {
-                await results;
-              }
-            } catch (error) {
-              // retry if error is a promise. This means that probably a QRL is not resolved yet and we need to retry it.
-              if (isPromise(error) && retryCount < MAX_RETRY_ON_PROMISE_COUNT) {
-                error.then(() => handleEvent(retryCount++));
-              } else {
-                emitEvent('qerror', { error, ...eventData });
-              }
-            } finally {
-              doc[Q_CONTEXT] = previousCtx;
+          try {
+            doc[Q_CONTEXT] = [element, ev, url];
+            isSync || emitEvent<QwikSymbolEvent>('qsymbol', { ...eventData });
+            const results = handler(ev, element);
+            // only await if there is a promise returned
+            if (isPromise(results)) {
+              await results;
             }
-          };
-          handleEvent();
+          } catch (error) {
+            emitEvent('qerror', { error, ...eventData });
+          } finally {
+            doc[Q_CONTEXT] = previousCtx;
+          }
         }
       }
     }