hmm

apps/repl/app/utils/editor-text.ts

@@ -161,10 +161,11 @@ export class FileURIComponent {
   #frame?: number;
   #qps: URLSearchParams | undefined;
   #updateQPs = async (rawText: string, format: Format) => {
-    await waitForPromise(this.#_updateQPs(rawText, format));
+    waitForPromise(this.#_updateQPs(rawText, format));
   };
 
   #_updateQPs = async (rawText: string, format: Format) => {
+    console.debug('queueing QPs');
     if (this.#frame) cancelAnimationFrame(this.#frame);
 
     let encoded = compressToEncodedURIComponent(rawText);
@@ -182,35 +183,42 @@ export class FileURIComponent {
       ...qps,
     };
 
-    this.#frame = requestAnimationFrame(async () => {
-      if (isDestroyed(this) || isDestroying(this)) {
-        return;
-      }
+    return new Promise<void>((resolve) => {
+      this.#frame = requestAnimationFrame(async () => {
+        if (isDestroyed(this) || isDestroying(this)) {
+          resolve();
 
-      /**
-       * Debounce so we are kinder on the CPU
-       */
-      await new Promise((resolve) => setTimeout(resolve, DEBOUNCE_MS));
+          return;
+        }
 
-      if (isDestroyed(this) || isDestroying(this)) {
-        return;
-      }
+        /**
+         * Debounce so we are kinder on the CPU
+         */
+        await new Promise((resolve) => setTimeout(resolve, DEBOUNCE_MS));
 
-      // On initial load, if we call #updateQPs,
-      // we may not have a currentURL, because the first transition has yet to complete
-      let base = this.router.currentURL?.split('?')[0];
+        if (isDestroyed(this) || isDestroying(this)) {
+          resolve();
 
-      if (macroCondition(isTesting())) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        base ??= (this.router as any) /* private API? */?.location?.path;
-      } else {
-        base ??= window.location.pathname;
-      }
+          return;
+        }
+
+        // On initial load, if we call #updateQPs,
+        // we may not have a currentURL, because the first transition has yet to complete
+        let base = this.router.currentURL?.split('?')[0];
+
+        if (macroCondition(isTesting())) {
+          // eslint-disable-next-line @typescript-eslint/no-explicit-any
+          base ??= (this.router as any) /* private API? */?.location?.path;
+        } else {
+          base ??= window.location.pathname;
+        }
 
-      let next = `${base}?${qps}`;
+        let next = `${base}?${qps}`;
 
-      this.router.replaceWith(next);
-      this.#text = rawText;
+        this.router.replaceWith(next);
+        this.#text = rawText;
+        resolve();
+      });
     });
   };
 }

apps/repl/tests/application/-page/index.ts

@@ -52,7 +52,7 @@ export class Page extends PageObject {
 
     let url = currentURL();
 
-    assert(`Expected an URL, got ${url}`, url);
+    assert(`Expected an URL -- via currentURL(), got ${url}`, url);
 
     let [, search] = url.split('?');
     let query = new URLSearchParams(search);

apps/repl/tests/test-helper.ts

@@ -22,6 +22,9 @@ Object.assign(window, { getSettledState, getPendingWaiterState, currentURL, curr
 
 setup(QUnit.assert);
 
+QUnit.testStart(() => {
+  localStorage.clear();
+});
 QUnit.testDone(resetOnerror);
 
 start();