|
|
|
@@ -1,6 +1,7 @@
|
|
|
|
|
# M02-GMP-MOVEMENT-001 — Local movement state/controller
|
|
|
|
|
|
|
|
|
|
<!-- OPENWC_CLAIM:M02-GMP-MOVEMENT-001:sindo-main-codex:2026-07-16 -->
|
|
|
|
|
<!-- OPENWC_HANDOFF:READY:M02-GMP-MOVEMENT-001:435e1c9 -->
|
|
|
|
|
|
|
|
|
|
## Ownership
|
|
|
|
|
|
|
|
|
@@ -76,8 +77,41 @@ sandbox motion.
|
|
|
|
|
|
|
|
|
|
## Status
|
|
|
|
|
|
|
|
|
|
- State: active
|
|
|
|
|
- Done: claim and isolated worktree from updated master
|
|
|
|
|
- Next: implement controller, migrate scene consumer, test and document
|
|
|
|
|
- State: ready-for-review
|
|
|
|
|
- Done: scene-free movement state/controller, player scene migration, twelve
|
|
|
|
|
numeric cases, two flight transitions, integrated regression and module documentation
|
|
|
|
|
- Next: M02 integrator reviews and merges before TerrainQuery extraction
|
|
|
|
|
- Blocked by:
|
|
|
|
|
|
|
|
|
|
## Handoff
|
|
|
|
|
|
|
|
|
|
- Commit: `435e1c9`
|
|
|
|
|
- Branch: `work/sindo-main-codex/m02-local-movement`
|
|
|
|
|
- Outcome: `ThirdPersonWowController` no longer owns velocity, flight state,
|
|
|
|
|
acceleration or directional speed calculation. It selects an explicit
|
|
|
|
|
Godot-world basis, applies returned displacement, and retains terrain and
|
|
|
|
|
presentation adapters for later M02 packages.
|
|
|
|
|
- Public contract: additive `LocalPlayerMovementController` with read-only
|
|
|
|
|
Godot-world velocity/flight state, `toggle_sandbox_flight` and `advance`.
|
|
|
|
|
Input, coordinate, cache and renderer contracts remain unchanged.
|
|
|
|
|
- Verification: `LOCAL_PLAYER_MOVEMENT PASS cases=12 state_transitions=2
|
|
|
|
|
scene_boundary=1`; `PLAYER_INPUT PASS actions=12 intent_cases=6 controller=1
|
|
|
|
|
regression_scene=1`; asset-free scene smoke exited `0`; coordinate boundary
|
|
|
|
|
and StreamingFocus passed; renderer dry-run passed all eight pre-capture gates
|
|
|
|
|
and seven checkpoint plans; coordination/documentation/diff gates passed.
|
|
|
|
|
- Fidelity: exact existing defaults are regression-locked—forward `7.0`,
|
|
|
|
|
backward/strafe `4.5`, acceleration `28.0`, sandbox sprint `6x`, vertical
|
|
|
|
|
flight `7.0`, velocity reset on toggle and camera-relative pitched flight.
|
|
|
|
|
This preserves the sandbox baseline but is not original-client evidence.
|
|
|
|
|
- Local verification inputs: the isolated worktree reused the existing ignored
|
|
|
|
|
native DLL and generated ADT resource scripts. Proprietary extracted data and
|
|
|
|
|
character GLB were absent, so renderer dry-run logged expected missing-corpus
|
|
|
|
|
diagnostics while every contract step returned success.
|
|
|
|
|
- Rebuild/migration: none; transient movement state is not persisted.
|
|
|
|
|
- Remaining risks: exported speed changes after `_ready` do not reconfigure the
|
|
|
|
|
controller; exact build-12340 movement semantics, terrain/collision state,
|
|
|
|
|
jump/fall/swim, server prediction and debug profile gating remain open.
|
|
|
|
|
- Documentation: inline public API plus `docs/modules/local-player-movement.md`
|
|
|
|
|
with inputs/outputs, data-flow, state and sequence diagrams, ownership,
|
|
|
|
|
recovery, capability status, risks and source map; player-input consumer
|
|
|
|
|
diagrams/status were updated.
|
|
|
|
|