BUILD PR: centralize readHostStatus/readHostSnapshot across exact engine 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_44_TO_RESULT_DEBUG_COMMAND_BATCH.md exactly.
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH.md exactly.
 Edit only these files:
-- src/engine/debug/shared/debugCommandResultUtils.js (new file)
+- src/engine/debug/network/shared/hostReadUtils.js (new file)
 - src/engine/debug/inspectors/commands/registerInspectorCommands.js
-- tools/dev/presets/debugPresetApplier.js
+- src/engine/debug/network/dashboard/registerDashboardCommands.js
 Do not expand scope.
-Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_44_TO_RESULT_DEBUG_COMMAND_BATCH_delta.zip
+Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-BUILD PR: centralize toResult across exact debug-command batch.
+BUILD PR: centralize readHostStatus/readHostSnapshot across exact engine batch.

docs/dev/NEXT_COMMAND.txt

@@ -1 +1 @@
-Next: run BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH after this batch.
+Next: run BUILD_PR_SHARED_EXTRACTION_46_STRINGIFY_VALUE_INSPECTOR_BATCH after this batch.

docs/dev/reports/change_summary.txt

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

docs/pr/BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH.md

@@ -0,0 +1,93 @@
+# BUILD_PR_SHARED_EXTRACTION_45_READ_HOST_STATUS_SNAPSHOT_BATCH
+
+## Purpose
+Centralize duplicated dashboard/inspector host reader helpers across the exact engine batch identified in the duplicate report.
+
+## Single PR Purpose
+Normalize ONLY these helpers:
+
+- `readHostStatus(host)`
+- `readHostSnapshot(host)`
+
+## Exact Files Allowed
+
+### New shared file
+1. `src/engine/debug/network/shared/hostReadUtils.js`
+
+### Consumer files
+2. `src/engine/debug/inspectors/commands/registerInspectorCommands.js`
+3. `src/engine/debug/network/dashboard/registerDashboardCommands.js`
+
+Do not edit any other file.
+
+## Source of Truth
+This exact scope comes from the provided duplicate report entries for:
+- `function readHostStatus(host)`
+- `function readHostSnapshot(host)`
+
+Only the 2 listed consumer files are in scope.
+
+## Exact Shared Helper Creation
+Create:
+
+`src/engine/debug/network/shared/hostReadUtils.js`
+
+Export exactly:
+- `readHostStatus`
+- `readHostSnapshot`
+
+Implementation rules:
+- use ONE existing local implementation as source-of-truth for each helper
+- do NOT merge logic
+- do NOT generalize behavior
+- preserve signatures exactly:
+  - `readHostStatus(host)`
+  - `readHostSnapshot(host)`
+
+## Exact Consumer Changes
+For each of the 2 listed consumer files:
+
+If the file contains local function definitions matching:
+```js
+function readHostStatus(host)
+function readHostSnapshot(host)
+```
+then:
+- remove the local function definition(s)
+- import the helper(s) from the correct relative path to:
+  - `src/engine/debug/network/shared/hostReadUtils.js`
+- if the file already imports from that module, add the needed helper(s) 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 versions, leave it unchanged.
+
+## Relative Import Rule
+Use the correct relative path from each consumer file to:
+
+`src/engine/debug/network/shared/hostReadUtils.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 dashboard/inspector cleanup
+- no behavior changes
+- keep one PR purpose only
+
+## Validation Checklist
+1. Confirm no more than the 3 listed files changed
+2. Confirm `src/engine/debug/network/shared/hostReadUtils.js` exists and exports:
+   - `readHostStatus`
+   - `readHostSnapshot`
+3. Confirm local function definitions no longer exist in changed listed consumer files
+4. Confirm changed consumer files import the helpers from the correct relative path to `src/engine/debug/network/shared/hostReadUtils.js`
+5. Confirm no unrelated files changed
+6. Confirm no behavior changes were made
+
+## Non-Goals
+- no cleanup of other dashboard or inspector helpers
+- no cleanup outside the 2 listed consumer files
+- no refactor beyond this exact duplicate-removal batch

src/engine/debug/inspectors/commands/registerInspectorCommands.js

@@ -11,18 +11,10 @@ import {
   sanitizeText
 } from "../shared/inspectorUtils.js";
 import { toResult } from "../../shared/debugCommandResultUtils.js";
-
-function readHostStatus(host) {
-  return host && typeof host.getStatus === "function"
-    ? asObject(host.getStatus())
-    : {};
-}
-
-function readHostSnapshot(host) {
-  return host && typeof host.getSnapshot === "function"
-    ? asObject(host.getSnapshot())
-    : {};
-}
+import {
+  readHostSnapshot,
+  readHostStatus
+} from "../../network/shared/hostReadUtils.js";
 
 export function createInspectorCommandPack(options = {}) {
   const source = asObject(options);

src/engine/debug/network/dashboard/registerDashboardCommands.js

@@ -8,6 +8,10 @@ registerDashboardCommands.js
 import { asArray, asObject, sanitizeText } from "../shared/networkDebugUtils.js";
 import { normalizeServerDashboardSnapshot } from "./serverDashboardProviders.js";
 import { listServerDashboardRefreshModes } from "./serverDashboardRefreshModes.js";
+import {
+  readHostSnapshot,
+  readHostStatus
+} from "../shared/hostReadUtils.js";
 
 function toCommandResult(title, lines, code = "DASHBOARD_OK", status = "ready") {
   return {
@@ -18,20 +22,6 @@ function toCommandResult(title, lines, code = "DASHBOARD_OK", status = "ready")
   };
 }
 
-function readHostStatus(host) {
-  if (!host || typeof host.getStatus !== "function") {
-    return {};
-  }
-  return asObject(host.getStatus());
-}
-
-function readHostSnapshot(host) {
-  if (!host || typeof host.getSnapshot !== "function") {
-    return normalizeServerDashboardSnapshot({});
-  }
-  return normalizeServerDashboardSnapshot(host.getSnapshot());
-}
-
 function formatStatusLines(status) {
   return [
     `running=${Boolean(status.running)}`,
@@ -132,7 +122,7 @@ export function createServerDashboardCommandPack(options = {}) {
         summary: "Show normalized dashboard snapshot summary.",
         usage: "dashboard.snapshot",
         handler() {
-          const snapshot = readHostSnapshot(host);
+          const snapshot = normalizeServerDashboardSnapshot(readHostSnapshot(host));
           const players = asArray(snapshot.players);
           const selectedRows = players.slice(0, 5).map((player) => {
             const row = asObject(player);

src/engine/debug/network/shared/hostReadUtils.js

@@ -0,0 +1,13 @@
+import { asObject } from "../../inspectors/shared/inspectorUtils.js";
+
+export function readHostStatus(host) {
+  return host && typeof host.getStatus === "function"
+    ? asObject(host.getStatus())
+    : {};
+}
+
+export function readHostSnapshot(host) {
+  return host && typeof host.getSnapshot === "function"
+    ? asObject(host.getSnapshot())
+    : {};
+}