fix(sdk-coin-flrp): update fromAddresses in delegator tx

[joshisakshi]

TICKET: SC-5648

Summary:

• Change fromAddressesBytes from [userAddress] to sorted _fromAddresses

• Sort fromAddressesBytes before passing to FlareJS to match sorted UTXO addresses

• Fixes 'No addresses match UTXO owners' error in HSM firmware tests

• Clarify comments about reward distribution (determined by stake output addresses, not rewardAddresses parameter)"

• Change output threshold

  • Issue: FlareJS defaults change outputs to threshold=1 for multisig wallets
  • Impact: Any single compromised key could steal change funds
  • Fix: Apply threshold correction (same pattern as ExportInPTxBuilder)

• Add comprehensive test coverage (33 tests)

  • Core validation (nodeID, amount, duration, required fields)
  • BitGo multisig patterns (2-of-3, 2-of-2)
  • Multiple UTXOs, change outputs, round-trip serialization
  • Real-world scenarios (multiple/sequential delegations)

[Gautam2305]

In ExportInPTxBuilder , after the threshold fix, the code also reconstructs _utxos from the corrected transaction's inputs and builds proper credentials + addressMaps using createCredentialForUtxo / createAddressMapForUtxo with sigIndices.
Here, you're passing empty arrays [] for both credentials and addressMaps. Is this intentional? I assume the delegator tx builder handles credential creation differently (perhaps during signing), but wanted to confirm this doesn't break
signing for multisig wallets.

[joshisakshi]

Good catch , have addressed the issue