Skip to content

Conversation

jamesmcnamara
Copy link
Contributor

@jamesmcnamara jamesmcnamara commented Sep 8, 2025

Adds documentation for the Sourcegraph MCP server added in v6.8

Pull Request approval

You will need to get your PR approved by at least one member of the Sourcegraph team. For reviews of docs formatting, styles, and component usage, please tag the docs team via the #docs Slack channel.

Copy link

vercel bot commented Sep 8, 2025

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

Project Deployment Preview Comments Updated (UTC)
sourcegraph-docs Ready Ready Preview Comment Sep 9, 2025 9:50pm

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jamesmcnamara jamesmcnamara requested a review from a team September 8, 2025 22:05
@jamesmcnamara jamesmcnamara marked this pull request as ready for review September 8, 2025 22:05
- `limit` - Maximum results per page (optional, default 50)
- `after`/`before` - Pagination cursors (optional)

### Code Search
Copy link
Contributor

Choose a reason for hiding this comment

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

What do you think @janhartman ?

@mmanela mmanela requested review from a team and janhartman September 8, 2025 22:26
Copy link
Contributor

@janhartman janhartman left a comment

Choose a reason for hiding this comment

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

Seems pretty good! I wonder if we should generate full descriptions from our code and put them in the docs (collapsed by default). Could be valuable for an interested reader as otherwise they'd have to connect to the server first. But maybe hiding all the details is a feature? 😆


#### Prerequisites

Before using the device flow, you must create an OAuth application in your Sourcegraph instance:
Copy link
Member

Choose a reason for hiding this comment

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

We can predefine oauth clients as well, I wonder if we should do that.

@unknwon do you have thoughts here?

Then users would just need to copy the details here. No way to get a client secret then, so PKCE or RFC8628 device flow auth are required; which is probably what most clients should do anyways.

Copy link
Contributor

Choose a reason for hiding this comment

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

My 2 cents: If we are aiming enterprises to use MCP feature to start with, let's ask users to create OAuth applications for now because our OAuth scopes are currently still all or nothing. Ideally, we could give away some less sensitive scopes OOTB for a predefined client, then if users want to grant more scopes, they can choose to create a different OAuth client for their MCP needs.

Copy link
Member

Choose a reason for hiding this comment

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

Alright!

@jamesmcnamara
Copy link
Contributor Author

Seems pretty good! I wonder if we should generate full descriptions from our code and put them in the docs (collapsed by default). Could be valuable for an interested reader as otherwise they'd have to connect to the server first. But maybe hiding all the details is a feature? 😆

@janhartman What do you mean by full descriptions? Do you mean including the full tool description here (but collapsed)?

@jamesmcnamara jamesmcnamara force-pushed the jamesmcnamara-splf-1364-add-public-docs-for-sg-mcp branch from f6daeef to 9a58daa Compare September 9, 2025 21:47
@jamesmcnamara jamesmcnamara merged commit e376711 into main Sep 10, 2025
6 checks passed
@jamesmcnamara jamesmcnamara deleted the jamesmcnamara-splf-1364-add-public-docs-for-sg-mcp branch September 10, 2025 17:53
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.

6 participants