Improve app start integration

[denrase]

📜 Description

• Always create root TTID transaction/span in app start integration
• Move app start processor functionality to integration

💡 Motivation and Context

Closes #2259
Closes #2103
Closes #2233

💚 How did you test it?

• Unit tests

📝 Checklist

• I reviewed submitted code
• I added tests to verify changes
• No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
• I updated the docs if needed
• All tests passing
• No breaking changes

[github-actions]

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

Generated by 🚫 dangerJS against 02ce10e

[codecov]

Codecov Report

Attention: Patch coverage is 93.27731% with 8 lines in your changes missing coverage. Please review.

Project coverage is 90.71%. Comparing base (fe6dcac) to head (02ce10e).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...lib/src/integrations/native_app_start_handler.dart	95.91%	4 Missing ⚠️
flutter/lib/src/sentry_flutter.dart	50.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2266      +/-   ##
==========================================
+ Coverage   88.01%   90.71%   +2.70%     
==========================================
  Files         247       73     -174     
  Lines        8608     2370    -6238     
==========================================
- Hits         7576     2150    -5426     
+ Misses       1032      220     -812     

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

[github-actions]

iOS Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1245.53 ms	1274.51 ms	28.98 ms
Size	8.38 MiB	9.73 MiB	1.35 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
f1314d5	1246.46 ms	1270.92 ms	24.46 ms
04bd9e6	1230.78 ms	1250.71 ms	19.94 ms
64e4616	1242.80 ms	1270.07 ms	27.27 ms
e6b16cd	1226.20 ms	1246.52 ms	20.32 ms
a5031f1	1244.14 ms	1268.79 ms	24.64 ms
3ba8d7e	1224.14 ms	1226.12 ms	1.98 ms
ef31c7f	1272.43 ms	1295.71 ms	23.29 ms
33527b4	1225.17 ms	1252.23 ms	27.06 ms
1edf30e	1254.43 ms	1272.82 ms	18.39 ms
f9d18f3	1240.20 ms	1242.78 ms	2.57 ms

App size

Revision	Plain	With Sentry	Diff
f1314d5	8.10 MiB	9.08 MiB	1004.30 KiB
04bd9e6	8.33 MiB	9.61 MiB	1.27 MiB
64e4616	8.28 MiB	9.34 MiB	1.05 MiB
e6b16cd	8.33 MiB	9.54 MiB	1.22 MiB
a5031f1	8.32 MiB	9.50 MiB	1.18 MiB
3ba8d7e	8.29 MiB	9.36 MiB	1.07 MiB
ef31c7f	8.09 MiB	9.07 MiB	1000.79 KiB
33527b4	8.34 MiB	9.66 MiB	1.32 MiB
1edf30e	8.16 MiB	9.17 MiB	1.01 MiB
f9d18f3	8.29 MiB	9.36 MiB	1.07 MiB

Previous results on branch: feat/improve-app-start-integration

Startup times

Revision	Plain	With Sentry	Diff
24d43f6	1253.29 ms	1279.98 ms	26.69 ms
f8e4ca3	1253.12 ms	1277.04 ms	23.92 ms
7246d5e	1240.63 ms	1257.26 ms	16.62 ms
17e05e9	1255.80 ms	1281.59 ms	25.80 ms
9e7c614	1246.83 ms	1273.22 ms	26.38 ms
e423a36	1250.56 ms	1279.17 ms	28.61 ms

App size

Revision	Plain	With Sentry	Diff
24d43f6	8.38 MiB	9.73 MiB	1.35 MiB
f8e4ca3	8.38 MiB	9.73 MiB	1.35 MiB
7246d5e	8.38 MiB	9.73 MiB	1.35 MiB
17e05e9	8.38 MiB	9.73 MiB	1.36 MiB
9e7c614	8.38 MiB	9.73 MiB	1.35 MiB
e423a36	8.38 MiB	9.73 MiB	1.35 MiB

[github-actions]

Android Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	434.65 ms	477.33 ms	42.68 ms
Size	6.49 MiB	7.55 MiB	1.06 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
3751dbc	486.20 ms	538.68 ms	52.48 ms
24f71aa	358.49 ms	455.90 ms	97.41 ms
6aab859	306.27 ms	377.92 ms	71.65 ms
4efee31	308.92 ms	368.68 ms	59.76 ms
abfcdb5	416.55 ms	498.88 ms	82.33 ms
dd76eef	461.37 ms	540.55 ms	79.18 ms
aed5947	295.70 ms	348.18 ms	52.48 ms
0ceb89c	304.57 ms	357.18 ms	52.61 ms
25161f4	353.98 ms	431.94 ms	77.96 ms
d301b11	337.96 ms	395.04 ms	57.08 ms

App size

Revision	Plain	With Sentry	Diff
3751dbc	6.49 MiB	7.55 MiB	1.06 MiB
24f71aa	6.06 MiB	7.03 MiB	990.30 KiB
6aab859	6.15 MiB	7.13 MiB	999.97 KiB
4efee31	5.94 MiB	6.92 MiB	1003.76 KiB
abfcdb5	6.35 MiB	7.40 MiB	1.05 MiB
dd76eef	6.35 MiB	7.40 MiB	1.05 MiB
aed5947	5.94 MiB	6.96 MiB	1.02 MiB
0ceb89c	5.94 MiB	6.95 MiB	1.01 MiB
25161f4	6.27 MiB	7.20 MiB	960.44 KiB
d301b11	6.06 MiB	7.09 MiB	1.03 MiB

Previous results on branch: feat/improve-app-start-integration

Startup times

Revision	Plain	With Sentry	Diff
7246d5e	375.66 ms	398.65 ms	22.99 ms
e423a36	414.00 ms	467.22 ms	53.22 ms
17e05e9	483.30 ms	537.00 ms	53.70 ms
f8e4ca3	494.22 ms	557.17 ms	62.95 ms
9e7c614	509.15 ms	568.47 ms	59.31 ms
24d43f6	490.56 ms	552.66 ms	62.10 ms

App size

Revision	Plain	With Sentry	Diff
7246d5e	6.52 MiB	7.61 MiB	1.08 MiB
e423a36	6.52 MiB	7.61 MiB	1.08 MiB
17e05e9	6.49 MiB	7.55 MiB	1.06 MiB
f8e4ca3	6.49 MiB	7.55 MiB	1.06 MiB
9e7c614	6.49 MiB	7.55 MiB	1.06 MiB
24d43f6	6.49 MiB	7.55 MiB	1.06 MiB

[buenaflor]

adding this as a reminder that we probably don't need to return early

in case it is really null we only lose the sentryStartup span and firstFrameRender span

let's do that in a separate PR

[buenaflor]

@denrase can you fix the last test pls

[buenaflor]

nice!

only got one last comment

[denrase]

We have some 403 errors in CI. I'm gonna go ahead and merge, since they are not related to this PR.

Error: Exception in thread "main" org.kohsuke.github.HttpException: {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest","status":"403"}