NEO-83: Reconcile plan with code review Bruno follow-up.

Document one-pulse cast→GET smoke request and review-driven handler/test changes.
pull/117/head
VinPropane 2026-05-25 11:23:41 -04:00
parent 50a2c9ca9d
commit 24eb355a26
1 changed files with 3 additions and 1 deletions

View File

@ -97,6 +97,7 @@
| `server/NeonSprawl.Server/Game/Combat/CombatTargetsListDtos.cs` | Response envelope + row JSON types. |
| `server/NeonSprawl.Server.Tests/Game/Combat/CombatTargetsWorldApiTests.cs` | AAA integration tests: fresh, post-cast, defeat. |
| `bruno/neon-sprawl-server/combat-targets/Get combat targets.bru` | Bruno happy path. |
| `bruno/neon-sprawl-server/combat-targets/Get combat targets after one cast.bru` | Fast one-pulse cast→GET smoke (code review follow-up). |
| `bruno/neon-sprawl-server/combat-targets/Get combat targets after cast spine.bru` | Bruno cast → GET damage + defeat AC spine. |
| `bruno/neon-sprawl-server/combat-targets/folder.bru` | Folder docs + run order. |
| `docs/plans/NEO-83-implementation-plan.md` | This plan. |
@ -141,4 +142,5 @@ Bruno spine covers manual HTTP verification; no **`docs/manual-qa/NEO-83.md`** (
- **`CombatTargetsWorldApi`** + **`CombatTargetsListDtos`** in **`Game/Combat/`**; **`PrototypeTargetRegistry.GetPrototypeTargetIdsInOrder()`**.
- **3** AAA integration tests in **`CombatTargetsWorldApiTests`** (fresh GET, post-cast damage, four-pulse defeat).
- Bruno happy GET + cast→GET defeat spine; **`server/README.md`**, E5M1 backlog, E5_M1 module doc, dependency register, alignment table updated.
- Bruno happy GET + one-pulse cast→GET smoke + four-pulse defeat spine; **`server/README.md`**, E5M1 backlog, E5_M1 module doc, dependency register, alignment table updated.
- Code review follow-up: **`TryGet` miss throws** instead of silent skip; one-pulse test asserts cast **`combatResolution`** coherence.