neon-sprawl/docs/reviews
VinPropane d6ef4fd811 NEO-24: race-free target/select via optional positionHint
Followup #5: after locking alpha, walking into beta's ring, stopping and
pressing Tab could still return `out_of_range` even when client and server
agreed on position (Δ=0). The freshness-kick move-stream POST and the
target/select POST race on separate HTTP connections so the select can be
validated against a stale stored snap.

Extend TargetSelectRequest v1 with an optional positionHint {x,y,z}.
Server uses the hint for the radius check and for the echoed
targetState.validity; the hint is advisory and does not write to the
position store (move-stream stays the sole write path). Client attaches
the hint automatically whenever set_freshness_kick(...) is wired, so
headless unit tests that skip the wiring fall through to the no-hint
behavior. Freshness kick is kept in place to refresh the stored snap for
later validity GETs but is no longer load-bearing for denial correctness.

Server: 3 new tests (hint accepts when stored is stale, hint-far denies,
hint never writes to position store). Client: 1 new test asserting the
hint is in the POST body when wired. All 62 server + 96 client tests
pass.

Plan Decision 8, review follow-up #5, manual QA section 6, and NEO-23
contract notes updated. New bruno request exercises the hint path.
2026-04-21 23:52:20 -04:00
..
2026-03-29-NEO-6.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-03-30-NEO-7.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-03-30-NEO-8.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-03-31-game-design-skills-coherence.md docs: drop classless terminology; rename Epic 2 to skills and progression 2026-04-03 21:33:53 -04:00
2026-04-03-branch-chore-game-design-docs-vs-main.md docs: E2.M1 contract column lists allowedXpSourceKinds (nit #2) 2026-04-03 21:34:49 -04:00
2026-04-03-final-branch-docs-review.md chore: mark vision-plan suggestion done in final docs review 2026-04-03 22:58:14 -04:00
2026-04-03-game-design-branch-docs-review.md chore: stable Epic 3 slice anchors; deep-links from gather/craft docs 2026-04-03 22:14:47 -04:00
2026-04-04-NEO-9.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-05-NEO-11.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-05-NEO-14.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-07-NEO-15.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-08-NEO-13.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-08-NEO-16.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-08-NEO-17.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-09-NEO-20.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-10-NEO-14-followup.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-10-NEO-18.md chore: point docs and rules at Linear NEO issue ids 2026-04-13 13:15:32 -04:00
2026-04-17-NEO-22.md NEO-22: Cache scrape contact in WASD tick; close review items 2026-04-17 20:21:13 -04:00
2026-04-17-NEO-23.md NEO-23: code review follow-up (E1.M3 docs, GET 404 test, radius bounds) 2026-04-17 21:14:43 -04:00
2026-04-18-e1m3-documentation.md chore: address E1.M3 docs review (Slice 3 anchor, Linear pointers) 2026-04-17 20:48:53 -04:00
2026-04-21-NEO-24.md NEO-24: race-free target/select via optional positionHint 2026-04-21 23:52:20 -04:00