Rebuild Palette Browser as first session-backed Tool v2 lane - PR 11.188

Author: DavidQ

docs/dev/NEXT_RESTART.md

@@ -1,6 +1,6 @@
-# NEXT RESTART - TOOL V2 MIGRATION
+# SOD 2026-05-01.s1 - PR 11.188 Restart
 
-## Open repo
+## Open Repo
 
 ```powershell
 cd C:\Users\davidq\Documents\GitHub\HTML-JavaScript-Gaming
@@ -14,46 +14,54 @@ git status
 git pull
 ```
 
-## Current Decision
+## Direction Lock
 
-Stop patching legacy Workspace Manager/tool shell code.
+```text
+NO schema changes
+NO sample changes
+NO game changes
+NO old Workspace Manager fixes
+NO legacy tool patches
+NO copying old code
+NEW Tool v2 lane only
+```
 
-Start clean Tool v2 lane:
+## Start PR
 
 ```text
-Palette first
-Session-backed shared data
-One tool at a time
-Reverse engineer, do not copy
-Move old tools to <tool>-v1
-No schema/sample/game changes
+PR_11_188_PALETTE_REVERSE_ENGINEER_AND_REBUILD
 ```
 
-## Supported load paths
+## Added Rules
 
 ```text
-1. Workspace from URL -> load data -> session -> tool
-2. Tool from URL -> load data -> session -> tool
-3. Tool from session loaded by Workspace
+tools/shared/ is deprecated for new Tool v2 work
+Use tools/common/ for new shared Tool v2 foundation
+No code that is just one stream file
+Controls should have focused classes/modules
+Two menus: menuTool and menuWorkspace
+Only controls for the current launch are visible
+Workspace menu updates workspace/session only
+Plan sidebar accordion control types before expanding UI
 ```
 
-## Next PR
+## Do Not Use
 
 ```text
-PR_11_188_PALETTE_V2_REVERSE_ENGINEER_AND_REBUILD
+platformShell
+assetUsageIntegration
+shared handoff
+tool aliases
+fallback data
+default anything
+Workspace ?tool= auto-open
 ```
 
-## Next PR must do
+## Launch Behavior
 
 ```text
-Reverse engineer Palette Browser
-Move tools/Palette Browser -> tools/Palette Browser-v1
-Create clean tools/Palette Browser
-Add/use shared CSS
-Use session context only
-No platformShell
-No assetUsageIntegration
-No shared handoff
-No fallback data
-No schema/sample/game changes
+Workspace opens clean
+User selects Palette Browser / Manager
+Workspace writes session
+Palette Browser reads session via hostContextId
 ```

docs/dev/codex_commands.md

@@ -1,31 +1,49 @@
-# Codex Commands - PR 11.187
+# Codex Commands - PR 11.188
 
-Model: GPT-5 high
+Model: GPT-5.4
 Reasoning: high
 
-Commit/restart documentation only.
+Run one BUILD only:
 
-Do not modify:
-- schemas
-- samples
-- games
-- runtime tool code
+```text
+BUILD_PR_LEVEL_11_188_PALETTE_REVERSE_ENGINEER_AND_REBUILD
+```
 
-Create/keep docs that define the restart lane:
+Use the repo instructions and this bundle as the source of truth.
 
-- Tool v2 migration
-- Palette first
-- session-backed shared data
-- reverse engineer tools, do not copy old code
-- move old tools to `<tool>-v1`
-- support data load paths:
-  1. workspace from URL -> session -> tool
-  2. tool from URL -> session -> tool
-  3. tool from workspace session
+Hard requirements:
 
-Validation:
-- docs only; no runtime validation required
-- full samples smoke skipped because this is commit/restart documentation
+- one PR purpose only
+- no schemas
+- no samples
+- no games
+- no old Workspace Manager fixes
+- no legacy tool patches
+- no `tools/shared/**` changes
+- move `tools/Palette Browser/` to `tools/Palette Browser-v1/`
+- create clean `tools/Palette Browser/`
+- create `tools/common/toolLayout.css`
+- create `tools/common/sessionContext.js`
+- create `tools/common/toolContract.js`
+- reverse engineer first and document findings
+- no fallback/default data
+- no `platformShell`
+- no shared handoff
+- no copied legacy implementation code
+- visible tool name must not include `v2`
+
+Validation required:
+
+```powershell
+node --check "tools/common/sessionContext.js"
+node --check "tools/common/toolContract.js"
+node --check "tools/Palette Browser/main.js"
+```
+
+Do not run full samples smoke unless required by a broad shared sample loader change. Record the skip/run decision and reason.
 
 Return ZIP artifact at:
-`<project folder>/tmp/PR_11_187_20260430_01.zip`
+
+```text
+<project folder>/tmp/PR_11_188_20260501_01.zip
+```

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Document restart for Palette-first Tool v2 migration with session-backed data - PR 11.187
+Rebuild Palette Browser as first session-backed Tool v2 lane - PR 11.188

docs/dev/reports/pr_11_188_bundle_validation.md

@@ -0,0 +1,37 @@
+# PR 11.188 Bundle Validation
+
+## Bundle Source
+This BUILD used `docs/pr/BUILD_PR_LEVEL_11_188_PALETTE_REVERSE_ENGINEER_AND_REBUILD.md` and the PR 11.187 restart lane as the source of truth.
+
+## Scope Confirmation
+Runtime scope was limited to:
+
+- `tools/Palette Browser/**`
+- `tools/Palette Browser-v1/**`
+- `tools/common/**`
+
+Documentation scope was limited to:
+
+- `docs/pr/**`
+- `docs/dev/**`
+- `docs/dev/reports/**`
+
+No schemas, samples, games, `tools/shared/**`, old Workspace Manager fixes, or unrelated legacy tool patches were changed.
+
+## Validation
+Required targeted validation passed:
+
+```powershell
+node --check "tools/common/sessionContext.js"
+node --check "tools/common/toolContract.js"
+node --check "tools/Palette Browser/main.js"
+```
+
+Full samples smoke skipped: targeted Palette Browser rebuild and new `tools/common/` foundation only; no broad shared sample loader change was made.
+
+## Artifact
+Expected ZIP:
+
+```text
+tmp/PR_11_188_20260501_01.zip
+```