Merge branch '118-min-satoshi' into 'dev'

Resolve "set minimum satoshi"

Closes #118

See merge request ergo/rosen-bridge/rosen-chains!142

.changeset/gold-fans-breathe.md

@@ -0,0 +1,5 @@
+---
+'@rosen-chains/bitcoin': minor
+---
+
+remove legacy and script address payment restrictions in BitcoinChain

.changeset/quiet-snakes-crash.md

@@ -0,0 +1,5 @@
+---
+'@rosen-chains/bitcoin': minor
+---
+
+set minimum satoshi to 546

packages/chains/bitcoin/lib/BitcoinChain.ts

@@ -167,8 +167,14 @@ class BitcoinChain extends AbstractUtxoChain<BitcoinTx, BitcoinUtxo> {
       if (order.assets.tokens.length) {
         throw Error('Bitcoin does not support tokens in payment order');
       }
-      if (order.address.slice(0, 4) !== 'bc1q') {
-        throw Error('Bitcoin does not support payment to non-segwit addresses');
+      if (
+        order.address.slice(0, 4) !== 'bc1q' &&
+        order.address[0] !== '1' &&
+        order.address[0] !== '3'
+      ) {
+        throw Error(
+          'Bitcoin supports payments only to native-segwit, legacy or script addresses'
+        );
       }
       const orderBtc = this.unwrapBtc(order.assets.nativeToken).amount;
 
@@ -492,8 +498,7 @@ class BitcoinChain extends AbstractUtxoChain<BitcoinTx, BitcoinUtxo> {
    * @returns the minimum amount
    */
   getMinimumNativeToken = (): bigint => {
-    // there is no token in bitcoin
-    return 0n;
+    return 546n; // smallest non-dust value
   };
 
   /**
@@ -646,8 +651,11 @@ class BitcoinChain extends AbstractUtxoChain<BitcoinTx, BitcoinUtxo> {
    */
   minimumMeaningfulSatoshi = (feeRatio: number): bigint => {
     return BigInt(
-      Math.ceil(
-        (feeRatio * SEGWIT_INPUT_WEIGHT_UNIT) / 4 // estimate fee per weight and convert to virtual size
+      Math.max(
+        Math.ceil(
+          (feeRatio * SEGWIT_INPUT_WEIGHT_UNIT) / 4 // estimate fee per weight and convert to virtual size
+        ),
+        Number(this.getMinimumNativeToken())
       )
     );
   };

packages/chains/bitcoin/tests/BitcoinChain.spec.ts

@@ -87,9 +87,8 @@ describe('BitcoinChain', () => {
       const expectedRequiredAssets = structuredClone(
         testData.transaction2Order[0].assets
       );
-      expectedRequiredAssets.nativeToken += BigInt(
-        Math.ceil(SEGWIT_INPUT_WEIGHT_UNIT / 4)
-      );
+      expectedRequiredAssets.nativeToken +=
+        bitcoinChain.getMinimumNativeToken();
       expect(getCovBoxesSpy).toHaveBeenCalledWith(
         testUtils.configs.addresses.lock,
         expectedRequiredAssets,