mcp-migration-advisor 0.2.3 → 0.2.4

package/build/analyzers/lock-risk.js

@@ -147,6 +147,17 @@ function analyzeStatement(stmt) {
             recommendation: "Avoid explicit locks in migrations. Use row-level locking or redesign the migration.",
         });
     }
+    // TRUNCATE acquires ACCESS EXCLUSIVE lock — same severity as DROP TABLE
+    if (stmt.type === "OTHER" && /\bTRUNCATE\b/i.test(stmt.raw)) {
+        const tableMatch = stmt.raw.match(/TRUNCATE\s+(?:TABLE\s+)?(?:`|"|)?(\w+)/i);
+        risks.push({
+            severity: "HIGH",
+            statement: truncate(stmt.raw),
+            tableName: tableMatch?.[1] || null,
+            risk: "TRUNCATE acquires ACCESS EXCLUSIVE lock, blocking all concurrent reads and writes for the duration.",
+            recommendation: "On large tables, prefer DELETE with a WHERE clause in batches. If speed is critical, ensure a maintenance window.",
+        });
+    }
     return risks;
 }
 function truncate(s, max = 120) {

package/build/index.js

@@ -33,7 +33,7 @@ function formatParserWarnings(migration) {
 }
 // Handle --help
 if (process.argv.includes("--help") || process.argv.includes("-h")) {
-    console.log(`mcp-migration-advisor v0.1.0 — MCP server for database migration risk analysis
+    console.log(`mcp-migration-advisor v0.2.4 — MCP server for database migration risk analysis
 
 Usage:
   mcp-migration-advisor [options]
@@ -52,7 +52,7 @@ Tools provided:
 }
 const server = new McpServer({
     name: "mcp-migration-advisor",
-    version: "0.1.0",
+    version: "0.2.4",
 });
 // Tool 1: analyze_migration
 server.tool("analyze_migration", "Analyze a SQL migration file for lock risks, data loss potential, and unsafe patterns. Supports Flyway (V__*.sql) and plain SQL.", {

package/build/parsers/flyway-sql.js

@@ -31,7 +31,8 @@ export function parseFlywayFilename(filename) {
 }
 /**
  * Split SQL text into individual statements.
- * Handles semicolons, ignoring those inside strings and comments.
+ * Handles semicolons, ignoring those inside string literals and comments.
+ * Single-quoted strings are tracked; escaped quotes ('') are handled correctly.
  */
 function splitStatements(sql) {
     // Remove block comments
@@ -40,16 +41,38 @@ function splitStatements(sql) {
     cleaned = cleaned.replace(/--.*$/gm, "");
     const stmts = [];
     let current = "";
-    for (const char of cleaned) {
-        if (char === ";") {
+    let inString = false;
+    let i = 0;
+    while (i < cleaned.length) {
+        const char = cleaned[i];
+        if (char === "'" && !inString) {
+            inString = true;
+            current += char;
+            i++;
+        }
+        else if (char === "'" && inString) {
+            current += char;
+            i++;
+            // '' is the SQL escape for a literal single quote inside a string
+            if (i < cleaned.length && cleaned[i] === "'") {
+                current += cleaned[i];
+                i++;
+            }
+            else {
+                inString = false;
+            }
+        }
+        else if (char === ";" && !inString) {
             const trimmed = current.trim();
             if (trimmed.length > 0) {
                 stmts.push(trimmed);
             }
             current = "";
+            i++;
         }
         else {
             current += char;
+            i++;
         }
     }
     const last = current.trim();

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "mcp-migration-advisor",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "MCP server for database migration risk analysis — Flyway and Liquibase XML/YAML/SQL support with lock detection and conflict analysis",
+  "mcpName": "Migration Advisor",
   "main": "build/index.js",
   "bin": {
     "mcp-migration-advisor": "build/index.js"