pr feedback

Author: DatGuyJonathan

apps/framework-cli/src/cli.rs

@@ -253,7 +253,13 @@ fn override_project_config_from_url(
         )
     })?;
 
+    let clusters = project.clickhouse_config.clusters.clone();
+    let additional_databases = project.clickhouse_config.additional_databases.clone();
+
     project.clickhouse_config = clickhouse_config;
+    project.clickhouse_config.clusters = clusters;
+    project.clickhouse_config.additional_databases = additional_databases;
+
     info!(
         "Overriding project ClickHouse config from CLI: database = {}",
         project.clickhouse_config.db_name

apps/framework-cli/src/cli/routines.rs

@@ -1170,8 +1170,16 @@ pub async fn remote_gen_migration(
         },
     );
 
+    // Validate the plan before generating migration files
+    let plan = InfraPlan {
+        target_infra_map: local_infra_map.clone(),
+        changes,
+    };
+
+    plan_validator::validate(project, &plan)?;
+
     let db_migration =
-        MigrationPlan::from_infra_plan(&changes, &project.clickhouse_config.db_name)?;
+        MigrationPlan::from_infra_plan(&plan.changes, &project.clickhouse_config.db_name)?;
 
     Ok(MigrationPlanWithBeforeAfter {
         remote_state: remote_infra_map,