Skip to content

Commit a355cbc

Browse files
author
DavidQ
committed
Capture palette contract evidence for explicit normalization follow-up - PR 10.6D
1 parent 1aef2dd commit a355cbc

25 files changed

Lines changed: 15464 additions & 23 deletions

‎docs/dev/codex_commands.md‎

Lines changed: 2 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,8 @@
1-
# CODEX COMMANDS — LEVEL 10.6B Palette Contract Alignment
1+
# Codex Commands
22

33
Model: GPT-5.4
44
Reasoning: high
55

6-
## PR Purpose
7-
Normalize palette payload contracts so palette data in manifest inputs and tool JSON inputs use the same canonical shape.
8-
9-
## Codex Command
106
```powershell
11-
codex --model gpt-5.4 --reasoning high "Implement LEVEL_10_6B_PALETTE_CONTRACT_ALIGNMENT. One PR purpose only. Do not add new features, schemas, tools, fallback data, hardcoded asset paths, or silent default loading. Fix palette contract drift only. Ensure palette-bearing standalone samples pass explicit data flow: sample -> manifest -> normalized input -> tool -> UI/state. Manifest palette payloads and tool palette JSON files must share the same canonical palette object shape: schema, version, name, source, swatches. Remove legacy leading metadata such as $schema from runtime palette payloads where it causes mismatch. Replace source values such as engine/paletteList with manifest or another explicit manifest-derived source where appropriate. Palette Browser and palette-dependent tools must consume payload.palette directly, then bind state.palette to UI. Do not reshape, merge, inject defaults, or auto-load hidden palettes inside tools. Update the standalone data-flow report/status only as needed after running tests. Run npm run test:launch-smoke:games and npm run test:sample-standalone:data-flow."
12-
```
13-
14-
## Required Validation
15-
```powershell
16-
npm run test:launch-smoke:games
17-
npm run test:sample-standalone:data-flow
18-
```
19-
20-
## Report to Inspect
21-
```powershell
22-
docs/dev/reports/level_10_6_standalone_tool_data_flow_report.md
23-
```
24-
25-
## Commit Comment
26-
```text
27-
Normalize palette contract to manifest SSoT and remove tool-level schema drift - PR 10.6B
7+
codex --model gpt-5.4 --reasoning high "Run BUILD_PR_LEVEL_10_6D_PALETTE_CONTRACT_EVIDENCE_CAPTURE exactly as documented in docs/pr/BUILD_PR_LEVEL_10_6D_PALETTE_CONTRACT_EVIDENCE_CAPTURE.md. Diagnostics only. Do not normalize palette files. Run npm run test:sample-standalone:data-flow fresh, collect the exact palette manifest/tool/schema files, produce docs/dev/reports/level_10_6d_palette_contract_evidence/palette_contract_evidence_report.md, and package repo-relative evidence into tmp/BUILD_PR_LEVEL_10_6D_PALETTE_CONTRACT_EVIDENCE_CAPTURE.zip."
288
```

‎docs/dev/commit_comment.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Normalize palette contract to manifest SSoT and remove tool-level schema drift - PR 10.6B
1+
Capture palette contract evidence for explicit normalization follow-up - PR 10.6D

‎docs/dev/reports/level_10_6d_palette_contract_evidence/palette_contract_evidence_report.md‎

Lines changed: 114 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,114 @@
1+
# level_10_6d_palette_contract_evidence_report
2+
3+
## command run
4+
- `npm run test:sample-standalone:data-flow`
5+
6+
## timestamps
7+
- reportGeneratedAt: 2026-04-27T17:04:41.161Z
8+
- testSummaryGeneratedAt: 2026-04-27T17:02:16.473Z
9+
10+
## test result summary
11+
- status: PASS
12+
- totalSampleToolPayloadFiles: 62
13+
- totalSamplePaletteFiles: 20
14+
- totalRoundtripRows: 62
15+
- schemaFailures: 0
16+
- contractFailures: 0
17+
- roundtripPathFailures: 0
18+
- genericFailures: 0
19+
- paletteBrowserGenericChecks: 3
20+
- paletteBrowserTargetedChecks: 3
21+
22+
## palette-related samples checked
23+
- sampleId 0213 -> toolId palette-browser
24+
- sampleId 0308 -> toolId palette-browser
25+
- sampleId 0313 -> toolId palette-browser
26+
27+
## sample manifest palette shape vs tool expected palette shape
28+
- test expectation source:
29+
- `tests/runtime/SampleStandaloneToolDataFlow.test.mjs:1162-1168` reads `testCase.presetPayload?.config?.palette` and requires non-empty `name` and `swatches`.
30+
- runtime ingestion source:
31+
- `tools/Palette Browser/main.js:531-553` extracts palette payload via aliases (`payload.palette`, `config.palette`, `payload.swatches`, `config.swatches`).
32+
- `tools/Palette Browser/main.js:506-529` validates/normalizes imported palette shape.
33+
- comparison (manifest palette object vs paired `sample.<id>.palette.json`):
34+
- Sample 0213:
35+
- swatchesMatch: true
36+
- keysOnlyInManifest: (none)
37+
- keysOnlyInPaletteFile: $schema, locked, sourceId
38+
- value mismatch: source => manifest="manifest" | paletteFile="engine/paletteList"
39+
- Sample 0308:
40+
- swatchesMatch: true
41+
- keysOnlyInManifest: (none)
42+
- keysOnlyInPaletteFile: $schema
43+
- value mismatch: source => manifest="manifest" | paletteFile="generated-from-sample-colors"
44+
- Sample 0313:
45+
- swatchesMatch: true
46+
- keysOnlyInManifest: (none)
47+
- keysOnlyInPaletteFile: $schema
48+
- value mismatch: source => manifest="manifest" | paletteFile="generated-from-sample-colors"
49+
50+
## exact mismatches found
51+
- Cross-file mismatches detected (manifest `config.palette` vs sample-side `sample.<id>.palette.json`):
52+
- Sample 0213:
53+
- swatchesMatch: true
54+
- keysOnlyInManifest: (none)
55+
- keysOnlyInPaletteFile: $schema, locked, sourceId
56+
- value mismatch: source => manifest="manifest" | paletteFile="engine/paletteList"
57+
- Sample 0308:
58+
- swatchesMatch: true
59+
- keysOnlyInManifest: (none)
60+
- keysOnlyInPaletteFile: $schema
61+
- value mismatch: source => manifest="manifest" | paletteFile="generated-from-sample-colors"
62+
- Sample 0313:
63+
- swatchesMatch: true
64+
- keysOnlyInManifest: (none)
65+
- keysOnlyInPaletteFile: $schema
66+
- value mismatch: source => manifest="manifest" | paletteFile="generated-from-sample-colors"
67+
- Test contract mismatches during fresh run: none (all failure arrays empty in `tmp/sample-standalone-tool-data-flow-results.json`).
68+
69+
## required status checks
70+
- `$schema` remains: yes.
71+
- Tool payload files contain top-level `$schema` references to `tools/schemas/tools/palette-browser.schema.json`.
72+
- Sample palette files contain top-level `$schema` references to `tools/schemas/palette.schema.json`.
73+
- `engine` / `paletteList` remains: yes.
74+
- Sample 0213 palette source remains `engine/paletteList`.
75+
- Palette Browser still loads `../../src/engine/paletteList.js` and reads `globalThis.palettesList`.
76+
- `wrapper` remains: no in audited sample payloads.
77+
- Test harness still rejects wrapper fields `documentKind`, `id`, `type` for sample tool payload contracts.
78+
- `alias` remains: yes.
79+
- `extractPaletteFromSamplePreset` accepts `payload.palette`, `config.palette`, `payload.swatches`, and `config.swatches`.
80+
- `reshaping` remains: yes.
81+
- `normalizePaletteDocument` reshapes legacy `entries[]`/`colors[]` into `swatches[]`.
82+
- `fallback data` remains: yes.
83+
- Built-in palette fallback source defaults to `{}` when `globalThis.palettesList` is absent.
84+
- Symbol/name/version/source defaults are still applied during normalization.
85+
- `hardcoded paths` remain: partial.
86+
- Fixed engine script include path remains in tool HTML (`../../src/engine/paletteList.js`).
87+
- Preset ingest path handling is constrained to `/samples/`, `./samples/`, or `samples/` and rejects `..`.
88+
89+
## exact copied file paths
90+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/metadata/samples.index.metadata.json
91+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-02/0213/sample.0213.palette-browser.json
92+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-03/0308/sample.0308.palette-browser.json
93+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-03/0313/sample.0313.palette-browser.json
94+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-02/0213/sample.0213.palette.json
95+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-03/0308/sample.0308.palette.json
96+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/samples/phase-03/0313/sample.0313.palette.json
97+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/schemas/tools/palette-browser.schema.json
98+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/schemas/palette.schema.json
99+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/Palette Browser/index.html
100+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/Palette Browser/main.js
101+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/shared/paletteDocumentContract.js
102+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tools/toolRegistry.js
103+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tests/runtime/SampleStandaloneToolDataFlow.test.mjs
104+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/src/engine/paletteList.js
105+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/tmp/sample-standalone-tool-data-flow-results.json
106+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/docs/dev/reports/level_10_6d_palette_contract_evidence/test_sample_standalone_data_flow_output.txt
107+
- docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/docs/dev/reports/level_10_6d_palette_contract_evidence/palette_shape_diff.json
108+
109+
## fresh test output artifacts
110+
- docs/dev/reports/level_10_6d_palette_contract_evidence/test_sample_standalone_data_flow_output.txt
111+
- tmp/sample-standalone-tool-data-flow-results.json
112+
113+
## next-step recommendation (normalization only)
114+
- Normalize palette contract boundaries so sample manifest palette payload and sample palette data file metadata are explicitly aligned by policy, with a single accepted source semantic and a narrowed ingest alias surface (no behavior changes beyond normalization scope).

‎docs/dev/reports/level_10_6d_palette_contract_evidence/palette_shape_diff.json‎

Lines changed: 105 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,105 @@
1+
{
2+
"generatedAt": "2026-04-27T17:02:54.163Z",
3+
"comparisons": [
4+
{
5+
"sampleId": "0213",
6+
"manifestPath": "samples/phase-02/0213/sample.0213.palette-browser.json",
7+
"palettePath": "samples/phase-02/0213/sample.0213.palette.json",
8+
"manifestKeys": [
9+
"name",
10+
"schema",
11+
"source",
12+
"swatches",
13+
"version"
14+
],
15+
"paletteKeys": [
16+
"$schema",
17+
"locked",
18+
"name",
19+
"schema",
20+
"source",
21+
"sourceId",
22+
"swatches",
23+
"version"
24+
],
25+
"keysOnlyInManifest": [],
26+
"keysOnlyInPaletteFile": [
27+
"$schema",
28+
"locked",
29+
"sourceId"
30+
],
31+
"valueMismatches": [
32+
{
33+
"key": "source",
34+
"manifest": "manifest",
35+
"paletteFile": "engine/paletteList"
36+
}
37+
],
38+
"swatchesMatch": true
39+
},
40+
{
41+
"sampleId": "0308",
42+
"manifestPath": "samples/phase-03/0308/sample.0308.palette-browser.json",
43+
"palettePath": "samples/phase-03/0308/sample.0308.palette.json",
44+
"manifestKeys": [
45+
"name",
46+
"schema",
47+
"source",
48+
"swatches",
49+
"version"
50+
],
51+
"paletteKeys": [
52+
"$schema",
53+
"name",
54+
"schema",
55+
"source",
56+
"swatches",
57+
"version"
58+
],
59+
"keysOnlyInManifest": [],
60+
"keysOnlyInPaletteFile": [
61+
"$schema"
62+
],
63+
"valueMismatches": [
64+
{
65+
"key": "source",
66+
"manifest": "manifest",
67+
"paletteFile": "generated-from-sample-colors"
68+
}
69+
],
70+
"swatchesMatch": true
71+
},
72+
{
73+
"sampleId": "0313",
74+
"manifestPath": "samples/phase-03/0313/sample.0313.palette-browser.json",
75+
"palettePath": "samples/phase-03/0313/sample.0313.palette.json",
76+
"manifestKeys": [
77+
"name",
78+
"schema",
79+
"source",
80+
"swatches",
81+
"version"
82+
],
83+
"paletteKeys": [
84+
"$schema",
85+
"name",
86+
"schema",
87+
"source",
88+
"swatches",
89+
"version"
90+
],
91+
"keysOnlyInManifest": [],
92+
"keysOnlyInPaletteFile": [
93+
"$schema"
94+
],
95+
"valueMismatches": [
96+
{
97+
"key": "source",
98+
"manifest": "manifest",
99+
"paletteFile": "generated-from-sample-colors"
100+
}
101+
],
102+
"swatchesMatch": true
103+
}
104+
]
105+
}

‎docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/docs/dev/reports/level_10_6d_palette_contract_evidence/palette_shape_diff.json‎

Lines changed: 105 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,105 @@
1+
{
2+
"generatedAt": "2026-04-27T17:02:54.163Z",
3+
"comparisons": [
4+
{
5+
"sampleId": "0213",
6+
"manifestPath": "samples/phase-02/0213/sample.0213.palette-browser.json",
7+
"palettePath": "samples/phase-02/0213/sample.0213.palette.json",
8+
"manifestKeys": [
9+
"name",
10+
"schema",
11+
"source",
12+
"swatches",
13+
"version"
14+
],
15+
"paletteKeys": [
16+
"$schema",
17+
"locked",
18+
"name",
19+
"schema",
20+
"source",
21+
"sourceId",
22+
"swatches",
23+
"version"
24+
],
25+
"keysOnlyInManifest": [],
26+
"keysOnlyInPaletteFile": [
27+
"$schema",
28+
"locked",
29+
"sourceId"
30+
],
31+
"valueMismatches": [
32+
{
33+
"key": "source",
34+
"manifest": "manifest",
35+
"paletteFile": "engine/paletteList"
36+
}
37+
],
38+
"swatchesMatch": true
39+
},
40+
{
41+
"sampleId": "0308",
42+
"manifestPath": "samples/phase-03/0308/sample.0308.palette-browser.json",
43+
"palettePath": "samples/phase-03/0308/sample.0308.palette.json",
44+
"manifestKeys": [
45+
"name",
46+
"schema",
47+
"source",
48+
"swatches",
49+
"version"
50+
],
51+
"paletteKeys": [
52+
"$schema",
53+
"name",
54+
"schema",
55+
"source",
56+
"swatches",
57+
"version"
58+
],
59+
"keysOnlyInManifest": [],
60+
"keysOnlyInPaletteFile": [
61+
"$schema"
62+
],
63+
"valueMismatches": [
64+
{
65+
"key": "source",
66+
"manifest": "manifest",
67+
"paletteFile": "generated-from-sample-colors"
68+
}
69+
],
70+
"swatchesMatch": true
71+
},
72+
{
73+
"sampleId": "0313",
74+
"manifestPath": "samples/phase-03/0313/sample.0313.palette-browser.json",
75+
"palettePath": "samples/phase-03/0313/sample.0313.palette.json",
76+
"manifestKeys": [
77+
"name",
78+
"schema",
79+
"source",
80+
"swatches",
81+
"version"
82+
],
83+
"paletteKeys": [
84+
"$schema",
85+
"name",
86+
"schema",
87+
"source",
88+
"swatches",
89+
"version"
90+
],
91+
"keysOnlyInManifest": [],
92+
"keysOnlyInPaletteFile": [
93+
"$schema"
94+
],
95+
"valueMismatches": [
96+
{
97+
"key": "source",
98+
"manifest": "manifest",
99+
"paletteFile": "generated-from-sample-colors"
100+
}
101+
],
102+
"swatchesMatch": true
103+
}
104+
]
105+
}

‎docs/dev/reports/level_10_6d_palette_contract_evidence/repo_relative/docs/dev/reports/level_10_6d_palette_contract_evidence/test_sample_standalone_data_flow_output.txt‎

35.6 KB
Binary file not shown.

0 commit comments

Comments
 (0)