Merge pull request #389 from sourabhxyz/388-dont-require-collateral-w…

…hen-simple-script

feat(#388): require collateral only for plutus scripts & ignore for n…

.changeset/two-shrimps-refuse.md

@@ -0,0 +1,5 @@
+---
+"@lucid-evolution/lucid": patch
+---
+
+add collateral only when Script is not NativeScript

packages/lucid/src/tx-builder/internal/CompleteTxBuilder.ts

@@ -118,9 +118,11 @@ export const complete = (
 
     // Execute programs sequentially
     yield* Effect.all(config.programs);
-    const hasScriptExecutions: boolean = config.scripts.size > 0;
+    const hasPlutusScriptExecutions: boolean = Array.from(
+      config.scripts.values(),
+    ).some((value) => value.type !== "Native");
     // Set collateral input if there are script executions
-    if (hasScriptExecutions) {
+    if (hasPlutusScriptExecutions) {
       const collateralInput = yield* findCollateral(
         config.lucidConfig.protocolParameters.coinsPerUtxoByte,
         setCollateral,
@@ -141,7 +143,7 @@ export const complete = (
     // Second round of coin selection by including script execution costs in fee estimation.
     // UPLC evaluation need to be performed again if new inputs are selected during coin selection.
     // Because increasing the inputs can increase the script execution budgets.
-    if (hasScriptExecutions)
+    if (hasPlutusScriptExecutions)
       yield* selectionAndEvaluation(
         config,
         walletInputs,