[PM-38144] feat: Add bank account add/edit screens

[SaintPatrck]

🎟️ Tracking

• https://bitwarden.atlassian.net/browse/PM-38144

📔 Objective

Add the add/edit screen for Bank Account items. The form renders all 10 fields — bank name, name on account, account type (picker), account number, routing number, branch number, PIN, SWIFT code, IBAN, and bank contact phone — with reveal toggles on the account number, PIN, and IBAN. Saving round-trips through the SDK, and Bank Account is now selectable from the add-item picker and the Bank accounts group's add button, gated behind the newItemTypes feature flag.

📸 Screenshots

Add

Header	Header

Edit

Header	Header

[github-actions]

Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR (PM-38144) wires up the Bank Account add/edit screens, mirroring the established Card/Driver's License pattern across 25 files. BankAccountItemState covers all 10 fields with reveal toggles defaulted hidden for the account number, PIN, and IBAN; the visibility booleans are intentionally omitted from BankAccountView so they never reach the SDK. Bank Account is added to CipherType.canCreateCases (kept behind .newItemTypes via newItemTypesGatedCases), wired through AddEditItemState/AddEditItemProcessor/AddEditItemView, restored as a .button in VaultGroupState, and round-trips through BitwardenSdk.BankAccount.init(cipherBankAccountModel:) / CipherView.bankAccountItemState(). Unit and ViewInspector coverage exercises every field, every visibility toggle, and the unknown-account-type mapping.

Code Review Details

No findings.

Notes on patterns intentionally preserved (not findings):

• Unknown server accountType values map to .default and are dropped on re-save — same pattern as Card brand.
• Visibility-toggle events (parallel to cipherClientToggledCardCodeVisible) are not emitted for PIN/account number/IBAN — matches the Driver's License precedent; a new EventType case is not in scope here.
• Reveal-toggle booleans reset to false when streamCipherDetails re-applies the cipher — consistent with Card/Driver's License behavior.
• accountNumber, routingNumber, branchNumber, and pin use a default text-field configuration rather than .numeric(...) — appropriate, since international values are not strictly numeric.

[codecov]

Codecov Report

❌ Patch coverage is 88.68895% with 88 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.92%. Comparing base (89c925a) to head (e540865).

Files with missing lines	Patch %	Lines
...itBankAccountItem/AddEditBankAccountItemView.swift	73.71%	46 Missing ⚠️
...tem/AddEditBankAccountItemView+SnapshotTests.swift	0.00%	42 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2757      +/-   ##
==========================================
- Coverage   87.85%   86.92%   -0.94%     
==========================================
  Files        1712     1901     +189     
  Lines      166702   180569   +13867     
==========================================
+ Hits       146463   156958   +10495     
- Misses      20239    23611    +3372     

☔ 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.