Skip to content

feat: persist required assets in transaction meta#7820

Merged
matthewwalsh0 merged 5 commits intomainfrom
feat/transaction-required-assets-v2
Feb 4, 2026
Merged

feat: persist required assets in transaction meta#7820
matthewwalsh0 merged 5 commits intomainfrom
feat/transaction-required-assets-v2

Conversation

@matthewwalsh0
Copy link
Member

@matthewwalsh0 matthewwalsh0 commented Feb 3, 2026

Explanation

Adds support for persisting requiredAssets on transaction metadata.

Use if available when generating required tokens.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Medium Risk
Adds a new persisted requiredAssets field to transaction/batch metadata and changes required-token computation to prefer this data when present, which can affect downstream transaction-pay behavior. Changes are optional/guarded but touch shared types and request plumbing used by multiple consumers.

Overview
Adds support for persisting requiredAssets on transactions and batches. addTransaction now accepts requiredAssets and stores it on TransactionMeta, and the batch (EIP-7702) flow forwards requiredAssets from the batch request into the created transaction.

Updates transaction-pay required-token generation to prefer explicit assets. parseRequiredTokens now builds required tokens from transactionMeta.requiredAssets when provided, falling back to parsing token transfers/gas fees only when absent/empty; tests and changelogs were updated accordingly.

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

@matthewwalsh0 matthewwalsh0 force-pushed the feat/transaction-required-assets-v2 branch from df20b41 to 439c6cf Compare February 3, 2026 20:51
@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review February 3, 2026 21:00
@matthewwalsh0 matthewwalsh0 requested review from a team as code owners February 3, 2026 21:00
@matthewwalsh0 matthewwalsh0 force-pushed the feat/transaction-required-assets-v2 branch from e254b7c to 72a8d15 Compare February 4, 2026 11:55
@matthewwalsh0 matthewwalsh0 force-pushed the feat/transaction-required-assets-v2 branch from 72a8d15 to be34464 Compare February 4, 2026 15:46
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

@matthewwalsh0 matthewwalsh0 added this pull request to the merge queue Feb 4, 2026
Merged via the queue into main with commit 33f666e Feb 4, 2026
302 checks passed
@matthewwalsh0 matthewwalsh0 deleted the feat/transaction-required-assets-v2 branch February 4, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants