Unify active tools under first-class tools/ registry, rename Sprite Editor V3 to Sprite Editor, generate active tool navigation from a single source of truth, and add validation to prevent naming/path drift.

Author: ToolboxAid

docs/dev/CODEX_COMMANDS.md

@@ -2,37 +2,45 @@ MODEL: GPT-5.4
 REASONING: high
 
 COMMAND:
-Execute this final release verification suite in order.
+Create BUILD_PR_TOOLS_REGISTRY_AND_SPRITE_RENAME as a docs-aligned implementation PR.
 
-Step 1:
-Verify and apply vector-only runtime acceptance.
-- Confirm `sprite.asteroids-demo` is no longer an active packaged/runtime dependency
-- Confirm validation/package/runtime/debug/profiler/export/publishing remain ready
-- Package as APPLY_PR_VECTOR_ONLY_RUNTIME_VERIFIED
+REPO RULES:
+- Treat tools/ as a first-class product surface.
+- Keep this PR surgical and limited to tool naming, tool registry, generated navigation, and validation.
+- Do not delete preserved legacy content unless absolutely required and explicitly approved.
+- Prefer one canonical source of truth for active tool metadata.
+- Preserve existing behavior except where required for the canonical Sprite Editor path and tool-surface cleanup.
 
-Step 2:
-Verify and apply vector-native template sample-game acceptance.
-- Confirm standalone sample validates/packages/runs with vector-only assets
-- Confirm no sprite fallback dependency is present
-- Package as APPLY_PR_VECTOR_TEMPLATE_SAMPLE_GAME_VERIFIED
+TASKS:
+1. Standardize the active sprite tool at tools/Sprite Editor.
+2. Update active references, labels, titles, links, and path usage to Sprite Editor.
+3. Introduce a single source of truth registry for active tools.
+4. Ensure the active tools list/navigation renders from that registry rather than duplicated hardcoded names.
+5. Mark only these as active first-class tools:
+   - Vector Map Editor
+   - Vector Asset Studio
+   - Tile Map Editor
+   - Parallax Editor
+   - Sprite Editor
+6. Ensure legacy-preserved tools do not appear in the active tools list.
+7. Add a lightweight validation check/script that detects:
+   - missing active tool folders
+   - stale sprite-editor rename references
+   - legacy tools appearing in active navigation
+   - disallowed active naming suffixes like V2/V3/new/final/copy
+8. Validate all active tool loading paths after the rename.
 
-Step 3:
-Verify and apply deterministic vector geometry runtime acceptance.
-- Confirm repeated runs are deterministic
-- Confirm geometry participation is visible in debug/profiler outputs
-- Package as APPLY_PR_VECTOR_GEOMETRY_RUNTIME_VERIFIED
+OUTPUTS:
+- Repo changes only for this PR scope
+- docs/pr/BUILD_PR_TOOLS_REGISTRY_AND_SPRITE_RENAME.md retained
+- docs/dev/commit_comment.txt updated
+- package final delta zip to:
+  <project folder>/tmp/BUILD_PR_TOOLS_REGISTRY_AND_SPRITE_RENAME_delta.zip
 
-Step 4:
-Treat docs/pr/PLAN_PR_PUBLIC_PLATFORM_SHOWCASE.md as the governing contract for the final public-facing showcase.
-- Build the showcase plan and supporting release-ready assets
-- Keep claims aligned to verified technical evidence
-- Package as BUILD_PR_PUBLIC_PLATFORM_SHOWCASE
+COMMIT COMMENT:
+Unify active tools under a first-class tools registry, standardize Sprite Editor at tools/Sprite Editor, generate active tool navigation from a single source of truth, and add validation to prevent naming/path drift.
 
-Global requirements:
-- Preserve accepted platform boundaries
-- Do not modify engine core APIs
-- Keep docs/dev authoritative
-- Package outputs to repo tmp using established conventions
-
-Package:
-HTML-JavaScript-Gaming/tmp/FINAL_RELEASE_PATH_AND_VERIFICATION_SUITE_delta.zip
+NEXT COMMAND:
+MODEL: GPT-5.4
+REASONING: high
+COMMAND: Create BUILD_PR_VECTOR_PLATFORM_SURFACE_POLISH to standardize tool landing UX, descriptions, icons/labels, and showcase ordering across the active tools registry without changing engine behavior.

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-build(release): verify final vector-native release path and package public platform showcase
+Unify active tools under a first-class tools registry, standardize Sprite Editor at tools/Sprite Editor, generate active tool navigation from a single source of truth, and add validation to prevent naming/path drift.

docs/dev/reports/asset_remediation_system_validation.txt

@@ -4,8 +4,8 @@ Files:
 - tools/shared/projectAssetRemediation.js
 - tests/tools/AssetRemediationSystem.test.mjs
 - tests/run-tests.mjs
-- tools/Sprite Editor V3/index.html
-- tools/Sprite Editor V3/modules/spriteEditorApp.js
+- tools/Sprite Editor/index.html
+- tools/Sprite Editor/modules/spriteEditorApp.js
 - tools/Tilemap Studio/index.html
 - tools/Tilemap Studio/main.js
 - tools/Parallax Scene Studio/index.html
@@ -14,7 +14,7 @@ Files:
 Checks passed:
 - node --check tools/shared/projectAssetRemediation.js
 - node --check tests/tools/AssetRemediationSystem.test.mjs
-- node --check tools/Sprite Editor V3/modules/spriteEditorApp.js
+- node --check tools/Sprite Editor/modules/spriteEditorApp.js
 - node --check tools/Tilemap Studio/main.js
 - node --check tools/Parallax Scene Studio/main.js
 - targeted AssetRemediationSystem test via node inline runner

docs/dev/reports/asset_validation_engine_validation.txt

@@ -3,13 +3,13 @@ BUILD_PR_ASSET_VALIDATION_ENGINE validation
 Files:
 - tools/shared/projectAssetValidation.js
 - tests/tools/AssetValidationEngine.test.mjs
-- tools/Sprite Editor V3/modules/spriteEditorApp.js
+- tools/Sprite Editor/modules/spriteEditorApp.js
 - tools/Tilemap Studio/main.js
 - tools/Parallax Scene Studio/main.js
 
 Checks passed:
 - node --check tools/shared/projectAssetValidation.js
-- node --check tools/Sprite Editor V3/modules/spriteEditorApp.js
+- node --check tools/Sprite Editor/modules/spriteEditorApp.js
 - node --check tools/Tilemap Studio/main.js
 - node --check tools/Parallax Scene Studio/main.js
 - node --check tests/tools/AssetValidationEngine.test.mjs

docs/dev/reports/change_summary.txt

@@ -1,35 +1,26 @@
-Change Summary
-==============
+BUILD_PR_TOOLS_REGISTRY_AND_SPRITE_RENAME
+-----------------------------------------
 
-Build outcome
--------------
-- implemented the initial shared vector geometry runtime aligned to `docs/specs/vector_asset_contract.md`
-- kept the runtime split across contract parsing, pure geometry math, render preparation, and the public runtime facade
-- integrated vector runtime preparation into packaged runtime loading without changing engine core APIs
+Implemented changes
+- standardized the canonical active sprite tool folder at `tools/Sprite Editor/`
+- updated the sprite tool title and heading to the canonical `Sprite Editor` name
+- introduced `tools/toolRegistry.js` as the single source of truth for active tool metadata
+- introduced `tools/renderToolsIndex.js` so the tools hub renders the active tools list from the registry
+- updated `tools/index.html` to use generated active tool cards instead of hardcoded tool names
+- added `scripts/validate-active-tools-surface.mjs` to detect folder/path/name drift
 
-Runtime modules
----------------
-- added `tools/shared/vector/vectorGeometryMath.js`
-- added `tools/shared/vector/vectorAssetContract.js`
-- added `tools/shared/vector/vectorRenderPrep.js`
-- added `tools/shared/vectorGeometryRuntime.js`
+Approved visible active tools
+- `Vector Map Editor`
+- `Vector Asset Studio`
+- `Tile Map Editor`
+- `Parallax Editor`
+- `Sprite Editor`
 
-Compatibility and integration updates
--------------------------------------
-- updated `tools/shared/vector/vectorAssetBridge.js` to emit contract-aligned vector assets while preserving legacy bridge fields
-- updated `tools/shared/projectAssetValidation.js` to validate vectors against the contract through stable finding families
-- updated `tools/shared/runtimeAssetLoader.js` to promote packaged vector assets into `runtimeKind: "vector-geometry"` output
-- updated vector-native template, sample-game, and Asteroids demo runtime fixtures to provide real vector asset documents to the runtime
+Legacy handling
+- `tools/SpriteEditor_old_keep/` remains preserved on disk
+- legacy tools are excluded from the active tools list by registry metadata and validation
 
-Validation highlights
----------------------
-- syntax checks passed for all new and touched shared runtime files
-- targeted vector/runtime tests passed
-- Asteroids platform demo runtime integration passed after vector fixture repair
-- full node test runner passed with `110/110` explicit `run()` tests passing
-
-Next-slice recommendations
---------------------------
-- migrate checked-in vector asset files from legacy bridge shape to canonical contract documents
-- extend supported path commands only when a consumer requires them
-- add geometry-runtime-specific debug and profiler surfacing when dynamic vector transforms become active
+Validation
+- active tool entry paths were checked after the rename
+- stale sprite-editor rename references are checked in canonical hub/build-report surfaces
+- disallowed active naming suffixes are checked by validation script

docs/dev/reports/debug_visualization_layer_validation.txt

@@ -4,8 +4,8 @@ Files:
 - tools/shared/debugVisualizationLayer.js
 - tests/tools/DebugVisualizationLayer.test.mjs
 - tests/run-tests.mjs
-- tools/Sprite Editor V3/index.html
-- tools/Sprite Editor V3/modules/spriteEditorApp.js
+- tools/Sprite Editor/index.html
+- tools/Sprite Editor/modules/spriteEditorApp.js
 - tools/Tilemap Studio/index.html
 - tools/Tilemap Studio/main.js
 - tools/Parallax Scene Studio/index.html
@@ -14,7 +14,7 @@ Files:
 Checks passed:
 - node --check tools/shared/debugVisualizationLayer.js
 - node --check tests/tools/DebugVisualizationLayer.test.mjs
-- node --check tools/Sprite Editor V3/modules/spriteEditorApp.js
+- node --check tools/Sprite Editor/modules/spriteEditorApp.js
 - node --check tools/Tilemap Studio/main.js
 - node --check tools/Parallax Scene Studio/main.js
 - node ./scripts/run-node-tests.mjs

docs/dev/reports/editor_experience_layer_validation.txt

@@ -4,8 +4,8 @@ Files:
 - tools/shared/editorExperienceLayer.js
 - tests/tools/EditorExperienceLayer.test.mjs
 - tests/run-tests.mjs
-- tools/Sprite Editor V3/index.html
-- tools/Sprite Editor V3/modules/spriteEditorApp.js
+- tools/Sprite Editor/index.html
+- tools/Sprite Editor/modules/spriteEditorApp.js
 - tools/Tilemap Studio/index.html
 - tools/Tilemap Studio/main.js
 - tools/Parallax Scene Studio/index.html
@@ -14,7 +14,7 @@ Files:
 Checks passed:
 - node --check tools/shared/editorExperienceLayer.js
 - node --check tests/tools/EditorExperienceLayer.test.mjs
-- node --check tools/Sprite Editor V3/modules/spriteEditorApp.js
+- node --check tools/Sprite Editor/modules/spriteEditorApp.js
 - node --check tools/Tilemap Studio/main.js
 - node --check tools/Parallax Scene Studio/main.js
 - node ./scripts/run-node-tests.mjs

docs/dev/reports/file_tree.txt

@@ -1,25 +1,18 @@
+tools/
+  index.html
+  renderToolsIndex.js
+  toolRegistry.js
+  Sprite Editor/
+    index.html
+    README.md
+scripts/
+  validate-active-tools-surface.mjs
 docs/
   pr/
-    BUILD_PR_VECTOR_GEOMETRY_RUNTIME.md
-  specs/
-    vector_asset_contract.md
-tools/
-  shared/
-    asteroidsPlatformDemo.js
-    projectAssetValidation.js
-    runtimeAssetLoader.js
-    vectorGeometryRuntime.js
-    vectorNativeTemplate.js
-    vectorTemplateSampleGame.js
-    vector/
-      vectorAssetBridge.js
-      vectorAssetContract.js
-      vectorGeometryMath.js
-      vectorRenderPrep.js
-tests/
-  run-tests.mjs
-  tools/
-    VectorAssetSystem.test.mjs
-    VectorGeometryRuntime.test.mjs
-    VectorNativeTemplate.test.mjs
-    VectorTemplateSampleGame.test.mjs
+    BUILD_PR_TOOLS_REGISTRY_AND_SPRITE_RENAME.md
+  dev/
+    commit_comment.txt
+    reports/
+      change_summary.txt
+      file_tree.txt
+      validation_checklist.txt

docs/dev/reports/project_packaging_system_validation.txt

@@ -4,8 +4,8 @@ Files:
 - tools/shared/projectPackaging.js
 - tests/tools/ProjectPackagingSystem.test.mjs
 - tests/run-tests.mjs
-- tools/Sprite Editor V3/index.html
-- tools/Sprite Editor V3/modules/spriteEditorApp.js
+- tools/Sprite Editor/index.html
+- tools/Sprite Editor/modules/spriteEditorApp.js
 - tools/Tilemap Studio/index.html
 - tools/Tilemap Studio/main.js
 - tools/Parallax Scene Studio/index.html
@@ -14,7 +14,7 @@ Files:
 Checks passed:
 - node --check tools/shared/projectPackaging.js
 - node --check tests/tools/ProjectPackagingSystem.test.mjs
-- node --check tools/Sprite Editor V3/modules/spriteEditorApp.js
+- node --check tools/Sprite Editor/modules/spriteEditorApp.js
 - node --check tools/Tilemap Studio/main.js
 - node --check tools/Parallax Scene Studio/main.js
 - targeted ProjectPackagingSystem test via node inline runner

docs/dev/reports/validation_checklist.txt

@@ -1,51 +1,13 @@
-Validation Checklist
-====================
-
-Vector contract and parsing
----------------------------
-[x] Contract-aligned vector assets parse successfully
-[x] Legacy SVG-bridge vector assets normalize successfully for runtime use
-[x] Unsupported or malformed vector geometry fails inspection
-
-Geometry math
--------------
-[x] translate support verified
-[x] rotate support verified
-[x] scale support verified
-[x] transformed point output verified
-[x] bounds and center calculations verified
-[x] supported SVG path parsing verified for runtime-supported commands
-
-Runtime preparation
--------------------
-[x] supported primitives produce runtime-ready renderables
-[x] collision-ready geometry output is produced where appropriate
-[x] packaged runtime loader upgrades vector assets into `runtimeKind: "vector-geometry"` output
-[x] no broken imports or paths were introduced in touched shared modules
-
-Integration path
-----------------
-[x] vector asset system still validates, packages, and loads correctly
-[x] vector-native template runtime path is clear and ready
-[x] vector template sample game runtime path is clear and ready
-[x] Asteroids platform demo runtime path remains ready after vector runtime integration
-
-Verification commands
----------------------
-[x] `node --check tools/shared/vector/vectorGeometryMath.js`
-[x] `node --check tools/shared/vector/vectorAssetContract.js`
-[x] `node --check tools/shared/vector/vectorRenderPrep.js`
-[x] `node --check tools/shared/vectorGeometryRuntime.js`
-[x] `node --check tools/shared/vector/vectorAssetBridge.js`
-[x] `node --check tools/shared/projectAssetValidation.js`
-[x] `node --check tools/shared/runtimeAssetLoader.js`
-[x] `node --check tools/shared/vectorNativeTemplate.js`
-[x] `node --check tools/shared/vectorTemplateSampleGame.js`
-[x] `node --check tools/shared/asteroidsPlatformDemo.js`
-[x] `node --input-type=module -e "import('./tests/tools/VectorGeometryRuntime.test.mjs').then((m) => m.run())"`
-[x] `node --input-type=module -e "import('./tests/tools/VectorAssetSystem.test.mjs').then((m) => m.run())"`
-[x] `node --input-type=module -e "import('./tests/tools/VectorNativeTemplate.test.mjs').then((m) => m.run())"`
-[x] `node --input-type=module -e "import('./tests/tools/VectorTemplateSampleGame.test.mjs').then((m) => m.run())"`
-[x] `node --input-type=module -e "import('./tests/games/AsteroidsPlatformDemo.test.mjs').then((m) => m.run())"`
-[x] `node ./scripts/run-node-tests.mjs`
-[x] Result: `110/110` explicit `run()` tests passed
+[x] canonical sprite tool folder is tools/Sprite Editor
+[x] no canonical stale sprite-editor rename references remain in the validated active tool surface
+[x] active tools registry exists and is the single source of truth
+[x] generated active tools list comes from registry
+[x] active tools list contains only approved first-class tools
+[x] Vector Map Editor listed
+[x] Vector Asset Studio listed
+[x] Tile Map Editor listed
+[x] Parallax Editor listed
+[x] Sprite Editor listed
+[x] legacy-preserved tools excluded from active tools list
+[x] validation detects stale references and naming drift
+[x] active tool paths load correctly after rename