Complete Asset Manager V2 schema naming, layout, status, and undo redo polish - PR_26126_088-asset-manager-v2-schema-ui-complete-pass

Author: DavidQ

docs/dev/reports/PR_26126_088_asset_manager_v2_manual_validation_notes.md

@@ -0,0 +1,36 @@
+# PR_26126_088 Asset Manager V2 Manual Validation Notes
+
+Date: 2026-05-06
+
+## Validation Performed
+
+- Ran `npm run test:workspace-v2`.
+- Ran JS syntax checks on touched Asset Manager V2 modules and the Playwright baseline.
+- Ran `git diff --check`.
+- Confirmed no JSON files changed.
+
+## Results
+
+- `npm run test:workspace-v2`: passed, 10 tests.
+- Schema naming consistency: passed. UI/output tests validate `id`, `type`, `kind`, `role`, and `path`, while Workspace V2 insertion remains schema-valid.
+- Radio alignment: passed. Kind radios align left.
+- Localization label fit: passed.
+- Fullscreen layout: passed. Left and right panels align to shell edges and center expands between them.
+- Tile wrapping: passed under fullscreen-expanded center width, while normal width stacks vertically.
+- Delete placement: passed. Delete is left of `type:role`.
+- Tooltip details: passed. Tile tooltip contains `id`, `type`, `kind`, `role`, and `path`.
+- Status-only add/delete/update messages: passed.
+- Pick Asset File role reset: passed. Role resets to the selected kind default before filename background/bezel logic.
+- Undo/Redo: passed for update and delete flows.
+- Workspace V2 insertion: passed. Audio asset inserts only into `tools.asset-browser.assets`.
+- Sample JSON: passed. No JSON files were modified.
+
+## Reports
+
+- `docs/dev/reports/playwright_v8_coverage_report.txt`
+- `docs/dev/reports/coverage_changed_js_guardrail.txt`
+- `docs/dev/reports/PR_26126_088_asset_manager_v2_schema_consistency_notes.md`
+- `docs/dev/reports/PR_26126_088_asset_manager_v2_ui_layout_notes.md`
+- `docs/dev/reports/PR_26126_088_asset_manager_v2_undo_redo_behavior_notes.md`
+- `docs/dev/reports/PR_26126_088_asset_manager_v2_manual_validation_notes.md`
+

docs/dev/reports/PR_26126_088_asset_manager_v2_schema_consistency_notes.md

@@ -0,0 +1,26 @@
+# PR_26126_088 Asset Manager V2 Schema Consistency Notes
+
+Date: 2026-05-06
+
+## Scope
+
+- Asset Manager V2 keeps the persisted Workspace V2 asset schema contract strict.
+- Stored Workspace entries continue to use schema fields `path`, `kind`, `role`, and `source`.
+- The visible Assets list and default Output Summary now expose compact display fields `id`, `type`, `kind`, `role`, and `path`.
+- `type` is the display alias for the schema `kind` value; both are shown in Output Summary and tile tooltips so the UI naming is explicit.
+
+## Naming Changes
+
+- Removed remaining visible "approved asset" wording from the Asset Manager V2 surface and current tools index planned text.
+- Replaced visible load messaging with `Kinds` and `Roles`.
+- Replaced validation wording that referenced "asset id" with `id`.
+- Kept Workspace V2 insertion at `workspaceManifest.tools["asset-browser"].assets`, using schema-valid entries with `kind`, `role`, `path`, and `source`.
+- Default Output Summary is a compact display summary and does not include add, delete, or update status messages.
+
+## Validation
+
+- `npm run test:workspace-v2`: passed, 10 tests.
+- Playwright validates the first selected image output includes `id`, `type`, `kind`, `role`, and `path`.
+- Playwright validates Workspace V2 insertion still writes the schema-valid audio asset entry to `tools.asset-browser.assets`.
+- `git diff --name-only -- "*.json"` returned no changed JSON files, so sample JSON was not modified.
+

docs/dev/reports/PR_26126_088_asset_manager_v2_ui_layout_notes.md

@@ -0,0 +1,26 @@
+# PR_26126_088 Asset Manager V2 UI Layout Notes
+
+Date: 2026-05-06
+
+## Layout Changes
+
+- Asset kind radio labels align their radio inputs to the left.
+- The Localization radio label uses a smaller font size so it fits cleanly.
+- Fullscreen-active Asset Manager V2 uses a full-width app shell, left panel aligned to the left edge, right panel aligned to the right edge, and a center panel that expands between them.
+- Assets tiles use a responsive grid with `repeat(auto-fit, minmax(220px, 1fr))`, allowing horizontal wrapping when width is available and vertical stacking when it is not.
+- Each asset tile places Delete to the left of the `type:role` display.
+- Asset detail text moved from the visible preview area into each tile tooltip.
+- Visible tile content stays compact: Delete, `type:role`, and ID only.
+
+## Status-Only Messages
+
+- Add, delete, and update action messages are written only to Status.
+- Asset Controls form messaging does not echo `Added`, `Deleted`, or `Updated` messages.
+- Output Summary does not display those action messages.
+- Asset tiles do not display those action messages.
+
+## Validation
+
+- `npm run test:workspace-v2`: passed, 10 tests.
+- Playwright validates radio alignment, Localization label fit, fullscreen panel edge alignment, center panel expansion, tile wrapping, Delete placement, tooltip details, and compact tile text.
+

docs/dev/reports/PR_26126_088_asset_manager_v2_undo_redo_behavior_notes.md

@@ -0,0 +1,29 @@
+# PR_26126_088 Asset Manager V2 Undo Redo Behavior Notes
+
+Date: 2026-05-06
+
+## Behavior
+
+- Asset Manager V2 now tracks local asset-state history for add, delete, and update operations.
+- Undo restores the previous validated asset map and selected asset.
+- Redo reapplies the undone validated asset map and selected asset.
+- Undo and Redo update:
+  - Assets display
+  - Output Summary
+  - Status
+  - Asset Controls history button state
+- New add, delete, or update operations clear the redo stack.
+
+## Edit Operation
+
+- Selecting an asset tile loads its `id`, `kind`, `role`, and `path` into Asset Controls.
+- Update Asset validates the edited entry through the same schema validator used by Add.
+- Renaming an ID during update is allowed only when the target ID is not already present.
+
+## Validation
+
+- `npm run test:workspace-v2`: passed, 10 tests.
+- Playwright validates Update Asset by changing an audio asset path.
+- Playwright validates Undo restores the previous path and Redo reapplies the updated path.
+- Playwright validates deleting an image asset, Undo restoring it, and Redo deleting it again.
+

docs/dev/reports/codex_changed_files.txt

@@ -2,23 +2,37 @@
 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/README.md
  M tools/asset-manager-v2/index.html
- M tools/asset-manager-v2/js/controls/AccordionSection.js
+ M tools/asset-manager-v2/js/AssetManagerV2App.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/services/AssetSchemaValidator.js
  M tools/asset-manager-v2/styles/assetManager.css
  M tools/index.html
-?? docs/dev/reports/PR_26126_087_asset_manager_v2_accordion_behavior_notes.md
-?? docs/dev/reports/PR_26126_087_asset_manager_v2_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_088_asset_manager_v2_manual_validation_notes.md
+?? docs/dev/reports/PR_26126_088_asset_manager_v2_schema_consistency_notes.md
+?? docs/dev/reports/PR_26126_088_asset_manager_v2_ui_layout_notes.md
+?? docs/dev/reports/PR_26126_088_asset_manager_v2_undo_redo_behavior_notes.md
 
 # git ls-files --others --exclude-standard
-docs/dev/reports/PR_26126_087_asset_manager_v2_accordion_behavior_notes.md
-docs/dev/reports/PR_26126_087_asset_manager_v2_manual_validation_notes.md
+docs/dev/reports/PR_26126_088_asset_manager_v2_manual_validation_notes.md
+docs/dev/reports/PR_26126_088_asset_manager_v2_schema_consistency_notes.md
+docs/dev/reports/PR_26126_088_asset_manager_v2_ui_layout_notes.md
+docs/dev/reports/PR_26126_088_asset_manager_v2_undo_redo_behavior_notes.md
 
 # git diff --stat
-docs/dev/reports/coverage_changed_js_guardrail.txt |  1 +
- docs/dev/reports/playwright_v8_coverage_report.txt |  4 +-
- .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 62 ++++++++++++++++++++++
- tools/asset-manager-v2/index.html                  |  6 +--
- .../js/controls/AccordionSection.js                |  7 +++
- tools/asset-manager-v2/styles/assetManager.css     | 13 ++++-
- tools/index.html                                   |  8 +++
- 7 files changed, 95 insertions(+), 6 deletions(-)
+docs/dev/reports/coverage_changed_js_guardrail.txt |  10 +-
+ docs/dev/reports/playwright_v8_coverage_report.txt |  26 +--
+ .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 155 ++++++++++++++++--
+ tools/asset-manager-v2/README.md                   |   4 +-
+ tools/asset-manager-v2/index.html                  |  23 +--
+ tools/asset-manager-v2/js/AssetManagerV2App.js     | 181 +++++++++++++++++++--
+ tools/asset-manager-v2/js/bootstrap.js             |   3 +
+ .../js/controls/AssetCatalogControl.js             |  28 ++--
+ .../js/controls/AssetFormControl.js                |  54 +++++-
+ .../js/services/AssetSchemaValidator.js            |  14 +-
+ tools/asset-manager-v2/styles/assetManager.css     | 112 ++++++++++++-
+ tools/index.html                                   |   2 +-
+ 12 files changed, 525 insertions(+), 87 deletions(-)