[PM-33315] Create @bitwarden/logging

[dani-garcia]

🎟️ Tracking

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

📔 Objective

Create the @bitwarden/logging-angular library, which contains an angular-specific wrapper of FlightRecorder for dependency injection.

📸 Screenshots

[github-actions]

Checkmarx One – Scan Summary & Details – c29ea652-0a86-484c-9905-3450953e25fd

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.96%. Comparing base (087fcd8) to head (bbb2b81).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #20197   +/-   ##
=======================================
  Coverage   46.96%   46.96%           
=======================================
  Files        3911     3913    +2     
  Lines      118361   118367    +6     
  Branches    18111    18111           
=======================================
+ Hits        55585    55592    +7     
  Misses      58602    58602           
+ Partials     4174     4173    -1     

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR scaffolds a new @bitwarden/logging-angular library that exposes FlightRecorderService, an @Injectable subclass of the framework-agnostic FlightRecorder from @bitwarden/logging. The service wires SdkLoadService.Ready automatically so Angular consumers can inject it via DI. The new library follows established monorepo conventions (README, eslint/jest/tsconfig wiring, project.json, CODEOWNERS, tsconfig.base.json path alias, root jest.config.js registration), and the previously flagged alphabetical ordering of the path alias has been addressed.

Code Review Details

No findings. The implementation is minimal and correct: the service's sole responsibility is DI wiring, and the underlying FlightRecorder is already covered by @bitwarden/logging's tests. The co-located spec verifies injectability, root-singleton semantics, and inheritance — which is the appropriate scope for a thin Angular wrapper. No new external dependencies were introduced; the package-lock.json change is standard workspace-link churn.

[djsmith85]

Changes are looking good!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud