Migrate mobile docs to @walletconnect/universal-provider#2
Open
CassioMG wants to merge 9 commits intoadd-developer-docsfrom
Open
Migrate mobile docs to @walletconnect/universal-provider#2CassioMG wants to merge 9 commits intoadd-developer-docsfrom
CassioMG wants to merge 9 commits intoadd-developer-docsfrom
Conversation
Replace @walletconnect/sign-client with @walletconnect/universal-provider in installation instructions. Add Stellar Wallets Kit alternative note and link to Reown docs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace SignClient connect/event/disconnect patterns with UniversalProvider equivalents. Add public key extraction example. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace all client.request() calls with provider.request() pattern. Update full connect-sign-submit example. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the Mobile integration docs to use WalletConnect’s @walletconnect/universal-provider API (instead of @walletconnect/sign-client) across installation, connection, and signing examples, aligning the guide with current Reown/WalletConnect documentation for non-EVM chains.
Changes:
- Swaps installation and initialization examples to
@walletconnect/universal-provider. - Updates connection flow to use
display_uri+provider.connect()(noapproval()step). - Migrates all signing examples to
provider.request(payload, chain)and refreshes the end-to-end sample accordingly.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| mobile/installation.md | Updates install/init instructions for @walletconnect/universal-provider and adds an alternatives note + link to current Reown docs. |
| mobile/connecting.md | Documents the display_uri event and provider.connect() resolving on approval; simplifies disconnect to provider.disconnect(). |
| mobile/signing.md | Converts method examples and the full flow to provider.request(..., "stellar:pubnet") and replaces SignClient usage with UniversalProvider. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The in-app browser detection is a Stellar Wallets Kit implementation detail, not needed by dapp developers using UniversalProvider directly. Simplified the full example and clarified the in-app vs external browser UX difference in prose instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add @reown/appkit to installation alongside universal-provider
- Initialize createAppKit with manualWCControl for standalone modal
- Use modal.open({ uri }) in display_uri handler instead of console.log
- Add links to Reown UniversalProvider and AppKit modal docs
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Wire the provider to the modal via the universalProvider option, matching the pattern from the Reown migration guide. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Critical:
- Replace display_uri + modal.open({ uri }) with modal.open() before
provider.connect() — AppKit handles URI display automatically when
manualWCControl and universalProvider are set
Important:
- Add comment explaining mainnet placeholder in AppKit networks config
- Fix README chainId hint to reference provider.request() argument
- Add session null check after provider.connect()
Suggestions:
- Switch to named import { UniversalProvider } matching Reown docs
- Add cross-file reference notes to connecting.md and signing.md
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
CassioMG
commented
Apr 11, 2026
|
|
||
| ## Alternatives | ||
|
|
||
| [Stellar Wallets Kit](https://stellarwalletskit.dev/) provides a multi-wallet interface that includes WalletConnect support, but currently only supports `stellar_signXDR` and `stellar_signAndSubmitXDR`. To use all four Freighter Mobile methods — including `stellar_signMessage` and `stellar_signAuthEntry` — implement the WalletConnect integration directly as shown in this guide. |
Author
There was a problem hiding this comment.
created a ticket to work with Enrique to add those 2 extra methods to SWK: stellar/freighter-mobile#815
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
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.
Summary
@walletconnect/sign-clientwith@walletconnect/universal-providerin all mobile code samples@reown/appkitmodal for QR code display and wallet selection (viacreateAppKitwithmanualWCControl)signMessage/signAuthEntrysupportisFreighterInAppBrowserbranching from full example (Stellar Wallets Kit implementation detail, not needed by dapp developers)provider.request(payload, chain)vsclient.request({ topic, chainId, request })Context
@walletconnect/sign-clientis no longer referenced in the official WalletConnect/Reown documentation.@walletconnect/universal-provideris the actively documented package for non-EVM chains.@reown/appkitwithmanualWCControl: truereplaces the deprecated@walletconnect/modalfor displaying QR codes and wallet lists.Files changed
mobile/installation.md— install bothuniversal-providerand@reown/appkit, init provider + modal, alternatives sectionmobile/connecting.md—display_uriopens AppKit modal,provider.connect()pattern, simplified disconnectmobile/signing.md— all 4 method examples migrated toprovider.request(), full example with modal🤖 Generated with Claude Code