BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP

- Added strict manifest payload expectation tests
- Removed legacy root sections from game manifests
- Removed invalid unused Asteroids tool sections
- Ensured Asteroids vector data belongs to Vector Asset Studio

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

Author: DavidQ

docs/dev/reports/expected_delta_template.md

@@ -1,21 +1,29 @@
 # Expected Codex Return / Delta Template
 
 ## Expected Changed Files
-- Workspace Manager loader/binding file(s)
-- `tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs` only if needed
-- `docs/dev/reports/level_10_2b_workspace_manager_palette_binding_report.md`
+- `games/*/game.manifest.json`
+- `tests/runtime/GameManifestPayloadExpectations.test.mjs` or equivalent
+- `tests/runtime/GamesIndexWorkspaceManagerOpen.test.mjs` if asset expectation checks are extended
+- `docs/dev/reports/level_10_2c_manifest_payload_expectation_report.md`
+- `docs/dev/reports/level_10_2c_manifest_cleanup_report.md`
 - `docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md` if status update needed
 
 ## Expected Validation Summary
-- `workspace_manager_reads_palette_browser_palette=true`
-- `bouncing_ball_shared_palette_present=true`
-- `games_with_manifest_palette_bound=<count>`
-- `games_still_missing_palette=<count>`
-- `direct_launch_regression=false`
-- `validators_added=0`
+- `games_scanned=<count>`
+- `root_lineage_remaining=0`
+- `root_sources_remaining=0`
+- `root_assets_remaining=0`
+- `sourcePath_remaining=0`
+- `legacy_json_path_refs_remaining=0`
+- `tool_sections_missing_metadata=0`
+- `asteroids_vector_count_gt_zero=true`
+- `asteroids_invalid_sprite_editor_present=false`
+- `asteroids_invalid_tile_map_editor_present=false`
+- `asteroids_invalid_parallax_editor_present=false`
+- `workspace_manager_vector_asset_none_when_vectors_exist=false`
 - `start_of_day_changes=0`
 
 ## Expected Delta ZIP
 Codex must create:
 
-`tmp/BUILD_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX_delta.zip`
+`tmp/BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP_delta.zip`

docs/dev/reports/launch_smoke_report.md

@@ -1,6 +1,6 @@
 # Launch Smoke Report
 
-Generated: 2026-04-27T01:16:49.548Z
+Generated: 2026-04-27T01:43:12.462Z
 
 Filters: games=true, samples=false, tools=false, sampleRange=all
 

docs/dev/reports/level_10_2c_manifest_cleanup_report.md

@@ -0,0 +1,49 @@
+# Level 10.2C Manifest Cleanup Report
+
+## BUILD
+- `BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP`
+
+## Cleanup Scope Applied
+- removed disallowed root manifest blocks: `lineage`, `sources`, `assets`
+- removed `sourcePath` fields from all game manifests
+- removed legacy catalog path references to `workspace.asset-catalog.json` and `tools.manifest.json`
+- preserved singleton palette at `tools["palette-browser"].palette`
+- kept tool metadata fields (`schema`, `version`, `name`, `source`) on all tool sections
+
+## Cleaned Game Manifests
+- `games/AITargetDummy/game.manifest.json`
+- `games/Asteroids/game.manifest.json`
+- `games/Bouncing-ball/game.manifest.json`
+- `games/Breakout/game.manifest.json`
+- `games/GravityWell/game.manifest.json`
+- `games/Pacman/game.manifest.json`
+- `games/Pong/game.manifest.json`
+- `games/SolarSystem/game.manifest.json`
+- `games/SpaceDuel/game.manifest.json`
+- `games/SpaceInvaders/game.manifest.json`
+- `games/vector-arcade-sample/game.manifest.json`
+- `games/_template/game.manifest.json`
+
+## Asteroids Specific
+- kept `tools["vector-asset-studio"].vectors` with actual vector payloads
+- removed `tools["sprite-editor"]`
+- removed `tools["tile-map-editor"]`
+- removed `tools["parallax-editor"]`
+- removed `tools["vector-asset-studio"].libraries` (reference/index-only metadata)
+
+## Bouncing-ball Specific
+- removed root `lineage`, `sources`, `assets`
+- kept `tools["palette-browser"].palette`
+- kept `tools["primitive-skin-editor"].skins`
+- removed stale external palette/skin JSON references from manifest payload
+
+## Post-Cleanup Validation
+- `npm run test:manifest-payload:games` -> `PASS`
+- `npm run test:workspace-manager:games` -> `PASS`
+- `npm run test:launch-smoke:games` -> `PASS`
+
+## Summary
+- manifests cleaned: `12`
+- disallowed root blocks remaining: `0`
+- `sourcePath` keys remaining: `0`
+- legacy catalog path references remaining: `0`

docs/dev/reports/level_10_2c_manifest_payload_expectation_report.md

@@ -0,0 +1,40 @@
+# Level 10.2C Manifest Payload Expectation Report
+
+## BUILD
+- `BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP`
+
+## Strict Test Added
+- `tests/runtime/GameManifestPayloadExpectations.test.mjs`
+- npm command: `npm run test:manifest-payload:games`
+
+## Test Contract (Enforced)
+- no root `lineage`, `sources`, `assets`, `palette`, `palettes`
+- no `sourcePath` keys anywhere in game manifests
+- no legacy `workspace.asset-catalog.json`/`tools.manifest.json` references
+- every `tools[*]` section includes `schema`, `version`, `name`, `source`
+- singleton palette is only `tools["palette-browser"].palette`
+- Asteroids requires `tools["vector-asset-studio"].vectors` with count > 0 and disallows `sprite-editor`, `tile-map-editor`, `parallax-editor`, `libraries`
+- Bouncing-ball requires `tools["palette-browser"].palette` + `tools["primitive-skin-editor"].skins` and no stale external palette/skin JSON references
+
+## Validation Results
+- command: `npm run test:manifest-payload:games`
+- result: `PASS`
+- manifests checked: `12`
+- failures: `0`
+
+## Asteroids Assertions
+- vector asset count: `5`
+- disallowed Asteroids tool sections removed: `sprite-editor`, `tile-map-editor`, `parallax-editor`
+- vector `libraries` reference/index block removed
+
+## Workspace Manager Runtime Cross-Check
+- command: `npm run test:workspace-manager:games`
+- result: `PASS`
+- Asteroids shared asset label: `vector.asteroids.ship`
+- Bouncing-ball shared palette label: `Bouncing Ball Palette`
+- Bouncing-ball shared asset label: `Bouncing Ball Classic Skin`
+
+## Acceptance Mapping
+- bad payload shape is test-covered: `yes`
+- Asteroids vector tool has assets: `yes`
+- invalid unused Asteroids tool sections removed: `yes`

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

@@ -901,4 +901,5 @@
 - [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.
+- [x] Manifest payload expectation test + cleanup completed (no root legacy manifest blocks, no `sourcePath`, palette singleton enforced, Asteroids tool payloads pruned to active data domains).
 - [x] Workspace Manager shared palette binding now hydrates from `game.manifest.json` `tools["palette-browser"].palette` (with root `palette` compatibility fallback only).

docs/operations/dev/codex_commands.md

@@ -2,32 +2,51 @@ MODEL: GPT-5.3-codex
 REASONING: high
 
 TASK:
-Apply BUILD_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX.
+Apply BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP.
 
 STEPS:
-1. Read docs/pr/PLAN_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX.md.
-2. Read docs/pr/BUILD_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX.md.
-3. Inspect Workspace Manager game manifest loading/binding code.
-4. Find where shared palette is selected/displayed.
-5. Bind shared palette from:
-   - gameManifest.tools["palette-browser"].palette
-6. Allow temporary fallback from root `palette` only for compatibility if present.
-7. Do not create root palette objects.
-8. Verify Bouncing-ball no longer shows:
-   - Shared Palette: No shared palette selected
-9. Run/update Level 10.2A Workspace Manager asset presence test.
-10. Ensure direct game launch remains unchanged.
-11. Write docs/dev/reports/level_10_2b_workspace_manager_palette_binding_report.md.
-12. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only if needed:
+1. Read docs/pr/PLAN_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP.md.
+2. Read docs/pr/BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP.md.
+3. Add a strict manifest payload expectation test.
+4. Clean all game manifests:
+   - remove root lineage
+   - remove root sources
+   - remove generic root assets
+   - remove sourcePath fields
+   - remove legacy JSON path references
+5. Ensure palette is exactly:
+   - tools["palette-browser"].palette
+6. Ensure every tool section has metadata:
+   - schema
+   - version
+   - name
+   - source
+7. Asteroids:
+   - keep vector-asset-studio with actual vectors
+   - remove sprite-editor if no actual sprite data
+   - remove tile-map-editor if no actual tilemap data
+   - remove parallax-editor if no actual parallax data
+   - remove libraries if it is only reference/index metadata
+8. Bouncing-ball:
+   - remove lineage/sources/assets
+   - keep palette-browser.palette
+   - keep primitive-skin-editor.skins
+9. Update Workspace Manager test if needed so it validates real payload sections, not shell load only.
+10. Write reports:
+   - docs/dev/reports/level_10_2c_manifest_payload_expectation_report.md
+   - docs/dev/reports/level_10_2c_manifest_cleanup_report.md
+11. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only if needed:
     - [ ] -> [.]
     - [.] -> [x]
     - no prose rewrite/delete
-13. Do not add validators.
-14. Do not modify start_of_day.
-15. Create Codex delta ZIP:
-    tmp/BUILD_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX_delta.zip
+12. Do not add validators.
+13. Do not modify start_of_day.
+14. Create Codex delta ZIP:
+    tmp/BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP_delta.zip
 
 ACCEPTANCE:
-- shared palette binds from palette-browser singleton palette
-- Bouncing-ball palette appears
+- bad payload shape is test-covered
+- manifests cleaned
+- Asteroids vector tool has assets
+- invalid unused tool sections removed
 - delta ZIP exists

docs/operations/dev/commit_comment.txt

@@ -1,10 +1,11 @@
-BUILD_PR_LEVEL_10_2B_WORKSPACE_MANAGER_PALETTE_BINDING_FIX
+BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP
 
-- Bound Workspace Manager shared palette from game manifest palette-browser singleton
-- Fixed missing shared palette display for game workspaces
-- Preserved direct game launch behavior
+- Added strict manifest payload expectation tests
+- Removed legacy root sections from game manifests
+- Removed invalid unused Asteroids tool sections
+- Ensured Asteroids vector data belongs to Vector Asset Studio
 
 PR Details:
-- Addresses Bouncing-ball missing palette regression
+- Fixes page-load-only false positives
 - No validators
 - No start_of_day changes

docs/pr/BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP.md

@@ -0,0 +1,145 @@
+# BUILD_PR_LEVEL_10_2C_MANIFEST_PAYLOAD_EXPECTATION_TESTS_AND_CLEANUP
+
+## Objective
+Fix the gap exposed by review: tests validate load/open state, but not expected manifest payload.
+
+## Current Bad Example
+Bouncing-ball manifest contains:
+- `lineage`
+- `sources`
+- `assets`
+- `sourcePath`
+- references to old JSON
+- copied legacy catalog data
+
+These should not be present in the final clean manifest model.
+
+## Direct Answers / Final Rules
+
+### 1. `lineage`
+Delete it from game manifests.
+
+Reason:
+- It is migration history/debug metadata.
+- It is not runtime/tool data.
+- It creates legacy dependency noise.
+
+### 2. `sources`
+Delete it from game manifests.
+
+Reason:
+- It points to old external JSON sources.
+- Final manifest should not be a reference map.
+
+### 3. `assets`
+Delete generic root `assets`.
+
+Reason:
+- It is a copied catalog/index.
+- It duplicates tool-owned data.
+- External media file references belong only under `tools["asset-browser"].assets`.
+
+### 4. Asteroids vector data
+Asteroids ships/asteroids/title vectors must be actual Vector Asset Studio data under:
+
+```json
+tools["vector-asset-studio"].vectors
+```
+
+Workspace Manager should show Vector Asset Studio with a selected/available asset, not `Asset: none`.
+
+### 5. `sprite-editor`
+Remove from Asteroids if Asteroids has only vector assets.
+
+### 6. `tile-map-editor`
+Remove from Asteroids if Asteroids does not use tile maps.
+
+### 7. `parallax-editor`
+Remove from Asteroids if Asteroids does not use parallax.
+
+### 8. `libraries`
+Delete if it is only an index/reference list.
+
+Keep only if it contains actual tool-owned reusable data that the tool requires. For Asteroids vector data, prefer actual entries in `vectors`, not a separate reference library.
+
+## Clean Manifest Model
+
+Allowed root fields:
+- `schema`
+- `version`
+- `game`
+- `launch` only if still intentionally used
+- `tools`
+
+Disallowed root fields:
+- `lineage`
+- `sources`
+- generic `assets`
+- copied legacy catalog data
+- root `palette`
+- root `palettes`
+
+Allowed tool sections only when valid:
+- `palette-browser`
+- `primitive-skin-editor`
+- `asset-browser`
+- `vector-asset-studio`
+- `sprite-editor` only for sprite games/data
+- `tile-map-editor` only for tile games/data
+- `parallax-editor` only for parallax games/data
+
+## Required Test
+Add or update a manifest payload expectation test.
+
+Suggested file:
+
+```text
+tests/runtime/GameManifestPayloadExpectations.test.mjs
+```
+
+The test must validate:
+
+### For all games
+- no root `lineage`
+- no root `sources`
+- no generic root `assets`
+- no `sourcePath`
+- no stale external JSON paths
+- no old `workspace.asset-catalog.json`
+- no old `tools.manifest.json`
+- every tool section has `schema`, `version`, `name`, `source`
+- exactly one palette at `tools["palette-browser"].palette`
+- no root `palette`
+- no root `palettes`
+
+### For Asteroids
+- has `tools["vector-asset-studio"].vectors`
+- vector count > 0
+- does not have `sprite-editor` unless actual sprite data exists
+- does not have `tile-map-editor` unless actual tilemap data exists
+- does not have `parallax-editor` unless actual parallax data exists
+- does not have `libraries` unless actual non-reference library data is required
+- Workspace Manager must not show Vector Asset Studio `Asset: none` if vectors exist
+
+### For Bouncing-ball
+- no lineage/sources/assets
+- has `palette-browser.palette`
+- has `primitive-skin-editor.skins`
+- no stale references to external palette/skin JSON files
+
+## Required Cleanup
+Clean all game manifests to satisfy the above test.
+
+## Required Reports
+Create:
+- `docs/dev/reports/level_10_2c_manifest_payload_expectation_report.md`
+- `docs/dev/reports/level_10_2c_manifest_cleanup_report.md`
+
+## Acceptance
+- Tests fail on the old bad manifest shape.
+- Tests pass after cleanup.
+- Bouncing-ball has no `lineage`, `sources`, or generic `assets`.
+- Asteroids has Vector Asset Studio vectors and no invalid unused tool sections.
+- Workspace Manager asset presence test validates actual payload expectations.
+- No validators added.
+- No start_of_day changes.