Skip to content

Commit b588ff7

Browse files
author
DavidQ
committed
Rename Puckman → Pacman across repo
BUILD_PR_LEVEL_19_9_RENAME_PUCKMAN_TO_PACMAN
1 parent b0fb62a commit b588ff7

38 files changed

Lines changed: 279 additions & 238 deletions

docs/dev/CODEX_COMMANDS.md

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

44
COMMAND:
5-
1. Run:
6-
node tools/dev/checkSharedExtractionGuard.mjs
7-
8-
2. Capture output differences.
9-
10-
3. Locate baseline file (likely in tools/dev or reports).
11-
12-
4. Determine:
13-
- Are new shared entries valid? If YES → update baseline
14-
- If NO → fix classification logic
15-
16-
5. Apply smallest change:
17-
- update baseline JSON OR
18-
- adjust guard filters
19-
20-
6. Run:
21-
npm test
22-
23-
7. Confirm:
24-
pretest passes (no baseline drift)
25-
26-
8. Package:
27-
<project folder>/tmp/BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE.zip
5+
1. Search repo for "Puckman"
6+
2. Replace all occurrences with "Pacman"
7+
3. Rename files/folders if needed
8+
4. Fix any import paths
9+
5. Run tests:
10+
node ./scripts/run-node-tests.mjs
11+
6. Package ZIP:
12+
<project folder>/tmp/BUILD_PR_LEVEL_19_9_RENAME_PUCKMAN_TO_PACMAN.zip
2813

2914
CONSTRAINTS:
30-
- Do not modify unrelated systems
31-
- Do not remove guard
32-
- Preserve intent of guard
15+
- No logic changes
16+
- Keep changes minimal and focused

docs/dev/COMMIT_COMMENT.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
Fix shared extraction guard baseline drift
1+
Rename Puckman → Pacman across repo
22

3-
BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE
3+
BUILD_PR_LEVEL_19_9_RENAME_PUCKMAN_TO_PACMAN
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# Pacman Naming Correction
2+
3+
## Issue
4+
Repo contained mixed naming:
5+
- Pacman (original arcade internal name)
6+
- Pacman (standard public name)
7+
8+
## Resolution
9+
Standardized to "Pacman" across entire repo.
10+
11+
## Result
12+
- Consistent naming
13+
- Cleaner documentation
14+
- Avoid confusion

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-17T15:32:24.952Z
3+
Generated: 2026-04-17T15:45:00.839Z
44

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
Phase 08 status correction after BUILD_PR_LEVEL_08_04_PUCKMAN_BOUNDARY_NORMALIZATION
1+
Phase 08 status correction after BUILD_PR_LEVEL_08_04_PACMAN_BOUNDARY_NORMALIZATION
22

33
Earned now
4-
- [x] puckman normalized
4+
- [x] pacman normalized
55

66
Still not earned repo-wide
77
- [ ] game flow pattern standardized (`flow/attract.js`, `flow/intro.js`, `flow/highscore.js`)
@@ -11,5 +11,5 @@ Still not earned repo-wide
1111
- [ ] future games follow template-first path
1212

1313
Notes
14-
- This PR normalizes Puckman only.
14+
- This PR normalizes Pacman only.
1515
- Repo-wide completion markers remain unchanged unless globally satisfied.

docs/dev/reports/puckman_boundary_model.txt

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
1-
PR: BUILD_PR_LEVEL_08_04_PUCKMAN_BOUNDARY_NORMALIZATION
1+
PR: BUILD_PR_LEVEL_08_04_PACMAN_BOUNDARY_NORMALIZATION
22

3-
Puckman boundary normalization (smallest scope)
4-
- Added game-local boundary folders under games/Puckman.
5-
- Added rules source of truth at games/Puckman/rules/gameFlowRules.js.
3+
Pacman boundary normalization (smallest scope)
4+
- Added game-local boundary folders under games/Pacman.
5+
- Added rules source of truth at games/Pacman/rules/gameFlowRules.js.
66
- Added flow modules consuming rules constants:
7-
- games/Puckman/flow/attract.js
8-
- games/Puckman/flow/intro.js
9-
- games/Puckman/flow/highscore.js
7+
- games/Pacman/flow/attract.js
8+
- games/Pacman/flow/intro.js
9+
- games/Pacman/flow/highscore.js
1010
- Added gameplay runtime/scene modules consuming rules constants:
11-
- games/Puckman/game/PuckmanRuntime.js
12-
- games/Puckman/game/PuckmanGameScene.js
11+
- games/Pacman/game/PacmanRuntime.js
12+
- games/Pacman/game/PacmanGameScene.js
1313

1414
Boundary ownership
1515
- flow: descriptors only, no duplicated constants

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# MASTER ROADMAP - HIGH LEVEL (v7 ADDITIVE)
1+
# MASTER ROADMAP - HIGH LEVEL (v7 ADDITIVE)
22

33
## Status Key
44
- [x] complete
@@ -232,7 +232,7 @@
232232
### Track A - Debug Foundation
233233
- [x] Dev Console (input + command execution)
234234
- [x] Debug Overlay (visual panels)
235-
- [x] Console ↔ Overlay Boundary
235+
- [x] Console ↔ Overlay Boundary
236236
- [x] Panel Registry
237237
- [x] Data Providers (read-only model)
238238
- [x] Operator Commands (control surface)
@@ -409,7 +409,7 @@
409409
- [x] game-specific asset ownership enforced
410410
- [x] shared-vs-game utility boundaries enforced
411411
- [x] space_invaders normalized
412-
- [x] puckman normalized
412+
- [x] pacman normalized
413413
- [x] future games follow template-first path
414414

415415
---
@@ -707,7 +707,7 @@
707707
## 18. Engine Finalization & Hardening (Deferred Execution)
708708

709709
> Purpose: convert the built system into a clean, stable, production-grade engine after all major capability tracks are complete.
710-
> This is NOT a scaffolding phase it is a consolidation + enforcement phase.
710+
> This is NOT a scaffolding phase — it is a consolidation + enforcement phase.
711711
712712
### Scope Rules
713713
- additive only (no blind refactors)
@@ -721,36 +721,36 @@
721721
[ ] Clean boundaries
722722
[ ] Document contracts
723723

724-
### Track A Engine Usage Enforcement
724+
### Track A — Engine Usage Enforcement
725725
- [x] verify all `samples/` use engine systems (no local reimplementation)
726726
- [.] verify all `games/` use engine systems
727727
- [x] migrate any local logic into engine/shared where appropriate
728728
- [ ] remove sample-specific logic from engine paths
729729

730-
### Track B Boundary Hardening
730+
### Track B — Boundary Hardening
731731
- [.] enforce engine vs shared vs game vs tool boundaries
732732
- [.] eliminate cross-layer leakage
733733
- [.] validate dependency direction rules across repo
734734
- [ ] remove accidental coupling
735735

736-
### Track C Contract Stabilization
736+
### Track C — Contract Stabilization
737737
- [.] finalize engine public APIs
738738
- [.] finalize shared contracts
739739
- [ ] ensure selectors/providers are stable
740740
- [.] remove unstable or experimental surfaces
741741

742-
### Track D Codebase Consistency
742+
### Track D — Codebase Consistency
743743
- [ ] single class per file enforcement
744744
- [ ] remove duplicate helpers
745745
- [ ] normalize naming consistency
746746
- [ ] eliminate import/export anti-patterns
747747

748-
### Track E CSS & UI Normalization
748+
### Track E — CSS & UI Normalization
749749
- [ ] flatten CSS layers
750750
- [ ] enforce shared UI classes
751751
- [ ] remove redundant styles
752752

753-
### Track F Docs System Cleanup
753+
### Track F — Docs System Cleanup
754754
[ ] Docs organization: classify all `./docs/` into buckets.
755755

756756
[ ] Arrange docs into classification buckets.
@@ -761,15 +761,15 @@
761761
- [ ] remove move-only historical docs (after validation)
762762
- [ ] align docs to feature-based structure
763763

764-
### Track G Repo Hygiene
764+
### Track G — Repo Hygiene
765765
[ ] Remove imports to export (should not be import x, export x)
766766
[ ] Other than templates (games/samples/tools), remove the .keep file, if the folder is empty, delete
767767
- [ ] remove unnecessary `.keep` files
768768
- [ ] remove empty folders
769769
- [ ] validate folder ownership rules
770770
- [ ] enforce clean repo structure
771771

772-
### Track H PR Consolidation Strategy
772+
### Track H — PR Consolidation Strategy
773773
- [ ] bundle related PRs into capability-level units
774774
- [ ] reduce multi-PR fragmentation
775775
- [ ] ensure each PR represents a complete capability
@@ -786,7 +786,7 @@
786786
- no experimental work
787787
- only stabilization, validation, and extensibility
788788

789-
### Track A System Integration Validation
789+
### Track A — System Integration Validation
790790
- [.] validate all major systems working together:
791791
- rendering
792792
- input
@@ -797,45 +797,45 @@
797797
- [.] verify no hidden coupling
798798
- [.] verify predictable system interaction
799799

800-
### Track B Runtime Lifecycle Validation
801-
- [ ] validate boot run shutdown lifecycle
800+
### Track B — Runtime Lifecycle Validation
801+
- [ ] validate boot ? run ? shutdown lifecycle
802802
- [ ] validate hot reload / reset flows
803803
- [ ] validate error handling paths
804804
- [ ] validate long-running stability
805805

806-
### Track C Performance & Scaling
806+
### Track C — Performance & Scaling
807807
- [.] validate large scene performance
808808
- [.] validate stress scenarios (1k+ entities)
809809
- [.] validate memory stability
810810
- [.] identify bottlenecks
811811

812-
### Track D Debug & Observability Maturity
812+
### Track D — Debug & Observability Maturity
813813
- [ ] ensure all systems expose debug data
814814
- [x] ensure providers are complete and consistent
815815
- [x] validate debug panels across systems
816816
- [x] confirm production-safe debug toggling
817817

818-
### Track E Toolchain Validation
818+
### Track E — Toolchain Validation
819819
- [ ] verify tools integrate cleanly with engine
820820
- [ ] validate asset pipelines end-to-end
821-
- [ ] validate editor runtime consistency
821+
- [ ] validate editor ? runtime consistency
822822
- [ ] confirm no tool-specific logic leaks into engine
823823

824-
### Track F Sample & Game Validation
824+
### Track F — Sample & Game Validation
825825
- [.] Organize/rebuild `samples/` and `games/` as if newly constructed, with proper classes/data in proper folders.
826826
- [.] Simulated code (for example, some network samples) should be converted to real networks, with tests as needed.
827827
- [.] verify all samples still function correctly
828828
- [ ] verify curriculum progression remains intact
829829
- [ ] validate games follow template strictly
830830
- [ ] confirm no regression across phases
831831

832-
### Track G Extensibility Readiness
832+
### Track G — Extensibility Readiness
833833
- [x] validate plugin/extension patterns
834834
- [x] validate adding new systems is clean
835835
- [x] validate external integration points
836836
- [x] ensure future phases can build cleanly
837837

838-
### Track H Final Stability Gate
838+
### Track H — Final Stability Gate
839839
- [x] full-repo validation sweep
840840
- [x] zero regression requirement
841841
- [x] contract freeze readiness

docs/pr/BUILD_PR_LEVEL_08_04_PUCKMAN_BOUNDARY_NORMALIZATION.md

Lines changed: 23 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
# BUILD_PR_LEVEL_08_04_PUCKMAN_BOUNDARY_NORMALIZATION
1+
# BUILD_PR_LEVEL_08_04_PACMAN_BOUNDARY_NORMALIZATION
22

33
## Purpose
4-
Normalize **Puckman** to the Phase 08 games-layer boundary model only.
4+
Normalize **Pacman** to the Phase 08 games-layer boundary model only.
55

66
This PR is intentionally narrow:
77
- no engine edits
@@ -20,37 +20,37 @@ What is now true:
2020
What is still not true:
2121
- repo-wide flow standardization is not earned
2222
- shared-vs-game utility enforcement is not earned
23-
- Puckman normalization is still not earned
23+
- Pacman normalization is still not earned
2424
- full current-games migration is still not earned
2525

2626
The smallest valid next move is:
27-
1. normalize **Puckman** to the same game-local boundary model used for focused game migrations,
27+
1. normalize **Pacman** to the same game-local boundary model used for focused game migrations,
2828
2. keep the PR game-local and surgical,
2929
3. update Phase 08 status only where actually earned.
3030

3131
## Required boundary target
32-
Puckman should follow this ownership model:
32+
Pacman should follow this ownership model:
3333

34-
- `games/Puckman/flow/*`
34+
- `games/Pacman/flow/*`
3535
- attract / intro / highscore flow modules when those states exist
3636
- no duplicated rule constants
37-
- `games/Puckman/game/*`
37+
- `games/Pacman/game/*`
3838
- runtime orchestration and scene wiring only
39-
- `games/Puckman/entities/*`
39+
- `games/Pacman/entities/*`
4040
- player, ghost, pellet, fruit, tunnel, collision-domain entities
41-
- `games/Puckman/levels/*`
41+
- `games/Pacman/levels/*`
4242
- maze/layout/start-state progression definitions
43-
- `games/Puckman/rules/*`
43+
- `games/Pacman/rules/*`
4444
- authoritative game-local constants and flow/rules contracts
45-
- `games/Puckman/assets/*` and platform-local assets
46-
- only Puckman-owned media/content
47-
- `games/Puckman/systems/*` and `games/Puckman/utils/*`
45+
- `games/Pacman/assets/*` and platform-local assets
46+
- only Pacman-owned media/content
47+
- `games/Pacman/systems/*` and `games/Pacman/utils/*`
4848
- game-specific helpers only
4949

5050
## Implementation goals
5151
Codex should make only the smallest changes needed to achieve the following:
5252

53-
1. Create or normalize a Puckman rules surface
53+
1. Create or normalize a Pacman rules surface
5454
- add a game-local rules module if missing
5555
- move duplicated flow/gameplay constants there
5656
- make flow + gameplay import from one source of truth
@@ -65,7 +65,7 @@ Codex should make only the smallest changes needed to achieve the following:
6565
- keep scene/runtime files focused on orchestration
6666

6767
4. Preserve game-local ownership
68-
- do not move Puckman assets/utilities into shared unless they are already clearly cross-game
68+
- do not move Pacman assets/utilities into shared unless they are already clearly cross-game
6969
- do not pull engine utilities into the game folder
7070

7171
5. Update status truthfully
@@ -82,14 +82,14 @@ Codex should make only the smallest changes needed to achieve the following:
8282
- no Phase 09+ work
8383

8484
## Expected touched areas
85-
- `games/Puckman/flow/*`
86-
- `games/Puckman/game/*`
87-
- `games/Puckman/rules/*`
88-
- optional: `games/Puckman/entities/*`, `levels/*`, `systems/*`, `utils/*`
85+
- `games/Pacman/flow/*`
86+
- `games/Pacman/game/*`
87+
- `games/Pacman/rules/*`
88+
- optional: `games/Pacman/entities/*`, `levels/*`, `systems/*`, `utils/*`
8989
- roadmap/status docs only if already part of the repo’s active tracking files
9090

9191
## Acceptance criteria
92-
- Puckman has a clear local boundary split for flow/game/rules
92+
- Pacman has a clear local boundary split for flow/game/rules
9393
- no duplicated flow-rule constants across flow and gameplay
9494
- gameplay scene/runtime files consume rules constants instead of defining them
9595
- no engine files changed
@@ -108,14 +108,14 @@ Still not earned from the uploaded PRs:
108108
- [ ] gameplay/entities/levels/rules/assets boundaries normalized
109109
- [ ] shared-vs-game utility boundaries enforced
110110
- [ ] space_invaders normalized
111-
- [ ] puckman normalized
111+
- [ ] pacman normalized
112112
- [ ] future games follow template-first path
113113

114114
## Suggested commit title
115-
`build(games): normalize puckman boundaries for phase 08`
115+
`build(games): normalize pacman boundaries for phase 08`
116116

117117
## After this PR
118-
Phase 08 should be reassessed strictly from repo truth. Likely remaining work after Puckman will be:
118+
Phase 08 should be reassessed strictly from repo truth. Likely remaining work after Pacman will be:
119119
- Space Invaders normalization, if not already fully implemented in repo
120120
- shared-vs-game utility boundary enforcement
121121
- final truthful status alignment

docs/pr/BUILD_PR_LEVEL_08_05_SPACE_INVADERS_COMPLETION.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# BUILD_PR — LEVEL 08.05 SPACE INVADERS COMPLETION
22

33
## Purpose
4-
Complete Space Invaders boundary normalization to match Asteroids and Puckman.
4+
Complete Space Invaders boundary normalization to match Asteroids and Pacman.
55

66
## Scope
77
- Only Space Invaders game folder

0 commit comments

Comments
 (0)