Skip to content

Refactor API and add transaction list tool #42

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.

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

Merged
merged 2 commits into from
Apr 13, 2025
Merged

Conversation

dcramer
Copy link
Member

@dcramer dcramer commented Apr 13, 2025

  • Add 'list_transactions' endpoint
  • Move API into its own submodule (getting a little large)
  • Improve some tool responses and expand coverage

We cant yet reliable export a trace until Sentry lands some new APIs, so considering this a PoC.

Copy link

codecov bot commented Apr 13, 2025

Codecov Report

Attention: Patch coverage is 91.71484% with 43 lines in your changes missing coverage. Please review.

Project coverage is 82.18%. Comparing base (4ccd825) to head (2cfc6c7).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/lib/sentry-api/client.ts 75.00% 18 Missing ⚠️
src/lib/sentry-msw.ts 93.46% 13 Missing ⚠️
src/mcp/tools.ts 90.72% 9 Missing ⚠️
src/lib/sentry-api/utils.ts 91.89% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   79.09%   82.18%   +3.08%     
==========================================
  Files          15       18       +3     
  Lines        1765     2110     +345     
  Branches       97      135      +38     
==========================================
+ Hits         1396     1734     +338     
- Misses        369      376       +7     
Flag Coverage Δ
evals 25.54% <30.63%> (?)
unittests 80.57% <91.68%> (+1.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dcramer dcramer force-pushed the feat/transaction-list branch from db97ae8 to e9d79fd Compare April 13, 2025 15:51
@dcramer dcramer force-pushed the feat/transaction-list branch from e9d79fd to bf5780e Compare April 13, 2025 15:51
@getsentry getsentry deleted a comment from github-actions bot Apr 13, 2025
@dcramer dcramer force-pushed the feat/transaction-list branch from fe301ef to 2f7674a Compare April 13, 2025 16:00
@dcramer dcramer force-pushed the feat/transaction-list branch from 2f7674a to c4af964 Compare April 13, 2025 16:03
@dcramer dcramer force-pushed the feat/transaction-list branch from c4af964 to 6a13e9d Compare April 13, 2025 16:06
@dcramer dcramer force-pushed the feat/transaction-list branch from 6a13e9d to 9406d9a Compare April 13, 2025 16:08
@dcramer dcramer force-pushed the feat/transaction-list branch from 9406d9a to f9fc565 Compare April 13, 2025 16:12
@dcramer dcramer force-pushed the feat/transaction-list branch from f9fc565 to 9fec0b0 Compare April 13, 2025 16:12
@dcramer dcramer force-pushed the feat/transaction-list branch from 9fec0b0 to 05c454c Compare April 13, 2025 16:16
@dcramer dcramer force-pushed the feat/transaction-list branch from 6981004 to 128ae67 Compare April 13, 2025 16:26
- Add 'list_transactions' endpoint
- Move API into its own submodule (getting a little large)
- Improve some tool responses and expand coverage

We cant yet reliable export a trace until Sentry lands some new APIs, so considering this a PoC.
@dcramer dcramer force-pushed the feat/transaction-list branch from b0a22f7 to 2cfc6c7 Compare April 13, 2025 18:07
@getsentry getsentry deleted a comment from github-actions bot Apr 13, 2025
@dcramer dcramer merged commit 92b1387 into main Apr 13, 2025
8 checks passed
@dcramer dcramer deleted the feat/transaction-list branch April 13, 2025 18:23
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.

None yet

1 participant