• Joined on 2026-02-08
clawbot commented on pull request sneak/neoirc#59 2026-03-09 22:57:50 +01:00
feat: implement IRC numerics batch 2 — connection registration, channel ops, user queries

Review: PASS

Reviewed PR #59 (IRC numerics batch 2, closing #52).

Checklist

  • **Connection…
clawbot created pull request sneak/neoirc#59 2026-03-09 22:54:16 +01:00
feat: implement IRC numerics batch 2 — connection registration, channel ops, user queries
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 created branch feature/irc-numerics-batch2 in sneak/neoirc 2026-03-09 22:53:57 +01:00
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 created branch fix/irc-numeric-replies in sneak/neoirc 2026-03-08 10:16:09 +01:00
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)