[PM-8836] Move ownership of biometrics to key-management

[quexten]

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-8836

📔 Objective

Move ownership of biometrics on all platforms to key-management. This aims to not change any functionality. However we do break out a few parts (especially the biometrics handling in desktop-credential-storage-listener.ts). Actual refactor of these will come later.

📸 Screenshots

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[github-actions]

Checkmarx One – Scan Summary & Details – 4f622938-2505-4cc7-b6e0-1b5658e56cc4

No New Or Fixed Issues Found

[jlf0dev]

🚀

[codecov]

Codecov Report

Attention: Patch coverage is 11.11111% with 72 lines in your changes missing coverage. Please review.

Project coverage is 33.23%. Comparing base (f1b343e) to head (59c8d5e).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...ment/biometrics/biometric.renderer-ipc.listener.ts	0.00%	30 Missing ⚠️
apps/desktop/src/key-management/preload.ts	0.00%	11 Missing ⚠️
...nagement/biometrics/electron-biometrics.service.ts	0.00%	7 Missing ⚠️
apps/desktop/src/main.ts	0.00%	5 Missing ⚠️
apps/browser/src/background/main.background.ts	0.00%	2 Missing ⚠️
...anagement/biometrics/browser-biometrics.service.ts	0.00%	2 Missing ⚠️
apps/browser/src/popup/services/services.module.ts	0.00%	2 Missing ⚠️
apps/desktop/src/app/services/services.module.ts	0.00%	2 Missing ⚠️
apps/browser/src/auth/popup/lock.component.ts	0.00%	1 Missing ⚠️
...th/popup/settings/account-security-v1.component.ts	0.00%	1 Missing ⚠️
... and 9 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10818      +/-   ##
==========================================
- Coverage   33.27%   33.23%   -0.05%     
==========================================
  Files        2731     2730       -1     
  Lines       85326    85258      -68     
  Branches    16229    16221       -8     
==========================================
- Hits        28390    28332      -58     
+ Misses      54678    54670       -8     
+ Partials     2258     2256       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mzieniukbw]

Codeowners file contains typos.

[mzieniukbw]

LGTM!

[JaredSnider-Bitwarden]

LGTM at a quick pass!