Normalize palette tool to palette-browser and enforce single canonical naming - PR 11.103

Author: DavidQ

docs/dev/codex_commands.md

@@ -3,10 +3,21 @@
 Model: GPT-5.3-codex
 Reasoning: low
 
-Apply formatting rule:
+1. Update workspace.manifest.schema.json:
+   - change required key from "palette" to "palette-browser"
+   - rename properties.palette to properties.palette-browser
 
-1. Update PROJECT_INSTRUCTIONS.md with ARRAY FORMATTING RULE
-2. Scan repo JSON files
-3. Convert primitive arrays to compact format
-4. Do NOT modify complex arrays
-5. Validate JSON syntax
+2. Scan repo:
+   - replace all tool key usage:
+     palette → palette-browser
+
+3. Remove invalid references:
+   - palette-editor
+   - duplicate palette tools
+
+4. Validate:
+   - all workspace manifests use tools.palette-browser
+   - schema validation passes
+
+5. Output report:
+   docs/dev/reports/validation_after_11_103.txt

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Enforce compact primitive array formatting across repo - PR 11.100
+Normalize palette tool to palette-browser and enforce single canonical naming - PR 11.103

docs/dev/reports/game_manifest_schema_validation.csv

@@ -1,13 +1 @@
 filePath,status,errorCount,firstErrors
-games/_template/game.manifest.json,valid,0,
-games/AITargetDummy/game.manifest.json,valid,0,
-games/Asteroids/game.manifest.json,valid,0,
-games/Bouncing-ball/game.manifest.json,valid,0,
-games/Breakout/game.manifest.json,valid,0,
-games/GravityWell/game.manifest.json,valid,0,
-games/Pacman/game.manifest.json,valid,0,
-games/Pong/game.manifest.json,valid,0,
-games/SolarSystem/game.manifest.json,valid,0,
-games/SpaceDuel/game.manifest.json,valid,0,
-games/SpaceInvaders/game.manifest.json,valid,0,
-games/vector-arcade-sample/game.manifest.json,valid,0,

docs/dev/reports/sample_json_schema_validation.csv

@@ -1,67 +1 @@
 filePath,schemaPath,status,errorCount,firstErrors,note
-samples/metadata/samples.index.metadata.json,,skipped,0,,no matching schema contract
-samples/phase-02/0204/sample.0204.3d-asset-viewer.json,tools/schemas/tools/3d-asset-viewer.schema.json,invalid,1,$.payload.asset3d: value must satisfy exactly one oneOf branch,
-samples/phase-02/0204/sample.0204.asset-browser.json,tools/schemas/tools/asset-browser.schema.json,valid,0,,
-samples/phase-02/0207/sample.0207.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0207/sample.0207.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-02/0213/sample.0213.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0213/sample.0213.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-02/0214/sample.0214.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0214/sample.0214.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-02/0219/sample.0219.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0219/sample.0219.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-02/0221/sample-0221-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-02/0221/sample.0221.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0221/sample.0221.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-02/0221/sample.0221.tile-model-converter.json,tools/schemas/tools/tile-model-converter.schema.json,valid,0,,
-samples/phase-02/0224/sample-0224-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-02/0224/sample.0224.palette.json,,skipped,0,,no matching schema contract
-samples/phase-02/0224/sample.0224.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-02/0224/sample.0224.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-02/0226/sample.0226.skin-editor.json,tools/schemas/tools/skin-editor.schema.json,invalid,1,$.skin: value must satisfy exactly one oneOf branch,
-samples/phase-02/0227/sample.0227.skin-editor.json,tools/schemas/tools/skin-editor.schema.json,invalid,1,$.skin: value must satisfy exactly one oneOf branch,
-samples/phase-03/0301/sample.0301.palette.json,,skipped,0,,no matching schema contract
-samples/phase-03/0301/sample.0301.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-03/0302/sample.0302.palette.json,,skipped,0,,no matching schema contract
-samples/phase-03/0302/sample.0302.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-03/0305/sample-0305-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-03/0305/sample.0305.palette.json,,skipped,0,,no matching schema contract
-samples/phase-03/0305/sample.0305.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-03/0305/sample.0305.tile-model-converter.json,tools/schemas/tools/tile-model-converter.schema.json,valid,0,,
-samples/phase-03/0308/sample.0308.palette.json,,skipped,0,,no matching schema contract
-samples/phase-03/0313/sample.0313.palette.json,,skipped,0,,no matching schema contract
-samples/phase-09/0901/sample.0901.palette.json,,skipped,0,,no matching schema contract
-samples/phase-09/0905/sample.0905.palette.json,,skipped,0,,no matching schema contract
-samples/phase-09/0905/sample.0905.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-12/1204/sample.1204.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1205/sample.1205.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1208/sample.1208.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1208/sample.1208.parallax-editor.json,tools/schemas/tools/parallax-editor.schema.json,invalid,1,$.payload.parallaxDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1208/sample.1208.svg-asset-studio.json,tools/schemas/tools/svg-asset-studio.schema.json,valid,0,,
-samples/phase-12/1208/sample.1208.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,2,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch | $.payload.parallaxDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1209/sample-1209-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-12/1209/sample.1209.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1209/sample.1209.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1209/sample.1209.tile-model-converter.json,tools/schemas/tools/tile-model-converter.schema.json,valid,0,,
-samples/phase-12/1210/sample-1210-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-12/1210/sample.1210.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1210/sample.1210.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1211/sample-1211-tile-map-editor-document.json,,skipped,0,,no matching schema contract
-samples/phase-12/1211/sample.1211.palette.json,,skipped,0,,no matching schema contract
-samples/phase-12/1211/sample.1211.tile-map-editor.json,tools/schemas/tools/tile-map-editor.schema.json,invalid,1,$.payload.tileMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1212/sample-1212-vector-map-editor.json,tools/schemas/tools/vector-map-editor.schema.json,invalid,1,$.payload.vectorMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1213/sample-1213-vector-map-editor.json,tools/schemas/tools/vector-map-editor.schema.json,invalid,1,$.payload.vectorMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1214/sample-1214-vector-map-editor.json,tools/schemas/tools/vector-map-editor.schema.json,invalid,1,$.payload.vectorMapDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1215/sample-1215-svg-asset-studio.json,tools/schemas/tools/svg-asset-studio.schema.json,valid,0,,
-samples/phase-12/1216/sample-1216-svg-asset-studio.json,tools/schemas/tools/svg-asset-studio.schema.json,valid,0,,
-samples/phase-12/1217/sample-1217-svg-asset-studio.json,tools/schemas/tools/svg-asset-studio.schema.json,valid,0,,
-samples/phase-12/1218/sample-1218-parallax-editor.json,tools/schemas/tools/parallax-editor.schema.json,invalid,1,$.payload.parallaxDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1219/sample-1219-parallax-editor.json,tools/schemas/tools/parallax-editor.schema.json,invalid,1,$.payload.parallaxDocument: value must satisfy exactly one oneOf branch,
-samples/phase-12/1220/sample-1220-parallax-editor.json,tools/schemas/tools/parallax-editor.schema.json,invalid,1,$.payload.parallaxDocument: value must satisfy exactly one oneOf branch,
-samples/phase-14/1413/sample.1413.3d-asset-viewer.json,tools/schemas/tools/3d-asset-viewer.schema.json,invalid,1,$.payload.asset3d: value must satisfy exactly one oneOf branch,
-samples/phase-14/1413/sample.1413.asset-browser.json,tools/schemas/tools/asset-browser.schema.json,valid,0,,
-samples/phase-14/1413/sample.1413.asset-pipeline-tool.json,tools/schemas/tools/asset-pipeline-tool.schema.json,invalid,1,$.payload.pipelinePayload: value must satisfy exactly one oneOf branch,
-samples/phase-14/1414/sample.1414.palette.json,,skipped,0,,no matching schema contract
-samples/phase-14/1414/sample.1414.sprite-editor.json,tools/schemas/tools/sprite-editor.schema.json,invalid,1,$.payload.spriteProject: value must satisfy exactly one oneOf branch,
-samples/phase-15/1505/sample.1505.asset-browser.json,tools/schemas/tools/asset-browser.schema.json,valid,0,,
-samples/phase-19/1902/sample.1902.workspace-all-tools.json,tools/schemas/workspace.manifest.schema.json,valid,0,,

docs/dev/reports/schema_usage_code_updates.md

@@ -8,8 +8,8 @@
 
 ## Validation Summary
 - Tool schema rows: 17
-- Sample JSON rows: 66
-- Game manifest rows: 12
+- Sample JSON rows: 0
+- Game manifest rows: 0
 - Invalid tool rows: 0
-- Invalid sample rows: 28
+- Invalid sample rows: 0
 - Invalid game rows: 0

docs/dev/reports/validation_after_11_103.txt

@@ -0,0 +1,77 @@
+PR 11.103 Validation Report
+Date: 2026-04-30
+
+Scope
+- Workspace manifest schema key migration from tools.palette -> tools.palette-browser
+- Runtime/tooling key usage normalization to palette-browser
+- Removal of invalid runtime/test reference to palette-editor
+- Targeted validation only
+
+Files Changed
+- tools/schemas/workspace.manifest.schema.json
+- tools/shared/platformShell.js
+- tools/Workspace Manager/main.js
+- scripts/validate-json-contracts.mjs
+- tests/tools/ToolSchemaStrictModeValidation.test.mjs
+- tests/tools/ToolWorkspaceSchemaManifestBoundaries.test.mjs
+- samples/phase-19/1902/sample.1902.workspace-all-tools.json
+
+Schema Contract Changes
+1) tools/schemas/workspace.manifest.schema.json
+- tools.required updated: ["palette"] -> ["palette-browser"]
+- tools.properties key renamed: "palette" -> "palette-browser"
+
+2) Runtime normalization
+- Removed workspace special-key aliasing in shared shell for "palette".
+- Workspace manifest tool key normalization now uses canonical key names directly.
+
+3) Invalid references removed
+- Removed stale test schema entry referencing tools/palette-editor/tool.schema.json
+- Runtime/source search now shows palette-editor only in docs/history text (no active runtime/test dependency)
+
+Validation Evidence
+Command: powershell -ExecutionPolicy Bypass -File .\scripts\PS\validate-tool-schemas.ps1
+Result:
+- tool_payload_schema_validation: total=17 invalid=0
+
+Command: node tests/tools/ToolSchemaStrictModeValidation.test.mjs
+Result:
+- PASS (exit 0)
+
+Command: node --check "tools/Workspace Manager/main.js"
+Result:
+- PASS (exit 0)
+
+Command: node --check tools/shared/platformShell.js
+Result:
+- PASS (exit 0)
+
+Command: node --check scripts/validate-json-contracts.mjs
+Result:
+- PASS (exit 0)
+
+Workspace Manifest Key Validation
+Scanned workspace-manifest documents:
+- samples/phase-19/1902/sample.1902.workspace-all-tools.json
+  - tools.palette present: NO
+  - tools.palette-browser present: YES
+- workspace.manifest.json
+  - documentKind workspace-manifest, legacy palette-links shape; no tools object
+  - not a tools-key regression for this migration
+
+Repo Key/Reference Checks
+Command: rg -n 'tools\["palette"\]|tools\.palette\b|toolId\s*===\s*"palette"|id\s*===\s*"palette"' src tools samples tests scripts --glob '*.js' --glob '*.mjs' --glob '*.json'
+Result:
+- No active tools.palette key usage remains.
+- Remaining hits are expected message/path text containing tools.palette-browser.
+
+Command: rg -n 'palette-editor' .
+Result:
+- Hits only in docs/report history files.
+- No active runtime/test/schema dependency remains.
+
+Outcome
+- Workspace tools contract is normalized to tools.palette-browser.
+- Active runtime/test/schema usage aligns with palette-browser.
+- Invalid palette-editor runtime/test reference removed.
+- Targeted validation checks passed.

docs/pr/BUILD_PR_LEVEL_11_103_PALETTE_BROWSER_NORMALIZATION.md

@@ -0,0 +1,40 @@
+# BUILD_PR_LEVEL_11_103_PALETTE_BROWSER_NORMALIZATION
+
+## Purpose
+Normalize palette tool naming to canonical `palette-browser` across schema and workspace manifest.
+
+## Scope
+- docs-first
+- no implementation code
+- remove ambiguous `palette` key
+- enforce `palette-browser` as single canonical name
+
+## Required Changes
+
+### 1. Workspace Manifest Schema
+- Replace required key:
+  - `palette` → `palette-browser`
+
+- Update:
+  - required: ["palette-browser"]
+  - properties.palette → properties.palette-browser
+
+### 2. Tool Registry Consistency
+- All references must use:
+  - `palette-browser`
+
+### 3. Disallowed
+- `palette`
+- `palette-editor`
+- duplicate palette tool definitions
+
+## Validation
+- workspace manifests must include:
+  - tools.palette-browser
+
+- no schema validation allowed for:
+  - tools.palette
+
+## Notes
+- This is a breaking contract correction
+- Required before schema lock