Skip to content

Commit 8f54478

Browse files
author
DavidQ
committed
Add TESTABILITY GATE and BUILD quality requirement without modifying existing content.
1 parent 7407041 commit 8f54478

5 files changed

Lines changed: 46 additions & 9 deletions

File tree

docs/dev/CODEX_COMMANDS.md

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
11
MODEL: GPT-5.4-codex
2-
REASONING: high
3-
2+
REASONING: low
43
COMMAND:
5-
Execute exactly docs/pr/BUILD_PR_GAMES_94A_ASTEROIDS_NEW_PARALLEL_BURST_ALPHA.md.
6-
Modify only the exact target files listed in the PR doc.
7-
Do not expand scope.
8-
Package the delta zip to:
9-
<project folder>/tmp/BUILD_PR_GAMES_94A_ASTEROIDS_NEW_PARALLEL_BURST_ALPHA_delta.zip
4+
Append provided content to RULES.md and WORKFLOW.md without deleting or modifying existing content.

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Accelerate asteroids_new with one larger non-destructive parallel adoption burst covering world, debug, and two entity files.
1+
Add TESTABILITY GATE and BUILD quality requirement without modifying existing content.

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
After commit, inspect the delta and either pull one additional gameplay slice or validate before widening into systems.
1+
Next: enforce testable vertical slice builds across roadmap.

docs/dev/start_of_day/codex/RULES.md

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,3 +77,29 @@ Codex must report:
7777
- whether scope stayed clean
7878
- whether engine-core was untouched if expected
7979
- the ZIP output path under `<project folder>/tmp/`
80+
81+
82+
83+
## 🚫 TESTABILITY GATE (NEW — REQUIRED)
84+
85+
A BUILD PR must produce a testable result.
86+
87+
Definition of testable:
88+
- Can be executed, loaded, or meaningfully validated (runtime, UI, or module level)
89+
- Not just structural (folders, placeholders, or isolated files)
90+
- Not a partial dependency chain that cannot run
91+
92+
FAIL FAST if:
93+
- The change only copies files without enabling execution
94+
- The result cannot be smoke-tested
95+
- The change introduces a partial system with no entry point
96+
97+
DO NOT EXECUTE:
98+
- Scaffolding-only PRs
99+
- Single-file migrations that do not integrate
100+
- Folder-only or placeholder-only changes
101+
102+
Instead:
103+
- Expand the BUILD scope to a vertical slice
104+
- Or request a corrected BUILD PR with sufficient scope
105+

docs/dev/start_of_day/codex/WORKFLOW.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,3 +30,19 @@
3030
- modify roadmap wording
3131
- touch start_of_day files
3232
- continue after a failed execution threshold
33+
34+
35+
36+
## ✅ BUILD PR Quality Requirement (TESTABILITY)
37+
38+
Every BUILD PR must result in a testable state.
39+
40+
Preferred:
41+
- Vertical slice (feature, flow, or runnable subset)
42+
43+
Avoid:
44+
- Incremental fragments that cannot run
45+
46+
Rule of thumb:
47+
If it cannot be executed or meaningfully validated, it is not a valid BUILD.
48+

0 commit comments

Comments
 (0)