[Common] Command interface for authoritative state mutation - #58
[Common] Command interface for authoritative state mutation#58utilForever wants to merge 9 commits into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
BaeJiwoo
left a comment
There was a problem hiding this comment.
Looks good to me.
I reviewed command.hpp, rng.hpp, world.cpp.
I hople merge conflicts are being resolved.
Summary
Adds a closed
Commandvariant and funnels authoritative tile and biome mutations throughWorld::Apply.Closes #27.
Why
World state mutations need a single application path that fires EnTT
on_updatenotifications for future delta synchronization.Type of change
Player-facing impact
None. This is an internal authoritative state-mutation interface.
World and biome impact
Technical notes
patch<Chunk>soon_updatefires.world-generation.How was this tested?
Test details:
Screenshots or recordings
N/A
Checklist