[PM-39974] Remove Biometric Integration Setting Fully

[quexten]

🎟️ Tracking

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

📔 Objective

The introduction of shared unlock removed the browser integration setting, but not the state (incorrectly). Further, it conflicted with the addition of the new desktop settings dialog. This leads to a situation, where currently only the new desktop settings dialog component did not regress, and correctly allows enabling biometric unlock.

This PR fully removes the setting in UI and in state and enables the setting always.

📸 Screenshots

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the full removal of the browserIntegrationEnabled desktop setting across UI, DesktopSettingsService, persisted state, and the biometric message handler. Verified that browser integration is genuinely always-on now, since generateManifests() runs unconditionally at desktop startup (apps/desktop/src/main.ts:419), making the removed toggle and its dead code paths safe to delete. The new state migration (81) correctly follows the established key-definition pattern and is properly wired into the migration chain. No security, correctness, or breaking-change concerns found.

Code Review Details

No findings.

Verification notes:

• State migration 81-remove-browser-integration-enabled.ts matches the desktopSettings / browserIntegrationEnabled global key definition used in migration 66, is registered 80 → 81 with CURRENT_VERSION bumped, and has spec coverage for present/absent/rollback (IRREVERSIBLE).
• Removed dead paths (connected-apps clearing in biometric-message-handler.service.ts, DDG shared-key clearing in settings-dialog.component.ts) were both gated on the now-nonexistent "integration disabled" state — no behavioral or security regression.
• Only the en locale was edited; remaining non-en keys are Crowdin-managed per repo i18n rules. No dangling references to removed symbols or i18n keys remain in .ts/.html code.

[quexten]

Lint errors are unrelated.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.72%. Comparing base (0151e15) to head (a99bdf2).
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #21684   +/-   ##
=======================================
  Coverage   49.72%   49.72%           
=======================================
  Files        4114     4115    +1     
  Lines      129004   128987   -17     
  Branches    19750    19745    -5     
=======================================
- Hits        64148    64145    -3     
+ Misses      60096    60082   -14     
  Partials     4760     4760           

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.