Fix markdown log RPC title rendering - remove double wrapping

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Author: Copilot

internal/logger/markdown_logger.go

@@ -147,7 +147,15 @@ func (ml *MarkdownLogger) Log(level LogLevel, category, format string, args ...i
 	// Format as markdown bullet point with emoji
 	// Use code blocks for multi-line content or technical details
 	var logLine string
-	if strings.Contains(message, "\n") || strings.Contains(message, "command=") || strings.Contains(message, "args=") {
+	
+	// Check if message is already pre-formatted (RPC messages with markdown formatting)
+	// RPC messages start with ** and contain → or ← arrows
+	isPreformatted := strings.HasPrefix(message, "**") && (strings.Contains(message, "→") || strings.Contains(message, "←"))
+	
+	if isPreformatted {
+		// Pre-formatted content (like RPC messages) - just add bullet and emoji
+		logLine = fmt.Sprintf("- %s %s %s\n", emoji, category, message)
+	} else if strings.Contains(message, "\n") || strings.Contains(message, "command=") || strings.Contains(message, "args=") {
 		// Multi-line or technical content - use code block
 		logLine = fmt.Sprintf("- %s **%s**\n  ```\n  %s\n  ```\n", emoji, category, message)
 	} else {

internal/logger/markdown_logger_test.go

@@ -309,3 +309,58 @@ func TestMarkdownLoggerFallback(t *testing.T) {
 	// Should not panic when logging
 	LogInfoMd("test", "This should not crash")
 }
+
+func TestMarkdownLoggerRPCFormatting(t *testing.T) {
+	tmpDir := t.TempDir()
+	logDir := filepath.Join(tmpDir, "logs")
+	fileName := "rpc-format-test.md"
+
+	err := InitMarkdownLogger(logDir, fileName)
+	if err != nil {
+		t.Fatalf("InitMarkdownLogger failed: %v", err)
+	}
+
+	// Test RPC-style pre-formatted messages (should NOT be wrapped in code blocks)
+	LogDebugMd("rpc", "**github**→`tools/list`")
+	LogDebugMd("rpc", "**safeoutputs**→`tools/call`\n\n```json\n{\n  \"id\": 1\n}\n```")
+	LogDebugMd("rpc", "**backend**←`resp`")
+
+	// Test regular messages (should use code blocks for multi-line)
+	LogInfoMd("backend", "command=/usr/bin/docker args=[run]")
+	LogInfoMd("backend", "Multi\nline\ncontent")
+
+	CloseMarkdownLogger()
+
+	// Read the log file
+	logPath := filepath.Join(logDir, fileName)
+	content, err := os.ReadFile(logPath)
+	if err != nil {
+		t.Fatalf("Failed to read log file: %v", err)
+	}
+
+	logContent := string(content)
+
+	// RPC messages should be on single lines (not wrapped in code blocks)
+	// Check that "- 🔍 rpc **github**→`tools/list`" is on one line
+	if !strings.Contains(logContent, "- 🔍 rpc **github**→`tools/list`") {
+		t.Errorf("RPC message should be on single line without extra code block wrapping")
+	}
+
+	// Check that RPC messages with JSON blocks are properly formatted
+	// The title should be on one line, followed by the JSON block
+	if !strings.Contains(logContent, "- 🔍 rpc **safeoutputs**→`tools/call`") {
+		t.Errorf("RPC message with JSON should have title on single line")
+	}
+
+	// Regular multi-line messages should still use code blocks
+	if !strings.Contains(logContent, "- ✓ **backend**\n  ```\n  command=") {
+		t.Errorf("Regular multi-line messages should still use code blocks")
+	}
+
+	// Count occurrences of nested code blocks (should not happen)
+	// A nested code block would look like: ``` \n  **server** \n ```
+	nestedPattern := "```\n  **"
+	if strings.Contains(logContent, nestedPattern) {
+		t.Errorf("Found nested code blocks - RPC messages should not be double-wrapped")
+	}
+}