Integrates validation runner with standard workflows, adds exit code handling, and provides pre-commit/CI usage guidance.

BUILD_PR_LEVEL_10_03_VALIDATION_INTEGRATION

Author: DavidQ

.githooks/pre-commit.example.ps1

@@ -0,0 +1,26 @@
+[CmdletBinding()]
+param()
+
+Set-StrictMode -Version Latest
+$ErrorActionPreference = "Stop"
+
+$repoRoot = [System.IO.Path]::GetFullPath((Join-Path $PSScriptRoot ".."))
+$validateAllPath = Join-Path $repoRoot "scripts\PS\validate\Validate-All.ps1"
+
+if (-not (Test-Path -LiteralPath $validateAllPath)) {
+    Write-Host "Pre-commit validation: FAIL"
+    Write-Host "Reason: Missing validator script at $validateAllPath"
+    exit 1
+}
+
+& $validateAllPath
+$exitCode = if ($null -eq $LASTEXITCODE) { 0 } else { [int]$LASTEXITCODE }
+
+if ($exitCode -ne 0) {
+    Write-Host "Pre-commit validation: FAIL"
+    Write-Host "Commit aborted. Fix validation errors before committing."
+    exit $exitCode
+}
+
+Write-Host "Pre-commit validation: PASS"
+exit 0

docs/dev/CODEX_COMMANDS.md

@@ -2,6 +2,12 @@ MODEL: GPT-5.4
 REASONING: high
 
 COMMAND:
-- create Validate-All.ps1
-- call Validate-ScriptStructure.ps1
-- output pass/fail summary
+- ensure Validate-All.ps1 sets exit codes (0 pass, non-zero fail)
+- ensure clear PASS/FAIL output
+- add optional pre-commit hook example under .githooks/
+- keep validation scripts under scripts/PS/validate/
+- do not modify engine/runtime code
+- commit format:
+  description first line
+  PR name last line
+- update roadmap status only

docs/dev/COMMIT_COMMENT.txt

@@ -1,3 +1,2 @@
-Adds runnable validation entry point for all repo validation scripts.
-
-BUILD_PR_LEVEL_10_02_SCRIPT_VALIDATION_RUNNER
+Harden Validate-All exit-code behavior, keep clear PASS/FAIL summaries, and add an optional pre-commit hook example under .githooks.
+BUILD_PR_LEVEL_10_03_VALIDATION_INTEGRATION

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-BUILD_PR_LEVEL_10_03_VALIDATION_INTEGRATION
+BUILD_PR_LEVEL_10_04_VALIDATION_EXTENSIONS

docs/dev/reports/change_summary.txt

@@ -1 +1,20 @@
-Adds executable validation runner script.
+BUILD_PR_LEVEL_10_03_VALIDATION_INTEGRATION
+
+Scope:
+- Validation-lane changes only.
+- No engine/runtime/tool code touched.
+
+Implemented:
+- Updated `scripts/PS/validate/Validate-All.ps1`:
+  - explicit `0` exit on PASS
+  - non-zero exit on failure
+  - clear PASS/FAIL summary output
+  - guarded error path with failure summary and reason
+- Added optional pre-commit hook example:
+  - `.githooks/pre-commit.example.ps1`
+  - calls `scripts/PS/validate/Validate-All.ps1`
+  - blocks commit on validation failure
+
+Constraints:
+- Validation scripts remain under `scripts/PS/validate/`.
+- Deployment, engine, and runtime behavior were not modified.

docs/dev/reports/validation_checklist.txt

@@ -1,2 +1,6 @@
-[ ] run scripts/PS/validate/Validate-All.ps1
-[ ] PASS/FAIL output shown
+[x] `scripts/PS/validate/Validate-All.ps1` runs successfully in repo
+[x] PASS path returns exit code `0`
+[x] failure simulation returns non-zero exit code
+[x] clear PASS/FAIL summary output is present
+[x] optional `.githooks/pre-commit.example.ps1` calls `Validate-All.ps1`
+[x] roadmap updated with status-only change

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

@@ -550,7 +550,7 @@
 - [.] unit coverage aligned to engine/shared/games
 - [.] integration coverage aligned to state/replay/rendering/tools
 - [x] smoke validation aligned to samples/tools/games
-- [ ] fixtures/helpers organization normalized
+- [.] fixtures/helpers organization normalized
 - [x] move/refactor validation strategy documented
 - [x] post-PR acceptance criteria consistently enforced
 

docs/pr/BUILD_PR_LEVEL_10_03_VALIDATION_INTEGRATION.md

@@ -0,0 +1,24 @@
+# BUILD_PR — LEVEL 10_03 — VALIDATION INTEGRATION
+
+## Objective
+Integrate validation runner into standard workflows (pre-commit and CI-ready), and standardize exit codes so failures block bad structure.
+
+## What to Execute
+PowerShell:
+scripts/PS/validate/Validate-All.ps1
+
+## Expected Behavior
+- Returns exit code 0 on PASS
+- Returns non-zero on FAIL
+- Emits clear PASS/FAIL summary
+
+## Optional Local Hook (example)
+PowerShell:
+git config core.hooksPath .githooks
+
+Create .githooks/pre-commit that runs:
+powershell -NoProfile -ExecutionPolicy Bypass -File scripts/PS/validate/Validate-All.ps1
+if ($LASTEXITCODE -ne 0) { exit 1 }
+
+## CI Note
+This same script can be called from CI to gate merges.

scripts/PS/validate/Validate-All.ps1

@@ -11,18 +11,29 @@ if (-not (Test-Path -LiteralPath $validatorPath)) {
     throw "Missing validation entrypoint: $validatorPath"
 }
 
-$global:LASTEXITCODE = 0
-if ($Json.IsPresent) {
-    & $validatorPath -Json
+try {
+    $global:LASTEXITCODE = 0
+    if ($Json.IsPresent) {
+        & $validatorPath -Json
+    }
+    else {
+        & $validatorPath
+    }
 }
-else {
-    & $validatorPath
+catch {
+    Write-Host "Validate-All summary: FAIL"
+    Write-Host "Reason: $($_.Exception.Message)"
+    exit 1
 }
 
 $validatorExitCode = if ($null -eq $LASTEXITCODE) { 0 } else { [int]$LASTEXITCODE }
 $summary = if ($validatorExitCode -eq 0) { "PASS" } else { "FAIL" }
-Write-Host "Validate-All summary: $summary"
+if (-not $Json.IsPresent) {
+    Write-Host "Validate-All summary: $summary"
+}
 
 if ($validatorExitCode -ne 0) {
     exit $validatorExitCode
 }
+
+exit 0