Review: PASS
Independent re-review of f932a86 against next (9313b0f). Round-1 BLOCKING 1 is genuinely
fixed. I reproduced the central claim and probed it harder than the author did; no…
Reworked, head f932a86. One commit still; body updated.
BLOCKING 1 — upper bound restored. The barrier no longer opens at the concurrency-th holder. A second sync.WaitGroup of…
PASS
Head f346625; one commit on next b573959 (fast-forward, no conflict), title ends (closes #187), TODO.md untouched, no attribution trailers or vendor references anywhere in the…
Review: FAIL — needs-rework
Independent review of b8940c0 against next (b573959). One blocking finding: the de-flake of
TestLoginGuard_SemaphoreBoundsConcurrentVerifications traded…
Disclosure: this landed as a merge commit (9313b0f) rather than a squash, unlike every other unit on next. My mistake — I passed the merge style under the wrong parameter name, it was…