Skip to content

Comments

Sign-in wallet prioritization logic#166

Draft
tansawit wants to merge 2 commits intomainfrom
cursor/sign-in-wallet-prioritization-logic-e9f3
Draft

Sign-in wallet prioritization logic#166
tansawit wants to merge 2 commits intomainfrom
cursor/sign-in-wallet-prioritization-logic-e9f3

Conversation

@tansawit
Copy link
Contributor

@tansawit tansawit commented Feb 18, 2026

Prioritize owned popular wallets in sign-in suggestions when the number of ready wallets exceeds the display limit.

This ensures that users with many installed wallets see their popular ones (e.g., MetaMask, Rabby) first in the limited 5-slot display on the sign-in page, improving discoverability without reordering when within the limit.


Open in Cursor Open in Web


Note

Low Risk
Small, UI-only ordering change gated to cases with many ready wallets; includes unit tests and doesn’t affect connection/auth flows.

Overview
Updates the sign-in wallet suggestion list to prioritize installed wallets that match a curated popular-wallet set (by id or normalized name) when there are more ready connectors than the 5-slot display limit, while preserving the original ordering when within the limit.

Introduces reusable prioritizeSignInWallets logic plus Vitest coverage, and replaces the previous simple slice(0, 5) selection in SignIn.tsx with this prioritization and a shared SIGN_IN_WALLET_LIMIT constant.

Written by Cursor Bugbot for commit 58c4a92. This will update automatically on new commits. Configure here.

Co-authored-by: Sawit Trisirisatayawong <tansawit@users.noreply.github.com>
@cursor
Copy link

cursor bot commented Feb 18, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 18, 2026

Deploying interwovenkit with  Cloudflare Pages  Cloudflare Pages

Latest commit: 58c4a92
Status: ✅  Deploy successful!
Preview URL: https://d50fb316.interwovenkit.pages.dev
Branch Preview URL: https://cursor-sign-in-wallet-priori.interwovenkit.pages.dev

View logs

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 18, 2026

Deploying interwovenkit-staging with  Cloudflare Pages  Cloudflare Pages

Latest commit: 58c4a92
Status: ✅  Deploy successful!
Preview URL: https://d6406cd3.interwovenkit-staging.pages.dev
Branch Preview URL: https://cursor-sign-in-wallet-priori.interwovenkit-staging.pages.dev

View logs

@coderabbitai
Copy link

coderabbitai bot commented Feb 18, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cursor/sign-in-wallet-prioritization-logic-e9f3

Comment @coderabbitai help to get the list of available commands and usage tips.

@cloudflare-workers-and-pages
Copy link

Deploying interwovenkit-testnet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 58c4a92
Status: ✅  Deploy successful!
Preview URL: https://80c1388e.interwovenkit-testnet.pages.dev
Branch Preview URL: https://cursor-sign-in-wallet-priori.interwovenkit-testnet.pages.dev

View logs

@tansawit tansawit self-assigned this Feb 18, 2026
@tansawit tansawit added the type:enhancement New feature or request label Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:low type:enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants