Skip to content

test(mcp): cover startup summary states#396

Merged
esengine merged 1 commit intoesengine:mainfrom
ctharvey:test/mcp-startup-summary-states
May 8, 2026
Merged

test(mcp): cover startup summary states#396
esengine merged 1 commit intoesengine:mainfrom
ctharvey:test/mcp-startup-summary-states

Conversation

@ctharvey
Copy link
Copy Markdown
Contributor

@ctharvey ctharvey commented May 7, 2026

Part of #196 — MCP cleanup roadmap item 1.

Summary

  • add startup-state coverage at the chatCommand() seam
  • cover the three startup handoff states the issue thread called out: live bridged, disabled with no live summary, and unbridged startup failure
  • preserve configured MCP specs at startup so /mcp can still surface configured-but-unbridged servers on the marketplace fallback path

Why this shape

The issue comment asked for startup summary state coverage before the larger UI/view-model extractions. This keeps the slice narrow and behavior-focused: one small startup handoff fix in chat.tsx, plus focused tests for the state transitions the MCP hub depends on.

Testing

  • npm run verify

Refs #196

Copy link
Copy Markdown
Owner

@esengine esengine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified locally — rebased onto current main and ran the full suite (136 files, 2180 passed). Narrow change to chat.tsx, focused tests for the three startup-handoff states. Thanks!

@esengine esengine merged commit 0d89ab8 into esengine:main May 8, 2026
3 checks passed
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.

2 participants