1.8 KiB
1.8 KiB
Manual QA — NEO-64 (gather telemetry hook sites)
Reference: implementation plan, NEO-62 implementation plan (gather engine), NEO-63 implementation plan (interact wiring).
Preconditions
- Run
NeonSprawl.Server(dotnet runfromserver/NeonSprawl.Server; defaulthttp://localhost:5253) if exercising Bruno; not required for code review only. - This story adds comments only; gather behavior matches NEO-62/NEO-63.
Code review check
- Open
server/NeonSprawl.Server/Game/Gathering/GatherOperations.cs. - Confirm
resource_gatheredcomment block on the successful gather path (after depletion commit, beforeGatherResultreturn). - Confirm
gather_node_depletedcomment block is insideif (remainingAfterGather == 0)(not on pre-gathernode_depleteddeny). - Confirm
TODO(E9.M1)markers and planned payload fields; no newILoggeror metrics calls. - Confirm no duplicate hook comments in
InteractionApi.cs. - Open
GatherResult.cs— summary cross-references NEO-64 hook anchor inTryGather. - Open server README — Gather telemetry hooks (NEO-64) — NEO-64 subsection present.
Regression (optional)
cd server && dotnet test NeonSprawl.sln
Expect all tests pass (comments-only change).
Bruno (optional sanity)
From bruno/neon-sprawl-server/interaction/ against a running server:
- Post interact prototype resource node gather —
allowed: trueunchanged. - Post interact gather then get inventory — scrap quantity increases unchanged.
- Post interact depleted node deny —
allowed: false,reasonCode: node_depletedunchanged.