Optimize redundant string operations and array iterations

[Copilot]

Identified and fixed several inefficient code patterns causing unnecessary CPU cycles and memory allocations.

Changes

Server-side stderr processing (server/src/index.ts)

• Cache chunk.toString() result instead of calling 3× per chunk
• Reuse trimmed string for case conversion instead of converting original chunk

// Before: 3 toString() calls per chunk
if (chunk.toString().includes("MODULE_NOT_FOUND")) { ... }
let message = chunk.toString().trim();
let ucMsg = chunk.toString().toUpperCase();

// After: 1 toString() call
const chunkStr = chunk.toString();
if (chunkStr.includes("MODULE_NOT_FOUND")) { ... }
let message = chunkStr.trim();
const ucMsg = message.toUpperCase();

Client-side JSON parsing (client/src/components/JsonView.tsx)

• Eliminate duplicate tryParseJson() invocation in useMemo by caching parse result

Schema normalization (client/src/utils/schemaUtils.ts)

• Replace multiple .some() iterations with single-pass type detection
• Reduces O(n×m) to O(n) where n=array length, m=type checks
• Consolidates 96 lines to 33 lines

Array operations (client/src/utils/jsonUtils.ts)

• Extend arrays in-place instead of creating intermediate copies

Header processing (server/src/index.ts)

• Hoist header lookup outside conditional to avoid redundant access

Original prompt

Identify and suggest improvements to slow or inefficient code

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.