Skip to content

feat: adopt @sentry/api SDK for 38 of 40 API call sites #931

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
MathurAditya724 wants to merge 33 commits into
base: main
Choose a base branch
from
Open

feat: adopt @sentry/api SDK for 38 of 40 API call sites #931

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
6906b31
feat: adopt @sentry/api SDK for 28 API call sites
MathurAditya724 May 5, 2026
84f5766
test: fix MSW handler mismatches after SDK migration
MathurAditya724 May 5, 2026
f9f3776
fix: replace delete operator with Reflect.deleteProperty for Biome lint
MathurAditya724 May 5, 2026
0a1ab48
fix: restore missing query params and improve error extraction
MathurAditya724 May 5, 2026
abb3fdc
test: update 404 error snapshot for new unwrapSdkResult message format
MathurAditya724 May 5, 2026
f6eeee8
test(core): Restore LLM env vars in test setup
dcramer May 5, 2026
99fbc48
fix(core): Preserve replay environment filters
dcramer May 5, 2026
36e9413
fix(core): Preserve SDK query parameter validation
dcramer May 5, 2026
aa5564b
test(core): Clear LLM env before tests
dcramer May 5, 2026
a994785
test(core): Validate tag key schema rejection
dcramer May 5, 2026
cd1f116
chore: bump @sentry/api to ^0.141.0
MathurAditya724 May 6, 2026
f15d51e
feat: adopt auto-generated Zod schemas from @sentry/api/zod
MathurAditya724 May 6, 2026
0fceb91
fix(core): Preserve issue event query shape
dcramer May 6, 2026
99c7c83
fix: pass API error detail as message in unwrapSdkResult for multi-pr…
MathurAditya724 May 6, 2026
7d1b8e6
fix(core): Detect SDK multi-project errors
dcramer May 6, 2026
fb38abb
feat: migrate 9 remaining endpoints to @sentry/api SDK v0.171.0
MathurAditya724 Jun 3, 2026
38396ed
refactor: remove redundant inner type casts on SDK call sites
MathurAditya724 Jun 3, 2026
3a5b50a
refactor: rebase TagSchema on @sentry/api/zod's zTagKeyDetailsDict
MathurAditya724 Jun 3, 2026
e65a5f8
fix: forward client identity headers through SDK calls
MathurAditya724 Jun 3, 2026
2063a8d
chore: bump @sentry/api to ^0.172.0 and document SdkResult
MathurAditya724 Jun 3, 2026
e2d1eeb
fix: use API detail text in SDK error messages, remove dead code
MathurAditya724 Jun 3, 2026
917acd0
fix(api-client): bump @sentry/api to 0.174.0 and remove now-unnecessa…
betegon Jun 3, 2026
61c94c4
feat: migrate listTags to SDK via listAnOrganization_sTags
MathurAditya724 Jun 3, 2026
0c9a035
fix: pass allowAggregateConditions as string '0' not boolean false
MathurAditya724 Jun 3, 2026
f445beb
fix(api-client): remove dead params project and useRpc from SDK calls
betegon Jun 4, 2026
84eb184
fix(api-client): inline typed query for searchReplays, revert listTag…
betegon Jun 4, 2026
1a285de
fix(api-client): restore project=-1 in listReplayIdsForIssue
betegon Jun 4, 2026
950be09
fix(api): Preserve SDK query behavior after rebase
dcramer Jun 12, 2026
e91521d
chore(api-client): bump @sentry/api to 0.228.0 and adopt token operat…
betegon Jun 18, 2026
5a598a2
refactor(api-client): use SDK parseSentryLinkHeader for pagination
betegon Jun 18, 2026
55cbe83
refactor(api-client): type unwrapSdkResult structurally instead of any
betegon Jun 18, 2026
52b2cd5
fix(api-client): forward replay project slug verbatim, don't coerce t…
betegon Jun 18, 2026
975604c
fix(api-client): stop coercing profiling project slugs to NaN; bump n…
betegon Jun 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"type": "module",
"packageManager": "pnpm@10.15.1",
"engines": {
"node": ">=20"
"node": ">=22"
},
"license": "FSL-1.1-ALv2",
"author": "Sentry",
Expand Down
11 changes: 4 additions & 7 deletions packages/mcp-core/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,25 +5,21 @@
"private": true,
"packageManager": "pnpm@10.8.1",
"engines": {
"node": ">=20"
"node": ">=22"
},
"license": "FSL-1.1-ALv2",
"author": "Sentry",
"description": "Sentry MCP Core - Shared code for MCP transports",
"homepage": "https://github.com/getsentry/sentry-mcp",
"keywords": [
"sentry"
],
"keywords": ["sentry"],
Comment thread
cursor[bot] marked this conversation as resolved.
"bugs": {
"url": "https://github.com/getsentry/sentry-mcp/issues"
},
"repository": {
"type": "git",
"url": "git@github.com:getsentry/sentry-mcp.git"
},
"files": [
"./dist/*"
],
"files": ["./dist/*"],
"exports": {
"./api-client": {
"types": "./dist/api-client/index.d.ts",
Expand Down Expand Up @@ -168,6 +164,7 @@
"@logtape/logtape": "^1.1.1",
"@logtape/sentry": "^1.1.1",
"@modelcontextprotocol/sdk": "catalog:",
"@sentry/api": "^0.228.0",

Check warning on line 167 in packages/mcp-core/package.json

View check run for this annotation

@sentry/warden / warden: code-review

Migrated SDK API calls no longer map network errors (ENOTFOUND/ECONNREFUSED/ETIMEDOUT) to ConfigurationError, causing spurious Sentry reports and unhelpful guidance 

The 38 methods migrated to the @sentry/api SDK bypass the legacy request() path, which detected DNS/connection error codes and threw ConfigurationError. For network failures these methods now surface a non-ConfigurationError error, so formatErrorForUser falls through to its final branch — calling logIssue (a spurious Sentry event) and telling the user to contact support — instead of the actionable "**Configuration Error**" guidance users previously got for a misconfigured self-hosted SENTRY_URL or transient network failure.
"@sentry/core": "catalog:",
"ai": "catalog:",
"dotenv": "catalog:",
Expand Down
Loading
Loading