Skip to content

fix(validation): align celestial-v1 filename tiling - #260

Merged
zoeyrose merged 2 commits into
mainfrom
fix/tools-525-celestial-tiling
Sep 3, 2026
Merged

fix(validation): align celestial-v1 filename tiling#260
zoeyrose merged 2 commits into
mainfrom
fix/tools-525-celestial-tiling

Conversation

@zoeyrose

@zoeyrose zoeyrose commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

Align authored-exit validation with Classic's filename-derived horizontal tiling for celestial_schema 1 maps.

Implementation / behavior

  • Parse signed v1 map coordinates using Classic's filename convention.
  • Derive only existing horizontal neighbors for empty slots; explicit paths remain authoritative.
  • Keep vertical links explicit for reciprocal celestial stack validation.
  • Add fixtures for derived, missing, overridden, and both edge-stair landing cases.

Tracks the Classic runtime issue without closing it: atrinik/classic#525

Validation

  • Focused authored-exit tests and repository content validation.
  • git diff --check.

Limitations / follow-up

This pull request owns static validation; the coordinated Classic pull request owns runtime behavior.

Behavior clarification

Static validation mirrors Classic's general coordinate-filename lookup for every map, regardless of any celestial_schema marker. It fills empty slots only when candidate files exist and leaves explicit paths authoritative. Celestial-specific validation remains separate from this path-discovery rule.

@zoeyrose
zoeyrose marked this pull request as ready for review September 2, 2026 19:59
@zoeyrose zoeyrose moved this from Inbox to Review in Atrinik work Sep 2, 2026
@zoeyrose
zoeyrose merged commit 08e8bc8 into main Sep 3, 2026
9 checks passed
@zoeyrose
zoeyrose deleted the fix/tools-525-celestial-tiling branch September 3, 2026 01:57
@github-project-automation github-project-automation Bot moved this from Review to Done in Atrinik work Sep 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant