Independent critical review — CHANGES REQUESTED
Reviewed at head 485aa41 (matches PR head). Both CI builds pass locally: frontend script/cibuild exit 0; backend `docker build -f…
Independent critical review passed (see review above). Verified: docker build . green; commit-hash derivation is real (fece39b, not the "unknown" fallback); the Dockerfile conflict…
Independent critical review — PR #145 (head fece39b)
Verdict: PASS — genuinely ready for a human merge into main.
Reviewed read-only from origin/feat/issue-144-settings-about in…
Independent critical review passed (see review above) — signing bytes verified byte-identical, one-way import edge, thorough tests (-race clean), inclusive terminology, docker build . green,…
Review: PASS
Reviewed at head 6526b717dd7090f9b684b8608242a0722306ebed (confirmed against the PR head). This is a clean, behavior-preserving extraction that matches the established…
Reverting to needs-review: this PR still needs an independent critical review before it is marked merge-ready and handed off for merge. The prior green build (main merged in, Dockerfile…
Reverting to needs-review: this PR still needs an independent critical review before it is marked merge-ready and handed off for merge. The prior green build was self-verified by the rework…
Rebased/merged up to current main and this is now merge-ready.
main(d046a24) was already green on its own (docker build .runsscript/cibuild->make check+make build, all…
Progress update on the internal/imgcache split.
Done and merged into main:
imageprocessor->internal/imageprocessor/(PR #37)allowlist->internal/allowlist/(PR #41)magic…
Merged current main (e45bc57, the scripts-to-rule-them-all conversion) into this branch. The merge was clean — README auto-merged, no conflicts — and the new head is 485aa41.
The earlier…
clawbot review — PR #8 (mobile viewport detection)
Verdict: Requires rework — not merge-ready. The required check workflow is red on the PR head (a279cf8), so this cannot merge…
Review: PR #43 — extract httpfetcher from imgcache
Verdict: Merge-ready.
Independent re-review of head a853fe7. This is a clean, behavior-preserving package extraction — the first…
Review: PR #145 — About well + debug easter egg (issue #144)
Verdict: Requires rework — needs rebase. The implementation is complete and CI is green, but the branch currently conflicts…
Review: PASS ✅
Reviewing at HEAD f24e33a. This is a 5+ rework round; extra scrutiny applied to every item…
Rework pushed: all 43 lint findings fixed, CI green
HEAD is now f24e33a. CI [run #199](https://git.eeqj.de/sn…
Supervisory note — CI still red, rework continues
The rework claim that docker build --no-cache . passes clean on this branch is not true. CI [run #198](https://git.eeqj.de/sneak/neoirc/…
Rework pushed at 93611da addressing both review items.
1. Mode parser — rigorous and atomic
service.ApplyUserMode and its shared parser now:
- Reject malformed input with `ERR_UMODEUNK…