Add Color usage ID segment and adjust Asset Manager V2 detail height - PR_26126_100-asset-manager-v2-color-usage-and-detail-height

Author: DavidQ

docs/dev/reports/PR_26126_100_asset_manager_v2_color_usage_id_notes.md

@@ -0,0 +1,17 @@
+# PR_26126_100 Asset Manager V2 Color Usage ID Notes
+
+## Usage Field
+- Color assets now expose a required `Usage` textbox directly under `Role`.
+- `Usage` is hidden and disabled for non-Color asset types.
+- The typed Usage value remains editable as entered; normalization is applied only when generating the asset ID segment.
+
+## Color ID Shape
+- Color IDs now use `assets.<type>.<role>.<usage>.<colorName>`.
+- Example: Usage `Primary HUD` plus swatch `Sky Blue` becomes `assets.color.hud.primary-hud.sky-blue`.
+- The original palette swatch name is not altered in color metadata.
+- Color path generation is unchanged; swatch `Sky Blue` still resolves to `palette://workspace/sky-blue`.
+
+## Validation
+- Selecting a Color swatch without Usage leaves the generated ID empty and keeps `Add Asset` disabled.
+- Missing Usage validation is routed to Status only: `Selected color validation failed: Color usage is required for color assets.`
+- Schema validation rejects Color asset IDs that do not include both Usage and color name segments.

docs/dev/reports/PR_26126_100_asset_manager_v2_manual_validation_notes.md

@@ -0,0 +1,25 @@
+# PR_26126_100 Asset Manager V2 Manual Validation Notes
+
+## Commands
+- `node --check tools/asset-manager-v2/js/controls/AssetFormControl.js`
+- `node --check tools/asset-manager-v2/js/assetManagerMetadata.js`
+- `node --check tools/asset-manager-v2/js/services/AssetSchemaValidator.js`
+- `node --check tests/playwright/PreviewGeneratorV2Baseline.spec.mjs`
+- `git diff --check`
+- `npm run test:workspace-v2`
+
+## Results
+- JavaScript syntax checks passed for changed runtime and Playwright files.
+- Required workspace-v2 Playwright gate passed: 12 tests passed.
+- `git diff --check` passed with no whitespace errors.
+- `docs/dev/reports/playwright_v8_coverage_report.txt` was regenerated by the required workspace-v2 validation run.
+
+## Manual Checks Covered By Tests
+- Selected Asset Detail is 20px taller through the embedded preview area while retaining the PR_26126_099 bottom-spacing guard.
+- Usage appears directly under Role only for Color assets.
+- Usage is hidden and disabled for non-Color assets.
+- Color swatch selection without Usage writes validation failure to Status only and keeps Add Asset disabled.
+- Color IDs include normalized Usage and normalized swatch name: `assets.<type>.<role>.<usage>.<colorName>`.
+- Color paths and original swatch names are preserved.
+- Workspace V2 insertion uses the new Color ID structure.
+- Sample JSON files were not modified.

docs/dev/reports/PR_26126_100_asset_manager_v2_selected_detail_height_notes.md

@@ -0,0 +1,13 @@
+# PR_26126_100 Asset Manager V2 Selected Detail Height Notes
+
+## Height Adjustment
+- Selected Asset Detail remains content-controlled; no fixed `300px` height was restored.
+- The embedded preview area min-height increased from `104px` to `124px`, raising the selected detail area by 20px through real content.
+
+## Spacing Guard
+- The PR_26126_099 trailing whitespace checks remain in workspace-v2 coverage.
+- Asset Controls, Assets, and Selected Asset Detail still validate with no extra trailing body gap after their final visible content.
+
+## Validation
+- Playwright verifies the Selected Asset Detail preview min-height is `124px`.
+- Playwright verifies the Selected Asset Detail section height remains content-driven and not fixed at `300px`.

docs/dev/reports/codex_changed_files.txt

@@ -3,18 +3,28 @@ M docs/dev/reports/coverage_changed_js_guardrail.txt
  M docs/dev/reports/playwright_v8_coverage_report.txt
  M tests/playwright/PreviewGeneratorV2Baseline.spec.mjs
  M tools/asset-manager-v2/index.html
+ M tools/asset-manager-v2/js/assetManagerMetadata.js
+ M tools/asset-manager-v2/js/bootstrap.js
+ M tools/asset-manager-v2/js/controls/AssetFormControl.js
+ M tools/asset-manager-v2/js/services/AssetSchemaValidator.js
  M tools/asset-manager-v2/styles/assetManager.css
-?? docs/dev/reports/PR_26126_099_asset_manager_v2_accordion_spacing_notes.md
-?? docs/dev/reports/PR_26126_099_asset_manager_v2_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_100_asset_manager_v2_color_usage_id_notes.md
+?? docs/dev/reports/PR_26126_100_asset_manager_v2_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_100_asset_manager_v2_selected_detail_height_notes.md
 
 # git ls-files --others --exclude-standard
-docs/dev/reports/PR_26126_099_asset_manager_v2_accordion_spacing_notes.md
-docs/dev/reports/PR_26126_099_asset_manager_v2_manual_validation_notes.md
+docs/dev/reports/PR_26126_100_asset_manager_v2_color_usage_id_notes.md
+docs/dev/reports/PR_26126_100_asset_manager_v2_manual_validation_notes.md
+docs/dev/reports/PR_26126_100_asset_manager_v2_selected_detail_height_notes.md
 
 # git diff --stat
-docs/dev/reports/coverage_changed_js_guardrail.txt |  2 --
- docs/dev/reports/playwright_v8_coverage_report.txt |  4 ---
- .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 38 +++++++++++++++++++---
- tools/asset-manager-v2/index.html                  | 17 +++++++---
- tools/asset-manager-v2/styles/assetManager.css     | 18 ++++++----
- 5 files changed, 57 insertions(+), 22 deletions(-)
+docs/dev/reports/coverage_changed_js_guardrail.txt |  8 ++--
+ docs/dev/reports/playwright_v8_coverage_report.txt | 22 ++++-----
+ .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 53 ++++++++++++++++++----
+ tools/asset-manager-v2/index.html                  |  4 ++
+ tools/asset-manager-v2/js/assetManagerMetadata.js  | 11 +++++
+ tools/asset-manager-v2/js/bootstrap.js             |  2 +
+ .../js/controls/AssetFormControl.js                | 44 ++++++++++++++++--
+ .../js/services/AssetSchemaValidator.js            | 11 ++++-
+ tools/asset-manager-v2/styles/assetManager.css     |  4 +-
+ 9 files changed, 128 insertions(+), 31 deletions(-)