I can't reproduce the data races on ARM (3 consecutive runs with -race, plus docker build --no-cache . all pass). Could you paste the race detector output from CI? I need to see which…
Rebased onto current main (e62962d, includes PR #93 in-memory SQLite fix). The data races were caused by the old file-backed SQLite tests interacting…
Re-Review: PR #92 — Tier 2 Channel Modes (post-rework)
Result: PASS ✅
Verification
The previous review found one blocking issue: README RPL_MYINFO (004) example showed "mnst" but…
Fixed the blocking review finding: updated RPL_MYINFO (004) example in README.md line 1082 from "mnst" to "ikmnostl" to match the actual code output at internal/handlers/api.go:450.
`docker…
Review: PR #41 (post-rework) — allowlist rename
Sneak's Feedback Verification
Review: PR #92 — Tier 2 Channel Modes (+b/+i/+s/+k/+l)
Result: FAIL — one blocking documentation inaccuracy.
Checklist
channel_banstable with proper schema (id, channel_id…
Rework: rename to allowlist, fix stuttering
Addressed the review feedback:
- Package renamed
internal/whitelist/→internal/allowlist/ - Type renamed
HostWhitelist→…
This issue does not apply to dnswatcher. The repo uses JSON file-based state persistence (internal/state/state.go → DATA_DIR/state.json), not SQLite. There are:
- No
.sqlfiles…