Replace Asset Manager V2 palette UAT with workspace UAT and separate Playwright suites - PR_26126_106-asset-manager-v2-workspace-uat-and-test-separation

Author: DavidQ

docs/dev/reports/PR_26126_093_asset_manager_v2_manual_validation_notes.md

@@ -4,7 +4,7 @@
 - `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/TemporaryUatWorkspace.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`
@@ -19,7 +19,7 @@
 ## Manual Checks Covered By Tests
 - Disabled and grayed ID textbox.
 - Color ID generation from normalized swatch names.
-- Temporary `?palette=sample` UAT palette loading.
+- Temporary `?workspace=UAT` 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.

docs/dev/reports/PR_26126_093_asset_manager_v2_palette_sample_notes.md

@@ -1,9 +1,9 @@
 # PR_26126_093 Asset Manager V2 Palette Sample Notes
 
 ## Temporary UAT Loader
-- Added `tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js`.
+- Added `tools/asset-manager-v2/js/services/TemporaryUatWorkspace.js`.
 - The loader is explicitly marked temporary UAT-only and isolated for later removal.
-- It is activated only by `?palette=sample`.
+- It is activated only by `?workspace=UAT`.
 - Normal Workspace V2 palette loading remains separate.
 
 ## Palette Format
@@ -19,7 +19,7 @@
 - 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.
+- `?workspace=UAT` 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`.
 

docs/dev/reports/PR_26126_096_asset_manager_v2_manual_validation_notes.md

@@ -3,7 +3,7 @@
 ## Commands
 - `node --check tools/asset-manager-v2/js/assetPreviewHelpers.js`
 - `node --check tools/asset-manager-v2/js/services/WorkspaceBridge.js`
-- `node --check tools/asset-manager-v2/js/services/TemporaryUatSamplePalette.js`
+- `node --check tools/asset-manager-v2/js/services/TemporaryUatWorkspace.js`
 - `node --check tools/asset-manager-v2/js/AssetManagerV2App.js`
 - `node --check tests/playwright/PreviewGeneratorV2Baseline.spec.mjs`
 - `npx playwright test tests/playwright/PreviewGeneratorV2Baseline.spec.mjs --project=playwright --reporter=list -g "Asset Manager V2"`
@@ -20,6 +20,6 @@
 - Selected Asset Detail height is increased to at least 220px.
 - Asset preview remains inside Selected Asset Detail.
 - Workspace preview context is games-only.
-- `?palette=sample` temporary UAT root uses `games/Asteroids/` for preview/path testing only.
-- `?palette=sample` image previews resolve under `/games/Asteroids/assets/...`.
+- `?workspace=UAT` temporary UAT workspace root uses `games/Asteroids/` for preview/path testing only.
+- `?workspace=UAT` image previews resolve under `/games/Asteroids/assets/...`.
 - Sample JSON files were not modified.

docs/dev/reports/PR_26126_096_asset_manager_v2_temporary_uat_session_root_notes.md

@@ -1,18 +1,18 @@
 # PR_26126_096 Asset Manager V2 Temporary UAT Session Root Notes
 
 ## Temporary Behavior
-- `?palette=sample` remains isolated in `TemporaryUatSamplePalette.js`.
-- For UAT preview/path testing only, `?palette=sample` now provides the temporary game root `games/Asteroids/`.
+- `?workspace=UAT` remains isolated in `TemporaryUatWorkspace.js`.
+- For UAT preview/path testing only, `?workspace=UAT` now provides the temporary game root `games/Asteroids/`.
 - The temporary root is used only by Asset Manager V2 preview options.
 - Asset records still store normalized asset paths such as `assets/images/uat-preview.png`; the game root is not persisted into asset entries.
 - The code and Status log both mark this behavior as temporary UAT-only.
 
 ## Preview Result
-- During `?palette=sample`, file-backed previews for `assets/...` paths resolve through `/games/Asteroids/assets/...`.
+- During `?workspace=UAT`, file-backed previews for `assets/...` paths resolve through `/games/Asteroids/assets/...`.
 - The temporary preview path does not use Sample or Tool roots.
 - Palette color behavior remains unchanged and still uses the temporary sample palette swatches only.
 
 ## Validation
 - Playwright validates the temporary Status message for `games/Asteroids/`.
-- Playwright validates a `?palette=sample` image asset preview renders with `/games/Asteroids/assets/images/uat-preview.png`.
+- Playwright validates a `?workspace=UAT` image asset preview renders with `/games/Asteroids/assets/images/uat-preview.png`.
 - Playwright validates the UAT preview path excludes `/samples/` and `/tools/`.

docs/dev/reports/PR_26126_097_asset_manager_v2_font_preview_notes.md

@@ -8,7 +8,7 @@
 
 ## Path Resolution
 - Font preview URLs continue to resolve through the active Workspace game root.
-- During temporary `?palette=sample` UAT testing, `assets/fonts/vector-battle.ttf` resolves to `/games/Asteroids/assets/fonts/vector-battle.ttf`.
+- During temporary `?workspace=UAT` UAT testing, `assets/fonts/vector-battle.ttf` resolves to `/games/Asteroids/assets/fonts/vector-battle.ttf`.
 - Existing Workspace V2 font previews continue to resolve under `/games/Asteroids/assets/...` when `gameId=Asteroids` is active.
 
 ## Failure Reporting

docs/dev/reports/PR_26126_105_asset_manager_v2_launch_guard_notes.md

@@ -3,7 +3,7 @@
 ## Guarded Launches
 - Direct Asset Manager V2 launch without Workspace Manager context now shows the launch guard overlay before controls are mounted.
 - Workspace Manager launch without an active palette now shows the same guard overlay.
-- The temporary UAT-only `?palette=sample` context remains allowed and supplies the sample Asteroids game root for preview/path testing.
+- The temporary UAT-only `?workspace=UAT` context remains allowed and supplies the sample Asteroids game root for preview/path testing.
 
 ## Overlay Message
 - The guard message states: `Asset Manager V2 is only available through Workspace Manager with a game workspace and palette.`

docs/dev/reports/PR_26126_105_asset_manager_v2_manual_validation_notes.md

@@ -13,7 +13,7 @@
 - `npm run test:workspace-v2` passed 13/13 Playwright tests.
 - Playwright validates direct Asset Manager V2 launch shows the launch guard overlay.
 - Playwright validates Workspace launch without an active palette shows the launch guard overlay.
-- Playwright validates `?palette=sample` remains a valid temporary UAT context.
+- Playwright validates `?workspace=UAT` remains a valid temporary UAT context.
 - Playwright validates Path is disabled and visually matches the disabled ID field.
 - Playwright validates click/tap asset selection continues to update selected tile styling, Selected Asset Detail, and preview.
 - The keyboard cleanup `rg` search returned no matches in Asset Manager V2 source or the touched Playwright spec.

docs/dev/reports/PR_26126_106_asset_manager_v2_keyboard_cleanup_review_notes.md

@@ -0,0 +1,14 @@
+# PR_26126_106 Keyboard Cleanup Review Notes
+
+## Review Scope
+- Re-reviewed Asset Manager V2 source and Playwright tests after moving the Asset Manager coverage to its own spec.
+- Searched Asset Manager source, `AssetManagerV2.spec.mjs`, and `PreviewGeneratorV2Baseline.spec.mjs` for the retired navigation keys and keyboard terms.
+
+## Result
+- No Asset Manager V2 handling or assertions remain for KeyD, KeyA, KeyS, KeyW, ArrowLeft, ArrowRight, ArrowUp, ArrowDown, Home, End, PageUp, PageDown, or Enter navigation.
+- No WASD, arrow-key, page-key, or `keydown` asset-navigation references remain in the reviewed source/tests.
+- Asset selection remains click/tap driven and is validated in `AssetManagerV2.spec.mjs`.
+
+## Validation
+- `npm run test:workspace-v2` passed 14/14 tests.
+- No sample JSON was modified.

docs/dev/reports/PR_26126_106_asset_manager_v2_test_separation_notes.md

@@ -0,0 +1,15 @@
+# PR_26126_106 Test Separation Notes
+
+## Dedicated Spec
+- Asset Manager V2 Playwright coverage moved from `tests/playwright/PreviewGeneratorV2Baseline.spec.mjs` to `tests/playwright/AssetManagerV2.spec.mjs`.
+- Asset Manager launch, UAT palette/session, import/export, preview, Workspace insertion, and launch-guard assertions now live in the dedicated Asset Manager V2 spec.
+- `PreviewGeneratorV2Baseline.spec.mjs` no longer contains Asset Manager V2 helpers or assertions.
+
+## Workspace Gate
+- `npm run test:workspace-v2` now runs both `PreviewGeneratorV2Baseline.spec.mjs` and `AssetManagerV2.spec.mjs` with one worker.
+- Both specs share `tests/helpers/workspaceV2CoverageReporter.mjs` so the Playwright/V8 report remains combined across the separated suite.
+- The required validation passed with 14/14 Playwright tests.
+
+## Scope
+- The separated suite preserves Palette Manager V2 coverage naming and Asset Manager V2 runtime coverage in the combined V8 report.
+- No sample JSON was modified.

docs/dev/reports/PR_26126_106_asset_manager_v2_workspace_uat_notes.md

@@ -0,0 +1,17 @@
+# PR_26126_106 Workspace UAT Notes
+
+## Launch Modes
+- Asset Manager V2 now allows only Workspace Manager session launches or temporary `?workspace=UAT`.
+- Direct launch without Workspace Manager session context hard-fails to the launch guard overlay.
+- `?workspace=PROD` hard-fails to the launch guard overlay; production launch is only supported through Workspace Manager session state.
+
+## Temporary UAT Context
+- `?workspace=UAT` seeds the temporary Asset Manager V2 UAT session with `gameRoot = games/Asteroids/`.
+- `?workspace=UAT` seeds `assetsPath = games/Asteroids/assets`.
+- `?workspace=UAT` seeds the temporary UAT sample palette for Color asset validation and palette-only picking.
+- Preview paths under UAT continue to resolve through `/games/Asteroids/assets/...`.
+
+## Retired Query
+- Removed the old sample-palette query source helper path, messages, tests, and report references.
+- The isolated helper is now `TemporaryUatWorkspace.js`.
+- No sample JSON was modified.