From 2e3a00f22680410d01351c3bef89268838168d9a Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 29 Apr 2026 05:20:11 +0000 Subject: [PATCH 1/2] Replace standard log calls with logHealth in health_monitor.go The health_monitor.go file declared logHealth = logger.New("launcher:health") but two functions (Start and handleErrorState) still used the standard library log.Printf instead of the project's structured logger. - Replace log.Printf("[HEALTH] Starting...") with logHealth.Printf - Replace log.Printf("[HEALTH] Successfully restarted...") with logHealth.Printf - Remove now-unused "log" standard library import This makes logging consistent with the rest of the file and the project's debug logging conventions described in AGENTS.md. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- internal/launcher/health_monitor.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/internal/launcher/health_monitor.go b/internal/launcher/health_monitor.go index 4bfd42051..864d3f5b0 100644 --- a/internal/launcher/health_monitor.go +++ b/internal/launcher/health_monitor.go @@ -1,7 +1,6 @@ package launcher import ( - "log" "time" "github.com/github/gh-aw-mcpg/internal/logger" @@ -47,7 +46,7 @@ func NewHealthMonitor(l *Launcher, interval time.Duration) *HealthMonitor { // Start begins periodic health checks in a background goroutine. func (hm *HealthMonitor) Start() { - log.Printf("[HEALTH] Starting health monitor (interval=%s)", hm.interval) + logHealth.Printf("Starting health monitor: interval=%v", hm.interval) logger.LogInfo("startup", "Health monitor started (interval=%s)", hm.interval) go hm.run() } @@ -129,6 +128,6 @@ func (hm *HealthMonitor) handleErrorState(serverID string, state ServerState) { } hm.consecutiveFailures[serverID] = 0 - log.Printf("[HEALTH] Successfully restarted server %q", serverID) + logHealth.Printf("Successfully restarted server: serverID=%s", serverID) logger.LogInfo("backend", "Health check: successfully restarted server %q", serverID) } From bd0041c694002d203f1be2b2e24edb3fccb7b16f Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Wed, 29 Apr 2026 08:09:29 -0700 Subject: [PATCH 2/2] Update internal/launcher/health_monitor.go Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- internal/launcher/health_monitor.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/launcher/health_monitor.go b/internal/launcher/health_monitor.go index 864d3f5b0..1bd02b5a8 100644 --- a/internal/launcher/health_monitor.go +++ b/internal/launcher/health_monitor.go @@ -47,7 +47,7 @@ func NewHealthMonitor(l *Launcher, interval time.Duration) *HealthMonitor { // Start begins periodic health checks in a background goroutine. func (hm *HealthMonitor) Start() { logHealth.Printf("Starting health monitor: interval=%v", hm.interval) - logger.LogInfo("startup", "Health monitor started (interval=%s)", hm.interval) + logger.LogInfo("startup", "Health monitor started (interval=%v)", hm.interval) go hm.run() }