Canonicalize schema filenames and restore compact primitive array enforcement - PR 11.104

Author: DavidQ

docs/dev/PROJECT_INSTRUCTIONS.md

@@ -292,7 +292,9 @@ Treat this as a recovery/stabilization PR only. Do not expand into a broader gam
 
 ## ARRAY FORMATTING RULE
 
-- Primitive-only arrays in JSON must use compact single-line formatting.
+- Primitive-only arrays in JSON must use compact grouped formatting.
 - Primitive values are: string, number, boolean, and null.
-- Complex arrays (arrays containing objects or nested arrays) must not be compacted.
+- Valid compact form example:
+  `[1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1]`
+- Do not compact arrays of objects, nested arrays, or complex structures.
 - Do not change JSON contracts or semantics while applying array formatting.

docs/dev/codex_commands.md

@@ -3,21 +3,58 @@
 Model: GPT-5.3-codex
 Reasoning: low
 
-1. Update workspace.manifest.schema.json:
-   - change required key from "palette" to "palette-browser"
-   - rename properties.palette to properties.palette-browser
+## PR
+BUILD_PR_LEVEL_11_104_SCHEMA_FILENAME_CANONICALIZATION_AND_ARRAY_RULE_RESTORE
 
-2. Scan repo:
-   - replace all tool key usage:
-     palette → palette-browser
+## Execute
 
-3. Remove invalid references:
-   - palette-editor
-   - duplicate palette tools
+1. Rename schema files:
+   - tools/schemas/tools/3d-json-payload.schema.json -> tools/schemas/tools/3d-json-payload.schema.json
+   - tools/schemas/tools/asset-pipeline.schema.json -> tools/schemas/tools/asset-pipeline.schema.json
 
-4. Validate:
-   - all workspace manifests use tools.palette-browser
-   - schema validation passes
+2. Update every repo reference to the renamed schema files:
+   - `$ref`
+   - schema ids
+   - validation scripts
+   - docs
+   - tool/sample/game JSON
 
-5. Output report:
-   docs/dev/reports/validation_after_11_103.txt
+3. Validate all JSON against strict schemas.
+
+4. Fix JSON/schema mismatches only:
+   - stale schema references
+   - stale tool ids
+   - invalid/unknown properties
+   - missing required fields
+   - wrong canonical names
+
+5. Restore the Project Instructions compact primitive-array formatting rule if it is missing or changed:
+   - simple primitive arrays must use compact grouped form:
+     [1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1]
+   - do not apply to arrays of objects, nested arrays, or complex structures
+
+6. Normalize repo JSON primitive arrays to match the rule.
+
+7. Confirm no old names remain:
+   - 3d-json-payload.schema.json
+   - asset-pipeline.schema.json
+   - palette
+   - palette-browser
+
+8. Write reports:
+   - docs/dev/reports/schema_filename_canonicalization_11_104.txt
+   - docs/dev/reports/json_schema_alignment_11_104.txt
+   - docs/dev/reports/array_formatting_rule_11_104.txt
+
+9. Roadmap:
+   - only update status markers if execution-backed
+   - do not rewrite roadmap text
+   - do not delete roadmap text
+
+## Constraints
+
+- Do not loosen schemas.
+- Do not add aliases.
+- Do not add fallback/default asset data.
+- Do not write implementation code unless required only to update validation/reference paths.
+- Do not run full samples smoke test unless validation proves shared sample loader/framework was changed.

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Normalize palette tool to palette-browser and enforce single canonical naming - PR 11.103
+Canonicalize schema filenames and restore compact primitive array enforcement - PR 11.104

docs/dev/reports/PR_10_27_DUPLICATE_CLASSIFICATION_AND_SAFE_CLEANUP_report.md

@@ -39,7 +39,7 @@ PASS
 | Hash (short) | Representative File | Reason |
 |---|---|---|
 | 971437553d34 | `docs/design/gdd/Capcom/Final Fight/gdd.txt` | Potential intentional/doc-specific duplication or unclear ownership |
-| df859606f030 | `tools/palette-editor/tool.schema.json` | Potential intentional/doc-specific duplication or unclear ownership |
+| df859606f030 | `tools/palette-browser/tool.schema.json` | Potential intentional/doc-specific duplication or unclear ownership |
 | f01a374e9c81 | `games/Pacman/assets/.gitkeep` | Potential intentional/doc-specific duplication or unclear ownership |
 | e3b0c44298fc | `docs/design/gdd/Nameco/Galaga/gdd.txt` | Potential intentional/doc-specific duplication or unclear ownership |
 
@@ -78,7 +78,7 @@ PASS
 | 2fb3e00d7762 | 3 | sample-variant-intentional | kept-no-action | high | `samples/phase-12/1218/assets/data/parallax/mountain-ridge-tall.svg` |
 | 6435cd00ef69 | 9 | sample-variant-intentional | kept-no-action | high | `games/AITargetDummy/rules/gameFlowRules.js` |
 | 9a97352ae4b6 | 9 | sample-variant-intentional | kept-no-action | high | `games/Pacman/rules/gameFlowRules.js` |
-| df859606f030 | 3 | ambiguous-no-action | kept-no-action | high | `tools/palette-editor/tool.schema.json` |
+| df859606f030 | 3 | ambiguous-no-action | kept-no-action | high | `tools/palette-browser/tool.schema.json` |
 | 9454a8b0c6e3 | 2 | sample-variant-intentional | kept-no-action | high | `samples/phase-12/1220/assets/data/parallax/mario_style_sky.svg` |
 | 7fac6c68998c | 2 | report-evidence | demoted-moved | high | `docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/schemas/tools/palette-browser.schema.json` |
 | 3a6e82fbca11 | 2 | template-intentional | kept-no-action | medium | `docs/operations/dev/templates/CODEX_AUTO_COMMAND.md` |

docs/dev/reports/PR_11_14_WORKSPACE_SCHEMA_PALETTE_TOOL_AND_1902_REBUILD_report.md

@@ -12,8 +12,8 @@
 - 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`).
+- Added required singular `tools.palette-browser`.
+- Added shape checks for `tools.palette-browser` (`id`, `name`, `swatches`).
 
 ## Final 1902 JSON Structure
 - Top-level keys:
@@ -24,12 +24,12 @@
   - `name`
   - `tools`
 - Canonical palette location:
-  - `tools.palette`
+  - `tools.palette-browser`
 - 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`).
+- `tools.palette-browser` exists and is the canonical workspace palette payload.
+- Only one canonical palette object exists in the manifest (`tools.palette-browser`).
 - 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.

docs/dev/reports/PR_11_17_SCHEMA_SET_NORMALIZATION_AND_WORKSPACE_REF_ENFORCEMENT_report.md

@@ -16,7 +16,7 @@ File:
 Verified:
 - Top-level `palettes` is removed.
 - Top-level `tools` is required.
-- `tools.palette` is singular and required.
+- `tools.palette-browser` is singular and required.
 - `tools.properties` explicitly lists all active workspace-supported registry ids.
 - `tools.additionalProperties` is `false`.
 - `$ref` entries are used for each declared tool/palette schema target.
@@ -36,9 +36,9 @@ Verified resolvable and valid JSON:
 - `./tools/replay-visualizer.schema.json`
 - `./tools/performance-profiler.schema.json`
 - `./tools/physics-sandbox.schema.json`
-- `./tools/asset-pipeline-tool.schema.json`
+- `./tools/asset-pipeline.schema.json`
 - `./tools/tile-model-converter.schema.json`
-- `./tools/3d-json-payload-normalizer.schema.json`
+- `./tools/3d-json-payload.schema.json`
 - `./tools/3d-asset-viewer.schema.json`
 - `./tools/3d-camera-path-editor.schema.json`
 
@@ -48,7 +48,7 @@ File:
 
 Verified:
 - No top-level `palettes`.
-- Contains `tools.palette` singleton payload.
+- Contains `tools.palette-browser` singleton payload.
 - Uses canonical registry-id tool keys plus `palette`.
 - No `sample.1902.palette.json` file present.
 - No `sample.1902.palette.json` references in sample 1902 payload.

docs/dev/reports/PR_11_18_FULL_STRICT_SCHEMA_MODE_report.md

@@ -9,9 +9,9 @@
 - `tools/schemas/tool.manifest.schema.json`
 - `tools/schemas/tools/3d-asset-viewer.schema.json`
 - `tools/schemas/tools/3d-camera-path-editor.schema.json`
-- `tools/schemas/tools/3d-json-payload-normalizer.schema.json`
+- `tools/schemas/tools/3d-json-payload.schema.json`
 - `tools/schemas/tools/asset-browser.schema.json`
-- `tools/schemas/tools/asset-pipeline-tool.schema.json`
+- `tools/schemas/tools/asset-pipeline.schema.json`
 - `tools/schemas/tools/palette-browser.schema.json`
 - `tools/schemas/tools/palette.schema.json`
 - `tools/schemas/tools/parallax-editor.schema.json`

docs/dev/reports/PR_11_19_SCHEMA_CLEANUP_TOOL_PAYLOADS_AND_PALETTE_CANONICALIZATION_report.md

@@ -57,9 +57,9 @@
 - `tools/schemas/sample.tool-payload.schema.json`
 - `tools/schemas/tools/3d-asset-viewer.schema.json`
 - `tools/schemas/tools/3d-camera-path-editor.schema.json`
-- `tools/schemas/tools/3d-json-payload-normalizer.schema.json`
+- `tools/schemas/tools/3d-json-payload.schema.json`
 - `tools/schemas/tools/asset-browser.schema.json`
-- `tools/schemas/tools/asset-pipeline-tool.schema.json`
+- `tools/schemas/tools/asset-pipeline.schema.json`
 - `tools/schemas/tools/palette-browser.schema.json`
 - `tools/schemas/tools/parallax-editor.schema.json`
 - `tools/schemas/tools/performance-profiler.schema.json`
@@ -78,7 +78,7 @@
   - `samples/phase-19/1902/sample.1902.workspace-all-tools.json`
 - Changes include:
   - tool entry `config` -> `payload`
-  - `tools.palette` migrated to canonical palette-browser payload shape
+  - `tools.palette-browser` migrated to canonical palette-browser payload shape
   - `tools.palette-browser` given explicit strict payload
   - generic `projectId` replacements
   - removed nested tile payload `$schema` reference to `sample.tool-payload.schema.json`

docs/dev/reports/PR_11_20_WORKSPACE_LOADER_SCHEMA_V2_TOOLS_PAYLOAD_SUPPORT_report.md

@@ -10,11 +10,11 @@ PASS
 
 ## Old Loader Assumptions Found
 - Workspace Manager previously derived available tools from registry/game filters, not strictly from `Object.keys(manifest.tools)` when a sample workspace manifest was provided.
-- Shared shell sample preset scoping previously matched `manifest.tools` keys by exact tool id only, so a singular `tools.palette` key would not resolve for `palette-browser` launches.
+- Shared shell sample preset scoping previously matched `manifest.tools` keys by exact tool id only, so a singular `tools.palette-browser` key would not resolve for `palette-browser` launches.
 
 ## New Loader Path
 - Workspace Manager now reads `samplePresetPath` workspace manifest, classifies `manifest.tools` keys, validates each key against tool host registry IDs (with explicit singular palette mapping), validates payload `tool` id equality, and uses accepted tool IDs as the active tool list filter.
-- Shared shell scoped preset resolution now supports singular `tools.palette` mapped to `palette-browser` and rejects mismatched/malformed tool payload entries.
+- Shared shell scoped preset resolution now supports singular `tools.palette-browser` mapped to `palette-browser` and rejects mismatched/malformed tool payload entries.
 
 ## Sample 1902 Manifest Tool Key Evidence
 Source:

docs/dev/reports/PR_11_21_WORKSPACE_MANAGER_TOOL_PRESENT_DETECTION_FIX_report.md

@@ -21,7 +21,7 @@ PASS
 - Classification now records and validates, per key:
   - raw key
   - normalized key
-  - palette alias mapping (`tools.palette` -> `palette-browser`)
+  - palette alias mapping (`tools.palette-browser` -> `palette-browser`)
   - registry match
   - workspace-schema allowed key
   - tool-schema validity