Skip to content

Conversation

antonis
Copy link
Contributor

@antonis antonis commented Oct 10, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

This is a followup to #5065 removing the remaining usages in the project

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • 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.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@antonis antonis marked this pull request as ready for review October 10, 2025 15:36
Copy link
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 494.13 ms 547.44 ms 53.30 ms
Size 17.75 MiB 19.70 MiB 1.95 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
77061ed+dirty 369.55 ms 408.35 ms 38.80 ms
a2bb688+dirty 409.65 ms 410.45 ms 0.80 ms
000da7a 454.46 ms 445.00 ms -9.46 ms
a0b15d6 423.06 ms 437.77 ms 14.71 ms
af9331b 449.77 ms 479.20 ms 29.43 ms
69602ce 417.47 ms 443.52 ms 26.05 ms
7480abe+dirty 411.60 ms 405.81 ms -5.78 ms
8a4ce6f 422.88 ms 408.33 ms -14.55 ms
bc9680d 375.15 ms 401.12 ms 25.97 ms
46da307 455.92 ms 443.79 ms -12.13 ms

App size

Revision Plain With Sentry Diff
77061ed+dirty 17.75 MiB 19.68 MiB 1.94 MiB
a2bb688+dirty 17.75 MiB 19.70 MiB 1.95 MiB
000da7a 17.75 MiB 19.68 MiB 1.94 MiB
a0b15d6 17.75 MiB 20.15 MiB 2.41 MiB
af9331b 17.75 MiB 19.68 MiB 1.94 MiB
69602ce 17.75 MiB 19.68 MiB 1.94 MiB
7480abe+dirty 17.75 MiB 19.68 MiB 1.94 MiB
8a4ce6f 17.75 MiB 19.68 MiB 1.94 MiB
bc9680d 17.75 MiB 20.15 MiB 2.41 MiB
46da307 17.75 MiB 19.68 MiB 1.93 MiB

Copy link
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 353.48 ms 398.83 ms 45.35 ms
Size 7.15 MiB 8.43 MiB 1.28 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
21c9e75+dirty 356.73 ms 381.06 ms 24.33 ms
a0b15d6+dirty 414.33 ms 448.85 ms 34.52 ms
77061ed+dirty 359.40 ms 424.10 ms 64.70 ms
818a608+dirty 350.29 ms 397.38 ms 47.09 ms
b3b5b0d+dirty 361.42 ms 403.90 ms 42.48 ms
1853710+dirty 360.67 ms 396.28 ms 35.61 ms
a2bb688+dirty 371.19 ms 389.18 ms 17.99 ms
20daa0a+dirty 352.33 ms 424.30 ms 71.98 ms
a02e30b+dirty 346.13 ms 381.76 ms 35.62 ms
e2fa43d+dirty 326.56 ms 372.88 ms 46.32 ms

App size

Revision Plain With Sentry Diff
21c9e75+dirty 7.15 MiB 8.42 MiB 1.27 MiB
a0b15d6+dirty 7.15 MiB 8.42 MiB 1.27 MiB
77061ed+dirty 7.15 MiB 8.41 MiB 1.26 MiB
818a608+dirty 7.15 MiB 8.41 MiB 1.26 MiB
b3b5b0d+dirty 7.15 MiB 8.41 MiB 1.26 MiB
1853710+dirty 7.15 MiB 8.41 MiB 1.26 MiB
a2bb688+dirty 7.15 MiB 8.43 MiB 1.28 MiB
20daa0a+dirty 7.15 MiB 8.42 MiB 1.27 MiB
a02e30b+dirty 7.15 MiB 8.42 MiB 1.27 MiB
e2fa43d+dirty 7.15 MiB 8.42 MiB 1.27 MiB

Copy link
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1215.73 ms 1231.62 ms 15.89 ms
Size 2.63 MiB 3.99 MiB 1.36 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
1bea095+dirty 1234.14 ms 1233.96 ms -0.18 ms
20daa0a+dirty 1233.12 ms 1233.35 ms 0.22 ms
ba75c7c+dirty 1235.86 ms 1226.45 ms -9.41 ms
6479fd5+dirty 1228.48 ms 1231.32 ms 2.84 ms
7480abe+dirty 1220.53 ms 1244.18 ms 23.65 ms
c4e097a+dirty 1229.09 ms 1242.61 ms 13.53 ms
69602ce+dirty 1235.65 ms 1230.82 ms -4.83 ms
d916aa3+dirty 1221.02 ms 1228.98 ms 7.96 ms
d751a5d+dirty 1215.57 ms 1220.56 ms 4.99 ms
955f2eb+dirty 1235.06 ms 1253.88 ms 18.81 ms

App size

Revision Plain With Sentry Diff
1bea095+dirty 2.63 MiB 3.99 MiB 1.35 MiB
20daa0a+dirty 2.63 MiB 3.81 MiB 1.18 MiB
ba75c7c+dirty 2.63 MiB 3.81 MiB 1.18 MiB
6479fd5+dirty 2.63 MiB 3.96 MiB 1.33 MiB
7480abe+dirty 2.63 MiB 3.96 MiB 1.33 MiB
c4e097a+dirty 2.63 MiB 3.87 MiB 1.24 MiB
69602ce+dirty 2.63 MiB 3.91 MiB 1.28 MiB
d916aa3+dirty 2.63 MiB 3.81 MiB 1.18 MiB
d751a5d+dirty 2.63 MiB 3.98 MiB 1.34 MiB
955f2eb+dirty 2.63 MiB 3.98 MiB 1.35 MiB

Copy link
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1223.08 ms 1223.90 ms 0.81 ms
Size 3.19 MiB 4.56 MiB 1.37 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
1bea095+dirty 1218.93 ms 1212.37 ms -6.57 ms
20daa0a+dirty 1227.71 ms 1233.72 ms 6.01 ms
ba75c7c+dirty 1236.14 ms 1240.69 ms 4.55 ms
6479fd5+dirty 1207.80 ms 1215.12 ms 7.32 ms
7480abe+dirty 1219.84 ms 1223.60 ms 3.76 ms
c4e097a+dirty 1231.35 ms 1246.91 ms 15.56 ms
69602ce+dirty 1230.59 ms 1230.84 ms 0.24 ms
d916aa3+dirty 1211.02 ms 1221.33 ms 10.31 ms
d751a5d+dirty 1212.22 ms 1217.94 ms 5.71 ms
955f2eb+dirty 1225.78 ms 1239.27 ms 13.49 ms

App size

Revision Plain With Sentry Diff
1bea095+dirty 3.19 MiB 4.55 MiB 1.37 MiB
20daa0a+dirty 3.19 MiB 4.38 MiB 1.19 MiB
ba75c7c+dirty 3.19 MiB 4.38 MiB 1.19 MiB
6479fd5+dirty 3.19 MiB 4.53 MiB 1.35 MiB
7480abe+dirty 3.19 MiB 4.53 MiB 1.35 MiB
c4e097a+dirty 3.19 MiB 4.44 MiB 1.25 MiB
69602ce+dirty 3.19 MiB 4.48 MiB 1.29 MiB
d916aa3+dirty 3.19 MiB 4.38 MiB 1.19 MiB
d751a5d+dirty 3.19 MiB 4.54 MiB 1.36 MiB
955f2eb+dirty 3.19 MiB 4.55 MiB 1.36 MiB

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