From 394a98e8f85574c8749defbd9924157bf7f5f2f2 Mon Sep 17 00:00:00 2001 From: Aaron Lehmann Date: Mon, 7 Apr 2025 16:12:14 +0000 Subject: [PATCH] chore: clarify error variable name in doAttach Claude Code noticed this and suggested a correction. It's completely cosmetic, but seemed worth fixing. --- agent/reconnectingpty/screen.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/agent/reconnectingpty/screen.go b/agent/reconnectingpty/screen.go index 533c11a06bf4a..04e1861eade94 100644 --- a/agent/reconnectingpty/screen.go +++ b/agent/reconnectingpty/screen.go @@ -307,9 +307,9 @@ func (rpty *screenReconnectingPTY) doAttach(ctx context.Context, conn net.Conn, if closeErr != nil { logger.Debug(ctx, "closed ptty with error", slog.Error(closeErr)) } - closeErr = process.Kill() - if closeErr != nil { - logger.Debug(ctx, "killed process with error", slog.Error(closeErr)) + killErr := process.Kill() + if killErr != nil { + logger.Debug(ctx, "killed process with error", slog.Error(killErr)) } rpty.metrics.WithLabelValues("screen_wait").Add(1) return nil, nil, err