Repository navigation
FRLG maps: one map set with per-map status - #81
Merged
Merged
Conversation
Merge runtime/ (57 approved maps) and source-candidate/ (140 generated maps) into the single directory runtime/, which the published manifest and the Arduino-Source resource pin already read. Each FRLG manifest entry now has a status: verified (approved-runtime-pack, or a console-run record with run, date and checkpoint) or candidate (generated from pret, unreviewed). For the 45 maps in both, the reviewed runtime bytes are kept unchanged. The 95 candidate-only maps join as candidates, without the in-file generation_status (status lives in the manifest, so promotion never changes map bytes or hashes). frlg_map_set checks statuses, syncs candidate entries and promotes a map after a console pass. The generator --check (--manifest) and the import audit validate the new layout; the pack audit allows only scope boundaries and candidate-to-verified links pending a relink. Mainland Pokemon Centers and elevators move to a scripted-in-autostory scope group. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss
dayfine
marked this pull request as ready for review
November 10, 2026 02:00
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.
FRLG maps are now one set in
data/games/frlg/maps/runtime/, with each map'sstatus(verifiedorcandidate) recorded in the manifest instead of split across two directories.verifiedwith byte-identical files, keeping their review notes (verification,terrain_corrections). For the 45 maps that had both copies, the reviewed runtime file wins. The 95 candidate-only maps join ascandidate, andsource-candidate/is gone.frlg_map_setmodule providescheck,sync-candidatesandpromote --run --date --checkpoint.--check(--manifest) and the import audit now validate the single set. The pack audit allows only scope boundaries and candidate→verified warps that are waiting for the verified map to be relinked (31, all listed in the README). Fixing those means editing reviewed maps, which is left for a follow-up.scripted-in-autostoryscope group: they are scripted in AutoStory, not mapped. The 8 Centers verified earlier stay in the set.6c2f5c8, because906956erejects the new glyphs.Testing
contract_format --check,json_schema --check-published, the generator--checkagainst the pinned pret cache, and the import audit with--schema.consumer_sync update/prepare/check --layout resourcespass with this branch's tools and with main6c2f5c8. Tools at906956efail with "unknown tile glyph".🤖 Generated with Claude Code
https://claude.ai/code/session_013VUubhHeYSMGSSDXZkKZss