Epic: EventHandler Deprecation and Removal #192

New issue

Open

opened 2026-03-15 23:08:54 +00:00 by freemo · 0 comments

freemo commented

2026-03-15 23:08:54 +00:00

Owner

Background and Context

The EventHandler processes events using reflection: Module.const_get(event.type).send(event.action, ...). It is retained only for combat delayed events (punch_result, slash_result, kick_result) and NPC Reacts action sequences. The spec has a 4-phase deprecation plan: freeze, extract CombatService, migrate Reacts, remove.

Demonstrable Outcome

EventHandler class deleted. Combat uses CombatService. Reacts uses PriorityQueue.

Acceptance Criteria

EventHandler class deleted
@event_handler removed from Manager
Combat delayed events use CombatService
Reacts NPC sequences use PriorityQueue

## Background and Context The `EventHandler` processes events using reflection: `Module.const_get(event.type).send(event.action, ...)`. It is retained only for combat delayed events (punch_result, slash_result, kick_result) and NPC Reacts action sequences. The spec has a 4-phase deprecation plan: freeze, extract CombatService, migrate Reacts, remove. ## Demonstrable Outcome `EventHandler` class deleted. Combat uses `CombatService`. Reacts uses `PriorityQueue`. ## Acceptance Criteria - EventHandler class deleted - `@event_handler` removed from Manager - Combat delayed events use CombatService - Reacts NPC sequences use PriorityQueue