BUILD_PR_LEVEL_10_1B_TOOL_SECTION_METADATA_AND_PALETTE_TOOL_SINGLETON

- Moved singleton palette into Palette Browser tool section
- Removed root-level palette ownership
- Added required metadata to tool sections
- Kept Primitive Skin Editor focused on skins only

PR Details:
- Corrects tool ownership model
- No palette JSON files
- No validators
- No start_of_day changes

BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT

- Added games/index Workspace Manager open-action validation
- Verified gameId + mount=game query contract
- Audited tools/shared/asteroidsPlatformDemo.js boundary ownership

PR Details:
- Supports manifest/tool alignment work
- No start_of_day changes
- No validators

Author: DavidQ

docs/dev/reports/expected_delta_template.md

@@ -1,23 +1,25 @@
 # Expected Codex Return / Delta Template
 
-## Expected Changed Files
-- `games/*/game.manifest.json`
-- `docs/dev/reports/level_10_1a_palette_singleton_correction_report.md`
+## Expected Changed/Added 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/roadmaps/MASTER_ROADMAP_ENGINE.md` if status update needed
 
 ## Expected Validation Summary
-- `games_scanned=<count>`
-- `games_with_root_palette=<count>`
-- `tool_owned_palettes_remaining=0`
-- `root_palettes_collections_remaining=0`
-- `palette_json_files_created=0`
-- `duplicate_palette_swatches_removed=<count>`
-- `symbol_conflicts_fixed=<count>`
-- `opaque_alpha_ff_remaining=0`
-- `validators_added=0`
+- `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`
 - `start_of_day_changes=0`
 
 ## Expected Delta ZIP
 Codex must create:
 
-`tmp/BUILD_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION_delta.zip`
+`tmp/BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT_delta.zip`

docs/dev/reports/level_10_1b_tool_section_metadata_and_palette_tool_singleton_report.md

@@ -0,0 +1,67 @@
+# Level 10.1B Tool Section Metadata And Palette Tool Singleton Report
+
+## BUILD
+- `BUILD_PR_LEVEL_10_1B_TOOL_SECTION_METADATA_AND_PALETTE_TOOL_SINGLETON`
+
+## Scope Completed
+- Scanned all `games/*/game.manifest.json`.
+- Moved singleton palette ownership to `tools["palette-browser"].palette`.
+- Removed root `palette` and root `palettes`.
+- Removed all `tools.*.palettes` collections and non-`palette-browser` `tools.*.palette` duplicates.
+- Added required metadata to every existing tool section:
+  - `schema`
+  - `version`
+  - `name`
+  - `source`
+- Ensured `primitive-skin-editor` exists with metadata and `skins: []`.
+- Ensured `palette-browser` exists with metadata and singleton `palette`.
+
+## Read-Path Adjustment
+- Updated old Asteroids manifest fragment references in:
+  - `tools/shared/asteroidsPlatformDemo.js`
+- Old reference:
+  - `games/Asteroids/game.manifest.json#palette`
+- New reference:
+  - `games/Asteroids/game.manifest.json#tools.palette-browser.palette`
+
+## Structural Validation
+- Validation result: `issues=0` for all launchable game manifests.
+- Checks performed:
+  - No root `palette`.
+  - No root `palettes`.
+  - Exactly one `tools["palette-browser"].palette` per game manifest.
+  - No remaining `tools.*.palettes`.
+  - No non-`palette-browser` `tools.*.palette`.
+  - `primitive-skin-editor` has metadata and `skins` only.
+  - All tool sections include metadata (`schema/version/name/source`).
+  - Palette swatches normalized:
+    - uppercase `#RRGGBB`
+    - no opaque `FF`
+    - single-character symbols
+
+## Runtime Validation
+- Executed: `npm run test:launch-smoke:games`
+- Result:
+  - `filters: games=true samples=false tools=false`
+  - `PASS=12 FAIL=0 TOTAL=12`
+
+## Game Summary
+| Game | Palette Location | Primitive Skin Editor | Tool Metadata Coverage |
+| --- | --- | --- | --- |
+| `_template` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `AITargetDummy` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `Asteroids` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `Bouncing-ball` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `Breakout` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `GravityWell` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `Pacman` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `Pong` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `SolarSystem` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `SpaceDuel` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `SpaceInvaders` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+| `vector-arcade-sample` | `tools.palette-browser.palette` | metadata + `skins: []` | complete |
+
+## Guardrails
+- No new palette JSON files were created.
+- No validators were added.
+- No `start_of_day` files were modified.

docs/dev/reports/level_10_2_asteroids_platform_demo_boundary_audit.md

@@ -0,0 +1,8 @@
+# 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.

docs/dev/reports/level_10_2_workspace_manager_open_test_report.md

@@ -0,0 +1,9 @@
+# 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.

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

@@ -898,3 +898,4 @@
 - [x] Game palette completeness is normalized with manifest-owned palette data under `game.manifest.json` tool sections.
 - [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).

docs/operations/dev/codex_commands.md

@@ -2,37 +2,39 @@ MODEL: GPT-5.3-codex
 REASONING: high
 
 TASK:
-Apply BUILD_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION.
+Apply BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT.
 
 STEPS:
-1. Read docs/pr/PLAN_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION.md.
-2. Read docs/pr/BUILD_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION.md.
-3. Scan all `games/*/game.manifest.json`.
-4. For each manifest:
-   - find any `tools.*.palettes`
-   - move/merge palette data into root `palette`
-   - remove tool-owned `palettes`
-   - ensure only one root `palette` exists
-5. Normalize swatches:
-   - uppercase hex
-   - remove opaque `FF`
-   - single-character symbols
-   - dedupe by hex where safe
-6. Ensure Primitive Skin Editor owns skins only, not palette.
-7. Ensure Palette Browser does not own a duplicate palette object.
-8. Update any game/tool manifest readers only if required to read root `palette`.
-9. Do not create palette JSON files.
-10. Write docs/dev/reports/level_10_1a_palette_singleton_correction_report.md.
-11. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md status only if needed:
-    - [ ] -> [.]
-    - [.] -> [x]
-    - no prose rewrite/delete
-12. Do not add validators.
-13. Do not modify start_of_day.
-14. Create Codex delta ZIP:
-    tmp/BUILD_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION_delta.zip
+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.
+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
 
 ACCEPTANCE:
-- exactly one root `palette` per game manifest
-- no tool-owned palettes remain
+- games index Workspace Manager action test exists
+- all game actions use gameId + mount=game
+- boundary audit exists
 - delta ZIP exists

docs/operations/dev/commit_comment.txt

@@ -1,12 +1,10 @@
-BUILD_PR_LEVEL_10_1A_PALETTE_STANDALONE_SINGLETON_CORRECTION
+BUILD_PR_LEVEL_10_2_WORKSPACE_MANAGER_OPEN_TEST_AND_SHARED_BOUNDARY_AUDIT
 
-- Moved game palettes out of Primitive Skin Editor
-- Enforced one root palette per game/workspace
-- Removed tool-owned palette duplication
+- Added games/index Workspace Manager open-action validation
+- Verified gameId + mount=game query contract
+- Audited tools/shared/asteroidsPlatformDemo.js boundary ownership
 
 PR Details:
-- Palette is standalone shared data
-- Tools consume the shared root palette
-- No palette JSON files created
-- No validators
+- Supports manifest/tool alignment work
 - No start_of_day changes
+- No validators

docs/pr/BUILD_PR_LEVEL_10_1B_TOOL_SECTION_METADATA_AND_PALETTE_TOOL_SINGLETON.md

@@ -0,0 +1,130 @@
+# BUILD_PR_LEVEL_10_1B_TOOL_SECTION_METADATA_AND_PALETTE_TOOL_SINGLETON
+
+## Problem
+The previous correction moved palette to root:
+
+```json
+"palette": {
+  "schema": "html-js-gaming.palette",
+  "version": 1,
+  "name": "_template Palette",
+  "source": "manifest-and-runtime-color-scan",
+  "swatches": []
+}
+```
+
+That is wrong because palette is a first-class tool/domain.
+
+Also, Primitive Skin Editor sections were missing standard metadata:
+
+```json
+"primitive-skin-editor": {
+  "skins": []
+}
+```
+
+## Correct Shape
+
+```json
+{
+  "tools": {
+    "palette-browser": {
+      "schema": "html-js-gaming.tool.palette-browser",
+      "version": 1,
+      "name": "Palette Browser",
+      "source": "manifest",
+      "palette": {
+        "schema": "html-js-gaming.palette",
+        "version": 1,
+        "name": "Game Palette",
+        "source": "manifest-and-runtime-color-scan",
+        "swatches": []
+      }
+    },
+
+    "primitive-skin-editor": {
+      "schema": "html-js-gaming.tool.primitive-skin-editor",
+      "version": 1,
+      "name": "Primitive Skin Editor",
+      "source": "manifest",
+      "skins": []
+    }
+  }
+}
+```
+
+## Rules
+
+### Palette
+- Palette lives under `tools["palette-browser"].palette`.
+- Exactly one palette per game/workspace.
+- No root `palette`.
+- No root `palettes`.
+- No `tools.*.palettes` collection.
+- No duplicate palettes under other tools.
+
+### Primitive Skin Editor
+Primitive Skin Editor owns:
+- skins
+- HUD skins
+- primitive style groups
+- mappings to palette symbols
+
+Primitive Skin Editor does NOT own:
+- palette swatches
+- palette definitions
+- palette collections
+
+### Tool Section Metadata
+Every first-class tool section under `tools` must have:
+
+```json
+{
+  "schema": "html-js-gaming.tool.<tool-id>",
+  "version": 1,
+  "name": "<Display Name>",
+  "source": "manifest"
+}
+```
+
+Tool-specific data is added beside those fields.
+
+## Required Migration
+For every `games/*/game.manifest.json`:
+
+1. If root `palette` exists:
+   - move it to `tools["palette-browser"].palette`
+   - remove root `palette`
+
+2. If `tools["primitive-skin-editor"].palettes` exists:
+   - move/merge into `tools["palette-browser"].palette`
+   - remove `tools["primitive-skin-editor"].palettes`
+
+3. Ensure `tools["primitive-skin-editor"]` has:
+   - schema
+   - version
+   - name
+   - source
+
+4. Ensure `tools["palette-browser"]` has:
+   - schema
+   - version
+   - name
+   - source
+   - palette
+
+5. Normalize palette:
+   - one palette only
+   - swatches use single-character symbols
+   - hex uppercase
+   - no opaque alpha `FF`
+
+## Acceptance
+- No root `palette`.
+- No root `palettes`.
+- Exactly one `tools["palette-browser"].palette` per game.
+- Primitive Skin Editor has metadata and no palette ownership.
+- Every existing tool section has schema/version/name/source.
+- No new palette JSON files.
+- No validators.
+- No `start_of_day` changes.