From 8397dfeb3eed43d20db3e4809db52614bdcc3100 Mon Sep 17 00:00:00 2001 From: Neil <4138956+nwparker@users.noreply.github.com> Date: Mon, 27 Jul 2026 13:30:49 -0700 Subject: [PATCH] test(terminal): note where the xterm gate-order double is pinned for real The unit double encodes RenderService's paused-then-latch gate order, which can drift on an xterm upgrade. Point at the e2e oracle that pins the same order against the real renderer, so a future upgrade has a trail to the authoritative check. Refs STA-2694. Co-authored-by: Orca --- .../pane-manager/reveal-repaint-synchronized-output.test.ts | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/renderer/src/lib/pane-manager/reveal-repaint-synchronized-output.test.ts b/src/renderer/src/lib/pane-manager/reveal-repaint-synchronized-output.test.ts index 28a726f470b..4d3519726b7 100644 --- a/src/renderer/src/lib/pane-manager/reveal-repaint-synchronized-output.test.ts +++ b/src/renderer/src/lib/pane-manager/reveal-repaint-synchronized-output.test.ts @@ -16,8 +16,10 @@ import type { ManagedPaneInternal } from './pane-manager-types' type GateState = { synchronizedOutput: boolean; rendered: number } -// Mirrors RenderService.refreshRows (RenderService.ts:155-181): paused first, -// then the synchronized-output buffer gate, then the actual render. +// Mirrors RenderService.refreshRows: paused first, then the synchronized-output +// buffer gate, then the actual render. This double can drift from xterm on an +// upgrade, so the gate order it encodes is independently pinned against the real +// renderer in tests/e2e/terminal-reveal-draw-command-probe.spec.ts. function createGatedPane(options: { synchronizedOutput: boolean }): { pane: ManagedPaneInternal gate: GateState