[Quartermaster] Sprint: Level-Up Wizard Correctness & Debt

[LordOfMyatar]

Summary

Corrects the remaining level-up math errors and retires the duplicated logic that causes them.
Five items, sequenced so the correctness fixes land before the refactors.

Every claim was challenged against HEAD during pre-warm before planning. Three issue bodies
needed correction first: #2580's headline ("~600k uncached 2DA reads") was refuted — the 2DA
layer already caches, and its row counts came from ?? fallback constants — so it is rescoped
to the real hot spots. #2581 understated itself: all six auto-assign priority branches have
drifted, not one. #2578 was upgraded to confirmed-reachable.

Work Items

• ✅ [Quartermaster] Level-Up: Auto-Assign Skills caps ranks at wrong level for multi-level level-ups #2576 — Auto-Assign Skills caps ranks at the final level, not current+1
• ✅ [Quartermaster] Level-Up: inline skill-point calc omits level-1 x4 multiplier (level-0 creature) #2578 — Level-1 x4 skill-point multiplier, delegated to the shared service
• ✅ [Quartermaster] tech-debt: Wizards reimplement point-buy/feat-table/gender logic that already exists in services #2581 — NCW routed through the point-buy and gender services; one shared class-feat-table reader; [Quartermaster] Bug: LUW auto-assign may use wrong package for feat/ability suggestions #1737 priority order made canonical (derived-projection cleanup landed with [Quartermaster] Level-Up: consolidated multi-level apply re-implements single-level path (drift risk) #2575)
• 🔄 [Quartermaster] perf: Wizard feat list does ~300-row 2DA scan per feat, rebuilt on every click #2580 — NCW skill rows update incrementally; TwoDAFile column-index cache. LUW feat re-sort and the class-feat HashSet are deferred, tracked in [Quartermaster] perf: Wizard feat list does ~300-row 2DA scan per feat, rebuilt on every click #2580
• ✅ [Quartermaster] Level-Up: consolidated multi-level apply re-implements single-level path (drift risk) #2575 — one span-aware apply path (single + multi-level); ability map derived by a unit-tested helper, three hand-synced projections retired

Related Issues

• Closes [Quartermaster] Sprint: Level-Up Wizard Correctness & Debt #2677 (sprint), [Quartermaster] Level-Up: Auto-Assign Skills caps ranks at wrong level for multi-level level-ups #2576, [Quartermaster] Level-Up: inline skill-point calc omits level-1 x4 multiplier (level-0 creature) #2578, [Quartermaster] tech-debt: Wizards reimplement point-buy/feat-table/gender logic that already exists in services #2581, [Quartermaster] Level-Up: consolidated multi-level apply re-implements single-level path (drift risk) #2575
• Relates to [Quartermaster] perf: Wizard feat list does ~300-row 2DA scan per feat, rebuilt on every click #2580 (partial — remainder stays open), [Quartermaster] epic: Level-Up wizard transactional refactor (clone-and-commit + service-side validation) #2571 (parent epic), [Quartermaster] Level-Up: consolidated multi-level apply drops CE extra ability increases #2696, [Radoub] perf: Cross-tool performance tune (app stalling, cold-start latency) #2128

Checklist

• Implementation complete
• Tests added/updated
• CHANGELOG updated with date
• Manual spot-checks verified

---

🤖 Generated with Claude Code