Promote preview.html into Preview Generator V2 tool preserving behavior and aligning to Palette Manager layout - PR_26126_009-preview-generator-v2-from-preview-htm.l

Author: DavidQ

docs/dev/codex_commands.md

@@ -1,24 +1,25 @@
-# Codex Commands - PR_26126_008-preview-generator-v2-first-pass
+# Codex Commands - PR_26126_010-preview-generator-v2-reskin-correction
 
 ```bash
-codex run "Create PR_26126_008-preview-generator-v2-first-pass. Build Preview Generator V2 in one pass using Palette Manager as the base layout/code pattern. Use the exact Palette Manager platform header and NAV style. NAV buttons must be Generate Preview, Apply to Game, Export Image, with Generate Preview in NAV. Do not show JSON UI. Preview Generator V2 owns no schema. Implement native radio target selection with name=\"previewTarget\" and values including game. Build 3-column UI: left target/source/status, center main preview plus Last Generated Preview at bottom, right render/output controls. For Game target, generated preview should update asset-browser-compatible destination data for that game, validated against the destination schema only. No fallback data, no hidden defaults, no partial render on invalid destination data. Include targeted Playwright coverage for skeleton render, radio selection, generate/apply/export button states, and no visible JSON controls. Do not modify samples. Produce review artifacts."
+codex run "Create PR_26126_010-preview-generator-v2-reskin-correction. Correct Preview Generator V2 as a reskin of existing preview.html, not a rewrite. Review and preserve all existing preview.html functionality, labels, inputs, ids where possible, and behavior. Regroup the existing controls into the Palette Manager-style layout. Left column must contain: Repo Destination (rename Game Destination; it opens a folder, not a file), Target Source, and Render Controls. Center column must replace Main Preview with Paths or IDs using an input box. Right column must place Status under Output Summary. Header/NAV must remove Apply to Game and Export Image; keep Generate Preview only in the Palette Manager-style NAV. Do not add JSON UI. Do not create a Preview Generator V2 schema. Do not modify samples. Update Playwright/tests only for this corrected reskin behavior. Produce review artifacts."
 ```
 
 ## Validation Commands
 
 ```bash
-node --check tools/preview/main.js
-node --check tools/preview/previewGeneratorShell.js
+node --check tools/preview-generator-v2/main.js
+node --check tools/preview-generator-v2/previewGeneratorShell.js
 node --check tests/tools/PreviewGeneratorV2FirstPass.test.mjs
 node tests/tools/PreviewGeneratorV2FirstPass.test.mjs
-git diff --check -- tools/preview/index.html tools/preview/main.js tools/preview/previewGeneratorShell.js tools/preview/previewGeneratorV2.css tests/tools/PreviewGeneratorV2FirstPass.test.mjs docs/dev/codex_commands.md docs/dev/commit_comment.txt
+git diff --check -- tools/preview-generator-v2/index.html tools/preview-generator-v2/main.js tools/preview-generator-v2/previewGeneratorShell.js tools/preview-generator-v2/previewGeneratorV2.css tools/toolRegistry.js tests/tools/PreviewGeneratorV2FirstPass.test.mjs docs/dev/codex_commands.md docs/dev/commit_comment.txt
+npm run test:workspace-v2
 npm run codex:review-artifacts
 ```
 
-## Playwright
+## Notes
 
-Targeted Playwright coverage is in `tests/tools/PreviewGeneratorV2FirstPass.test.mjs`.
+`tools/preview/preview_svg_generator.html` was used as the existing preview generator base because there is no literal `preview.html` file in the repository.
 
-## Full Samples
+`npm run test:workspace-v2` was attempted, but the script is not defined in this checkout.
 
-Full samples smoke test was skipped because this PR is scoped to the Preview Generator V2 tool and targeted Playwright coverage.
+Full samples smoke test was skipped because this PR is scoped to Preview Generator V2.

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Build Preview Generator V2 first-pass UI and coverage - PR_26126_008-preview-generator-v2-first-pass
+Correct Preview Generator V2 reskin layout - PR_26126_010-preview-generator-v2-reskin-correction

docs/dev/reports/codex_changed_files.txt

@@ -1,18 +1,24 @@
 # git status --short
 M docs/dev/codex_commands.md
  M docs/dev/commit_comment.txt
- A tests/tools/PreviewGeneratorV2FirstPass.test.mjs
- A tools/preview/index.html
- A tools/preview/main.js
- A tools/preview/previewGeneratorShell.js
- A tools/preview/previewGeneratorV2.css
+ M docs/dev/reports/codex_changed_files.txt
+ M docs/dev/reports/codex_review.diff
+ M tests/tools/PreviewGeneratorV2FirstPass.test.mjs
+ A tools/preview-generator-v2/index.html
+ A tools/preview-generator-v2/main.js
+ A tools/preview-generator-v2/previewGeneratorShell.js
+ A tools/preview-generator-v2/previewGeneratorV2.css
+ M tools/toolRegistry.js
 
 # git diff --stat
-docs/dev/codex_commands.md                       |  15 +-
- docs/dev/commit_comment.txt                      |   2 +-
- tests/tools/PreviewGeneratorV2FirstPass.test.mjs | 127 ++++++
- tools/preview/index.html                         | 171 ++++++++
- tools/preview/main.js                            | 493 +++++++++++++++++++++
- tools/preview/previewGeneratorShell.js           | 186 ++++++++
- tools/preview/previewGeneratorV2.css             | 534 +++++++++++++++++++++++
- 7 files changed, 1521 insertions(+), 7 deletions(-)
+docs/dev/codex_commands.md                         |   19 +-
+ docs/dev/commit_comment.txt                        |    2 +-
+ docs/dev/reports/codex_changed_files.txt           |   32 +-
+ docs/dev/reports/codex_review.diff                 | 6646 ++++++++++++++++++--
+ tests/tools/PreviewGeneratorV2FirstPass.test.mjs   |  144 +-
+ tools/preview-generator-v2/index.html              |  188 +
+ tools/preview-generator-v2/main.js                 |  932 +++
+ .../preview-generator-v2/previewGeneratorShell.js  |  186 +
+ tools/preview-generator-v2/previewGeneratorV2.css  |  490 ++
+ tools/toolRegistry.js                              |   18 +
+ 10 files changed, 8041 insertions(+), 616 deletions(-)