Skip to content

Commit 38f6bd0

Browse files
author
DavidQ
committed
Contract stabilization progress (Level 18 Track C)
1 parent d512f9f commit 38f6bd0

9 files changed

Lines changed: 122 additions & 57 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 7 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,11 @@
11
MODEL: GPT-5.3-codex
22
REASONING: high
33

4-
Execute BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.
4+
Execute BUILD_PR_LEVEL_18_5_CONTRACT_STABILIZATION_PROGRESS:
55

6-
Required behavior:
7-
1. Use the prior reviewed recommendations as the source of truth.
8-
2. Execute only the approved reclassification moves for:
9-
- games/Bouncing-ball
10-
- games/Gravity
11-
- games/Thruster
12-
- games/ProjectileLab
13-
- games/Orbit
14-
- games/PaddleIntercept
15-
- games/MultiBallChaos
16-
3. Do not change games/PacmanLite.
17-
4. Apply only the exact path/reference updates needed to keep the moved entries valid.
18-
5. Do not expand into unrelated cleanup or refactors.
19-
6. Write execution-backed reports to docs/dev/reports.
20-
7. Update docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md in place under these rules:
21-
- never delete roadmap content
22-
- never rewrite roadmap text
23-
- only update status markers:
24-
- [ ] -> [.]
25-
- [.] -> [x]
26-
- append additive content only if explicitly required by this PR
27-
28-
Packaging:
29-
- produce final ZIP at:
30-
<project folder>/tmp/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.zip
6+
- inspect engine public APIs
7+
- inspect shared contracts
8+
- identify unstable surfaces
9+
- stabilize or remove where safe
10+
- validate selectors/providers
11+
- output report

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1 @@
1-
Execute approved game-to-sample reclassification moves.
2-
3-
BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION
4-
- move approved game/demo entries into recommended sample phase targets
5-
- leave PacmanLite unchanged
6-
- update exact references required by the moves
7-
- update master roadmap in place under roadmap guard rules
1+
Contract stabilization progress (Level 18 Track C)
Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
# BUILD_PR_LEVEL_18_5_CONTRACT_STABILIZATION_PROGRESS
2+
3+
## Purpose
4+
Advance Level 18 Track C contract stabilization by auditing engine public APIs and shared contracts, then applying the smallest safe stabilization.
5+
6+
## Scope Executed
7+
- inspected engine public API barrels in `src/engine/**/index.js` with focus on debug provider/panel surfaces
8+
- inspected shared contract/state surfaces in:
9+
- `src/shared/contracts/index.js`
10+
- `src/shared/state/index.js`
11+
- `src/shared/state/selectors.js`
12+
- `src/shared/state/publicSelectors.js`
13+
- validated selector and provider stability using focused runtime tests
14+
15+
## Unstable Surfaces Identified
16+
1. `src/engine/debug/standard/threeD/index.js` did not export the current inspector-era provider/panel contracts introduced by Track H follow-up work.
17+
- Impact: consumers using the threeD public barrel could not depend on stable imports for:
18+
- transform inspector
19+
- render pipeline stages
20+
- collision overlays
21+
- scene graph inspector
22+
23+
## Stabilization Actions Executed (Safe)
24+
- Updated `src/engine/debug/standard/threeD/index.js` to explicitly export current inspector-era provider and panel contracts, while preserving existing legacy summary exports for compatibility.
25+
- No selector behavior changes were made.
26+
- No provider runtime behavior changes were made.
27+
28+
## Files Changed
29+
- `src/engine/debug/standard/threeD/index.js`
30+
- `docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md` (status-only marker update)
31+
32+
## Roadmap Status Update (Execution-Backed)
33+
- Track C:
34+
- `ensure selectors/providers are stable` updated from `[ ]` to `[.]`
35+
36+
## Validation Commands Run
37+
1. Barrel contract export validation:
38+
```bash
39+
node (inline) -> imports ./src/engine/debug/standard/threeD/index.js and asserts required inspector-era exports exist
40+
```
41+
Result: PASS
42+
43+
2. Focused shared selector/provider regression checks:
44+
```bash
45+
node (inline) -> run():
46+
- tests/shared/SharedFoundationCombinedPass.test.mjs
47+
- tests/shared/GetStateVariantClassification.test.mjs
48+
- tests/tools/TransformInspectorDebugPanel.test.mjs
49+
- tests/tools/RenderPipelineStagesDebugPanel.test.mjs
50+
- tests/tools/CollisionOverlaysDebugPanel.test.mjs
51+
- tests/tools/SceneGraphInspectorDebugPanel.test.mjs
52+
```
53+
Result: 6/6 PASS
54+
55+
## Bounded Caveats
56+
- Legacy 3D summary provider/panel exports remain intentionally exposed for backward compatibility and should be retired in a dedicated deprecation/removal PR, not this progress slice.
Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1 @@
1-
Docs-first execution bundle for the approved game-to-sample reclassification lane.
2-
3-
This PR asks Codex to:
4-
- execute the approved reclassification moves
5-
- leave PacmanLite alone
6-
- update only exact references required by those moves
7-
- update the master roadmap in place under the roadmap guard
1+
Docs-only PR

docs/dev/reports/file_tree.txt

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1 @@
1-
docs/
2-
docs/pr/
3-
docs/pr/BUILD_PR_LEVEL_18_4_GAME_TO_SAMPLE_RECLASSIFICATION_EXECUTION.md
4-
docs/dev/
5-
docs/dev/codex_commands.md
6-
docs/dev/commit_comment.txt
7-
docs/dev/reports/
8-
docs/dev/reports/change_summary.txt
9-
docs/dev/reports/validation_checklist.txt
10-
docs/dev/reports/file_tree.txt
1+
generated
Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,3 @@
1-
[ ] Prior reviewed recommendations used as source of truth
2-
[ ] Approved entries moved from games to samples
3-
[ ] PacmanLite left unchanged
4-
[ ] Exact path/reference updates completed
5-
[ ] Reports written to docs/dev/reports
6-
[ ] Master roadmap updated in place under guard rules
1+
[ ] unstable surfaces identified
2+
[ ] stabilization applied
3+
[ ] validation done

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -736,7 +736,7 @@
736736
### Track C � Contract Stabilization
737737
- [.] finalize engine public APIs
738738
- [.] finalize shared contracts
739-
- [ ] ensure selectors/providers are stable
739+
- [.] ensure selectors/providers are stable
740740
- [.] remove unstable or experimental surfaces
741741

742742
### Track D � Codebase Consistency
Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# BUILD_PR_LEVEL_18_5_CONTRACT_STABILIZATION_PROGRESS
2+
3+
## Purpose
4+
Advance Level 18 Track C by progressing engine and shared contract stabilization.
5+
6+
## Scope
7+
- docs-only
8+
- no implementation authored by ChatGPT
9+
- smallest scoped stabilization step
10+
11+
## Codex Responsibilities
12+
- inspect engine public APIs and shared contracts
13+
- identify unstable or experimental surfaces
14+
- stabilize or mark for removal
15+
- ensure selectors/providers remain stable
16+
17+
## Acceptance
18+
- unstable surfaces identified
19+
- stabilization actions executed where safe
20+
- validation report produced

src/engine/debug/standard/threeD/index.js

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,45 @@ export {
2626
PROVIDER_3D_SCENE_GRAPH_SUMMARY,
2727
createSceneGraphSummaryProvider
2828
} from "./providers/sceneGraphSummaryProvider.js";
29+
export {
30+
PROVIDER_3D_TRANSFORM_INSPECTOR,
31+
createTransformInspectorProvider
32+
} from "./providers/transformInspectorProvider.js";
33+
export {
34+
PROVIDER_3D_RENDER_PIPELINE_STAGES,
35+
createRenderPipelineStagesProvider
36+
} from "./providers/renderPipelineStagesProvider.js";
37+
export {
38+
PROVIDER_3D_COLLISION_OVERLAYS,
39+
createCollisionOverlaysProvider
40+
} from "./providers/collisionOverlaysProvider.js";
41+
export {
42+
PROVIDER_3D_SCENE_GRAPH_INSPECTOR,
43+
createSceneGraphInspectorProvider
44+
} from "./providers/sceneGraphInspectorProvider.js";
2945

3046
export { createStandard3dPanels, registerStandard3dPanels } from "./panels/registerStandard3dPanels.js";
3147
export { PANEL_3D_TRANSFORM, create3dTransformPanel } from "./panels/panel3dTransform.js";
3248
export { PANEL_3D_CAMERA, create3dCameraPanel } from "./panels/panel3dCamera.js";
3349
export { PANEL_3D_RENDER_STAGES, create3dRenderStagesPanel } from "./panels/panel3dRenderStages.js";
3450
export { PANEL_3D_COLLISION, create3dCollisionPanel } from "./panels/panel3dCollision.js";
3551
export { PANEL_3D_SCENE_GRAPH, create3dSceneGraphPanel } from "./panels/panel3dSceneGraph.js";
52+
export {
53+
PANEL_3D_TRANSFORM_INSPECTOR,
54+
create3dTransformInspectorPanel
55+
} from "./panels/panel3dTransformInspector.js";
56+
export {
57+
PANEL_3D_RENDER_PIPELINE_STAGES,
58+
create3dRenderPipelineStagesPanel
59+
} from "./panels/panel3dRenderPipelineStages.js";
60+
export {
61+
PANEL_3D_COLLISION_OVERLAYS,
62+
create3dCollisionOverlaysPanel
63+
} from "./panels/panel3dCollisionOverlays.js";
64+
export {
65+
PANEL_3D_SCENE_GRAPH_INSPECTOR,
66+
create3dSceneGraphInspectorPanel
67+
} from "./panels/panel3dSceneGraphInspector.js";
3668

3769
export {
3870
STANDARD_3D_DEBUG_PRESETS,

0 commit comments

Comments
 (0)