Skip to content

feat(sdk-core): add removePasskeyFromAccount function#8639

Draft
derranW26 wants to merge 2 commits intomasterfrom
passkey/ticket-6-remove-passkey-from-account
Draft

feat(sdk-core): add removePasskeyFromAccount function#8639
derranW26 wants to merge 2 commits intomasterfrom
passkey/ticket-6-remove-passkey-from-account

Conversation

@derranW26
Copy link
Copy Markdown
Contributor

Summary

  • Adds modules/sdk-core/src/bitgo/passkey/ module with removePasskeyFromAccount, WebAuthnOtpDevice type, and barrel index.ts
  • removePasskeyFromAccount issues DELETE /api/v2/user/otp/{device.id} using device.id (not credentialId)
  • Exports passkey module from modules/sdk-core/src/bitgo/index.ts
  • Unit tests verify correct device.id usage and no credentialId leakage

Notes

  • WebAuthnOtpDevice is a local stub — swap to @bitgo/public-types once that PR merges (TODO comment in types.ts)
  • Callers must invoke removePasskeyFromWallet() for all affected wallets before calling this function (JSDoc warning on the function)

Ticket

WCN-191

Pure cryptographic primitives for WebAuthn PRF-based key derivation:
- derivePassword: converts ArrayBuffer PRF result to hex walletPassphrase
- deriveEnterpriseSalt: HMAC-SHA256 via SJCL matching retail implementation exactly

TICKET: WCN-186
@derranW26 derranW26 force-pushed the passkey/ticket-6-remove-passkey-from-account branch from a2b2564 to c18a4d4 Compare April 28, 2026 14:39
@derranW26 derranW26 changed the base branch from passkey/ticket-1-passkey-crypto-package to master April 28, 2026 14:39
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