feat: use wasm-utxo for address generation on testnets

Use wasm-utxo's fixedScriptWallet.address for testnets instead of
utxo-lib's address derivation. This improves compatibility with the
native implementation.

Co-authored-by: llm-git <llm-git@ttll.de>

Ticket: BTC-2720

TICKET: BTC-2720

Author: lcovar

modules/abstract-utxo/src/address/fixedScript.ts

@@ -14,6 +14,7 @@ import {
 import * as utxolib from '@bitgo/utxo-lib';
 import { bitgo } from '@bitgo/utxo-lib';
 import { bip32 } from '@bitgo/secp256k1';
+import * as wasmUtxo from '@bitgo/wasm-utxo';
 
 type ScriptType2Of3 = bitgo.outputScripts.ScriptType2Of3;
 
@@ -56,6 +57,15 @@ export function generateAddressWithChainAndIndex(
   index: number,
   format: CreateAddressFormat | undefined
 ): string {
+  if (utxolib.isTestnet(network)) {
+    return wasmUtxo.fixedScriptWallet.address(
+      keychains.map((k) => k.pub) as [string, string, string],
+      chain,
+      index,
+      network
+    );
+  }
+
   const path = '0/0/' + chain + '/' + index;
   const hdNodes = keychains.map(({ pub }) => bip32.fromBase58(pub));
   const derivedKeys = hdNodes.map((hdNode) => hdNode.derivePath(sanitizeLegacyPath(path)).publicKey);

modules/abstract-utxo/test/unit/util/unspents.ts

@@ -1,5 +1,6 @@
 import * as utxolib from '@bitgo/utxo-lib';
 import { getSeed } from '@bitgo/sdk-test';
+import * as wasmUtxo from '@bitgo/wasm-utxo';
 
 import { getReplayProtectionAddresses } from '../../../src';
 
@@ -31,6 +32,9 @@ export function getWalletAddress(
   chain = defaultChain,
   index = 0
 ): string {
+  if (utxolib.isTestnet(network)) {
+    return wasmUtxo.fixedScriptWallet.address(walletKeys, chain, index, network);
+  }
   return utxolib.address.fromOutputScript(getOutputScript(walletKeys, chain, index).scriptPubKey, network);
 }
 
@@ -49,8 +53,7 @@ export function mockWalletUnspent<TNumber extends number | bigint = number>(
   if (chain === undefined) {
     throw new Error(`unspent chain must be set`);
   }
-  const derived = getOutputScript(walletKeys, chain, index);
-  const deriveAddress = utxolib.address.fromOutputScript(derived.scriptPubKey, network);
+  const deriveAddress = getWalletAddress(network, walletKeys, chain, index);
   if (address) {
     if (address !== deriveAddress) {
       throw new Error(`derivedAddress mismatch: ${address} derived=${deriveAddress}`);

modules/utxo-lib/package.json

@@ -48,6 +48,7 @@
   "dependencies": {
     "@bitgo/blake2b": "^3.2.4",
     "@bitgo/secp256k1": "^1.7.0",
+    "@bitgo/wasm-utxo": "1.1.0",
     "@brandonblack/musig": "^0.0.1-alpha.0",
     "bech32": "^2.0.0",
     "bip174": "npm:@bitgo-forks/bip174@3.1.0-master.4",

modules/utxo-lib/src/bitgo/wallet/WalletScripts.ts

@@ -1,7 +1,8 @@
-import { Network, address } from '../..';
+import { Network, address, isTestnet } from '../..';
 import { ChainCode, outputScripts } from '..';
 import { RootWalletKeys } from './WalletKeys';
 import { scriptTypeForChain, SpendableScript } from '../outputScripts';
+import * as wasmUtxo from '@bitgo/wasm-utxo';
 
 export function getWalletOutputScripts(keys: RootWalletKeys, chain: ChainCode, index: number): SpendableScript {
   return outputScripts.createOutputScript2of3(
@@ -11,5 +12,8 @@ export function getWalletOutputScripts(keys: RootWalletKeys, chain: ChainCode, i
 }
 
 export function getWalletAddress(keys: RootWalletKeys, chain: ChainCode, index: number, network: Network): string {
+  if (isTestnet(network)) {
+    return wasmUtxo.fixedScriptWallet.address(keys, chain, index, network);
+  }
   return address.fromOutputScript(getWalletOutputScripts(keys, chain, index).scriptPubKey, network);
 }

modules/utxo-lib/src/testutil/mock.ts

@@ -1,7 +1,7 @@
 import { ok as assert } from 'assert';
 import { BIP32Interface } from '@bitgo/secp256k1';
 import { payments } from '..';
-import { getMainnet, Network, networks } from '../networks';
+import { getMainnet, Network, networks, isTestnet } from '../networks';
 
 import {
   ChainCode,
@@ -22,6 +22,7 @@ import {
 } from '../bitgo';
 import { fromOutputScript } from '../address';
 import { createOutputScript2of3, createOutputScriptP2shP2pk } from '../bitgo/outputScripts';
+import * as wasmUtxo from '@bitgo/wasm-utxo';
 
 import { getDefaultWalletKeys, getKey } from './keys';
 
@@ -94,14 +95,17 @@ export function mockWalletUnspent<TNumber extends number | bigint>(
     id,
   }: { chain?: ChainCode; index?: number; keys?: RootWalletKeys; vout?: number; id?: string } = {}
 ): WalletUnspent<TNumber> | NonWitnessWalletUnspent<TNumber> {
-  const derivedKeys = keys.deriveForChainAndIndex(chain, index);
-  const address = fromOutputScript(
-    createOutputScript2of3(derivedKeys.publicKeys, scriptTypeForChain(chain)).scriptPubKey,
-    network
-  );
+  const address = isTestnet(network)
+    ? wasmUtxo.fixedScriptWallet.address(keys, chain, index, network)
+    : fromOutputScript(
+        createOutputScript2of3(keys.deriveForChainAndIndex(chain, index).publicKeys, scriptTypeForChain(chain))
+          .scriptPubKey,
+        network
+      );
   if (id && typeof id === 'string') {
     return { id, address, chain, index, value };
   } else {
+    const derivedKeys = keys.deriveForChainAndIndex(chain, index);
     const prevTransaction = mockPrevTx(
       vout,
       createOutputScript2of3(derivedKeys.publicKeys, scriptTypeForChain(chain), network).scriptPubKey,