Move the MCP logo to the Conformance sidebar item. - #3662
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_a4ff8b35-80bd-4745-a201-648dc41804e3) |
📄 Knowledge reviewDosu skipped reviewing this PR because your organization has used its |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughUpdated the sidebar icon import and assignments. Connect and Servers now use Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
mcpjam-inspector/client/src/components/mcp-sidebar.tsxOops! Something went wrong! :( ESLint: 8.57.1 Error: ESLint configuration in --config is invalid:
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 |
Note
Low Risk
Cosmetic sidebar icon changes only; no routing, auth, or behavior impact.
Overview
Updates sidebar icons in
mcp-sidebar.tsxso Connect and Servers use Lucide Cable instead of MCPIcon, and Conformance uses MCPIcon instead of FlaskConical.Nav URLs, feature flags, and labels are unchanged—this is icon-only.
Reviewed by Cursor Bugbot for commit ba33079. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by cubic
Moved the MCP logo (
MCPIcon) to the Conformance sidebar item, replacingFlaskConical. Updated the Connect and Servers items to useCableinstead ofMCPIconfor clearer semantics.Written for commit ba33079. Summary will update on new commits.