fix: don't initialize wallet in state.keyring effect

[gomesalexandre]

Description

Fixes redundant initialization - we already do this in the load() method.
This is only a performance bottleneck for most wallets, however, for Ledger, this means we attempt pairing the device without explicitly having Ledger as a selected wallet.

Pull Request Type

• 🐛 Bug fix (Non-breaking Change: Fixes an issue)
• 🛠️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
• 💅 New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

• relates to Identify and implement lazy imports #5445

Risk

• Wallets pairing might be broken, test with all wallets. Tested with Ledger, Native, KK, MM and WCV2

Testing

• Ensure wallet pairing is still happy against prod for all wallets

Engineering

• ☝🏽

Operations

• ☝🏽

Screenshots (if applicable)

[gomesalexandre]

Current dependencies on/for this PR:

• develop
  • PR fix: don't initialize wallet in state.keyring effect #5462 👈
    • PR feat: lazy load hdwallet adapter modules part 1 #5463
      • PR feat: actually lazy load hdwallet adapter modules #5466
        • PR feat: walletprovider adapters type narrowing #5468
          • PR perf: defi only render current step if active #5476

This comment was auto-generated by Graphite.