Implement tool launch SSoT routing with exact launch labels

PR Details:
- Routes sample-launched tools through explicit tools/<tool>/index.html targets using Open <tool>.
- Routes game workspace flows through tools/Workspace Manager/index.html using Open with Workspace Manager.
- Clears launch memory for external sample/game launches.
- Removes default/fallback behavior from touched launch flow.
- Adds validation report for SSoT routing and launch labels.

Author: DavidQ

docs/dev/codex_rules.md

@@ -10,6 +10,16 @@ Read and obey:
 
 - docs/dev/specs/TOOL_LAUNCH_SSOT.md
 
+## Required UI Labels
+
+Samples:
+- use `Open <tool>` for sample-to-tool launch actions
+- do NOT use `Open with Workspace Manager`
+
+Games:
+- use `Open with Workspace Manager` for game-to-workspace launch actions
+- do NOT use `Open <tool>`
+
 ## This PR
 
 Allowed:
@@ -25,6 +35,7 @@ Forbidden:
 - implementation outside touched launch path
 - start_of_day changes
 - roadmap text rewrite
+- changing required UI label semantics
 
 ## Anti-Patterns Forbidden
 

docs/dev/reports/tool_launch_ssot_routing_validation.md

@@ -2,28 +2,31 @@
 
 ## Changed Files
 
-- `tools/shared/toolLaunchSSoT.js`
 - `games/index.render.js`
 - `docs/dev/reports/tool_launch_ssot_routing_validation.md`
 
 ## Tested UAT Paths
 
-1. Samples hub roundtrip launch href resolution via `resolveSampleToolLaunchHref(...)`.
-2. Games hub workspace launch href resolution via `resolveGameWorkspaceLaunchHref(...)`.
-3. External launch memory clearing via `clearExternalToolWorkspaceMemory()` and `launchWithExternalToolWorkspaceReset(...)`.
-4. Missing launch context handling for:
-   - sample launch missing `sampleId`
-   - games metadata entries missing valid `href`
+1. Samples hub: sample-to-tool action label and launch href generation.
+2. Games hub: game-to-workspace action label and launch href generation.
+3. External launch flow: storage reset before navigation.
+4. Missing launch context behavior: sample missing `sampleId`; game entries missing valid `href`.
 
-## Proof: Samples Route To `tools/<tool>/index.html`
+## Proof Sample Actions Are Labeled `Open <tool>`
 
-Command run:
+Code proof:
 
-```bash
-node --input-type=module -
+- `samples/index.render.js:102`
+
+```text
+const label = `Open ${normalize(tool.displayName) || normalize(tool.name) || toolId}`;
 ```
 
-Script output:
+This preserves required sample launch semantics and does not use `Open with Workspace Manager` for sample-to-tool actions.
+
+## Proof Samples Route To `tools/<tool>/index.html`
+
+Command output:
 
 ```text
 sampleLaunch {
@@ -32,21 +35,30 @@ sampleLaunch {
 }
 ```
 
-Implementation references:
+Code proof:
 
 - `tools/shared/toolLaunchSSoT.js:75`
-- `tools/shared/toolLaunchSSoT.js:91`
+- `tools/shared/toolLaunchSSoT.js:86`
 - `samples/index.render.js:92`
+- `samples/index.render.js:345`
 
-## Proof: Games Route To `tools/Workspace Manager/index.html`
+## Proof Game Actions Are Labeled `Open with Workspace Manager`
 
-Command run:
+Code proof:
 
-```bash
-node --input-type=module -
+- `games/index.render.js:260`
+
+```text
+Open with Workspace Manager
 ```
 
-Script output:
+Additional V2 hardening:
+
+- Removed unlabeled game launch triggers (`game-title-link`, card-click launch) so workspace launch is exposed through the required labeled action path.
+
+## Proof Games Route To `tools/Workspace Manager/index.html`
+
+Command output:
 
 ```text
 gameLaunch {
@@ -55,59 +67,58 @@ gameLaunch {
 }
 ```
 
-Implementation references:
+Code proof:
 
 - `tools/shared/toolLaunchSSoT.js:105`
 - `tools/shared/toolLaunchSSoT.js:117`
 - `tools/shared/toolLaunchSSoT.js:118`
 - `games/index.render.js:147`
+- `games/index.render.js:407`
 
-## Proof: External Launch Memory Is Cleared
+## Proof External Launch Memory Is Cleared
 
-Command run:
-
-```bash
-node --input-type=module -
-```
-
-Script output:
+Command output:
 
 ```text
 clearResult true
 localAfterClear [ [ 'keep.one', 'x' ] ]
 sessionAfterClear [ [ 'keep.two', 'y' ] ]
 launchResult true
-assignCalls [ '/tools/Sprite%20Editor/index.html?sampleId=1208' ]
+assignCalls [ '/tools/Workspace%20Manager/index.html?gameId=2001&mount=game' ]
 localAfterLaunch [ [ 'keep.one', 'x' ] ]
 sessionAfterLaunch [ [ 'keep.two', 'y' ] ]
 ```
 
-Implementation references:
+Code proof:
 
 - `tools/shared/toolLaunchSSoT.js:123`
 - `tools/shared/toolLaunchSSoT.js:140`
 - `samples/index.render.js:537`
-- `games/index.render.js:424`
-- `games/index.render.js:444`
+- `games/index.render.js:416`
 
-## Proof: Missing Context Does Not Fallback
+## Proof Missing Context Does Not Fallback
 
-1. Sample launch now requires `sampleId`; missing context returns explicit error and empty href:
+1. Missing sample launch context is explicit and non-routable:
 
 ```text
 missingSampleContext { href: '', error: 'Tool "sprite-editor" launch is missing sampleId.' }
 ```
 
-2. Games rows missing valid game `href` now emit explicit visible launch errors instead of silent empty launch state:
+2. Missing game launch context is visible in games UI (`Workspace launch error`) instead of silently falling back:
 
 - `games/index.render.js:147`
-- `games/index.render.js:271`
-- `games/index.render.js:336`
+- `games/index.render.js:264`
+- `games/index.render.js:330`
 
-3. Workspace launch links are only bound when explicit `workspaceHref` exists, so missing context cannot route through a fallback launcher:
+Metadata scan used during validation:
 
-- `games/index.render.js:266`
-- `games/index.render.js:416`
+```text
+{ totalGames: 29, missingValidHref: 18 }
+```
+
+3. No hidden alternate game workspace launch action remains in touched flow:
+
+- `games/index.render.js` contains no `game-title-link`, `handleCardLaunch`, or `data-workspace-href` launch path.
 
 ## Static Validation
 
@@ -123,8 +134,8 @@ All commands exited successfully.
 
 ## Anti-Pattern Self-Check
 
-- No new duplicate launch state introduced.
-- No new hidden/default/fallback launch route added in touched flow.
-- No duplicate event listeners added.
-- No new globals/managers/factories introduced.
-- No unrelated files changed for behavior.
+- No duplicate launch state introduced.
+- No hidden/default/fallback route/tool/workspace behavior added in touched flow.
+- No duplicate event listeners introduced.
+- No globals/managers/factories/service layers added.
+- No unrelated cleanup or scope expansion.

docs/operations/dev/codex_commands.md

@@ -1,4 +1,4 @@
-# Codex Commands — BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING
+# Codex Commands — BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING_V2
 
 ## Model
 GPT-5.4 or GPT-5.3-codex
@@ -12,14 +12,16 @@ High
 Read docs/dev/codex_rules.md first.
 Read docs/dev/specs/TOOL_LAUNCH_SSOT.md second.
 
-Execute BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING.
+Execute BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING_V2.
 
 Goal:
-Implement the first runtime slice of TOOL_LAUNCH_SSOT.
+Implement the first runtime slice of TOOL_LAUNCH_SSOT with exact UI launch wording.
 
 Required behavior:
 - samples launch tools through tools/<tool>/index.html
+- sample launch actions must be labeled: Open <tool>
 - games launch workspace flows through tools/Workspace Manager/index.html
+- game launch actions must be labeled: Open with Workspace Manager
 - external launches from samples/games clear launch memory before loading
 - launch data comes from the existing single source of truth
 - no default/fallback route/tool/workspace behavior in touched launch flow
@@ -32,17 +34,20 @@ Hard constraints:
 - no start_of_day changes
 - no roadmap text rewrite except status markers
 - no anti-patterns listed in docs/dev/codex_rules.md
+- do not alter the required label meanings
 
 Validation:
 Create docs/dev/reports/tool_launch_ssot_routing_validation.md with:
 - changed files
 - tested UAT paths
+- proof sample actions are labeled Open <tool>
 - proof samples route to tools/<tool>/index.html
+- proof game actions are labeled Open with Workspace Manager
 - proof games route to tools/Workspace Manager/index.html
 - proof external launch memory is cleared
 - proof missing context does not fallback
 - anti-pattern self-check
 
 Return ZIP at:
-tmp/BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING.zip
+tmp/BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING_V2.zip
 ```

docs/operations/dev/commit_comment.txt

@@ -1,8 +1,8 @@
-Implement tool launch SSoT routing
+Implement tool launch SSoT routing with exact launch labels
 
 PR Details:
-- Routes sample-launched tools through explicit tools/<tool>/index.html targets.
-- Routes game workspace flows through tools/Workspace Manager/index.html.
+- Routes sample-launched tools through explicit tools/<tool>/index.html targets using Open <tool>.
+- Routes game workspace flows through tools/Workspace Manager/index.html using Open with Workspace Manager.
 - Clears launch memory for external sample/game launches.
 - Removes default/fallback behavior from touched launch flow.
-- Adds validation report for SSoT routing.
+- Adds validation report for SSoT routing and launch labels.

docs/pr/BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING_V2.md

@@ -0,0 +1,114 @@
+# BUILD_PR_LEVEL_20_8_IMPLEMENT_TOOL_LAUNCH_SSOT_ROUTING_V2
+
+## Purpose
+
+Implement the first runtime slice of the launch SSoT contract with exact UI launch wording:
+
+- sample-launched tools use `Open <tool>`
+- game-launched Workspace Manager flows use `Open with Workspace Manager`
+
+## Scope
+
+One PR purpose only:
+
+- route sample-launched tools through explicit `tools/<tool>/index.html` targets via `Open <tool>`
+- route game-launched workspace flows through `tools/Workspace Manager/index.html` via `Open with Workspace Manager`
+- clear launch memory for external launches from samples or games
+- remove default/fallback launch behavior only where touched by this launch path
+
+## Required Source of Truth
+
+Codex must use:
+
+- `docs/dev/specs/TOOL_LAUNCH_SSOT.md`
+- existing launch metadata/config files already present in the repo
+
+Codex must not invent a second source of truth.
+
+## Required UI Contract
+
+### Samples UI
+
+Samples must expose tool launch actions as:
+
+```text
+Open <tool>
+```
+
+Examples:
+
+```text
+Open Vector Map Editor
+Open Vector Asset Studio
+Open Tilemap Studio
+```
+
+Samples must not use `Open with Workspace Manager` for sample-to-tool launch actions.
+
+### Games UI
+
+Games must expose Workspace Manager launch actions as:
+
+```text
+Open with Workspace Manager
+```
+
+Games must not use `Open <tool>` for game-to-workspace launch actions.
+
+## Hard Constraints
+
+Codex must NOT:
+
+- rewrite unrelated tools
+- rewrite unrelated games
+- rewrite unrelated samples
+- change route names unless required by the SSoT
+- change the required launch labels
+- introduce default tool fallback
+- introduce default workspace fallback
+- silently redirect missing launch context
+- preserve stale memory on external launches
+- add managers/factories/service layers unless already part of the existing pattern
+- modify `start_of_day`
+- rewrite roadmap text except status markers
+
+## Anti-Pattern Guards
+
+Do not introduce:
+
+- alias variables
+- pass-through variables
+- duplicate state
+- stored derived state
+- vague names
+- magic route strings outside the SSoT path
+- duplicate event listeners
+- broad truthy/falsy checks that change behavior
+- hidden fallback behavior
+
+## Required Validation
+
+Codex must document validation in:
+
+- `docs/dev/reports/tool_launch_ssot_routing_validation.md`
+
+Validation must include:
+
+- sample tile/action labels use `Open <tool>`
+- sample tile/action launches resolve to explicit `tools/<tool>/index.html`
+- game tile/action labels use `Open with Workspace Manager`
+- game tile/action launches resolve to `tools/Workspace Manager/index.html`
+- external launch memory is cleared before loading
+- missing launch data fails visibly
+- no default/fallback path remains in touched launch flow
+- no anti-patterns introduced in touched files
+
+## Acceptance
+
+- Runtime launch behavior follows `TOOL_LAUNCH_SSOT.md`.
+- Sample launch UI uses `Open <tool>`.
+- Game launch UI uses `Open with Workspace Manager`.
+- SSoT remains the authority.
+- UAT path from `games/index.html` to Workspace Manager works.
+- UAT path from `samples/index.html` to tool index works.
+- No unrelated implementation changes.