Logger should use slog.LevelVar for dynamic level switching #8

New Issue

clawbot · 2026-03-01T19:07:35+01:00

clawbot commented

2026-03-01 19:07:35 +01:00

Per GO_HTTP_SERVER_CONVENTIONS §9 (Logger Setup with TTY Detection), the logger should use slog.LevelVar to allow dynamic log level changes without recreating the handler:

l.level = new(slog.LevelVar)
l.level.Set(slog.LevelInfo)
// ...
opts := &slog.HandlerOptions{Level: l.level}

Currently, internal/logger/logger.go EnableDebugLogging() recreates the entire slog.Handler and slog.Logger just to change the level. This means any component that cached the *slog.Logger from Get() before EnableDebugLogging() was called will still use the old info-level logger.

Using slog.LevelVar allows changing the level atomically without replacing the handler, and all existing references to the logger automatically respect the new level.

Per GO_HTTP_SERVER_CONVENTIONS §9 (Logger Setup with TTY Detection), the logger should use `slog.LevelVar` to allow dynamic log level changes without recreating the handler: ```go l.level = new(slog.LevelVar) l.level.Set(slog.LevelInfo) // ... opts := &slog.HandlerOptions{Level: l.level} ``` Currently, `internal/logger/logger.go` `EnableDebugLogging()` recreates the entire `slog.Handler` and `slog.Logger` just to change the level. This means any component that cached the `*slog.Logger` from `Get()` before `EnableDebugLogging()` was called will still use the old info-level logger. Using `slog.LevelVar` allows changing the level atomically without replacing the handler, and all existing references to the logger automatically respect the new level.

clawbot added the

bot

label 2026-03-01 19:07:35 +01:00

clawbot referenced this issue

2026-03-01 19:12:13 +01:00

docs: comprehensive README rewrite with complete service specification #13

clawbot referenced this issue

2026-03-02 00:38:37 +01:00

1.0/mvp #15

sneak referenced this issue from a commit

2026-03-02 00:43:56 +01:00

docs: comprehensive README rewrite with complete service specification (#13)