umputun / remark42 / 19895196749

84%

master: 84%

Build Type

Pull #1919

github

Committed by paskal

Commit Message

Get rid of github.com/go-chi/render use

Replace go-chi/render with go-pkgz/rest for JSON responses and custom
helpers for HTML/plain text responses.

Key changes:
- Replace render.JSON/render.Status with rest.RenderJSON and explicit
  w.WriteHeader() calls
- Replace render.DecodeJSON with json.NewDecoder().Decode()
- Add SendErrorJSON helper that sets Content-Type header before
  WriteHeader (required since rest.RenderJSON can't set headers after
  WriteHeader is called)
- Add HTMLResponse and PlainTextResponse helpers

Fix export double-execution in migrator.go:
The original code called Export twice - once to io.Discard to check for
errors, then again to actually write. This was wasteful and had a race
condition risk. Now file mode buffers to memory first for atomic
success/failure, while stream mode writes directly with proper error
handling.

Pull Request Pull Request #1919: Get rid of github.com/go-chi/render use

Run Details

70 of 79 new or added lines in 6 files covered. (88.61%)

3 existing lines in 2 files now uncovered.

6053 of 7173 relevant lines covered (84.39%)

33.78 hits per line