Refactor: Leverage EvmBaseAdapter for Ledger app gating across all EVM chain adapters

[coderabbitai]

Context

In PR #11450, Ledger app verification was added to the Sui chain adapter by manually calling verifyLedgerAppOpen in multiple methods (getAddress, signTransaction, signAndBroadcastTransaction).

For chain adapters that extend EvmBaseAdapter, this creates code duplication. We should leverage OOP principles to move the Ledger app gating logic to the base adapter implementation.

Objective

Move Ledger app-gating logic to EvmBaseAdapter so that:

• Child adapters automatically inherit the gating behavior
• Child adapters can extend with their own logic as needed
• Code duplication is eliminated across second-class EVM chain adapters

Proposed Changes

1. Add verifyLedgerAppOpen calls to the base implementation in EvmBaseAdapter for:

   • getAddress
   • signTransaction
   • signAndBroadcastTransaction

2. Remove duplicate verifyLedgerAppOpen calls from child adapter implementations that extend EvmBaseAdapter

3. Ensure child adapters can still extend with their own logic when needed

Effort Estimate

Approximately 1 hour

References

• PR feat: add Ledger app gate to Sui chain adapter #11450: feat: add Ledger app gate to Sui chain adapter #11450
• Comment: feat: add Ledger app gate to Sui chain adapter #11450 (comment)

Requested by: @gomesalexandre

[gomesalexandre]

Took a look at this and probably not worth the high refactor effort for something a gotcha in Claude skill solves. What a world we live in where duplication kills the need for abstractions.