Skip to content

Conversation

@thostetler
Copy link
Member

@thostetler thostetler commented Jan 28, 2026

Adds custom sentry spans that mirror the ones added to nectar so that the dashboards can have the same charts

Add initialScope.tags.app = 'bumblebee' to Sentry config to enable
filtering and comparison with Nectar (SciX) performance dashboards.
Create AMD utility module for Sentry span instrumentation with:
- PERF_SPANS constants matching Nectar naming convention
- trackUserFlow/trackDeferred for async operation spans
- startRenderSpan for UI rendering spans
- getQueryType and getResultCountBucket helpers for tagging

Register as 'performance' path in RequireJS config.
Add search.submit.total span tracking:
- Results widget publishes timing:search-started event on search init
- Navigator creates span on timing:search-started
- Navigator ends span on timing:results-loaded

Includes query_type tag from performance utility.
Wrap export API request in trackDeferred with export.api.request span.
Includes export_format tag for filtering by export type.
Add performance tracking for library operations:
- library.list.load for fetching library metadata
- library.create.total for creating new libraries
- library.add.total for adding papers to libraries
Add orcid.profile.load span tracking for getUserProfile requests
to measure ORCID integration performance.
- Remove unused performance import from query_mediator
- Add try/catch around all Sentry span operations
- Extract endSpan helper with defensive error handling
- Ensure tracking code never breaks the main application flow
@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 52ca8f2 into adsabs:master Feb 5, 2026
1 check passed
@thostetler thostetler deleted the feature/sentry-performance-dashboards branch February 5, 2026 22:33
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