wip

Author: howjmay

bindings/go/examples/prepare_split_coins/main.go

@@ -22,7 +22,6 @@ func main() {
 		sender,
 		[]*sdk.PtbArgument{sdk.PtbArgumentRes("coin1"), sdk.PtbArgumentRes("coin2"), sdk.PtbArgumentRes("coin3")},
 	)
-	builder.Gas(coinObjId).GasBudget(1000000000)
 
 	txn, err := builder.Finish()
 	if err.(*sdk.SdkFfiError) != nil {
@@ -36,14 +35,13 @@ func main() {
 	log.Printf("Signing Digest: %v", sdk.HexEncode(txn.SigningDigest()))
 	log.Printf("Txn Bytes: %v", sdk.Base64Encode(txnBytes))
 
-	res, err := builder.DryRun(false)
+	skipChecks := bool(false)
+	res, err := client.DryRunTx(txn, &skipChecks)
 	if err.(*sdk.SdkFfiError) != nil {
-		log.Fatalf("Failed to split coins: %v", err)
+		log.Fatalf("Failed to dry run split coins: %v", err)
 	}
-
 	if res.Error != nil {
 		log.Fatalf("Failed to split coins: %v", *res.Error)
 	}
-
 	log.Print("Split coins dry run was successful!")
 }

crates/iota-sdk-types/src/effects/v1.rs

@@ -118,6 +118,8 @@ pub struct ChangedObject {
     /// Optional object type information. This is not part of the BCS protocol
     /// data but can be populated from other sources when available.
     #[cfg_attr(feature = "serde", serde(skip))]
+    #[cfg_attr(feature = "proptest", strategy(proptest::strategy::Just(None)))]
+    #[cfg_attr(feature = "schemars", schemars(skip))]
     pub object_type: Option<String>,
 }
 

crates/iota-transaction-builder/src/builder/mod.rs

@@ -4,7 +4,7 @@
 //! Builder for Programmable Transactions.
 
 use std::{
-    collections::{BTreeMap, HashMap},
+    collections::{BTreeMap, HashMap, HashSet},
     marker::PhantomData,
     time::Duration,
 };
@@ -730,7 +730,25 @@ impl<L> TransactionBuilder<Client, L> {
         let mut inputs = Vec::new();
         let mut gas = Vec::new();
         let mut input_map = HashMap::new();
+
         if default_gas && !self.data.inputs.values().any(|i| i.is_gas) {
+            let mut unusable_object_ids = HashSet::new();
+            for cmd in &self.data.commands {
+                for arg in match cmd {
+                    Command::MoveCall(MoveCall { arguments, .. }) => arguments.as_slice(),
+                    Command::TransferObjects(TransferObjects { objects, .. }) => objects.as_slice(),
+                    Command::MergeCoins(MergeCoins { coins_to_merge, .. }) => {
+                        coins_to_merge.as_slice()
+                    }
+                    _ => &[],
+                } {
+                    if let Argument::Input(idx) = arg {
+                        if let Some(obj_id) = self.data.inputs[idx].object_id() {
+                            unusable_object_ids.insert(*obj_id);
+                        }
+                    }
+                }
+            }
             for coin in self
                 .client
                 .objects(
@@ -745,10 +763,12 @@ impl<L> TransactionBuilder<Client, L> {
                 .map_err(Error::Client)?
                 .data
             {
-                self.set_input(
-                    InputKind::Input(iota_types::Input::ImmutableOrOwned(coin.object_ref())),
-                    true,
-                );
+                if !unusable_object_ids.contains(&coin.object_id()) {
+                    self.set_input(
+                        InputKind::Input(iota_types::Input::ImmutableOrOwned(coin.object_ref())),
+                        true,
+                    );
+                }
             }
         }
         for (id, input) in std::mem::take(&mut self.data.inputs) {