The extension should use MD fromat as well to lower the amount of complex serialize and deserialize the data

Author: CapulusCodeNinja

Directory.Build.props

@@ -36,7 +36,7 @@
   <PropertyGroup Label="Default Versioning">
     <!-- Default version for all projects (can be overridden per project) -->
     <VersionPrefix>1.1.0</VersionPrefix>
-        <VersionSuffix>60</VersionSuffix>
+        <VersionSuffix>61</VersionSuffix>
 
     <!-- Calculated version -->
     <Version Condition="'$(Version)' == ''">$(VersionPrefix).$(VersionSuffix)</Version>

nexus_engine/nexus_engine_extensions/ExtensionScripts/basic_crash_analysis/basic_crash_analysis.ps1

@@ -55,36 +55,47 @@ try {
 
     Write-NexusLog "Basic crash analysis completed successfully (5 commands executed, 4 using async batching)" -Level Information
 
-    # Return structured result
-    $result = @{
-        success = $true
-        workflow = "basic_crash_analysis"
-        steps = @{
-            automaticAnalysis = @{
-                command = "!analyze -v"
-                output = $results["analyze"]
-            }
-            threadInfo = @{
-                command = "!threads"
-                output = $results["threads"]
-            }
-            allThreadStacks = @{
-                command = "~*k"
-                output = $results["allStacks"]
-            }
-            locks = @{
-                command = "!locks"
-                output = $results["locks"]
-            }
-            cpuUsage = @{
-                command = "!runaway"
-                output = $results["runaway"]
-            }
-        }
-        message = "Basic crash analysis completed successfully using async batching. Review the outputs to identify faulting thread, exception type, and crash context."
-    } | ConvertTo-Json -Depth 10
-
-    Write-Output $result
+    # Compose Markdown report
+    $now = Get-Date -Format "yyyy-MM-dd HH:mm:ss"
+    $md = @"
+## Basic Crash Analysis
+
+**Workflow:** `basic_crash_analysis`
+**Executed:** $now
+
+### !analyze -v
+
+```
+$($results["analyze"])
+```
+
+### !threads
+
+```
+$($results["threads"])
+```
+
+### ~*k
+
+```
+$($results["allStacks"])
+```
+
+### !locks
+
+```
+$($results["locks"])
+```
+
+### !runaway
+
+```
+$($results["runaway"])
+```
+
+"@
+
+    Write-Output $md
     exit 0
 }
 catch {

nexus_engine/nexus_engine_extensions/ExtensionScripts/stack_with_sources/stack_with_sources.ps1

@@ -78,17 +78,21 @@ try {
 
         Write-NexusLog "[$threadDisplay] Thread not found or invalid (ThreadId: $ThreadId)" -Level Error
 
-        $result = @{
-            success = $false
-            threadId = $ThreadId
-            totalFrames = 0
-            sourcesDownloaded = 0
-            addresses = @()
-            error = "Thread '$ThreadId' not found. The thread ID may be invalid or the thread may not exist in this dump file."
-            suggestion = "Use the '!threads' or '~' command to list available threads, then try again with a valid thread ID."
-            stackTrace = $stackOutput
-        } | ConvertTo-Json -Depth 10
-        Write-Output $result
+        $md = @"
+## Stack With Sources
+
+**Success:** False
+**Thread ID:** `$ThreadId`
+**Message:** Thread '$ThreadId' not found. The thread ID may be invalid or may not exist in this dump.
+**Suggestion:** Use `!threads` or `~` to list available threads, then retry with a valid thread ID.
+
+### Stack Trace
+
+```
+$stackOutput
+```
+"@
+        Write-Output $md
         exit 0
     }
 
@@ -116,16 +120,20 @@ try {
 
     if ($addresses.Count -eq 0) {
         Write-NexusLog "No valid return addresses found in stack trace" -Level Warning
-        $result = @{
-            success = $false
-            threadId = $ThreadId
-            totalFrames = 0
-            sourcesDownloaded = 0
-            addresses = @()
-            error = "No valid return addresses found in stack trace"
-            stackTrace = $stackOutput
-        } | ConvertTo-Json -Depth 10
-        Write-Output $result
+        $md = @"
+## Stack With Sources
+
+**Success:** False
+**Thread ID:** `$ThreadId`
+**Message:** No valid return addresses found in stack trace
+
+### Stack Trace
+
+```
+$stackOutput
+```
+"@
+        Write-Output $md
         exit 0
     }
 
@@ -161,21 +169,31 @@ try {
 
     if ($addressesWithSymbols.Count -eq 0) {
         Write-NexusLog "No addresses with symbols found to download sources for" -Level Warning
-        $result = @{
-            success = $true
-            threadId = $ThreadId
-            totalFrames = $addresses.Count
-            framesWithSymbols = 0
-            sourcesDownloaded = 0
-            sourcesMissing = 0
-            successRate = 0
-            addresses = $addresses
-            failedAddresses = @()
-            message = "No addresses with symbols found, nothing to download"
-            stackTrace = $stackOutput
-            sourceOutputs = @{}
-        } | ConvertTo-Json -Depth 10
-        Write-Output $result
+        $addrList = ($addresses -join "\n")
+        $md = @"
+## Stack With Sources
+
+**Success:** True
+**Thread ID:** `$ThreadId`
+**Total Frames:** $($addresses.Count)
+**Frames With Symbols:** 0
+**Sources Downloaded:** 0
+**Success Rate:** 0%
+**Message:** No addresses with symbols found, nothing to download
+
+### Stack Trace
+
+```
+$stackOutput
+```
+
+### Addresses (All)
+
+```
+$addrList
+```
+"@
+        Write-Output $md
         exit 0
     }
 
@@ -232,24 +250,64 @@ try {
         Write-NexusLog "Failed to download sources for $($failedAddresses.Count) addresses" -Level Warning
     }
 
-    # Return structured result with all command outputs
-    $result = @{
-        success = $true
-        threadId = $ThreadId
-        totalFrames = $addresses.Count
-        framesWithSymbols = $addressesWithSymbols.Count
-        sourcesDownloaded = $downloadedCount
-        sourcesMissing = $failedAddresses.Count
-        successRate = $successRate
-        addresses = $addresses
-        addressesWithSymbols = $addressesWithSymbols
-        failedAddresses = $failedAddresses
-        message = "Successfully downloaded and verified $downloadedCount of $($addressesWithSymbols.Count) source files (filtered from $($addresses.Count) total addresses)"
-        stackTrace = $stackOutput
-        sourceOutputs = $sourceOutputs
-    } | ConvertTo-Json -Depth 10
-
-    Write-Output $result
+    # Compose Markdown report with summary and outputs
+    $now = Get-Date -Format "yyyy-MM-dd HH:mm:ss"
+    $addrAll = ($addresses -join "\n")
+    $addrWithSymbols = ($addressesWithSymbols -join "\n")
+    $failedList = ($failedAddresses -join "\n")
+
+    # Flatten sourceOutputs into a readable section
+    $sourceOutputsText = New-Object System.Text.StringBuilder
+    foreach ($kv in $sourceOutputs.GetEnumerator()) {
+        [void]$sourceOutputsText.AppendLine("[$($kv.Key)]")
+        [void]$sourceOutputsText.AppendLine($kv.Value)
+        [void]$sourceOutputsText.AppendLine()
+    }
+
+    $md = @"
+## Stack With Sources
+
+**Success:** True
+**Thread ID:** `$ThreadId`
+**Executed:** $now
+**Total Frames:** $($addresses.Count)
+**Frames With Symbols:** $($addressesWithSymbols.Count)
+**Sources Downloaded:** $downloadedCount
+**Sources Missing:** $($failedAddresses.Count)
+**Success Rate:** $successRate%
+
+### Stack Trace
+
+```
+$stackOutput
+```
+
+### Addresses With Symbols
+
+```
+$addrWithSymbols
+```
+
+### Failed Addresses
+
+```
+$failedList
+```
+
+### All Addresses
+
+```
+$addrAll
+```
+
+### Source Verification Outputs
+
+```
+$($sourceOutputsText.ToString())
+```
+"@
+
+    Write-Output $md
     exit 0
 }
 catch {

nexus_protocol/Middleware/JsonRpcLoggingMiddleware.cs

@@ -15,7 +15,7 @@ internal class JsonRpcLoggingMiddleware
 {
     private readonly RequestDelegate m_Next;
     private readonly Logger m_Logger;
-    private const int MaxLength = 500;
+    private const int MaxLength = 1500;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="JsonRpcLoggingMiddleware"/> class.

nexus_protocol/Tools/ReadDumpAnalyzeCommandResultTool.cs

@@ -51,7 +51,7 @@ public static async Task<object> nexus_read_dump_analyze_command_result(
 
             if (!string.IsNullOrEmpty(commandInfo.AggregatedOutput))
             {
-                markdown += MarkdownFormatter.CreateCodeBlock(commandInfo.AggregatedOutput, "Output");
+                markdown += MarkdownFormatter.AppendOutputForCommand(commandInfo.Command, commandInfo.AggregatedOutput, "Output");
             }
 
             if (!string.IsNullOrEmpty(commandInfo.ErrorMessage))

nexus_protocol/Utilities/MarkdownFormatter.cs

@@ -440,6 +440,27 @@ public static string CreateOperationResult(
         return markdown.ToString();
     }
 
+    /// <summary>
+    /// Appends command output according to its origin. For extension commands (prefix "Extension: "),
+    /// returns the output verbatim (expected to be Markdown). For all other commands, wraps output
+    /// in a code block with an optional title.
+    /// </summary>
+    /// <param name="command">The command label as stored in command info.</param>
+    /// <param name="output">The raw output to append.</param>
+    /// <param name="titleForNonExtension">Optional title when wrapping non-extension output.</param>
+    /// <returns>Markdown to append for the given output.</returns>
+    public static string AppendOutputForCommand(string? command, string? output, string? titleForNonExtension = "Output")
+    {
+        if (string.IsNullOrEmpty(output))
+        {
+            return string.Empty;
+        }
+
+        return !string.IsNullOrEmpty(command) && command.StartsWith("Extension: ", StringComparison.Ordinal)
+            ? output
+            : CreateCodeBlock(output, titleForNonExtension);
+    }
+
     /// <summary>
     /// Helper method to get property value from anonymous object.
     /// </summary>

unittests/nexus_protocol_unittests/Tools/ReadDumpAnalyzeCommandResultToolTests.cs

@@ -0,0 +1,47 @@
+using FluentAssertions;
+
+using Nexus.Protocol.Utilities;
+
+using Xunit;
+
+namespace Nexus.Protocol.Tests.Tools;
+
+/// <summary>
+/// Tests for output wrapping decision for extension vs non-extension commands.
+/// </summary>
+public class ReadDumpAnalyzeCommandResultToolTests
+{
+    /// <inheritdoc/>
+    [Fact]
+    public void AppendOutputForCommand_NonExtension_WrapsInCodeBlock()
+    {
+        // Arrange
+        var command = "!threads";
+        var output = "line1\nline2";
+
+        // Act
+        var result = MarkdownFormatter.AppendOutputForCommand(command, output, "Output");
+
+        // Assert
+        _ = result.Should().Contain("```");
+        _ = result.Should().Contain("line1");
+        _ = result.Should().Contain("line2");
+    }
+
+    /// <inheritdoc/>
+    [Fact]
+    public void AppendOutputForCommand_Extension_ReturnsVerbatim()
+    {
+        // Arrange
+        var command = "Extension: basic_crash_analysis";
+        var output = "## Some Markdown\n\n``\ncode\n``".Replace("``", "```");
+
+        // Act
+        var result = MarkdownFormatter.AppendOutputForCommand(command, output, "Output");
+
+        // Assert
+        _ = result.Should().Be(output);
+    }
+}
+
+