Remove deprecated Skin Editor tooling and skin.main.json files - PR_26139_029-remove-skin-editor-and-skin-main-files

Author: DavidQ

docs/design/tools/TOOLS_REENGINEERING_INDEX.md

@@ -11,7 +11,6 @@ This index lists the remaining rebuild anchors, schema baseline, and support fol
 | `asset-pipeline` | `tools/Asset Pipeline` | rebuildable tool | core-03 | tools.asset-pipeline | core rebuild lane |
 | `svg-asset-studio` | `tools/SVG Asset Studio` | rebuildable tool | core-04 | tools.svg-asset-studio | core rebuild lane |
 | `sprite-editor` | `tools/Sprite Editor` | rebuildable tool | core-05 | tools.sprite-editor | core rebuild lane |
-| `skin-editor` | `tools/Skin Editor` | rebuildable tool | core-06 | tools.skin-editor | core rebuild lane |
 | `tile-map-editor` | `tools/Tilemap Studio` | rebuildable tool | core-07 | tools.tile-map-editor | core rebuild lane |
 | `tile-model-converter` | `tools/Tile Model Converter` | rebuildable tool | core-08 | tools.tile-model-converter | core rebuild lane |
 | `parallax-editor` | `tools/Parallax Scene Studio` | rebuildable tool | core-09 | tools.parallax-editor | core rebuild lane |

docs/design/tools/skin-editor/REENGINEERING_DESIGN.md

@@ -727,52 +727,6 @@ layer list empty after successful fetch
 seeded/demo layers shown instead of loaded layers
 ```
 
-## 12. Skin Editor (`skin-editor`)
-
-### Required inputs
-
-```text
-gameId context
-skin payload json when samplePresetPath is provided
-palette/context handoff from workspace or game context
-```
-
-### Required controls
-
-```text
-object list/workbench
-selected object controls
-palette list
-preview canvas
-status/summary/context fields
-```
-
-### Required output fields
-
-```text
-status text that reports Loaded preset or load failure
-active game context output
-palette availability output
-selected object output
-```
-
-### Success
-
-```text
-game context fields match loaded launch context
-palette list is populated when palette context is required
-selected object controls bind loaded skin payload
-preview canvas renders loaded object state
-```
-
-### Failure
-
-```text
-missing game context hidden
-palette context missing while editor reports ready
-preview uses fallback/default object data
-```
-
 ## 13. State Inspector (`state-inspector`)
 
 ### Required inputs

docs/dev/dod/tool_ui_readiness_dod.md

@@ -0,0 +1,28 @@
+# PR_26139_029 Remove Skin Editor And Skin Main Files
+
+## Summary
+
+- Removed the deprecated Skin Editor tool folder, registry entry, workspace schema reference, tool schema, design doc, and active shell/tool-hint references.
+- Removed all `skin.main.json` files and their `workspace.asset-catalog.json` entries.
+- Removed the runtime `skin.main` loader path and booted Pong, Breakout, Bouncing Ball, and Solar System with their existing built-in scene defaults.
+- Removed Skin Editor sample slices 0226/0227 and the Skin Editor payload from sample 1902.
+- Updated active tests and active generated validation reports so current surfaces no longer expect Skin Editor.
+
+## Validation
+
+- PASS: `npm run build:manifest`
+- PASS: `node scripts\validate-json-contracts.mjs --mode=games --details`
+  - `game_manifest_schema_validation: total=11 invalid=0`
+- PASS: `node scripts\validate-json-contracts.mjs --mode=all --details`
+  - Regenerated active schema reports; game manifest rows remain valid. Existing non-game strict schema/sample rows still report their current known invalid entries.
+- PASS: `node scripts\validate-tool-registry.mjs`
+- PASS: `npm run test:workspace-v2`
+  - 58 passed
+- PASS: `node tests\tools\ToolSchemaStrictModeValidation.test.mjs`
+- PASS: direct tools index/registry assertion for removed Skin Editor.
+- PASS: active exact-reference sweeps for `skin-editor`, `Skin Editor`, `Primitive Skin Editor`, `primitive-skin-editor`, `skin.main.json`, `skin.main`, `loadGameSkin`, and `gameSkinLoader`.
+
+## Notes
+
+- Historical reports under `docs/dev/reports` still contain old PR/audit prose mentioning Skin Editor; active current surfaces and regenerated active reports do not.
+- Full launch smoke was not run because it creates workspace files under `tmp/`; the user explicitly reserved `tmp/` for the delta zip.