Fix lint

Author: owjs3901

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

@@ -30,9 +30,7 @@ fn parse_fill_with_args(args: &[String]) -> HashMap<(String, String), String> {
 
 /// Format the type info string for display.
 fn format_type_info(column_type: Option<&String>) -> String {
-    column_type
-        .map(|t| format!(" ({})", t))
-        .unwrap_or_default()
+    column_type.map(|t| format!(" ({})", t)).unwrap_or_default()
 }
 
 /// Format a single fill_with item for display.
@@ -757,12 +755,8 @@ mod tests {
 
     #[test]
     fn test_print_fill_with_item_and_get_prompt_no_type() {
-        let prompt = print_fill_with_item_and_get_prompt(
-            "orders",
-            "status",
-            None,
-            "ModifyColumnNullable",
-        );
+        let prompt =
+            print_fill_with_item_and_get_prompt("orders", "status", None, "ModifyColumnNullable");
         assert!(prompt.contains("Enter fill value for"));
         assert!(prompt.contains("orders"));
         assert!(prompt.contains("status"));
@@ -795,7 +789,8 @@ mod tests {
         let mut fill_values = HashMap::new();
 
         // Mock prompt function that returns a fixed value
-        let mock_prompt = |_prompt: &str| -> Result<String> { Ok("'test@example.com'".to_string()) };
+        let mock_prompt =
+            |_prompt: &str| -> Result<String> { Ok("'test@example.com'".to_string()) };
 
         let result = collect_fill_with_values(&missing, &mut fill_values, mock_prompt);
         assert!(result.is_ok());
@@ -932,7 +927,8 @@ mod tests {
         let mut fill_values = HashMap::new();
 
         // Mock prompt function
-        let mock_prompt = |_prompt: &str| -> Result<String> { Ok("'test@example.com'".to_string()) };
+        let mock_prompt =
+            |_prompt: &str| -> Result<String> { Ok("'test@example.com'".to_string()) };
 
         let result = handle_missing_fill_with(&mut plan, &mut fill_values, mock_prompt);
         assert!(result.is_ok());
@@ -1018,7 +1014,8 @@ mod tests {
         let mut fill_values = HashMap::new();
 
         // Mock prompt that returns an error
-        let mock_prompt = |_prompt: &str| -> Result<String> { Err(anyhow::anyhow!("user cancelled")) };
+        let mock_prompt =
+            |_prompt: &str| -> Result<String> { Err(anyhow::anyhow!("user cancelled")) };
 
         let result = handle_missing_fill_with(&mut plan, &mut fill_values, mock_prompt);
         assert!(result.is_err());

crates/vespertide-core/src/action.rs

@@ -108,15 +108,23 @@ impl fmt::Display for MigrationAction {
                 ..
             } => {
                 let nullability = if *nullable { "NULL" } else { "NOT NULL" };
-                write!(f, "ModifyColumnNullable: {}.{} -> {}", table, column, nullability)
+                write!(
+                    f,
+                    "ModifyColumnNullable: {}.{} -> {}",
+                    table, column, nullability
+                )
             }
             MigrationAction::ModifyColumnDefault {
                 table,
                 column,
                 new_default,
             } => {
                 if let Some(default) = new_default {
-                    write!(f, "ModifyColumnDefault: {}.{} -> {}", table, column, default)
+                    write!(
+                        f,
+                        "ModifyColumnDefault: {}.{} -> {}",
+                        table, column, default
+                    )
                 } else {
                     write!(f, "ModifyColumnDefault: {}.{} -> (none)", table, column)
                 }
@@ -132,7 +140,11 @@ impl fmt::Display for MigrationAction {
                     } else {
                         comment.clone()
                     };
-                    write!(f, "ModifyColumnComment: {}.{} -> '{}'", table, column, display)
+                    write!(
+                        f,
+                        "ModifyColumnComment: {}.{} -> '{}'",
+                        table, column, display
+                    )
                 } else {
                     write!(f, "ModifyColumnComment: {}.{} -> (none)", table, column)
                 }
@@ -537,10 +549,7 @@ mod tests {
         },
         "ModifyColumnDefault: users.status -> (none)"
     )]
-    fn test_display_modify_column_default(
-        #[case] action: MigrationAction,
-        #[case] expected: &str,
-    ) {
+    fn test_display_modify_column_default(#[case] action: MigrationAction, #[case] expected: &str) {
         assert_eq!(action.to_string(), expected);
     }
 
@@ -561,10 +570,7 @@ mod tests {
         },
         "ModifyColumnComment: users.email -> (none)"
     )]
-    fn test_display_modify_column_comment(
-        #[case] action: MigrationAction,
-        #[case] expected: &str,
-    ) {
+    fn test_display_modify_column_comment(#[case] action: MigrationAction, #[case] expected: &str) {
         assert_eq!(action.to_string(), expected);
     }
 

crates/vespertide-planner/src/diff.rs

@@ -1815,7 +1815,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string(), "tenant_id".to_string()]
                 )
             });
-            assert!(has_remove, "Should have RemoveConstraint for old composite PK");
+            assert!(
+                has_remove,
+                "Should have RemoveConstraint for old composite PK"
+            );
 
             let has_add = plan.actions.iter().any(|a| {
                 matches!(
@@ -1826,7 +1829,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string()]
                 )
             });
-            assert!(has_add, "Should have AddConstraint for new single-column PK");
+            assert!(
+                has_add,
+                "Should have AddConstraint for new single-column PK"
+            );
         }
 
         #[test]
@@ -1913,7 +1919,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string()]
                 )
             });
-            assert!(has_remove, "Should have RemoveConstraint for old single-column PK");
+            assert!(
+                has_remove,
+                "Should have RemoveConstraint for old single-column PK"
+            );
 
             let has_add = plan.actions.iter().any(|a| {
                 matches!(
@@ -1928,7 +1937,10 @@ mod tests {
                     ]
                 )
             });
-            assert!(has_add, "Should have AddConstraint for new 3-column composite PK");
+            assert!(
+                has_add,
+                "Should have AddConstraint for new 3-column composite PK"
+            );
         }
 
         #[test]
@@ -1971,7 +1983,10 @@ mod tests {
                     ]
                 )
             });
-            assert!(has_remove, "Should have RemoveConstraint for old 3-column composite PK");
+            assert!(
+                has_remove,
+                "Should have RemoveConstraint for old 3-column composite PK"
+            );
 
             let has_add = plan.actions.iter().any(|a| {
                 matches!(
@@ -1982,7 +1997,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string()]
                 )
             });
-            assert!(has_add, "Should have AddConstraint for new single-column PK");
+            assert!(
+                has_add,
+                "Should have AddConstraint for new single-column PK"
+            );
         }
 
         #[test]
@@ -2022,7 +2040,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string(), "tenant_id".to_string()]
                 )
             });
-            assert!(has_remove, "Should have RemoveConstraint for old PK with tenant_id");
+            assert!(
+                has_remove,
+                "Should have RemoveConstraint for old PK with tenant_id"
+            );
 
             let has_add = plan.actions.iter().any(|a| {
                 matches!(
@@ -2033,7 +2054,10 @@ mod tests {
                     } if table == "users" && columns == &vec!["id".to_string(), "region_id".to_string()]
                 )
             });
-            assert!(has_add, "Should have AddConstraint for new PK with region_id");
+            assert!(
+                has_add,
+                "Should have AddConstraint for new PK with region_id"
+            );
         }
     }
 
@@ -2577,32 +2601,26 @@ mod tests {
             // Column changing both nullable and comment
             let from = vec![table(
                 "users",
-                vec![
-                    col("id", ColumnType::Simple(SimpleColumnType::Integer)),
-                    {
-                        let mut c =
-                            col_with_comment("email", ColumnType::Simple(SimpleColumnType::Text), None);
-                        c.nullable = true;
-                        c
-                    },
-                ],
+                vec![col("id", ColumnType::Simple(SimpleColumnType::Integer)), {
+                    let mut c =
+                        col_with_comment("email", ColumnType::Simple(SimpleColumnType::Text), None);
+                    c.nullable = true;
+                    c
+                }],
                 vec![],
             )];
 
             let to = vec![table(
                 "users",
-                vec![
-                    col("id", ColumnType::Simple(SimpleColumnType::Integer)),
-                    {
-                        let mut c = col_with_comment(
-                            "email",
-                            ColumnType::Simple(SimpleColumnType::Text),
-                            Some("Required email"),
-                        );
-                        c.nullable = false;
-                        c
-                    },
-                ],
+                vec![col("id", ColumnType::Simple(SimpleColumnType::Integer)), {
+                    let mut c = col_with_comment(
+                        "email",
+                        ColumnType::Simple(SimpleColumnType::Text),
+                        Some("Required email"),
+                    );
+                    c.nullable = false;
+                    c
+                }],
                 vec![],
             )];
 
@@ -2744,8 +2762,8 @@ mod tests {
                 vec![
                     col("id", ColumnType::Simple(SimpleColumnType::Integer)),
                     col_nullable("email", ColumnType::Simple(SimpleColumnType::Text), false), // nullable -> non-nullable
-                    col_nullable("name", ColumnType::Simple(SimpleColumnType::Text), true),   // non-nullable -> nullable
-                    col_nullable("phone", ColumnType::Simple(SimpleColumnType::Text), true),  // no change
+                    col_nullable("name", ColumnType::Simple(SimpleColumnType::Text), true), // non-nullable -> nullable
+                    col_nullable("phone", ColumnType::Simple(SimpleColumnType::Text), true), // no change
                 ],
                 vec![],
             )];
@@ -2765,7 +2783,10 @@ mod tests {
                     } if table == "users" && column == "email"
                 )
             });
-            assert!(has_email_change, "Should detect email nullable -> non-nullable");
+            assert!(
+                has_email_change,
+                "Should detect email nullable -> non-nullable"
+            );
 
             let has_name_change = plan.actions.iter().any(|a| {
                 matches!(
@@ -2778,7 +2799,10 @@ mod tests {
                     } if table == "users" && column == "name"
                 )
             });
-            assert!(has_name_change, "Should detect name non-nullable -> nullable");
+            assert!(
+                has_name_change,
+                "Should detect name non-nullable -> nullable"
+            );
         }
 
         #[test]

crates/vespertide-planner/src/error.rs

@@ -39,7 +39,9 @@ pub enum PlannerError {
 }
 
 #[derive(Debug, Error)]
-#[error("enum '{enum_name}' in column '{table_name}.{column_name}' has invalid {value_type} value '{value}': not in allowed values [{allowed}]")]
+#[error(
+    "enum '{enum_name}' in column '{table_name}.{column_name}' has invalid {value_type} value '{value}': not in allowed values [{allowed}]"
+)]
 pub struct InvalidEnumDefaultError {
     pub enum_name: String,
     pub table_name: String,

crates/vespertide-planner/src/lib.rs

@@ -1,13 +1,15 @@
-pub mod apply;
-pub mod diff;
-pub mod error;
-pub mod plan;
-pub mod schema;
-pub mod validate;
-
-pub use apply::apply_action;
-pub use diff::diff_schemas;
-pub use error::PlannerError;
-pub use plan::{plan_next_migration, plan_next_migration_with_baseline};
-pub use schema::schema_from_plans;
-pub use validate::{find_missing_fill_with, validate_migration_plan, validate_schema, FillWithRequired};
+pub mod apply;
+pub mod diff;
+pub mod error;
+pub mod plan;
+pub mod schema;
+pub mod validate;
+
+pub use apply::apply_action;
+pub use diff::diff_schemas;
+pub use error::PlannerError;
+pub use plan::{plan_next_migration, plan_next_migration_with_baseline};
+pub use schema::schema_from_plans;
+pub use validate::{
+    FillWithRequired, find_missing_fill_with, validate_migration_plan, validate_schema,
+};