Skip to content

Endings survive a restore; companion menu works on piped input; stale room-name tests - #84

Merged
corepunch merged 1 commit into
mainfrom
fix/limehouse-tests
Sep 29, 2026
Merged

corepunch merged 1 commit into
mainfrom
fix/limehouse-tests

Conversation

@corepunch

Copy link
Copy Markdown
Owner
  • Restored saves re-entered MAIN_LOOP outside the QUIT/RESTART handler, so Limehouse's winning accusation crashed llm.lua; both entries now share it, and the ending's text is returned (TAKE_OUTPUT) instead of dropped.
  • main.lua --companion hung on piped input (widget looped at EOF); EOF now stops it and non-terminal input gets the numbered menu.
  • Tests expecting room names on first visit (no longer printed by DESCRIBE-ROOM) now match room prose or check HERE; two Blackwood-based tests search the papers first; the companion test joins test-limehouse-walkthrough.

All of test-pure-zil, test-horror-all, test-limehouse-walkthrough (630/630, companion 13/13), test-wondertown-descriptions and lint-zil pass.

🤖 Generated with Claude Code

A restored save re-entered MAIN_LOOP outside the handler for QUIT and
RESTART, so Limehouse's winning accusation escaped llm.lua as a raw
control-signal table. Both entries now share one handler. The story's last
words, printed after the final READ, were dropped when the game ended; the
game now returns them (TAKE_OUTPUT), so hosts show the ending.

main.lua --companion hung forever on piped input: the arrow-key widget read
nil at end of input and looped. End of input now stops it, and non-terminal
input gets the numbered menu again.

Tests still expected room names on first visits, which DESCRIBE-ROOM no
longer prints (hosts set them as headings). Limehouse tests match each
room's prose; the Zork I walkthrough checks HERE. Two Blackwood-based tests
search the papers before taking the key. The companion test runs with the
Limehouse walkthrough target.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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