mcp-jvm-diagnostics 0.1.11 → 0.1.13

package/build/index.js

@@ -13,7 +13,7 @@ import { parseJfrSummary } from "./parsers/jfr-summary.js";
 import { generateReportFromThreadDump, analyzeThreadDumpMarkdown, } from "./reporting.js";
 // Handle --help
 if (process.argv.includes("--help") || process.argv.includes("-h")) {
-    console.log(`mcp-jvm-diagnostics v0.1.11 — MCP server for JVM diagnostics
+    console.log(`mcp-jvm-diagnostics v0.1.12 — MCP server for JVM diagnostics
 
 Usage:
   mcp-jvm-diagnostics [options]
@@ -33,7 +33,7 @@ Tools provided:
 }
 const server = new McpServer({
     name: "mcp-jvm-diagnostics",
-    version: "0.1.11",
+    version: "0.1.12",
 });
 // --- Tool: analyze_thread_dump ---
 server.tool("analyze_thread_dump", "Parse a JVM thread dump (jstack output) and analyze thread states, detect deadlocks, identify lock contention hotspots, and find thread starvation patterns. Handles both platform threads and virtual threads (Java 21+). Note: deadlock detection covers synchronized monitor locks only — java.util.concurrent.locks.ReentrantLock and other j.u.c lock types do not expose their waiters in thread dump lock info and will not be detected.", {
@@ -211,10 +211,13 @@ server.tool("analyze_heap_histo", "Parse jmap -histo output and detect memory le
 server.tool("compare_heap_histos", "Compare two jmap -histo snapshots taken at different times to detect memory growth patterns, leak candidates, and new allocations. Captures what is growing between snapshots.", {
     before: z
         .string()
-        .describe("The FIRST (earlier) jmap -histo output"),
+        .describe("The FIRST (earlier) jmap -histo output. Capture from a stable baseline — before the suspected leak window or after application startup has settled. " +
+        "Use `jmap -histo:live <pid>` to force a GC before capturing, ensuring unreachable objects are excluded and the snapshot reflects true live objects."),
     after: z
         .string()
-        .describe("The SECOND (later) jmap -histo output"),
+        .describe("The SECOND (later) jmap -histo output. Capture after reproducing the suspected leak or after running under load. " +
+        "Allow at least 5-10 minutes between snapshots — short gaps may not accumulate enough growth to distinguish real leaks from normal allocation churn. " +
+        "Use the same `jmap -histo:live` flag as the before snapshot for a fair comparison."),
 }, async ({ before, after }) => {
     try {
         const report = compareHeapHistos(before, after);
@@ -349,11 +352,11 @@ server.tool("diagnose_jvm", "Unified JVM diagnosis combining thread dump and GC
     thread_dump: z
         .string()
         .optional()
-        .describe("Thread dump text (from jstack, kill -3, or VisualVM). Capture at the same time as the GC log for accurate cross-correlation."),
+        .describe("Thread dump text (from jstack, kill -3, or VisualVM). Capture during active symptoms for meaningful cross-correlation — a dump from a healthy period will not correlate with GC pressure visible in the log."),
     gc_log: z
         .string()
         .optional()
-        .describe("GC log text (from -Xlog:gc* for Java 9+, or -verbose:gc for Java 8). Capture at the same time as the thread dump for accurate cross-correlation."),
+        .describe("GC log text (from -Xlog:gc* for Java 9+, or -verbose:gc for Java 8). Provide the portion covering the time window when the thread dump was taken — trim the log to the symptom window if it is large. Logs from a different time period produce spurious correlations."),
 }, async ({ thread_dump, gc_log }) => {
     try {
         if (!thread_dump && !gc_log) {

package/build/parsers/gc-log.js

@@ -122,6 +122,7 @@ export function parseGcLog(text) {
                 heapAfterMb: Math.round(parseInt(parallelRegionMatch[3], 10) / 1024),
                 heapTotalMb: Math.round(parseInt(parallelRegionMatch[4], 10) / 1024),
             });
+            continue;
         }
     }
     // Calculate time span

package/build/parsers/jfr-summary.js

@@ -142,7 +142,8 @@ function analyzeJfrEvents(events, totalEvents, totalSize) {
         .filter(Boolean)
         .reduce((sum, e) => sum + e.count, 0);
     if (ioCount > 5000) {
-        issues.push(`High I/O activity: ${ioCount.toLocaleString()} file/socket events. Consider batching or buffering.`);
+        issues.push(`High I/O activity: ${ioCount.toLocaleString()} file/socket events recorded.`);
+        recommendations.push("Profile I/O call sites with async-profiler or VisualVM. Common causes: unbuffered writes, per-record DB queries, missing connection pooling. Consider batching writes and reads.");
     }
     // Check for compilation events
     const compilation = byName.get("jdk.Compilation");

package/build/reporting.js

@@ -6,7 +6,7 @@ import { analyzeContention } from "./analyzers/contention.js";
 import { detectDeadlocks } from "./analyzers/deadlock.js";
 import { parseThreadDump } from "./parsers/thread-dump.js";
 const PRODUCT_NAME = "jvm-diagnostics";
-const TOOL_VERSION = "0.1.7";
+const TOOL_VERSION = "0.1.12";
 export const MISSING_LICENSE_INPUT_ERROR = "Missing required input: license_key";
 export const MISSING_THREAD_DUMP_ERROR = "Missing required input: thread_dump";
 export const MISSING_ENV_LICENSE_ERROR = "MCP_LICENSE_KEY environment variable is not set. generate_report requires a valid Pro license key. Free-tier analysis tools remain available without it.";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-jvm-diagnostics",
-  "version": "0.1.11",
+  "version": "0.1.13",
   "description": "MCP server for JVM diagnostics — analyze thread dumps, detect deadlocks, parse GC logs, and get JVM tuning recommendations",
   "mcpName": "io.github.dmitriusan/mcp-jvm-diagnostics",
   "author": "Dmytro Lisnichenko",