main.go error-color split breaks on multi-line error bodies #603

New issue

Closed

opened 2026-08-08 08:23:08 +00:00 by the.auditor · 1 comment

the.auditor commented

2026-08-08 08:23:08 +00:00

Owner

main() in cmd/tpagectl/main.go colors the first line of clierr.Render(err) red by splitting on the first \n. When the friendly status detail embeds a multi-line response body (a 5xx/401 with an indented or multi-line body), the split lands mid-error: only the first fragment gets the red style and the hint: line is folded into the remaining text via the second Fprintln. All text is still printed, but the styling and hint separation are mangled.

Split on the hint: marker (or color the whole message) instead of the first newline.

See #602

`main()` in `cmd/tpagectl/main.go` colors the first line of `clierr.Render(err)` red by splitting on the first `\n`. When the friendly status detail embeds a multi-line response body (a 5xx/401 with an indented or multi-line body), the split lands mid-error: only the first fragment gets the red style and the `hint:` line is folded into the remaining text via the second `Fprintln`. All text is still printed, but the styling and hint separation are mangled. Split on the `hint:` marker (or color the whole message) instead of the first newline. See https://git.lan.thwap.org/thwap/thwap-pagesd/pulls/602

the.auditor added the

Kind/Bug

Priority

Low

labels

2026-08-08 08:23:11 +00:00

fuzzy referenced this issue from a commit

2026-08-11 01:29:27 +00:00

fix(tpagectl): address CLI issues

automation referenced this issue

2026-08-11 01:29:32 +00:00

fix(tpagectl): address CLI issues #634

automation closed this issue

2026-08-11 01:29:47 +00:00

automation commented

2026-08-11 01:29:47 +00:00

Owner

Fixed in #634: main() colors only the leading error: prefix and prints the whole rendered message unchanged, so multi-line bodies and hint: lines are no longer mangled.

Fixed in #634: `main()` colors only the leading `error:` prefix and prints the whole rendered message unchanged, so multi-line bodies and `hint:` lines are no longer mangled.

fuzzy referenced this issue from a commit

2026-08-11 01:29:56 +00:00

fix(tpagectl): address CLI issues