Emit movement events from Manager when event sourcing is enabled #40

New issue

Open

opened 2026-03-14 23:54:57 +00:00 by freemo · 0 comments

freemo commented

2026-03-14 23:54:57 +00:00

Owner

Metadata

Commit Message: feat(event-sourcing): emit container change events on player movement
Branch: feature/m1-v12-movement-events

Background

Movement actions change a player's @container (the room they're in) but do not emit GameObjectContainerUpdated events through the event sourcing system.

Expected Behavior

When event_sourcing_enabled is true, the Manager's container-change path emits UpdateGameObjectContainer commands for player movement.

Acceptance Criteria

Manager or GameObject#container= emits UpdateGameObjectContainer on container change
Events are only emitted when ServerConfig[:event_sourcing_enabled] is true
Errors are caught and logged without crashing the game

Subtasks

Verify GameObject#container= already emits (check existing hook)
If not, add command emission in the setter
Tests: Unit test verifying event emission on movement
Run bundle exec rake unit, fix any errors

Definition of Done

This issue is complete when all subtasks are done, commit created on the specified branch, PR submitted and merged.

## Metadata - **Commit Message**: `feat(event-sourcing): emit container change events on player movement` - **Branch**: `feature/m1-v12-movement-events` ## Background Movement actions change a player's `@container` (the room they're in) but do not emit `GameObjectContainerUpdated` events through the event sourcing system. ## Expected Behavior When `event_sourcing_enabled` is true, the Manager's container-change path emits `UpdateGameObjectContainer` commands for player movement. ## Acceptance Criteria - [ ] `Manager` or `GameObject#container=` emits `UpdateGameObjectContainer` on container change - [ ] Events are only emitted when `ServerConfig[:event_sourcing_enabled]` is true - [ ] Errors are caught and logged without crashing the game ## Subtasks - [ ] Verify `GameObject#container=` already emits (check existing hook) - [ ] If not, add command emission in the setter - [ ] Tests: Unit test verifying event emission on movement - [ ] Run `bundle exec rake unit`, fix any errors ## Definition of Done This issue is complete when all subtasks are done, commit created on the specified branch, PR submitted and merged.