Skip to content

docs+feat: expand MCP connection states into reference table and add tooltip to State column header - #5785

Merged
Pratham-Mishra04 merged 1 commit into
devfrom
08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table
Aug 8, 2026
Merged

docs+feat: expand MCP connection states into reference table and add tooltip to State column header#5785
Pratham-Mishra04 merged 1 commit into
devfrom
08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table

Conversation

@Pratham-Mishra04

Copy link
Copy Markdown
Collaborator

Summary

Improves discoverability of MCP client connection states by expanding the auth overview docs and adding an inline tooltip to the "State" column header in the MCP clients table.

Changes

  • Replaced the prose description of connection states in docs/mcp/auth/overview.mdx with a structured reference table covering all states: connected, disconnected, error, pending_tools, disabled, pending_verification, and needs_reauth, including what each means and which auth types it applies to.
  • Added pending_tools and disabled as explicitly documented states, which were previously unlisted.
  • Clarified that pending_verification and needs_reauth are the two states that apply across all auth types, not just server-level clients.
  • Updated the needs_reauth description to distinguish server-level behavior from per-user behavior more precisely, and noted it is a display-level state for per-user clients.
  • Added a HeaderWithTooltip component to the MCP clients table that renders an info icon next to the "State" column header; hovering it shows a brief explanation of connection state semantics and links to the full reference table in the docs.

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Chore/CI

Affected areas

  • Core (Go)
  • Transports (HTTP)
  • Providers/Integrations
  • Plugins
  • UI (React)
  • Docs

How to test

  1. Navigate to the MCP clients table in the dashboard.
  2. Hover over the "State" column header and confirm the tooltip appears with a description and a link to the docs.
  3. Verify the link opens https://docs.getbifrost.ai/mcp/auth/overview#connection-states and the anchor resolves to the new connection states table.
cd ui
pnpm i || npm i
pnpm build || npm run build

Screenshots/Recordings

Before: "State" column header was plain text with no additional context.

After: "State" column header includes a small info icon; hovering reveals a tooltip explaining connection state semantics and linking to the full reference.

Breaking changes

  • No

Related issues

Security considerations

None.

Checklist

  • I read docs/contributing/README.md and followed the guidelines
  • I added/updated tests where appropriate
  • I updated documentation where needed
  • I verified builds succeed (Go and UI)
  • I verified the CI pipeline passes locally if applicable

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

This was referenced Aug 3, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/mcp/auth/overview.mdx`:
- Line 57: Update the auth-state documentation to map only per_user_oauth and
per_user_headers clients to MCP Sessions, excluding token_exchange. Verify the
current state projection and, if unchanged, scope needs_reauth to oauth,
per_user_oauth, per_user_headers, and token_exchange rather than all auth types;
also qualify the manual-repair guidance as applying only to server-level
clients.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 00da3f26-3c3b-4af9-ba32-bacc1e2eb74a

📥 Commits

Reviewing files that changed from the base of the PR and between 4379093 and 85ff18a.

📒 Files selected for processing (4)
  • docs/mcp/auth/overview.mdx
  • docs/mcp/connecting-to-servers.mdx
  • docs/mcp/overview.mdx
  • ui/app/workspace/mcp-registry/views/mcpClientsTable.tsx
🚧 Files skipped from review as they are similar to previous changes (3)
  • docs/mcp/overview.mdx
  • ui/app/workspace/mcp-registry/views/mcpClientsTable.tsx
  • docs/mcp/connecting-to-servers.mdx

Comment thread docs/mcp/auth/overview.mdx Outdated
@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 07-31-feat_add_token-exchange_demo_mcp_server_for_testing_against_real_identity_providers branch from 4379093 to 33dcddc Compare August 6, 2026 21:53
@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table branch from 85ff18a to 14b254c Compare August 6, 2026 21:53
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

coderabbitai[bot]
coderabbitai Bot previously approved these changes Aug 6, 2026
@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table branch from 14b254c to d0a54e1 Compare August 8, 2026 08:43
@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 07-31-feat_add_token-exchange_demo_mcp_server_for_testing_against_real_identity_providers branch 2 times, most recently from 0f63fe2 to e141ca5 Compare August 8, 2026 10:13
@Pratham-Mishra04
Pratham-Mishra04 requested a review from a team as a code owner August 8, 2026 10:13
@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table branch from d0a54e1 to b54596d Compare August 8, 2026 10:14

Pratham-Mishra04 commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator Author

Merge activity

  • Aug 8, 10:33 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Aug 8, 10:41 AM UTC: Graphite rebased this pull request as part of a merge.
  • Aug 8, 10:42 AM UTC: @Pratham-Mishra04 merged this pull request with Graphite.

@Pratham-Mishra04
Pratham-Mishra04 changed the base branch from 07-31-feat_add_token-exchange_demo_mcp_server_for_testing_against_real_identity_providers to graphite-base/5785 August 8, 2026 10:37
@Pratham-Mishra04
Pratham-Mishra04 changed the base branch from graphite-base/5785 to dev August 8, 2026 10:40
@Pratham-Mishra04
Pratham-Mishra04 dismissed coderabbitai[bot]’s stale review August 8, 2026 10:40

The base branch was changed.

@Pratham-Mishra04
Pratham-Mishra04 force-pushed the 08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table branch from b54596d to 9b9630a Compare August 8, 2026 10:40
@Pratham-Mishra04
Pratham-Mishra04 merged commit 61583ed into dev Aug 8, 2026
14 of 15 checks passed
@Pratham-Mishra04
Pratham-Mishra04 deleted the 08-01-feat_add_a_states-explainer_tooltip_to_the_mcp_servers_table branch August 8, 2026 10:42
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