Skip to content

test(dungeons): verify encounter doors survive restart - #85

Merged
Mathih13 merged 2 commits into
mainfrom
feat/vanilla-compatibility-first-pass
Oct 15, 2026
Merged

Mathih13 merged 2 commits into
mainfrom
feat/vanilla-compatibility-first-pass

Conversation

@Mathih13

@Mathih13 Mathih13 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Deadmines had no Package-owned test of encounter progress across a restart. The new durable test completes part of the dungeon, restarts a private persistent Standalone, then finishes the bosses and restarts again. It checks all four doors and their saved encounter states at each step.

The test uses the existing Package API and production kill and gameobject-use paths. Normal Module persistence already preserves the door rows, so no recovery behavior is added. The Smite fixture now checks the issued movement destination because movement finishes on later ticks. Fixture setup refuses imported content, and a missing-door log now identifies the Instance.

What you verified

Local checks used Core tree 5003aca5, based on 9be23cd1, with Package revision cee7c6d installed.

  • ./.github/check-core-tip.sh with all Official Packages: 37 Package API checks, 1,706 Module unit tests and the debug-reducer Wasm build passed.
  • Package test runner strict Clippy passed.
  • deadmines_doors: the durable test passed through two real process restarts, checking partial and completed encounter progress.
  • Existing Core deadmines_package durable test passed, including the Smite movement assertion.
  • Generated bindings passed the freshness check.
  • Rebase onto the merged identity refresh changed no files under dungeons/.

Independent adversarial review passed for the final Package scope. Comment review passed for all seven changed files, including new files, removed comments and retained contracts. README pointers resolve.

What you did not verify

No real-client door animation Verification. These tests cover persistent Standalone restarts; they do not add or claim Instance recreation support.

This PR targets Package main. It uses the existing Package API and can merge independently of the Core stack.

The same-name Core branch provides the paired CI build: behavior evidence inventory.

Models: GPT-6.1 Sol and GPT-6 Astra through Codex, with adversarial review by Claude Opus 5.5 through Claude Agent, in T3 Code.

@Mathih13
Mathih13 force-pushed the feat/vanilla-compatibility-first-pass branch from c241ec4 to e65a603 Compare October 15, 2026 13:02
@Mathih13 Mathih13 changed the title fix(dungeons): reopen boss doors from saved completion test(dungeons): verify encounter doors survive restart Oct 15, 2026
@Mathih13
Mathih13 merged commit 5248c22 into main Oct 15, 2026
21 checks passed
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