radio-t / super-bot / 32317846857

77%

push

github

Apply go fix (#162)

`go fix` in Go 1.26 runs the modernizer analysers, so it rewrites
`strings.HasPrefix`/`TrimPrefix` pairs into `strings.CutPrefix`,
`sort.Slice` into `slices.Sort`, counted `for` loops into `for range n`,
`strings.Fields` into `strings.FieldsSeq`, string accumulation into
`strings.Builder`, and per-test contexts into `t.Context()`.

It also drops `omitempty` from the two struct-typed fields of
`bot.Message`, where the option has never had any effect: encoding/json
treats only false, 0, nil pointers and empty arrays, slices, maps and
strings as empty, so a struct is always encoded. The marshalled form of
the message log written in `app/reporter/reporter.go` and read back in
`app/reporter/export.go` is unchanged, and existing logs still parse.
The alternative `omitzero` rewrite, which would change behaviour, is
deliberately not applied.

The `strings.Builder` rewrites left two `WriteString(fmt.Sprintf(...))`
calls that staticcheck flags as QF1012; they are written as
`fmt.Fprintf` instead.

Co-authored-by: Umputun <umputun@gmail.com>

23 of 23 new or added lines in 7 files covered. (100.0%)

2709 of 3607 relevant lines covered (75.1%)

54.23 hits per line