Skip to content

Upgrade Ledger app to v1.3.7 with Nano S+ and Zemu 0.55.3 #2986

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 5, 2025
Merged

Upgrade Ledger app to v1.3.7 with Nano S+ and Zemu 0.55.3 #2986

merged 1 commit into from
Nov 5, 2025

Conversation

@felipemadero
Copy link
Collaborator

@felipemadero felipemadero commented Nov 5, 2025 •
edited
Loading

Why this should be merged

This PR upgrades the test infrastructure to use the latest Ledger app version (v1.3.7) with Nano S+ device support.
It also fixes logic that caused gRPC timeout errors in e2e tests.

How this works

Keychain Optimization:

  • Changed getLedgerIndices() to use PubKeys() instead of looping PubKey()
  • Changed searchForFundedLedgerIndices() similarly

Ledger Infrastructure Updates:

  • Upgraded Avalanche Ledger app from v0.7.2 to v1.3.7
  • Replaced app_s.elf (Nano S) with app_s2.elf (Nano S+, 381KB)
  • Upgraded Zemu simulator from 0.36.0 to 0.55.3
  • Updated to Docker image: zondax/builder-zemu:speculos-d076634af0e38fb24844ed85417286c50441d494
  • Updated Zemu API usage for 0.55.3 compatibility

How this was tested

  • Ledger E2E

@felipemadero
Upgrade Ledger app to v1.3.7 with Nano S+ and Zemu 0.55.3
de32646 
Update test infrastructure to support newer Ledger hardware and
simulator, fixing ledger SDK migration issues in the process.
@felipemadero felipemadero requested a review from a team as a code owner November 5, 2025 00:42
@sukantoraymond sukantoraymond merged commit 2a32985 into main Nov 5, 2025
121 of 122 checks passed
@sukantoraymond sukantoraymond deleted the fix-ledger-sdk-migration branch November 5, 2025 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sukantoraymond sukantoraymond sukantoraymond approved these changes

@jasonatran jasonatran Awaiting requested review from jasonatran jasonatran is a code owner automatically assigned from ava-labs/tooling

@artemanosov artemanosov Awaiting requested review from artemanosov artemanosov is a code owner automatically assigned from ava-labs/tooling

@meaghanfitzgerald meaghanfitzgerald Awaiting requested review from meaghanfitzgerald meaghanfitzgerald is a code owner automatically assigned from ava-labs/tooling

@emilypi-avalabs emilypi-avalabs Awaiting requested review from emilypi-avalabs emilypi-avalabs is a code owner automatically assigned from ava-labs/tooling

Assignees

@felipemadero felipemadero

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@felipemadero @sukantoraymond