BUILD PR: centralize toResult across exact debug-command batch.

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_38_CODE_TO_LETTER_HIGHSCORE_BATCH.md exactly.
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_44_TO_RESULT_DEBUG_COMMAND_BATCH.md exactly.
 Edit only these files:
-- src/shared/utils/initialsEntryUtils.js (new file)
-- games/Asteroids/systems/AsteroidsInitialsEntry.js
-- games/SpaceDuel/game/SpaceDuelInitialsEntry.js
-- games/SpaceInvaders/game/SpaceInvadersInitialsEntry.js
+- src/engine/debug/shared/debugCommandResultUtils.js (new file)
+- src/engine/debug/inspectors/commands/registerInspectorCommands.js
+- tools/dev/presets/debugPresetApplier.js
 Do not expand scope.
-Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_38_CODE_TO_LETTER_HIGHSCORE_BATCH_delta.zip
+Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_44_TO_RESULT_DEBUG_COMMAND_BATCH_delta.zip

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-BUILD PR: centralize codeToLetter across exact initials-entry batch.
+BUILD PR: centralize toResult across exact debug-command batch.

docs/dev/NEXT_COMMAND.txt

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

docs/dev/reports/change_summary.txt

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

docs/dev/reports/validation_checklist.txt

@@ -1 +1 @@
-Exact file list present; one PR purpose only; no guessing.
+Exact file list present; exact helper list present; one PR purpose only.

docs/pr/BUILD_PR_SHARED_EXTRACTION_44_TO_RESULT_DEBUG_COMMAND_BATCH.md

@@ -0,0 +1,92 @@
+# BUILD_PR_SHARED_EXTRACTION_44_TO_RESULT_DEBUG_COMMAND_BATCH
+
+## Purpose
+Centralize duplicated `toResult(status, title, code, lines, details = {})` implementations across the exact debug-command batch identified in the duplicate report.
+
+## Single PR Purpose
+Normalize ONLY this helper:
+
+- `toResult(status, title, code, lines, details = {})`
+
+## Exact Files Allowed
+
+### New shared file
+1. `src/engine/debug/shared/debugCommandResultUtils.js`
+
+### Consumer files
+2. `src/engine/debug/inspectors/commands/registerInspectorCommands.js`
+3. `tools/dev/presets/debugPresetApplier.js`
+
+Do not edit any other file.
+
+## Source of Truth
+This exact scope comes from the provided duplicate report entry for:
+
+`function toResult(status, title, code, lines, details = {})`
+
+Only the 2 listed consumer files are in scope.
+
+## Exact Shared Helper Creation
+Create:
+
+`src/engine/debug/shared/debugCommandResultUtils.js`
+
+Export exactly:
+
+```js
+export function toResult(status, title, code, lines, details = {}) {
+  // 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:
+  - `toResult(status, title, code, lines, details = {})`
+
+## Exact Consumer Changes
+For each of the 2 listed consumer files:
+
+If the file contains a local function definition matching:
+```js
+function toResult(status, title, code, lines, details = {})
+```
+then:
+- remove the local `toResult(...)` function definition
+- import `toResult` from the correct relative path to:
+  - `src/engine/debug/shared/debugCommandResultUtils.js`
+- if the file already imports from that module, add `toResult` 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 `toResult`, leave it unchanged.
+
+## Relative Import Rule
+Use the correct relative path from each consumer file to:
+
+`src/engine/debug/shared/debugCommandResultUtils.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 debug helper 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/shared/debugCommandResultUtils.js` exists and exports `toResult`
+3. Confirm local `function toResult(...)` definitions no longer exist in changed listed consumer files
+4. Confirm changed consumer files import `toResult` from the correct relative path to `src/engine/debug/shared/debugCommandResultUtils.js`
+5. Confirm no unrelated files changed
+6. Confirm no behavior changes were made
+
+## Non-Goals
+- no cleanup of `createResult`
+- no cleanup outside the 2 listed consumer files
+- no refactor beyond this exact duplicate-removal batch

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

@@ -10,16 +10,7 @@ import {
   asObject,
   sanitizeText
 } from "../shared/inspectorUtils.js";
-
-function toResult(status, title, code, lines, details = {}) {
-  return {
-    status: status === "failed" ? "failed" : "ready",
-    title,
-    code,
-    lines: asArray(lines).map((line) => String(line)),
-    details: asObject(details)
-  };
-}
+import { toResult } from "../../shared/debugCommandResultUtils.js";
 
 function readHostStatus(host) {
   return host && typeof host.getStatus === "function"

src/engine/debug/shared/debugCommandResultUtils.js

@@ -0,0 +1,14 @@
+import {
+  asArray,
+  asObject
+} from "../inspectors/shared/inspectorUtils.js";
+
+export function toResult(status, title, code, lines, details = {}) {
+  return {
+    status: status === "failed" ? "failed" : "ready",
+    title,
+    code,
+    lines: asArray(lines).map((line) => String(line)),
+    details: asObject(details)
+  };
+}

tools/dev/presets/debugPresetApplier.js

@@ -6,9 +6,9 @@ debugPresetApplier.js
 */
 
 import { sanitizeText } from "../../../src/engine/debug/inspectors/shared/inspectorUtils.js";
-import { isObject } from "../../../src/shared/utils/objectUtils.js";
 import { cloneJson } from "../../../src/shared/utils/jsonUtils.js";
 import { getRuntimeAndRegistry } from "../shared/runtimeRegistryUtils.js";
+import { toResult } from "../../../src/engine/debug/shared/debugCommandResultUtils.js";
 
 function getAllPanels(panelRegistry) {
   const panels = panelRegistry.getOrderedPanels(true);
@@ -38,16 +38,6 @@ function toPanelMap(panelRegistry) {
   return map;
 }
 
-function toResult(status, title, code, lines, details = {}) {
-  return {
-    status: status === "failed" ? "failed" : "ready",
-    title,
-    code,
-    lines: Array.isArray(lines) ? lines : [],
-    details: isObject(details) ? details : {}
-  };
-}
-
 export class DebugPresetApplier {
   constructor(options = {}) {
     this.presetRegistry = options.presetRegistry || null;