Merge pull request #135 from dev-five-git/chage-fk-issue

Chage fk issue

Author: owjs3901

.changepacks/changepack_log_2Lq3nw6Achu2Z4knMTxg6.json

@@ -0,0 +1 @@
+{"changes":{"crates/vespertide-cli/Cargo.toml":"Patch","crates/vespertide/Cargo.toml":"Patch","crates/vespertide-loader/Cargo.toml":"Patch","crates/vespertide-config/Cargo.toml":"Patch","crates/vespertide-core/Cargo.toml":"Patch","crates/vespertide-exporter/Cargo.toml":"Patch","crates/vespertide-naming/Cargo.toml":"Patch","crates/vespertide-macro/Cargo.toml":"Patch","crates/vespertide-planner/Cargo.toml":"Patch","crates/vespertide-query/Cargo.toml":"Patch"},"note":"Fix change fk issue","date":"2026-04-14T04:54:12.310894100Z"}

.gitignore

@@ -4,3 +4,4 @@ settings.local.json
 coverage
 lcov.info
 .sisyphus
+.omc

Cargo.lock

@@ -184,6 +184,19 @@ fn format_action(action: &MigrationAction) -> String {
                 table.bright_cyan()
             )
         }
+        MigrationAction::ReplaceConstraint {
+            table, from, to, ..
+        } => {
+            format!(
+                "{} {} {} {} {} {}",
+                "Replace constraint:".bright_yellow(),
+                format_constraint_type(from).bright_cyan().bold(),
+                "->".bright_white(),
+                format_constraint_type(to).bright_cyan().bold(),
+                "on".bright_white(),
+                table.bright_cyan()
+            )
+        }
     }
 }
 
@@ -234,7 +247,9 @@ mod tests {
     use std::path::PathBuf;
     use tempfile::tempdir;
     use vespertide_config::VespertideConfig;
-    use vespertide_core::{ColumnDef, ColumnType, SimpleColumnType, TableConstraint, TableDef};
+    use vespertide_core::{
+        ColumnDef, ColumnType, ReferenceAction, SimpleColumnType, TableConstraint, TableDef,
+    };
 
     struct CwdGuard {
         original: PathBuf,
@@ -565,6 +580,28 @@ mod tests {
             "This is a very long comment...".bright_cyan().bold()
         )
     )]
+    #[case(
+        MigrationAction::ReplaceConstraint {
+            table: "posts".into(),
+            from: vespertide_core::TableConstraint::ForeignKey {
+                name: Some("fk_user".into()),
+                columns: vec!["user_id".into()],
+                ref_table: "users".into(),
+                ref_columns: vec!["id".into()],
+                on_delete: None,
+                on_update: None,
+            },
+            to: vespertide_core::TableConstraint::ForeignKey {
+                name: Some("fk_user".into()),
+                columns: vec!["user_id".into()],
+                ref_table: "users".into(),
+                ref_columns: vec!["id".into()],
+                on_delete: Some(ReferenceAction::Cascade),
+                on_update: None,
+            },
+        },
+        format!("{} {} {} {} {} {}", "Replace constraint:".bright_yellow(), "fk_user FK (user_id) -> users".bright_cyan().bold(), "->".bright_white(), "fk_user FK (user_id) -> users".bright_cyan().bold(), "on".bright_white(), "posts".bright_cyan())
+    )]
     #[serial]
     fn format_action_cases(#[case] action: MigrationAction, #[case] expected: String) {
         assert_eq!(format_action(&action), expected);

crates/vespertide-cli/src/commands/diff.rs

@@ -602,7 +602,8 @@ fn rewrite_plan_for_recreation(
             | MigrationAction::ModifyColumnDefault { table, .. }
             | MigrationAction::ModifyColumnComment { table, .. }
             | MigrationAction::AddConstraint { table, .. }
-            | MigrationAction::RemoveConstraint { table, .. } => Some(table.as_str()),
+            | MigrationAction::RemoveConstraint { table, .. }
+            | MigrationAction::ReplaceConstraint { table, .. } => Some(table.as_str()),
             _ => None,
         };
         table.is_none_or(|t| !tables_to_recreate.contains(t))