Skip to content

feat: narrow EdDSA MPC R/G share API and keep full state on AWM - #273

Merged
danielpeng1 merged 1 commit into
masterfrom
WCN-1930/eddsa-mpc-r-share
Sep 25, 2026
Merged

danielpeng1 merged 1 commit into
masterfrom
WCN-1930/eddsa-mpc-r-share

Conversation

@danielpeng1

Copy link
Copy Markdown
Contributor

Tighten EdDSA MPC signing so the R round response and G round request match the updated AWM contract.

  • AWM: R round returns EddsaUserToBitgoRShare only; G round decrypts encryptedUserToBitgoRShare + encryptedDataKey server-side (drop userToBitgoRShare from the API)
  • Master-express: client types + G call forward commitment-round encrypted fields; R handler builds a minimal SDK SignShare (only rShares[BITGO] is used by the SDK)
  • Breaking AWM ↔ master-express contract change; deploy both together

Tests:

  • signMpcTransaction.test.ts: R response keys, G uses encrypted state, G fails without it
  • Master eddsa / sendMany / signAndSendTxRequest mocks updated for new R/G bodies

Ticket: WCN-1930

@danielpeng1 danielpeng1 self-assigned this Sep 25, 2026
@linear-code

linear-code Bot commented Sep 25, 2026

Copy link
Copy Markdown

WCN-1930

@danielpeng1
danielpeng1 marked this pull request as ready for review September 25, 2026 16:06
@danielpeng1
danielpeng1 requested review from a team as code owners September 25, 2026 16:06
@danielpeng1
danielpeng1 merged commit b6b35bf into master Sep 25, 2026
23 checks passed
@danielpeng1
danielpeng1 deleted the WCN-1930/eddsa-mpc-r-share branch September 25, 2026 17:22
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