3a2a045 · crush · git.iko.soy

git.iko.soy/crush

fix: improve logs, standardize capitalized (#2047)

* fix: improve logs, standarize capitalized Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> * Update Taskfile.yaml Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com> * chore: lint Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> --------- Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com> Co-authored-by: Andrey Nering <andreynering@users.noreply.github.com>

commit3a2a045c3edb8e53b36cb71951f91213e4c3fb5c authorCarlos Alexandro Becker <caarlos0@users.noreply.github.com> dateThu 29 Jan 2026, 21:12 (9 months ago) parentc3ae230

20 files changed+88−61

compare with main

unifiedsplit

AGENTS.md+2

@@ -26,6 +26,8 @@

2626 need of a temporary directory. This directory does not need to be removed.

2727- **JSON tags**: Use snake_case for JSON field names

2828- **File permissions**: Use octal notation (0o755, 0o644) for file permissions

29+- **Log messages**: Log messages must start with a capital letter (e.g., "Failed to save session" not "failed to save session")

30+ - This is enforced by `task lint:log` which runs as part of `task lint`

2931- **Comments**: End comments in periods unless comments are at the end of the line.

3032

3133## Testing with Mock Providers

Taskfile.yaml+6

@@ -23,10 +23,16 @@ tasks:

2323 lint:

2424 desc: Run base linters

2525 cmds:

26+ - task: lint:log

2627 - golangci-lint run --path-mode=abs --config=".golangci.yml" --timeout=5m

2728 env:

2829 GOEXPERIMENT: null

2930

31+ lint:log:

32+ desc: Check that log messages start with capital letters

33+ cmds:

34+ - ./scripts/check_log_capitalization.sh

35+

3036 lint:fix:

3137 desc: Run base linters and fix issues

3238 cmds:

internal/agent/agent.go+14−14

@@ -802,22 +802,22 @@ func (a *sessionAgent) generateTitle(ctx context.Context, sessionID string, user

802802 resp, err := agent.Stream(ctx, streamCall)

803803 if err == nil {

804804 // We successfully generated a title with the small model.

805− slog.Info("generated title with small model")

805+ slog.Debug("Generated title with small model")

806806 } else {

807807 // It didn't work. Let's try with the big model.

808− slog.Error("error generating title with small model; trying big model", "err", err)

808+ slog.Error("Error generating title with small model; trying big model", "err", err)

809809 model = largeModel

810810 agent = newAgent(model.Model, titlePrompt, maxOutputTokens)

811811 resp, err = agent.Stream(ctx, streamCall)

812812 if err == nil {

813− slog.Info("generated title with large model")

813+ slog.Debug("Generated title with large model")

814814 } else {

815815 // Welp, the large model didn't work either. Use the default

816816 // session name and return.

817− slog.Error("error generating title with large model", "err", err)

817+ slog.Error("Error generating title with large model", "err", err)

818818 saveErr := a.sessions.UpdateTitleAndUsage(ctx, sessionID, defaultSessionName, 0, 0, 0)

819819 if saveErr != nil {

820− slog.Error("failed to save session title and usage", "error", saveErr)

820+ slog.Error("Failed to save session title and usage", "error", saveErr)

821821 }

822822 return

823823 }

@@ -826,10 +826,10 @@ func (a *sessionAgent) generateTitle(ctx context.Context, sessionID string, user

826826 if resp == nil {

827827 // Actually, we didn't get a response so we can't. Use the default

828828 // session name and return.

829− slog.Error("response is nil; can't generate title")

829+ slog.Error("Response is nil; can't generate title")

830830 saveErr := a.sessions.UpdateTitleAndUsage(ctx, sessionID, defaultSessionName, 0, 0, 0)

831831 if saveErr != nil {

832− slog.Error("failed to save session title and usage", "error", saveErr)

832+ slog.Error("Failed to save session title and usage", "error", saveErr)

833833 }

834834 return

835835 }

@@ -843,7 +843,7 @@ func (a *sessionAgent) generateTitle(ctx context.Context, sessionID string, user

843843

844844 title = strings.TrimSpace(title)

845845 if title == "" {

846− slog.Warn("empty title; using fallback")

846+ slog.Debug("Empty title; using fallback")

847847 title = defaultSessionName

848848 }

849849

@@ -878,7 +878,7 @@ func (a *sessionAgent) generateTitle(ctx context.Context, sessionID string, user

878878 // concurrent session updates.

879879 saveErr := a.sessions.UpdateTitleAndUsage(ctx, sessionID, title, promptTokens, completionTokens, cost)

880880 if saveErr != nil {

881− slog.Error("failed to save session title and usage", "error", saveErr)

881+ slog.Error("Failed to save session title and usage", "error", saveErr)

882882 return

883883 }

884884}

@@ -921,25 +921,25 @@ func (a *sessionAgent) Cancel(sessionID string) {

921921 // fully completes (including error handling that may access the DB).

922922 // The defer in processRequest will clean up the entry.

923923 if cancel, ok := a.activeRequests.Get(sessionID); ok && cancel != nil {

924− slog.Info("Request cancellation initiated", "session_id", sessionID)

924+ slog.Debug("Request cancellation initiated", "session_id", sessionID)

925925 cancel()

926926 }

927927

928928 // Also check for summarize requests.

929929 if cancel, ok := a.activeRequests.Get(sessionID + "-summarize"); ok && cancel != nil {

930− slog.Info("Summarize cancellation initiated", "session_id", sessionID)

930+ slog.Debug("Summarize cancellation initiated", "session_id", sessionID)

931931 cancel()

932932 }

933933

934934 if a.QueuedPrompts(sessionID) > 0 {

935− slog.Info("Clearing queued prompts", "session_id", sessionID)

935+ slog.Debug("Clearing queued prompts", "session_id", sessionID)

936936 a.messageQueue.Del(sessionID)

937937 }

938938}

939939

940940func (a *sessionAgent) ClearQueue(sessionID string) {

941941 if a.QueuedPrompts(sessionID) > 0 {

942− slog.Info("Clearing queued prompts", "session_id", sessionID)

942+ slog.Debug("Clearing queued prompts", "session_id", sessionID)

943943 a.messageQueue.Del(sessionID)

944944 }

945945}

@@ -1099,7 +1099,7 @@ func (a *sessionAgent) workaroundProviderMediaLimitations(messages []fantasy.Mes

10991099 if media, ok := fantasy.AsToolResultOutputType[fantasy.ToolResultOutputContentMedia](toolResult.Output); ok {

11001100 decoded, err := base64.StdEncoding.DecodeString(media.Data)

11011101 if err != nil {

1102− slog.Warn("failed to decode media data", "error", err)

1102+ slog.Warn("Failed to decode media data", "error", err)

11031103 textParts = append(textParts, part)

11041104 continue

11051105 }

internal/agent/coordinator.go+6−6

@@ -151,7 +151,7 @@ func (c *coordinator) Run(ctx context.Context, sessionID string, prompt string,

151151 mergedOptions, temp, topP, topK, freqPenalty, presPenalty := mergeCallOptions(model, providerCfg)

152152

153153 if providerCfg.OAuthToken != nil && providerCfg.OAuthToken.IsExpired() {

154− slog.Info("Token needs to be refreshed", "provider", providerCfg.ID)

154+ slog.Debug("Token needs to be refreshed", "provider", providerCfg.ID)

155155 if err := c.refreshOAuth2Token(ctx, providerCfg); err != nil {

156156 return nil, err

157157 }

@@ -176,18 +176,18 @@ func (c *coordinator) Run(ctx context.Context, sessionID string, prompt string,

176176 if c.isUnauthorized(originalErr) {

177177 switch {

178178 case providerCfg.OAuthToken != nil:

179− slog.Info("Received 401. Refreshing token and retrying", "provider", providerCfg.ID)

179+ slog.Debug("Received 401. Refreshing token and retrying", "provider", providerCfg.ID)

180180 if err := c.refreshOAuth2Token(ctx, providerCfg); err != nil {

181181 return nil, originalErr

182182 }

183− slog.Info("Retrying request with refreshed OAuth token", "provider", providerCfg.ID)

183+ slog.Debug("Retrying request with refreshed OAuth token", "provider", providerCfg.ID)

184184 return run()

185185 case strings.Contains(providerCfg.APIKeyTemplate, "$"):

186− slog.Info("Received 401. Refreshing API Key template and retrying", "provider", providerCfg.ID)

186+ slog.Debug("Received 401. Refreshing API Key template and retrying", "provider", providerCfg.ID)

187187 if err := c.refreshApiKeyTemplate(ctx, providerCfg); err != nil {

188188 return nil, originalErr

189189 }

190− slog.Info("Retrying request with refreshed API key", "provider", providerCfg.ID)

190+ slog.Debug("Retrying request with refreshed API key", "provider", providerCfg.ID)

191191 return run()

192192 }

193193 }

@@ -428,7 +428,7 @@ func (c *coordinator) buildTools(ctx context.Context, agent config.Agent) ([]fan

428428 }

429429 if len(agent.AllowedMCP) == 0 {

430430 // No MCPs allowed

431− slog.Debug("no MCPs allowed", "tool", tool.Name(), "agent", agent.Name)

431+ slog.Debug("No MCPs allowed", "tool", tool.Name(), "agent", agent.Name)

432432 break

433433 }

434434

internal/agent/hyper/provider.go+1−1

@@ -49,7 +49,7 @@ var Enabled = sync.OnceValue(func() bool {

4949var Embedded = sync.OnceValue(func() catwalk.Provider {

5050 var provider catwalk.Provider

5151 if err := json.Unmarshal(embedded, &provider); err != nil {

52− slog.Error("could not use embedded provider data", "err", err)

52+ slog.Error("Could not use embedded provider data", "err", err)

5353 }

5454 return provider

5555})

internal/agent/tools/mcp/init.go+6−6

@@ -140,7 +140,7 @@ func Initialize(ctx context.Context, permissions permission.Service, cfg *config

140140 for name, m := range cfg.MCP {

141141 if m.Disabled {

142142 updateState(name, StateDisabled, nil, nil, Counts{})

143− slog.Debug("skipping disabled mcp", "name", name)

143+ slog.Debug("Skipping disabled MCP", "name", name)

144144 continue

145145 }

146146

@@ -162,7 +162,7 @@ func Initialize(ctx context.Context, permissions permission.Service, cfg *config

162162 err = fmt.Errorf("panic: %v", v)

163163 }

164164 updateState(name, StateError, err, nil, Counts{})

165− slog.Error("panic in mcp client initialization", "error", err, "name", name)

165+ slog.Error("Panic in MCP client initialization", "error", err, "name", name)

166166 }

167167 }()

168168

@@ -174,7 +174,7 @@ func Initialize(ctx context.Context, permissions permission.Service, cfg *config

174174

175175 tools, err := getTools(ctx, session)

176176 if err != nil {

177− slog.Error("error listing tools", "error", err)

177+ slog.Error("Error listing tools", "error", err)

178178 updateState(name, StateError, err, nil, Counts{})

179179 session.Close()

180180 return

@@ -182,7 +182,7 @@ func Initialize(ctx context.Context, permissions permission.Service, cfg *config

182182

183183 prompts, err := getPrompts(ctx, session)

184184 if err != nil {

185− slog.Error("error listing prompts", "error", err)

185+ slog.Error("Error listing prompts", "error", err)

186186 updateState(name, StateError, err, nil, Counts{})

187187 session.Close()

188188 return

@@ -277,7 +277,7 @@ func createSession(ctx context.Context, name string, m config.MCPConfig, resolve

277277 transport, err := createTransport(mcpCtx, m, resolver)

278278 if err != nil {

279279 updateState(name, StateError, err, nil, Counts{})

280− slog.Error("error creating mcp client", "error", err, "name", name)

280+ slog.Error("Error creating MCP client", "error", err, "name", name)

281281 cancel()

282282 cancelTimer.Stop()

283283 return nil, err

@@ -319,7 +319,7 @@ func createSession(ctx context.Context, name string, m config.MCPConfig, resolve

319319 }

320320

321321 cancelTimer.Stop()

322− slog.Info("MCP client initialized", "name", name)

322+ slog.Debug("MCP client initialized", "name", name)

323323 return session, nil

324324}

325325

internal/agent/tools/mcp/prompts.go+1−1

@@ -49,7 +49,7 @@ func GetPromptMessages(ctx context.Context, clientName, promptName string, args

4949func RefreshPrompts(ctx context.Context, name string) {

5050 session, ok := sessions.Get(name)

5151 if !ok {

52− slog.Warn("refresh prompts: no session", "name", name)

52+ slog.Warn("Refresh prompts: no session", "name", name)

5353 return

5454 }

5555

internal/agent/tools/mcp/tools.go+1−1

@@ -111,7 +111,7 @@ func RunTool(ctx context.Context, name, toolName string, input string) (ToolResu

111111func RefreshTools(ctx context.Context, name string) {

112112 session, ok := sessions.Get(name)

113113 if !ok {

114− slog.Warn("refresh tools: no session", "name", name)

114+ slog.Warn("Refresh tools: no session", "name", name)

115115 return

116116 }

117117

internal/app/app.go+15−11

@@ -135,7 +135,7 @@ func (app *App) Config() *config.Config {

135135

136136// RunNonInteractive runs the application in non-interactive mode with the

137137// given prompt, printing to stdout.

138−func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt, largeModel, smallModel string, quiet bool) error {

138+func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt, largeModel, smallModel string, hideSpinner bool) error {

139139 slog.Info("Running in non-interactive mode")

140140

141141 ctx, cancel := context.WithCancel(ctx)

@@ -160,10 +160,9 @@ func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt,

160160 }

161161 stderrTTY = term.IsTerminal(os.Stderr.Fd())

162162 stdinTTY = term.IsTerminal(os.Stdin.Fd())

163−

164163 progress = app.config.Options.Progress == nil || *app.config.Options.Progress

165164

166− if !quiet && stderrTTY {

165+ if !hideSpinner && stderrTTY {

167166 t := styles.CurrentTheme()

168167

169168 // Detect background color to set the appropriate color for the

@@ -188,7 +187,7 @@ func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt,

188187

189188 // Helper function to stop spinner once.

190189 stopSpinner := func() {

191− if !quiet && spinner != nil {

190+ if !hideSpinner && spinner != nil {

192191 spinner.Stop()

193192 spinner = nil

194193 }

@@ -245,6 +244,7 @@ func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt,

245244

246245 messageEvents := app.Messages.Subscribe(ctx)

247246 messageReadBytes := make(map[string]int)

247+ var printed bool

248248

249249 defer func() {

250250 if progress && stderrTTY {

@@ -268,7 +268,7 @@ func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt,

268268 stopSpinner()

269269 if result.err != nil {

270270 if errors.Is(result.err, context.Canceled) || errors.Is(result.err, agent.ErrRequestCancelled) {

271− slog.Info("Non-interactive: agent processing cancelled", "session_id", sess.ID)

271+ slog.Debug("Non-interactive: agent processing cancelled", "session_id", sess.ID)

272272 return nil

273273 }

274274 return fmt.Errorf("agent processing failed: %w", result.err)

@@ -294,7 +294,11 @@ func (app *App) RunNonInteractive(ctx context.Context, output io.Writer, prompt,

294294 if readBytes == 0 {

295295 part = strings.TrimLeft(part, " \t")

296296 }

297− fmt.Fprint(output, part)

297+ // Ignore initial whitespace-only messages.

298+ if printed || strings.TrimSpace(part) != "" {

299+ printed = true

300+ fmt.Fprint(output, part)

301+ }

298302 messageReadBytes[msg.ID] = len(content)

299303 }

300304

@@ -433,20 +437,20 @@ func setupSubscriber[T any](

433437 select {

434438 case event, ok := <-subCh:

435439 if !ok {

436− slog.Debug("subscription channel closed", "name", name)

440+ slog.Debug("Subscription channel closed", "name", name)

437441 return

438442 }

439443 var msg tea.Msg = event

440444 select {

441445 case outputCh <- msg:

442446 case <-time.After(2 * time.Second):

443− slog.Warn("message dropped due to slow consumer", "name", name)

447+ slog.Debug("Message dropped due to slow consumer", "name", name)

444448 case <-ctx.Done():

445− slog.Debug("subscription cancelled", "name", name)

449+ slog.Debug("Subscription cancelled", "name", name)

446450 return

447451 }

448452 case <-ctx.Done():

449− slog.Debug("subscription cancelled", "name", name)

453+ slog.Debug("Subscription cancelled", "name", name)

450454 return

451455 }

452456 }

@@ -511,7 +515,7 @@ func (app *App) Subscribe(program *tea.Program) {

511515// Shutdown performs a graceful shutdown of the application.

512516func (app *App) Shutdown() {

513517 start := time.Now()

514− defer func() { slog.Info("Shutdown took " + time.Since(start).String()) }()

518+ defer func() { slog.Debug("Shutdown took " + time.Since(start).String()) }()

515519

516520 // First, cancel all agents and wait for them to finish. This must complete

517521 // before closing the DB so agents can finish writing their state.

internal/app/lsp.go+1−1

@@ -140,7 +140,7 @@ func (app *App) createAndStartLSPClient(ctx context.Context, name string, config

140140 updateLSPState(name, lsp.StateReady, nil, lspClient, 0)

141141 }

142142

143− slog.Info("LSP client initialized", "name", name)

143+ slog.Debug("LSP client initialized", "name", name)

144144

145145 // Add to map with mutex protection before starting goroutine

146146 app.LSPClients.Set(name, lspClient)

internal/cmd/run.go+11−1

@@ -8,6 +8,7 @@ import (

88 "os/signal"

99 "strings"

1010

11+ "charm.land/log/v2"

1112 "github.com/charmbracelet/crush/internal/event"

1213 "github.com/spf13/cobra"

1314)

@@ -29,9 +30,13 @@ crush run "What is this code doing?" <<< prrr.go

2930

3031# Run in quiet mode (hide the spinner)

3132crush run --quiet "Generate a README for this project"

33+

34+# Run in verbose mode

35+crush run --verbose "Generate a README for this project"

3236 `,

3337 RunE: func(cmd *cobra.Command, args []string) error {

3438 quiet, _ := cmd.Flags().GetBool("quiet")

39+ verbose, _ := cmd.Flags().GetBool("verbose")

3540 largeModel, _ := cmd.Flags().GetString("model")

3641 smallModel, _ := cmd.Flags().GetString("small-model")

3742

@@ -49,6 +54,10 @@ crush run --quiet "Generate a README for this project"

4954 return fmt.Errorf("no providers configured - please run 'crush' to set up a provider interactively")

5055 }

5156

57+ if verbose {

58+ slog.SetDefault(slog.New(log.New(os.Stderr)))

59+ }

60+

5261 prompt := strings.Join(args, " ")

5362

5463 prompt, err = MaybePrependStdin(prompt)

@@ -64,7 +73,7 @@ crush run --quiet "Generate a README for this project"

6473 event.SetNonInteractive(true)

6574 event.AppInitialized()

6675

67− return app.RunNonInteractive(ctx, os.Stdout, prompt, largeModel, smallModel, quiet)

76+ return app.RunNonInteractive(ctx, os.Stdout, prompt, largeModel, smallModel, quiet || verbose)

6877 },

6978 PostRun: func(cmd *cobra.Command, args []string) {

7079 event.AppExited()

@@ -73,6 +82,7 @@ crush run --quiet "Generate a README for this project"

7382

7483func init() {

7584 runCmd.Flags().BoolP("quiet", "q", false, "Hide spinner")

85+ runCmd.Flags().BoolP("verbose", "v", false, "Show logs")

7686 runCmd.Flags().StringP("model", "m", "", "Model to use. Accepts 'model' or 'provider/model' to disambiguate models with the same name across providers")

7787 runCmd.Flags().String("small-model", "", "Small model to use. If not provided, uses the default small model for the provider")

7888}

internal/config/config.go+2−2

@@ -317,7 +317,7 @@ func (m MCPConfig) ResolvedHeaders() map[string]string {

317317 var err error

318318 m.Headers[e], err = resolver.ResolveValue(v)

319319 if err != nil {

320− slog.Error("error resolving header variable", "error", err, "variable", e, "value", v)

320+ slog.Error("Error resolving header variable", "error", err, "variable", e, "value", v)

321321 continue

322322 }

323323 }

@@ -840,7 +840,7 @@ func resolveEnvs(envs map[string]string) []string {

840840 var err error

841841 envs[e], err = resolver.ResolveValue(v)

842842 if err != nil {

843− slog.Error("error resolving environment variable", "error", err, "variable", e, "value", v)

843+ slog.Error("Error resolving environment variable", "error", err, "variable", e, "value", v)

844844 continue

845845 }

846846 }

internal/fsext/ls.go+6−6

@@ -144,20 +144,20 @@ func (dl *directoryLister) shouldIgnore(path string, ignorePatterns []string) bo

144144 }

145145

146146 if commonIgnorePatterns().MatchesPath(relPath) {

147− slog.Debug("ignoring common pattern", "path", relPath)

147+ slog.Debug("Ignoring common pattern", "path", relPath)

148148 return true

149149 }

150150

151151 parentDir := filepath.Dir(path)

152152 ignoreParser := dl.getIgnore(parentDir)

153153 if ignoreParser.MatchesPath(relPath) {

154− slog.Debug("ignoring dir pattern", "path", relPath, "dir", parentDir)

154+ slog.Debug("Ignoring dir pattern", "path", relPath, "dir", parentDir)

155155 return true

156156 }

157157

158158 // For directories, also check with trailing slash (gitignore convention)

159159 if ignoreParser.MatchesPath(relPath + "/") {

160− slog.Debug("ignoring dir pattern with slash", "path", relPath+"/", "dir", parentDir)

160+ slog.Debug("Ignoring dir pattern with slash", "path", relPath+"/", "dir", parentDir)

161161 return true

162162 }

163163

@@ -166,7 +166,7 @@ func (dl *directoryLister) shouldIgnore(path string, ignorePatterns []string) bo

166166 }

167167

168168 if homeIgnore().MatchesPath(relPath) {

169− slog.Debug("ignoring home dir pattern", "path", relPath)

169+ slog.Debug("Ignoring home dir pattern", "path", relPath)

170170 return true

171171 }

172172

@@ -177,7 +177,7 @@ func (dl *directoryLister) checkParentIgnores(path string) bool {

177177 parent := filepath.Dir(filepath.Dir(path))

178178 for parent != "." && path != "." {

179179 if dl.getIgnore(parent).MatchesPath(path) {

180− slog.Debug("ingoring parent dir pattern", "path", path, "dir", parent)

180+ slog.Debug("Ignoring parent dir pattern", "path", path, "dir", parent)

181181 return true

182182 }

183183 if parent == dl.rootPath {

@@ -210,7 +210,7 @@ func ListDirectory(initialPath string, ignorePatterns []string, depth, limit int

210210 found := csync.NewSlice[string]()

211211 dl := NewDirectoryLister(initialPath)

212212

213− slog.Debug("listing directory", "path", initialPath, "depth", depth, "limit", limit, "ignorePatterns", ignorePatterns)

213+ slog.Debug("Listing directory", "path", initialPath, "depth", depth, "limit", limit, "ignorePatterns", ignorePatterns)

214214

215215 conf := fastwalk.Config{

216216 Follow: true,

internal/home/home.go+1−1

@@ -12,7 +12,7 @@ var homedir, homedirErr = os.UserHomeDir()

1212

1313func init() {

1414 if homedirErr != nil {

15− slog.Error("failed to get user home directory", "error", homedirErr)

15+ slog.Error("Failed to get user home directory", "error", homedirErr)

1616 }

1717}

1818

internal/lsp/client.go+4−4

@@ -317,12 +317,12 @@ func (c *Client) HandlesFile(path string) bool {

317317 // Check if file is within working directory.

318318 absPath, err := filepath.Abs(path)

319319 if err != nil {

320− slog.Debug("cannot resolve path", "name", c.name, "file", path, "error", err)

320+ slog.Debug("Cannot resolve path", "name", c.name, "file", path, "error", err)

321321 return false

322322 }

323323 relPath, err := filepath.Rel(c.workDir, absPath)

324324 if err != nil || strings.HasPrefix(relPath, "..") {

325− slog.Debug("file outside workspace", "name", c.name, "file", path, "workDir", c.workDir)

325+ slog.Debug("File outside workspace", "name", c.name, "file", path, "workDir", c.workDir)

326326 return false

327327 }

328328

@@ -339,11 +339,11 @@ func (c *Client) HandlesFile(path string) bool {

339339 suffix = "." + suffix

340340 }

341341 if strings.HasSuffix(name, suffix) || filetype == string(kind) {

342− slog.Debug("handles file", "name", c.name, "file", name, "filetype", filetype, "kind", kind)

342+ slog.Debug("Handles file", "name", c.name, "file", name, "filetype", filetype, "kind", kind)

343343 return true

344344 }

345345 }

346− slog.Debug("doesn't handle file", "name", c.name, "file", name)

346+ slog.Debug("Doesn't handle file", "name", c.name, "file", name)

347347 return false

348348}

349349

internal/session/session.go+1−1

@@ -203,7 +203,7 @@ func (s *service) List(ctx context.Context) ([]Session, error) {

203203func (s service) fromDBItem(item db.Session) Session {

204204 todos, err := unmarshalTodos(item.Todos.String)

205205 if err != nil {

206− slog.Error("failed to unmarshal todos", "session_id", item.ID, "error", err)

206+ slog.Error("Failed to unmarshal todos", "session_id", item.ID, "error", err)

207207 }

208208 return Session{

209209 ID: item.ID,

internal/ui/image/image.go+1−1

@@ -168,7 +168,7 @@ func (e Encoding) Transmit(id string, img image.Image, cs CellSize, cols, rows i

168168 return chunk

169169 },

170170 }); err != nil {

171− slog.Error("failed to encode image for kitty graphics", "err", err)

171+ slog.Error("Failed to encode image for kitty graphics", "err", err)

172172 return uiutil.InfoMsg{

173173 Type: uiutil.InfoTypeError,

174174 Msg: "failed to encode image",

internal/ui/model/history.go+1−1

@@ -27,7 +27,7 @@ func (m *UI) loadPromptHistory() tea.Cmd {

2727 messages, err = m.com.App.Messages.ListAllUserMessages(ctx)

2828 }

2929 if err != nil {

30− slog.Error("failed to load prompt history", "error", err)

30+ slog.Error("Failed to load prompt history", "error", err)

3131 return promptHistoryLoadedMsg{messages: nil}

3232 }

3333

internal/ui/model/ui.go+3−3

@@ -330,7 +330,7 @@ func (m *UI) loadCustomCommands() tea.Cmd {

330330 return func() tea.Msg {

331331 customCommands, err := commands.LoadCustomCommands(m.com.Config())

332332 if err != nil {

333− slog.Error("failed to load custom commands", "error", err)

333+ slog.Error("Failed to load custom commands", "error", err)

334334 }

335335 return userCommandsLoadedMsg{Commands: customCommands}

336336 }

@@ -341,7 +341,7 @@ func (m *UI) loadMCPrompts() tea.Cmd {

341341 return func() tea.Msg {

342342 prompts, err := commands.LoadMCPPrompts()

343343 if err != nil {

344− slog.Error("failed to load mcp prompts", "error", err)

344+ slog.Error("Failed to load MCP prompts", "error", err)

345345 }

346346 if prompts == nil {

347347 // flag them as loaded even if there is none or an error

@@ -683,7 +683,7 @@ func (m *UI) Update(msg tea.Msg) (tea.Model, tea.Cmd) {

683683 }

684684 case uv.KittyGraphicsEvent:

685685 if !bytes.HasPrefix(msg.Payload, []byte("OK")) {

686− slog.Warn("unexpected Kitty graphics response",

686+ slog.Warn("Unexpected Kitty graphics response",

687687 "response", string(msg.Payload),

688688 "options", msg.Options)

689689 }

scripts/check_log_capitalization.shnew file+5

@@ -0,0 +1,5 @@

1+#!/bin/bash

3+ echo "❌ Log messages must start with a capital letter. Found lowercase logs above."

4+ exit 1

5+fi