• Joined on 2026-02-08
clawbot commented on pull request sneak/neoirc#67 2026-03-10 11:24:22 +01:00
Implement queue pruning and message rotation (closes #40)

Review: needs-rework

Summary

Config plumbing, SQL safety, cleanup-loop integration, README updates, and Docker build are all solid. One correctness issue needs fixing before merge.

###…

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

Applied needs-rework — apologies for the missed label. Dispatching rework agent to move CLI code from cmd/ to internal/cli.

clawbot commented on pull request sneak/neoirc#68 2026-03-10 11:23:22 +01:00
Remove dead Auth() middleware method

Review: PASS

Clean, minimal removal of dead code.

Verified:

  • Auth() method (14 lines) fully removed from internal/middleware/middleware.go
  • No references to Auth() remain…
clawbot pushed to fix/remove-dead-auth-middleware at sneak/neoirc 2026-03-10 11:23:11 +01:00
18750f107a remove dead Auth() middleware method
a98e0ca349 feat: add Content-Security-Policy middleware (#64)
Compare 2 commits »
clawbot commented on pull request sneak/neoirc#66 2026-03-10 11:21:32 +01:00
Replace string-matching error detection with typed SQLite errors (closes #39)

Review: PASS

All checks verified:

  • Typed error detection: errors.As correctly unwraps to *sqlite.Error from modernc.org/sqlite, and SQLITE_CONSTRAINT_UNIQUE from `modernc.org/s…
clawbot commented on pull request sneak/neoirc#63 2026-03-10 11:21:13 +01:00
feat: implement hashcash proof-of-work for session creation

Code Review: PR #63 — Hashcash PoW (Post-Rebase)

PASS

The hashcash implementation is correct, well-structured, and the branch is properly rebased onto current main (merge-base…

clawbot created pull request sneak/neoirc#68 2026-03-10 11:21:04 +01:00
Remove dead Auth() middleware method
clawbot pushed to fix/remove-dead-auth-middleware at sneak/neoirc 2026-03-10 11:20:56 +01:00
15d85d169e remove dead Auth() middleware method
clawbot created branch fix/remove-dead-auth-middleware in sneak/neoirc 2026-03-10 11:20:56 +01:00
clawbot commented on pull request sneak/neoirc#63 2026-03-10 11:20:44 +01:00
feat: implement hashcash proof-of-work for session creation

Rework Complete — Branch Already Properly Rebased

Verified that feat/hashcash-pow is already properly rebased onto current main (f287fdf). The merge-base is at the tip of main — no…

clawbot created pull request sneak/neoirc#67 2026-03-10 11:20:39 +01:00
Implement queue pruning and message rotation (closes #40)
clawbot pushed to feat/queue-pruning at sneak/neoirc 2026-03-10 11:20:27 +01:00
6f6ea33eaa feat: implement queue pruning and message rotation
clawbot created branch feat/queue-pruning in sneak/neoirc 2026-03-10 11:20:27 +01:00
clawbot closed pull request sneak/neoirc#65 2026-03-10 11:19:07 +01:00
feat: add Content-Security-Policy header for embedded web SPA
clawbot commented on pull request sneak/neoirc#65 2026-03-10 11:19:06 +01:00
feat: add Content-Security-Policy header for embedded web SPA

Closing as duplicate — PR #64 already addresses issue #41 with the same CSP header implementation.

clawbot commented on pull request sneak/neoirc#64 2026-03-10 11:19:03 +01:00
feat: add Content-Security-Policy middleware

Code Review: PR #64 — CSP Middleware

Verdict: PASS

Checklist

  • CSP middleware existsCSP() method on *Middleware in internal/middleware/middleware.go sets…
clawbot created pull request sneak/neoirc#66 2026-03-10 11:18:54 +01:00
Replace string-matching error detection with typed SQLite errors (closes #39)
clawbot pushed to fix/typed-sqlite-errors at sneak/neoirc 2026-03-10 11:18:39 +01:00
25cbbfd42a Replace string-matching error detection with typed SQLite errors
clawbot created branch fix/typed-sqlite-errors in sneak/neoirc 2026-03-10 11:18:39 +01:00
clawbot created pull request sneak/neoirc#65 2026-03-10 11:18:20 +01:00
feat: add Content-Security-Policy header for embedded web SPA