Make Workspace palette a singular tool payload and rebuild sample 1902 - PR 11.14

DavidQ · DavidQ · commit f0ba84fb1242 · 2026-04-28T21:59:12.000-04:00
diff --git a/docs/dev/codex_commands.md b/docs/dev/codex_commands.md
@@ -3,15 +3,18 @@
 model: gpt-5.3-codex
 reasoning: medium
 
-Apply PR_11_13_WORKSPACE_1902_SCHEMA_CONFORMANCE_FIX.
+Apply PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD.
 
 Required:
-- Validate sample 1902 against the actual Workspace schemas, not the sample tool-payload schema.
-- Rewrite `samples/phase-19/1902/sample.1902.workspace-all-tools.json` to conform to the actual Workspace schema/manifest shape.
-- Remove unsupported/misplaced fields such as `tool: workspace-all-tools-integration`, `activeWorkspaceTools`, duplicate `config`, duplicate `payload`, copied unrelated sample/game data, and palette sidecar references unless the actual schema requires them.
-- Ensure Workspace resolves all active workspace-supported tools from the schema-valid manifest, not just Palette.
+- Update `tools/schemas/workspace.manifest.schema.json` so Palette is a singular tool payload under `tools.palette`.
+- Remove top-level required `palettes` collection from the manifest schema unless migration compatibility is required and documented.
+- Rebuild only `samples/phase-19/1902/`.
+- Delete/remove `sample.1902.palette.json`.
+- Rebuild sample 1902 workspace payload so all tool data is under `tools`, including `tools.palette`.
+- Allow only one palette in the workspace manifest.
+- Ensure Workspace shows all valid tools, not only Palette.
+- Do not modify other samples.
 - Do not add fallback/default/hidden data.
-- Do not modify standalone tool samples.
 - Do not modify start_of_day folders.
-- Add validation report at docs/dev/reports/PR_11_13_WORKSPACE_1902_SCHEMA_CONFORMANCE_FIX_report.md.
-- Return ZIP artifact at tmp/PR_11_13_WORKSPACE_1902_SCHEMA_CONFORMANCE_FIX_delta.zip.
+- Add validation report at docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md.
+- Return ZIP artifact at tmp/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_delta.zip.
diff --git a/docs/dev/commit_comment.txt b/docs/dev/commit_comment.txt
@@ -1 +1 @@
-Fix sample 1902 Workspace JSON to conform to actual workspace schema - PR 11.13
+Make Workspace palette a singular tool payload and rebuild sample 1902 - PR 11.14
diff --git a/docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md b/docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md
@@ -0,0 +1,76 @@
+# PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD Report
+
+## Result
+- PASS
+
+## Scope Applied
+- Updated `tools/schemas/workspace.manifest.schema.json`.
+- Rebuilt only `samples/phase-19/1902/sample.1902.workspace-all-tools.json`.
+- Did not modify any other sample folder.
+
+## Schema Changes
+- Removed top-level required `palettes`.
+- Removed top-level `palettes` property from the workspace manifest schema.
+- Kept top-level `tools` required.
+- Added required singular `tools.palette`.
+- Added shape checks for `tools.palette` (`id`, `name`, `swatches`).
+
+## Final 1902 JSON Structure
+- Top-level keys:
+  - `documentKind`
+  - `schema`
+  - `version`
+  - `id`
+  - `name`
+  - `tools`
+- Canonical palette location:
+  - `tools.palette`
+- Tool payloads remain under `tools.<toolId>` for all active tools.
+
+## Palette Contract Confirmation
+- `tools.palette` exists and is the canonical workspace palette payload.
+- Only one canonical palette object exists in the manifest (`tools.palette`).
+- Top-level `palettes` collection is removed.
+- `sample.1902.palette.json` sidecar is not present.
+- No `sample.1902.palette.json` references remain in the 1902 workspace manifest.
+
+## Workspace Validation (More Than Palette)
+- Validation script result:
+  - `PR_11_14_validation PASS`
+  - `activeToolCount 17`
+  - `sampleToolCount 18`
+- `sampleToolCount` includes all 17 active tool payload keys plus canonical `palette`.
+- Launch smoke run:
+  - Command: `npm run test:launch-smoke -- --tools`
+  - Result: `PASS=287 FAIL=0 TOTAL=287`
+  - Includes `sample 1902` PASS and all tool entries PASS.
+
+## Tools Shown/Resolved in 1902
+- `vector-map-editor`
+- `vector-asset-studio`
+- `tile-map-editor`
+- `parallax-editor`
+- `sprite-editor`
+- `skin-editor`
+- `asset-browser`
+- `palette-browser`
+- `state-inspector`
+- `replay-visualizer`
+- `performance-profiler`
+- `physics-sandbox`
+- `asset-pipeline-tool`
+- `tile-model-converter`
+- `3d-json-payload-normalizer`
+- `3d-asset-viewer`
+- `3d-camera-path-editor`
+- `palette` (canonical workspace palette payload)
+
+## Change Boundary Confirmation
+- Sample changes were limited to `samples/phase-19/1902/`.
+- No `start_of_day` changes.
+- No fallback/default/hidden data was added.
+
+## Files Changed for This PR
+- `tools/schemas/workspace.manifest.schema.json`
+- `samples/phase-19/1902/sample.1902.workspace-all-tools.json`
+- `docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md`
diff --git a/docs/dev/reports/REPORT_PR_11_14.md b/docs/dev/reports/REPORT_PR_11_14.md
@@ -0,0 +1,10 @@
+# REPORT_PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD
+
+## Summary
+This PR updates the Workspace manifest contract so Palette is a singular tool payload under `tools.palette`, then rebuilds only sample 1902 against that schema.
+
+## Target
+- No top-level palette collection.
+- No palette sidecar.
+- One palette under tools.palette.
+- Workspace shows all valid tools for sample 1902.
diff --git a/docs/dev/reports/launch_smoke_report.md b/docs/dev/reports/launch_smoke_report.md
@@ -1,6 +1,6 @@
 # Launch Smoke Report
 
-Generated: 2026-04-29T01:39:23.947Z
+Generated: 2026-04-29T01:57:36.705Z
 
 Filters: games=true, samples=true, tools=true, sampleRange=all
 
diff --git a/docs/pr/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD/BUILD.md b/docs/pr/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD/BUILD.md
@@ -0,0 +1,69 @@
+# BUILD_PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD
+
+## Required Codex Work
+
+### 1. Update Workspace manifest schema
+Update:
+`tools/schemas/workspace.manifest.schema.json`
+
+Required model:
+- top-level `tools` remains required
+- top-level `palettes` must be removed unless retained only for migration with clear deprecation
+- Palette belongs under `tools.palette`
+- only one `tools.palette` entry is allowed
+- no multiple palette collection
+
+Preferred manifest shape:
+```json
+{
+  "tools": {
+    "asset-browser": {},
+    "sprite-editor": {},
+    "vector-map-editor": {},
+    "palette": {
+      "id": "sample-1902-palette",
+      "name": "Sample 1902 Palette",
+      "swatches": []
+    }
+  }
+}
+```
+
+### 2. Rebuild only sample 1902
+Update only:
+`samples/phase-19/1902/`
+
+Rules:
+- delete/remove any `sample.1902.palette.json`
+- remove top-level `palettes`
+- remove duplicated palette/config/payload blocks
+- put palette data at `tools.palette`
+- put each tool payload under its matching `tools.<toolId>` key
+- no copied unrelated sample/game dump data
+- no fallback/default/hidden data
+
+### 3. Workspace schema file
+Update the 1902 Workspace file so it uses the correct launcher/index schema and points to the updated manifest shape.
+
+If 1902 uses a single combined file, it must still match the updated schema expected by the loader. Do not invent unsupported fields.
+
+### 4. Validation
+Create:
+docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md
+
+Report must include:
+- schema changes
+- final 1902 JSON structure
+- confirmation palette is under `tools.palette`
+- confirmation only one palette exists
+- confirmation no palette sidecar remains
+- Workspace validation showing more than Palette is available
+- list of tools shown/resolved
+- confirmation only sample 1902 was changed
+- confirmation no start_of_day changes
+
+## Constraints
+- One PR purpose only.
+- Do not modify other samples.
+- Do not modify standalone sample tool behavior.
+- No broad Workspace refactor unless required to read the updated schema.
diff --git a/docs/pr/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD/PLAN.md b/docs/pr/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD/PLAN.md
@@ -0,0 +1,25 @@
+# PLAN_PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD
+
+## Purpose
+Update the Workspace manifest schema so Palette is a singular tool payload under `tools.palette`, then rebuild only sample 1902 to match.
+
+## Problem
+Workspace currently shows only Palette. The schema separates `palettes` from `tools`, but Palette is a tool and should be represented as a single allowed `tools.palette` payload. Sample 1902 must not use a separate palette sidecar or workspace-level palettes collection.
+
+## Scope
+- Update Workspace manifest schema contract.
+- Palette becomes singular and tool-owned:
+  - `tools.palette`
+  - only one palette allowed
+- Rebuild only sample 1902 against the updated schema.
+- Do not modify other samples.
+- Do not modify standalone tool samples.
+- Do not modify start_of_day folders.
+
+## Acceptance
+- `workspace.manifest.schema.json` no longer requires top-level `palettes`.
+- `tools.palette` is the single palette source for Workspace.
+- Only one palette payload is allowed in Workspace manifest.
+- Sample 1902 contains no `sample.1902.palette.json`.
+- Sample 1902 Workspace JSON validates against updated schema.
+- Workspace shows all valid tools, not only Palette.
diff --git a/samples/phase-19/1902/sample.1902.workspace-all-tools.json b/samples/phase-19/1902/sample.1902.workspace-all-tools.json
diff --git a/tools/schemas.zip b/tools/schemas.zip
diff --git a/tools/schemas/workspace.manifest.schema.json b/tools/schemas/workspace.manifest.schema.json

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-Fix sample 1902 Workspace JSON to conform to actual workspace schema - PR 11.13`
	`1`	`+Make Workspace palette a singular tool payload and rebuild sample 1902 - PR 11.14`