diff --git a/AGRARIAN_DEVELOPMENT_ROADMAP.md b/AGRARIAN_DEVELOPMENT_ROADMAP.md index ab39a1e..c112f57 100644 --- a/AGRARIAN_DEVELOPMENT_ROADMAP.md +++ b/AGRARIAN_DEVELOPMENT_ROADMAP.md @@ -857,7 +857,7 @@ Target deliverable: A small group can join a server, spawn into one biome, gathe - [x] Can launch packaged client. Added an MVP QA gate that requires the Windows package script, packaged executable, installed investor launchers, and the real-GPU visual QA readiness check before the client launch gate qualifies. - [x] Can launch server. Added an MVP QA gate for the Linux gameplay host requiring the true dedicated build path or current binary-engine fallback, deployment to `/opt/agrarian/server`, `agrarian-game-server.service` active state, UDP `7777` listener evidence, and Ground Zero map browse evidence. - [x] Can connect two clients. Added a two-client connection QA gate and Windows helper that checks the packaged client, launches two client instances against the same `play.agrariangame.com:7777` or LAN endpoint, and ties the manual observation steps to the multiplayer latency smoke plan. -- [ ] Can gather resources. +- [x] Can gather resources. Added a resource gathering QA gate tied to Ground Zero wood/fiber nodes, server-authoritative resource interaction, replicated harvest depletion, inventory grants, resource persistence coverage, and the natural shelter playable-loop smoke test. - [ ] Can craft a fire. - [ ] Can craft a shelter. - [ ] Can survive one full day/night cycle. diff --git a/Docs/QA/MvpQaGates.md b/Docs/QA/MvpQaGates.md index 11f7585..5357cf1 100644 --- a/Docs/QA/MvpQaGates.md +++ b/Docs/QA/MvpQaGates.md @@ -71,3 +71,22 @@ Required evidence: The manual observation steps continue in `Docs/Ops/MultiplayerLatencyTestPlan.md` until gameplay automation can drive two clients directly. + +## Resource Gathering + +The resource gathering gate proves a player can interact with placed Ground +Zero resource nodes and receive authoritative resource inventory changes. + +Required evidence: + +- Ground Zero includes placed wood and fiber resource nodes. +- `AAgrarianResourceNode` exposes `Gather` or `Gather by hand` prompts. +- Gathering runs only with server authority. +- Successful gathering decrements replicated `RemainingHarvests`. +- The gathered item stack is granted through the character inventory path. +- Resource depletion persistence is covered by the save/load gate. +- `Scripts/verify_playable_loop_smoke.py` includes wood and fiber gathering in + the natural shelter loop smoke test. + +This gate is gameplay/server-relevant and should be covered by the next server +package deployment. diff --git a/Scripts/verify_resource_gathering_qa_gate.py b/Scripts/verify_resource_gathering_qa_gate.py new file mode 100644 index 0000000..328bd95 --- /dev/null +++ b/Scripts/verify_resource_gathering_qa_gate.py @@ -0,0 +1,66 @@ +#!/usr/bin/env python3 +"""Verify the MVP resource gathering QA gate is covered.""" + +from pathlib import Path + + +ROOT = Path(__file__).resolve().parents[1] +ROADMAP = ROOT / "AGRARIAN_DEVELOPMENT_ROADMAP.md" +QA_DOC = ROOT / "Docs" / "QA" / "MvpQaGates.md" +RESOURCE_H = ROOT / "Source" / "AgrarianGame" / "AgrarianResourceNode.h" +RESOURCE_CPP = ROOT / "Source" / "AgrarianGame" / "AgrarianResourceNode.cpp" +SMOKE = ROOT / "Scripts" / "verify_playable_loop_smoke.py" +PERSISTENCE = ROOT / "Scripts" / "verify_resource_node_persistence.py" + +REQUIRED = { + QA_DOC: [ + "## Resource Gathering", + "Gather by hand", + "RemainingHarvests", + "character inventory path", + "verify_playable_loop_smoke.py", + ], + RESOURCE_H: [ + "RemainingHarvests", + "QuantityPerHarvest", + "YieldItemDefinition", + "OnRep_RemainingHarvests", + ], + RESOURCE_CPP: [ + "FText::FromString(TEXT(\"Gather\"))", + "FText::FromString(TEXT(\"Gather by hand\"))", + "if (!HasAuthority() || !Interactor || RemainingHarvests <= 0)", + "RemainingHarvests--;", + "Interactor->GetInventoryComponent()", + "Inventory->AddItem(Granted)", + "DOREPLIFETIME(AAgrarianResourceNode, RemainingHarvests);", + ], + SMOKE: [ + "WOOD_NODE_LABEL", + "FIBER_NODE_LABEL", + "run_natural_shelter_loop_smoke_test", + ], + PERSISTENCE: [ + "FAgrarianSavedResourceNode", + "RemainingHarvests", + ], + ROADMAP: [ + "[x] Can gather resources.", + ], +} + + +def main() -> None: + missing: list[str] = [] + for path, snippets in REQUIRED.items(): + text = path.read_text(encoding="utf-8") + for snippet in snippets: + if snippet not in text: + missing.append(f"{path.relative_to(ROOT)} missing {snippet!r}") + if missing: + raise SystemExit("FAILED: " + "; ".join(missing)) + print("OK: resource gathering QA gate is covered by gameplay and smoke-test hooks.") + + +if __name__ == "__main__": + main()