diff --git a/TODO.md b/TODO.md index 6b1a213..afbadf3 100644 --- a/TODO.md +++ b/TODO.md @@ -29,7 +29,9 @@ REWORK_PLAN.md. ### BUG-2: addParticipant allows duplicate id - **FIXED** (commit: addParticipant throws on dup id). - Test: `shared/tests/turn.characterization.test.js` 'addParticipant rejects - duplicate id' — GREEN. + duplicate id' --- GREEN. + +### bug-3 was a halucination has been removed ### BUG-4: hide-player-HP breaks display view (preexisting) - **Broader than hide-HP**: ALL 5 `storage.setDoc(getPath.activeDisplay(), ...)` calls @@ -72,7 +74,7 @@ REWORK_PLAN.md. - nextTurn rotates via `turnOrderIds` only → reorder has NO effect on combat rotation. Mid-encounter drag-drop = pointless. - replay-combat.js calls reorderParticipants with WRONG signature - `(enc, reorderedArray)` — swallowed by try/catch, silent no-op. So + `(enc, reorderedArray)` --- swallowed by try/catch, silent no-op. So replay never exercised real path either. - Fix: reorder must also update turnOrderIds to match new participant order (within same-initiative tie). @@ -88,7 +90,7 @@ REWORK_PLAN.md. - Display frozen until full reload. - Fix: `onclose` → reconnect + re-subscribe existing paths. -## Pipeline (bugs only — milestones live in REWORK_PLAN.md) +## Pipeline (bugs only --- milestones live in REWORK_PLAN.md) - [ ] BUG-4: fix setDoc→updateDoc for all 5 activeDisplay sites - [ ] BUG-5: fix computeTurnOrderAfterAddition currentTurn re-anchor - [ ] BUG-6: reorderParticipants update turnOrderIds