Skip to content

feat(gamestate): expose last_tarot_planet for The Fool - #221

Closed
BGarber42 wants to merge 1 commit into
coder:mainfrom
BGarber42:feat/gamestate-last-tarot-planet
Closed

feat(gamestate): expose last_tarot_planet for The Fool#221
BGarber42 wants to merge 1 commit into
coder:mainfrom
BGarber42:feat/gamestate-last-tarot-planet

Conversation

@BGarber42

Copy link
Copy Markdown

Summary

  • Expose optional last_tarot_planet on GameState from G.GAME.last_tarot_planet (card key The Fool would create, e.g. c_hermit).
  • Omitted when no Tarot/Planet has been used yet this run.
  • Document in OpenRPC, docs/api.md, and Lua types; add integration tests.

Test plan

  • uv run pytest tests/lua/endpoints/test_gamestate.py::TestGamestateTopLevel::test_last_tarot_planet_absent_at_run_start -v
  • uv run pytest tests/lua/endpoints/test_gamestate.py::TestGamestateTopLevel::test_last_tarot_planet_after_using_tarot -v
  • uv run pytest tests/lua/endpoints/test_gamestate.py::TestGamestateTopLevel::test_last_tarot_planet_after_using_planet -v
  • Manually: start a run, gamestate has no last_tarot_planet; use Hermit; gamestate shows last_tarot_planet: c_hermit`

Surface G.GAME.last_tarot_planet on GameState so clients know which Tarot/Planet The Fool would copy, instead of only the static effect text.
@S1M0N38

S1M0N38 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

close in favor of

Thanks for this PR. I credited you as co-author. These changes will land in balatrobot v2.

@S1M0N38 S1M0N38 closed this Jul 21, 2026
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.

2 participants