Merge pull request #228 from blocknative/fix/correct-derivation-paths

Make derivation paths match common implementations.

Closes #227

Author: lnbc1QWFyb24

src/modules/select/wallets/ledger.ts

@@ -148,12 +148,12 @@ async function ledgerProvider(options: {
 
       if (numberToGet > 1) {
         for (
-          let i = addressesAlreadyFetched;
+          let i = addressesAlreadyFetched > 1 ? addressesAlreadyFetched : 0;
           i < numberToGet + addressesAlreadyFetched;
           i++
         ) {
           const ledgerLive = `${basePath}/${i}'/0/0`
-          const legacy = `${basePath}/0'/${i}'`
+          const legacy = `${basePath}/0'/${i}`
 
           paths.push(ledgerLive, legacy)
         }

src/modules/select/wallets/trezor.ts

@@ -166,11 +166,11 @@ async function trezorProvider(options: {
     }
 
     for (
-      let i = addressesAlreadyFetched;
+      let i = addressesAlreadyFetched > 1 ? addressesAlreadyFetched : 0;
       i < accountsToGet + addressesAlreadyFetched;
       i++
     ) {
-      const path = `${basePath}/${i}'/0/0`
+      const path = `${basePath}/0'/0/${i}`
       bundle.push({ path, showOnTrezor: false })
     }