Add sol wallet signature auth to SDK#14019
Merged
raymondjacobson merged 8 commits intomainfrom Mar 28, 2026
Merged
Conversation
|
raymondjacobson
changed the title
Adds a WalletAuth module to @audius/sdk so apps can hand off wallet credentials (pubkey, message, signature) and have the SDK inject X-Solana-* headers automatically via middleware. Apps connect wallets however they want (Phantom, wallet-adapter, etc.) — the SDK owns the message format and request plumbing but bundles zero wallet dependencies. The coin-gated example now uses sdk.walletAuth.setCredential() and a single sdk.tracks.streamTrack() call for both OAuth and wallet auth paths, removing the manual fetch + header construction. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- walletAuth/ (4 files) → solWallet/ (2 files) - createAuthMessage → createSolWalletSignatureMessage - addWalletAuthMiddleware → addSolWalletSignatureMiddleware - sdk.walletAuth → sdk.solWallet - Strip superfluous comments, add usage example to middleware docstring Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Moves connect + message creation + signing + base58 encoding into SolanaWallet.auth(provider), so consumers just pass window.solana (or any wallet adapter). Includes a minimal inline base58 encoder to avoid adding a dependency. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove Authorization guard in SDK middleware so wallet headers are sent alongside OAuth tokens (API needs both to merge balances) - Add walletConnected to useGatedTracks query key so React Query refetches after wallet connect instead of serving stale cache Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
raymondjacobson
force-pushed
the
ray/sdk-wallet-auth
branch
from
16ac54d to
e58df77
Compare
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
🌐 Web preview readyPreview URL: https://audius-web-preview-pr-14019.audius.workers.dev Unique preview for this PR (deployed from this branch). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add
WalletAuthmodule to@audius/sdk— apps callsdk.walletAuth.setCredential()with a signed credential and the SDK injectsX-Solana-*headers into all requests via middleware🤖 Generated with Claude Code