PR 8.18: Launcher pairing contract

- Defined openTool(toolPayload, dataObjects)
- Advanced roadmap status

Author: DavidQ

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md

@@ -873,8 +873,8 @@
 - [x] Canonical 17 first-class tool inventory documented.
 - [x] Tool folder paths verified for all 17 first-class tools.
 - [x] Explicit schema paths recorded for all 17 first-class tools.
-- [.] Close first-class schema gap by adding `tools/schemas/tools/skin-editor.schema.json`.
-- [.] Define launcher pairing contract: pass tool payload object plus shared data objects.
+- [x] Close first-class schema gap by adding `tools/schemas/tools/skin-editor.schema.json`.
+- [x] Define launcher pairing contract: pass tool payload object plus shared data objects.
 - [ ] Verify all 17 tools have exactly one schema.
 - [ ] Verify sample tool payload files match workspace/game `tools[]` item shape.
 - [ ] Verify palette/shared data never embeds inside tool payload files.

docs/dev/roadmaps/MASTER_ROADMAP_ENGINE_LEVEL_8_UPDATE.md

@@ -0,0 +1,11 @@
+## Level 8 — Schema-Driven First-Class Tool Normalization
+
+- [x] Canonical 17 first-class tool inventory documented.
+- [x] Tool folder paths verified for all 17 first-class tools.
+- [x] Explicit schema paths recorded for all 17 first-class tools.
+- [x] Close first-class schema gap by adding `skin-editor.schema.json`.
+- [x] Define launcher pairing contract.
+- [ ] Verify all 17 tools have exactly one schema.
+- [ ] Verify sample tool payload files match workspace/game `tools[]` item shape.
+- [ ] Verify palette/shared data never embeds inside tool payload files.
+- [ ] Verify workspace schema has no sample-only concepts.

docs/dev/specs/LAUNCHER_PAIRING_CONTRACT.md

@@ -0,0 +1,22 @@
+# Launcher Pairing Contract
+
+## Purpose
+Define how launchers pass tool payload data and shared data objects without embedding shared data into tool payload files.
+
+## Contract
+`openTool(toolPayloadJson, dataObjects)`
+
+### `toolPayloadJson`
+- Source file pattern: `sample.<id>.<tool>.json`
+- Shape: identical to the corresponding `tools[]` item shape used in workspace manifests
+- Scope: tool payload only
+
+### `dataObjects`
+- Contains shared, externalized objects such as palette data and asset data
+- Passed separately from `toolPayloadJson`
+
+## Rules
+- Do not embed palette objects inside tool payload JSON files.
+- Do not embed shared asset objects inside tool payload JSON files.
+- Keep one tool payload shape across sample and workspace launch paths.
+- Launcher wiring composes `toolPayloadJson` with `dataObjects` at launch time.

docs/operations/dev/codex_commands.md

@@ -1,25 +1,5 @@
 MODEL: GPT-5.3-codex
-REASONING: medium
-
 TASK:
-Apply BUILD_PR_LEVEL_8_17_ROADMAP_AND_SKIN_EDITOR_SCHEMA.
-
-STEPS:
-1. Read docs/pr/PLAN_PR_LEVEL_8_17_ROADMAP_AND_SKIN_EDITOR_SCHEMA.md.
-2. Read docs/pr/BUILD_PR_LEVEL_8_17_ROADMAP_AND_SKIN_EDITOR_SCHEMA.md.
-3. Add tools/schemas/tools/skin-editor.schema.json exactly as provided unless an equivalent schema already exists.
-4. Update docs/dev/roadmaps/MASTER_ROADMAP_ENGINE.md:
-   - If the Level 8 schema-driven normalization section already exists, update status markers only.
-   - If it does not exist, append the Level 8 section from docs/dev/roadmaps/MASTER_ROADMAP_ENGINE_LEVEL_8_APPEND.md.
-   - Do not delete, rewrite, reflow, or paraphrase existing roadmap text.
-5. Add docs/dev/reports/level_8_17_roadmap_and_skin_editor_schema_report.md with validation findings.
-6. Verify first-class schema coverage is now 17/17.
-7. Do not modify runtime files.
-8. Do not add validators.
-9. Do not modify start_of_day.
-
-ACCEPTANCE:
-- Roadmap status advanced using only [ ] -> [.] or [.] -> [x].
-- skin-editor schema exists.
-- 17/17 first-class tools have exactly one schema.
-- Runtime/start_of_day unchanged.
+- Append/update roadmap Level 8 status only
+- Add launcher contract docs
+- No runtime changes

docs/operations/dev/commit_comment.txt

@@ -1,10 +1,4 @@
-BUILD_PR_LEVEL_8_17_ROADMAP_AND_SKIN_EDITOR_SCHEMA
+PR 8.18: Launcher pairing contract
 
-- Added roadmap tracking for Level 8 schema-driven first-class tool normalization
-- Added missing skin-editor tool schema
-- Advanced roadmap status without rewriting existing roadmap content
-
-PR Details:
-- Closes documented 16/17 schema coverage gap
-- Keeps all 17 first-class tools planned
-- Preserves docs-first workflow and start_of_day guard
+- Defined openTool(toolPayload, dataObjects)
+- Advanced roadmap status

docs/pr/BUILD_PR_LEVEL_8_18_LAUNCHER_PAIRING_CONTRACT_DOCS.md

@@ -0,0 +1,26 @@
+# BUILD_PR_LEVEL_8_18_LAUNCHER_PAIRING_CONTRACT_DOCS
+
+## Launcher Contract
+
+openTool(
+  toolPayloadJson,
+  dataObjects
+)
+
+### toolPayloadJson
+- sample.<id>.<tool>.json
+- identical to workspace tools[] entry
+
+### dataObjects
+- palette object
+- asset objects
+- passed separately
+
+## Rules
+- No palette/data inside tool payload
+- Tool schema remains consistent across sample + workspace
+- Launcher responsible for wiring
+
+## Acceptance
+- Contract documented
+- No schema changes

docs/pr/PLAN_PR_LEVEL_8_18_LAUNCHER_PAIRING_CONTRACT_DOCS.md

@@ -0,0 +1,9 @@
+# PLAN_PR_LEVEL_8_18_LAUNCHER_PAIRING_CONTRACT_DOCS
+
+## Purpose
+Define the launcher contract that passes tool payload + shared data objects (palette, assets) without embedding data in tool payloads.
+
+## Scope
+- Docs only
+- Roadmap advancement
+- No runtime changes