dev-docs (MCP server): Add tracing and errors documentation#14374
Merged
dev-docs (MCP server): Add tracing and errors documentation#14374
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
AbhiPrasad
reviewed
Jul 16, 2025
Contributor
AbhiPrasad
left a comment
There was a problem hiding this comment.
nice, this is what I think we want. Once we convert the errors spec to stop using tags we can merge!
develop-docs/sdk/expected-features/mcp-instrumentation/index.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
reviewed
Jul 17, 2025
develop-docs/sdk/expected-features/mcp-instrumentation/errors.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
reviewed
Jul 17, 2025
develop-docs/sdk/expected-features/mcp-instrumentation/errors.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
reviewed
Jul 17, 2025
develop-docs/sdk/expected-features/mcp-instrumentation/tracing.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
reviewed
Jul 17, 2025
develop-docs/sdk/expected-features/mcp-instrumentation/tracing.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
approved these changes
Jul 17, 2025
Contributor
coolguyzone
left a comment
There was a problem hiding this comment.
Looks good! Thanks for adding 🫡
Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
….mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
….mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
Member
Author
|
Hey @coolguyzone , thanks for reviewing it. I was waiting to update the erors docs to use |
AbhiPrasad
approved these changes
Jul 25, 2025
Contributor
AbhiPrasad
left a comment
There was a problem hiding this comment.
very happy with this, good starting point. Thanks for iterating @betegon
Member
Author
|
merging this, happy to push another PR if you review it @coolguyzone :) |
lucas-zimerman
pushed a commit
that referenced
this pull request
Jul 29, 2025
## DESCRIBE YOUR PR Closes #14106 Adds developer docs for our MCP Server instrumentation. Route for the documentation `sdk/expected-features/mcp-instrumentation` that includes: ```bash .mcp-instrumentation ├── index.mdx # brief description on current features and limitations ├── errors.mdx # draft errors doc └── tracing.mdx # Tracing and span conventions ``` Followed this file structure as a start to (in the future) have this structure for all the other features, having something like this below, where we have : ``` .[Feature 1] ├── index.mdx ├── logs.mdx ├── errors.mdx . . . .[Feature 2] ├── index.mdx ├── errors.mdx ├── tracing.mdx . . . ``` The structure idea is from @abhi, who helped me settle this <3. **NOTE:** Error docs are going to change, addressing feedback from [sentry-javascript/pull/16817](getsentry/sentry-javascript#16817) ## IS YOUR CHANGE URGENT? ## PRE-MERGE CHECKLIST *Make sure you've checked the following before merging your changes:* - [x] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs) --------- Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
DESCRIBE YOUR PR
Closes #14106
Adds developer docs for our MCP Server instrumentation.
Route for the documentation
sdk/expected-features/mcp-instrumentationthat includes:Followed this file structure as a start to (in the future) have this structure for all the other features, having something like this below, where we have :
The structure idea is from @abhi, who helped me settle this <3.
NOTE: Error docs are going to change, addressing feedback from sentry-javascript/pull/16817
IS YOUR CHANGE URGENT?
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: