fix(server): strip terminal escape sequences from opencode CLI output

[Exotic209093]

Problem

opencode <= 1.18 emits an OSC title set (ESC ]0;<cwd>: ready BEL) on stdout even when stdout is a pipe. The escape bytes leak into everything parsed from its CLI output: parseServerUrlFromOutput misses the ready line, model slugs gain a prefix, agent names pick up garbage, and skill JSON fails to decode.

Fix

Add a shared helper, @t3tools/shared/stripTerminalEscapes, that removes OSC sequences (terminated by BEL or ST) and CSI sequences while leaving ordinary text alone (unterminated prefixes are preserved). Apply it in opencodeRuntime.ts before parsing server output, models, agents, and skills. Since JSON.stringify encodes control bytes textually inside JSON values, all three CLI parsers also strip decoded string values via a JSON.parse reviver — model fields at any depth, agent permission rules, and skill metadata — while non-string values and object/array structure pass through untouched.

Tests: direct unit tests for the helper plus parser coverage for raw-byte titles, escapes embedded in decoded model/permission/skill fields, and unterminated-sequence preservation.

Note

Strip terminal escape sequences from opencode CLI output parsers

• Adds stripTerminalEscapes to @t3tools/shared, which removes OSC and CSI sequences from strings, exported via the new ./stripTerminalEscapes subpath
• Applies the helper and a stripTerminalEscapesJsonReviver to parseServerUrlFromOutput, parseModelsCliOutput, parseAgentListCliOutput, and parseSkillsCliOutput so CLI output is cleaned before line splitting and JSON decoding
• parseSkillsCliOutput now uses JSON.parse with the reviver before schema validation (decodeOpenCodeSkillsArrayExit) instead of decoding from JSON text, returning an empty array on parse failure
• Risk: parseSkillsCliOutput no longer validates raw JSON text via schema; malformed input returns [] instead of throwing, so downstream callers expecting schema errors on bad input will see an empty array

^{Macroscope summarized 842a0dd.}

[macroscopeapp]

Approvability

Verdict: Skipped

Macroscope did not run approvability analysis for this PR. Macroscope could not determine whether this PR modifies its approvability configuration, so the PR was not approved automatically. A PR that may change the rules that govern approval is never approved automatically.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9a4dd94c-1a3f-4ca4-8d13-5cb758b88c32

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}