fix: remove unnecessary waiting for navigation

Author: OrKoN

src/McpContext.ts

@@ -364,7 +364,6 @@ export class McpContext implements Context {
     const networkMultiplier = getNetworkMultiplierFromString(
       this.getNetworkConditions(),
     );
-
     const waitForHelper = this.getWaitForHelper(
       page,
       cpuMultiplier,

src/WaitForHelper.ts

@@ -126,18 +126,23 @@ export class WaitForHelper {
   async waitForEventsAfterAction(
     action: () => Promise<unknown>,
   ): Promise<void> {
-    const navigationStartedPromise = this.waitForNavigationStarted();
-
+    const navigationStartedPromise = this.waitForNavigationStarted().then(
+      navigationStated => {
+        if (navigationStated) {
+          return this.#page
+            .waitForNavigation({
+              timeout: this.#navigationTimeout,
+              signal: this.#abortController.signal,
+            })
+            .catch(error => logger(error));
+        }
+        return;
+      },
+    );
     await action();
 
     try {
-      const navigationStated = await navigationStartedPromise;
-      if (navigationStated) {
-        await this.#page.waitForNavigation({
-          timeout: this.#navigationTimeout,
-          signal: this.#abortController.signal,
-        });
-      }
+      await navigationStartedPromise;
 
       // Wait for stable dom after navigation so we execute in
       // the correct context