fix(abstract-utxo): normalize address comparison in parseOutput

When comparing addresses, normalize them to a standard script format to ensure
accurate matching regardless of address representation.

We can now pass `txParams.txRecipients` to `parseOutputs`. It previously passed
`expectedOutputs`, which did multiple things at once:

- Normalize the address format for comparison (which is what we wanted)
- Append the changeAddress in certain cases (which is not desired in
  parseOutput)

This allows us to change the `toExpectedOutputs` function.

Issue: BTC-2962

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

Author: OttoAllmendinger

modules/abstract-utxo/src/transaction/fixedScript/parseOutput.ts

@@ -14,6 +14,7 @@ import {
 
 import { AbstractUtxoCoin } from '../../abstractUtxoCoin';
 import { Output, FixedScriptWalletOutput } from '../types';
+import { fromExtendedAddressFormatToScript } from '../recipient';
 
 const debug = debugLib('bitgo:v2:parseoutput');
 
@@ -284,7 +285,9 @@ export async function parseOutput({
      */
     if (txParams.recipients !== undefined && txParams.recipients.length > RECIPIENT_THRESHOLD) {
       const isCurrentAddressInRecipients = txParams.recipients.some((recipient) =>
-        recipient.address.includes(currentAddress)
+        fromExtendedAddressFormatToScript(recipient.address, coin.name).equals(
+          fromExtendedAddressFormatToScript(currentAddress, coin.name)
+        )
       );
 
       if (isCurrentAddressInRecipients) {

modules/abstract-utxo/src/transaction/fixedScript/parseTransaction.ts

@@ -183,7 +183,7 @@ export async function parseTransaction<TNumber extends bigint | number>(
         keychainArray: toKeychainTriple(keychains),
         wallet,
         txParams: {
-          recipients: expectedOutputs,
+          recipients: txParams.recipients ?? [],
           changeAddress: txParams.changeAddress,
         },
         customChange,