Sort tools index groups and tool tiles while continuing root cleanup - PR_26154_014-tools-index-sort-cleanup

Author: ToolboxAid

docs_build/dev/codex_commands.md

@@ -1,60 +1,47 @@
 # Codex Commands
 
-Tasks:
+Task:
 
-- `PR_26154_011-theme-v1-final-teardown`
-- `PR_26154_012-root-asset-reference-cleanup`
-- `PR_26154_013-root-structure-inventory-closeout`
+- `PR_26154_014-tools-index-sort-cleanup`
 
 Commands run:
 
 - `Get-Content .codex/skills/repo-build/SKILL.md`
 - `Get-Content docs_build/dev/PROJECT_INSTRUCTIONS.md`
 - `git status --short --untracked-files=all`
-- Targeted `Get-ChildItem`, `Get-Content`, and `rg` inventory for:
-  - `src/engine/theme/`
-  - `assets/theme/v1/`
-  - active public/root pages
-  - active tool surfaces under `tools/`
-  - `games/`
-  - `docs/`
-  - stale path strings
-  - current root ownership folders
-- Updated `src/engine/theme/README.md` to document runtime ownership and the current direct helper import path.
-- Targeted stale-reference checks for:
-  - `GameFoundryStudio/`
-  - `src/engine/theme/index.js`
-  - `assets/theme/v2/assets/`
-  - `assets/theme/v2/images/games/`
-  - `favicon.ico`
-- Root structure inventory counts for:
-  - `assets/`
-  - `tools/`
-  - `old-tools/`
-  - `games/`
-  - `old_games/`
-  - `old_samples/`
-  - `docs/`
-  - `docs_build/`
-  - `schemas/`
-  - `src/`
-- Node targeted static validation for changed HTML, JS, CSS, JSON, and Markdown paths.
-- `git status --short -- start_of_day old_games old_samples`
+- `Get-Content tools/index.html`
+- `Get-Content tools/tools-page-accordions.js`
+- `rg -n "GameFoundryStudio/|src/engine/theme/|assets/theme/v2/assets/|assets/theme/v2/images/games/|favicon\\.ico" tools/index.html tools/tools-page-accordions.js`
+- Node scoped rewrite for `tools/tools-page-accordions.js` group and tile ordering.
+- `rg --files | rg -i "localization|localisation|localize|translate|translation"`
+- `Get-ChildItem assets/theme/v2/images/tools`
+- `rg -n "localization\\.png|localization-studio|Localization|localization" tools assets docs docs_build -g "!*old*"`
+- `rg -n "tools-page-accordions\\.js" -g "*.html" -g "*.js" -g "*.md" -g "*.json" .`
+- `node --check tools/tools-page-accordions.js`
+- Node targeted validator for:
+  - active Tools index group ordering
+  - tile ordering within each group
+  - active tool href/image file existence
+  - `tools/index.html` local ref file existence
+  - stale active path strings
+- Node static validation for changed HTML, JS, CSS, JSON, and Markdown files.
 - `git diff --check`
+- `git status --short -- start_of_day old_games old_samples`
 - `npm run codex:review-artifacts`
-- Python ZIP packaging for `tmp/PR_26154_011-013-root-structure-closeout_delta.zip`
+- ZIP packaging for `tmp/PR_26154_014-tools-index-sort-cleanup_delta.zip`
 
 Validation summary:
 
-- PASS no confirmed V1 legacy/static assets remain under `src/engine/theme/`.
-- PASS remaining `src/engine/theme/` files are runtime-owned or runtime documentation.
-- PASS `src/engine/theme/README.md` no longer references the removed `index.js` barrel.
-- PASS no active public/root page, active tool page, game page, public doc, asset, script, or source helper references remain to stale public asset paths.
-- PASS remaining `src/engine/theme/` references are intentional runtime/guard/test references or historical/generated records.
-- PASS root structure ownership inventory completed.
-- PASS `schemas/` absence confirmed; active schemas are under `tools/schemas/`.
-- PASS `start_of_day/`, `old_games/`, and `old_samples/` unchanged.
+- PASS `tools/tools-page-accordions.js` syntax validation.
+- PASS active Tools index group order is alphabetical.
+- PASS active Tools index tile order is alphabetical within every group.
+- PASS 19 active tool href/image references resolve.
+- PASS 4 `tools/index.html` local refs resolve.
+- PASS no targeted stale path strings remain in `tools/index.html` or `tools/tools-page-accordions.js`.
+- PASS static validation for changed HTML, JS, CSS, JSON, and Markdown files.
 - PASS `git diff --check`.
-- PASS `npm run codex:review-artifacts`.
+- PASS no `start_of_day/`, `old_games/`, or `old_samples/` changes.
+- PASS review artifact generation.
+- PASS repo-structured delta ZIP packaging.
 - SKIPPED `npm run test:workspace-v2`; active Workspace V2 launch/navigation behavior was not changed.
-- SKIPPED old_games, old_samples, and full samples smoke validation per request.
+- SKIPPED old games, old samples, and full samples smoke validation per request.

docs_build/dev/commit_comment.txt

@@ -1,12 +1,9 @@
-Theme V1 final teardown, root asset reference cleanup, and structure inventory closeout
+Tools index sort cleanup
 
-Stack:
-- PR_26154_011-theme-v1-final-teardown
-- PR_26154_012-root-asset-reference-cleanup
-- PR_26154_013-root-structure-inventory-closeout
+PR: PR_26154_014-tools-index-sort-cleanup
 
-- Confirm no further V1 static/theme assets can safely move from `src/engine/theme/` because the remaining files are runtime-owned.
-- Update `src/engine/theme/README.md` to remove a stale removed-barrel import and document current runtime ownership.
-- Audit active root/page/tool/game/doc references for stale path strings and report remaining intentional runtime, generated, or historical references.
-- Create root structure ownership inventory and cleanup candidate report without moving unrelated files.
-- Validate targeted references, changed static files, diff whitespace, review artifacts, and repo-structured delta ZIP packaging.
+- Sort active Tools index groups alphabetically by group name.
+- Sort tool tiles alphabetically by visible tool name within each existing group.
+- Preserve all existing group assignments, tool names, and launch hrefs.
+- Fix the active Localization tile image path to the existing `localization-studio.png` asset.
+- Validate active Tools index ordering, resolved local href/image paths, stale path strings, diff whitespace, review artifacts, and repo-structured delta ZIP packaging.

docs_build/dev/reports/tools_index_sort_cleanup_report.md

@@ -0,0 +1,64 @@
+# PR_26154_014 Tools Index Sort Cleanup Report
+
+Task: `PR_26154_014-tools-index-sort-cleanup`
+
+## Scope
+
+- Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- Kept `tools/index.html` layout and markup unchanged.
+- Updated the active Tools index data source: `tools/tools-page-accordions.js`.
+- Preserved existing group assignments and tool names.
+- Did not touch `old_games/`, `old_samples/`, or `start_of_day/`.
+- Did not run full samples smoke validation.
+
+## Ordering Changes
+
+`tools/index.html` loads `tools-page-accordions.js`, so the visible grouped order is controlled by `tools/tools-page-accordions.js`.
+
+Alphabetized group order:
+
+1. AI
+2. Assets
+3. Audio
+4. Colors
+5. Input
+6. Objects
+7. Worlds
+
+Alphabetized tool order by group:
+
+| Group | Tool Order |
+| --- | --- |
+| AI | AI Assistant |
+| Assets | Arcade, Cloud, Localization, Marketplace, Publish, Storage Inspector |
+| Audio | MIDI, Sound |
+| Colors | Palette Manager |
+| Input | Input |
+| Objects | Animation, Assets, Custom Extensions, Object Vector |
+| Worlds | Game Builder, Game Design, Particles, World Vector |
+
+## Targeted Path Cleanup
+
+| File | Previous Path | Updated Path | Reason |
+| --- | --- | --- | --- |
+| `tools/tools-page-accordions.js` | `../assets/theme/v2/images/tools/localization.png` | `../assets/theme/v2/images/tools/localization-studio.png` | Active Tools index image validation found the previous file missing; the existing matching Localization tile asset is `localization-studio.png`. |
+
+No tool hrefs were renamed or reassigned.
+
+## Validation
+
+- PASS: `node --check tools/tools-page-accordions.js`.
+- PASS: Targeted order validator confirmed group order is alphabetical.
+- PASS: Targeted order validator confirmed every group sorts tool tiles alphabetically by visible tool name.
+- PASS: Targeted link/path validator resolved 19 tool href/image references and 4 `tools/index.html` local refs.
+- PASS: Targeted stale reference check found no `GameFoundryStudio/`, `src/engine/theme/`, `assets/theme/v2/assets/`, `assets/theme/v2/images/games/`, or `favicon.ico` references in the active Tools index files.
+- PASS: Static validation covered changed HTML/JS/CSS/JSON/Markdown paths.
+- PASS: `git diff --check`.
+- PASS: `git status --short -- start_of_day old_games old_samples` returned no output.
+- SKIPPED: `npm run test:workspace-v2`; active Workspace V2 launch/navigation behavior was not changed.
+- SKIPPED: old games, old samples, and full samples smoke tests per request.
+
+## Notes
+
+- The active Tools index loads `tools/tools-page-accordions.js`.
+- A separate historical helper copy at `assets/theme/v2/js/tools-page-accordions.js` is not imported by `tools/index.html`; it was not changed in this scoped PR.