Add CactusLink wallet support#2097
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds Cactus Link as a supported wallet by upgrading Stellar Wallets Kit and registering the new module in the app’s wallet module list, along with updating E2E expectations for the wallet-connect modal.
Changes:
- Upgraded
@creit.tech/stellar-wallets-kitto^2.2.0. - Registered
CactusLinkModulein the WalletKit initialization modules list (affecting both testnet and mainnet via shared module arrays). - Updated the sign-transaction E2E test to expect 8 wallets and verify “Cactus Link” appears.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
package.json |
Bumps Stellar Wallets Kit dependency to enable Cactus Link module usage. |
src/components/WalletKit/WalletKitContextProvider.tsx |
Adds and registers CactusLinkModule in the initialized SWK modules list. |
tests/e2e/signTransactionPage.test.ts |
Updates wallet modal assertions for the new wallet entry. |
pnpm-lock.yaml |
Lockfile refresh due to dependency update (includes a concerning non-registry tarball entry). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Preview is available here: |
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
Preview is available here: |
quietbits
left a comment
There was a problem hiding this comment.
Looks good. I tested Wallet Kit with Freighter (Testnet) and Ledger (Mainnet).
|
Preview is available here: |
|
Preview is available here: |
Summary
Resolves #2075
Validation