Skip to content

Stop accepting platform-provided variant player counts#47

Open
KyleHerndon wants to merge 1 commit into
mainfrom
kyle/episode-roster-player-count
Open

Stop accepting platform-provided variant player counts#47
KyleHerndon wants to merge 1 commit into
mainfrom
kyle/episode-roster-player-count

Conversation

@KyleHerndon

Copy link
Copy Markdown
Contributor

Summary

  • Remove num_agents from commissioner VariantInfo.
  • Stop deriving minimum_champions from platform variant player counts.
  • Use commissioner-owned config or scheduled entries when legacy strategy code still needs an internal count.
  • Update strategy tests for roster-defined scheduling.

Validation

  • uv run pytest tests/test_commissioner_strategies.py -q
  • uv run python -m compileall commissioners tests -q

Notes

  • uv run ruff check is not available in this project environment.
  • A targeted websocket integration test timed out locally before this final validation pass, so I left it out of the required PR gate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant