BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION

- Strengthened Workspace Manager game-open test to validate asset presence
- Added shared palette and expected asset assertions
- Added Bouncing-ball missing palette regression coverage

PR Details:
- Prevents page-load-only false positives
- No validators
- No start_of_day changes

Author: DavidQ

docs/dev/reports/expected_delta_template.md

@@ -1,25 +1,22 @@
 # Expected Codex Return / Delta Template
 
-## Expected Changed/Added Files
+## Expected Changed Files
 - `tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs` or equivalent
-- `tests/run-tests.mjs` only if needed to include the new test in the correct suite
-- `package.json` only if a focused npm script is needed
-- `docs/dev/reports/level_10_2_workspace_manager_open_test_report.md`
-- `docs/dev/reports/level_10_2_asteroids_platform_demo_boundary_audit.md`
+- `docs/dev/reports/level_10_2a_workspace_manager_asset_presence_validation_report.md`
 - `docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md` if status update needed
 
 ## Expected Validation Summary
-- `games_index_test_added=true`
-- `games_discovered=<count>`
-- `workspace_manager_actions_checked=<count>`
-- `actions_with_gameId=<count>`
-- `actions_with_mount_game=<count>`
-- `actions_using_legacy_game_query=0`
-- `workspace_manager_diagnostic_failures=0`
-- `asteroids_platform_demo_boundary_audited=true`
+- `games_checked=<count>`
+- `workspace_manager_open_actions_valid=<count>`
+- `diagnostic_failures=0`
+- `games_with_shared_palette=<count>`
+- `games_missing_shared_palette=<count>`
+- `bouncing_ball_shared_palette_present=true`
+- `games_with_expected_assets=<count>`
+- `asset_presence_test_added=true`
 - `start_of_day_changes=0`
 
 ## Expected Delta ZIP
 Codex must create:
 
-`tmp/BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT_delta.zip`
+`tmp/BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION_delta.zip`

docs/dev/reports/level_10_2_asteroids_platform_demo_boundary_audit.md

@@ -1,8 +1,52 @@
 # Level 10.2 Asteroids Platform Demo Boundary Audit
 
-## To Be Completed By Codex
-- Identify all imports of `tools/shared/asteroidsPlatformDemo.js`.
-- Identify imports used by that file.
-- Determine whether it is tool-generic, game-specific, sample-specific, or engine-generic.
-- Recommend final owning folder.
-- State whether move is safe now or should be a follow-up PR.
+## File Audited
+- `tools/shared/asteroidsPlatformDemo.js`
+
+## Consumers (Who Imports It)
+- `tests/games/AsteroidsPlatformDemo.test.mjs`
+- `tests/games/AsteroidsAssetReferenceAdoption.test.mjs`
+
+No runtime game boot/module import was found outside tests.
+
+## Imports Used By `asteroidsPlatformDemo.js`
+- `./projectAssetValidation.js`
+- `./projectAssetRemediation.js`
+- `./projectPackaging.js`
+- `./runtimeAssetLoader.js`
+- `./gameplaySystemLayer.js`
+- `./gameTemplates.js`
+- `./multiTargetExport.js`
+- `./debugVisualizationLayer.js`
+- `./performanceProfiler.js`
+- `./publishingPipeline.js`
+- `./vector/vectorAssetBridge.js`
+- `../../src/shared/utils/jsonUtils.js`
+- `./pipeline/runtimeAssetLookup.js`
+
+## Domain Ownership Assessment
+- The file is strongly Asteroids/game-specific:
+  - hardcoded `Asteroids` identifiers
+  - `games/Asteroids/...` manifest fragment paths
+  - `games/Asteroids/main.js` runtime handoff expectations
+  - Asteroids-specific vector/tilemap/parallax/palette asset IDs
+- It orchestrates tool/shared pipeline modules, but the orchestration content is game-domain specific.
+
+## Boundary Conclusion
+- Current placement in `tools/shared/` is a boundary smell.
+- The file is not a tool-agnostic shared utility.
+
+## Recommended Target
+- Preferred ownership target:
+  - `games/Asteroids/shared/asteroidsPlatformDemo.js`
+- Alternative if retained as test fixture-only orchestration:
+  - `tests/fixtures/games/asteroidsPlatformDemo.js`
+
+## Move Decision For This PR
+- **No move in Level 10.2**.
+- Reason:
+  - safe move requires coordinated import updates and re-validation across the Asteroids demo tests and dependent pipeline assumptions.
+  - this PR scope is focused on Workspace Manager open-action runtime testing plus audit.
+
+## Follow-Up Recommendation
+- Create a focused follow-up BUILD to relocate `tools/shared/asteroidsPlatformDemo.js` to a game-owned or test-fixture-owned boundary and update imports atomically.

docs/dev/reports/level_10_2_workspace_manager_open_test_report.md

@@ -1,9 +1,71 @@
 # Level 10.2 Workspace Manager Open Test Report
 
-## To Be Completed By Codex
-- List games discovered from `games/index.html`.
-- List Workspace Manager actions found.
-- Confirm `gameId` query usage.
-- Confirm `mount=game`.
-- Confirm no `?game=` legacy query usage.
-- Confirm no Workspace Manager diagnostic appears.
+## Scope
+- BUILD: `BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT`
+- Focused runtime/browser test added:
+  - `tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs`
+- Focused npm command added:
+  - `npm run test:workspace-manager:games`
+
+## Test Coverage
+- Opens `/games/index.html` through local static server.
+- Discovers game cards from DOM (`#games-index-list article[data-game-id]`).
+- Discovers Workspace Manager actions (`a[data-workspace-launch-href]`) per launchable game.
+- Validates each Workspace Manager action URL:
+  - path resolves to `tools/Workspace Manager/index.html`
+  - query contains `gameId=<id>`
+  - query contains `mount=game`
+  - query does not use legacy `?game=`
+- Opens each Workspace Manager URL and verifies diagnostic text does not appear:
+  - `Workspace Manager game launch requires a valid gameId`
+
+## Validation Run
+- Command: `npm run test:workspace-manager:games`
+- Result: `PASS`
+
+## Discovery And Counts
+- Games discovered from metadata: `29`
+- Game cards discovered from DOM: `29`
+- Launchable games (expected Workspace Manager actions): `11`
+- Workspace Manager actions discovered: `11`
+
+## Game IDs Discovered From `games/index.html`
+- `AITargetDummy`
+- `ArenaShooter`
+- `Asteroids`
+- `Bouncing-ball`
+- `Breakout`
+- `ConnectFour`
+- `DefenderLite`
+- `DonkeyKong`
+- `DualStickArena`
+- `Frogger`
+- `FullArenaShooter`
+- `Galaga`
+- `GravityWell`
+- `KingOfTheIceberg`
+- `MultiplayerOptional`
+- `Othello`
+- `Pacman`
+- `PlatformerDemo`
+- `Pong`
+- `RallyX`
+- `ReplaySystem`
+- `Robotron2084`
+- `ScrollingWorldTest`
+- `SolarSystem`
+- `SpaceDuel`
+- `SpaceInvaders`
+- `TicTacToe`
+- `TileCollisionDemo`
+- `vector-arcade-sample`
+
+## Query Validation Summary
+- actions with `gameId=<id>`: `11/11`
+- actions with `mount=game`: `11/11`
+- actions using legacy `?game=`: `0`
+- Workspace Manager diagnostic failures after open: `0`
+- structural/action validation failures: `0`
+
+## Notes
+- The new test is games-index/workspace-manager specific and does not execute sample or tool launch suites.

docs/dev/reports/level_10_2a_workspace_manager_asset_presence_validation_report.md

@@ -0,0 +1,74 @@
+# Level 10.2A Workspace Manager Asset Presence Validation Report
+
+## Scope
+- BUILD: `BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION`
+- Extended runtime/browser test: `tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs`
+- Command: `npm run test:workspace-manager:games`
+- Samples/tools suites were not executed; this path only targets `games/index.html` Workspace Manager actions.
+
+## Existing Checks Preserved
+- actions with `gameId=<id>`: `11/11`
+- actions with `mount=game`: `11/11`
+- actions using legacy `?game=`: `0`
+- diagnostic failures: `0`
+
+## Added Asset Presence Checks
+- workspace loaded in mounted tool frame: `11/11`
+- shared palette present: `0/11`
+- shared assets present: `10/11`
+- asset presence failures: `24`
+
+## Bouncing-ball Regression Check
+- shared palette present: `False`
+- palette label: `No shared palette selected`
+- expected skin visible (`Bouncing Ball Classic Skin`): `True`
+- failure(s):
+  - Shared palette missing for Bouncing-ball.
+  - Manifest palette expected but UI shows missing palette for Bouncing-ball.
+  - Bouncing-ball regression: UI shows "Shared Palette: No shared palette selected".
+
+## Per-Game Results
+| Game | Open Action Valid | Diagnostic Absent | Workspace Loaded | Shared Palette Present | Shared Assets Present | Expected Skin | Observed Shared Asset | Failures |
+| --- | --- | --- | --- | --- | --- | --- | --- | --- |
+| Bouncing-ball | yes | yes | yes | no | yes | Bouncing Ball Classic Skin | Bouncing Ball Classic Skin | Shared palette missing for Bouncing-ball.; Manifest palette expected but UI shows missing palette for Bouncing-ball.; Bouncing-ball regression: UI shows "Shared Palette: No shared palette selected". |
+| Breakout | yes | yes | yes | no | yes | Breakout Classic Skin | Breakout Classic Skin | Shared palette missing for Breakout.; Manifest palette expected but UI shows missing palette for Breakout. |
+| SolarSystem | yes | yes | yes | no | yes | Solar System Classic Skin | Solar System Classic Skin | Shared palette missing for SolarSystem.; Manifest palette expected but UI shows missing palette for SolarSystem. |
+| GravityWell | yes | yes | yes | no | yes | n/a | image.gravitywell.preview | Shared palette missing for GravityWell.; Manifest palette expected but UI shows missing palette for GravityWell. |
+| Pong | yes | yes | yes | no | yes | Pong Classic Skin | Pong Classic Skin | Shared palette missing for Pong.; Manifest palette expected but UI shows missing palette for Pong. |
+| Asteroids | yes | yes | yes | no | no | n/a | No shared asset selected | Shared palette missing for Asteroids.; Shared assets missing for Asteroids.; Manifest palette expected but UI shows missing palette for Asteroids. |
+| SpaceInvaders | yes | yes | yes | no | yes | n/a | audio.space-invaders.shoot | Shared palette missing for SpaceInvaders.; Manifest palette expected but UI shows missing palette for SpaceInvaders. |
+| SpaceDuel | yes | yes | yes | no | yes | n/a | audio.space-duel.thrust | Shared palette missing for SpaceDuel.; Manifest palette expected but UI shows missing palette for SpaceDuel. |
+| AITargetDummy | yes | yes | yes | no | yes | n/a | image.ai-target-dummy.preview | Shared palette missing for AITargetDummy.; Manifest palette expected but UI shows missing palette for AITargetDummy. |
+| Pacman | yes | yes | yes | no | yes | n/a | image.pacman.preview-svg | Shared palette missing for Pacman.; Manifest palette expected but UI shows missing palette for Pacman. |
+| vector-arcade-sample | yes | yes | yes | no | yes | n/a | Vector Arcade Template Sprite | Shared palette missing for vector-arcade-sample.; Manifest palette expected but UI shows missing palette for vector-arcade-sample. |
+
+## Failure Summary
+- Test intentionally fails when required data is missing after page load. Current failures:
+  - Shared palette missing for Bouncing-ball.
+  - Manifest palette expected but UI shows missing palette for Bouncing-ball.
+  - Bouncing-ball regression: UI shows "Shared Palette: No shared palette selected".
+  - Shared palette missing for Breakout.
+  - Manifest palette expected but UI shows missing palette for Breakout.
+  - Shared palette missing for SolarSystem.
+  - Manifest palette expected but UI shows missing palette for SolarSystem.
+  - Shared palette missing for GravityWell.
+  - Manifest palette expected but UI shows missing palette for GravityWell.
+  - Shared palette missing for Pong.
+  - Manifest palette expected but UI shows missing palette for Pong.
+  - Shared palette missing for Asteroids.
+  - Shared assets missing for Asteroids.
+  - Manifest palette expected but UI shows missing palette for Asteroids.
+  - Shared palette missing for SpaceInvaders.
+  - Manifest palette expected but UI shows missing palette for SpaceInvaders.
+  - Shared palette missing for SpaceDuel.
+  - Manifest palette expected but UI shows missing palette for SpaceDuel.
+  - Shared palette missing for AITargetDummy.
+  - Manifest palette expected but UI shows missing palette for AITargetDummy.
+  - Shared palette missing for Pacman.
+  - Manifest palette expected but UI shows missing palette for Pacman.
+  - Shared palette missing for vector-arcade-sample.
+  - Manifest palette expected but UI shows missing palette for vector-arcade-sample.
+
+## Outcome
+- Result: `FAIL` (expected for current missing shared palette conditions).
+- Page-load-only false positives are prevented because the test now requires mounted shared palette/asset data, not just shell load.

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

@@ -899,3 +899,5 @@
 - [x] Tool input alignment contract is documented as `toolInput = gameManifest.tools[toolId]` (no game-owned file-path JSON input).
 - [x] Palette standalone singleton correction is complete (`game.manifest.json` root `palette`, no tool-owned `palettes`).
 - [x] Palette tool singleton + tool section metadata correction is complete (`tools["palette-browser"].palette` + metadata on all tool sections).
+- [x] Games index Workspace Manager open-action browser test is complete (`gameId` + `mount=game`, legacy `?game=` rejected, diagnostic check included).
+- [x] `tools/shared/asteroidsPlatformDemo.js` shared-boundary audit is complete with move recommendation captured for follow-up.

docs/operations/dev/codex_commands.md

@@ -2,39 +2,37 @@ MODEL: GPT-5.3-codex
 REASONING: high
 
 TASK:
-Apply BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT.
+Apply BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION.
 
 STEPS:
-1. Read docs/pr/PLAN_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT.md.
-2. Read docs/pr/BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT.md.
-3. Add a focused runtime/browser test for `games/index.html`:
-   - discover all games/cards/actions
-   - validate "Open with Workspace Manager" uses `gameId=<id>&mount=game`
-   - reject legacy `?game=`
-   - ensure Workspace Manager diagnostic does not appear when clicked/opened
-4. Ensure the new test does not run samples or tools.
-5. Add a focused npm script only if useful, such as:
-   - `test:workspace-manager:games`
-6. Audit `tools/shared/asteroidsPlatformDemo.js`:
-   - imports
-   - consumers
-   - domain ownership
-   - move recommendation
-7. Move `tools/shared/asteroidsPlatformDemo.js` only if safe and obvious; otherwise report follow-up.
-8. Write reports:
-   - docs/dev/reports/level_10_2_workspace_manager_open_test_report.md
-   - docs/dev/reports/level_10_2_asteroids_platform_demo_boundary_audit.md
-9. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only if needed:
-   - [ ] -> [.]
-   - [.] -> [x]
-   - no prose rewrite/delete
-10. Do not modify start_of_day.
+1. Read docs/pr/PLAN_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION.md.
+2. Read docs/pr/BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION.md.
+3. Open the Level 10.2 Workspace Manager open test.
+4. Extend it so it validates game data presence after opening each game through Workspace Manager.
+5. Keep existing checks:
+   - gameId query
+   - mount=game
+   - no ?game=
+   - no diagnostic
+6. Add asset presence checks:
+   - shared palette present
+   - expected shared assets/tool sections present
+   - skin data present when expected
+7. Specifically fail Bouncing-ball if the UI shows:
+   - `Shared Palette: No shared palette selected`
+8. Ensure this test does not run samples/tools.
+9. Write docs/dev/reports/level_10_2a_workspace_manager_asset_presence_validation_report.md.
+10. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only if needed:
+    - [ ] -> [.]
+    - [.] -> [x]
+    - no prose rewrite/delete
 11. Do not add validators.
-12. Create Codex delta ZIP:
-    tmp/BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT_delta.zip
+12. Do not modify start_of_day.
+13. Create Codex delta ZIP:
+    tmp/BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION_delta.zip
 
 ACCEPTANCE:
-- games index Workspace Manager action test exists
-- all game actions use gameId + mount=game
-- boundary audit exists
+- page-load-only false positives are prevented
+- missing palette is caught
+- Bouncing-ball missing palette issue is detected/fixed or reported
 - delta ZIP exists

docs/operations/dev/commit_comment.txt

@@ -1,10 +1,10 @@
-BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT
+BUILD_PR_LEVEL_10_2A_WORKSPACE_MANAGER_ASSET_PRESENCE_VALIDATION
 
-- Added games/index Workspace Manager open-action validation
-- Verified gameId + mount=game query contract
-- Audited tools/shared/asteroidsPlatformDemo.js boundary ownership
+- Strengthened Workspace Manager game-open test to validate asset presence
+- Added shared palette and expected asset assertions
+- Added Bouncing-ball missing palette regression coverage
 
 PR Details:
-- Supports manifest/tool alignment work
-- No start_of_day changes
+- Prevents page-load-only false positives
 - No validators
+- No start_of_day changes