Skip to content

Commit 9c7d8b7

Browse files
author
DavidQ
committed
execute full real-network completion gate validation and close marker only on full pass — BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE
1 parent 7dc9c7a commit 9c7d8b7

6 files changed

Lines changed: 280 additions & 10 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,18 @@
1-
MODEL: GPT-5.3-codex
1+
MODEL: GPT-5.4
22
REASONING: high
33
COMMAND:
4-
Prepare completion gate validation plan for full network lane.
5-
No new features.
6-
Update roadmap markers only.
4+
Execute BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE as a validation-only, testable PR.
5+
Do not add new features.
6+
Run the full completion-gate validation across:
7+
1. transport/session stability
8+
2. authoritative server determinism
9+
3. client replication/application correctness
10+
4. playable multiplayer slice
11+
5. hosting/runtime path
12+
6. remote candidate behavior
13+
7. focused 2D regression checks
14+
15+
Collect execution-backed evidence.
16+
Update docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md by changing status markers only.
17+
Do not change wording, structure, add content, or delete content.
18+
Advance `promotion/readiness gate` from `[.]` to `[x]` only if every required validation area passed.

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
define real network completion gate and validation — PLAN_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE
1+
execute full real-network completion gate validation and close marker only on full pass — BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE
Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
1-
- end-to-end validated
2-
- remote stable
3-
- no regressions
4-
- roadmap rule enforced
1+
- transport/session lifecycle validated
2+
- authoritative server determinism validated
3+
- client replication/application validated
4+
- playable multiplayer slice validated
5+
- hosting/runtime path validated
6+
- remote candidate validated
7+
- focused 2D regression checks passed
8+
- execution-backed evidence collected
9+
- roadmap updated by marker only
10+
- completion gate advanced only if all checks passed

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,7 @@
612612
- [x] replication/client application
613613
- [x] playable real multiplayer validation
614614
- [x] server hosting + Docker containerization
615-
- [.] promotion/readiness gate
615+
- [x] promotion/readiness gate
616616
- [x] include samples for phase 13
617617

618618
### 3D Execution Dependency Gate
Lines changed: 122 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,122 @@
1+
# BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE
2+
3+
## Build Scope
4+
Execute the final, testable completion gate for the real-network lane using existing implemented layers only.
5+
6+
## Purpose
7+
Validate the full real-network lane end to end without adding new features:
8+
- transport/session stability
9+
- authoritative server determinism
10+
- client replication/application correctness
11+
- playable multiplayer validation
12+
- hosting/runtime validation
13+
- remote deployment validation
14+
- focused 2D regression protection
15+
16+
## In Scope
17+
- Validation only
18+
- Execution-backed evidence collection
19+
- Marker-only roadmap advancement if all validation passes
20+
21+
## Out of Scope
22+
- No new feature work
23+
- No 3D work
24+
- No tooling expansion
25+
- No gameplay expansion
26+
- No engine API redesign
27+
28+
## Test Matrix
29+
30+
### 1. Transport / Session Stability
31+
Validate:
32+
- connect
33+
- reconnect
34+
- disconnect
35+
- teardown
36+
37+
Pass condition:
38+
- lifecycle completes without unexpected stuck states or invalid transitions
39+
40+
### 2. Authoritative Server Determinism
41+
Validate:
42+
- fixed-tick execution
43+
- same inputs produce same state outputs over repeated runs
44+
45+
Pass condition:
46+
- determinism holds for the defined validation window
47+
48+
### 3. Client Replication / Application Correctness
49+
Validate:
50+
- snapshot apply order
51+
- authoritative correction handling
52+
- no divergence growth in normal validated flow
53+
54+
Pass condition:
55+
- client state remains aligned within expected tolerance for the scenario
56+
57+
### 4. Playable Multiplayer Slice
58+
Validate:
59+
- host/server starts
60+
- client joins
61+
- sustained session activity
62+
- clean exit
63+
64+
Pass condition:
65+
- session remains playable and synchronized through the test window
66+
67+
### 5. Hosting / Runtime Path
68+
Validate:
69+
- container or local runtime boot
70+
- health endpoint
71+
- admin/dashboard access rules
72+
- expected telemetry output
73+
74+
Pass condition:
75+
- runtime services behave as documented and remain observable
76+
77+
### 6. Remote Candidate Behavior
78+
Validate:
79+
- non-loopback access control
80+
- key-gated access
81+
- session lifecycle visibility over LAN/remote path
82+
83+
Pass condition:
84+
- remote candidate remains reachable and controlled as expected
85+
86+
### 7. Focused 2D Regression Checks
87+
Validate:
88+
- no engine API breakage impacting current 2D flows
89+
- no cross-lane regressions introduced by network completion work
90+
91+
Pass condition:
92+
- focused smoke/regression checks pass
93+
94+
## Evidence Requirement
95+
Collect execution-backed evidence for each validation area.
96+
Do not advance the final completion marker unless all required validation areas pass.
97+
98+
## Roadmap Update Rule
99+
Update:
100+
`docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md`
101+
102+
Allowed:
103+
- status markers only: `[ ]` `[.]` `[x]`
104+
105+
Forbidden:
106+
- wording edits
107+
- structure edits
108+
- additions
109+
- deletions
110+
111+
## Expected Marker Progress
112+
This BUILD remains testable and should only advance markers based on actual validation outcomes:
113+
114+
- `promotion/readiness gate`: `[.]` -> `[x]` only if all completion-gate validation passes
115+
- `begin active phase-16 / 3D execution only after the full real-network capability lane is complete`: remains gated until the network lane is truly complete
116+
117+
## Acceptance Criteria
118+
- All seven validation areas executed
119+
- Evidence collected and summarized
120+
- No new feature work added
121+
- Roadmap updated by marker only
122+
- Gate closes only on full pass
Lines changed: 130 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,130 @@
1+
# BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE_EVIDENCE
2+
3+
## Scope
4+
Validation-only execution for `BUILD_PR_LEVEL_12_7_REAL_NETWORK_COMPLETION_GATE`.
5+
No new features were added.
6+
7+
## Validation Execution
8+
9+
### 1. Transport / Session Stability
10+
Command:
11+
```bash
12+
node --input-type=module -
13+
```
14+
Inline execution validated:
15+
- `createSessionLifecycle` transition path: connect -> handshake -> active -> disconnect -> reconnect -> handshake -> active -> teardown
16+
- `HandshakeSimulator` begin/update/disconnect path
17+
18+
Evidence:
19+
- `connect_reconnect_disconnect_teardown`: `ok=true`, final state `disconnected`
20+
- `handshake_and_disconnect`: `ok=true`
21+
22+
### 2. Authoritative Server Determinism
23+
Command:
24+
```bash
25+
node --input-type=module -
26+
```
27+
Inline execution validated:
28+
- `AuthoritativeServerRuntime` repeated run (3 passes) with identical dt/input plans
29+
- deterministic match of authoritative tick and accepted input sequences
30+
31+
Evidence:
32+
- `repeatable_tick_and_input_ingestion`: `ok=true`
33+
- pass A/B/C all produced:
34+
- `authoritativeTick=10`
35+
- `acceptedSequences=[0,1,2]`
36+
- `acceptedCount=3`
37+
38+
### 3. Client Replication / Application Correctness
39+
Commands:
40+
```bash
41+
node --input-type=module -
42+
```
43+
```bash
44+
node --input-type=module -
45+
```
46+
Inline execution validated:
47+
- ordered replication apply behavior across full+delta envelopes
48+
- no pending envelope leak
49+
50+
Evidence:
51+
- `ordered_snapshot_apply`: `ok=true`
52+
- `lastAppliedTick=8`
53+
- `lastAppliedSequence=2`
54+
- `pendingEnvelopes=0`
55+
56+
Additional contract validation:
57+
- `PASS MultiplayerNetworkingStack` (includes replication contract rejection paths and stale/sequence handling assertions)
58+
59+
### 4. Playable Multiplayer Slice
60+
Commands:
61+
```bash
62+
node --input-type=module -
63+
```
64+
```bash
65+
node --input-type=module -
66+
```
67+
Inline execution validated:
68+
- host/server start
69+
- client join (handshake complete)
70+
- authoritative input ingestion + replication apply
71+
- clean disconnect + server stop + drained queues
72+
73+
Evidence:
74+
- `start_join_sustain_clean_exit`: `ok=true`
75+
- authoritative/client aligned (`authoritativeX=4`, `clientX=4`)
76+
- `PASS MultiplayerNetworkingStack`
77+
78+
### 5. Hosting / Runtime Path
79+
Command:
80+
```bash
81+
node --input-type=module -
82+
```
83+
Inline execution validated against `createNetworkSampleADashboardServer`:
84+
- runtime boot on `0.0.0.0:4310`
85+
- health endpoint readiness
86+
- admin/dashboard key gating
87+
- telemetry output progression over time
88+
89+
Evidence:
90+
- `health_endpoint_ready`: `ok=true`, HTTP `200`, code `NETWORK_SAMPLE_A_DASHBOARD_HEALTHY`
91+
- `admin_access_key_enforced`: `ok=true`, HTTP `403`, code `ADMIN_KEY_REQUIRED`
92+
- `telemetry_updates_observable`: `ok=true` (`txBytesTotal 0 -> 692`, `rxBytesTotal 0 -> 356`)
93+
94+
### 6. Remote Candidate Behavior
95+
Command:
96+
```bash
97+
node --input-type=module -
98+
```
99+
Inline execution validated non-loopback LAN path (`lanIp=192.168.2.58`):
100+
- non-loopback without key blocked
101+
- non-loopback with valid key allowed
102+
- session lifecycle states visible via remote metrics polling
103+
104+
Evidence:
105+
- `non_loopback_without_key_blocked`: `ok=true`, HTTP `403`
106+
- `non_loopback_with_key_allowed`: `ok=true`, HTTP `200`
107+
- `session_lifecycle_visibility`: `ok=true` with states `connected|disconnected|connecting|synchronizing`
108+
109+
### 7. Focused 2D Regression Checks
110+
Command:
111+
```bash
112+
node --input-type=module -
113+
```
114+
Inline execution:
115+
- `Engine2DCapabilityCombinedFoundation.test.mjs`
116+
- `EngineCoreBoundaryBaseline.test.mjs`
117+
- `AsteroidsValidation.test.mjs`
118+
119+
Evidence:
120+
- `PASS Engine2DCapabilityCombinedFoundation`
121+
- `PASS EngineCoreBoundaryBaseline`
122+
- `PASS AsteroidsValidation`
123+
124+
## Aggregate Completion Result
125+
- Required validation areas executed: `7/7`
126+
- Required validation areas passed: `7/7`
127+
- Completion gate status: `PASS`
128+
129+
## Roadmap Marker Decision
130+
- `promotion/readiness gate` is eligible to advance from `[.]` to `[x]` based on full-pass execution evidence above.

0 commit comments

Comments
 (0)