Skip to content

[PM-35105] Extract shared key-envelope helpers in bitwarden-crypto#1203

Open
quexten wants to merge 8 commits into
km/hazmat-cose-symmetricfrom
km/safe-shared-helpers
Open

[PM-35105] Extract shared key-envelope helpers in bitwarden-crypto#1203
quexten wants to merge 8 commits into
km/hazmat-cose-symmetricfrom
km/safe-shared-helpers

Conversation

@quexten

@quexten quexten commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

Stacked PR 3/4 (on top of #1202).

Factors the duplicated key-decode and contained-key-id logic out of the symmetric-key and password-protected envelopes into shared helpers (decode_sealed_symmetric_key, extract_key_id, set_contained_key_id) on the safe module, and exposes set_header_value to the module. No behavior change.

Base: km/hazmat-cose-symmetric (#1202).

Factors the duplicated key-decode and contained-key-id logic out of the
symmetric-key and password-protected envelopes into shared helpers
(decode_sealed_symmetric_key, extract_key_id, set_contained_key_id) on the safe
module, and exposes set_header_value to the module. No behavior change.
@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

🔍 SDK Breaking Change Detection

SDK Version: km/safe-shared-helpers (6bfc6bf)

⚠️ If breaking changes are detected, a corresponding pull request addressing them must be ready for merge in the affected client repository.

Client Status Details
typescript ✅ No breaking changes detected Compilation passed with new SDK version - View Details
android ✅ No breaking changes detected Compilation passed with new SDK version - View Details

Breaking change detection uses the build of the SDK from this branch, including any incompatibities pre-existing on or merged into this branch. Check the workflow logs to confirm.
Results update as workflows complete.

@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 84.61538% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.16%. Comparing base (f26a006) to head (8d78925).
⚠️ Report is 1 commits behind head on km/hazmat-cose-symmetric.

Files with missing lines Patch % Lines
...crypto/src/safe/password_protected_key_envelope.rs 66.66% 6 Missing ⚠️
...itwarden-crypto/src/safe/symmetric_key_envelope.rs 62.50% 3 Missing ⚠️
crates/bitwarden-crypto/src/safe/mod.rs 97.36% 1 Missing ⚠️
Additional details and impacted files
@@                    Coverage Diff                    @@
##           km/hazmat-cose-symmetric    #1203   +/-   ##
=========================================================
  Coverage                     85.16%   85.16%           
=========================================================
  Files                           467      468    +1     
  Lines                         64291    64295    +4     
=========================================================
+ Hits                          54754    54759    +5     
+ Misses                         9537     9536    -1     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

bw-ghapp Bot and others added 6 commits June 22, 2026 09:27
…dc2f6b56abd6adf9 (#1200)

Updates the Identity bindings to
`408002d1b6e6c6b6aee907c1dc2f6b56abd6adf9`

Co-authored-by: bw-ghapp[bot] <178206702+bw-ghapp[bot]@users.noreply.github.com>
Comment thread crates/bitwarden-crypto/src/safe/mod.rs Outdated
Comment thread crates/bitwarden-crypto/src/safe/mod.rs Outdated
@sonarqubecloud

Copy link
Copy Markdown

@quexten quexten marked this pull request as ready for review June 23, 2026 03:03
@quexten quexten requested a review from a team as a code owner June 23, 2026 03:03
@quexten quexten requested review from mzieniukbw and removed request for a team June 23, 2026 03:03
@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Claude Code is reviewing this pull request...

If this comment does not update with results, check the Actions log.

@quexten quexten added the ai-review Request a Claude code review label Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant