Clarify the usage of the commands

Author: CapulusCodeNinja

mcp_nexus/Protocol/McpToolDefinitionService.cs

@@ -17,10 +17,10 @@ public McpToolSchema[] GetAllTools()
             return
             [
                 // ✅ NEXUS DEBUGGER COMMANDS - Core functionality for crash dump analysis
-                CreateNexusOpenDumpTool(),
-                CreateNexusExecDebuggerCommandAsyncTool(),
-                CreateNexusDebuggerCommandStatusTool(),
-                CreateNexusCloseDumpTool()
+                CreateNexusOpenDumpAnalyzeSessionTool(),
+                CreateNexusDumpAnalyzeSessionAsyncCommandTool(),
+                CreateNexusDumpAnalyzeSessionAsyncCommandStatusTool(),
+                CreateNexusCloseDumpAnalyzeSessionTool()
                 // NOTE: Remote debugging and command cancellation will be added in future releases
             ];
         }
@@ -36,23 +36,23 @@ public async Task NotifyToolsChanged()
             }
         }
 
-        private static McpToolSchema CreateNexusOpenDumpTool()
+        private static McpToolSchema CreateNexusOpenDumpAnalyzeSessionTool()
         {
             return new McpToolSchema
             {
-                Name = "nexus_open_dump",
-                Description = "🚀 STEP 1 - START HERE: Open and analyze a Windows crash dump file (.dmp). " +
+                Name = "nexus_open_dump_analyze_session",
+                Description = "🚀 STEP 1 - START HERE: Open and analyze a Windows crash dump file (.dmp) by creating a dedicated analysis session. " +
                     "🚨 CRITICAL RETURN VALUE: This command RETURNS a sessionId in the response JSON that you MUST EXTRACT and SAVE! " +
                     "📤 RESPONSE CONTAINS: {\"sessionId\": \"sess-000001-abc12345\", ...} " +
                     "📝 YOU MUST: Parse the response JSON and extract the 'sessionId' field value! " +
                     "💾 SAVE IT: Store this sessionId string and use it in ALL subsequent commands! " +
                     "🔄 MANDATORY WORKFLOW: " +
-                    "1️⃣ nexus_open_dump → EXTRACT 'sessionId' from response JSON → SAVE IT! " +
-                    "2️⃣ nexus_exec_debugger_command_async + SAVED sessionId → get commandId " +
-                    "3️⃣ nexus_debugger_command_status + commandId → get results " +
-                    "4️⃣ nexus_close_dump + SAVED sessionId → CLOSE session when done (EXPECTED!) " +
+                    "1️⃣ nexus_open_dump_analyze_session → EXTRACT 'sessionId' from response JSON → SAVE IT! " +
+                    "2️⃣ nexus_dump_analyze_session_async_command + SAVED sessionId → get commandId " +
+                    "3️⃣ nexus_dump_analyze_session_async_command_status + commandId → get results " +
+                    "4️⃣ nexus_close_dump_analyze_session + SAVED sessionId → CLOSE session when done (EXPECTED!) " +
                     "❌ DO NOT MAKE UP sessionId VALUES! Use only what this command returns! " +
-                    "🧹 CLEANUP EXPECTATION: You SHOULD call nexus_close_dump when finished analyzing to properly release resources and close the debugging session. While sessions auto-expire after 30 minutes, explicit closure is the expected and professional approach!",
+                    "🧹 CLEANUP EXPECTATION: You SHOULD call nexus_close_dump_analyze_session when finished analyzing to properly release resources and close the debugging session. While sessions auto-expire after 30 minutes, explicit closure is the expected and professional approach!",
                 InputSchema = new
                 {
                     type = "object",
@@ -67,45 +67,45 @@ private static McpToolSchema CreateNexusOpenDumpTool()
         }
 
 
-        private static McpToolSchema CreateNexusCloseDumpTool()
+        private static McpToolSchema CreateNexusCloseDumpAnalyzeSessionTool()
         {
             return new McpToolSchema
             {
-                Name = "nexus_close_dump",
-                Description = "🔚 STEP 4 - CLEANUP: Close the current crash dump session and release resources. " +
+                Name = "nexus_close_dump_analyze_session",
+                Description = "🔚 STEP 4 - CLEANUP: Close the current crash dump analysis session and release resources. " +
                     "⭐ EXPECTED BEHAVIOR: You SHOULD call this when done analyzing a dump file! " +
                     "🧹 PROFESSIONAL PRACTICE: While sessions auto-expire after 30 minutes, explicit closure is the expected and responsible approach. " +
-                    "🔄 NEXT SESSION: After closing, you'll need nexus_open_dump again to analyze another dump. " +
+                    "🔄 NEXT SESSION: After closing, you'll need nexus_open_dump_analyze_session again to analyze another dump. " +
                     "💡 AI CLIENT TIP: Always close sessions when finished - it's good resource management!",
                 InputSchema = new
                 {
                     type = "object",
                     properties = new
                     {
-                        sessionId = new { type = "string", description = "🚨 REQUIRED: Session ID that you EXTRACTED from nexus_open_dump response JSON. Use the EXACT value (e.g., 'sess-000001-abc12345')" }
+                        sessionId = new { type = "string", description = "🚨 REQUIRED: Session ID that you EXTRACTED from nexus_open_dump_analyze_session response JSON. Use the EXACT value (e.g., 'sess-000001-abc12345')" }
                     },
                     required = new[] { "sessionId" }
                 }
             };
         }
 
 
-        private static McpToolSchema CreateNexusExecDebuggerCommandAsyncTool()
+        private static McpToolSchema CreateNexusDumpAnalyzeSessionAsyncCommandTool()
         {
             return new McpToolSchema
             {
-            Name = "nexus_exec_debugger_command_async",
+            Name = "nexus_dump_analyze_session_async_command",
                 Description = "⚡ STEP 2 - EXECUTE COMMANDS: Run debugger commands like '!analyze -v', 'k', 'lm', 'dt', etc. " +
                     "🚨 ASYNC WORKFLOW - READ CAREFULLY: " +
                     "1️⃣ This command ONLY QUEUES the command and returns a commandId " +
                     "2️⃣ It does NOT return the actual debugger output! " +
-                    "3️⃣ You MUST call nexus_debugger_command_status(commandId) to get results " +
+                    "3️⃣ You MUST call nexus_dump_analyze_session_async_command_status(commandId) to get results " +
                     "4️⃣ Commands execute asynchronously in background queue " +
-                    "⏰ POLLING REQUIRED: Check nexus_debugger_command_status EVERY 3-5 SECONDS until status is 'completed' " +
-                    "🔄 EXACT WORKFLOW: nexus_exec_debugger_command_async → GET commandId → nexus_debugger_command_status(commandId) → REPEAT until complete " +
-                    "🎯 MANDATORY sessionId: You MUST include the sessionId from nexus_open_dump response! " +
+                    "⏰ POLLING REQUIRED: Check nexus_dump_analyze_session_async_command_status EVERY 3-5 SECONDS until status is 'completed' " +
+                    "🔄 EXACT WORKFLOW: nexus_dump_analyze_session_async_command → GET commandId → nexus_dump_analyze_session_async_command_status(commandId) → REPEAT until complete " +
+                    "🎯 MANDATORY sessionId: You MUST include the sessionId from nexus_open_dump_analyze_session response! " +
                     "❌ MISSING sessionId = ERROR: This command will FAIL without a valid sessionId parameter! " +
-                    "📝 HOW TO GET sessionId: Call nexus_open_dump first, extract 'sessionId' from response JSON, then use it here " +
+                    "📝 HOW TO GET sessionId: Call nexus_open_dump_analyze_session first, extract 'sessionId' from response JSON, then use it here " +
                     "💡 COMMON COMMANDS: " +
                     "• '!analyze -v' - Detailed crash analysis " +
                     "• 'k' - Call stack " +
@@ -118,7 +118,7 @@ private static McpToolSchema CreateNexusExecDebuggerCommandAsyncTool()
                 properties = new
                 {
                     command = new { type = "string", description = "WinDbg/CDB command like '!analyze -v', 'k', 'lm', etc." },
-                            sessionId = new { type = "string", description = "🚨 REQUIRED: Session ID that you EXTRACTED from nexus_open_dump response JSON. This must be the EXACT value from the 'sessionId' field (e.g., 'sess-000001-abc12345'). DO NOT make up your own values!" }
+                            sessionId = new { type = "string", description = "🚨 REQUIRED: Session ID that you EXTRACTED from nexus_open_dump_analyze_session response JSON. This must be the EXACT value from the 'sessionId' field (e.g., 'sess-000001-abc12345'). DO NOT make up your own values!" }
                 },
                 required = new[] { "command", "sessionId" }
             }
@@ -127,25 +127,27 @@ private static McpToolSchema CreateNexusExecDebuggerCommandAsyncTool()
 
 
 
-        private static McpToolSchema CreateNexusDebuggerCommandStatusTool()
+        private static McpToolSchema CreateNexusDumpAnalyzeSessionAsyncCommandStatusTool()
         {
             return new McpToolSchema
             {
-                Name = "nexus_debugger_command_status",
+                Name = "nexus_dump_analyze_session_async_command_status",
                 Description = "📋 STEP 3 - GET RESULTS: Retrieve the output from a previously queued command. " +
                     "This is the ONLY way to get actual debugger command results! " +
+                    "🚨 WORKFLOW DEPENDENCY: You can ONLY call this AFTER calling nexus_dump_analyze_session_async_command! " +
+                    "🔄 REQUIRED SEQUENCE: nexus_open_dump_analyze_session → nexus_dump_analyze_session_async_command → nexus_dump_analyze_session_async_command_status " +
                     "📊 STATUS FLOW: queued → executing → completed " +
                     "✅ When status='completed', the 'result' field contains the debugger output. " +
                     "⏳ If status='executing' or 'queued', wait 3-5 seconds and call this again! " +
                     "🔄 KEEP POLLING: Call this repeatedly every 3-5 seconds until status='completed' " +
-                    "❌ NEVER GIVE UP: If status is not 'completed', you MUST try again later! " +
+                    "❌ NEVER SKIP STEPS: You cannot make up commandId values or skip nexus_dump_analyze_session_async_command! " +
                     "💡 SMART TIP: Listen for notifications/commandStatus to know when to check instead of polling constantly.",
                 InputSchema = new
                 {
                     type = "object",
                     properties = new
                     {
-                        commandId = new { type = "string", description = "The commandId returned by nexus_exec_debugger_command_async" }
+                        commandId = new { type = "string", description = "🚨 REQUIRED: The EXACT commandId that was returned by nexus_dump_analyze_session_async_command. Format: 'cmd-sess-XXXXXX-YYYYYYYY-ZZZZ'. DO NOT make up your own values!" }
                     },
                     required = new[] { "commandId" }
                 }

mcp_nexus/Protocol/McpToolExecutionService.cs

@@ -17,10 +17,10 @@ public async Task<object> ExecuteTool(string toolName, JsonElement arguments)
             {
                 return toolName switch
                 {
-                    "nexus_open_dump" => await ExecuteOpenWindbgDump(arguments),
-                    "nexus_close_dump" => await ExecuteCloseWindbgDump(arguments),
-                    "nexus_exec_debugger_command_async" => await ExecuteRunWindbgCmdAsync(arguments),
-                    "nexus_debugger_command_status" => await ExecuteGetCommandStatus(arguments),
+                    "nexus_open_dump_analyze_session" => await ExecuteOpenWindbgDump(arguments),
+                    "nexus_close_dump_analyze_session" => await ExecuteCloseWindbgDump(arguments),
+                    "nexus_dump_analyze_session_async_command" => await ExecuteRunWindbgCmdAsync(arguments),
+                    "nexus_dump_analyze_session_async_command_status" => await ExecuteGetCommandStatus(arguments),
                     _ => throw new McpToolException(-32602, $"Unknown tool: {toolName}")
                 };
             }
@@ -48,7 +48,7 @@ private async Task<object> ExecuteOpenWindbgDump(JsonElement arguments)
                     "💡 EXAMPLE: {\"dumpPath\": \"C:\\\\path\\\\to\\\\crash.dmp\"}");
 
             var symbolsPath = GetOptionalStringArgument(arguments, "symbolsPath");
-            var result = await sessionAwareWindbgTool.nexus_open_dump(dumpPath, symbolsPath);
+            var result = await sessionAwareWindbgTool.nexus_open_dump_analyze_session(dumpPath, symbolsPath);
 
             // Return the structured response object directly
             return result;
@@ -66,7 +66,7 @@ private async Task<object> ExecuteCloseWindbgDump(JsonElement arguments)
 
             logger.LogInformation("Manual session closure requested for session: {SessionId}", sessionId);
 
-            var result = await sessionAwareWindbgTool.nexus_close_dump(sessionId);
+            var result = await sessionAwareWindbgTool.nexus_close_dump_analyze_session(sessionId);
             return result;
         }
 
@@ -96,7 +96,7 @@ private async Task<object> ExecuteRunWindbgCmdAsync(JsonElement arguments)
 
             try
             {
-                var result = await sessionAwareWindbgTool.nexus_exec_debugger_command_async(sessionId, command);
+                var result = await sessionAwareWindbgTool.nexus_dump_analyze_session_async_command(sessionId, command);
                 return result;
             }
             catch (Exception ex)
@@ -121,7 +121,7 @@ private async Task<object> ExecuteGetCommandStatus(JsonElement arguments)
 
             try
             {
-                var result = await sessionAwareWindbgTool.nexus_debugger_command_status(commandId);
+                var result = await sessionAwareWindbgTool.nexus_dump_analyze_session_async_command_status(commandId);
                 // Return the structured response object directly
                 return result;
             }

mcp_nexus/Session/Models/SessionModels.cs

@@ -118,6 +118,9 @@ public class SessionContext
         /// <summary>Human-readable session description</summary>
         public string Description { get; set; } = string.Empty;
 
+        /// <summary>Path to the dump file being debugged</summary>
+        public string? DumpPath { get; set; }
+        
         /// <summary>Session creation time</summary>
         public DateTime CreatedAt { get; set; }
 

mcp_nexus/Session/ThreadSafeSessionManager.cs

@@ -343,6 +343,7 @@ public SessionContext GetSessionContext(string sessionId)
             {
                 SessionId = sessionId,
                 Description = $"Debugging session for {Path.GetFileName(session.DumpPath ?? "Unknown")}",
+                DumpPath = session.DumpPath,
                 CreatedAt = session.CreatedAt,
                 LastActivity = session.LastActivity,
                 Status = session.Status.ToString(),