Skip to content

Conversation

@arlert-armin
Copy link
Contributor

@arlert-armin arlert-armin commented Nov 26, 2025

Summary

This PR adds Cosmos (Cosmostation) support to the Hub and integrates its connection and account model into the Hub’s Cosmos namespace architecture.
While core functionality works, Cosmostation introduces several important limitations account switching, and auto-connect behavior. These limitations originate from Cosmostation itself and not the Hub integration.

Implementation notes / limitations

Feature

⚠️ Switch Account

Cosmostation does not provide a working account-switch notification mechanism for Cosmos.
When a user switches accounts inside the wallet:

  • The dApp is not notified.
  • The Hub continues using the old account, since no event or update is emitted.

A reconnect is required to update the account state.

⚠️ Auto Connect (Cosmos)

Cosmos eager-connect behavior is tied to EVM connectivity:

  • Eager-connect for Cosmos only works if EVM is already connected.

  • If EVM is not connected, Cosmos eager-connect will not trigger.

  • As a result:

    • Auto-connect fails for Cosmos unless EVM is connected.

Fixes # (issue)

How did you test this change?

I validated Cosmostation under the Hub integration across multiple scenarios:

  • Confirmed initial Cosmos connection works.

  • Verified account retrieval and standard Cosmos operations.

  • Tested auto-connect:

    • Confirmed eager-connect works only when EVM is connected first.
    • Verified it fails when EVM is not connected (expected limitation).
  • Ensured that reconnecting restores the correct account state.

  • Test A

  • Test B

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

@arlert-armin arlert-armin force-pushed the feat/migrate-keplr-to-the-hub branch 2 times, most recently from 8435f28 to b4057b2 Compare November 30, 2025 07:16
@arlert-armin arlert-armin force-pushed the feat/rf-2783-migrate-cosmostation-to-the-hub branch from d0a3120 to 27a098e Compare November 30, 2025 07:27
@arlert-armin arlert-armin force-pushed the feat/rf-2783-migrate-cosmostation-to-the-hub branch from 27a098e to 69592f0 Compare November 30, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants