fix: update squawk-syntax to 2.58.0 to fix global cargo install failures

Author: dsecurity49

CHANGELOG.md

@@ -2,6 +2,14 @@
 
 All notable changes to safe-migrate are documented here.
 
+## v0.3.2
+
+Official support for `squawk-syntax` v2.58.0. 
+
+- Unpinned the `squawk-syntax` dependency from `=2.56.0` and updated the expression visitor to handle upstream breaking AST changes. 
+- `CallExpr` argument lists are now correctly mapped through the newly introduced `Arg` struct wrapper, and the `BinOp::Escape` variant is now exhaustively handled.
+- This ensures `cargo install safe-migrate` works natively with the latest parser ecosystem out-of-the-box without requiring the `--locked` flag.
+
 ## v0.3.1
 
 Hotfix for a broken `cargo install` on v0.3.0.

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "safe-migrate"
-version = "0.3.1"
+version = "0.3.2"
 edition = "2024"
 description = "Lint PostgreSQL migrations against live database statistics to prevent blocking locks"
 license = "MIT OR Apache-2.0"
@@ -10,7 +10,7 @@ keywords = ["postgres", "migration", "linter", "ast", "database"]
 categories = ["command-line-utilities", "database"]
 
 [dependencies]
-squawk-syntax = "=2.56.0"
+squawk-syntax = "2.58.0"
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 toml = "0.8"

Cargo.toml

@@ -1,14 +1,14 @@
-# safe-migrate v0.3.1
+# safe-migrate v0.3.2
 
 A PostgreSQL migration linter that **executes a bi-directional state machine simulation** over your SQL, combining static typed AST analysis with live database statistics to prevent blocking locks before they reach production.
 
 **The Problem:** `ALTER TABLE users ADD COLUMN status TEXT` is safe on 500 rows. On 50M rows, it acquires an `ACCESS EXCLUSIVE` lock that takes down your app. Standard linters only look at the SQL. **safe-migrate looks at the SQL AND the size of the tables it affects.**
 
 ---
 
-## What's New in v0.3.1
+## What's New in v0.3.2
 
-v0.3.1 is a complete internal rewrite. Earlier versions parsed migrations with regex and substring matching, which broke on quoted identifiers, schemas, and anything non-trivial. safe-migrate now walks a typed PostgreSQL AST and runs a full state machine simulation of the migration — including transaction rollbacks, cascading drops, and partition hierarchies — before evaluating any rule.
+v0.3.2 is a complete internal rewrite. Earlier versions parsed migrations with regex and substring matching, which broke on quoted identifiers, schemas, and anything non-trivial. safe-migrate now walks a typed PostgreSQL AST and runs a full state machine simulation of the migration — including transaction rollbacks, cascading drops, and partition hierarchies — before evaluating any rule.
 
 Highlights:
 - Typed AST parsing via `squawk_syntax` (no more string matching)

README.md

@@ -49,9 +49,15 @@ impl ExprVisitor {
             })
             .unwrap_or_else(|| "<fn>".into());
 
+        // FIX for squawk_syntax >= 2.58.0: filter_map through the new `Arg` wrapper
         let args = ce
             .arg_list()
-            .map(|al| al.args().map(Self::convert).collect())
+            .map(|al| {
+                al.args()
+                    .filter_map(|arg| arg.expr())
+                    .map(Self::convert)
+                    .collect()
+            })
             .unwrap_or_default();
 
         ExprIr::FunctionCall { name, args }
@@ -106,6 +112,8 @@ impl ExprVisitor {
                 BinOp::NotSimilarTo(n) => n.syntax().text().to_string(),
                 BinOp::OperatorCall(n) => n.syntax().text().to_string(),
                 BinOp::SimilarTo(n) => n.syntax().text().to_string(),
+                // FIX for squawk_syntax >= 2.58.0: New Escape operator coverage
+                BinOp::Escape(t) => t.text().to_string(),
             })
             .unwrap_or_else(|| "<op>".into());
 
@@ -134,7 +142,6 @@ impl ExprVisitor {
     fn convert_case_expr(ce: squawk_syntax::ast::CaseExpr) -> ExprIr {
         let mut branches: Vec<ExprIr> = Vec::new();
 
-        // Index 0: Base expression (or omitted if standalone WHEN clauses)
         branches.push(ce.expr().map(Self::convert).unwrap_or(ExprIr::Omitted));
 
         if let Some(wcl) = ce.when_clause_list() {
@@ -148,7 +155,6 @@ impl ExprVisitor {
             }
         }
 
-        // Final Index: Else clause (or omitted)
         branches.push(
             ce.else_clause()
                 .and_then(|ec| ec.expr())