BUILD PR: consolidate duplicate definitions and export consistency inside shared util files only.

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,13 +1,10 @@
 MODEL: GPT-5.3-codex
 REASONING: high
 COMMAND:
-Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_09_CLEAN_UNUSED_HELPER_IMPORTS_ADVANCED_ONLY.md exactly.
-
-Edit ONLY:
-- src/advanced/promotion/createPromotionGate.js
-- src/advanced/state/createWorldGameStateSystem.js
-
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_10_SHARED_UTIL_CONSOLIDATION_CHECK.md exactly.
+Edit only these files:
+- src/shared/utils/numberUtils.js
+- src/shared/utils/objectUtils.js
+Fail fast if either file does not exist.
 Do not expand scope.
-
-Package to:
-<project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_09_CLEAN_UNUSED_HELPER_IMPORTS_ADVANCED_ONLY_delta.zip
+Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_10_SHARED_UTIL_CONSOLIDATION_CHECK_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-BUILD PR: remove unused helper imports after shared extraction (advanced modules only).
+BUILD PR: consolidate duplicate definitions and export consistency inside shared util files only.

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-Next: continue executable BUILD chain
+Next: if clean, continue with the next executable BUILD. If blocked, capture the exact blocker and narrow further without changing PR purpose.

docs/dev/reports/change_summary.txt

@@ -1 +1 @@
-Cleanup pass for unused imports after extraction steps.
+Built the next executable shared-extraction step focused only on internal consistency of the shared utility files.

docs/dev/reports/file_tree.txt

@@ -1 +1,11 @@
-docs only
+docs/
+  pr/
+    BUILD_PR_SHARED_EXTRACTION_10_SHARED_UTIL_CONSOLIDATION_CHECK.md
+  dev/
+    codex_commands.md
+    commit_comment.txt
+    next_command.txt
+    reports/
+      change_summary.txt
+      validation_checklist.txt
+      file_tree.txt

docs/dev/reports/validation_checklist.txt

@@ -1 +1,8 @@
-Unused imports removed only.
+SESSION VALIDATION GATE
+- Bundle type: BUILD
+- One PR purpose only: yes
+- Exact target files listed: yes
+- Fail-fast rule explicit: yes
+- No consumer changes allowed: yes
+- No guessing: yes
+- ZIP repo-structured and execution-ready: yes

docs/pr/BUILD_PR_SHARED_EXTRACTION_10_SHARED_UTIL_CONSOLIDATION_CHECK.md

@@ -0,0 +1,79 @@
+# BUILD_PR_SHARED_EXTRACTION_10_SHARED_UTIL_CONSOLIDATION_CHECK
+
+## Purpose
+Validate and enforce consistency inside the existing shared utility files only.
+
+## Single PR Purpose
+Consolidate duplicate helper definitions and export consistency ONLY inside:
+
+- `src/shared/utils/numberUtils.js`
+- `src/shared/utils/objectUtils.js`
+
+Do not touch any consumer file.
+
+## Exact Files Allowed
+Edit only these 2 files:
+
+1. `src/shared/utils/numberUtils.js`
+2. `src/shared/utils/objectUtils.js`
+
+Do not edit any other file.
+
+## Exact Allowed Actions
+
+### In `src/shared/utils/numberUtils.js`
+Allowed:
+- confirm `asFiniteNumber` is exported exactly once
+- confirm `asPositiveInteger` is exported exactly once
+- if duplicate definitions of either helper exist within this same file, remove duplicates and keep one canonical implementation
+- if both named exports and bottom export lists are present, make the minimum change needed so each helper is exported exactly once
+
+Not allowed:
+- no renaming
+- no behavior changes
+- no new helper additions
+- no consumer/import updates
+
+### In `src/shared/utils/objectUtils.js`
+Allowed:
+- confirm `isPlainObject` is exported exactly once
+- if duplicate definitions of `isPlainObject` exist within this same file, remove duplicates and keep one canonical implementation
+- if both named exports and bottom export lists are present, make the minimum change needed so the helper is exported exactly once
+
+Not allowed:
+- no renaming
+- no behavior changes
+- no new helper additions
+- no consumer/import updates
+
+## Fail-Fast Rule
+If either shared utility file does not exist:
+- stop
+- report blocker
+- make no changes
+- do not produce a delta ZIP
+
+## Hard Constraints
+- do not edit any file other than the 2 listed above
+- do not change helper behavior
+- do not change import paths
+- do not touch advanced files
+- do not touch engine files
+- do not perform repo-wide cleanup
+- keep one PR purpose only
+
+## Validation Checklist
+1. Confirm only the 2 listed files changed
+2. Confirm `src/shared/utils/numberUtils.js` exists
+3. Confirm `src/shared/utils/objectUtils.js` exists
+4. Confirm `asFiniteNumber` is exported exactly once from `src/shared/utils/numberUtils.js`
+5. Confirm `asPositiveInteger` is exported exactly once from `src/shared/utils/numberUtils.js`
+6. Confirm `isPlainObject` is exported exactly once from `src/shared/utils/objectUtils.js`
+7. Confirm no helper behavior was changed
+8. Confirm no consumer files were touched
+
+## Non-Goals
+- no consumer cleanup
+- no import normalization
+- no new shared file creation
+- no refactor beyond duplicate-removal/export-consistency inside the 2 exact files