Skip to content

Commit dbdb8e5

Browse files
author
DavidQ
committed
Move remaining string normalizers into shared utils
1 parent a19210e commit dbdb8e5

7 files changed

Lines changed: 35 additions & 29 deletions

docs/dev/CODEX_COMMANDS.md

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

44
COMMAND:
5-
Execute exactly docs/pr/BUILD_PR_REPO_STRUCTURE_66_MOVE_MAP_SANITIZETEXT_TO_SHARED_STRINGUTILS.md.
6-
Modify only the exact target files listed in the PR doc.
7-
Do not expand scope.
8-
Package the delta zip to:
9-
<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_66_MOVE_MAP_SANITIZETEXT_TO_SHARED_STRINGUTILS_delta.zip
5+
Execute docs/pr/BUILD_PR_REPO_STRUCTURE_67_MOVE_MAP_STRING_NORMALIZERS_TO_SHARED.md exactly.
6+
Modify only listed files.
7+
Package to:
8+
<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_67_MOVE_MAP_STRING_NORMALIZERS_TO_SHARED_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Move sanitizeText into src/shared/utils/stringUtils.js and update the exact tool consumers in the shared/tools boundary lane.
1+
Move remaining string normalizers into shared utils

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Next: inspect remaining shared/tools boundary candidates and build the next exact slice only if the target files are explicit.
1+
Next: evaluate remaining shared/tools boundary candidates
Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1 @@
1-
- continue shared/tools boundary enforcement with one exact reusable string helper
2-
- move canonical sanitizeText ownership into shared string utils
3-
- update only the listed tool consumers; preserve engine-facing behavior
1+
string normalization consolidation

docs/dev/reports/file_tree.txt

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1 @@
1-
docs/pr/BUILD_PR_REPO_STRUCTURE_66_MOVE_MAP_SANITIZETEXT_TO_SHARED_STRINGUTILS.md
2-
docs/dev/codex_commands.md
3-
docs/dev/commit_comment.txt
4-
docs/dev/next_command.txt
5-
docs/dev/reports/file_tree.txt
6-
docs/dev/reports/change_summary.txt
7-
docs/dev/reports/validation_checklist.txt
8-
src/engine/debug/inspectors/shared/inspectorUtils.js
9-
src/shared/utils/stringUtils.js
10-
tools/dev/commandPacks/groupCommandPack.js
11-
tools/dev/commandPacks/overlayCommandPack.js
12-
tools/dev/commandPacks/inspectorCommandPack.js
13-
tools/dev/presets/debugPresetApplier.js
1+
inspectorUtils + stringUtils only
Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1 @@
1-
- single PR purpose only
2-
- exact target files only
3-
- no broad repo scan
4-
- no unrelated helper moves
5-
- no behavior change
6-
- imports/exports resolve after the move
1+
exact files only, no behavior change
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
# BUILD PR — Repo Structure Move Map (String Normalizers to Shared)
2+
3+
## Purpose
4+
Continue shared/tools boundary enforcement by consolidating remaining string normalization helpers into the shared string utils layer.
5+
6+
## Exact Target Files
7+
- src/engine/debug/inspectors/shared/inspectorUtils.js
8+
- src/shared/utils/stringUtils.js
9+
10+
## Required Code Changes
11+
1. In src/shared/utils/stringUtils.js
12+
- add/export any remaining normalization helpers found in inspectorUtils (trimSafe, normalizeWhitespace, etc.)
13+
14+
2. In src/engine/debug/inspectors/shared/inspectorUtils.js
15+
- remove local implementations of those helpers
16+
- import from shared string utils instead
17+
18+
## Constraints
19+
- exact files only
20+
- no other tools touched in this PR
21+
- no behavior changes
22+
- no refactor
23+
24+
## Acceptance Criteria
25+
- string normalization helpers live only in shared utils
26+
- inspectorUtils delegates to shared

0 commit comments

Comments
 (0)