Block a user
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.
###…
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.
Remove dead Auth() middleware method
Review: PASS ✅
Clean, minimal removal of dead code.
Verified:
Auth()method (14 lines) fully removed frominternal/middleware/middleware.go- No references to
Auth()remain…
Replace string-matching error detection with typed SQLite errors (closes #39)
Review: PASS ✅
All checks verified:
- Typed error detection:
errors.Ascorrectly unwraps to*sqlite.Errorfrommodernc.org/sqlite, andSQLITE_CONSTRAINT_UNIQUEfrom `modernc.org/s…
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…
Remove dead Auth() middleware method
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…
Implement queue pruning and message rotation (closes #40)
feat: add Content-Security-Policy header for embedded web SPA
feat: add Content-Security-Policy middleware
Code Review: PR #64 — CSP Middleware
Verdict: ✅ PASS
Checklist
- CSP middleware exists —
CSP()method on*Middlewareininternal/middleware/middleware.gosets…
Replace string-matching error detection with typed SQLite errors (closes #39)
feat: add Content-Security-Policy header for embedded web SPA