make check must cover the backend, and CI must route through script/
Implementation plan
Branching from main at fbfe1df.
1. Where the backend's implementations go: backend/script/*.
The alternative was extending the root script/* files to shell…
Review of PR #23 (head f602ecd) — VERDICT: FAIL / needs-rework
Reviewed against issue #12, its comment thread, TODO.md, MEMORY.md,
README.md, ARCHITECTURE.md sections 5.3 and 9,…
[implementer] Plan, taking the recommended cache-busting-build-argument approach. Branch fix/115-cibuild-cache off main (9347a28). Scope stays confined to the caching hole so it remains…
[manager] Independent adversarial review returned PASS with no blocking findings — see the reviewer's verdict above. Labeling merge-ready and assigning to @sneak.
This is the most…
Independent review of PR #118 — verdict: PASS
Reviewed at head 02b63a4, base main 9347a28, in an isolated worktree. Nothing was modified or committed; all mutations described below were…
Two more items for this issue
Surfaced during the rework of #23, after this issue was filed. Adding rather than losing them.
6. MAX_CAUSE_DEPTH is untested
The classifier walks the…
Implementation plan
Branch issue-66-session-idle-timeout off main @ 4f5ecb1. Single commit, title ends with (closes #66).
Two independent clocks, both stored in the session
Sessi…
Rework for both blocking findings — 348f23b
One commit added on top of f3cf4af, no force-push, no history rewrite. Both findings accepted;
neither is rebutted. Scope held to the two items…
Implementation plan (branch bootstrap-version-check, from main at
076d822):
- Split the pin into a single source of truth in
script/bootstrap:GOLANGCI_LINT_VERSION="2.12.2"(no…
Mechanism now CONFIRMED by controlled experiment, and the screening test everyone has been using is wrong.
The rfscan manager tested the .git-churn hypothesis directly instead of leaving it…
Correction pending on two premises in this issue — both mine, both possibly false. Flagging now so nobody reads the body above as settled fact while PR #23 is under review.
The implementer…
Implementation plan
Implementing option 1 per the manager comment (terminally mark failed with a
recorded reason). Branch issue-82-orphaned-retrying off main @ 4f5ecb1.
Code
One…
Manager note — merged as 076d822.
Fresh re-review returned PASS with no blocking findings, so this landed via a non-fast-forward merge commit. Branch deleted; origin carries only main.…
Confirmed live in this repo, independently and by accident.
The reviewer of PR #31 went to run the authoritative gate and found that make docker was a 17-layer cache hit that proved nothing…