[PM-38154] feat: Add Passport view screen

[SaintPatrck]

🎟️ Tracking

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

📔 Objective

Adds the read-only detail view for passport vault items. The passport details section renders given name, surname, date of birth, sex, birth place, nationality, passport number, passport type, national identification number, issuing country, issuing authority/office, issue date, and expiration date. Each field is hidden when empty, and the section is hidden entirely when the passport has no details.

The passport number and national identification number are masked with a reveal toggle; the name and number fields support copy to clipboard.

📸 Screenshots

Figma	Actual

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

Reviewed the read-only Passport detail view (ViewPassportItemView), its action enum, the ViewItemProcessor passport handler, the ViewVaultItemState protocol addition, new CopyableField cases, and the accompanying ViewInspector and snapshot tests. The implementation closely mirrors the established ViewBankAccountItemView / ViewDriversLicenseItemView patterns: sensitive fields (passport number, national identification number) are masked behind reveal toggles, copyable fields route through the standard copyPressed action, and each field hides when empty with the whole section hidden when isPassportDetailsSectionEmpty. Error logging uses static strings only, preserving zero-knowledge expectations.

Code Review Details

No findings. The change is consistent with existing vault item view conventions, the processor guards mirror handleBankAccountAction (loaded-data and type checks), the new ViewVaultItemState.passportItemState requirement is already satisfied by the sole conformer CipherItemState, and tests cover copy buttons, visibility toggles, the empty state, and both error paths.

[claude]

⚠️ IMPORTANT: Build artifacts committed to the repository

Details and fix

This PR adds the entire export/build.xcresult/ bundle (binary Data/* blobs plus this Info.plist). These are Xcode test/build result artifacts that should not be tracked in git — they bloat the repository and history permanently, and are unrelated to the passport view feature.

Fix:

• git rm -r --cached export/ and remove the directory from the commit
• Add export/ (or *.xcresult) to .gitignore so it cannot be re-added accidentally

Confirmed these paths do not exist on origin/main and are not currently covered by .gitignore.

[SaintPatrck]

Resolved in 6d243a7

[codecov]

Codecov Report

❌ Patch coverage is 97.69737% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.98%. Comparing base (8ba847e) to head (c24317b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...Vault/VaultItem/ViewItem/ViewItemDetailsView.swift	0.00%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2770      +/-   ##
==========================================
+ Coverage   80.91%   80.98%   +0.06%     
==========================================
  Files        1023     1024       +1     
  Lines       65297    65599     +302     
==========================================
+ Hits        52836    53124     +288     
- Misses      12461    12475      +14     

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