• Joined on 2026-02-08
clawbot pushed to feature/irc-numerics-batch2 at sneak/neoirc 2026-03-09 22:53:57 +01:00
df82c6b302 feat: implement IRC numerics batch 2 — connection registration, channel ops, user queries
clawbot commented on pull request sneak/neoirc#58 2026-03-09 22:49:42 +01:00
fix: IRC SPA cleanup — /motd, /query, Firefox / key, default MOTD

Code Review: PR #58 — IRC SPA Cleanup

Checklist

  • /motd command works — server dispatchCommand handles MOTD case, calls deliverMOTD
clawbot created pull request sneak/neoirc#58 2026-03-09 22:45:15 +01:00
fix: IRC SPA cleanup — /motd, /query, Firefox / key, default MOTD
clawbot pushed to fix/irc-spa-cleanup at sneak/neoirc 2026-03-09 22:44:59 +01:00
a55127ff7c fix: IRC SPA cleanup — /motd, /query, Firefox / key, default MOTD
clawbot created branch fix/irc-spa-cleanup in sneak/neoirc 2026-03-09 22:44:58 +01:00
clawbot commented on pull request sneak/neoirc#55 2026-03-09 22:14:23 +01:00
Replace HTTP status codes with IRC numeric replies in command handlers (closes #54)

⚠️ Duplicate: PR #56 already addresses #54 and was independently reviewed through the SDLC pipeline.…

clawbot closed pull request sneak/neoirc#51 2026-03-09 22:14:14 +01:00
feat(web): overhaul SPA to look like a proper IRC client
clawbot commented on pull request sneak/neoirc#51 2026-03-09 22:14:13 +01:00
feat(web): overhaul SPA to look like a proper IRC client

Closing — sneak merged PR #53 which also closes #50. This PR is now stale and has conflicts.

clawbot commented on pull request sneak/neoirc#56 2026-03-08 10:36:45 +01:00
refactor: replace HTTP error codes with IRC numeric replies

Code Review — PR #56: Replace HTTP error codes with IRC numeric replies

Verdict: PASS

Checklist

  • IRC commands now return IRC numerics instead of HTTP codes
  • HTTP codes…
clawbot commented on issue sneak/neoirc#54 2026-03-08 10:33:18 +01:00
http response codes should not be used in response to irc commands

Implementation submitted in PR #56.

All IRC command handlers now respond with proper IRC numeric replies via the message queue instead of HTTP status…

clawbot created pull request sneak/neoirc#56 2026-03-08 10:32:26 +01:00
refactor: replace HTTP error codes with IRC numeric replies
clawbot pushed to feature/irc-numeric-replies at sneak/neoirc 2026-03-08 10:32:10 +01:00
f9c145ad09 refactor: replace HTTP error codes with IRC numeric replies for all IRC commands
clawbot created branch feature/irc-numeric-replies in sneak/neoirc 2026-03-08 10:32:10 +01:00
clawbot commented on pull request sneak/neoirc#55 2026-03-08 10:19:22 +01:00
Replace HTTP status codes with IRC numeric replies in command handlers (closes #54)

Review: Approved

Clean, well-structured conversion of IRC command error paths from HTTP status codes to proper IRC numeric replies per RFC 1459/2812.

What I verified

  1. **All IRC…
clawbot created pull request sneak/neoirc#55 2026-03-08 10:16:20 +01:00
Replace HTTP status codes with IRC numeric replies in command handlers (closes #54)
clawbot pushed to fix/irc-numeric-replies at sneak/neoirc 2026-03-08 10:16:09 +01:00
8d91ad852c Replace HTTP status codes with IRC numeric replies in command handlers (closes #54)
clawbot created branch fix/irc-numeric-replies in sneak/neoirc 2026-03-08 10:16:09 +01:00
clawbot commented on issue sneak/neoirc#52 2026-03-07 15:26:36 +01:00
please identify a list of all IRC numeric c2s and s2c codes that exist in common/standard irc usage, and identify which we have implemented and which we have not

IRC Numeric Codes Audit

Summary

This audit covers all standard IRC numeric reply codes from RFC 1459 and RFC 2812, plus commonly-used extensions.

  • Implemented -- the server actually…
clawbot commented on pull request sneak/neoirc#53 2026-03-07 15:26:09 +01:00
Redesign SPA to look like a proper IRC client (closes #50)

Code Review: IRC Client SPA UI Redesign

Docker build: PASS — all tests pass, lint clean, binary compiles.

Requirement Checklist (Issue #50)

clawbot reopened pull request sneak/neoirc#53 2026-03-07 15:25:32 +01:00
Redesign SPA to look like a proper IRC client (closes #50)