fix: copilot suggestions

Author: Luisfc68

contracts/PegOutContract.sol

@@ -348,7 +348,7 @@ contract PegOutContract is
             requiredAmount = quote.value - (quote.value % _SAT_TO_WEI_CONVERSION);
         }
         uint256 paidAmount = outputs[_PAY_TO_ADDRESS_OUTPUT].value * _SAT_TO_WEI_CONVERSION;
-        if (paidAmount < requiredAmount) revert InsufficientAmount(requiredAmount, paidAmount);
+        if (paidAmount < requiredAmount) revert InsufficientAmount(paidAmount, requiredAmount);
     }
 
     /// @notice This function is used to validate the null data of the Bitcoin transaction. The null data
@@ -363,11 +363,10 @@ contract PegOutContract is
             revert MalformedTransaction(scriptContent);
         }
 
-        // shift the array to remove the first byte (the size)
-        for (uint8 i = 0 ; i < scriptLength - 1; ++i) {
-            scriptContent[i] = scriptContent[i + 1];
+        bytes32 txQuoteHash;
+        assembly {
+            txQuoteHash := mload(add(scriptContent, 33)) // 32 bytes after the first byte
         }
-        bytes32 txQuoteHash = abi.decode(scriptContent, (bytes32));
         if (quoteHash != txQuoteHash) revert InvalidQuoteHash(quoteHash, txQuoteHash);
     }
 }

contracts/test-contracts/PegOutChangeReceiver.sol

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity 0.8.25;
 
-import {IPegOut} from "../interfaces/PegOut.sol";
+import {IPegOut} from "../interfaces/IPegOut.sol";
 import {Quotes} from "../libraries/Quotes.sol";
 
 

test/pegout/lp-refund.test.ts

@@ -131,7 +131,7 @@ describe("PegOutContract refundPegOut function should", () => {
     const otherQuote = getTestPegoutQuote({
       lbcAddress: await contract.getAddress(),
       liquidityProvider: usedLp,
-      refundAddress: quote.rskRefundAddress.toString(), // eslint-disable-line @typescript-eslint/no-base-to-string
+      refundAddress: String(quote.rskRefundAddress), // eslint-disable-line @typescript-eslint/no-base-to-string
       value: ethers.parseEther("1"),
     });
     const otherQuoteHash = await contract.hashPegOutQuote(otherQuote);
@@ -278,7 +278,7 @@ describe("PegOutContract refundPegOut function should", () => {
             )
         )
           .to.be.revertedWithCustomError(contract, "InsufficientAmount")
-          .withArgs(quote.value, satToWei(satAmount));
+          .withArgs(satToWei(satAmount), quote.value);
       });
     }
   );

test/pegout/user-refund.test.ts

@@ -195,9 +195,6 @@ describe("PegOutContract refundUserPegOut function should", () => {
 
     const tx = contract.connect(user).refundUserPegOut(getBytes(quoteHash));
 
-    await expect(tx)
-      .to.emit(contract, "PegOutUserRefunded")
-      .withArgs(getBytes(quoteHash), quote.rskRefundAddress, totalQuoteValue);
     await expect(tx)
       .to.emit(contract, "PegOutUserRefunded")
       .withArgs(getBytes(quoteHash), quote.rskRefundAddress, totalQuoteValue);