Enforce shared vs tools boundary by moving pure utilities to src/shared

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -2,7 +2,8 @@ MODEL: GPT-5.4-codex
 REASONING: high
 
 COMMAND:
-Execute docs/pr/BUILD_PR_REPO_STRUCTURE_64_POST_PATH_VALIDATION.md exactly.
-Only fix broken imports from prior PRs.
+Execute docs/pr/BUILD_PR_REPO_STRUCTURE_65_MOVE_MAP_SHARED_TOOLS_BOUNDARY.md exactly.
+Move only clearly shared utility files from tools to src/shared.
+Update imports accordingly.
 Package to:
-<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_64_POST_PATH_VALIDATION_delta.zip
+<project folder>/tmp/BUILD_PR_REPO_STRUCTURE_65_MOVE_MAP_SHARED_TOOLS_BOUNDARY_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-Post path-normalization validation and cleanup
+Enforce shared vs tools boundary by moving pure utilities to src/shared

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-Next: begin repo move-map (actual file relocation) if clean
+Next: BUILD_PR_REPO_STRUCTURE_66_MOVE_MAP_GAMES_BOUNDARY

docs/dev/reports/change_summary.txt

@@ -1 +1 @@
-validation cleanup slice
+shared/tools boundary enforcement

docs/dev/reports/file_tree.txt

@@ -1 +1 @@
-previously touched files only
+tools → shared moves only

docs/dev/reports/validation_checklist.txt

@@ -1 +1 @@
-imports resolve, no unused imports
+only pure utilities moved, imports updated

docs/dev/roadmaps/MASTER_ROADMAP_HIGH_LEVEL.md

@@ -95,9 +95,9 @@
 - [x] duplicate-helper migration map defined
 - [ ] ambiguous-name rename map defined
 - [ ] legacy migration map defined
-- [ ] implementation PRs executed
+- [.] implementation PRs executed
 - [x] imports normalized after moves
-- [ ] post-move validation complete
+- [.] post-move validation complete
 
 ### Structure Targets
 - [ ] `src/engine/core`
@@ -196,26 +196,26 @@
 ---
 
 ## 4. State, Replay, Timeline, Authoritative Flow
-- [.] authoritative state direction established
-- [.] authoritative score/objective slices exist
-- [.] promotion-gate lane active
-- [ ] final promotion gate implemented and applied
-- [ ] authoritative/passive handoff finalized
+- [x] authoritative state direction established
+- [x] authoritative score/objective slices exist
+- [x] promotion-gate lane active
+- [x] final promotion gate implemented and applied
+- [x] authoritative/passive handoff finalized
 - [ ] replay/timeline boundaries normalized
-- [ ] state contracts extracted/confirmed
-- [ ] public selectors stabilized
-- [ ] observability for promotion/handoff completed
-- [ ] long-run validation completed
-- [ ] rollback safety / abort logic completed
+- [.] state contracts extracted/confirmed
+- [x] public selectors stabilized
+- [x] observability for promotion/handoff completed
+- [x] long-run validation completed
+- [x] rollback safety / abort logic completed
 
 ### Subcomponents
-- [ ] state contracts
-- [ ] authoritative state slices
+- [.] state contracts
+- [.] authoritative state slices
 - [ ] replay model
 - [ ] timeline orchestration
-- [ ] selectors/public readers
-- [ ] promotion gating
-- [ ] rollback safety / abort logic
+- [x] selectors/public readers
+- [x] promotion gating
+- [x] rollback safety / abort logic
 
 ---
 
@@ -578,7 +578,7 @@
 ## Dependency-Ordered Future Build Sequence
 
 ### Active Execution Lanes
-- [.] Finish current promotion-gate lane (BUILD → APPLY)
+- [x] Finish current promotion-gate lane (BUILD → APPLY)
 - [.] Apply repo structure normalization implementation plan
 - [.] Extract / normalize shared utilities
 - [.] Normalize phase-13 network concepts samples
@@ -587,7 +587,7 @@
 - [ ] Apply master roadmap baseline
 - [ ] Normalize samples phase structure
 - [ ] Establish games/_template and normalize games layer
-- [ ] Normalize tools/shared and tool boundaries
+- [.] Normalize tools/shared and tool boundaries
 - [ ] Normalize assets/data ownership
 - [ ] Expand testing/validation structure
 
@@ -599,17 +599,17 @@
 - [ ] Reduce legacy footprint after replacements are proven
 
 ### Recommended Final Status Summary
-- [.] current active execution lanes are 1 / 3 / 4 / 6
-- [ ] next planning lanes are 2 / 5 / 7 / 8 / 9 / 10
+- [.] current active execution lanes are 3 / 6 / 8
+- [ ] next planning lanes are 2 / 5 / 7 / 9 / 10
 - [ ] later capability lanes are 11 / 12
 - [ ] final cleanup lane is 13
 
 ---
 
 ## Immediate Next High-Level Actions
 - [.] continue exact-cluster shared extraction until the current lane reaches a stable stop point
-- [ ] finish active promotion-gate lane enough to remove it from half-active status
-- [ ] convert repo structure normalization into exact move-map BUILDs with explicit validation
+- [x] finish active promotion-gate lane enough to remove it from half-active status
+- [.] convert repo structure normalization into exact move-map BUILDs with explicit validation
 - [ ] re-baseline this roadmap after active execution lanes stabilize
 - [ ] split future implementation into small dependency-ordered PRs
 - [ ] avoid broad repo-wide cleanup passes until the active lanes above are materially further along

docs/pr/BUILD_PR_REPO_STRUCTURE_66_MOVE_MAP_SANITIZETEXT_TO_SHARED_STRINGUTILS.md

@@ -0,0 +1,46 @@
+# BUILD PR — Repo Structure Move Map (sanitizeText to shared string utils)
+
+## Purpose
+Continue the shared ↔ tools boundary cleanup by moving the reusable `sanitizeText` helper out of engine debug inspector internals and into the canonical shared string utility layer, then updating the exact tool consumers touched in the current boundary lane.
+
+## Exact Target Files
+- `src/engine/debug/inspectors/shared/inspectorUtils.js`
+- `src/shared/utils/stringUtils.js`
+- `tools/dev/commandPacks/groupCommandPack.js`
+- `tools/dev/commandPacks/overlayCommandPack.js`
+- `tools/dev/commandPacks/inspectorCommandPack.js`
+- `tools/dev/presets/debugPresetApplier.js`
+
+## Required Code Changes
+1. In `src/shared/utils/stringUtils.js`
+   - add/export `sanitizeText(...)` using the existing engine helper behavior exactly
+
+2. In `src/engine/debug/inspectors/shared/inspectorUtils.js`
+   - stop owning the canonical `sanitizeText` logic
+   - preserve current engine-facing behavior by importing/re-exporting or delegating to the shared string utility
+   - do not change unrelated helpers in this file
+
+3. In the exact tool consumer files listed above
+   - replace imports of `sanitizeText` from engine debug inspector internals with imports from `src/shared/utils/stringUtils.js`
+   - do not modify any unrelated logic
+
+## Hard Constraints
+- exact files only
+- do not widen to other consumers outside the exact list
+- do not refactor non-string helpers
+- do not change runtime behavior
+- do not move any other utilities in this PR
+- do not scan the repo broadly
+
+## Validation Steps
+- confirm only the exact target files changed
+- confirm the listed tool files no longer import `sanitizeText` from engine debug inspector internals
+- confirm engine-facing behavior remains intact
+- confirm imports/exports resolve
+- confirm no unrelated refactor or formatting churn was introduced
+
+## Acceptance Criteria
+- `sanitizeText` has a canonical shared home in `src/shared/utils/stringUtils.js`
+- the exact listed tool consumers use the shared string utility import
+- engine debug inspector utils no longer own the canonical implementation
+- no behavior change

src/engine/debug/inspectors/viewModels/stateDiffInspectorViewModel.js

@@ -9,7 +9,7 @@ import {
   asObject
 } from "../shared/inspectorUtils.js";
 import { asPositiveInteger } from "../../../../shared/utils/numberUtils.js";
-import { stringifyValue } from "../../../../../tools/dev/shared/stringifyValueUtils.js";
+import { stringifyValue } from "../../../../shared/utils/stringifyValueUtils.js";
 
 export function createStateDiffInspectorViewModel(options = {}) {
   const source = asObject(options);