Correct the V2 tool implementation direction so static page structure lives in tools/<tool>-v2/index.html. Enforce HTML-first V2 tool shells across active re-engineer lane - PR 11.193. Re-engineer V2 tools to HTML-first shells with behavior-only JS and shared header mount - PR 11.193C

Author: DavidQ

docs/dev/codex_commands.md

@@ -1,24 +1,68 @@
-# Codex Commands — PR_11_192
+# Codex Commands — PR_11_193C
 
 ## Model
-GPT-5.4
-
-## Reasoning
-high
+Use GPT-5.3-codex or available Codex implementation model.
+Reasoning: high.
 
 ## Command
-codex exec --model gpt-5.4 --reasoning high "Apply PR_11_192 exactly as documented in docs/pr/PR_11_192_v2_html_shell_correction.md. Correct V2 tools so index.html owns the static shell, CSS links, shared header mount, page layout, menuTool/menuWorkspace containers, and module script. Keep index.js behavior-only: title/tool id setup, session read, validation, DOM population, rendering, and empty/error states. Do not inject CSS or replace document.body.innerHTML from JS. Do not touch schemas, samples, games, Workspace Manager v1, platformShell, tools/shared, or legacy tools. Return a ZIP at tmp/PR_11_192_20260501_01.zip."
+From repo root:
+
+```powershell
+cd C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming
+```
+
+Run Codex with this exact task:
+
+```text
+Implement PR_11_193C exactly as described in docs/pr/PR_11_193C_V2_HTML_FIRST_BATCH.md.
+
+This is a V2 re-engineer batch, not legacy copy/paste.
+
+You must write the implementation code. ChatGPT did not provide implementation files.
+
+Scope only:
+- Palette Manager V2
+- SVG Asset Studio V2
+- Vector Map Editor V2
+- Tilemap Studio V2
+- Asset Browser V2
+
+Do not change schemas, samples, games, Workspace Manager v1, platformShell, assetUsageIntegration, tools/shared/*, or start_of_day folders.
+
+Make each V2 tool HTML-first:
+- tools/<tool>-v2/index.html owns CSS links, shared header mount, static layout, static state containers, menus, and module script tags.
+- index.html must include <div id="shared-theme-header"></div>.
+- index.js must be behavior-only: document title/dataset, session read, validation, event binding, dynamic render into existing DOM nodes.
+- Remove JS-driven page construction such as document.body.innerHTML, document.head.insertAdjacentHTML style blocks, dynamic header script injection, and static layout template strings.
+- Keep one class per tool file.
+- No helper classes, no alias variables, no pass-through variables, no abstraction layers.
+- No fallback/default data.
+- Session-backed data only.
+- Tool names must end with V2.
+
+Validation required:
+- Run syntax checks for each changed JS file.
+- Run targeted tool launch/HTML validation available in the repo for these tools.
+- Do not run full samples smoke unless you changed shared sample loader/framework code.
+- Write validation evidence to docs/dev/reports/PR_11_193C_validation.md.
+
+Before finishing:
+- Review diff for unrelated changes and remove them.
+- Confirm no implementation touched banned paths.
+- Confirm no schema/sample/game changes.
+- Create final Codex ZIP at tmp/PR_11_193C_20260501_Codex.zip preserving repo structure.
+```
 
-## Targeted validation
-node --check tools/palette-manager-v2/index.js
-node --check tools/svg-asset-studio-v2/index.js
-node --check tools/vector-map-editor-v2/index.js
+## Expected Evidence File
+Codex must create:
 
-Skip any validation command only if the file does not exist and document why in docs/dev/reports/PR_11_192_validation.md.
+```text
+docs/dev/reports/PR_11_193C_validation.md
+```
 
-## Manual validation
-Open each corrected V2 tool directly and verify:
-- shared header renders from `<div id="shared-theme-header"></div>`
-- static page shell is visible before session data
-- empty session shows explicit empty state
-- valid session renders without legacy coupling
+It must include:
+- files changed
+- tests run
+- full smoke skipped/run decision and reason
+- banned path check result
+- V2 header compliance result

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Move V2 static shell into index.html and keep tool JavaScript behavior-only - PR 11.192
+Re-engineer V2 tools to HTML-first shells with behavior-only JS and shared header mount - PR 11.193C

docs/dev/reports/PR_11_192_validation.md

@@ -1,7 +1,7 @@
 # PR_11_192 Validation
 
 ## Purpose
-Correct V2 tools so `index.html` owns the static shell and `index.js` remains behavior-only.
+Correct V2 tools so `index.html` owns the static shell and `index.js` remains behavior-only. This `_04` pass revalidated the corrected V2 lane and packages the requested artifact.
 
 ## Changed Files
 - `tools/palette-manager-v2/index.html`
@@ -12,15 +12,36 @@ Correct V2 tools so `index.html` owns the static shell and `index.js` remains be
 - `tools/vector-map-editor-v2/index.js`
 - `docs/dev/reports/PR_11_192_validation.md`
 
-## Removed Superseded V2 Paths
-- `tools/Palette Manager v2/main.js`
-- `tools/SVG Asset Studio v2/main.js`
+## HTML Shell Evidence
+Each corrected V2 `index.html` owns:
+- static CSS links
+- `<div id="shared-theme-header"></div>`
+- static page shell
+- accordion layout
+- `menuTool` container
+- `menuWorkspace` container
+- shared header module script
+- tool module script: `./index.js`
+
+Confirmed for:
+- `tools/palette-manager-v2/index.html`
+- `tools/svg-asset-studio-v2/index.html`
+- `tools/vector-map-editor-v2/index.html`
+
+## Behavior-Only JS Evidence
+Each corrected V2 `index.js` contains one class and keeps behavior in JS only:
+- title/tool id setup
+- session read
+- validation
+- DOM population
+- rendering
+- empty/error states
 
-## HTML Shell Correction
-- Each V2 tool now has a static `index.html`.
-- Each `index.html` owns CSS links, `<div id="shared-theme-header"></div>`, page shell, accordion, `menuTool`, `menuWorkspace`, stable DOM nodes, shared header module script, and tool module script.
-- Each `index.js` is behavior-only: title/tool id setup, session read, validation, DOM population, rendering, and empty/error states.
-- JS no longer injects CSS, appends the shared header script, or replaces `document.body.innerHTML`.
+Confirmed no `index.js` contains:
+- `document.body.innerHTML`
+- `insertAdjacentHTML`
+- dynamic shared header script creation/append
+- injected `<style>` shell CSS
 
 ## Targeted Validation
 All requested validation commands were run. No validation command was skipped.
@@ -51,7 +72,7 @@ A targeted file-content guard scan checked the corrected V2 tool folders for for
 - `fallback`
 - `default data`
 - `demo data`
-- static imports/exports
+- `../shared`
 
 Result: passed. No matches.
 
@@ -78,5 +99,5 @@ Reason: PR 11.192 is limited to V2 static shell separation and targeted direct t
 ## ZIP Artifact
 
 ```text
-tmp/PR_11_192_20260501_01.zip
+tmp/PR_11_192_20260501_04.zip
 ```

docs/dev/reports/PR_11_193C_validation.md

@@ -0,0 +1,129 @@
+# PR_11_193C Validation
+
+## Purpose
+Implement the V2 HTML-first batch for Palette Manager V2, SVG Asset Studio V2, Vector Map Editor V2, Tilemap Studio V2, and Asset Browser V2.
+
+## Files Changed
+- `tools/tilemap-studio-v2/index.html`
+- `tools/tilemap-studio-v2/index.js`
+- `tools/asset-browser-v2/index.html`
+- `tools/asset-browser-v2/index.js`
+- `docs/dev/reports/PR_11_193C_validation.md`
+
+Existing active V2 tools verified without additional implementation edits in this pass:
+- `tools/palette-manager-v2/index.html`
+- `tools/palette-manager-v2/index.js`
+- `tools/svg-asset-studio-v2/index.html`
+- `tools/svg-asset-studio-v2/index.js`
+- `tools/vector-map-editor-v2/index.html`
+- `tools/vector-map-editor-v2/index.js`
+
+Existing PR source doc was present in the worktree and included in the ZIP:
+- `docs/pr/PR_11_193C_V2_HTML_FIRST_BATCH.md`
+
+## Implementation Summary
+- Added `Tilemap Studio V2` as `tools/tilemap-studio-v2/`.
+- Added `Asset Browser V2` as `tools/asset-browser-v2/`.
+- Both new tools are HTML-first: `index.html` owns static shell, CSS links, shared header mount, layout, menus, state containers, and module scripts.
+- Both new tools keep `index.js` behavior-only: document title/tool id setup, session read, contract validation, event binding, dynamic rendering into existing DOM nodes, and empty/error states.
+- No legacy implementation files were copied into V2. Legacy files were used only to infer high-level contract names and user-facing data shape.
+
+## V2 Header Compliance Result
+Passed for all five scoped V2 tools:
+- `tools/palette-manager-v2/index.html`
+- `tools/svg-asset-studio-v2/index.html`
+- `tools/vector-map-editor-v2/index.html`
+- `tools/tilemap-studio-v2/index.html`
+- `tools/asset-browser-v2/index.html`
+
+Each contains:
+- `<div id="shared-theme-header"></div>`
+- `../../src/engine/theme/main.css`
+- `../../src/engine/ui/hubCommon.css`
+- `<main class="page-shell">`
+- `data-menu-tool` / `menuTool`
+- `data-menu-workspace` / `menuWorkspace`
+- `<script type="module" src="../../src/engine/theme/mount-shared-header.js"></script>`
+- `<script type="module" src="./index.js"></script>`
+
+## Behavior-Only JS Result
+Passed for all five scoped V2 tools. Each `index.js` contains one class and avoids static shell construction.
+
+Forbidden markers checked and not found in scoped V2 `index.js` files:
+- `document.body.innerHTML`
+- `document.head.insertAdjacentHTML`
+- dynamic script creation/append for the shared header
+- inline `<style>` shell injection
+- `platformShell`
+- `assetUsageIntegration`
+- `tools/shared`
+- `../shared`
+- Workspace Manager coupling
+- handoff logic
+- fallback/default/demo data markers
+
+## Required Runtime Logs
+Present in scoped V2 tools:
+- `[PALETTE_V2_ENTRY]`
+- `[SVG_V2_ENTRY]`
+- `[VECTOR_MAP_V2_ENTRY]`
+- `[TILEMAP_V2_ENTRY]`
+- `[ASSET_BROWSER_V2_ENTRY]`
+- `[SESSION_CONTEXT_READ]`
+- `[PALETTE_V2_CONTRACT_LOADED]`
+- `[SVG_V2_CONTRACT_LOADED]`
+- `[VECTOR_MAP_V2_CONTRACT_LOADED]`
+- `[TILEMAP_V2_CONTRACT_LOADED]`
+- `[ASSET_BROWSER_V2_CONTRACT_LOADED]`
+
+## Tests Run
+Syntax checks:
+
+```powershell
+node --check tools/palette-manager-v2/index.js
+node --check tools/svg-asset-studio-v2/index.js
+node --check tools/vector-map-editor-v2/index.js
+node --check tools/tilemap-studio-v2/index.js
+node --check tools/asset-browser-v2/index.js
+```
+
+Results:
+- `node --check tools/palette-manager-v2/index.js` passed.
+- `node --check tools/svg-asset-studio-v2/index.js` passed.
+- `node --check tools/vector-map-editor-v2/index.js` passed.
+- `node --check tools/tilemap-studio-v2/index.js` passed.
+- `node --check tools/asset-browser-v2/index.js` passed.
+
+Targeted direct-entry/HTML validation:
+- Verified all five scoped `index.html` files have the required static shell/header/menu/script markers.
+- Verified all five scoped `index.js` files have one class and behavior-only markers.
+- Verified all five scoped `index.js` files omit forbidden shell-building and legacy coupling markers.
+
+Result: passed.
+
+## Banned Path Check Result
+Scoped banned path status check found no changes under:
+- schemas
+- samples
+- games
+- `start_of_day/**`
+- Workspace Manager v1
+- `tools/shared/**`
+- `platformShell`
+- `assetUsageIntegration`
+
+No schemas, samples, or games were changed.
+
+## Diff Review
+Scoped diff review found this PR's implementation changes limited to allowed V2 folders plus this report. Existing unrelated dirty worktree files were not modified for this PR and are not included as implementation changes.
+
+## Full Samples Smoke Decision
+Full samples smoke test skipped.
+
+Reason: PR 11.193C changes isolated V2 tool entry shells and behavior-only JS only; it does not modify shared sample loader/framework code.
+
+## ZIP Artifact
+
+```text
+tmp/PR_11_193C_20260501_Codex.zip
+```

docs/dev/reports/PR_11_193_V2_HTML_FIRST_BATCH_validation.md

@@ -0,0 +1,103 @@
+# PR_11_193 V2 HTML First Batch Validation
+
+## Purpose
+Batch-correct and verify the active V2 tool lane so `index.html` owns static shell/CSS/header mount/page layout/menu containers/module scripts, while `index.js` remains behavior-only.
+
+## Active V2 Folders Found
+- `tools/palette-manager-v2/`
+- `tools/svg-asset-studio-v2/`
+- `tools/vector-map-editor-v2/`
+
+## Listed V2 Folders Not Present
+- `tools/tilemap-studio-v2/` - NOT PRESENT - SKIPPED
+- `tools/asset-browser-v2/` - NOT PRESENT - SKIPPED
+
+No unrelated scaffolding was created for absent V2 tools.
+
+## Files Verified For Batch Correction
+- `tools/palette-manager-v2/index.html`
+- `tools/palette-manager-v2/index.js`
+- `tools/svg-asset-studio-v2/index.html`
+- `tools/svg-asset-studio-v2/index.js`
+- `tools/vector-map-editor-v2/index.html`
+- `tools/vector-map-editor-v2/index.js`
+
+The active V2 files were already in the corrected HTML-first shape at execution time; this pass verified and packaged that active lane.
+
+## HTML Shell Evidence
+Each active V2 `index.html` contains:
+- document title ending with `V2`
+- `../../src/engine/theme/main.css`
+- `../../src/engine/ui/hubCommon.css`
+- `<body data-tool-id="<tool-id>-v2">`
+- `<div id="shared-theme-header"></div>`
+- static `<main class="page-shell">` shell
+- static `menuTool` container
+- static `menuWorkspace` container
+- `<script type="module" src="../../src/engine/theme/mount-shared-header.js"></script>`
+- `<script type="module" src="./index.js"></script>`
+
+## Behavior-Only JS Evidence
+Each active V2 `index.js` contains one class and keeps JavaScript limited to behavior:
+- title/tool id setup
+- session read
+- session contract validation
+- DOM population into existing nodes
+- rendering into existing containers
+- explicit empty/error states
+
+Guard scan confirmed no active V2 `index.js` contains:
+- `document.body.innerHTML`
+- `document.head.insertAdjacentHTML`
+- inline `<style>` injection
+- dynamic shared header script creation/append
+- legacy v1/shared dependencies
+- fallback/default/demo data markers
+
+## Targeted Validation
+Commands run:
+
+```powershell
+node --check tools/palette-manager-v2/index.js
+node --check tools/svg-asset-studio-v2/index.js
+node --check tools/vector-map-editor-v2/index.js
+```
+
+Results:
+- `node --check tools/palette-manager-v2/index.js` passed.
+- `node --check tools/svg-asset-studio-v2/index.js` passed.
+- `node --check tools/vector-map-editor-v2/index.js` passed.
+
+Skipped commands:
+- `node --check tools/tilemap-studio-v2/index.js` - NOT PRESENT - SKIPPED
+- `node --check tools/asset-browser-v2/index.js` - NOT PRESENT - SKIPPED
+
+## Additional Guard Checks
+- HTML/JS compliance scan passed for the three active V2 folders.
+- Forbidden-pattern scan passed for active V2 `index.js` files.
+
+## Scope Guard
+No changes were made to:
+- schemas
+- samples
+- games
+- Workspace Manager v1
+- legacy tools without `-v2`
+- `platformShell`
+- `tools/shared/*`
+- root `/index.html`
+- `/tools/index.html`
+
+## Roadmap
+`docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md` was not updated. No exact execution-backed status marker change was required by this batch verification.
+
+## Full Samples Smoke Test
+Skipped as required.
+
+Reason: this PR is limited to isolated V2 tool entry shells and behavior-only JS separation, not shared sample loader/framework logic.
+
+## ZIP Artifact
+
+```text
+tmp/PR_11_193_V2_HTML_FIRST_BATCH.zip
+```