Skip to content

Commit 3cb09fc

Browse files
ToolboxAidToolboxAid
committed
Applied the PR at the docs stage by replacing the placeholder with a full apply record in APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md, and refreshed the apply reports in change_summary.txt, file_tree.txt, and validation_checklist.txt.
1 parent c7adfff commit 3cb09fc

4 files changed

Lines changed: 83 additions & 29 deletions

File tree

‎docs/dev/reports/change_summary.txt‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
1-
Created implementation bundle for BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.
1+
Created APPLY bundle for APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.
22

33
Summary:
4-
- Extends the shared registry helper with additive `tilemaps` support.
5-
- Updates Tile Map Editor to load/save `project.assets.json` and persist `assetRefs.tilemapId` plus `assetRefs.tilesetId`.
6-
- Registers `tilemaps` and relevant `tilesets` additively while preserving existing registry groups and unknown safe fields.
7-
- Keeps scope surgical: no engine changes and no Parallax Editor work in this PR.
4+
- Confirms the built Tile Map Editor registry adoption slice is ready to apply as implemented.
5+
- Verifies build/package scope stays limited to the approved Tile Map Editor registry files and docs.
6+
- Reaffirms legacy compatibility, additive registry preservation, and no engine or Parallax scope expansion.

‎docs/dev/reports/file_tree.txt‎

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
1-
BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta/
2-
├── docs/
3-
│ ├── dev/
4-
│ │ └── reports/
5-
│ │ ├── change_summary.txt
6-
│ │ ├── file_tree.txt
7-
│ │ └── validation_checklist.txt
8-
│ └── pr/
9-
│ └── BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md
10-
└── tools/
11-
├── Tile Map Editor/
12-
│ ├── README.md
13-
│ └── main.js
14-
└── shared/
15-
└── projectAssetRegistry.js
1+
APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta/
2+
└── docs/
3+
├── dev/
4+
│ └── reports/
5+
│ ├── change_summary.txt
6+
│ ├── file_tree.txt
7+
│ └── validation_checklist.txt
8+
└── pr/
9+
└── APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md

‎docs/dev/reports/validation_checklist.txt‎

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
1-
BUILD VALIDATION CHECKLIST
1+
APPLY VALIDATION CHECKLIST
22

3-
- [x] Tile Map Editor registry adoption implemented without engine core API changes
4-
- [x] Shared registry helpers reused and extended additively for `tilemaps`
5-
- [x] `project.assets.json` load/save supported in Tile Map Editor
6-
- [x] Tile map documents persist optional `assetRefs.tilemapId` and `assetRefs.tilesetId`
7-
- [x] Registry updates additively preserve existing groups and unknown safe fields
8-
- [x] Legacy tile map JSON remains loadable when `assetRefs` are absent
9-
- [x] No Parallax Editor implementation work included in this PR
10-
- [x] Syntax validation passed for changed JS files
3+
- [x] Build artifact `tmp/BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta.zip` exists
4+
- [x] Build artifact contents confirmed limited to approved PR files
5+
- [x] Plan/build/apply alignment confirmed for Tile Map Editor registry adoption
6+
- [x] Legacy compatibility and additive registry preservation confirmed in apply notes
7+
- [x] No engine core API changes or Parallax Editor scope expansion introduced during APPLY
8+
- [x] Syntax validation re-confirmed for changed JS files
9+
- [x] APPLY bundle is docs-only

‎docs/pr/APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md‎

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
Toolbox Aid
2+
David Quesenberry
3+
04/04/2026
4+
APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md
5+
6+
# APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION
7+
8+
## Goal
9+
Apply the approved `BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION` slice exactly as built, verify readiness, confirm legacy compatibility, and document the Tile Map Editor registry behavior now accepted into the repo without adding new implementation scope.
10+
11+
## Apply Scope
12+
In scope:
13+
- Validation and acceptance of the completed Tile Map Editor registry adoption slice
14+
- Documentation/report refresh for APPLY stage
15+
- Final APPLY delta ZIP packaging for this docs-only step
16+
17+
Out of scope:
18+
- New implementation code
19+
- Engine/core refactor
20+
- Sprite Editor rework
21+
- Parallax Editor work
22+
23+
## Readiness Verification
24+
- Build artifact confirmed: `tmp/BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta.zip`
25+
- Build artifact contents confirmed limited to approved Tile Map Editor registry files and supporting docs
26+
- Plan/build alignment confirmed against:
27+
- `docs/pr/PLAN_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md`
28+
- `docs/pr/BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION.md`
29+
- Working tree reviewed before final APPLY packaging
30+
31+
## Compatibility Verification
32+
- Legacy tile map JSON remains backward-loadable when `assetRefs` is absent.
33+
- Tile Map Editor continues to treat registry load as optional rather than required.
34+
- `assetRefs.tilemapId` and `assetRefs.tilesetId` are optional on input and normalize safely.
35+
- Missing registry entries degrade with status messaging and fallback behavior instead of hard failure.
36+
- Existing registry groups and unknown safe fields remain preserved through additive helper behavior.
37+
38+
## User-Visible Behavior Changes (Confirmed)
39+
- Tile Map Editor can load and save `project.assets.json`.
40+
- Saving a tile map now additively registers a `tilemaps` entry and relevant `tilesets` metadata.
41+
- Tile map JSON now persists optional `assetRefs.tilemapId` and `assetRefs.tilesetId`.
42+
- Tile set registry references can restore tileset path metadata when local path data is absent.
43+
- Soft status messaging communicates whether registry resolution succeeded or remained unresolved.
44+
45+
## Validation Executed During APPLY
46+
- Confirmed build ZIP exists at `tmp/BUILD_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta.zip`
47+
- Confirmed build ZIP contents are limited to approved PR files
48+
- `node --check tools/shared/projectAssetRegistry.js`
49+
- `node --check tools/Tile Map Editor/main.js`
50+
- Confirmed APPLY bundle scope is docs-only for this step
51+
52+
## Apply Decision
53+
Approved to apply.
54+
55+
## Commit Comment
56+
`build(tilemap-editor): adopt shared asset registry contracts for tilemaps`
57+
58+
## Package
59+
`tmp/APPLY_PR_TILEMAP_EDITOR_ASSET_REGISTRY_ADOPTION_delta.zip`
60+
61+
## Next Command
62+
`PLAN_PR_PARALLAX_EDITOR_ASSET_REGISTRY_ADOPTION`

0 commit comments

Comments
 (0)