* Initial plan
* Refactor timer system to run on main thread instead of child processes
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Remove obsolete GameProcess.js and simplify TimerManager to stub
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Address code review feedback: add comments and reduce duplication
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Remove comments, delete TimerManager, and simplify SOURCE_TIMER_EVENT
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Fix cross-instance timer communication and expired timer handling
- GET_TIME_REMAINING now publishes response through Redis for cross-instance communication
- SOURCE_TIMER_EVENT handles GET_TIME_REMAINING even when timer is not running locally
- Returns 0 time remaining when timer has expired or timerParams is null
- Updated test to expect Redis publish instead of direct method call
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Simplify GET_TIME_REMAINING flow and add ended flag
- Add ended: true flag to timerParams when timer completes
- Simplify TIMER_EVENT to check ended flag first, then local timer, then publish SOURCE_TIMER_EVENT
- SOURCE_TIMER_EVENT only publishes GET_TIME_REMAINING if it has the timer
- Remove GET_TIME_REMAINING from handleTimerCommand helper
- Direct socket emission for local timers instead of Redis publish
- Updated test to verify socket emission
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Refactor GET_TIME_REMAINING into handleTimerCommand switch case
- Moved GET_TIME_REMAINING logic from TIMER_EVENT handler into handleTimerCommand function
- Simplified TIMER_EVENT to just call handleTimerCommand for all timer operations
- Maintains same flow: check ended flag, check local timer, publish SOURCE_TIMER_EVENT if needed
- All tests passing
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Remove unused getTimeRemaining function from GameManager
- Function is no longer called after refactoring GET_TIME_REMAINING into handleTimerCommand
- All timer time remaining logic now handled in Events.js handleTimerCommand switch case
- Tests still passing
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* Reset timerParams.ended flag when restarting game
- Added reset of ended flag to false in restartGame function
- Ensures restarted games don't have timer marked as already ended
- Placed right after timer cleanup for logical grouping
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
* remuse unused constant
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: AlecM33 <24642328+AlecM33@users.noreply.github.com>
Co-authored-by: AlecM33 <leohfx@gmail.com>
