Skip to content

feat(sdk-core): add getEddsaMPCv2RecoveryKeyShares helper#8918

Open
vibhavgo wants to merge 1 commit into
masterfrom
WCI-396/sdk-core
Open

feat(sdk-core): add getEddsaMPCv2RecoveryKeyShares helper#8918
vibhavgo wants to merge 1 commit into
masterfrom
WCI-396/sdk-core

Conversation

@vibhavgo
Copy link
Copy Markdown
Contributor

@vibhavgo vibhavgo commented Jun 2, 2026

Decrypt both keycards in parallel, validate pub and rootChainCode separately with distinct errors, and surface a clear message when keycard material is missing or a v2 Argon2id envelope is detected.

  • Adds getEddsaMPCv2RecoveryKeyShares helper to sdk-core
  • Adds 7 unit tests covering happy path, field validation, and pub/rootChainCode mismatch cases
  • Fix cbor-x missing-package CI error: export MPSUtil.cborEncode from sdk-lib-mpc (which already declares cbor-x as a dependency) and use that in the sdk-core test instead of require('cbor-x') directly

Ticket: WCI-396

@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 2, 2026

WCI-396

@vibhavgo vibhavgo force-pushed the WCI-396/sdk-core branch 2 times, most recently from 2451c14 to e1e5266 Compare June 3, 2026 18:57
Ticket: WCI-396

Decrypt both keycards in parallel, validate pub and rootChainCode
separately with distinct errors, and surface a clear message when
keycard material is missing or a v2 Argon2id envelope is detected.
Adds 7 unit tests covering happy path, field validation, and
pub/rootChainCode mismatch cases.

Fix cbor-x missing-package CI error by exporting MPSUtil.cborEncode
from sdk-lib-mpc (which already declares cbor-x) and using that in
the sdk-core test instead of require('cbor-x') directly.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vibhavgo vibhavgo force-pushed the WCI-396/sdk-core branch from e1e5266 to db49315 Compare June 3, 2026 19:10
@vibhavgo vibhavgo marked this pull request as ready for review June 3, 2026 19:35
@vibhavgo vibhavgo requested review from a team as code owners June 3, 2026 19: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.

1 participant