chore: remove hardware wallet dead code

[mathieuartu]

Description

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1496

Manual testing steps

no manual testing steps

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I've included tests if applicable
• I've documented my code using JSDoc format if applicable
• I've applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Low Risk
Mostly deletes unused Ledger Connect/BLE scanning UI and related hooks/tests; functional change is limited to how Ledger device model is sourced when forgetting a device, which could affect metrics labeling but not core wallet logic.

Overview
Removes the legacy Ledger BLE connection/scan flow UI (LedgerConnect, Scan, LedgerConnectionError) along with associated hooks (useBluetooth, useBluetoothDevices, useLedgerBluetooth, useLedgerDeviceForAccount) and their unit/snapshot tests.

Updates AccountActions hardware-account removal to stop relying on useLedgerDeviceForAccount; when a Ledger keyring becomes empty it now calls getDeviceId() from core/Ledger/Ledger before forgetLedger() to populate the HARDWARE_WALLET_FORGOTTEN metric, and adjusts mocks accordingly.

^{Written by Cursor Bugbot for commit 8bcc324. This will update automatically on new commits. Configure here.}

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[gantunesr]

lgtm but the CI is failing

[montelaidev]

Tested adding accounts and signing with ledger.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

[github-actions]

🔍 Smart E2E Test Selection

• Selected E2E tags: SmokeAccounts
• Selected Performance tags: None (no tests recommended)
• Risk Level: medium
• AI Confidence: 82%

click to see 🤖 AI reasoning details

E2E Test Selection:
This PR is a refactoring/cleanup of Ledger hardware wallet Bluetooth connection infrastructure:

1. Deleted components/hooks: The entire LedgerConnect/ directory (index.tsx, Scan.tsx, LedgerConnectionError.tsx, index.styles.ts) and several Ledger Bluetooth hooks (useBluetooth.ts, useBluetoothDevices.ts, useLedgerBluetooth.ts, useLedgerDeviceForAccount.ts, loadBleTransport.ts) have been completely removed.

2. Modified AccountActions.tsx: The useLedgerDeviceForAccount hook was removed and replaced with a direct getDeviceId() async call in the "forget Ledger device" flow. This is the key behavioral change - instead of scanning for BLE devices to get the device ID, it now calls getDeviceId() directly from the Ledger core module.

3. Impact: The changes affect the Ledger hardware wallet account management flow, specifically:

   • The "forget device" action in AccountActions (when removing a Ledger hardware wallet account)
   • The Ledger BLE connection UI (which appears to have been moved/replaced elsewhere)

4. SmokeAccounts is the appropriate tag because it covers hardware wallet account management, account actions (including the AccountActions component that was modified), and account management workflows. The AccountActions component is directly modified in this PR.

5. No other tags are needed - this is purely a Ledger hardware wallet refactoring with no impact on confirmations, network management, trading, identity sync, or other feature areas.

6. No performance tests needed - these are UI/hook refactoring changes with no performance-sensitive code paths (no list rendering, no data loading changes, no startup/initialization changes).

Performance Test Selection:
The changes are a refactoring of Ledger Bluetooth connection hooks and UI components. No performance-sensitive code paths are affected - no list rendering, no data loading, no state management changes, no startup/initialization changes. The AccountActions change is a simple replacement of a hook with a direct async function call.

View GitHub Actions results

[github-actions]

✅ E2E Fixture Validation — Schema is up to date
17 value mismatches detected (expected — fixture represents an existing user).
View details

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud