Skip to content

feat: use distinct span operations for app start phases#8003

Merged
itaybre merged 4 commits into
mainfrom
itay/update_spans_operations
Jun 9, 2026
Merged

feat: use distinct span operations for app start phases#8003
itaybre merged 4 commits into
mainfrom
itay/update_spans_operations

Conversation

@itaybre

@itaybre itaybre commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Description

Standalone app start child spans now use specific operation names instead of the generic app.start operation. This improves observability and allows filtering by individual app start phases.

Operation mapping (standalone spans only)

Span description Before After
Pre Runtime Init app.start app.start.pre_runtime_init
Runtime Init to Pre Main Initializers app.start app.start.runtime_init
UIKit Init app.start app.start.uikit_init
Application Init app.start app.start.application_init
Extended App Start app.start app.start.extended_app_start

Spotlight confirmation of the rename
image

Non-standalone spans retain their existing app.start operation unchanged.

Standalone app start child spans now use specific operation names
(e.g. app.start.pre_runtime_init, app.start.uikit_init) instead of
the generic app.start, improving observability and filtering.
@itaybre itaybre added the ready-to-merge Use this label to trigger all PR workflows label Jun 5, 2026
@codecov

codecov Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.406%. Comparing base (72ea638) to head (b8a8779).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #8003       +/-   ##
=============================================
+ Coverage   87.308%   87.406%   +0.097%     
=============================================
  Files          553       553               
  Lines        31967     31976        +9     
  Branches     13151     13161       +10     
=============================================
+ Hits         27910     27949       +39     
+ Misses        4009      3980       -29     
+ Partials        48        47        -1     
Files with missing lines Coverage Δ
Sources/Sentry/SentryBuildAppStartSpans.m 100.000% <100.000%> (ø)
...StartTracking/SentryExtendedAppLaunchManager.swift 95.238% <100.000%> (ø)

... and 5 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72ea638...b8a8779. Read the comment docs.

@sentry

sentry Bot commented Jun 5, 2026

Copy link
Copy Markdown

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
SDK-Size io.sentry.sample.SDK-Size 9.16.1 (1) Release

⚙️ sentry-cocoa Build Distribution Settings

@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1211.56 ms 1249.20 ms 37.64 ms
Size 24.14 KiB 1.18 MiB 1.15 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
962c7bc 1235.49 ms 1262.79 ms 27.30 ms
9da166b 1222.40 ms 1250.24 ms 27.84 ms
39655d1 1224.05 ms 1258.38 ms 34.33 ms
31a83fd 1218.63 ms 1247.18 ms 28.55 ms
1770336 1225.09 ms 1251.32 ms 26.23 ms
8d75f5a 1223.90 ms 1255.94 ms 32.04 ms
943b250 1214.69 ms 1257.32 ms 42.63 ms
6feb25a 1233.71 ms 1261.45 ms 27.73 ms
e44b6f8 1225.98 ms 1249.20 ms 23.23 ms
ddbec00 1229.12 ms 1261.33 ms 32.20 ms

App size

Revision Plain With Sentry Diff
962c7bc 24.14 KiB 1.16 MiB 1.13 MiB
9da166b 24.14 KiB 1.16 MiB 1.14 MiB
39655d1 24.14 KiB 1.16 MiB 1.14 MiB
31a83fd 24.14 KiB 1.17 MiB 1.15 MiB
1770336 24.14 KiB 1.15 MiB 1.13 MiB
8d75f5a 24.14 KiB 1.17 MiB 1.15 MiB
943b250 24.14 KiB 1.17 MiB 1.15 MiB
6feb25a 24.14 KiB 1.17 MiB 1.14 MiB
e44b6f8 24.14 KiB 1.15 MiB 1.13 MiB
ddbec00 24.14 KiB 1.17 MiB 1.15 MiB

Previous results on branch: itay/update_spans_operations

Startup times

Revision Plain With Sentry Diff
d4a8c2a 1212.39 ms 1246.71 ms 34.33 ms
3379181 1233.89 ms 1254.81 ms 20.92 ms
8bc0f14 1218.65 ms 1253.47 ms 34.82 ms

App size

Revision Plain With Sentry Diff
d4a8c2a 24.14 KiB 1.17 MiB 1.15 MiB
3379181 24.14 KiB 1.17 MiB 1.15 MiB
8bc0f14 24.14 KiB 1.17 MiB 1.15 MiB

@itaybre itaybre changed the title impr: use distinct span operations for app start phases feat: use distinct span operations for app start phases Jun 5, 2026
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against b8a8779

@itaybre itaybre marked this pull request as ready for review June 5, 2026 18:54

@philprime philprime 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

@itaybre itaybre enabled auto-merge (squash) June 9, 2026 17:07
@itaybre itaybre merged commit a22d778 into main Jun 9, 2026
251 of 254 checks passed
@itaybre itaybre deleted the itay/update_spans_operations branch June 9, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants