Validate and close remaining non-3D roadmap residue in one combined pass

DavidQ · DavidQ · commit 4b24b6c09db7 · 2026-04-14T23:15:10.000-04:00
BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED
diff --git a/docs/dev/CODEX_COMMANDS.md b/docs/dev/CODEX_COMMANDS.md
@@ -2,10 +2,7 @@ MODEL: GPT-5.4
 REASONING: high
 
 COMMAND:
-Create `BUILD_PR_REMAINING_ROADMAP_VALIDATE_OR_CLOSEOUT_COMBINED`.
-
-Goal:
-Validate the remaining listed roadmap items against repo truth and close them in one combined pass wherever possible.
+Create BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED.
 
 Validate-first requirements:
 1. Inspect repo truth for each target item before changing anything.
@@ -18,41 +15,17 @@ Validate-first requirements:
 4. If partially complete or incomplete:
    - do the smallest valid work needed to close it
    - avoid unrelated expansion
+   - do not open any 3D lane work
 
 Target items:
-
-Tooling Strategy By Need
-- 2D tool stabilization before 3D tool expansion
-
-Next Planning / Normalization Lanes
-- Apply master roadmap baseline
-- Normalize samples phase structure
-
-Repo Operator + Asset Conversion Scripting Lanes
 - Existing games asset folders updated so existing images / vectors / related runtime assets can be transformed into tool-editable `data/` objects, with corresponding project JSON updates
-
-Later Capability Lanes
-- FEATURE: Fullscreen Bezel Overlay System
-
-Final Cleanup Lane
+- Execute 2D capability polish lanes
 - Reduce legacy footprint after replacements are proven
 
-Required work:
-1. Reuse already-completed repo work wherever truth supports closure.
-2. Perform only the smallest valid residue work for any incomplete items.
-3. Keep changes coherent and surgical.
-4. Update roadmap status markers only.
-5. Report:
-   - what was already complete
-   - what was completed in this PR
-   - what remains open, if anything, with exact blockers
+Roadmap/meta handling:
+- update summary/status rows that are now outdated because repo truth changed
+- status markers only
+- do NOT rewrite roadmap text
 
 Final packaging step is REQUIRED:
-- package ALL changed files into this exact repo-structured ZIP:
-  `<project folder>/tmp/BUILD_PR_REMAINING_ROADMAP_VALIDATE_OR_CLOSEOUT_COMBINED.zip`
-
-Hard rules:
-- validate first, build second
-- combine aggressively to reduce PR count
-- no unrelated repo changes
-- no missing ZIP
+<project folder>/tmp/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED.zip
diff --git a/docs/dev/COMMIT_COMMENT.txt b/docs/dev/COMMIT_COMMENT.txt
@@ -1,2 +1,2 @@
-Validate and close remaining roadmap residue in one combined pass
-BUILD_PR_REMAINING_ROADMAP_VALIDATE_OR_CLOSEOUT_COMBINED
+Validate and close remaining non-3D roadmap residue in one combined pass
+BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED
diff --git a/docs/dev/NEXT_COMMAND.txt b/docs/dev/NEXT_COMMAND.txt
@@ -1 +1 @@
-BUILD_PR_REMAINING_ROADMAP_RESIDUE_ONLY_IF_NEEDED
+BUILD_PR_REMAINING_NON_3D_RESIDUE_ONLY_IF_NEEDED
diff --git a/docs/dev/reports/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED_report.md b/docs/dev/reports/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED_report.md
@@ -0,0 +1,47 @@
+# BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED Report
+
+## Validate-First Classification
+
+### 1) Existing games asset folders updated so existing images / vectors / related runtime assets can be transformed into tool-editable `data/` objects, with corresponding project JSON updates
+- classification: already complete (for qualifying existing games with substantive runtime-domain assets)
+- evidence:
+  - substantive runtime-domain asset games found: `Asteroids` only (`sprites`, `tilemaps`, `parallax`, `vectors` with non-data runtime files)
+  - `games/Asteroids/assets/tools.manifest.json` exists
+  - `games/Asteroids/assets/{sprites,tilemaps,parallax,vectors}/data/` all exist with real data JSON files
+  - focused ownership validator passes: `node scripts/validate-asset-ownership-strategy.mjs`
+- action in this PR:
+  - roadmap status marker updated to `[x]`
+
+### 2) Execute 2D capability polish lanes
+- classification: already complete
+- evidence:
+  - Section 12 (`2D Capability Track`) is fully complete in active roadmap
+  - focused 2D capability test passes: `node tests/core/Engine2DCapabilityCombinedFoundation.test.mjs`
+- action in this PR:
+  - roadmap status marker updated to `[x]`
+
+### 3) Reduce legacy footprint after replacements are proven
+- classification: partially complete
+- evidence:
+  - legacy inventory/policy groundwork is complete (Section 15 markers mostly closed in prior lanes)
+  - replacement-proven final reduction execution is not complete yet
+  - retained legacy surfaces remain by policy/traceability:
+    - `docs/archive/tools/SpriteEditor_old_keep/`
+    - `docs/dev/reports/classes_old_keep_policy_decision.md`
+- action in this PR:
+  - left open (`[ ]`) by truth
+
+## Summary/Status Row Updates
+- updated summary row marker by status only:
+  - `later capability lanes are 11 / 12` -> `[x]`
+
+## What Was Completed In This PR
+- closed by status reconciliation:
+  - existing games asset-folder conversion lane (non-3D target item)
+  - execute 2D capability polish lanes
+- updated one now-truthful summary/status row:
+  - later capability lanes 11 / 12
+
+## What Remains Open + Exact Blocker
+- `Reduce legacy footprint after replacements are proven`
+  - blocker: final replacement-proven legacy reduction execution lane has not been completed yet; remaining legacy surfaces are intentionally retained until that dedicated cleanup execution is done.
diff --git a/docs/dev/reports/change_summary.txt b/docs/dev/reports/change_summary.txt
@@ -1,3 +1,3 @@
-- Added validate-first combined closeout PR for the remaining listed roadmap items
-- Requires repo-truth inspection before any new work is created
-- Intended to reduce PR count by closing already-complete items with status-only updates and building only true residue
+- Added validate-first combined closeout PR for the remaining non-3D roadmap items
+- Focuses on asset conversion scripting, 2D polish residue, and legacy footprint reduction
+- Requires status-only roadmap/meta updates after repo-truth validation
diff --git a/docs/dev/reports/validation_checklist.txt b/docs/dev/reports/validation_checklist.txt
@@ -1,7 +1,6 @@
-- each listed item inspected before any creation/work
+- each listed non-3D item inspected before any creation/work
 - already-complete items closed by status update only
 - only incomplete residue received implementation work
+- no 3D lane work introduced
 - roadmap updated by status markers only
 - any remaining blockers are explicit
-- output ZIP created at:
-  <project folder>/tmp/BUILD_PR_REMAINING_ROADMAP_VALIDATE_OR_CLOSEOUT_COMBINED.zip
diff --git a/docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md b/docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md
@@ -600,7 +600,7 @@
 - [x] Expand testing/validation structure
 
 ### Repo Operator + Asset Conversion Scripting Lanes
-- [.] Existing games asset folders updated so existing images / vectors / related runtime assets can be transformed into tool-editable `data/` objects, with corresponding project JSON updates
+- [x] Existing games asset folders updated so existing images / vectors / related runtime assets can be transformed into tool-editable `data/` objects, with corresponding project JSON updates
 - [x] Add the ability for a PowerShell script to create a new game from template, including a project scaffold for the tools
 - [x] Add scripts to prep / update / delete the repo so it can be placed on a website
 - [x] Add scripts to switch between Pay-as-you-go and Codex plan modes, and scripts to input API key material and validate it
@@ -609,7 +609,7 @@
 - [x] FEATURE: Fullscreen Bezel Overlay System - Render game in full screen with optional bezel artwork layer (static or animated) surrounding the active playfield, preserving aspect ratio and supporting per-game/theme bezel assets without modifying core engine rendering.
 
 ### Later Capability Lanes
-- [.] Execute 2D capability polish lanes
+- [x] Execute 2D capability polish lanes
 - [ ] Execute phase-16 / 3D capability lanes
 
 ### Final Cleanup Lane
@@ -619,7 +619,7 @@
 ### Recommended Final Status Summary
 - [ ] current active execution lanes are 3 / 6 / 8
 - [ ] next planning lanes are 2 / 5 / 7 / 9 / 10
-- [ ] later capability lanes are 11 / 12
+- [x] later capability lanes are 11 / 12
 - [ ] final cleanup lane is 13
 
 ---
diff --git a/docs/pr/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED.md b/docs/pr/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED.md
@@ -0,0 +1,30 @@
+# BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED
+
+## Purpose
+Close the remaining non-3D roadmap items in one validate-first combined pass.
+
+## Validate-first rule
+For each target item:
+- inspect repo truth first
+- if already complete, mark complete
+- if partially complete, do the smallest valid remaining work
+- if incomplete, build only the smallest necessary residue
+- do not recreate work that already exists
+
+## Target non-3D items
+- Existing games asset folders updated so existing images / vectors / related runtime assets can be transformed into tool-editable `data/` objects, with corresponding project JSON updates
+- Execute 2D capability polish lanes
+- Reduce legacy footprint after replacements are proven
+
+## Roadmap/meta handling
+- update summary/status rows if repo truth now supports it
+- status markers only
+- no roadmap text rewrite
+
+## Constraints
+- no 3D work
+- no broad repo-wide cleanup
+- no unrelated repo changes
+
+## Packaging
+<project folder>/tmp/BUILD_PR_REMAINING_NON_3D_VALIDATE_OR_CLOSEOUT_COMBINED.zip

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-BUILD_PR_REMAINING_ROADMAP_RESIDUE_ONLY_IF_NEEDED`
	`1`	`+BUILD_PR_REMAINING_NON_3D_RESIDUE_ONLY_IF_NEEDED`