Fix: update isConnected method to exclude wallets without Identity contract

[blueogin]

Description

Base has no Identity contract, so MultiWallet.isConnected() was failing when it ran on BaseAdminWallet and broke /verify/topwallet. We now skip BaseAdminWallet in the isConnected check so only wallets with Identity (e.g. Celo) are used.
You can refer msgs on Slack

About # (link your issue here)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist:

• PR title matches follow: (Feature|Bug|Chore) Task Name
• My code follows the style guidelines of this project
• I have followed all the instructions described in the initial task (check Definitions of Done)
• I have performed a self-review of my own code
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have added reference to a related issue in the repository
• I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
• I have added screenshots related to my pull request (for frontend tasks)
• I have pasted a gif showing the feature.
• @mentions of the person or team responsible for reviewing proposed changes

Summary by Sourcery

Bug Fixes:

• Prevent MultiWallet.isConnected from failing on BaseAdminWallet by excluding wallets without an Identity contract from the connection check.

[sourcery-ai]

Hey - I've left some high level feedback:

• Relying on instanceof BaseAdminWallet inside MultiWallet tightly couples this class to a specific wallet implementation; consider adding a capability flag or interface method (e.g. supportsIdentity() or similar) on wallets and filtering based on that instead.
• If all wallets are filtered out (e.g. only admin wallets are configured), isConnected will always return false; consider explicitly handling/logging this scenario so it's easier to debug unexpected connectivity results.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Relying on `instanceof BaseAdminWallet` inside `MultiWallet` tightly couples this class to a specific wallet implementation; consider adding a capability flag or interface method (e.g. `supportsIdentity()` or similar) on wallets and filtering based on that instead.
- If all wallets are filtered out (e.g. only admin wallets are configured), `isConnected` will always return false; consider explicitly handling/logging this scenario so it's easier to debug unexpected connectivity results.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}