Skip to content

Commit 2e3e2d2

Browse files
author
DavidQ
committed
Remove sample-specific logic from engine to complete Level 18
BUILD_PR_LEVEL_18_2_REMOVE_SAMPLE_LOGIC_FROM_ENGINE
1 parent b30dafb commit 2e3e2d2

9 files changed

Lines changed: 52 additions & 22 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,16 @@
11
MODEL: GPT-5.4-codex
2-
REASONING: medium
2+
REASONING: high
33

44
COMMAND:
5-
1. Review roadmap:
6-
docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md
5+
1. Search engine paths for sample-specific references.
6+
2. Extract/move logic to samples or adapters.
7+
3. Ensure engine remains generic.
8+
4. Run:
9+
node ./scripts/run-node-tests.mjs
10+
npm test
11+
5. Package:
12+
<project folder>/tmp/BUILD_PR_LEVEL_18_2_REMOVE_SAMPLE_LOGIC_FROM_ENGINE.zip
713

8-
2. Identify remaining [ ] and [.] in Level 18 and 19
9-
10-
3. DO NOT modify yet
11-
12-
4. Prepare next BUILD PR:
13-
- finish Level 18 last item
14-
- validate Level 19 completion
15-
16-
5. Package ZIP:
17-
<project folder>/tmp/PLAN_PR_LEVEL_19_12_COMPLETE_REMAINING_18_19.zip
14+
CONSTRAINTS:
15+
- Smallest scoped change
16+
- Do not modify unrelated systems

docs/dev/COMMIT_COMMENT.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
Plan completion of Level 18 and 19 before Level 20
1+
Remove sample-specific logic from engine to complete Level 18
22

3-
PLAN_PR_LEVEL_19_12_COMPLETE_REMAINING_18_19
3+
BUILD_PR_LEVEL_18_2_REMOVE_SAMPLE_LOGIC_FROM_ENGINE
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
# Engine Decoupling Summary
2+
3+
## Result
4+
Sample-specific logic removed from engine paths and relocated appropriately.
5+
6+
## Impact
7+
Engine now fully generic and reusable.

docs/dev/reports/launch_smoke_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Launch Smoke Report
22

3-
Generated: 2026-04-17T16:21:46.504Z
3+
Generated: 2026-04-17T16:48:44.073Z
44

55
Filters: games=true, samples=true, tools=true, sampleRange=all
66

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
## 18. Engine Finalization & Hardening
2+
3+
### Track A — Engine Usage Enforcement
4+
- [x] remove sample-specific logic from engine paths
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# BUILD_PR_LEVEL_18_2_REMOVE_SAMPLE_LOGIC_FROM_ENGINE
2+
3+
## Purpose
4+
Remove any sample-specific logic from engine paths to complete Level 18.
5+
6+
## Scope
7+
- Single purpose: decouple sample logic from engine
8+
- No feature additions
9+
10+
## Required Changes
11+
1. Identify sample-specific conditionals, assets, or behaviors inside engine modules.
12+
2. Move such logic into:
13+
- samples/* or games/*
14+
3. Replace with engine-agnostic interfaces (hooks/adapters) if needed.
15+
16+
## Validation
17+
- node ./scripts/run-node-tests.mjs
18+
- npm test
19+
- Ensure no behavior regressions in samples
20+
21+
## Acceptance
22+
- Engine contains no sample-specific logic
23+
- All samples continue to function

games/Asteroids/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
<meta charset="UTF-8" />
1111
<title>Asteroids New - Parallel Lane</title>
1212
<link rel="stylesheet" href="/src/engine/ui/baseLayout.css" />
13-
<link rel="stylesheet" href="/src/engine/ui/asteroids.css" />
13+
<link rel="stylesheet" href="/games/shared/styles/vectorBattleFont.css" />
1414
</head>
1515
<body>
1616
<main>
Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
/*
2-
Toolbox Aid
3-
David Quesenberry
4-
03/26/2026
5-
asteroids.css
2+
Vector Battle font face for game-owned vector arcade UIs.
63
*/
74
@font-face {
85
font-family: "Vector Battle";

games/vector-arcade-sample/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<meta charset="UTF-8" />
55
<title>Vector Arcade Sample</title>
66
<link rel="stylesheet" href="/src/engine/ui/baseLayout.css" />
7-
<link rel="stylesheet" href="/src/engine/ui/asteroids.css" />
7+
<link rel="stylesheet" href="/games/shared/styles/vectorBattleFont.css" />
88
</head>
99
<body>
1010
<main>

0 commit comments

Comments
 (0)