PM-40317: bug: All users must follow Master Password policies

[david-livefront]

🎟️ Tracking

PM-40317

📔 Objective

This PR updates the policy rules to apply MasterPassword policies to all users.

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR adds PolicyType.MASTER_PASSWORD to the non-exempt branch of isOrganizationExemptFromPolicies in PolicyManagerImpl, so that org owners and admins are no longer exempt from Master Password policies in the legacy filtering flow. The change is minimal, matches the pattern of the other non-exempt policy types, and aligns with the stated objective (PM-40317). The accompanying test edit is a harmless type-parameter correction (SyncResponseJson.Policy → PolicyView in an emptyList assertion).

Code Review Details

No blocking findings.

• PolicyManagerImpl.kt:161 — Behavior change only applies to the legacy flow (isPoliciesInAcceptedStateEnabled == false); the accepted-state flow delegates exemption to authSdkSource.filterPolicies. Confirmed this matches intent and existing conventions.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.36%. Comparing base (006bdbf) to head (fa68245).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7163      +/-   ##
==========================================
- Coverage   86.43%   86.36%   -0.07%     
==========================================
  Files         912      912              
  Lines       65520    65633     +113     
  Branches     9610     9610              
==========================================
+ Hits        56631    56684      +53     
- Misses       5444     5504      +60     
  Partials     3445     3445              

Flag	Coverage Δ
app-data	17.41% <ø> (-0.02%)	⬇️
app-ui-auth-tools	18.81% <ø> (+0.22%)	⬆️
app-ui-platform	16.80% <ø> (-0.03%)	⬇️
app-ui-vault	28.14% <ø> (+<0.01%)	⬆️
authenticator	6.13% <ø> (-0.05%)	⬇️
lib-core-network-bridge	4.05% <ø> (-0.04%)	⬇️
lib-data-ui	1.14% <ø> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[david-livefront]

Minor issue of mismatched types. The test was still passing because the type was erased during runtime but I fixed it all the same 😄

[aj-rosado]

LGTM

[david-livefront]

Thanks @aj-rosado