fix(master-express): forward walletPubs in consolidateUnspents and accelerate

[bitgo-ai-agent-dev]

Summary

• consolidateUnspents and accelerate were failing for UTXO coins (BTC/tBTC) with "pubs are required for fixed script signing" because makeCustomSigningFunction did not forward walletPubs to awmClient.signMultisig
• Both handlers now fetch all 3 keychains (user, backup, bitgo) in parallel and pass walletPubs into makeCustomSigningFunction, matching the fix already in place for sendMany
• makeCustomSigningFunction in utils.ts updated to accept and forward the optional walletPubs parameter

Root Cause

makeCustomSigningFunction only forwarded txPrebuild, source, and pub to the AWM client. Without walletPubs, coin.signTransaction() on the AWM side was called without pubs, hitting the assertion added in commit c25b99b.

Test Plan

• Two new tests added to consolidateUnspents.test.ts: verify walletPubs is passed to AWM sign request, and verify it is omitted when any keychain lacks a pub
• Two new tests added to accelerate.test.ts with identical coverage
• All existing tests for both endpoints remain unchanged and should continue to pass

Ticket: WCN-447

[linear-code]

WCN-447

[pranishnepal]

can we create a util func for this:

const [userKeychain, backupKeychain, bitgoKeychain] = await Promise.all([  
    baseCoin.keychains().get({ id: wallet.keyIds()[KeyIndices.USER] }),
    baseCoin.keychains().get({ id: wallet.keyIds()[KeyIndices.BACKUP] }),
    baseCoin.keychains().get({ id: wallet.keyIds()[KeyIndices.BITGO] }),
  ]);
  const walletPubs =
    userKeychain?.pub && backupKeychain?.pub && bitgoKeychain?.pub
      ? [userKeychain.pub, backupKeychain.pub, bitgoKeychain.pub]
      : undefined;

return walletPubs;

and use it in both the places so we can just edit things in one place later?

[pranishnepal]

should we do the same at handleConsolidate or no?

advanced-wallets/src/masterBitgoExpress/handlers/handleConsolidate.ts

Lines 85 to 89 in dcee4fc

	customSigningFunction: makeCustomSigningFunction({
	awmClient,
	source: params.source,
	pub: signingKeychain.pub!,
	}),

[pranishnepal]

Testing

After applying the change, tested consolidation -- reference