From 70c585e8b23e9b600a220d95dec191a8c2e1736e Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 20 Dec 2025 17:49:21 +0000 Subject: [PATCH 1/2] Initial plan From 0401d3ecad54ef7c8511777bbe72dfebfc4dd5d0 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 20 Dec 2025 17:58:43 +0000 Subject: [PATCH 2/2] Fix golangci-lint gosimple violation in update_check.go Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- pkg/cli/update_check.go | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/pkg/cli/update_check.go b/pkg/cli/update_check.go index 5aef17ebaa1..e458eeb7063 100644 --- a/pkg/cli/update_check.go +++ b/pkg/cli/update_check.go @@ -92,12 +92,7 @@ func shouldCheckForUpdate(noCheckUpdate bool) bool { // This is a heuristic - we can't reliably detect this, so we're conservative func isRunningAsMCPServer() bool { // Check for MCP_SERVER environment variable that could be set by the MCP server - if os.Getenv("GH_AW_MCP_SERVER") != "" { - return true - } - // Additional heuristic: check if we're likely being invoked by MCP server - // MCP server tools typically run with minimal environment - return false + return os.Getenv("GH_AW_MCP_SERVER") != "" } var (