fix(test): widen the replay-callback meta type for snapshot dimensions

The replay meta now carries snapshotCols/snapshotRows, but seven local callback
types in the test file still declared only clearBeforeReplay, so the web
typecheck failed on two assignment sites while the suites themselves passed.
This commit is contained in:
Brennan Benson
2026-08-14 16:12:38 -07:00
parent 3b03789893
commit db34fa92e5
@@ -309,7 +309,10 @@ type ConnectCallbacks = {
data: string,
meta?: { seq?: number; rawLength?: number; background?: boolean; droppedOutput?: boolean }
) => void
onReplayData?: (data: string, meta?: { clearBeforeReplay?: boolean }) => void
onReplayData?: (
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void
onError?: (msg: string) => void
onWriteUnavailable?: () => void
onOutputPauseChanged?: (paused: boolean, supported: boolean) => void
@@ -9524,7 +9527,12 @@ describe('connectPanePty', () => {
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; pendingEscapeTailAnsi?: string }
meta?: {
clearBeforeReplay?: boolean
pendingEscapeTailAnsi?: string
snapshotCols?: number
snapshotRows?: number
}
) => void)
| null
} = { current: null }
@@ -17304,7 +17312,12 @@ describe('connectPanePty', () => {
enableActiveRuntimeEnvironment()
const transport = createMockTransport('remote:env-1@@terminal-1')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null
@@ -17342,7 +17355,12 @@ describe('connectPanePty', () => {
enableActiveRuntimeEnvironment()
const transport = createMockTransport('remote:env-1@@terminal-1')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null
@@ -17381,7 +17399,12 @@ describe('connectPanePty', () => {
enableActiveRuntimeEnvironment()
const transport = createMockTransport('remote:env-1@@terminal-1')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null
@@ -17415,7 +17438,12 @@ describe('connectPanePty', () => {
enableActiveRuntimeEnvironment()
const transport = createMockTransport('remote:env-1@@terminal-1')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null
@@ -17457,7 +17485,12 @@ describe('connectPanePty', () => {
const { connectPanePty } = await import('./pty-connection')
const transport = createMockTransport('pty-id')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null
@@ -17492,7 +17525,12 @@ describe('connectPanePty', () => {
const { connectPanePty } = await import('./pty-connection')
const transport = createMockTransport('pty-id')
const capturedReplayCallback: {
current: ((data: string, meta?: { clearBeforeReplay?: boolean }) => void) | null
current:
| ((
data: string,
meta?: { clearBeforeReplay?: boolean; snapshotCols?: number; snapshotRows?: number }
) => void)
| null
} = { current: null }
transport.connect.mockImplementation(async ({ callbacks }: { callbacks: ConnectCallbacks }) => {
capturedReplayCallback.current = callbacks.onReplayData ?? null