Run full schema validation sweep and fix violations - PR 11.144

Author: DavidQ

docs/dev/codex_commands.md

@@ -11,21 +11,21 @@ ALLOWED FILES:
 TASK:
 
 1. For each schema:
-   - ensure additionalProperties: false
-   - remove any wrapper acceptance
-   - ensure strict payload validation
+   - test valid payload → must pass
+   - test wrapper JSON → must fail
+   - test parent JSON → must fail
 
-2. Validate:
-   - valid JSON passes
-   - wrapper fails
-   - parent fails
+2. If violation:
+   - fix schema ONLY
 
 3. DO NOT:
-   - modify runtime
-   - modify routing
-   - modify samples
+   - expand schema
+   - add compatibility
+
+4. VERIFY:
+   - all schemas strict
 
 REPORT:
-docs/dev/reports/schema_lock_enforcement_11_143.txt
+docs/dev/reports/schema_validation_sweep_11_144.txt
 
-FAIL if any schema remains loose
+FAIL if any schema loose

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Lock schemas as single source of truth with strict validation - PR 11.143
+Run full schema validation sweep and fix violations - PR 11.144

docs/dev/reports/schema_validation_sweep_11_144.txt

@@ -0,0 +1,36 @@
+Schema Validation Sweep Report: 11_144
+Date: 2026-04-30
+Repo: C:/Users/davidq/Documents/GitHub/HTML-JavaScript-Gaming
+Mode: STRICT SCOPE
+
+Scope
+- Target schemas: tools/schemas/**/*.json
+- Runtime/routing/samples: not modified
+
+Validation Performed
+1) Per-schema contract test (all schema JSON files under tools/schemas)
+- Test A: valid payload (synthesized from each schema) -> must pass
+- Test B: wrapper JSON -> must fail
+- Test C: parent/workspace JSON -> must fail
+
+Result Matrix
+- All schemas: valid=PASS
+- All schemas: wrapper=FAIL (expected)
+- All schemas: parent=FAIL (expected)
+- Summary: SUMMARY_FAILURES=0
+
+2) Strict object lock audit
+- Verified every `type: object` schema node declares `additionalProperties`.
+- Result: TOTAL_OBJECT_SCHEMA_MISSING_ADDITIONAL_PROPERTIES=0
+
+Schema Fixes
+- None required. No violations found, so no schema edits were applied.
+
+Compliance With Constraints
+- No schema expansion performed.
+- No compatibility paths added.
+- No runtime/routing/sample files changed.
+
+Final Outcome
+- PASS
+- All schemas are strict under required sweep conditions.

docs/pr/BUILD_PR_LEVEL_11_144_SCHEMA_LOCK_VALIDATION_SWEEP.md

@@ -0,0 +1,54 @@
+# BUILD_PR_LEVEL_11_144_SCHEMA_LOCK_VALIDATION_SWEEP
+
+## Purpose
+Run a full schema validation sweep to ensure all tool schemas correctly enforce strict contracts after lock.
+
+## Scope
+- STRICT SCOPE
+- schemas only
+- validation only
+- no runtime changes
+- no routing changes
+
+## ALLOWED FILES
+- tools/schemas/**/*.json
+
+## ALLOWED CHANGES
+- fix ONLY schema violations found during validation
+- no expansion of schema
+- no compatibility additions
+
+## REQUIRED VALIDATION
+
+For ALL schemas:
+
+1. Validate:
+   - valid payload JSON → PASS
+   - wrapper JSON → FAIL
+   - parent JSON → FAIL
+
+2. Ensure:
+   - additionalProperties: false everywhere
+   - no fallback logic
+   - no wrapper support
+
+## REQUIRED FIXES
+
+If schema fails:
+- tighten validation
+- remove extra fields
+- correct required fields
+
+## REPORT
+
+docs/dev/reports/schema_validation_sweep_11_144.txt:
+- schemas tested
+- failures found
+- fixes applied
+- final pass status
+
+## FAILURE
+
+FAIL if any schema:
+- passes invalid JSON
+- allows extra properties