v2.3 review: drop a duplicate logIdentityMode call; README wording; review notes

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-09-25 20:17:03 -07:00
parent 7a12ddcf5a
commit 9ff38dfe4e
3 changed files with 12 additions and 2 deletions
@@ -112,3 +112,14 @@ it; (b) task — the task text did not state that `httputil.ReverseProxy` aborts
 test design — timing-based tests (limiter, queue, spread, cancel) have margins tuned for an idle
 host; widen or retry in a later plan.

+
+### v2.3 review (owner, 2026-09-25)
+
+Checked: gate, `-race -count=3` on proxy and limiter, smoke (check 6: dedicated listener). Task 01
+clean (nit: the Debug log block copies the Info block's fields). Task 02: release-on-first-flush
+reverted by the owner (limiter stopped limiting streams; cause was the owner's racy given test,
+now fixed, with `TestLoadIsHeldForTheWholeStream`). Task 03: correct; `main.go` called
+`logIdentityMode` twice (removed). Task 04: stopped correctly on the owner's missed v2.1 router
+test; resumed after the replacement. README: "Boxmaker's router" → "Boxmaker's `inferproxy`".
+Model faults this plan: one timing hack (logged as a deviation), one refusal-ending, one
+malformed tool call ending a session with no change. Owner faults: racy test, missed router test.