Skip to content

Commit 662b0f7

Browse files
author
DavidQ
committed
Post path-normalization validation and cleanup
1 parent 78c2bc2 commit 662b0f7

8 files changed

+55
-8
lines changed

docs/dev/CODEX_COMMANDS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ MODEL: GPT-5.4-codex
22
REASONING: high
33

44
COMMAND:
5-
Execute docs/pr/BUILD_PR_REPO_STRUCTURE_63_MOVE_MAP_GAMES_PATHS.md exactly.
6-
Only update incorrect games import paths.
5+
Execute docs/pr/BUILD_PR_REPO_STRUCTURE_64_POST_PATH_VALIDATION.md exactly.
6+
Only fix broken imports from prior PRs.
77
Package to:
8-
<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_63_MOVE_MAP_GAMES_PATHS_delta.zip
8+
<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_64_POST_PATH_VALIDATION_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Normalize remaining games import paths
1+
Post path-normalization validation and cleanup

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Next: inspect whether a post-path validation or exact move-map file relocation slice is needed.
1+
Next: begin repo move-map (actual file relocation) if clean
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
games path normalization slice
1+
validation cleanup slice

docs/dev/reports/file_tree.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
targeted games import updates only
1+
previously touched files only
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
no broad scan, exact replacements only
1+
imports resolve, no unused imports
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# BUILD PR — Repo Structure Post Path Validation
2+
3+
## Purpose
4+
Validate that all import path normalization changes are correct and no broken imports remain.
5+
6+
## Exact Target Files
7+
- ONLY files already modified in previous path normalization PRs (59–63)
8+
9+
## Required Code Changes
10+
- Fix any broken imports detected
11+
- Remove unused imports created by path changes
12+
13+
## Constraints
14+
- DO NOT introduce new paths
15+
- DO NOT scan unrelated files
16+
- DO NOT refactor logic
17+
- DO NOT move files
18+
19+
## Acceptance Criteria
20+
- All imports resolve
21+
- No console/module resolution errors
22+
- No unused imports remain
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# BUILD PR — Repo Structure Move Map (Shared ↔ Tools Boundary)
2+
3+
## Purpose
4+
Enforce boundary between src/shared and tools by moving pure utility logic out of tools into shared.
5+
6+
## Exact Target Files
7+
- tools/** files that contain reusable, non-UI logic (identified during execution)
8+
- src/shared/** destination only (no new structure invention)
9+
10+
## Required Code Changes
11+
- MOVE qualifying utility code from tools → src/shared
12+
- UPDATE imports for moved files only
13+
14+
## Constraints
15+
- ONLY move files that are:
16+
- pure logic
17+
- not UI-bound
18+
- DO NOT refactor code
19+
- DO NOT change behavior
20+
- DO NOT scan entire repo — operate only on clearly identified files
21+
22+
## Acceptance Criteria
23+
- no shared logic remains incorrectly inside tools
24+
- moved files resolve correctly
25+
- imports updated and working

0 commit comments

Comments
 (0)