[PM-27137] feat: Add credential deletion and debug info to Use Passkey screen - #2948
Conversation
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE Reviewed the final PR in the SDK-passkey TestHarness stack, which adds swipe-to-delete for registered credentials and surfaces cipher ID, user handle, and signature-counter usage on each credential row. The delete path traces cleanly from Code Review DetailsNo blocking or noteworthy code findings. PR Metadata Assessment
|
06b4697 to
df122c1
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## pm-27135-testharness-use-passkeys-ui #2948 +/- ##
========================================================================
- Coverage 79.47% 79.45% -0.02%
========================================================================
Files 1172 1172
Lines 75144 75144
========================================================================
- Hits 59724 59709 -15
- Misses 15420 15435 +15 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
df122c1 to
ef4d0a2
Compare
ef4d0a2 to
aea0e5e
Compare
aea0e5e to
dc0db85
Compare
dc0db85 to
2ea47d2
Compare
2ea47d2 to
33180e4
Compare
33180e4 to
f0b628f
Compare
…y screen Adds swipe-to-delete for registered credentials, threaded through DefaultFido2CredentialStore/PasskeyService/UsePasskeyProcessor, and surfaces cipher ID, user handle, and signature-counter usage on each credential row for easier debugging.
f0b628f to
dfce034
Compare
🎟️ Tracking
PM-27137
📔 Objective
Fourth and final PR in a 4-PR stack adding SDK-backed passkey test scenarios to TestHarness. Stacked on #2947.
SDKFido2CredentialStore/SDKPasskeyService/SDKUsePasskeyProcessor.📸 Screenshots
Stack: #2945 → #2946 → #2947 → #2948