mcp-migration-advisor 0.2.7 → 0.2.9

package/build/analyzers/data-loss.js

@@ -74,6 +74,11 @@ export function analyzeDataLoss(migration) {
                 break;
             }
             case "OTHER": {
+                // CREATE FUNCTION/PROCEDURE/TRIGGER bodies may contain TRUNCATE/DELETE/UPDATE
+                // statements that only execute when the function is invoked — not at migration time.
+                // Skip data manipulation checks for CREATE statements to avoid false positives.
+                if (/^CREATE\b/i.test(stmt.raw.trim()))
+                    break;
                 // Detect TRUNCATE
                 if (upper.includes("TRUNCATE")) {
                     const tableMatch = stmt.raw.match(/TRUNCATE\s+(?:TABLE\s+)?(?:`|"|)?(?:\w+\.)?(\w+)/i);

package/build/generators/rollback.js

@@ -66,13 +66,17 @@ function generateRollbackStatement(stmt) {
                 isReversible: true,
                 warning: null,
             };
-        case "DROP_TABLE":
+        case "DROP_TABLE": {
+            const hasCascade = stmt.details.cascade === "true";
             return {
                 forward: stmt.raw,
                 rollback: `-- Cannot reverse DROP TABLE ${stmt.tableName} without schema backup`,
                 isReversible: false,
-                warning: `DROP TABLE ${stmt.tableName} is irreversible — table structure and data are lost. Restore from backup.`,
+                warning: hasCascade
+                    ? `DROP TABLE ${stmt.tableName} CASCADE is irreversible — table and all dependent objects (views, foreign keys, indexes) are permanently destroyed. Each dependent object must be recreated manually.`
+                    : `DROP TABLE ${stmt.tableName} is irreversible — table structure and data are lost. Restore from backup.`,
             };
+        }
         case "CREATE_INDEX": {
             // Extract index name from the raw SQL
             const indexMatch = stmt.raw.match(/INDEX\s+(?:CONCURRENTLY\s+)?(?:IF\s+NOT\s+EXISTS\s+)?(?:`|"|)?(\w+)/i);

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.2.6 — MCP server for database migration risk analysis
+    console.log(`mcp-migration-advisor v0.2.9 — 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.2.6",
+    version: "0.2.9",
 });
 // 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.", {
@@ -62,7 +62,11 @@ server.tool("analyze_migration", "Analyze a SQL migration file for lock risks, d
     const migration = parseMigration(filename, sql);
     const lockRisks = analyzeLockRisks(migration);
     const dataLossIssues = analyzeDataLoss(migration);
-    const riskScore = calculateRiskScore(lockRisks);
+    const lockScore = calculateRiskScore(lockRisks);
+    const dataLossScore = Math.min(100, dataLossIssues.filter(i => i.risk === "CERTAIN").length * 25 +
+        dataLossIssues.filter(i => i.risk === "LIKELY").length * 15 +
+        dataLossIssues.filter(i => i.risk === "POSSIBLE").length * 5);
+    const riskScore = Math.min(100, lockScore + dataLossScore);
     let output = `## Migration Analysis: ${filename}\n\n`;
     // Migration info
     if (migration.version) {
@@ -112,7 +116,7 @@ server.tool("analyze_migration", "Analyze a SQL migration file for lock risks, d
     };
 });
 // Tool 2: analyze_liquibase
-server.tool("analyze_liquibase", "Analyze a Liquibase XML changelog for lock risks, data loss potential, and unsafe patterns.", {
+server.tool("analyze_liquibase", "Analyze a Liquibase XML changelog for lock risks, data loss potential, and unsafe patterns. Supports createTable, dropTable, addColumn, dropColumn, modifyDataType, createIndex, addForeignKeyConstraint, renameTable, renameColumn, and more.", {
     xml: z.string().describe("The Liquibase XML changelog content"),
 }, async ({ xml }) => {
     const migration = parseLiquibaseXml(xml);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-migration-advisor",
-  "version": "0.2.7",
+  "version": "0.2.9",
   "description": "MCP server for database migration risk analysis — Flyway and Liquibase XML/YAML/SQL support with lock detection and conflict analysis",
   "mcpName": "io.github.dmitriusan/mcp-migration-advisor",
   "main": "build/index.js",
@@ -40,7 +40,7 @@
   "author": "Dmytro Lisnichenko",
   "license": "MIT",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=20.0.0"
   },
   "repository": {
     "type": "git",
@@ -52,11 +52,11 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.27.1",
-    "zod": "^3.24.2"
+    "zod": "^4.0.0"
   },
   "devDependencies": {
-    "@types/node": "^22.0.0",
-    "typescript": "^5.8.2",
-    "vitest": "^4.0.18"
+    "@types/node": "^25.5.0",
+    "typescript": "^6.0.0",
+    "vitest": "^4.1.0"
   }
 }