Fix Asset Manager V2 palette color IDs preview paths and asset detail controls - PR_26126_093-asset-manager-v2-palette-preview-path-fix

Author: DavidQ

docs/dev/reports/PR_26126_093_asset_manager_v2_manual_validation_notes.md

@@ -0,0 +1,26 @@
+# PR_26126_093 Asset Manager V2 Manual Validation Notes
+
+## Commands
+- `node --check tools/asset-manager-v2/js/AssetManagerV2App.js`
+- `node --check tools/asset-manager-v2/js/controls/AssetFormControl.js`
+- `node --check tools/asset-manager-v2/js/controls/AssetCatalogControl.js`
+- `node --check tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js`
+- `node --check tools/asset-manager-v2/js/services/WorkspaceBridge.js`
+- `node --check src/shared/assets/assetPreviewHelpers.js`
+- `node --check tests/playwright/PreviewGeneratorV2Baseline.spec.mjs`
+- `npx playwright test tests/playwright/PreviewGeneratorV2Baseline.spec.mjs --project=playwright --reporter=list -g "Asset Manager V2"`
+- `npm run test:workspace-v2`
+
+## Results
+- Focused Asset Manager V2 Playwright slice passed: 4 tests passed.
+- Required workspace-v2 Playwright gate passed: 11 tests passed.
+- `docs/dev/reports/playwright_v8_coverage_report.txt` was regenerated by the full run.
+
+## Manual Checks Covered By Tests
+- Disabled and grayed ID textbox.
+- Color ID generation from normalized swatch names.
+- Temporary `?palette=sample` UAT palette loading.
+- Color picker replacing the file picker and Status-only empty palette messages.
+- Audio, font, and image preview path resolution from Workspace V2 game assets.
+- Selected Asset Detail accordion and Stretch Override fieldset styling.
+- Sample JSON files were not modified.

docs/dev/reports/PR_26126_093_asset_manager_v2_palette_sample_notes.md

@@ -0,0 +1,27 @@
+# PR_26126_093 Asset Manager V2 Palette Sample Notes
+
+## Temporary UAT Loader
+- Added `tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js`.
+- The loader is explicitly marked temporary UAT-only and isolated for later removal.
+- It is activated only by `?palette=sample`.
+- Normal Workspace V2 palette loading remains separate.
+
+## Palette Format
+- The sample palette uses the direct Palette Browser payload shape:
+  - `schema: "html-js-gaming.palette"`
+  - `version`
+  - `id`
+  - `name`
+  - `source`
+  - `sourceId`
+  - `locked`
+  - `swatches`
+- Swatches include only schema-valid `symbol`, `hex`, `name`, `source`, and `tags` fields.
+
+## Behavior
+- `?palette=sample` loads the palette into memory and populates the Asset Manager V2 Color picker.
+- The file picker stays hidden when Type is Color.
+- Color asset IDs are generated from swatch names, for example `Signal Violet!` becomes `assets.color.hud.signal-violet`.
+
+## Validation
+- Playwright validates the UAT query loader, 20px swatches, Status entry, Color ID generation, path generation, and color asset insertion.

docs/dev/reports/PR_26126_093_asset_manager_v2_preview_path_resolution_notes.md

@@ -0,0 +1,18 @@
+# PR_26126_093 Asset Manager V2 Preview Path Resolution Notes
+
+## Workspace Context
+- Asset Manager V2 now passes Workspace V2 preview context into shared asset preview helpers.
+- The preview context reads `gameId` from the Workspace V2 host context or query string.
+- When Workspace mode is active and an asset path starts with `assets/`, previews resolve from `games/<gameId>/assets/...`.
+
+## Media Coverage
+- Audio preview `src` resolves from the Workspace game asset root.
+- Image preview `src` resolves from the Workspace game asset root.
+- Font preview `@font-face src` resolves from the Workspace game asset root.
+- Absolute paths, protocol URLs, `palette://workspace/...`, and existing `games/...` paths are left unchanged.
+
+## Validation
+- Playwright validates these resolved paths with `gameId: "Asteroids"`:
+  - `/games/Asteroids/assets/audio/fire.wav`
+  - `/games/Asteroids/assets/fonts/vector_battle.ttf`
+  - `/games/Asteroids/assets/images/preview.png`

docs/dev/reports/PR_26126_093_asset_manager_v2_selected_asset_detail_notes.md

@@ -0,0 +1,26 @@
+# PR_26126_093 Asset Manager V2 Selected Asset Detail Notes
+
+## UI Placement
+- Selected asset metadata moved out of the preview card.
+- Asset metadata now appears in a dedicated `Selected Asset Detail` accordion under the Assets panel.
+- The preview card now focuses on media or inspection rendering.
+
+## Detail Fields
+- The selected detail control renders:
+  - `type/kind`
+  - `ID`
+  - `Type`
+  - `Kind`
+  - `Role`
+  - `Path`
+  - `Final ID`
+
+## Related Controls
+- Asset Controls `ID` is now disabled and visibly grayed out.
+- `Path` remains read-only.
+- Stretch Override is now a bordered fieldset matching the Type, Pick Asset, and History group styling.
+- Stretch Override remains visible only when Role is `bezel`.
+
+## Validation
+- Playwright validates the selected detail rows after asset insertion.
+- Playwright validates the disabled ID field and the bordered Stretch Override group.

docs/dev/reports/codex_changed_files.txt

@@ -3,40 +3,36 @@ M docs/dev/reports/coverage_changed_js_guardrail.txt
  M docs/dev/reports/playwright_v8_coverage_report.txt
  M src/shared/assets/assetPreviewHelpers.js
  M tests/playwright/PreviewGeneratorV2Baseline.spec.mjs
- M tools/asset-manager-v2/README.md
  M tools/asset-manager-v2/index.html
  M tools/asset-manager-v2/js/AssetManagerV2App.js
- M tools/asset-manager-v2/js/assetManagerMetadata.js
  M tools/asset-manager-v2/js/bootstrap.js
+ M tools/asset-manager-v2/js/controls/AssetCatalogControl.js
  M tools/asset-manager-v2/js/controls/AssetFormControl.js
- M tools/asset-manager-v2/js/controls/AssetManagerShellControl.js
- M tools/asset-manager-v2/js/services/AssetSchemaValidator.js
  M tools/asset-manager-v2/js/services/WorkspaceBridge.js
  M tools/asset-manager-v2/styles/assetManager.css
- M tools/schemas/tools/asset-browser.schema.json
-?? docs/dev/reports/PR_26126_092_asset_manager_v2_color_asset_schema_notes.md
-?? docs/dev/reports/PR_26126_092_asset_manager_v2_manual_validation_notes.md
-?? docs/dev/reports/PR_26126_092_asset_manager_v2_palette_picker_notes.md
+?? docs/dev/reports/PR_26126_093_asset_manager_v2_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_093_asset_manager_v2_palette_sample_notes.md
+?? docs/dev/reports/PR_26126_093_asset_manager_v2_preview_path_resolution_notes.md
+?? docs/dev/reports/PR_26126_093_asset_manager_v2_selected_asset_detail_notes.md
+?? tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js
 
 # git ls-files --others --exclude-standard
-docs/dev/reports/PR_26126_092_asset_manager_v2_color_asset_schema_notes.md
-docs/dev/reports/PR_26126_092_asset_manager_v2_manual_validation_notes.md
-docs/dev/reports/PR_26126_092_asset_manager_v2_palette_picker_notes.md
+docs/dev/reports/PR_26126_093_asset_manager_v2_manual_validation_notes.md
+docs/dev/reports/PR_26126_093_asset_manager_v2_palette_sample_notes.md
+docs/dev/reports/PR_26126_093_asset_manager_v2_preview_path_resolution_notes.md
+docs/dev/reports/PR_26126_093_asset_manager_v2_selected_asset_detail_notes.md
+tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js
 
 # git diff --stat
-docs/dev/reports/coverage_changed_js_guardrail.txt |  14 +-
- docs/dev/reports/playwright_v8_coverage_report.txt |  38 ++--
- src/shared/assets/assetPreviewHelpers.js           |   6 +
- .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 166 +++++++++++++--
- tools/asset-manager-v2/README.md                   |   4 +-
- tools/asset-manager-v2/index.html                  |  45 ++--
- tools/asset-manager-v2/js/AssetManagerV2App.js     |  42 ++++
- tools/asset-manager-v2/js/assetManagerMetadata.js  |  54 +++--
- tools/asset-manager-v2/js/bootstrap.js             |   3 +
- .../js/controls/AssetFormControl.js                | 227 ++++++++++++++++++++-
- .../js/controls/AssetManagerShellControl.js        |   2 +-
- .../js/services/AssetSchemaValidator.js            |  61 +++++-
- .../js/services/WorkspaceBridge.js                 |  16 ++
- tools/asset-manager-v2/styles/assetManager.css     |  74 +++++++
- tools/schemas/tools/asset-browser.schema.json      | 115 ++++++++++-
- 15 files changed, 762 insertions(+), 105 deletions(-)
+docs/dev/reports/coverage_changed_js_guardrail.txt |  13 +-
+ docs/dev/reports/playwright_v8_coverage_report.txt |  31 ++--
+ src/shared/assets/assetPreviewHelpers.js           |  29 +--
+ .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 196 ++++++++++++++++++---
+ tools/asset-manager-v2/index.html                  |  26 ++-
+ tools/asset-manager-v2/js/AssetManagerV2App.js     |  20 ++-
+ tools/asset-manager-v2/js/bootstrap.js             |   2 +
+ .../js/controls/AssetCatalogControl.js             |  37 +++-
+ .../js/controls/AssetFormControl.js                |  19 +-
+ .../js/services/WorkspaceBridge.js                 |  42 +++++
+ tools/asset-manager-v2/styles/assetManager.css     |  68 ++++++-
+ 11 files changed, 412 insertions(+), 71 deletions(-)