[PM-26063] Add flight recorder to Authenticator's settings

[matt-livefront]

🎟️ Tracking

PM-26063

📔 Objective

Adds the Flight Recorder options to Authenticator's settings view. This allows for enabling/disabling Flight Recorder and viewing the list of any logs.

📸 Screenshots

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2025-11-17.at.11.55.59.mov

⏰ Reminders before review

• Contributor guidelines followed
• All formatters and local linters executed and passed
• Written new unit and / or integration tests where applicable
• Protected functional changes with optionality (feature flags)
• Used internationalization (i18n) for all UI strings
• CI builds passed
• Communicated to DevOps any deployment requirements
• Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team

🦮 Reviewer guidelines

• 👍 (:+1:) or similar for great changes
• 📝 (:memo:) or ℹ️ (:information_source:) for notes or general info
• ❓ (:question:) for questions
• 🤔 (:thinking:) or 💭 (:thought_balloon:) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion
• 🎨 (:art:) for suggestions / improvements
• ❌ (:x:) or ⚠️ (:warning:) for more significant problems or concerns needing attention
• 🌱 (:seedling:) or ♻️ (:recycle:) for future improvements or indications of technical debt
• ⛏ (:pick:) for minor or nitpick changes

[claude]

Claude finished @matt-livefront's task —— View job

---

Code Review Complete

• Evaluate PR title and description
• Review architecture and code changes
• Check testing coverage
• Verify code style compliance
• Check security considerations
• Post comprehensive review findings

---

Summary

This PR integrates the Flight Recorder feature into the Authenticator app's settings screen. The implementation follows the established architecture pattern for the Authenticator app (coordinator → processor → view), adds the necessary navigation routes, and includes comprehensive test coverage.

Changes made:

• Added Flight Recorder section to Authenticator's settings UI
• Integrated FlightRecorderSettingsSectionView component from BitwardenKit
• Added navigation routes for enabling/disabling Flight Recorder and viewing logs
• Implemented streaming of active flight recorder log metadata
• Added corresponding test coverage (processor tests, ViewInspector tests, coordinator tests)

---

Findings

Finding 1: Missing DocC documentation in SettingsProcessor:328:streamFlightRecorderLog()

The streamFlightRecorderLog() method at SettingsProcessor.swift:211 lacks DocC documentation. Per project guidelines, all code requires DocC documentation.

Suggested documentation

/// Streams the flight recorder's active log metadata.
///
/// Subscribes to the flight recorder's active log publisher and updates the state
/// whenever the active log changes. This effect should be started when the settings
/// view appears and will run continuously until cancelled.
private func streamFlightRecorderLog() async {
    for await activeLog in await services.flightRecorder.activeLogPublisher().values {
        state.flightRecorderState.activeLog = activeLog
    }
}

---

Good Practices Observed

• Proper use of unidirectional data flow pattern (Actions → Processor → State → View)
• Comprehensive test coverage across all layers (processor, view, coordinator)
• Correct dependency injection through ServiceContainer
• Proper use of child stores for nested components (FlightRecorderSettingsSectionView)
• Tests follow naming conventions and are properly organized
• Effective reuse of existing FlightRecorderSettingsSectionView component from BitwardenKit
• Proper streaming pattern with AsyncPublisher for reactive log updates

---

Action Items

1. Add DocC documentation to the streamFlightRecorderLog() method

---

[github-actions]

Checkmarx One – Scan Summary & Details – 93fcc988-4102-454a-924b-6548cde3e18e

Great job! No new security vulnerabilities introduced in this pull request

[codecov]

Codecov Report

❌ Patch coverage is 92.62295% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.94%. Comparing base (b8161ee) to head (e50ffda).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...atorShared/UI/Platform/Application/AppModule.swift	0.00%	7 Missing ⚠️
...d/UI/Platform/Settings/Settings/SettingsView.swift	83.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2147      +/-   ##
==========================================
- Coverage   85.38%   83.94%   -1.44%     
==========================================
  Files        1725     1978     +253     
  Lines      145595   160936   +15341     
==========================================
+ Hits       124314   135099   +10785     
- Misses      21281    25837    +4556     

☔ View full report in Codecov by Sentry.
📢 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.