Skip to content

Auth/PM-40520 - Bump SDK versions in clients to get new registration open org invite crypto APIs - #22134

Merged
JaredSnider-Bitwarden merged 2 commits into
mainfrom
auth/pm-40520/update-sdk-to-latest-to-get-registration-open-org-invite-seal-apis
Jul 29, 2026
Merged

Auth/PM-40520 - Bump SDK versions in clients to get new registration open org invite crypto APIs#22134
JaredSnider-Bitwarden merged 2 commits into
mainfrom
auth/pm-40520/update-sdk-to-latest-to-get-registration-open-org-invite-seal-apis

Conversation

@JaredSnider-Bitwarden

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-40520
bitwarden/sdk-internal#1295

📔 Objective

Bump SDK versions in clients to get new registration open org invite crypto APIs

@JaredSnider-Bitwarden
JaredSnider-Bitwarden marked this pull request as ready for review July 29, 2026 18:16
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR bumps @bitwarden/sdk-internal and @bitwarden/commercial-sdk-internal from 0.2.0-main.926 to 0.2.0-main.927 in package.json, with a matching package-lock.json regeneration. No application code changed, and no other manifest in the repo pins these packages, so the bump is self-consistent. Both packages are pre-existing approved dependencies and this is a single prerelease build increment, so no AppSec dependency-approval or major-version migration concerns apply. The remaining lock file churn (newly hoisted chokidar/readdirp/utf-8-validate optional-peer entries and devOptionaldev reclassifications) is normal npm regeneration output.

Code Review Details

No code findings.

CI Observation

  • ❓ : Run Nx Affected Tasks is failing (~47m) while all Build/CLI/Web/Browser jobs and Run tests - CLI pass. Worth confirming this is unrelated to the SDK bump (e.g. a type break surfaced by the new SDK typings) before merging. The Rust/desktop_native SDK git rev in apps/desktop/desktop_native/Cargo.toml is intentionally unchanged and is tracked separately from the npm packages.

Dependency Changes

Package Change Ecosystem
@bitwarden/sdk-internal 0.2.0-main.926 → 0.2.0-main.927 npm
@bitwarden/commercial-sdk-internal 0.2.0-main.926 → 0.2.0-main.927 npm

@JaredSnider-Bitwarden JaredSnider-Bitwarden added the t:deps Change Type - Dependencies label Jul 29, 2026
@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.62%. Comparing base (45a557e) to head (2a43714).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #22134   +/-   ##
=======================================
  Coverage   52.62%   52.62%           
=======================================
  Files        4194     4194           
  Lines      132973   132973           
  Branches    20911    20911           
=======================================
+ Hits        69973    69974    +1     
  Misses      57886    57886           
+ Partials     5114     5113    -1     

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

@JaredSnider-Bitwarden
JaredSnider-Bitwarden merged commit 3586313 into main Jul 29, 2026
161 of 163 checks passed
@JaredSnider-Bitwarden
JaredSnider-Bitwarden deleted the auth/pm-40520/update-sdk-to-latest-to-get-registration-open-org-invite-seal-apis branch July 29, 2026 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:deps Change Type - Dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants