Skip to content

Conversation

@arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Nov 28, 2024

PR-Codex overview

This PR primarily focuses on refactoring and improving transaction handling in the codebase. It introduces new functions for retrieving transaction receipts, updates route registrations, and modifies existing transaction logic to enhance clarity and maintainability.

Detailed summary

  • Removed manuallyResentAt from QueuedTransaction.
  • Changed test case from test.only to test.
  • Updated route registrations to use new function names (syncRetryTransactionRoute, retryFailedTransactionRoute).
  • Introduced getReceiptForEOATransaction and getReceiptForUserOp functions for better receipt handling.
  • Refactored retryFailedTransaction and syncRetryTransaction to utilize new receipt retrieval methods.
  • Improved error handling for retried transactions, checking if they are already mined.
  • Simplified transaction mining logic in mineTransactionWorker.
  • Enhanced logging for transaction states and receipts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@arcoraven arcoraven force-pushed the ph/allowRetryUserOp branch from a5da4b7 to ad39e9a Compare December 5, 2024 21:36
@arcoraven arcoraven merged commit af85f61 into main Dec 6, 2024
4 checks passed
@arcoraven arcoraven deleted the ph/allowRetryUserOp branch December 6, 2024 02:51
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.

3 participants