Skip to content

feat(sdk-coin-tempo): implement transaction deserialization, parsing, and verification#8237

Merged
nayandas190 merged 1 commit intomasterfrom
cecho-286
Mar 4, 2026
Merged

feat(sdk-coin-tempo): implement transaction deserialization, parsing, and verification#8237
nayandas190 merged 1 commit intomasterfrom
cecho-286

Conversation

@nayandas190
Copy link
Contributor

@nayandas190 nayandas190 commented Mar 3, 2026

Ticket: CECHO-286

  • Implemented fromImplementation / fromTip20Transaction to deserialize type 0x76 Tempo transactions from RLP hex, restoring full builder state (nonce, gas, fees, calls, fee token, signature)
  • Overrode validateRawTransaction to handle 0x76 prefix before falling back to parent
  • Added parseTransaction and verifyTransaction overrides in tempo.ts
  • Fixed _outputs/_inputs population on Tip20Transaction and id getter (was returning 'pending')
  • Override isWalletAddress to bypass forwarder contract and MPC verification: validates that the base EVM portion of a memoId address matches the wallet's baseAddress (or rootAddress as fallback)

@nayandas190 nayandas190 force-pushed the cecho-286 branch 3 times, most recently from 618cdb9 to a336da3 Compare March 4, 2026 12:21
@nayandas190 nayandas190 changed the title feat(sdk-coin-tempo): implement tx deserialization, structured outputs, and field accessors feat(sdk-coin-tempo): implement transaction deserialization, parsing, and verification Mar 4, 2026
@nayandas190 nayandas190 marked this pull request as ready for review March 4, 2026 12:59
@nayandas190 nayandas190 requested review from a team as code owners March 4, 2026 12:59
Copy link
Contributor

@nvrakesh06 nvrakesh06 left a comment

Choose a reason for hiding this comment

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

Please take a look at the above comments

@nayandas190 nayandas190 merged commit 755490a into master Mar 4, 2026
21 checks passed
@nayandas190 nayandas190 deleted the cecho-286 branch March 4, 2026 14:35
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.

2 participants