this commit;

- makes interaction plans serializable
- fixes the shadowing bug where non-created tables were assumed to be created in the shadow tables map
- makes small changes to make clippy happy
- reorganizes simulation running flow to remove unnecessary plan regenerations while shrinking and double checking

simulator/runner/cli.rs

@@ -51,6 +51,7 @@ impl SimulatorCLI {
         if self.maximum_size < 1 {
             return Err("maximum size must be at least 1".to_string());
         }
+        // todo: fix an issue here where if minimum size is not defined, it prevents setting low maximum sizes.
         if self.minimum_size > self.maximum_size {
             return Err("Minimum size cannot be greater than maximum size".to_string());
         }

simulator/runner/env.rs

@@ -8,6 +8,7 @@ use crate::model::table::Table;
 
 use crate::runner::io::SimulatorIO;
 
+#[derive(Clone)]
 pub(crate) struct SimulatorEnv {
     pub(crate) opts: SimulatorOpts,
     pub(crate) tables: Vec<Table>,

simulator/runner/execution.rs

@@ -55,13 +55,14 @@ impl ExecutionResult {
 }
 
 pub(crate) fn execute_plans(
-    env: &mut SimulatorEnv,
+    env: Arc<Mutex<SimulatorEnv>>,
     plans: &mut [InteractionPlan],
     states: &mut [InteractionPlanState],
     last_execution: Arc<Mutex<Execution>>,
 ) -> ExecutionResult {
     let mut history = ExecutionHistory::new();
     let now = std::time::Instant::now();
+    let mut env = env.lock().unwrap();
     for _tick in 0..env.opts.ticks {
         // Pick the connection to interact with
         let connection_index = pick_index(env.connections.len(), &mut env.rng);
@@ -77,7 +78,7 @@ pub(crate) fn execute_plans(
         last_execution.interaction_index = state.interaction_pointer;
         last_execution.secondary_index = state.secondary_pointer;
         // Execute the interaction for the selected connection
-        match execute_plan(env, connection_index, plans, states) {
+        match execute_plan(&mut env, connection_index, plans, states) {
             Ok(_) => {}
             Err(err) => {
                 return ExecutionResult::new(history, Some(err));