BUILD PR: centralize stringifyValue across exact inspector/command-pack batch.

Author: DavidQ

docs/dev/CODEX_COMMANDS.md

@@ -1,10 +1,10 @@
 MODEL: GPT-5.3-codex
 REASONING: high
 COMMAND:
-Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH.md exactly.
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH.md exactly.
 Edit only these files:
-- src/engine/debug/network/shared/hostReadUtils.js (new file)
-- src/engine/debug/inspectors/commands/registerInspectorCommands.js
-- src/engine/debug/network/dashboard/registerDashboardCommands.js
+- tools/dev/shared/stringifyValueUtils.js (new file)
+- src/engine/debug/inspectors/viewModels/stateDiffInspectorViewModel.js
+- tools/dev/commandPacks/inspectorCommandPack.js
 Do not expand scope.
-Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH_delta.zip
+Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-BUILD PR: centralize readHostStatus/readHostSnapshot across exact engine batch.
+BUILD PR: centralize stringifyValue across exact inspector/command-pack batch.

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-Next: run BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH after this batch.
+Next: continue duplicate-family extraction from the provided duplicate report.

docs/dev/reports/change_summary.txt

@@ -1 +1 @@
-Execution-grade batch for host reader helpers using the exact duplicate-report file list.
+Execution-grade batch for stringifyValue using the exact duplicate-report file list.

docs/pr/BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH.md

@@ -0,0 +1,91 @@
+# BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH
+
+## Purpose
+Centralize duplicated `stringifyValue(value)` implementations across the exact inspector/command-pack batch identified in the duplicate report.
+
+## Single PR Purpose
+Normalize ONLY this helper:
+
+- `stringifyValue(value)`
+
+## Exact Files Allowed
+
+### New shared file
+1. `tools/dev/shared/stringifyValueUtils.js`
+
+### Consumer files
+2. `src/engine/debug/inspectors/viewModels/stateDiffInspectorViewModel.js`
+3. `tools/dev/commandPacks/inspectorCommandPack.js`
+
+Do not edit any other file.
+
+## Source of Truth
+This exact scope comes from the provided duplicate report entry for:
+
+`function stringifyValue(value)`
+
+Only the 2 listed consumer files are in scope.
+
+## Exact Shared Helper Creation
+Create:
+
+`tools/dev/shared/stringifyValueUtils.js`
+
+Export exactly:
+
+```js
+export function stringifyValue(value) {
+  // copy one existing implementation exactly
+}
+```
+
+Implementation rules:
+- use ONE existing local implementation as source-of-truth
+- do NOT merge logic
+- do NOT generalize behavior
+- preserve the function signature exactly:
+  - `stringifyValue(value)`
+
+## Exact Consumer Changes
+For each of the 2 listed consumer files:
+
+If the file contains a local function definition matching:
+```js
+function stringifyValue(value)
+```
+then:
+- remove the local `stringifyValue(value)` function definition
+- import `stringifyValue` from the correct relative path to:
+  - `tools/dev/shared/stringifyValueUtils.js`
+- if the file already imports from that module, add `stringifyValue` with the minimum safe edit
+- do not duplicate imports
+- do not touch unrelated helpers
+- do not change logic
+
+If a listed file already imports and uses shared `stringifyValue`, leave it unchanged.
+
+## Relative Import Rule
+Use the correct relative path from each consumer file to:
+
+`tools/dev/shared/stringifyValueUtils.js`
+
+Do not use aliases.
+Do not change `.js` extension usage.
+
+## Hard Constraints
+- no files outside the 2 listed consumers plus the one new shared file
+- no repo-wide stringify helper cleanup
+- no behavior changes
+- keep one PR purpose only
+
+## Validation Checklist
+1. Confirm no more than the 3 listed files changed
+2. Confirm `tools/dev/shared/stringifyValueUtils.js` exists and exports `stringifyValue`
+3. Confirm local `function stringifyValue(value)` definitions no longer exist in changed listed consumer files
+4. Confirm changed consumer files import `stringifyValue` from the correct relative path to `tools/dev/shared/stringifyValueUtils.js`
+5. Confirm no unrelated files changed
+6. Confirm no behavior changes were made
+
+## Non-Goals
+- no cleanup outside the 2 listed consumer files
+- no refactor beyond this exact duplicate-removal batch

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

@@ -9,15 +9,7 @@ import {
   asObject
 } from "../shared/inspectorUtils.js";
 import { asPositiveInteger } from "../../../../shared/utils/numberUtils.js";
-
-function stringifyValue(value) {
-  try {
-    const encoded = JSON.stringify(value);
-    return typeof encoded === "string" ? encoded : String(value);
-  } catch (_error) {
-    return String(value);
-  }
-}
+import { stringifyValue } from "../../../../../tools/dev/shared/stringifyValueUtils.js";
 
 export function createStateDiffInspectorViewModel(options = {}) {
   const source = asObject(options);

tools/dev/commandPacks/inspectorCommandPack.js

@@ -13,6 +13,7 @@ import {
   standardDetails,
   toLinePair
 } from "./packUtils.js";
+import { stringifyValue } from "../shared/stringifyValueUtils.js";
 
 import { sanitizeText } from "../../../src/engine/debug/inspectors/shared/inspectorUtils.js";
 
@@ -40,15 +41,6 @@ function findEntity(inspectors, entityId) {
   return entities.find((entry) => entry.id === id) || null;
 }
 
-function stringifyValue(value) {
-  try {
-    const json = JSON.stringify(value);
-    return typeof json === "string" ? json : String(value);
-  } catch (_error) {
-    return String(value);
-  }
-}
-
 function formatTime(timestamp) {
   return Number.isFinite(timestamp)
     ? new Date(timestamp).toISOString()

tools/dev/shared/stringifyValueUtils.js

@@ -0,0 +1,8 @@
+export function stringifyValue(value) {
+  try {
+    const encoded = JSON.stringify(value);
+    return typeof encoded === "string" ? encoded : String(value);
+  } catch (_error) {
+    return String(value);
+  }
+}