Skip to content

fix: add EdDSA MPCv2 chain code support for address derivation#8623

Open
Marzooqa wants to merge 1 commit intomasterfrom
WCI-230
Open

fix: add EdDSA MPCv2 chain code support for address derivation#8623
Marzooqa wants to merge 1 commit intomasterfrom
WCI-230

Conversation

@Marzooqa
Copy link
Copy Markdown
Contributor

  • Bump @bitgo/wasm-mps to 1.7.0 which exposes chaincode on the Share struct
  • Add shareChaincode field and getCommonKeychain() to DKG class in sdk-lib-mpc, returning pubkey+chaincode (128 hex chars) matching Eddsa.deriveUnhardened expectations
  • Use getCommonKeychain() in EddsaMPCv2Utils.createKeychains and pass full keychain to user/backup/bitgo keychains instead of just the 32-byte public key
  • Rename commonPublicKey to commonPublicKeychain in round 2 response handling to align with updated @bitgo/public-types and HSM response field name
  • Remove 64-char MPCv2 special case from getPublicKeyFromCommonKeychain; all EdDSA keychains are now uniformly 128 hex chars (pubkey + chaincode)
  • Update tests accordingly

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

TICKET: WCI-230

- Bump @bitgo/wasm-mps to 1.7.0 which exposes chaincode on the Share struct
- Add shareChaincode field and getCommonKeychain() to DKG class in sdk-lib-mpc,
  returning pubkey+chaincode (128 hex chars) matching Eddsa.deriveUnhardened expectations
- Use getCommonKeychain() in EddsaMPCv2Utils.createKeychains and pass full keychain
  to user/backup/bitgo keychains instead of just the 32-byte public key
- Rename commonPublicKey to commonPublicKeychain in round 2 response handling
  to align with updated @bitgo/public-types and HSM response field name
- Remove 64-char MPCv2 special case from getPublicKeyFromCommonKeychain; all
  EdDSA keychains are now uniformly 128 hex chars (pubkey + chaincode)
- Update tests accordingly

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

TICKET: WCI-230
@linear
Copy link
Copy Markdown

linear Bot commented Apr 23, 2026

@Marzooqa Marzooqa marked this pull request as ready for review April 24, 2026 10:14
@Marzooqa Marzooqa requested review from a team as code owners April 24, 2026 10:14
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.

1 participant