Skip to content

Commit 787c42d

Browse files
author
DavidQ
committed
PLAN PR: normalize helper imports to src/shared.
1 parent 350d005 commit 787c42d

7 files changed

Lines changed: 54 additions & 41 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1 @@
1-
# docs/dev/codex_commands.md
2-
3-
No Codex command included.
4-
5-
Reason:
6-
This is an APPLY bundle.
7-
Code execution was reported complete and clean.
8-
Do not send another BUILD for this same PR scope.
1+
No Codex command. PLAN only.

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
APPLY PR: accept shared helper extraction exact helper move after clean Codex execution.
1+
PLAN PR: normalize helper imports to src/shared.

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
Next step:
2-
Select the next PLAN or BUILD in the repo workflow.
3-
This PR is closed at APPLY.
1+
Next: BUILD_PR_SHARED_EXTRACTION_03_IMPORT_NORMALIZATION
Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1 @@
1-
Produced APPLY/acceptance docs for the completed shared helper extraction PR.
2-
Accepted based on clean execution report.
3-
No new code requested.
4-
No new BUILD command included.
1+
Defined import normalization plan.

docs/dev/reports/file_tree.txt

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1 @@
1-
docs/
2-
pr/
3-
APPLY_PR_SHARED_EXTRACTION_02_EXACT_HELPER_MOVE.md
4-
dev/
5-
codex_commands.md
6-
commit_comment.txt
7-
next_command.txt
8-
reports/
9-
change_summary.txt
10-
validation_checklist.txt
11-
file_tree.txt
1+
docs/pr + docs/dev structure
Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,4 @@
1-
SESSION VALIDATION GATE
2-
- Bundle type: APPLY
3-
- One PR purpose only: yes
4-
- Codex command removed: yes
5-
- Fake code claims avoided: yes
6-
- Start-of-day directories touched: no
7-
- Repo scan requested: no
8-
- ZIP repo-structured: yes
9-
10-
APPLY-SPECIFIC CHECKS
11-
- Scope limited to completed BUILD: yes
12-
- No new runtime work requested: yes
13-
- No new BUILD embedded: yes
14-
- Commit-ready docs included: yes
1+
- Imports point to src/shared
2+
- No relative helper imports
3+
- No duplicates
4+
- Build passes
Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,45 @@
1+
# PLAN_PR_SHARED_EXTRACTION_03_IMPORT_NORMALIZATION
2+
3+
## Purpose
4+
Normalize all imports to use src/shared as the single source of truth for extracted helpers.
5+
6+
## Scope
7+
- Replace relative helper imports with src/shared imports
8+
- Remove inline helper references
9+
- No logic changes
10+
- No file moves
11+
- No new helpers
12+
13+
## Rules
14+
- DO NOT scan entire repo blindly
15+
- ONLY update files that import moved helpers
16+
- DO NOT modify engine APIs
17+
- DO NOT rename helpers
18+
19+
## Normalization Targets
20+
21+
### Replace patterns like:
22+
../../utils/...
23+
../utils/...
24+
local helper definitions
25+
26+
### With:
27+
src/shared/utils/numberUtils.js
28+
src/shared/utils/objectUtils.js
29+
src/shared/state/getState.js
30+
31+
## Import Standard
32+
33+
Always use:
34+
import { helper } from "src/shared/...";
35+
36+
## Acceptance Criteria
37+
- All helper imports resolve to src/shared
38+
- No duplicate helper definitions remain
39+
- No relative helper imports remain
40+
- Build passes
41+
42+
## Non-Goals
43+
- No refactoring
44+
- No helper renaming
45+
- No additional extraction

0 commit comments

Comments
 (0)