Closing: completed by PR #59 (IRC numerics batch 2 — internal/irc/ package with 60+ named constants, command constants, reply format with code/command…
Rework: IRC Numerics Package + Reply Format + Dead Code Fix + Constant Deduplication
All FOUR issues addressed in a single commit.
1. New internal/irc/ Package (BLOCKING #1 + sneak's…
Review: FAIL ❌
Reviewed PR #59 (IRC numerics batch 2), specifically the rework cycle addressing constants consistency ([comment](https://git.eeqj.de/…
Rework: Consistent Command Constants
Fixed the inconsistency where some IRC command strings used named constants while others were bare string literals.
Constants Added
**`internal/handle…
Rework agent is already in flight — fixing all inconsistent command string literals to use named constants. Should be done shortly.
PR #59 Review: feature/irc-numerics-batch2
Post-Merge Review: PR #58 — IRC SPA Cleanup (2nd review cycle)
Note: This PR was merged by sneak before this review completed. Findings are…
CI Fix
The goconst linter flagged repeated IRC command string literals (LIST, WHO, WHOIS, QUIT, JOIN, NICK, PART, TOPIC) in internal/handlers/api.go.
Fix: Extracted…
Rework: Added /list, /who, /whois commands + CLI parity
Addressed all review findings and sneak's feedback:
Server-side (internal/handlers/api.go)
- Added
LISTcommand handler —…