[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 added for passport vault items, including the new ViewPassportItemView, the ViewPassportItemAction enum, the ViewItemProcessor passport action handler, the isPassportDetailsSectionEmpty state helper, and the new CopyableField cases. The implementation closely mirrors the existing ViewDriversLicenseItemView pattern: sensitive fields (passport number, national identification number) are masked behind reveal toggles via PasswordText/PasswordVisibilityButton, each field hides when empty, and the section hides when fully empty. Test coverage is strong across ViewInspector interaction tests, processor toggle/error-path tests, and (disabled) snapshot tests.

Code Review Details

No findings at or above the reporting threshold. The processor's handlePassportAction correctly guards both loaded-data and passport-type with error reporting (consistent with the SSH key handler), state wiring through CipherItemState is complete, and no plaintext sensitive data is logged or persisted.

Existing review threads (build artifacts, private-view docs, moving section-empty into state, test alphabetization) are already addressed by the author or are active human-reviewer discussions, so no duplicate comments were posted.

[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 89.52569% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.94%. Comparing base (8c9d9eb) to head (55a2e26).
⚠️ Report is 16 commits behind head on main.

Files with missing lines	Patch %	Lines
...sportItem/ViewPassportItemView+SnapshotTests.swift	0.00%	46 Missing ⚠️
...Vault/VaultItem/ViewItem/ViewItemDetailsView.swift	0.00%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2770      +/-   ##
==========================================
+ Coverage   87.93%   87.94%   +0.01%     
==========================================
  Files        1728     1741      +13     
  Lines      168727   170489    +1762     
==========================================
+ Hits       148365   149938    +1573     
- Misses      20362    20551     +189     

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