Upgrade wallet:multisig:sign to use ui.ledger (#5489)

* Upgrade wallet:multisig:sign to use ui.ledger This uses the new ledger UI action so that it will wait for the user to connect and unlock their ledger. * Update ironfish-cli/src/commands/wallet/multisig/sign.ts Co-authored-by: Hugh Cunningham <[email protected]> --------- Co-authored-by: Hugh Cunningham <[email protected]>
iron-fish · Oct 7, 2024 · d8c9965 · d8c9965
1 parent 72ac760
commit d8c9965
Showing 1 changed file with 21 additions and 16 deletions.
diff --git a/ironfish-cli/src/commands/wallet/multisig/sign.ts b/ironfish-cli/src/commands/wallet/multisig/sign.ts
@@ -73,15 +73,6 @@ export class SignMultisigTransactionCommand extends IronfishCommand {
 
     if (flags.ledger) {
       ledger = new LedgerMultiSigner(this.logger)
-      try {
-        await ledger.connect()
-      } catch (e) {
-        if (e instanceof Error) {
-          this.error(e.message)
-        } else {
-          throw e
-        }
-      }
     }
 
     let multisigAccountName: string
@@ -387,11 +378,16 @@ export class SignMultisigTransactionCommand extends IronfishCommand {
     const signingPackage = new multisig.SigningPackage(Buffer.from(signingPackageString, 'hex'))
 
     if (ledger) {
-      const frostSignatureShare = await ledger.dkgSign(
-        unsignedTransaction.publicKeyRandomness(),
-        signingPackage.frostSigningPackage().toString('hex'),
-        unsignedTransaction.hash().toString('hex'),
-      )
+      const frostSignatureShare = await ui.ledger({
+        ledger,
+        message: 'Sign Transaction',
+        action: () =>
+          ledger.dkgSign(
+            unsignedTransaction.publicKeyRandomness(),
+            signingPackage.frostSigningPackage().toString('hex'),
+            unsignedTransaction.hash().toString('hex'),
+          ),
+      })
 
       signatureShare = multisig.SignatureShare.fromFrost(
         frostSignatureShare,
@@ -515,7 +511,12 @@ export class SignMultisigTransactionCommand extends IronfishCommand {
 
     let commitment
     if (ledger) {
-      await ledger.reviewTransaction(unsignedTransactionHex)
+      await ui.ledger({
+        ledger,
+        message: 'Review Transaction',
+        action: () => ledger.reviewTransaction(unsignedTransactionHex),
+        approval: true,
+      })
 
       commitment = await this.createSigningCommitmentWithLedger(
         ledger,
@@ -545,7 +546,11 @@ export class SignMultisigTransactionCommand extends IronfishCommand {
     transactionHash: Buffer,
     signers: string[],
   ): Promise<string> {
-    const rawCommitments = await ledger.dkgGetCommitments(transactionHash.toString('hex'))
+    const rawCommitments = await ui.ledger({
+      ledger,
+      message: 'Get Commitments',
+      action: () => ledger.dkgGetCommitments(transactionHash.toString('hex')),
+    })
 
     const sigingCommitment = multisig.SigningCommitment.fromRaw(
       participant.identity,