feat: SDK update for version 22.2.0#330
Conversation
Greptile SummaryThis PR updates the CLI SDK to v22.2.0, adding OS keychain storage for OAuth refresh tokens (via
Confidence Score: 4/5Safe to merge with one fix: the error path after account verification fails in the OAuth login flow must call deleteStoredRefreshToken before removing the session from config, otherwise a keyring entry for the revoked session is silently orphaned. The keychain integration is careful and the happy path is correct. Every other session-removal site in this PR properly cleans up keyring storage before removing the config entry. The one exception is the catch block in loginWithOAuthDevice that fires when getCurrentAccount() fails after a successful token exchange — at that point setStoredRefreshToken has already written to the keychain, but the subsequent cleanup skips deleteStoredRefreshToken, leaving an orphaned (though revoked and harmless) keychain entry. lib/auth/login.ts — the catch block after getCurrentAccount() needs a deleteStoredRefreshToken(id) call before globalConfig.removeSession(id). Important Files Changed
Reviews (2): Last reviewed commit: "chore: update Command Line SDK to 22.2.0" | Re-trigger Greptile |
This PR contains updates to the SDK for version 22.2.0.
What's Changed
@napi-rs/keyring, falling back to config--typeoption tofunctions list-specificationsandsites list-specifications@appwrite.io/consoledependency to^15.1.1logoutwith a(current)markerrundll32for OAuth flows