Skip to content

fix(app): restore summary status indicator - #49470

Open
opencode-agent[bot] wants to merge 3 commits into
v2from
grouped-summary-status
Open

opencode-agent[bot] wants to merge 3 commits into
v2from
grouped-summary-status

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • restore the server status dot on the grouped Session details summary
  • keep the server indicator scoped to server health and connection state
  • surface MCP errors and attention states on the Session details trigger
  • mark MCP in the summary when one of its servers needs attention
  • add focused unit and E2E coverage

Validation

  • bun turbo typecheck --concurrency=1
  • bun run lint
  • bun test src/session/summary/indicator.test.ts src/session/summary/configured-lsp.test.ts
  • PLAYWRIGHT_WORKERS=1 bun run test:e2e:built -- e2e/regression/session-summary.spec.ts e2e/regression/session-summary-mcp.spec.ts -g "horizontal tabs|every MCP row" --reporter=line
  • inspected healthy and MCP-attention fixture screenshots
  • bench:entry was run before and after the latest follow-up; both runs retained the existing new-session fixture POST failures, while comparable cold-entry medians improved from 466.80→445.70ms first-correct and 554.05→527.80ms stable

Requested by: @Brendonovich (Brendan via Slack)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant