fix(sdk): NaN-safe expiry parsing + AsyncStorage token store tests

[rickyrombo]

getAccessTokenExpiry() / getRefreshTokenExpiry() returned NaN when localStorage or AsyncStorage held a non-numeric value (corruption, manual edits, format migration). NaN silently bypasses expiry checks since Date.now() >= NaN is always false, meaning expired sessions could be treated as valid.

Changes

• NaN safety (TokenStoreLocalStorage, TokenStoreAsyncStorage): replace bare Number(raw) with Number.isFinite() guard — invalid stored values return null instead of NaN
• Tests (tokenStoreAsyncStorage.test.ts): new test file mocking @react-native-async-storage/async-storage, covering token reads, expiry persistence, removal of expiry keys when not provided, NaN/corruption handling, and clear

// Before — NaN slips through
const raw = await AsyncStorage.getItem(key)
return raw ? Number(raw) : null  // returns NaN for "corrupted-value"

// After — invalid values treated as absent
const n = Number(raw)
return Number.isFinite(n) ? n : null

[changeset-bot]

🦋 Changeset detected

Latest commit: cd7e1fe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
@audius/sdk	Minor
@audius/sdk-legacy	Patch
@audius/protocol-dashboard	Patch
@audius/sp-actions	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[Copilot]

Pull request overview

This PR enhances the SDK OAuth session handling by persisting token expiry timestamps in token stores and using that information to proactively refresh/validate sessions, plus adding a single retry in getUser() after a token refresh on 401.

Changes:

• Extend OAuthTokenStore to track access/refresh token expiries and persist them (Memory, LocalStorage, AsyncStorage).
• Update OAuth.isAuthenticated() to consider expiry (including silent refresh) and update getUser() to retry once on 401 after refresh.
• Expand/adjust unit tests for the new expiry/refresh behavior and add a changeset entry.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/sdk/src/sdk/oauth/tokenStore.ts	Expands the public token store interface with expiry getters and optional expiry params in setTokens().
packages/sdk/src/sdk/oauth/TokenStoreMemory.ts	Stores expiry timestamps in-memory and clears them with tokens.
packages/sdk/src/sdk/oauth/TokenStoreLocalStorage.ts	Persists expiry timestamps to localStorage alongside tokens.
packages/sdk/src/sdk/oauth/tokenStoreLocalStorage.test.ts	Adds tests for expiry key persistence/removal/reading in localStorage store.
packages/sdk/src/sdk/oauth/TokenStoreAsyncStorage.ts	Persists expiry timestamps to AsyncStorage alongside tokens.
packages/sdk/src/sdk/oauth/tokenStore.test.ts	Updates memory-store tests to include expiry behavior.
packages/sdk/src/sdk/oauth/OAuth.ts	Implements expiry-aware isAuthenticated(), adds getUser() 401 refresh+retry, and persists token expiries from token responses.
packages/sdk/src/sdk/oauth/OAuth.test.ts	Adds/updates tests for expiry-aware authentication and getUser() retry behavior.
.changeset/oauth-token-expiry-tracking.md	Declares a release bump and describes the OAuth reliability improvements.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14078.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run