Skip to content
This repository was archived by the owner on Jun 25, 2026. It is now read-only.

Support division-specific ruleset events and game configs - #55

Open
KyleHerndon wants to merge 2 commits into
kyle/multi-leaderboard-tablesfrom
kyle/ruleset-division-events-game-configs
Open

Support division-specific ruleset events and game configs#55
KyleHerndon wants to merge 2 commits into
kyle/multi-leaderboard-tablesfrom
kyle/ruleset-division-events-game-configs

Conversation

@KyleHerndon

@KyleHerndon KyleHerndon commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add policy_membership_events as a division/stage ruleset hook alias for membership transitions
  • add division/stage game_config overrides that are emitted on scheduled episode requests
  • make episode timeout/throttle logic prefer per-episode game config when present

Validation

  • No repo lint command is configured in pyproject.toml/README; ran git diff --cached --check before commit and it passed.
  • uv run pytest tests/test_commissioner_strategies.py tests/test_commissioner_websocket.py::test_configured_episode_timeout_prefers_game_duration_from_ticks tests/test_commissioner_websocket.py::test_configured_episode_timeout_reads_round_timeout_seconds tests/test_commissioner_websocket.py::test_configured_episode_timeout_reads_explicit_nested_duration tests/test_commissioner_websocket.py::test_episode_duration_limit_has_five_minute_floor tests/test_commissioner_websocket.py::test_episode_duration_limit_doubles_timeout tests/test_commissioner_websocket.py::test_episode_duration_limit_doubles_ten_minute_round_timeout tests/test_commissioner_websocket.py::test_episode_duration_limit_prefers_episode_game_config -q
  • Note: the full websocket TestClient suite hangs in this environment even with a minimal FastAPI TestClient app, so I validated the non-TestClient timeout helpers plus the full ruleset strategy suite.

@KyleHerndon
KyleHerndon force-pushed the kyle/multi-leaderboard-tables branch from 2f9b959 to 5f8f2b4 Compare June 24, 2026 22:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant