fix(utxo-lib): update assert imports for Node 22

Replace "*" imports with specific imports from
assert module. When upgrading the test runner
from ts-node to tsx, we need to fix imports to
be strictly ESM to avoid:
"TypeError: assert is not a function"

TICKET: WP-5599

TICKET: WP-5599

Author: zahin-mohammad

modules/utxo-lib/src/bitgo/UtxoPsbt.ts

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import { Psbt as PsbtBase } from 'bip174';
 import {
   Bip32Derivation,

modules/utxo-lib/src/bitgo/UtxoTransaction.ts

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import * as bitcoinjs from 'bitcoinjs-lib';
 import * as varuint from 'varuint-bitcoin';
 import { toTNumber } from './tnumber';

modules/utxo-lib/src/bitgo/legacysafe/index.ts

@@ -58,11 +58,11 @@ export function createLegacySafeOutputScript2of3(
   });
 
   const script2of3 = bitcoinjs.payments.p2ms({ m: 2, pubkeys });
-  assert(script2of3.output);
+  assert.ok(script2of3.output);
 
   const scriptPubKey = bitcoinjs.payments.p2sh({ redeem: script2of3 });
-  assert(scriptPubKey);
-  assert(scriptPubKey.output);
+  assert.ok(scriptPubKey);
+  assert.ok(scriptPubKey.output);
 
   return {
     scriptPubKey: scriptPubKey.output,

modules/utxo-lib/src/bitgo/nonStandardHalfSigned.ts

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import * as opcodes from 'bitcoin-ops';
 import { classify, script as bscript, TxInput } from '../';
 

modules/utxo-lib/src/bitgo/outputScripts.ts

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import * as bitcoinjs from 'bitcoinjs-lib';
 
 import { Network, supportsSegwit, supportsTaproot } from '../networks';

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

@@ -463,7 +463,7 @@ export function isPsbtInputArray(inputs: PsbtInput[] | TxInput[]): inputs is Psb
  * @return true iff inputs array is of TxInput type
  * */
 export function isTxInputArray(inputs: PsbtInput[] | TxInput[]): inputs is TxInput[] {
-  assert(!!inputs.length, 'empty inputs array');
+  assert.ok(!!inputs.length, 'empty inputs array');
   return 'hash' in inputs[0];
 }
 
@@ -532,7 +532,7 @@ export function getSignatureValidationArrayPsbt(psbt: UtxoPsbt, rootWalletKeys:
  * The purpose is to provide backward compatibility to keyternal (KRS) that only supports network transaction and p2ms script types.
  */
 export function extractP2msOnlyHalfSignedTx(psbt: UtxoPsbt): UtxoTransaction<bigint> {
-  assert(!!(psbt.data.inputs.length && psbt.data.outputs.length), 'empty inputs or outputs');
+  assert.ok(!!(psbt.data.inputs.length && psbt.data.outputs.length), 'empty inputs or outputs');
   const tx = psbt.getUnsignedTx();
 
   function isP2msParsedPsbtInput(
@@ -543,10 +543,10 @@ export function extractP2msOnlyHalfSignedTx(psbt: UtxoPsbt): UtxoTransaction<big
 
   psbt.data.inputs.forEach((input, i) => {
     const parsed = parsePsbtInput(input);
-    assert(isP2msParsedPsbtInput(parsed), `unsupported script type ${parsed.scriptType}`);
-    assert(input.partialSig?.length === 1, `unexpected signature count ${input.partialSig?.length}`);
+    assert.ok(isP2msParsedPsbtInput(parsed), `unsupported script type ${parsed.scriptType}`);
+    assert.ok(input.partialSig?.length === 1, `unexpected signature count ${input.partialSig?.length}`);
     const [partialSig] = input.partialSig;
-    assert(
+    assert.ok(
       input.sighashType !== undefined && input.sighashType === bscript.signature.decode(partialSig.signature).hashType,
       'signature sighash does not match input sighash type'
     );

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

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 
 import { Payment, taproot } from 'bitcoinjs-lib';
 import { PsbtOutput, PsbtOutputUpdate } from 'bip174/src/lib/interfaces';

modules/utxo-lib/src/bitgo/wallet/psbt/RootNodes.ts

@@ -2,7 +2,7 @@
  * Contains helper methods for getting and sorting root nodes from a PSBT.
  */
 
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import * as bs58check from 'bs58check';
 
 import { UtxoPsbt } from '../../UtxoPsbt';

modules/utxo-lib/src/noble_ecc.ts

@@ -1,12 +1,12 @@
-import * as createHash from 'create-hash';
-import * as createHmac from 'create-hmac';
 import { ECPairAPI, ECPairFactory, ECPairInterface } from 'ecpair';
 import * as necc from '@noble/secp256k1';
 import { BIP32API, BIP32Factory, BIP32Interface } from 'bip32';
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-ignore base_crypto is exported as a subPath export, ignoring since compiler complains about importing like this
 import * as baseCrypto from '@brandonblack/musig/base_crypto';
 import { MuSig, MuSigFactory } from '@brandonblack/musig';
+const createHmac = require('create-hmac');
+const createHash = require('create-hash');
 
 necc.utils.sha256Sync = (...messages: Uint8Array[]): Uint8Array => {
   const sha256 = createHash('sha256');

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

@@ -1,4 +1,4 @@
-import * as assert from 'assert';
+import { ok as assert } from 'assert';
 import { BIP32Interface } from 'bip32';
 import * as noble from '@noble/secp256k1';
 import * as utxolib from '..';