Skip to content

fix: Mark multiple transactions as failed if a batch smart transaction fails#551

Merged
dan437 merged 3 commits intomainfrom
fail-batch-transactions
Dec 11, 2025
Merged

fix: Mark multiple transactions as failed if a batch smart transaction fails#551
dan437 merged 3 commits intomainfrom
fail-batch-transactions

Conversation

@dan437
Copy link
Copy Markdown
Collaborator

@dan437 dan437 commented Dec 11, 2025

Description

fix: Mark multiple transactions as failed if a batch smart transaction fails.

Steps to reproduce

Quickest way is to upgrade or downgrade an EIP-7702 account and use any gas fee token, with STX enabled (this error is being fixed separately by @matthewwalsh0).

Before this fix only one transaction from a batch would be marked as failed, with this all transactions in a batch are marked as failed. It only started happening after we started returning a txHash asap. Before that all transactions were being marked as failed in the TransactionController.


Note

Marks all related regular transactions as failed when a batch smart transaction fails by matching txHashes, updates controller usage, and adds tests; also records txHashes on submit (keeping legacy txHash).

  • Smart transactions logic:
    • Rename shouldMarkRegularTransactionAsFailedshouldMarkRegularTransactionsAsFailed and markRegularTransactionAsFailedmarkRegularTransactionsAsFailed.
    • Update failure handling to mark multiple regular TransactionMeta as failed by matching tx.hash in smartTransaction.txHashes or id equals transactionId; skip already-failed.
  • Controller:
    • Use new pluralized helpers to decide and apply failure updates.
    • On submit, compute and store txHashes (and keep txHash for backward compatibility).
  • Tests:
    • Update test names and expectations; add case asserting multiple transactions are marked failed when txHashes match.

Written by Cursor Bugbot for commit 2b8334b. This will update automatically on new commits. Configure here.

matthewwalsh0 and others added 3 commits December 11, 2025 09:54
…n fails

Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
@dan437 dan437 requested a review from a team as a code owner December 11, 2025 13:11
@dan437 dan437 merged commit 3e9e44c into main Dec 11, 2025
18 checks passed
@dan437 dan437 deleted the fail-batch-transactions branch December 11, 2025 13:27
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