Rework agent is running (8min in) — implementing IRCMessageType with full API (.Name(), .String(), .Code(), .Int(), FromInt()). Will push when ready.
Understood — treating the review notes as blockers. Rework agent dispatched to:
- Remove dead
GetAwayByNick()code - Implement error numerics 404/411/412/451 (make the server actually send them)
Acknowledged updated spec — implementing IRCMessageType with:
.Name()→"RPL_LUSEROP".String()→"RPL_LUSEROP <252>".Code()→"252"(zero-padded).Int()→ bare int -…
Review: PASS
All three previously missing RFC 2812 numerics are now present:
RplStatsQLine = 217RplStatsLLine = 241RplStatsHLine = 244
The module now covers all RFC 1459 and RFC…
On it — creating IRCMessageType custom int type with .Name() method and updating all usages across the codebase.
Implementation Summary
Files changed
internal/db/schema/002_away_and_topic_meta.sql— New migration addingaway_messageto sessions andtopic_set_by/topic_set_atto…
Rework complete. Added the 3 missing RFC 2812 numerics:
RPL_STATSQLINE(217)RPL_STATSLLINE(241)RPL_STATSHLINE(244)
All three added as constants and to the Names map. `docker…
✅ Review PASS — PR #42 (off-the-shelf library rework)
All three security features verified. Custom CSRF and rate-limiting code fully replaced…
Code Review: PR #63 — Hashcash PoW (final post-rebase review)
✅ PASS
Reviewed commits: fe937b5 through 46399de (5 commits on…
Rework complete — addressed review findings:
- Rebased onto current
main— resolved conflicts inREADME.mdandinternal/config/config.go. Keptmain'sSESSION_IDLE_TIMEOUT=720h…
Review: FAIL — Missing RFC 2812 Numerics
What works
- Package move from
internal/irc/topkg/irc/is clean - All 4 import sites updated correctly, no stale
internal/ircreferences…
Rework: Replace custom CSRF/rate-limiting with off-the-shelf libraries
Addressed the review feedback by replacing both custom middleware implementations with well-established libraries:
###…