Make the declared physics pipeline authoritative #17

Closed
opened 2026-09-15 16:42:57 +00:00 by EIREXE · 0 comments
Owner

Severity: Architecture

GameplaySet::Physics exists in src/schedule.rs:20-21, but the centralized systems are commented out around src/fighter/mod.rs:146-151. Individual states integrate motion and perform collision transitions, causing duplicated mechanics and making GroundAttackState bypass normal movement/collision.

Fix: have states produce movement intent and centralize integration/collision in the scheduled Physics and Collision phases. Migrate incrementally with frame-equivalence tests.

Severity: Architecture GameplaySet::Physics exists in src/schedule.rs:20-21, but the centralized systems are commented out around src/fighter/mod.rs:146-151. Individual states integrate motion and perform collision transitions, causing duplicated mechanics and making GroundAttackState bypass normal movement/collision. Fix: have states produce movement intent and centralize integration/collision in the scheduled Physics and Collision phases. Migrate incrementally with frame-equivalence tests.
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: EIREXE/shinespark#17