Skip to content

fix: retry transaction waits after RPC errors - #25274

Open
loglapa wants to merge 1 commit into
AztecProtocol:merge-train/spartanfrom
loglapa:vl/retry-transient-rpc-waits
Open

fix: retry transaction waits after RPC errors#25274
loglapa wants to merge 1 commit into
AztecProtocol:merge-train/spartanfrom
loglapa:vl/retry-transient-rpc-waits

Conversation

@loglapa

@loglapa loglapa commented Aug 20, 2026

Copy link
Copy Markdown

Fixes #23546

Summary

  • keep polling for transaction receipts after transient RPC failures
  • keep polling for L1-to-L2 message readiness after transient RPC failures
  • cover both wait helpers with regression tests

Testing

  • yarn workspace @aztec/aztec.js test utils/node.test.ts utils/cross_chain.test.ts
  • yarn build
  • yarn format aztec.js
  • yarn lint aztec.js

All commands were run in aztecprotocol/devbox:3.0.

@loglapa
loglapa marked this pull request as ready for review August 20, 2026 09:55
@loglapa
loglapa force-pushed the vl/retry-transient-rpc-waits branch from f6f80b9 to 4a22115 Compare August 21, 2026 10:53
@loglapa

loglapa commented Aug 21, 2026

Copy link
Copy Markdown
Author

Rebased onto the current merge-train/spartan tip. Validation in aztecprotocol/devbox:3.0: immutable install, full TypeScript build, focused node.test.ts and cross_chain.test.ts suites (23/23), format check, lint, and git diff --check all pass. The repository-wide bootstrap reaches the unchanged ipc-runtime prerequisite and stops because this devbox warns that Rust 1.89.0 is not installed; the yarn-project build and changed tests are green. The failed external check only reports the missing ci-external/ci-external-once label; a maintainer needs to add it because contributors cannot add repository labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant