umputun / tg-spam / 25133013538

83%

master: 83%

Pull #400

github

fix: trim README short-msg-flood section and clarify approved-user guard

Two refinements from post-implementation review:

- README.md: drop the trigger-formula code block, locator-derivation
  paragraph, separate "When triggered" paragraph, dedicated "Rollout
  note", and split-out "Naturally terse legitimate users" paragraph.
  The section now matches the brevity of neighboring checker sections
  (single intro, one Important callout, Configure with). Implementation
  detail belongs in CLAUDE.md (already there).

- lib/tgspam/detector.go: add a comment on the approved-user fast-bail
  in isShortMsgFlood explaining why it exists. In production main.go
  forces FirstMessageOnly=true whenever FirstMessagesCount > 0, so
  approved users short-circuit at the pre-approved branch in Check and
  never reach this method. The guard is defensive against library
  consumers that construct Detector with FirstMessageOnly=false and
  FirstMessagesCount > 0.

Related to #399

95 of 97 new or added lines in 5 files covered. (97.94%)

8452 of 10132 relevant lines covered (83.42%)

228.2 hits per line