Skip to content

feat(search): add Devin to Sim Search MCP clients - #8489

Merged
waleedlatif1 merged 1 commit into
stagingfrom
feat/search-mcp-devin
Sep 30, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
feat/search-mcp-devin

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Add Devin to the Sim Search MCP app picker
  • Devin connects remote MCP servers through its UI, so it shows the server URL with setup steps: Customize → MCPs → add a custom MCP, HTTP transport, OAuth, Personal access, then Connect
  • Personal access keeps search results scoped to each member's own Sim access

Type of Change

  • New feature

Testing

  • search-mcp-connection.test.tsx passes
  • bun run lint, check:audits (52 audits), and docs-manifest:check pass

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 30, 2026 11:31pm UTC

Request Review

@waleedlatif1
waleedlatif1 merged commit a1ee856 into staging Sep 30, 2026
23 checks passed
@waleedlatif1
waleedlatif1 deleted the feat/search-mcp-devin branch September 30, 2026 23:32
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Medium risk] Adds Devin to the search MCP client list.

The PR appears safe to merge, though a focused Devin connection test would protect the new setup flow.

Findings

  1. P2 Devin setup lacks test coverage ▶

Summary

Adds Devin to the Sim Search MCP app picker and provides URL-based setup guidance for its HTTP and OAuth connection.

  • Devin displays and copies the server URL rather than a terminal command.
  • The new client path lacks a focused test.

Reviews (1) · Last reviewed commit: "feat(search): add Devin to Sim Search MC..."

Comment on lines +70 to +71
: client === 'devin'
? 'In Devin, open Customize → MCPs and add a custom MCP with this URL. Choose HTTP, OAuth, and Personal access, then select Connect and sign in to Sim.'

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.

P2 Devin setup lacks test coverage The existing connection test selects only Claude Code. It would not catch a regression in Devin’s displayed URL, copied value, or setup guidance. A test that selects Devin and checks those details would protect this new flow.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

This branch was previously deployed

1 inactive deployment
Preview — 463e341a Deployed Sep 30, 2026 by vercel[bot]
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.

1 participant