From 5f36b79117c7d4dc5523de0735a2ce2b63ae87df Mon Sep 17 00:00:00 2001 From: brennanb2025 Date: Wed, 13 May 2026 16:13:27 -0700 Subject: [PATCH] test(e2e): replace node -e with printf for OSC title marker Same root cause as the prior commit's BEL fix: CI Linux launches the shell with PATH stripped of /usr/bin, so `node` is unresolvable and the marker bytes never reach xterm. printf is a shell builtin and emits the same OSC sequence. Co-authored-by: Orca --- tests/e2e/terminal-attention.spec.ts | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/e2e/terminal-attention.spec.ts b/tests/e2e/terminal-attention.spec.ts index b885c606c5c..e120615631f 100644 --- a/tests/e2e/terminal-attention.spec.ts +++ b/tests/e2e/terminal-attention.spec.ts @@ -184,11 +184,9 @@ test.describe('Terminal attention', () => { // async PTY pipeline. await emitBell(orcaPage, activePtyId) const MARKER_TITLE = 'focused-tab-bell-marker' - await execInTerminal( - orcaPage, - activePtyId, - `node -e "process.stdout.write('\\u001b]0;${MARKER_TITLE}\\u0007')"` - ) + // Why: printf is a shell builtin so it works even when CI launches the + // shell with a stripped PATH (no /usr/bin → no `node` resolvable). + await execInTerminal(orcaPage, activePtyId, `printf '\\033]0;${MARKER_TITLE}\\007'`) await expect .poll(