-
Notifications
You must be signed in to change notification settings - Fork 0
⭐ Code Review Audit
A per-module, per-file code review of all 13 added modules, run for real — a GitHub issue, a branch, and a merged PR for every source file, not a simulated pass. Parent tracking issue #32, one sub-issue per module (#33–#45). One module worked at a time, in the order modules were originally added.
Status: store complete — 27/27 files, 3 real bugs found and fixed, 1 test-coverage gap closed. entity complete — 9/9 files, 0 real bugs, 2 structural observations recorded. effects in progress — 9/18 files merged, 0 real bugs found so far. 10 modules still queued after effects.
All 3 real bugs found in store share one shape: a function's top-level entry point missing a null-state guard its own recursive/nested-value logic already had (freeze(), getUnserializable(), combineReducers()) — inherited from the original upstream import, not introduced by this repo, each reproduced with a failing regression test before being fixed. Full findings index, file by file, is docs/code-review.md; live per-file tracking table is in todo.md.