From 4c847e6de9fcbc0a0cbad0218b88b65721f22716 Mon Sep 17 00:00:00 2001
From: Kyle Isom <kyle@imap.cc>
Date: Sat, 28 Mar 2026 18:22:38 -0700
Subject: [PATCH] Fix extraneous blank lines in mcp logs output

Skip empty lines from the scanner that result from double newlines
(application slog trailing newline + container runtime newline).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 internal/agent/logs.go | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/internal/agent/logs.go b/internal/agent/logs.go
index f84377d..714a3e6 100644
--- a/internal/agent/logs.go
+++ b/internal/agent/logs.go
@@ -63,8 +63,12 @@ func (a *Agent) Logs(req *mcpv1.LogsRequest, stream mcpv1.McpAgentService_LogsSe
 
 	scanner := bufio.NewScanner(pr)
 	for scanner.Scan() {
+		line := scanner.Bytes()
+		if len(line) == 0 {
+			continue
+		}
 		if err := stream.Send(&mcpv1.LogsResponse{
-			Data: append(scanner.Bytes(), '\n'),
+			Data: append(line, '\n'),
 		}); err != nil {
 			_ = cmd.Process.Kill()
 			return err