docs: define alpha animation system architecture#279
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a7f1e9c761
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".
| | Concern | Owner | | ||
| | ---- | ---- | | ||
| | Movement position, velocity, interaction range | Fusion server | | ||
| | Combat range, facing, hit window, damage, death | Fusion server or Nakama path approved for prototype training | |
There was a problem hiding this comment.
Keep combat authority on Fusion
This row makes Nakama an allowed owner for combat range, hit windows, damage, and death in the alpha source-of-truth animation doc. That contradicts the repo's backend boundary that Photon Fusion remains authoritative for in-zone combat and that Fusion validates combat intents before BodyTime or other state changes; following this table would let a prototype implement damage/death in Nakama and later diverge from the server-authoritative architecture.
Useful? React with 👍 / 👎.
| | Group | Required Actions | | ||
| | ---- | ---- | | ||
| | Locomotion | Idle, Walk, Run, Stop, Turn | | ||
| | Dialogue | Talk, DialogIdle, Agree, Disagree, Listen | |
There was a problem hiding this comment.
Match required actions to the enum
This makes DialogIdle and Listen required alpha actions, but the current CharacterActionId contract only has Talk, Agree, and Disagree for dialogue and no entries for those two ids. Because this same doc later says to freeze CharacterActionId as the semantic contract, the next profile validation or missing-action smoke work will either fail against nonexistent ids or invent untracked names; please either add the enum entries or keep the required list to the ids that actually exist.
Useful? React with 👍 / 👎.
Summary
Verification
git diff --checktools/windows/check-backlog-sync.ps1Research sources used