mcp-migration-advisor 0.2.3 → 0.2.5

package/README.md

@@ -153,6 +153,13 @@ Returns a conflict report with severity levels, affected tables/columns, and whe
 - **No execution**: The advisor analyzes but never executes migrations. All recommendations are advisory.
 - **Database-specific DDL**: Parser targets PostgreSQL/MySQL DDL syntax. Oracle PL/SQL or SQL Server T-SQL may not be fully recognized.
 
+## Part of the MCP Java Backend Suite
+
+- [mcp-db-analyzer](https://www.npmjs.com/package/mcp-db-analyzer) — PostgreSQL/MySQL/SQLite schema analysis
+- [mcp-spring-boot-actuator](https://www.npmjs.com/package/mcp-spring-boot-actuator) — Spring Boot health, metrics, and bean analysis
+- [mcp-jvm-diagnostics](https://www.npmjs.com/package/mcp-jvm-diagnostics) — Thread dump and GC log analysis
+- [mcp-redis-diagnostics](https://www.npmjs.com/package/mcp-redis-diagnostics) — Redis memory, slowlog, and client diagnostics
+
 ## License
 
 MIT

package/build/analyzers/data-loss.js

@@ -8,16 +8,6 @@
  * - Table drops
  * - CASCADE operations
  */
-// Types that lose precision when converted
-const NARROWING_CONVERSIONS = {
-    "BIGINT": ["INTEGER", "SMALLINT", "TINYINT"],
-    "INTEGER": ["SMALLINT", "TINYINT"],
-    "DOUBLE PRECISION": ["REAL", "FLOAT4", "NUMERIC"],
-    "TEXT": ["VARCHAR", "CHAR"],
-    "VARCHAR": ["CHAR"],
-    "TIMESTAMP": ["DATE", "TIME"],
-    "TIMESTAMPTZ": ["DATE", "TIME", "TIMESTAMP"],
-};
 /**
  * Analyze a migration for data loss risks.
  */

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.5 — 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.5",
 });
 // 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.", {
@@ -202,29 +202,34 @@ server.tool("score_risk", "Calculate the overall risk score (0-100) for a SQL mi
     const highCount = lockRisks.filter(r => r.severity === "HIGH").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;
+    // Combine lock risk score with data loss severity for a complete picture
+    const dataLossScore = Math.min(100, dataLossCertain * 25 + dataLossLikely * 15 + dataLossPossible * 5);
+    const combinedScore = Math.min(100, riskScore + dataLossScore);
     let verdict;
-    if (riskScore >= 60 || dataLossCertain > 0) {
+    if (combinedScore >= 60 || dataLossCertain > 0) {
         verdict = "HIGH RISK — requires careful review and testing before deployment";
     }
-    else if (riskScore >= 30 || dataLossLikely > 0) {
+    else if (combinedScore >= 30 || dataLossLikely > 0) {
         verdict = "MODERATE RISK — review lock duration and test on staging";
     }
     else {
         verdict = "LOW RISK — standard migration, proceed with normal deployment";
     }
-    const output = `## Risk Score: ${riskScore}/100
+    const output = `## Risk Score: ${combinedScore}/100
 
 **Verdict**: ${verdict}
 
 ### Breakdown
 
-| Category | Count |
-|----------|-------|
-| CRITICAL lock risks | ${criticalCount} |
-| HIGH lock risks | ${highCount} |
-| Certain data loss | ${dataLossCertain} |
-| Likely data loss | ${dataLossLikely} |
-| Total statements | ${migration.statements.length} |
+| Category | Count | Score contribution |
+|----------|-------|--------------------|
+| CRITICAL lock risks | ${criticalCount} | ${criticalCount * 30} |
+| HIGH lock risks | ${highCount} | ${highCount * 20} |
+| Certain data loss | ${dataLossCertain} | ${dataLossCertain * 25} |
+| Likely data loss | ${dataLossLikely} | ${dataLossLikely * 15} |
+| Possible data loss | ${dataLossPossible} | ${dataLossPossible * 5} |
+| Total statements | ${migration.statements.length} | — |
 `;
     return {
         content: [{ type: "text", text: output }],

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.5",
   "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",
   "bin": {
     "mcp-migration-advisor": "build/index.js"