Skip to content

fix(sdk-coin-eth): fix the verify confidential transfer for zama#9130

Merged
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-696
Jun 29, 2026
Merged

fix(sdk-coin-eth): fix the verify confidential transfer for zama#9130
MohammedRyaan786 merged 1 commit into
masterfrom
CHALO-696

Conversation

@MohammedRyaan786

@MohammedRyaan786 MohammedRyaan786 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

TICKET: CHALO-696

This pull request adds support for verifying direct confidential token transfers for hot/TSS EOA wallets, in addition to the existing multisig/smart-contract wallet path. The main changes include updating the confidential transfer verification logic to handle both transaction shapes, adding a new decoder for direct confidentialTransfer calldata, and extending the test suite to cover the new path.

Confidential transfer verification improvements:

  • Updated Erc7984Token.verifyTransaction to support both sendMultiSig-wrapped (multisig/smart-contract wallet) and direct (hot/TSS EOA wallet) confidential transfer transactions. The code now detects the transaction shape by method ID and extracts the token contract address and transfer parameters accordingly. [1] [2] [3]
  • Added a new decoder function decodeDirectConfidentialTransferCalldata in utils.ts to parse direct confidentialTransfer calldata and extract the recipient, encrypted handle, and input proof.
  • Updated imports in erc7984Token.ts to include the new decoder and relevant method IDs.

Testing enhancements:

  • Added a comprehensive test suite for direct confidentialTransfer verification, including positive and negative cases, to ensure the new path is correctly handled.

@linear-code

linear-code Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

CHALO-696

@MohammedRyaan786 MohammedRyaan786 marked this pull request as ready for review June 29, 2026 10:32
@MohammedRyaan786 MohammedRyaan786 requested a review from a team as a code owner June 29, 2026 10:32
@MohammedRyaan786 MohammedRyaan786 merged commit 0a81a68 into master Jun 29, 2026
24 checks passed
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