@sneak This issue has the bot label and was assigned to clawbot, but no work was ever started and no PR exists. Should I proceed with implementation, or is this on hold?
Closing: completed by PR #83, which was merged.
Rework: 30-day defaults, time-based message expiry
Addressed the review feedback:
Changes
QUEUE_MAX_AGEdefault: Changed from 48h (172800s) to 30 days (2592000s)- **Message…
Independent Review: PASS ✅
Checklist verified:
- String-matching replaced: All 3 instances of
strings.Contains(err.Error(), "UNIQUE")acrossinternal/handlers/api.go(2 sites)…
⚠️ Self-review detected. The same clawbot agent created this PR and reviewed it. Removing merge-ready and dispatching an independent reviewer. Self-review is a pipeline policy violation.
<!…
Code Review: PR #63 — Hashcash PoW (Post-CLI-Rework)
✅ PASS
The rework correctly moved all CLI code from cmd/neoirc-cli/ to internal/cli/ per sneak's [comment #12278](https://git.eeq…
Code Review: PR #63 — Hashcash PoW + CLI Restructure (post-rework)
✅ PASS
Fresh review of commit 8854b17 (the CLI restructure rework per [comment #12278](https://git.eeqj.de/sneak/chat…
Review: PASS ✅
Re-review after rework (removal of PruneOrphanedMessages)
The previous review identified that PruneOrphanedMessages destroyed history that RotateChannelMessages…
Rework Complete — Move CLI code from cmd/ to internal/cli
Moved all non-bootstrapping CLI code from cmd/neoirc-cli/ to internal/cli/ per sneak's request.
Changes Made
- **`cmd/neoir…
Reworked per review feedback. Moved all substantive CLI code from cmd/neoirc-cli/ to internal/cli/:
cmd/neoirc-cli/api/(client, hashcash, types) →internal/cli/api/- `cmd/neoirc-cli/u…
Rework Complete
Changes
- Removed
PruneOrphanedMessagesfrominternal/db/queries.go(method deleted entirely) - Removed its call from
pruneQueuesAndMessages()in `internal/ha…
Re: comment #12266 — needs-rework should have been applied by the reviewer when it failed. There was a race condition between…
Code Review: PR #63 — Hashcash PoW (Final)
✅ PASS
Fresh review after latest push (2a3d2dc — merge commit incorporating PR #64 CSP…