mcp-migration-advisor 0.2.6 → 0.2.8

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

@@ -132,6 +132,22 @@ function generateRollbackStatement(stmt) {
                     warning: `Type change on ${stmt.tableName}.${stmt.columnName} requires knowing the original type. Check schema before migration.`,
                 };
             }
+            if (stmt.details.setDefault === "true") {
+                return {
+                    forward: stmt.raw,
+                    rollback: `ALTER TABLE ${stmt.tableName} ALTER COLUMN ${stmt.columnName} DROP DEFAULT`,
+                    isReversible: true,
+                    warning: null,
+                };
+            }
+            if (stmt.details.dropDefault === "true") {
+                return {
+                    forward: stmt.raw,
+                    rollback: `-- Cannot reverse DROP DEFAULT on ${stmt.tableName}.${stmt.columnName} — original default value unknown`,
+                    isReversible: false,
+                    warning: `DROP DEFAULT on ${stmt.tableName}.${stmt.columnName} requires knowing the original default value to reverse.`,
+                };
+            }
             return {
                 forward: stmt.raw,
                 rollback: `-- Cannot reverse: ${stmt.raw.slice(0, 80)}`,

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.5 — MCP server for database migration risk analysis
+    console.log(`mcp-migration-advisor v0.2.8 — 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.5",
+    version: "0.2.8",
 });
 // 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.", {
@@ -112,7 +112,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);
@@ -200,6 +200,8 @@ server.tool("score_risk", "Calculate the combined risk score (0-100) for a SQL m
     const riskScore = calculateRiskScore(lockRisks);
     const criticalCount = lockRisks.filter(r => r.severity === "CRITICAL").length;
     const highCount = lockRisks.filter(r => r.severity === "HIGH").length;
+    const mediumCount = lockRisks.filter(r => r.severity === "MEDIUM").length;
+    const lowCount = lockRisks.filter(r => r.severity === "LOW").length;
     const dataLossCertain = dataLossIssues.filter(i => i.risk === "CERTAIN").length;
     const dataLossLikely = dataLossIssues.filter(i => i.risk === "LIKELY").length;
     const dataLossPossible = dataLossIssues.filter(i => i.risk === "POSSIBLE").length;
@@ -226,6 +228,8 @@ server.tool("score_risk", "Calculate the combined risk score (0-100) for a SQL m
 |----------|-------|--------------------|
 | CRITICAL lock risks | ${criticalCount} | ${criticalCount * 30} |
 | HIGH lock risks | ${highCount} | ${highCount * 20} |
+| MEDIUM lock risks | ${mediumCount} | ${mediumCount * 10} |
+| LOW lock risks | ${lowCount} | ${lowCount * 5} |
 | Certain data loss | ${dataLossCertain} | ${dataLossCertain * 25} |
 | Likely data loss | ${dataLossLikely} | ${dataLossLikely * 15} |
 | Possible data loss | ${dataLossPossible} | ${dataLossPossible * 5} |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-migration-advisor",
-  "version": "0.2.6",
+  "version": "0.2.8",
   "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"
   }
 }