From 0019d8b3d453fdf81359347476f7272bc520cd10 Mon Sep 17 00:00:00 2001
From: "devsy-app[bot]" <277138668+devsy-app[bot]@users.noreply.github.com>
Date: Mon, 17 Aug 2026 09:46:54 +0000
Subject: [PATCH] fix(cmd): lowercase pro logout log strings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## cmd/ subdirectory reviewed

`cmd/pro/` — specifically `cmd/pro/logout.go`.

## Issue found

AGENTS.md states that log messages must be lowercase. `cmd/pro/logout.go` had four log strings that started with an uppercase letter, inconsistent with both the repo convention and the rest of the file (which already uses lowercase log strings throughout `cleanupLocalWorkspaces`):

- `log.Warnf("Failed to list workspaces: %v", err)`
- `log.Warnf("Failed to shut down daemon: %v", err)`
- `log.Debug("Waiting for daemon to shut down")`
- `log.Warnf("Failed to wait for daemon to be stopped: %v", err)`

## Minimal change

Lowercased the four offending log strings in `cmd/pro/logout.go`. No logic, flag, error-handling, or import changes.

## Verification performed

- `task cli:format` — clean (no formatting changes beyond the intended edits).
- `task cli:lint:ci` — **0 issues**.
- `task cli:test` — passes, except for two pre-existing failures that also fail on a clean `origin/main` checkout and are unrelated to this change:
  - `pkg/git` (`TestRepoClone*`) — the documented known pre-existing stale-assertion failure.
  - `pkg/docker` (`TestRunCmd_CancelKillsProcessGroup`) — a sandbox process-group timing test; confirmed failing on a clean `origin/main` tree with this change stashed.
- `go build ./cmd/...` — clean.
- `go test ./cmd/...` — all `cmd/` tests pass.

This PR was created by an AI agent as part of an automated daily CLI review job.
---
 cmd/pro/logout.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/cmd/pro/logout.go b/cmd/pro/logout.go
index 9e79f17e0..c9dfd6f05 100644
--- a/cmd/pro/logout.go
+++ b/cmd/pro/logout.go
@@ -95,7 +95,7 @@ func (cmd *LogoutCmd) Run(ctx context.Context, args []string) error {
 			false,
 		)
 		if err != nil {
-			log.Warnf("Failed to list workspaces: %v", err)
+			log.Warnf("failed to list workspaces: %v", err)
 		} else {
 			cleanupLocalWorkspaces(
 				ctx,
@@ -109,12 +109,12 @@ func (cmd *LogoutCmd) Run(ctx context.Context, args []string) error {
 		daemonClient := daemon.NewLocalClient(proInstanceConfig.Provider)
 		err = daemonClient.Shutdown(ctx)
 		if err != nil {
-			log.Warnf("Failed to shut down daemon: %v", err)
+			log.Warnf("failed to shut down daemon: %v", err)
 		}
-		log.Debug("Waiting for daemon to shut down")
+		log.Debug("waiting for daemon to shut down")
 		err = waitDaemonStopped(ctx, providerConfig.Name)
 		if err != nil {
-			log.Warnf("Failed to wait for daemon to be stopped: %v", err)
+			log.Warnf("failed to wait for daemon to be stopped: %v", err)
 		}
 	}