Skip to content

Update SDK to 02747b1 (2.0.0-4846-c74b7d1)#2575

Merged
LRNcardozoWDF merged 5 commits into
mainfrom
cmcg/sdk-update
Apr 24, 2026
Merged

Update SDK to 02747b1 (2.0.0-4846-c74b7d1)#2575
LRNcardozoWDF merged 5 commits into
mainfrom
cmcg/sdk-update

Conversation

@LRNcardozoWDF
Copy link
Copy Markdown
Member

@LRNcardozoWDF LRNcardozoWDF commented Apr 24, 2026

🎟️ Tracking

PM-35125

📔 Objective

Adapts the iOS client for two breaking changes introduced by the SDK update (2.0.0-4846-c74b7d1):

  • PM-28926 Implements the new setBulk, removeBulk, and removeAll methods on SdkCipherRepository to conform to the updated CipherRepository protocol. Includes tests and mock updates.
  • PM-31051 Passes upgradeToken: nil to InitUserCryptoRequest and adds getUpgradedUserKey to MockCryptoClient to conform to the updated CryptoClientProtocol.

What's Changed

@LRNcardozoWDF LRNcardozoWDF requested review from a team and matt-livefront as code owners April 24, 2026 17:20
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context t:deps Change Type - Dependencies labels Apr 24, 2026
@LRNcardozoWDF LRNcardozoWDF added ai-review Request a Claude code review and removed app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context labels Apr 24, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

❌ Patch coverage is 79.00000% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.08%. Comparing base (7fc13de) to head (7ec71b0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...sitories/TestHelpers/MockSdkCipherRepository.swift 26.66% 11 Missing ⚠️
...e/Auth/Services/TestHelpers/MockCryptoClient.swift 20.00% 4 Missing ⚠️
...e/Auth/Services/TestHelpers/MockCryptoClient.swift 20.00% 4 Missing ⚠️
.../Vault/Repositories/SdkCipherRepositoryTests.swift 94.11% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2575      +/-   ##
==========================================
- Coverage   87.16%   86.08%   -1.09%     
==========================================
  Files        1886     2116     +230     
  Lines      166723   181621   +14898     
==========================================
+ Hits       145331   156348   +11017     
- Misses      21392    25273    +3881     

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

Comment on lines +49 to +51
func removeBulk(keys: [String]) async throws {
for key in keys {
try await cipherDataStore.deleteCipher(id: key, userId: userId)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎨 This could be slow to delete one-by-one if there's a large number of ciphers being deleted. Would it be worth updating this and setBulk(values:) to use batch requests? Or at the very least create a ticket to follow-up on this since I don't think these are used yet.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch, I create a task to deal with this.

Comment thread BitwardenShared/Core/Vault/Repositories/TestHelpers/MockSdkCipherRepository.swift Outdated
Comment thread BitwardenShared/Core/Vault/Repositories/SdkCipherRepository.swift Outdated
@github-actions github-actions Bot added app:password-manager Bitwarden Password Manager app context app:authenticator Bitwarden Authenticator app context labels Apr 24, 2026
@LRNcardozoWDF LRNcardozoWDF enabled auto-merge (squash) April 24, 2026 22:05
@LRNcardozoWDF LRNcardozoWDF merged commit af25d88 into main Apr 24, 2026
18 of 23 checks passed
@LRNcardozoWDF LRNcardozoWDF deleted the cmcg/sdk-update branch April 24, 2026 22:49
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 app:authenticator Bitwarden Authenticator app context app:password-manager Bitwarden Password Manager app context t:deps Change Type - Dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants