Skip to content

Commit f384f55

Browse files
author
DavidQ
committed
PR 8.2: Schema enforcement + sample compliance
- Enforced $schema usage across samples - Aligned sample configs with tool schemas - Ensured palette schema compliance
1 parent b03cbc6 commit f384f55

77 files changed

Lines changed: 2295 additions & 185 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

docs/dev/codex_commands.md

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,12 @@ MODEL: GPT-5.3-codex
22
REASONING: medium
33

44
TASK:
5-
- Add schema files exactly as defined
6-
- Insert $schema references into JSON files
7-
- Remove ALL validation utilities
8-
- Do NOT add fallback logic
5+
- Add $schema to all sample JSON files
6+
- Align sample config.json to tool.schema.json
7+
- Align palette.json to shared schema
8+
- Remove any non-schema-compliant fields
9+
- Do NOT add validation utilities
10+
- Do NOT modify runtime logic
11+
12+
OUTPUT:
13+
- All samples schema-compliant

docs/dev/commit_comment.txt

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
PR 8.1: Schema-driven validation only
2-
- Added schemas
3-
- Removed validation utilities
4-
- Enforced no-fallback rule
1+
PR 8.2: Schema enforcement + sample compliance
2+
3+
- Enforced $schema usage across samples
4+
- Aligned sample configs with tool schemas
5+
- Ensured palette schema compliance
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# PR 8.2 — Schema Enforcement + Sample Compliance
2+
3+
## Purpose
4+
Enforce schema usage across all samples and ensure compliance with tool + shared schemas.
5+
6+
## Scope
7+
- Require $schema in all JSON assets
8+
- Validate samples against tool schemas
9+
- Ensure palettes align with shared schema
10+
- No runtime validation logic added
11+
12+
## Rules
13+
- Samples MUST reference tool schemas directly
14+
- All JSON MUST include $schema
15+
- No fallback or normalization allowed
16+
17+
## Validation Targets
18+
- samples/**/config.json -> tool.schema.json
19+
- samples/**/palette.json -> palette.schema.json
20+
- workspace.manifest.json -> workspace.schema.json
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# Sample Compliance Checklist
2+
3+
- [ ] All samples include config.json
4+
- [ ] config.json has $schema reference
5+
- [ ] palette.json has $schema reference
6+
- [ ] No extra fields outside schema
7+
- [ ] No missing required fields

samples/metadata/samples.curriculum.validation.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../tools/schemas/sample.tool-payload.schema.json",
23
"version": 1,
34
"progression": {
45
"phaseOrder": [

samples/metadata/samples.index.metadata.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../tools/schemas/sample.tool-payload.schema.json",
23
"version": 1,
34
"phases": [
45
{

samples/metadata/samples.runtime.validation.report.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../tools/schemas/sample.tool-payload.schema.json",
23
"version": 1,
34
"generatedAt": "2026-04-13T21:06:29.167Z",
45
"scope": "samples-runtime-only",

samples/metadata/samples.shared.boundaries.report.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../tools/schemas/sample.tool-payload.schema.json",
23
"version": 1,
34
"scope": "samples-only",
45
"generatedAt": "2026-04-13",

samples/phase-02/0201/sample-0201-3d-camera-path-editor.json

Lines changed: 31 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../../tools/schemas/sample.tool-payload.schema.json",
23
"sampleId": "0201",
34
"phase": "02",
45
"title": "Camera Follow",
@@ -13,18 +14,42 @@
1314
"waypoints": [
1415
{
1516
"t": 0,
16-
"position": { "x": -14, "y": 8, "z": -20 },
17-
"lookAt": { "x": 0, "y": 0, "z": 0 }
17+
"position": {
18+
"x": -14,
19+
"y": 8,
20+
"z": -20
21+
},
22+
"lookAt": {
23+
"x": 0,
24+
"y": 0,
25+
"z": 0
26+
}
1827
},
1928
{
2029
"t": 1500,
21-
"position": { "x": 0, "y": 10, "z": -14 },
22-
"lookAt": { "x": 2, "y": 0, "z": 0 }
30+
"position": {
31+
"x": 0,
32+
"y": 10,
33+
"z": -14
34+
},
35+
"lookAt": {
36+
"x": 2,
37+
"y": 0,
38+
"z": 0
39+
}
2340
},
2441
{
2542
"t": 3000,
26-
"position": { "x": 16, "y": 9, "z": -16 },
27-
"lookAt": { "x": 4, "y": 0, "z": 2 }
43+
"position": {
44+
"x": 16,
45+
"y": 9,
46+
"z": -16
47+
},
48+
"lookAt": {
49+
"x": 4,
50+
"y": 0,
51+
"z": 2
52+
}
2853
}
2954
]
3055
}

samples/phase-02/0202/sample-0202-3d-camera-path-editor.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
{
2+
"$schema": "../../../tools/schemas/sample.tool-payload.schema.json",
23
"sampleId": "0202",
34
"phase": "02",
45
"title": "Camera Bounds",

0 commit comments

Comments
 (0)