TOOLS: clean tools/index surface; keep Sprite Editor first-class; hide legacy; move/delete non-tool Advanced Systems & Extensions entries

Author: ToolboxAid

docs/dev/CODEX_COMMANDS.md

@@ -2,16 +2,26 @@ MODEL: GPT-5.4
 REASONING: high
 
 COMMAND:
-Create BUILD_PR_TOOL_REGISTRY_VALIDATOR_AND_SPRITE_FIRST_CLASS as a docs-aligned implementation PR.
+Create BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP as a docs-first, surgical surface cleanup PR for HTML-JavaScript-Gaming.
 
-REQUIREMENTS:
-1. Keep `tools/` first-class and registry-driven.
-2. Make `Sprite Editor` first-class and active in `tools/toolRegistry.js`.
-3. Keep `SpriteEditor_old_keep` preserved but excluded from active tool rendering.
-4. Update all active header/tool-surface rendering to use `entry.active === true`.
-5. Add a registry validator script that compares registry entries to filesystem directories.
-6. Fail validation on missing folders, missing registry entries, duplicate ids/names/paths, inactive `Sprite Editor`, or active legacy `SpriteEditor_old_keep`.
-7. Add a validation report output under `docs/dev/reports/` or equivalent repo-appropriate dev report location.
+GOAL:
+Make `tools/index.html` show only true first-class launchable tools. Keep `Sprite Editor` first-class and visible. Keep `SpriteEditor_old_keep` legacy and hidden. Remove `Advanced Systems & Extensions` from the tools landing page unless each item in that section is verified to be an actual active launchable tool under `tools/`.
+
+TARGET FILES TO AUDIT:
+- tools/index.html
+- tools/toolRegistry.js
+- tools/renderToolsIndex.js
+- any config/data feeding the tools page
+
+RULES:
+1. `tools/index.html` is a user-facing tool launcher, not a platform/dev page.
+2. Only render entries that are true first-class tools.
+3. Keep `Sprite Editor` active and visible.
+4. Keep `SpriteEditor_old_keep` hidden from active UI.
+5. If `Advanced Systems & Extensions` entries are platform/dev/internal, move or delete them.
+6. If they are duplicate/placeholder/dead, delete them.
+7. Do not expand scope beyond this cleanup.
+8. Preserve engine theme usage.
 
 EXPECTED ACTIVE TOOLS:
 - Vector Map Editor
@@ -20,8 +30,11 @@ EXPECTED ACTIVE TOOLS:
 - Parallax Scene Studio
 - Sprite Editor
 
-EXPECTED LEGACY TOOL:
-- SpriteEditor_old_keep
+EXPECTED RESULT:
+- no misleading `Advanced Systems & Extensions` on main tools page unless truly valid
+- no legacy tool leakage
+- registry and rendered landing page aligned
+- no broken links
 
 OUTPUT:
-<project folder>/tmp/BUILD_PR_TOOL_REGISTRY_VALIDATOR_AND_SPRITE_FIRST_CLASS.zip
+<project folder>/tmp/BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-Restore Sprite Editor as a first-class active tool, keep SpriteEditor_old_keep preserved but hidden, and add a filesystem-aware tool registry validator with report output.
+TOOLS: clean tools/index launcher surface; keep Sprite Editor first-class; hide legacy; remove non-tool sections

docs/dev/reports/change_summary.txt

@@ -1,6 +1,8 @@
-CHANGE SUMMARY
+BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP
 
-- Makes Sprite Editor first-class on the registry-driven tools surface
-- Preserves SpriteEditor_old_keep as legacy and non-rendered
-- Adds registry validator requirements to prevent registry/filesystem drift
-- Tightens rendering rules so only active registry entries appear in the header and tool landing surface
+Summary:
+- defines a strict rule for what may appear on tools/index.html
+- keeps Sprite Editor first-class and visible
+- keeps SpriteEditor_old_keep legacy and hidden
+- removes, relocates, or deletes Advanced Systems & Extensions items unless they are true launchable tools
+- keeps the cleanup surgical and surface-focused

docs/dev/reports/file_tree.txt

@@ -1,10 +1,11 @@
-docs/
-  pr/
-    BUILD_PR_TOOL_REGISTRY_VALIDATOR_AND_SPRITE_FIRST_CLASS.md
-  dev/
-    codex_commands.md
-    commit_comment.txt
-    reports/
-      change_summary.txt
-      validation_checklist.txt
-      file_tree.txt
+HTML-JavaScript-Gaming/
+  docs/
+    pr/
+      BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP.md
+    dev/
+      codex_commands.md
+      commit_comment.txt
+      reports/
+        change_summary.txt
+        validation_checklist.txt
+        file_tree.txt

docs/dev/reports/validation_checklist.txt

@@ -1,14 +1,9 @@
-VALIDATION CHECKLIST
-
-[ ] Sprite Editor folder exists under tools/
-[ ] Sprite Editor registry entry exists
-[ ] Sprite Editor is active=true
-[ ] Sprite Editor is legacy=false
-[ ] Sprite Editor appears in tools/index.html header
-[ ] SpriteEditor_old_keep does not appear in tools/index.html header
-[ ] Active render path is derived from registry active entries only
-[ ] Validator detects missing registry entries
-[ ] Validator detects missing folders
-[ ] Validator detects duplicate ids, names, and paths
-[ ] Validator fails if Sprite Editor is inactive
-[ ] Validator fails if SpriteEditor_old_keep is active
+[ ] tools/index.html audited for Advanced Systems & Extensions section
+[ ] tools/toolRegistry.js audited for Sprite Editor active entry
+[ ] tools/toolRegistry.js audited for SpriteEditor_old_keep legacy handling
+[ ] render logic verified to use active first-class tools only
+[ ] Sprite Editor visible on tools landing page
+[ ] SpriteEditor_old_keep hidden from tools landing page
+[ ] non-tool/internal/dev entries moved or deleted
+[ ] no dead links from tools landing page
+[ ] engine theme preserved across active tools page

docs/pr/BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP.md

@@ -0,0 +1,50 @@
+# BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP
+
+## Purpose
+Clean up `tools/index.html` so it acts as a user-facing launcher for true first-class tools only.
+
+## Scope
+In scope:
+- remove non-launchable planning and platform/internal sections from `tools/index.html`
+- keep the landing page aligned with the registry-driven first-class tool set
+- keep `Sprite Editor` first-class and visible
+- keep preserved legacy sprite tooling hidden
+- preserve engine theme usage
+
+Out of scope:
+- registry shape changes beyond what is needed for alignment
+- tool implementation changes
+- engine runtime changes
+- broader content refactors outside the tools landing surface
+
+## First-Class Launchable Tools
+- Vector Map Editor
+- Vector Asset Studio
+- Tilemap Studio
+- Parallax Scene Studio
+- Sprite Editor
+
+## Preserved But Hidden
+- `tools/SpriteEditor_old_keep/`
+
+## Implementation Summary
+- removed the `Planned Next` section from the tools landing page because it did not represent active launchable tools
+- removed the `Advanced Systems & Extensions` section from the tools landing page because those entries are platform/dev placeholders rather than actual tools under `tools/`
+- kept the landing page focused on the registry-driven first-class tool launcher only
+- preserved the engine theme and shared shell usage
+- kept registry and renderer behavior aligned with the active tool list
+
+## Files Changed
+- `tools/index.html`
+- `docs/pr/BUILD_PR_TOOLS_INDEX_SURFACE_CLEANUP.md`
+- `docs/dev/COMMIT_COMMENT.txt`
+
+## Validation Performed
+- `node scripts/validate-tool-registry.mjs`
+- `node scripts/validate-active-tools-surface.mjs`
+
+## Validation Summary
+- the active landing page remains aligned with the registry
+- `Sprite Editor` remains visible as a first-class tool
+- `SpriteEditor_old_keep` remains hidden from the active UI
+- no broken active tool links were introduced by this cleanup