Skip to content

Lock landing survival on MCP connect failure (CL-5600) - #458

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-5600-mcp-connection-errors-interrupt-the-default-loading-sequence
Aug 10, 2026
Merged

Lock landing survival on MCP connect failure (CL-5600)#458
TheGreatAxios merged 1 commit into
mainfrom
cl-5600-mcp-connection-errors-interrupt-the-default-loading-sequence

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Product path already routes MCP failures through surfaceSystemNotice so the landing mountain stays mounted.
  • Strengthens runtime-channels.test.ts to assert landing survival (isLanding, empty streamLog, statusFlash, mark-row stability) so a regression cannot reintroduce wipe-via-appendStreamRow without failing CI.

Closes CL-5600

Test plan

  • bun test src/tui/landing.test.ts src/tui/runtime-channels.test.ts
  • CI green

Product code already routes mcp.status failures through surfaceSystemNotice
(CL-5618). Strengthen the product-host channel test so a re-route through
appendStreamRow cannot wipe the mountain without failing the suite.
@linear-code

linear-code Bot commented Aug 10, 2026

Copy link
Copy Markdown

CL-5600

@TheGreatAxios
TheGreatAxios merged commit c00a11d into main Aug 10, 2026
2 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-5600-mcp-connection-errors-interrupt-the-default-loading-sequence branch August 10, 2026 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant