Question for sneak: is it right that mfer does not use sneak.berlin/go/simplelog?
The Go style guide says to import simplelog for sensible defaults. #141 (moving internal/log to log/slog, now on next) does not, because importing it, even blank, replaces slog's default handler in every program that imports package mfer: that handler writes to stdout (it would mix log lines into mfer export's JSON), prints JSON when stdout is not a terminal, and makes the program exit at startup when LOGGER_WEBHOOK_URL is malformed. Two reviewers checked these effects against the library's source.
Options:
A. (recommended) Keep it as landed: the library leaves slog's default alone, so programs that use mfer pick their own handler; the CLI uses its own small handler.
B. Import simplelog in cmd/mfer only; needs simplelog changed to write to stderr first, or mfer export output breaks.
Nothing waits on this; the change on next stands until you say otherwise.
Model: opus-5-5
Question for sneak: is it right that mfer does not use `sneak.berlin/go/simplelog`?
The Go style guide says to import `simplelog` for sensible defaults. https://git.eeqj.de/sneak/mfer/pulls/141 (moving `internal/log` to `log/slog`, now on `next`) does not, because importing it, even blank, replaces slog's default handler in every program that imports package `mfer`: that handler writes to stdout (it would mix log lines into `mfer export`'s JSON), prints JSON when stdout is not a terminal, and makes the program exit at startup when `LOGGER_WEBHOOK_URL` is malformed. Two reviewers checked these effects against the library's source.
Options:
A. (recommended) Keep it as landed: the library leaves slog's default alone, so programs that use `mfer` pick their own handler; the CLI uses its own small handler.
B. Import `simplelog` in `cmd/mfer` only; needs `simplelog` changed to write to stderr first, or `mfer export` output breaks.
Nothing waits on this; the change on `next` stands until you say otherwise.
Model: opus-5-5
sneak
was assigned by clawbot2026-10-04 16:32:10 +02:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Question for sneak: is it right that mfer does not use
sneak.berlin/go/simplelog?The Go style guide says to import
simplelogfor sensible defaults. #141 (movinginternal/logtolog/slog, now onnext) does not, because importing it, even blank, replaces slog's default handler in every program that imports packagemfer: that handler writes to stdout (it would mix log lines intomfer export's JSON), prints JSON when stdout is not a terminal, and makes the program exit at startup whenLOGGER_WEBHOOK_URLis malformed. Two reviewers checked these effects against the library's source.Options:
A. (recommended) Keep it as landed: the library leaves slog's default alone, so programs that use
mferpick their own handler; the CLI uses its own small handler.B. Import
simplelogincmd/mferonly; needssimplelogchanged to write to stderr first, ormfer exportoutput breaks.Nothing waits on this; the change on
nextstands until you say otherwise.Model: opus-5-5