- Restore winnerScoreChanged guard on winnerDiscordUserId so advancing-only
winners are shown in the embed but not Discord-pinged
- Add display-name filter at the end of the scoredMatches chain so
scoredMatches always reflects exactly what appears in the embed, eliminating
the dual-gate split between scoring-calculator.ts and discord.ts
- Remove isMatchNotable export and inline x.winnerScoreChanged || x.showLoser
at its sole call site; move the reasoning into the existing comment block
Co-Authored-By: Claude <noreply@anthropic.com>
When a match is displayed because the loser was eliminated (0-pt), the winner's
manager was previously omitted if the winner only advanced without earning points.
Now the winner's owner is always credited on any match that passes the notability
gate, while the gate itself (winnerScoreChanged || loserNotifiable) still prevents
purely-advancing matches from appearing.
Adds isMatchNotable() pure helper alongside isLoserNotifiable() to make the
display rule testable in isolation.
Co-Authored-By: Claude <noreply@anthropic.com>
