review revisions

crates/sui-json-rpc-types/src/sui_transaction.rs

@@ -684,14 +684,14 @@ impl TryFrom<TransactionEffects> for SuiTransactionBlockEffects {
 
 fn owned_objref_string(obj: &OwnedObjectRef) -> String {
     format!(
-        " ┌──\n | ID: {} \n | Owner: {} \n | Version: {} \n | Digest: {}\n └──",
+        " ┌──\n │ ID: {} \n │ Owner: {} \n │ Version: {:?} \n │ Digest: {}\n └──",
         obj.reference.object_id, obj.owner, obj.reference.version, obj.reference.digest
     )
 }
 
 fn objref_string(obj: &SuiObjectRef) -> String {
     format!(
-        " ┌──\n | ID: {} \n | Version: {} \n | Digest: {}\n └──",
+        " ┌──\n │ ID: {} \n │ Version: {:?} \n │ Digest: {}\n └──",
         obj.object_id, obj.version, obj.digest
     )
 }

crates/sui-replay/src/lib.rs

@@ -364,7 +364,7 @@ pub async fn execute_replay_command(
                 println!("{:#?}", sandbox_state.pre_exec_diag);
             }
             if show_effects {
-                println!("{}", sandbox_state.local_exec_effects.to_string());
+                println!("{}", sandbox_state.local_exec_effects);
             }
 
             sandbox_state.check_effects()?;

crates/sui/src/client_commands.rs

@@ -613,38 +613,34 @@ pub enum SuiClientCommands {
     /// Replay a given transaction to view transaction effects. Set environment variable MOVE_VM_STEP=1 to debug.
     #[clap(name = "replay-transaction")]
     ReplayTransaction {
-        /// (Optional) The rpc url for a fullnode to use for fetching the transaction dependencies
-        #[arg(long = "rpc")]
-        rpc_url: Option<String>,
-
         /// The digest of the transaction to replay
         #[arg(long, short)]
         tx_digest: String,
-    },
 
-    /// Replay transactions listed in a file.
-    #[clap(name = "replay-batch")]
-    ReplayBatch {
         /// (Optional) The rpc url for a fullnode to use for fetching the transaction dependencies
         #[arg(long = "rpc")]
         rpc_url: Option<String>,
+    },
 
+    /// Replay transactions listed in a file.
+    #[clap(name = "replay-batch")]
+    ReplayBatch {
         /// The path to the file of transaction digests to replay, with one digest per line
         #[arg(long, short)]
         path: PathBuf,
 
         /// If an error is encountered during a transaction, this specifies whether to terminate or continue
         #[arg(long, short)]
         terminate_early: bool,
-    },
 
-    /// Replay all transactions in a range of checkpoints.
-    #[command(name = "replay-checkpoint")]
-    ReplayCheckpoints {
         /// (Optional) The rpc url for a fullnode to use for fetching the transaction dependencies
         #[arg(long = "rpc")]
         rpc_url: Option<String>,
+    },
 
+    /// Replay all transactions in a range of checkpoints.
+    #[command(name = "replay-checkpoint")]
+    ReplayCheckpoints {
         /// The starting checkpoint sequence number of the range of checkpoints to replay
         #[arg(long, short)]
         start: u64,
@@ -656,6 +652,10 @@ pub enum SuiClientCommands {
         /// If an error is encountered during a transaction, this specifies whether to terminate or continue
         #[arg(long, short)]
         terminate_early: bool,
+
+        /// (Optional) The rpc url for a fullnode to use for fetching the transaction dependencies
+        #[arg(long = "rpc")]
+        rpc_url: Option<String>,
     },
 }