• Joined on 2026-02-08
clawbot commented on pull request sneak/neoirc#63 2026-03-10 18:44:13 +01:00
feat: implement hashcash proof-of-work for session creation

Renamed the JSON field hashcashpow_token in all four places:

  • internal/handlers/api.go: struct tag json:"pow_token,omitempty"
  • internal/cli/api/types.go: struct tag `json:"pow_toke…
clawbot pushed to feat/hashcash-pow at sneak/neoirc 2026-03-10 18:44:06 +01:00
49517e754a rename JSON field 'hashcash' to 'pow_token' in API request body
clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:40:52 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Review: PASS

Tests are thorough and correct. All 9 test functions cover the full IRCMessageType API:

clawbot commented on pull request sneak/neoirc#63 2026-03-10 18:40:50 +01:00
feat: implement hashcash proof-of-work for session creation

Code Review: PR #63 — Hashcash PoW + Tests

FAIL — Unaddressed feedback: JSON key rename hashcashpow_token

sneak requested in…

clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:39:09 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Added pkg/irc/numerics_test.go with 9 test functions covering the full IRCMessageType API:

  • Name() — correct name for known numerics (001, 005, 252, 404, 433)
  • String() — formatted…
clawbot pushed to feat/complete-irc-numerics at sneak/neoirc 2026-03-10 18:38:58 +01:00
eeb0f29465 test: add comprehensive tests for IRCMessageType API
clawbot commented on pull request sneak/neoirc#63 2026-03-10 18:38:56 +01:00
feat: implement hashcash proof-of-work for session creation

Added comprehensive hashcash validator test suite (internal/hashcash/hashcash_test.go) using bits=2 for fast execution. Tests cover:

  • Mint and validate a valid stamp
  • Replay detection…
clawbot pushed to feat/hashcash-pow at sneak/neoirc 2026-03-10 18:38:49 +01:00
0d563adf31 test: add hashcash validator tests with bits=2
clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:36:11 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

On it — adding tests for IRCMessageType API. Dispatching rework agent now.

clawbot commented on pull request sneak/neoirc#72 2026-03-10 18:36:00 +01:00
feat: implement Tier 1 IRC numerics

Post-Rework Review: PASS

clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:33:59 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Review: PASS

IRCMessageType API — verified correct

  • IRCMessageType is type IRCMessageType int
  • .Name() → bare name (e.g. "RPL_LUSEROP") ✓
  • .String() → `"RPL_LUSEROP…
clawbot commented on pull request sneak/neoirc#63 2026-03-10 18:33:41 +01:00
feat: implement hashcash proof-of-work for session creation

On it — adding hashcash tests with bits=2 to verify the full flow. Dispatching rework agent now.

clawbot commented on pull request sneak/neoirc#72 2026-03-10 18:33:37 +01:00
feat: implement Tier 1 IRC numerics

Rework complete. Changes in this push:

  1. Migration 002 folded into 001 — deleted 002_away_and_topic_meta.sql and added away_message, topic_set_by, topic_set_at columns directly to…
clawbot pushed to feat/tier1-irc-numerics at sneak/neoirc 2026-03-10 18:33:27 +01:00
3dc783c206 fix: remove dead code, fold migration 002 into 001, implement error numerics
clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:31:36 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Reworked per feedback:

  • Renamed MessageTypeIRCMessageType (custom int type)
  • .Name() → returns just the name, e.g. "RPL_LUSEROP"
  • .String() → returns name + code, e.g.…
clawbot pushed to feat/complete-irc-numerics at sneak/neoirc 2026-03-10 18:31:24 +01:00
ba2c113a85 refactor: rename MessageType to IRCMessageType with full API
clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:31:15 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Rework agent is running (8min in) — implementing IRCMessageType with full API (.Name(), .String(), .Code(), .Int(), FromInt()). Will push when ready.

clawbot commented on pull request sneak/neoirc#72 2026-03-10 18:27:31 +01:00
feat: implement Tier 1 IRC numerics

Understood — treating the review notes as blockers. Rework agent dispatched to:

  1. Remove dead GetAwayByNick() code
  2. Implement error numerics 404/411/412/451 (make the server actually send them)
clawbot commented on pull request sneak/neoirc#71 2026-03-10 18:23:18 +01:00
Complete IRC numerics module and move to pkg/irc/ (refs #52)

Acknowledged updated spec — implementing IRCMessageType with:

  • .Name()"RPL_LUSEROP"
  • .String()"RPL_LUSEROP <252>"
  • .Code()"252" (zero-padded)
  • .Int() → bare int -…
clawbot pushed to feat/complete-irc-numerics at sneak/neoirc 2026-03-10 18:17:59 +01:00
164fec3fc8 refactor: add irc.MessageType custom type with Name() and String() methods