BUILD PR: switch network_sample_c shared helper imports to @shared alias only.

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,11 +1,10 @@
 MODEL: GPT-5.3-codex
 REASONING: high
 COMMAND:
-Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_15B_ALIAS_CONFIG_BOOTSTRAP_JSCONFIG_ONLY.md exactly.
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_16_ALIAS_IMPORTS_NETWORK_SAMPLE_C.md exactly.
 Edit only these files:
-- jsconfig.json (new file)
-- src/advanced/promotion/createPromotionGate.js
-- src/advanced/state/createWorldGameStateSystem.js
-Fail fast if any required shared target file is missing.
+- games/network_sample_c/game/ReconciliationLayerAdapter.js
+- games/network_sample_c/game/StateTimelineBuffer.js
+Fail fast if jsconfig.json is missing, the @shared alias is not present, or required shared target files are missing.
 Do not expand scope.
-Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_15B_ALIAS_CONFIG_BOOTSTRAP_JSCONFIG_ONLY_delta.zip
+Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_16_ALIAS_IMPORTS_NETWORK_SAMPLE_C_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-BUILD PR: add minimal jsconfig @shared alias bootstrap and switch advanced shared imports only.
+BUILD PR: switch network_sample_c shared helper imports to @shared alias only.

docs/dev/reports/change_summary.txt

@@ -1 +1 @@
-Replaced the blocked alias BUILD with an execution-grade retry that explicitly creates a minimal jsconfig.json and switches only the two advanced files to @shared imports.
+Built the next executable alias rollout step, limited to network_sample_c only and assuming the existing @shared alias bootstrap.

docs/dev/reports/file_tree.txt

@@ -1,6 +1,6 @@
 docs/
   pr/
-    BUILD_PR_SHARED_EXTRACTION_15B_ALIAS_CONFIG_BOOTSTRAP_JSCONFIG_ONLY.md
+    BUILD_PR_SHARED_EXTRACTION_16_ALIAS_IMPORTS_NETWORK_SAMPLE_C.md
   dev/
     codex_commands.md
     commit_comment.txt

docs/dev/reports/validation_checklist.txt

@@ -1,9 +1,8 @@
 SESSION VALIDATION GATE
 - Bundle type: BUILD
 - One PR purpose only: yes
-- Exact files listed: yes
-- Exact new file content listed: yes
-- Prior failure cause addressed directly: yes
+- Exact target files listed: yes
+- Config untouched by design: yes
 - Fail-fast gate explicit: yes
 - Scope minimized: yes
 - ZIP repo-structured and execution-ready: yes

docs/pr/BUILD_PR_SHARED_EXTRACTION_16_ALIAS_IMPORTS_NETWORK_SAMPLE_C.md

@@ -0,0 +1,108 @@
+# BUILD_PR_SHARED_EXTRACTION_16_ALIAS_IMPORTS_NETWORK_SAMPLE_C
+
+## Purpose
+Extend the proven `@shared/*` alias usage to the network_sample_c consumer slice only.
+
+## Single PR Purpose
+Switch shared-helper imports in these 2 files from relative paths to the existing `@shared/*` alias:
+
+1. `games/network_sample_c/game/ReconciliationLayerAdapter.js`
+2. `games/network_sample_c/game/StateTimelineBuffer.js`
+
+This BUILD assumes the prior alias bootstrap is already present and does not modify config.
+
+## Exact Files Allowed
+Edit only these 2 files:
+
+1. `games/network_sample_c/game/ReconciliationLayerAdapter.js`
+2. `games/network_sample_c/game/StateTimelineBuffer.js`
+
+Do not edit any other file.
+
+## Alias Assumption
+The repo already contains alias support for:
+
+```js
+@shared/
+```
+
+resolving to:
+
+```js
+src/shared/
+```
+
+Do not modify `jsconfig.json` or any other config file in this PR.
+
+## Fail-Fast Gate
+Before making source edits, confirm:
+
+1. `jsconfig.json` exists
+2. it contains `@shared/*` -> `src/shared/*`
+3. these shared targets exist:
+   - `src/shared/utils/numberUtils.js`
+   - `src/shared/utils/objectUtils.js`
+
+If any condition is false:
+- stop
+- report blocker
+- make no changes
+- do not create a delta ZIP
+
+## Exact Source Changes
+
+### 1) `games/network_sample_c/game/ReconciliationLayerAdapter.js`
+Replace only shared-helper relative imports with alias imports.
+
+Target alias forms:
+```js
+import { asFiniteNumber, asPositiveInteger } from '@shared/utils/numberUtils.js';
+import { isPlainObject } from '@shared/utils/objectUtils.js';
+```
+
+Rules:
+- preserve existing imported specifiers
+- if imports from the same aliased module can be combined without changing behavior/style, minimum-edit combination is allowed
+- do not touch unrelated imports
+- do not change logic
+- do not add snapshot helper import unless this file already uses it
+
+### 2) `games/network_sample_c/game/StateTimelineBuffer.js`
+Replace only shared-helper relative imports with alias imports.
+
+Target alias forms:
+```js
+import { asPositiveInteger } from '@shared/utils/numberUtils.js';
+import { isPlainObject } from '@shared/utils/objectUtils.js';
+```
+
+Rules:
+- preserve existing imported specifiers
+- if imports from the same aliased module can be combined without changing behavior/style, minimum-edit combination is allowed
+- do not touch unrelated imports
+- do not change logic
+- do not add `asFiniteNumber` unless this file already imports/uses it
+
+## Hard Constraints
+- edit only the 2 listed files
+- no config changes
+- no engine changes
+- no advanced-file changes
+- no repo-wide alias rollout
+- no removal of `.js` extensions
+- no helper behavior changes
+- keep one PR purpose only
+
+## Validation Checklist
+1. Confirm only the 2 listed files changed
+2. Confirm both files now import shared helpers via `@shared/...`
+3. Confirm no unrelated imports changed
+4. Confirm no logic changed
+5. Confirm no config file was touched
+
+## Non-Goals
+- no alias rollout to engine
+- no alias rollout repo-wide
+- no import cleanup outside the 2 exact files
+- no toolchain redesign
+- no helper redesign

games/network_sample_c/game/ReconciliationLayerAdapter.js

@@ -5,7 +5,7 @@ David Quesenberry
 ReconciliationLayerAdapter.js
 */
 import StateTimelineBuffer from "./StateTimelineBuffer.js";
-import { asFiniteNumber, asPositiveInteger } from "../../../src/shared/utils/numberUtils.js";
+import { asFiniteNumber, asPositiveInteger } from "@shared/utils/numberUtils.js";
 
 const APPROVED_PUBLIC_EVENT_TYPES = new Set([
   "worldState.transition.applied",

games/network_sample_c/game/StateTimelineBuffer.js

@@ -5,7 +5,7 @@ David Quesenberry
 StateTimelineBuffer.js
 */
 
-import { asPositiveInteger } from "../../../src/shared/utils/numberUtils.js";
+import { asPositiveInteger } from "@shared/utils/numberUtils.js";
 
 function normalizeFrameId(frameId) {
   const numeric = Number(frameId);

tools/shared/powerShell/find_dupes_called.ps1

@@ -0,0 +1,23 @@
+# 1. Get all .js files recursively
+# 2. Look for function patterns
+# 3. Group them by the text found
+# 4. Filter for groups with more than 1 occurrence
+
+# .\find_dupes_called.ps1 > found_dupes_called.txt
+# .\find_dupes_called.ps1 | Out-File -FilePath "found_dupes_called.txt" -Encoding utf8
+
+# Goes up 3 levels to reach HTML-JavaScript-Gaming from tools\shared\powerShell\
+Get-ChildItem -Path "$PSScriptRoot\..\..\..\" -Recurse -Filter *.js | 
+    Select-String -Pattern "function\s+[a-zA-Z0-9_]*\(", "[a-zA-Z0-9_]*\s*=\s*function", "[a-zA-Z0-9_]*:\s*function" |
+    Group-Object Line | 
+    Where-Object { $_.Count -gt 1 } | 
+    ForEach-Object {
+        # Print the duplicate line and the count
+        "($($_.Count)) Duplicate line: $($_.Name.Trim())"
+        
+        # Loop through each occurrence in the group to show where it is
+        $_.Group | ForEach-Object {
+            "   -> Line $($_.LineNumber): $($_.Path)"
+        }
+        "" # Adds a blank line for better readability
+    }