Skip to content

refactor(maps): remove redundant filename-derived tiling metadata - #262

Merged
zoeyrose merged 3 commits into
mainfrom
issue/content-261-remove-redundant-tiling
Sep 3, 2026
Merged

refactor(maps): remove redundant filename-derived tiling metadata#262
zoeyrose merged 3 commits into
mainfrom
issue/content-261-remove-redundant-tiling

Conversation

@zoeyrose

@zoeyrose zoeyrose commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

Remove authored filename-redundant map tiling metadata from atrinik/content@main while preserving intentional topology policy and explicit overrides.

Implementation / behavior

  • Audit every coordinate-suffixed authored map against Classic's filename rules and the existing-neighbor check.
  • Remove eligible horizontal tile_path_1 through tile_path_8 records and their paired boundary records when no authored seam policy or override is present.
  • Apply the same review to vertical tile_path_9 and tile_path_10 only when the filename-derived vertical runtime contract and reciprocal stack checks are live and verified.
  • Preserve explicit overrides, policy-bearing seams, unparseable or legacy map names, missing neighbors, and unsupported vertical stacks.
  • Add a deterministic validation guard that reports filename-redundant authored tiling metadata before it can be reintroduced.
  • Keep generated Classic target artifacts derived from the authored source and preserve map layout and formatting conventions.

Validation

  • python3 tools/validate.py
  • Focused deterministic tiling-audit and validator tests.
  • The applicable Classic target/runtime validation from the same source revision.
  • git diff --check

Limitations / follow-up

Vertical cleanup is gated by the explicit filename-derived vertical runtime contract in atrinik/classic#530; no vertical record is removed until that contract and reciprocal checks are live and verified.

Closes #261

@zoeyrose zoeyrose moved this to Review in Atrinik work Sep 3, 2026
@zoeyrose
zoeyrose marked this pull request as ready for review September 3, 2026 19:20
@zoeyrose
zoeyrose merged commit 92a5b7c into main Sep 3, 2026
8 checks passed
@zoeyrose
zoeyrose deleted the issue/content-261-remove-redundant-tiling branch September 3, 2026 19:31
@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.

refactor(maps): remove redundant filename-derived tiling metadata

1 participant