fix: clearTimeout in waitForUserOperationReceipt (#4058)

* fix: clearTimeout in waitForUserOperationReceipt

* chore: add changeset

* Update clever-buttons-cut.md

---------

Co-authored-by: jxom <7336481+jxom@users.noreply.github.com>

Author: tuphan-dn

.changeset/clever-buttons-cut.md

@@ -0,0 +1,5 @@
+---
+"viem": patch
+---
+
+Fixed a bug where the timeout in `waitForUserOperationReceipt` was not properly cleared after the receipt was returned.

src/account-abstraction/actions/bundler/waitForUserOperationReceipt.ts

@@ -92,14 +92,26 @@ export function waitForUserOperationReceipt(
         unobserve()
       }
 
+      const timeoutId = timeout ? setTimeout(
+          () =>
+            done(() =>
+              emit.reject(
+                new WaitForUserOperationReceiptTimeoutError({ hash }),
+              ),
+            ),
+          timeout,
+        ) : undefined
+
       const unpoll = poll(
         async () => {
-          if (retryCount && count >= retryCount)
+          if (retryCount && count >= retryCount) {
+            clearTimeout(timeoutId)
             done(() =>
               emit.reject(
                 new WaitForUserOperationReceiptTimeoutError({ hash }),
               ),
             )
+          }
 
           try {
             const receipt = await getAction(
@@ -112,6 +124,8 @@ export function waitForUserOperationReceipt(
             const error = err as GetUserOperationReceiptErrorType
             if (error.name !== 'UserOperationReceiptNotFoundError')
               done(() => emit.reject(error))
+          } finally {
+            clearTimeout(timeoutId)
           }
 
           count++
@@ -122,17 +136,6 @@ export function waitForUserOperationReceipt(
         },
       )
 
-      if (timeout)
-        setTimeout(
-          () =>
-            done(() =>
-              emit.reject(
-                new WaitForUserOperationReceiptTimeoutError({ hash }),
-              ),
-            ),
-          timeout,
-        )
-
       return unpoll
     })
   })