fix: do not fall through on execution errors

wevm · Jan 29, 2025 · 3f8859f · 3f8859f
1 parent fe558fd
commit 3f8859f
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 4 deletions.
diff --git a/.changeset/four-beans-tan.md b/.changeset/four-beans-tan.md
@@ -0,0 +1,5 @@
+---
+"viem": patch
+---
+
+Fixed issue where Transports were falling back on execution errors.
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/src/clients/transports/fallback.ts b/src/clients/transports/fallback.ts
@@ -1,3 +1,4 @@
+import { ExecutionRevertedError } from '../../errors/node.js'
 import {
   TransactionRejectedRpcError,
   UserRejectedRequestError,
@@ -195,6 +196,7 @@ function shouldThrow(error: Error) {
     if (
       error.code === TransactionRejectedRpcError.code ||
       error.code === UserRejectedRequestError.code ||
+      ExecutionRevertedError.nodeMessage.test(error.message) ||
       error.code === 5000 // CAIP UserRejectedRequestError
     )
       return true