The log level is hardcoded to slog.LevelInfo in cmd/main.go:26. Operators cannot enable debug logging to troubleshoot without rebuilding the binary.
Proposed scope
- Add a
log_level field (LogLevel string) to Config in pkg/config/config.go, bound to NB_LOG_LEVEL (via v.BindEnv + v.SetDefault("log_level", "info")).
- Add a small helper
parseLogLevel(string) slog.Level supporting debug / info / warn / error (case-insensitive). Unknown value → fall back to info and log a warning.
- Use the parsed level in the
slog.NewJSONHandler options in cmd/main.go.
Acceptance criteria
- Unit test for
parseLogLevel covering each valid level plus an invalid value.
NB_LOG_LEVEL=debug produces debug output; default behavior (info) unchanged.- New
log_level key documented in docs/configuration.md.
Pointers
cmd/main.go:25-28 (handler construction)pkg/config/config.go (config struct + Load bindings)
Good first issue: self-contained, no live infrastructure needed.
The log level is hardcoded to
slog.LevelInfoincmd/main.go:26. Operators cannot enable debug logging to troubleshoot without rebuilding the binary.Proposed scope
log_levelfield (LogLevel string) toConfiginpkg/config/config.go, bound toNB_LOG_LEVEL(viav.BindEnv+v.SetDefault("log_level", "info")).parseLogLevel(string) slog.Levelsupportingdebug/info/warn/error(case-insensitive). Unknown value → fall back toinfoand log a warning.slog.NewJSONHandleroptions incmd/main.go.Acceptance criteria
parseLogLevelcovering each valid level plus an invalid value.NB_LOG_LEVEL=debugproduces debug output; default behavior (info) unchanged.log_levelkey documented indocs/configuration.md.Pointers
cmd/main.go:25-28(handler construction)pkg/config/config.go(config struct +Loadbindings)Good first issue: self-contained, no live infrastructure needed.