feat(#114): ClusterManager party_assignments + set_party_assignments()#116
Open
rebelmachina wants to merge 1 commit intomainfrom
Open
feat(#114): ClusterManager party_assignments + set_party_assignments()#116rebelmachina wants to merge 1 commit intomainfrom
rebelmachina wants to merge 1 commit intomainfrom
Conversation
Add party_assignments: HashMap<Uuid, Uuid> field (entity_id → party_id) defaulting to empty — no behaviour change for existing callers. Add set_party_assignments() to register group memberships from outside (e.g. arcane-clustering-sim at startup). Populate party_id from party_assignments when building PlayerInfo in run_evaluation_cycle(), wiring group signals into AffinityEngine Phase 1b. Test D: verify two entities sharing a party UUID appear with matching party_id in WorldStateView.players; unregistered entity has party_id = None. Closes #114. Part of epic #112. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of epic #112. Closes #114.
Summary
party_assignments: HashMap<Uuid, Uuid>toClusterManager(empty by default — zero behaviour change for existing code)set_party_assignments()methodparty_idfrom the map when buildingPlayerInfoinrun_evaluation_cycle()party_idin WorldStateView; unregistered entity hasNoneBackwards compatibility
All existing tests pass unchanged.
party_assignmentsdefaults to empty so all existingrun_evaluation_cyclebehaviour is identical.Test plan
4 tests pass (3 existing + Test D).