Skip to content

fix(abstract-eth): preserve zero wallet version for CCR - #9742

Draft
ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
SPT-452-self-serve-ccr-signing
Draft

ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
SPT-452-self-serve-ccr-signing

Conversation

@ralph-bitgo

@ralph-bitgo ralph-bitgo Bot commented Sep 15, 2026

Copy link
Copy Markdown

What

  • preserve wallet version 0 when rebuilding EthLike cross-chain recovery transactions
  • cover hot, cold, and build CCR flows with a version-zero response

Why

Version-zero wallets are valid legacy CCR wallets. Treating 0 as absent drops the wallet version during self-serve recovery signing and can produce an invalid transaction, blocking customer fund recovery.

Verification

  • BITGOJS_TEST_PASSWORD=test yarn unit-test --grep 'cross chain recovery' in modules/sdk-coin-ethlike — 15 passing
  • yarn build for @bitgo/abstract-eth and @bitgo/sdk-coin-ethlike

Ticket: SPT-452

Pass wallet version zero from cross-chain recovery build responses into the transaction builder and cover legacy CCR signing responses in EthLike tests.\n\nWithout this, a version-zero wallet is treated as missing during recovery signing, so self-serve CCR signing can produce an invalid transaction for affected customer funds.\n\nTicket: SPT-452

Session-Id: 43a697ad-f056-41b1-855b-c82224af5734
Task-Id: 72246e14-040e-4132-9fef-4e56482d1b19
@ralph-bitgo
ralph-bitgo Bot force-pushed the SPT-452-self-serve-ccr-signing branch from 8b80f56 to 6d1674b Compare September 15, 2026 17:37
@linear-code

linear-code Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

SPT-452

@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

⚠️ Unit tests are failing on Node 26.x (Current release line, non-blocking). This is not an LTS version yet, so it does not block merge, but it signals an incompatibility to fix before Node 26.x becomes LTS.

View run

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.

0 participants