Skip to content

feat(sentry): add performance instrumentation for dashboard comparison#783

Merged
thostetler merged 12 commits intoadsabs:masterfrom
thostetler:feature/sentry-performance-dashboards
Jan 30, 2026
Merged

feat(sentry): add performance instrumentation for dashboard comparison#783
thostetler merged 12 commits intoadsabs:masterfrom
thostetler:feature/sentry-performance-dashboards

Conversation

@thostetler
Copy link
Member

@thostetler thostetler commented Jan 27, 2026

Adds a bunch of custom sentry spans to help with creation of the dashboard

@thostetler thostetler force-pushed the feature/sentry-performance-dashboards branch from 7a58734 to 4a9408d Compare January 27, 2026 21:17
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 71.71053% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 61.2%. Comparing base (f41136e) to head (3b50e0a).

Files with missing lines Patch % Lines
src/api/orcid/orcid.ts 18.6% 22 Missing ⚠️
src/lib/performance.ts 88.9% 12 Missing ⚠️
src/api/search/search.ts 14.3% 6 Missing ⚠️
src/api/metrics/metrics.ts 40.0% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #783     +/-   ##
========================================
+ Coverage    61.1%   61.2%   +0.1%     
========================================
  Files         304     305      +1     
  Lines       35318   35443    +125     
  Branches     1534    1553     +19     
========================================
+ Hits        21566   21669    +103     
- Misses      13716   13738     +22     
  Partials       36      36             
Files with missing lines Coverage Δ
src/api/export/export.ts 98.6% <100.0%> (+0.1%) ⬆️
src/api/metrics/metrics.ts 56.0% <40.0%> (-0.2%) ⬇️
src/api/search/search.ts 45.2% <14.3%> (-0.2%) ⬇️
src/lib/performance.ts 88.9% <88.9%> (ø)
src/api/orcid/orcid.ts 26.8% <18.6%> (+0.5%) ⬆️

... and 1 file with indirect coverage changes

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

@thostetler thostetler force-pushed the feature/sentry-performance-dashboards branch from 242fe14 to 3b50e0a Compare January 28, 2026 17:42
@thostetler thostetler requested a review from shinyichen January 28, 2026 17:43
@thostetler thostetler marked this pull request as ready for review January 28, 2026 17:43
@thostetler thostetler merged commit a9b04d7 into adsabs:master Jan 30, 2026
5 checks passed
@thostetler thostetler deleted the feature/sentry-performance-dashboards branch January 30, 2026 16:51
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.

2 participants