Conversation
Rework the MCP servers page with a cleaner, more compact layout: - Split servers into Connected / Suggested sections - Add search bar with All / Connected filter tabs - Compact card design with smaller favicons, stripped URLs, quieter status indicators - Move Reconnect into three-dot overflow menu alongside new Disconnect option - Add disconnectMcpServer server action to remove a user's MCP credentials - Extract useConnectMcp hook for shared connect/reconnect logic
Redesign the workspace Ask Agent page with card-based layout, 3-stat strip, connector rows with status indicators, kebab menu, and Connect button with OAuth flow. Extract shared ConnectorRowInfo component for reuse between workspace and account settings pages.
Add "Configure connectors" link to the chat toolbar's Connectors submenu, pointing to /settings/workspaceAskAgent for workspace-level connector management. Keep existing "Manage connectors" link to /settings/accountAskAgent for personal connector setup.
Replace the full-card tool list with a compact badge grid. Clicking a badge reveals that tool's detail panel inline; only one detail is visible at a time. Also remove the focus ring from the tools trigger button.
Unify the duplicated card layout between AccountConnectedConnectorCard and WorkspaceConnectorCard into a single ConnectorCard component that owns the Collapsible state, Card shell, ConnectorRowInfo, and tool disclosure wiring. The divergent parts (status badge, action buttons) are passed as ReactNode slots.
* schema * feat
Move Ask/MCP chat features under ee/ and add entitlement messaging so the Sourcebot MCP server requires a paid subscription. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
# Conflicts: # docs/docs/features/mcp-server.mdx # docs/docs/misc/service-ping.mdx # packages/web/src/app/(app)/settings/accountAskAgent/page.tsx # packages/web/src/app/(app)/settings/layout.tsx # packages/web/src/app/(app)/settings/license/types.ts # packages/web/src/app/(app)/settings/workspaceAskAgent/page.test.tsx # packages/web/src/app/(app)/settings/workspaceAskAgent/page.tsx # packages/web/src/app/(app)/settings/workspaceAskAgent/workspaceAskAgentPage.tsx # packages/web/src/app/api/(client)/client.ts # packages/web/src/app/api/(server)/ee/askmcp/callback/route.test.ts # packages/web/src/app/api/(server)/ee/askmcp/callback/route.ts # packages/web/src/app/api/(server)/ee/askmcp/configuration/route.test.ts # packages/web/src/app/api/(server)/ee/askmcp/configuration/route.ts # packages/web/src/app/api/(server)/ee/askmcp/connect/route.test.ts # packages/web/src/app/api/(server)/ee/askmcp/connect/route.ts # packages/web/src/app/api/(server)/ee/askmcp/servers/route.test.ts # packages/web/src/app/api/(server)/ee/askmcp/servers/route.ts # packages/web/src/app/api/(server)/ee/askmcp/tools/route.test.ts # packages/web/src/app/api/(server)/ee/askmcp/tools/route.ts # packages/web/src/app/api/(server)/ee/chat/route.ts # packages/web/src/ee/features/chat/agent.ts # packages/web/src/ee/features/chat/components/chatThread/chatThread.tsx # packages/web/src/ee/features/chat/components/chatThread/chatThreadListItem.tsx # packages/web/src/ee/features/chat/components/chatThread/detailsCard.test.tsx # packages/web/src/ee/features/chat/components/chatThread/mcpFailedServersBanner.tsx # packages/web/src/ee/features/chat/components/chatThread/toolApprovalBanner.tsx # packages/web/src/ee/features/chat/components/chatThread/tools/jsonHighlighter.tsx # packages/web/src/ee/features/chat/components/chatThread/tools/mcpToolComponent.tsx # packages/web/src/ee/features/chat/components/chatThread/tools/toolSearchToolComponent.tsx # packages/web/src/ee/features/chat/components/chatThreadPanel.test.tsx # packages/web/src/ee/features/chat/components/chatThreadPanel.tsx # packages/web/src/ee/features/chat/mcp/actions.test.ts # packages/web/src/ee/features/chat/mcp/actions.ts # packages/web/src/ee/features/chat/mcp/components/connectorCard.tsx # packages/web/src/ee/features/chat/mcp/components/connectorToolDisclosure.test.tsx # packages/web/src/ee/features/chat/mcp/components/connectorToolDisclosure.tsx # packages/web/src/ee/features/chat/mcp/components/connectorToolUsageDisclosure.tsx # packages/web/src/ee/features/chat/mcp/hooks/useMcpToolMetadata.ts # packages/web/src/ee/features/chat/mcp/mcpClientFactory.test.ts # packages/web/src/ee/features/chat/mcp/mcpClientFactory.ts # packages/web/src/ee/features/chat/mcp/mcpToolSets.ts # packages/web/src/ee/features/chat/mcp/types.ts # packages/web/src/features/billing/planComparisonTable.tsx # packages/web/src/features/chat/components/chatBox/chatBoxPlusButton.tsx # packages/web/src/features/chat/mcpOAuthDraft.test.ts # packages/web/src/features/chat/mcpOAuthDraft.ts # packages/web/src/middleware/withAuth.test.ts
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
msukkari
changed the title
Move prismaOAuthClientProvider and mcpOAuthReturnTo (Ask-connector OAuth-client code) from ee/features/mcp (the MCP-server folder) into ee/features/chat/mcp, alongside the rest of the Ask connector code, so 'Sourcebot as an MCP server' and 'Ask connecting to external MCP servers' are cleanly separated at the folder level. Repoint importers (askmcp connect/callback routes, mcpClientFactory). No behavior change; all paths remain gated on the `ask` entitlement. Also remove two stale duplicate test files left behind by the EE migration (their sources moved to ee/): features/chat/agent.test.ts and features/chat/components/chatBox/chatBoxPlusButton.test.ts. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes SOU-1199
Fixes SOU-1200