Repository navigation
Replay current BDSP mine checkpoint from shared authored JSON - #74
Merged
Merged
Conversation
dayfine
force-pushed
the
story/bdsp-mine-authored
branch
from
November 8, 2026 23:31
9a16a85 to
90fed78
Compare
dayfine
marked this pull request as ready for review
November 8, 2026 23:36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Base:
story/parcel-replay(parent #73). This is the BDSP counterpart to the FRLG authored-JSON pilot. The native companion is branchbdsp/checkpoint-json, based on Arduino-Source #226 at0738cd55b.Author the current mine/Roark
01.03checkpoint as the same lean JSON the native table loads: 15 events, preserving its two-tile arrivals, wall/door variants and BDSP save-card/failure configuration. Barry/initial healing now belong to01.02b; the old 16-event draft is not used.Nine scenarios cover the complete maneuver and failed entry, transitions, wild interruption, Roark dialogue/departure, Hiker clearance and healing. Selected cases assert every semantic action. Both city exit tiles converge after 37/38 northward tiles; shortened legs cannot be repaired by assigning an authored arrival. An interrupted south-five leg retains its requested distance and exposes two uncredited candidate tiles with unknown facing.
How to review
Start with
data/story/bdsp-mine-pilot/README.md, the authored.plan.json, thenCASES.md. The first commit extracts the existing FRLG spatial runner intostory/spatial_replay.py; its tests still pass. The second addsgames/bdsp_story_replay.py, strict per-game schema/binding support and adjacent regression tests. Usegit diff --color-moved=dimmed-zebra --color-moved-ws=allow-indentation-change origin/story/parcel-replay...story/bdsp-mine-authoredto inspect the move separately.Pinned map bytes/resource revisions are unchanged. Missing mine warp links require separate scripted arrival evidence. NPC clearance windows and scene/heal outcomes are synthetic. Native robust planner costs, retry/save parity, analog/controller/detector timing and live recovery remain unclaimed. Native declared fallback beliefs are kept separate from physical pose uncertainty.
Validation: full local Python suite passed (535 tests, 50 optional dependency skips); the added unreachable-encounter guard then passed all 12 focused BDSP tests. Existing FRLG replay tests pass after the shared extraction. Compact JSON and whitespace checks pass. Full schema and Python-version CI are pending. No console/profile/save was used.