Freegle / Iznik / 4462

67%

master: 70%

Pull #164

circleci

fix: V1 parity for Received logs — byuser=NULL, text=messageid, reserved keyword

The first cut of the Received log used byuser=myid and text="", and
used an unquoted `user` column name. V1's Message::submit() writes
byuser NULL, text=messageid, and the Log class backticks column names.

- message.go: add logMessageReceived helper that writes byuser=NULL
  and text=messages.messageid. Backtick `user` in logModAction too
  since it's a MySQL reserved word.
- Call the helper from handleJoinAndPost and PutMessage.
- Backfill SQL: backtick `user`, use COALESCE(messageid, '') for text,
  pass NULL for byuser, exclude soft-deleted mg rows, and match the
  dedupe predicate on (msgid, groupid) so multi-group messages backfill
  once per group.
- Strengthen the JoinAndPost test to assert byuser IS NULL and text
  equals the originating messageid.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

8617 of 9983 branches covered (86.32%)

Branch coverage included in aggregate %.

14 of 20 new or added lines in 1 file covered. (70.0%)

82500 of 126017 relevant lines covered (65.47%)

14.16 hits per line