Skip to content

feat(passkey-crypto): add derivePasskeyPrfKey function#8679

Merged
derranW26 merged 1 commit intomasterfrom
passkey/ticket-7-derive-passkey-prf-key
May 6, 2026
Merged

feat(passkey-crypto): add derivePasskeyPrfKey function#8679
derranW26 merged 1 commit intomasterfrom
passkey/ticket-7-derive-passkey-prf-key

Conversation

@derranW26
Copy link
Copy Markdown
Contributor

@derranW26 derranW26 commented May 4, 2026 •
edited
Loading

https://linear.app/bitgo/issue/WCN-192/implement-derivepasskeyprfkey

Summary

  • Adds derivePasskeyPrfKey() to modules/sdk-core/src/bitgo/passkey/
  • Fetches wallet user keychain, builds PRF eval map, triggers WebAuthn assertion via WebAuthnProvider, and returns hex-encoded wallet passphrase
  • Imports buildEvalByCredential, matchDeviceByCredentialId, derivePassword from @bitgo/passkey-crypto
  • Adds @bitgo/passkey-crypto as a dependency in sdk-core/package.json and tsconfig.json project reference

Test plan

  • 5 unit tests pass: happy path, no devices, empty devices, no valid PRF salt, unknown credential ID
  • Error messages match retail exactly
  • No any types in implementation

Ticket: WCN-192

@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch 2 times, most recently from 043577b to adc92b6 Compare May 4, 2026 15:16
@derranW26 derranW26 marked this pull request as ready for review May 4, 2026 15:51
@derranW26 derranW26 requested review from a team as code owners May 4, 2026 15:52
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch from adc92b6 to 2db8709 Compare May 4, 2026 16:48
@derranW26
Copy link
Copy Markdown
Contributor Author

derranW26 commented May 4, 2026

@claude review this PR

@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch from 2db8709 to 2d56dae Compare May 4, 2026 18:03
@derranW26 derranW26 requested a review from a team as a code owner May 4, 2026 18:03
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch 4 times, most recently from e247cb7 to d1e85f3 Compare May 5, 2026 15:29
mohammadalfaiyazbitgo
mohammadalfaiyazbitgo requested changes May 5, 2026
View reviewed changes
Comment thread modules/passkey-crypto/src/derivePasskeyPrfKey.ts
@derranW26 derranW26 changed the title feat(sdk-core): add derivePasskeyPrfKey function feat(passkey-crypto): add derivePasskeyPrfKey function May 5, 2026
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch from d1e85f3 to 2df086f Compare May 5, 2026 17:30
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch 4 times, most recently from e2c5ccd to 09ca9e0 Compare May 6, 2026 15:24
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch from 09ca9e0 to f36c076 Compare May 6, 2026 16:14
@derranW26
feat(sdk-core): add derivePasskeyPrfKey function
74f10a0 
- fetch keychain webauthn devices and build PRF eval map
- fetch server-issued assertion challenge via bitgo
- trigger WebAuthn assertion via provider (navigator layer)
- derive hex wallet passphrase from PRF output

Ticket: WCN-192
@derranW26 derranW26 force-pushed the passkey/ticket-7-derive-passkey-prf-key branch from f36c076 to 74f10a0 Compare May 6, 2026 17:00
@derranW26 derranW26 merged commit 8f1038a into master May 6, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mohammadalfaiyazbitgo mohammadalfaiyazbitgo mohammadalfaiyazbitgo approved these changes

@kisslove-dewangan kisslove-dewangan Awaiting requested review from kisslove-dewangan kisslove-dewangan is a code owner automatically assigned from BitGo/wallet-core-india

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@derranW26 @mohammadalfaiyazbitgo