feat(attributes): Add app vitals start attributes#353
Merged
Conversation
Define the first interactive screen for app start spans so SDKs can report foreground starts and omit it for background starts without rendered UI. Co-authored-by: Cursor <cursoragent@cursor.com>
buenaflor
requested review from
a team,
Lms24,
cleptric,
lcian,
mjq and
nsdeschenes
as code owners
Semver Impact of This PR🟡 Minor (new features) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Attributes
Bug Fixes 🐛
🤖 This preview updates automatically when you update the PR. |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new semantic convention attribute to represent the first interactive UI screen after an app start, enabling SDKs to distinguish foreground app starts (UI rendered) from background app starts (no UI).
Changes:
- Introduced new attribute model:
app.vitals.start.screen - Regenerated TypeScript conventions output to include the new attribute constant/types/metadata
- Regenerated Python conventions output to include the new attribute name/metadata/typing
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| model/attributes/app/app__vitals__start__screen.json | Adds the source-of-truth attribute definition for app.vitals.start.screen. |
| javascript/sentry-conventions/src/attributes.ts | Regenerated TS exports/types/metadata to include the new attribute. |
| python/src/sentry_conventions/attributes.py | Regenerated Python constants/metadata/types to include the new attribute. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
cleptric
approved these changes
May 5, 2026
Propagate the app start screen description punctuation into generated TypeScript and Python convention files so CI generation stays clean. Co-authored-by: Cursor <cursoragent@cursor.com>
Define the launch trigger reason for app start spans so mobile SDKs can report normalized platform-specific causes alongside the start screen. Co-authored-by: Cursor <cursoragent@cursor.com>
buenaflor
changed the title
Keep the app start reason convention description focused on the trigger semantics and propagate the generated TypeScript and Python outputs. Co-authored-by: Cursor <cursoragent@cursor.com>
romtsn
reviewed
May 6, 2026
| }, | ||
| isInOtel: false, | ||
| example: 'push', | ||
| sdks: ['sentry.cocoa', 'sentry.java.android', 'sentry.javascript.react-native', 'sentry.dart.flutter'], |
Member
There was a problem hiding this comment.
dunno if we need to mention maui here?
Contributor
Author
There was a problem hiding this comment.
this sdks field is not really used anywhere, only informational
added it here as well though to be correct
List sentry.dotnet.maui for app start reason and screen attributes so SDK metadata reflects MAUI support in generated packages. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add
app.vitals.start.screenandapp.vitals.start.reasonfor app start spans.The screen attribute records the first interactive screen after launch, while the reason attribute records the normalized launch trigger such as
push. Omitting the screen attribute indicates a background app start where no UI was rendered.Generated the TypeScript and Python convention outputs from the new attribute models.