Repository navigation
docs: make the registry true — one false ✅, a number collision, and README rot - #23
Conversation
…EADME rot An audit of every SH-* ticket against the code found the docs wrong in BOTH directions. Nothing here changes behaviour; it stops the docs lying. SH-043 was the registry's only false ✅. It claimed "shipped — src/strings.ts"; that file has NEVER existed and there is no STRINGS reference in src/ (verified, not inferred). docs/aaa-round-1.md had the honest version the whole time. Now 🟡 with a part-by-part table: shake slider + colourblind cues + captions shipped; the localization table and rebindable keys have NOT. SH-027 v2 is closed — its header said "gated on EN-033" for weeks after EN-033 landed. The gate was imaginary; the remaining gap (pitch) is now closed by EN-039 + PR #22. SH-046 was a NUMBER COLLISION, and mine: I wrote the boot sequence as "SH-046" in code while the registry already used SH-046 for character fidelity. The boot work is renumbered SH-049 across boot.ts / main.ts / menu.ts / combat.ts — comments only, no behaviour. SH-047 (body turns to face movement) and SH-048 (crouch) existed only as code comments and are now registered, SH-048 explicitly 🟡 with its capsule-resize gap stated rather than implied. README: it said Esc QUITS (it pauses — and until yesterday it really did quit, from anywhere, including out of the settings screen), "five alien kinds / two weapons" (seven and four), and "World data is hardcoded in TS until JSON.parse is fixed" — the world has loaded from JSON at runtime since the baker was deleted. Controls table now documents aim/sprint/crouch/dodge, which shipped undocumented. CLAUDE.md contradicted itself on the same JSON.parse point (§144 records it fixed; §200 told you to work around it) and repeated the wrong counts; both corrected, and it now says plainly that only ONE arena is playable. Claude-Session: https://claude.ai/code/session_01J1UWgMcrTNvwWeXcJ1T3Fp
|
Warning Review limit reached
Next review available in: 5 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Auditing every SH-* ticket against the code found the docs wrong in both
directions. No behaviour changes here — this stops the docs lying.
The false ✅ (the one that costs someone a day)
SH-043 claimed "shipped —
src/strings.ts". That file has neverexisted, and there is no
STRINGSreference insrc/— verified, notinferred.
docs/aaa-round-1.mdhad the honest version all along ("Partial.Localization table not done."). Now 🟡 with a part-by-part table: shake slider,
colourblind cues and captions shipped; the localization table and rebindable
keys have not.
A gate that was imaginary
SH-027's header said v2 was "still gated on EN-033" for weeks after
EN-033 landed. A ticket left open past its landing isn't clerical — it stops
downstream work that was already free. v2 is now closed (EN-039 + #22).
A number collision, and it's mine
I wrote the boot sequence as SH-046 in code while the registry already used
SH-046 for character fidelity — two tickets, one number. Boot is renumbered
SH-049 across
boot.ts/main.ts/menu.ts/combat.ts. Comments only.Tickets that existed only as code comments
SH-047 (body turns to face movement) and SH-048 (crouch) are now
registered — SH-048 explicitly 🟡 with its capsule-resize gap stated rather
than implied.
README rot
Esc │ QuitCLAUDE.mdcontradicted itself on the JSON.parse point (§144 records it fixed;§200 told you to work around it), and repeated the wrong counts. Both corrected,
and it now says plainly that only one arena is playable.
Build + smoke verified — the rename touches comments only.
https://claude.ai/code/session_01J1UWgMcrTNvwWeXcJ1T3Fp