feat: add Aria Wallet support#4245
Conversation
|
@dblackstone1 is attempting to deploy a commit to the OsmoLabs Team on Vercel. A member of the Team first needs to authorize it. |
|
trying to test this. just running into build issues |
|
Updated details with results seen as reason for why testing could not occur. |
WalkthroughTwo new wallet options (Aria Extension and Aria Mobile) are registered in the CosmosKit wallet registry by adding imports and entries to the wallet list generator and registry configuration files. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
packages/web/package.jsonis excluded by!**/*.jsonpackages/web/public/wallets/aria.pngis excluded by!**/*.png,!**/*.png
📒 Files selected for processing (2)
packages/web/config/generate-cosmos-kit-wallet-list.tspackages/web/config/wallet-registry.ts
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Summary
🔇 Additional comments (2)
packages/web/config/generate-cosmos-kit-wallet-list.ts (2)
5-6: LGTM! Import pattern follows existing conventions.The import statements correctly follow the established pattern:
ariaExtensionInfouses camelCase (consistent with other extension imports)AriaMobileInfois aliased toariaMobileInfoto maintain naming consistency (matches the pattern on lines 13-14 for Leap and OKX mobile wallets)
43-44: LGTM! Wallet entries added correctly.Both Aria wallet variants are properly added to the
CosmosKitWalletListarray following the established pattern.
|
Updated with test details |
|
Set to draft until 2.5.0 is approved by Google extension store |
|
2.5 was accepted |
What is the purpose of the change:
Adds Aria wallet support to Osmosis, enabling users to connect using the Aria browser extension and mobile wallet. This follows the same integration pattern used for other cosmos-kit wallets like CDC Wallet and Station.
Linear Task
N/A - External contribution
Brief Changelog
Added @cosmos-kit/aria package dependency
Added Aria extension and mobile wallet imports to generate-cosmos-kit-wallet-list.ts
Added Aria extension and mobile wallet entries to wallet-registry.ts
Added Aria wallet logo
Testing and Verifying
This change has been tested locally by rebuilding the website and verified content and links are expected. Build setup required:
Ran npx tsx for generate scripts (Windows PATH workaround)
Fixed import: @cosmos-kit/aria-mobile exports AriaMobileInfo (PascalCase), not ariaMobileInfo
✅ Verified:
App loads and displays UI correctly
Aria wallet extension detected ([Aria Injected] logs in console)
Wallet connects successfully (auto-connected on page load)
Wallet communication working (getKey requests relayed via content script)
Quote routing functional (local.quoteRouter.routeTokenOutGivenIn queries succeed)
Wallet list generation succeeds
Aria entries correctly added to generated cosmos-kit-wallet-list.ts
❔ Not verified (infrastructure issues unrelated to this PR):
Swap execution blocked by rest.lavenderfive.com RPC returning 503 errors
Balance fetching blocked by same RPC CORS/availability issues