centralize ensureArray helper

Author: DavidQ

MASTER_ROADMAP_HIGH_LEVEL_status_only_updated.md

@@ -1,11 +1,6 @@
 MODEL: GPT-5.3-codex
 REASONING: high
 COMMAND:
-Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_35_NETWORK_DEBUG_SERIALIZATION_BATCH.md exactly.
-Edit only these files:
-- src/shared/utils/networkDebugUtils.js (new file)
-- games/network_sample_a/debug/networkSampleADebug.js
-- games/network_sample_b/debug/networkSampleBDebug.js
-- games/network_sample_c/debug/networkSampleCDebug.js
-Do not expand scope.
-Package the delta output to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_35_NETWORK_DEBUG_SERIALIZATION_BATCH_delta.zip
+Execute docs/pr/BUILD_PR_SHARED_EXTRACTION_36_ARRAY_UTILS_ENSURE_ARRAY_BATCH.md exactly.
+Edit only listed files.
+Package to <project folder>/tmp/BUILD_PR_SHARED_EXTRACTION_36_ARRAY_UTILS_ENSURE_ARRAY_BATCH_delta.zip

UPDATED_MASTER_ROADMAP_HIGH_LEVEL_v2.md

@@ -1 +1 @@
-BUILD PR: centralize network debug serialization helpers across exact network debug batch.
+centralize ensureArray helper

docs/dev/CODEX_COMMANDS.md

@@ -1 +1 @@
-Next: continue duplicate-family extraction from the provided duplicate report.
+Next: string utils batch

docs/dev/COMMIT_COMMENT.txt

@@ -1 +1 @@
-Execution-grade batch for network debug serialization helpers using the exact duplicate-report file list.
+ensureArray centralized

docs/dev/NEXT_COMMAND.txt

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

docs/dev/reports/change_summary.txt

@@ -0,0 +1,31 @@
+# BUILD_PR_SHARED_EXTRACTION_36_ARRAY_UTILS_ENSURE_ARRAY_BATCH
+
+## Purpose
+Centralize duplicated `ensureArray(value)` helper across debug/tools domains.
+
+## Single PR Purpose
+Normalize ONLY:
+
+- ensureArray(value)
+
+## Exact Files Allowed
+
+### Shared
+1. src/shared/utils/arrayUtils.js
+
+### Consumers (from dupes report)
+2. tools/dev/devConsoleIntegration.js
+3. tools/dev/inspectors/inspectorStore.js
+4. tools/shared/runtimeAssetLoader.js
+5. tools/shared/vectorGeometryRuntime.js
+
+## Rules
+- remove local ensureArray implementations
+- import from shared
+- no behavior change
+- no scope expansion
+
+## Validation
+- helper exists once
+- imports correct
+- no local duplicates remain

docs/dev/reports/validation_checklist.txt

@@ -0,0 +1,3 @@
+export function ensureArray(value) {
+  return Array.isArray(value) ? value : [];
+}

docs/pr/BUILD_PR_SHARED_EXTRACTION_36_ARRAY_UTILS_ENSURE_ARRAY_BATCH.md

@@ -22,6 +22,7 @@ import { createToggleCommandPack } from "./commandPacks/toggleCommandPack.js";
 import { createValidationCommandPack } from "./commandPacks/validationCommandPack.js";
 import { createInspectorStore } from "./inspectors/inspectorStore.js";
 import { createDebugPluginRegistry } from "./plugins/debugPluginSystem.js";
+import { ensureArray } from "../../src/shared/utils/arrayUtils.js";
 import { cloneJson } from "../../src/shared/utils/jsonUtils.js";
 
 import {
@@ -178,12 +179,12 @@ function isPrintableCharacter(key) {
 
 function flattenOverlaySections(sections, maxLines = 9) {
   const lines = [];
-  const source = Array.isArray(sections) ? sections : [];
+  const source = ensureArray(sections);
 
   for (let sectionIndex = 0; sectionIndex < source.length; sectionIndex += 1) {
     const section = source[sectionIndex];
     lines.push(`${sanitizeText(section?.title) || "Panel"}:`);
-    const sectionLines = Array.isArray(section?.lines) ? section.lines : [];
+    const sectionLines = ensureArray(section?.lines);
     for (let lineIndex = 0; lineIndex < sectionLines.length; lineIndex += 1) {
       lines.push(`  ${sanitizeText(sectionLines[lineIndex])}`);
       if (lines.length >= maxLines) {
@@ -277,7 +278,7 @@ export function createSampleGameDevConsoleIntegration(options = {}) {
     limits: isObject(options?.pluginLimits) ? options.pluginLimits : {}
   });
   let pluginActivityReports = [];
-  const pluginDescriptors = Array.isArray(options?.plugins) ? options.plugins : [];
+  const pluginDescriptors = ensureArray(options?.plugins);
   const pluginBootstrapReports = pluginRegistry.registerPlugins(
     pluginDescriptors,
     options?.activatePluginsOnInit === true
@@ -390,7 +391,7 @@ export function createSampleGameDevConsoleIntegration(options = {}) {
   }
 
   function pushConsoleOutputLines(lines) {
-    const source = Array.isArray(lines) ? lines : [];
+    const source = ensureArray(lines);
     source.forEach((line) => pushConsoleOutputLine(line));
   }
 
@@ -406,7 +407,7 @@ export function createSampleGameDevConsoleIntegration(options = {}) {
     const names = new Set(["help", "status"]);
     if (typeof commandRegistry?.listCommands === "function") {
       const commands = commandRegistry.listCommands();
-      const source = Array.isArray(commands) ? commands : [];
+      const source = ensureArray(commands);
       source.forEach((entry) => {
         const name = sanitizeText(typeof entry === "string" ? entry : entry?.name);
         if (name) {
@@ -418,7 +419,7 @@ export function createSampleGameDevConsoleIntegration(options = {}) {
   }
 
   function findLongestCommonPrefix(values) {
-    const source = Array.isArray(values) ? values.filter(Boolean) : [];
+    const source = ensureArray(values).filter(Boolean);
     if (source.length === 0) {
       return "";
     }
@@ -892,7 +893,7 @@ export function createSampleGameDevConsoleIntegration(options = {}) {
     const diagnosticsContext = isObject(frame.diagnosticsContext) ? frame.diagnosticsContext : {};
     const diagnosticsResult = runtime.collectDiagnostics(diagnosticsContext);
     diagnosticsSnapshot = diagnosticsResult?.diagnostics || null;
-    diagnosticsReports = Array.isArray(diagnosticsResult?.reports) ? diagnosticsResult.reports.slice() : [];
+    diagnosticsReports = ensureArray(diagnosticsResult?.reports).slice();
     inspectorStore.update({
       diagnosticsSnapshot: diagnosticsSnapshot || {},
       diagnosticsContext,