Skip to content

feat(MCP): Set up Sentry error reporting#7879

Merged
emyller merged 6 commits into
mainfrom
feat/mcp-sentry
Jun 30, 2026
Merged

feat(MCP): Set up Sentry error reporting#7879
emyller merged 6 commits into
mainfrom
feat/mcp-sentry

Conversation

@emyller

@emyller emyller commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

Contributes to #7771

Set up Sentry in the MCP server for error reporting.

How did you test this code?

New tests.

@emyller emyller self-assigned this Jun 25, 2026
@emyller emyller requested a review from a team as a code owner June 25, 2026 22:38
@emyller emyller requested review from khvn26 and removed request for a team June 25, 2026 22:38
@vercel

vercel Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown

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

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview Jun 30, 2026 3:03pm
flagsmith-frontend-preview Ignored Ignored Preview Jun 30, 2026 3:03pm
flagsmith-frontend-staging Ignored Ignored Preview Jun 30, 2026 3:03pm

Request Review

@github-actions github-actions Bot added the feature New feature or request label Jun 25, 2026
@github-actions

github-actions Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-e2e:pr-7879 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-7879 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-api-test:pr-7879 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-api:pr-7879 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7879 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith:pr-7879 Finished ✅ Results ✅
ghcr.io/flagsmith/flagsmith-e2e:pr-7879 Finished ✅ Skipped

@github-actions

github-actions Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  41.7 seconds
commit  7266d75
info  🔄 Run: #17840 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  44.7 seconds
commit  7266d75
info  🔄 Run: #17840 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  3 passed

Details

stats  3 tests across 3 suites
duration  34 seconds
commit  7266d75
info  🔄 Run: #17840 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  1 minute, 13 seconds
commit  7266d75
info  🔄 Run: #17840 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  5 passed

Details

stats  5 tests across 3 suites
duration  34.6 seconds
commit  809fce5
info  🔄 Run: #17934 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  5 passed

Details

stats  5 tests across 4 suites
duration  33.8 seconds
commit  809fce5
info  🔄 Run: #17934 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  18.7 seconds
commit  809fce5
info  🔄 Run: #17934 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  54.5 seconds
commit  809fce5
info  🔄 Run: #17934 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  5 passed

Details

stats  5 tests across 3 suites
duration  33.8 seconds
commit  5eff3b2
info  🔄 Run: #17935 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  45.6 seconds
commit  5eff3b2
info  🔄 Run: #17935 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  10.9 seconds
commit  5eff3b2
info  🔄 Run: #17935 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  34.1 seconds
commit  5eff3b2
info  🔄 Run: #17935 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  3 passed

Details

stats  3 tests across 2 suites
duration  13.1 seconds
commit  acfe484
info  🔄 Run: #17971 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  5 passed

Details

stats  5 tests across 3 suites
duration  38.2 seconds
commit  16472e0
info  🔄 Run: #17972 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  11.3 seconds
commit  acfe484
info  🔄 Run: #17971 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  11.7 seconds
commit  16472e0
info  🔄 Run: #17972 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  4 passed

Details

stats  4 tests across 4 suites
duration  18.9 seconds
commit  16472e0
info  🔄 Run: #17972 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  4 passed

Details

stats  4 tests across 3 suites
duration  54.3 seconds
commit  16472e0
info  🔄 Run: #17972 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  6 passed

Details

stats  6 tests across 4 suites
duration  19.2 seconds
commit  acfe484
info  🔄 Run: #17971 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  52.4 seconds
commit  acfe484
info  🔄 Run: #17971 (attempt 1)

@github-actions

github-actions Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Visual Regression

17 screenshots compared. See report for details.
View full report

khvn26
khvn26 reviewed Jun 26, 2026
View reviewed changes

@khvn26 khvn26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a couple of comments.

Comment thread mcp/src/flagsmith_mcp/config.py Outdated
Comment thread mcp/src/flagsmith_mcp/config.py Outdated
Comment thread mcp/src/flagsmith_mcp/config.py Outdated
@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Jun 29, 2026
@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Jun 29, 2026
khvn26
khvn26 previously approved these changes Jun 30, 2026
View reviewed changes

@khvn26 khvn26 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions Bot added feature New feature or request and removed feature New feature or request labels Jun 30, 2026
@github-actions github-actions Bot removed the feature New feature or request label Jun 30, 2026
@github-actions github-actions Bot added the feature New feature or request label Jun 30, 2026
@emyller emyller merged commit 5c55544 into main Jun 30, 2026
32 of 33 checks passed
@emyller emyller deleted the feat/mcp-sentry branch June 30, 2026 20:32
@flagsmith-engineering flagsmith-engineering Bot mentioned this pull request Jun 30, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@khvn26 khvn26 khvn26 approved these changes

Assignees

@emyller emyller

@khvn26 khvn26

Labels

feature New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@emyller @khvn26