Skip to content

Conversation

@antonis
Copy link
Contributor

@antonis antonis commented Nov 11, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Breaking changes:

SubPRs:

💡 Motivation and Context

See #5273

💚 How did you test it?

CI, Manual

📝 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

@antonis
Copy link
Contributor Author

antonis commented Nov 11, 2025

Marking as Blocked while the Cocoa v9 is in Alpha

itaybre and others added 3 commits November 11, 2025 16:38
* fix: Fixes SentryScreenFrames use after being converted to Swift

* Fix build

---------

Co-authored-by: Antonis Lilis <antonis.lilis@sentry.io>
* fix: Fixes SentryScreenFrames use after being converted to Swift

* Fix build

* Use non-experimental enableLogs

* bump target

* remove enable tracing

* Update enableLogs tests

---------

Co-authored-by: Itay Brenner <itay.brenner@sentry.io>
Co-authored-by: Itay Brenner <itaybrenner@hotmail.com>
Co-authored-by: Antonis Lilis <antonis.lilis@sentry.io>
* fix: Fixes SentryScreenFrames use after being converted to Swift

* Fix build

* Use non-experimental enableLogs

* bump target

* remove enable tracing

* Update enableLogs tests

* chore: Fix RNSentry after SentryFramesTracker conversion to swift

* Run linter

* Remove unused debug image provider imports

* Remove imports

* chore: Remove deprecated user property

* chore: Remove use of deprecate integrations API (#5304)

* Remove use of integrations in tests

* chore: Remove use of deprecated prop (#5322)

* ref: Move options to wrapper

---------

Co-authored-by: Antonis Lilis <antonis.lilis@sentry.io>
Co-authored-by: Denis Andrasec <denrase@gmail.com>
Co-authored-by: Noah Martin <noahm444@gmail.com>
@antonis antonis added the ready-to-merge Triggers the full CI test suite label Nov 11, 2025
@antonis
Copy link
Contributor Author

antonis commented Nov 11, 2025

Adding the ready-to-merge label to trigger all tests but it's not really ready to merge 😅

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 370.87 ms 383.05 ms 12.17 ms
Size 43.94 MiB 48.85 MiB 4.91 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
55b77fc+dirty 410.46 ms 414.11 ms 3.65 ms
266bc7e+dirty 378.00 ms 392.81 ms 14.81 ms
21c9e75+dirty 356.73 ms 381.06 ms 24.33 ms
0b64753+dirty 358.55 ms 429.16 ms 70.61 ms
8d89cc9+dirty 357.69 ms 415.79 ms 58.10 ms
2b89ce9+dirty 372.22 ms 417.06 ms 44.84 ms
c7f264b+dirty 356.98 ms 407.46 ms 50.48 ms
1d62dde+dirty 366.59 ms 408.80 ms 42.21 ms
df83aad+dirty 493.94 ms 521.76 ms 27.82 ms
8d20c8e+dirty 469.90 ms 502.00 ms 32.10 ms

App size

Revision Plain With Sentry Diff
55b77fc+dirty 43.94 MiB 48.82 MiB 4.88 MiB
266bc7e+dirty 43.94 MiB 48.82 MiB 4.88 MiB
21c9e75+dirty 7.15 MiB 8.42 MiB 1.27 MiB
0b64753+dirty 7.15 MiB 8.42 MiB 1.27 MiB
8d89cc9+dirty 7.15 MiB 8.41 MiB 1.26 MiB
2b89ce9+dirty 7.15 MiB 8.41 MiB 1.26 MiB
c7f264b+dirty 7.15 MiB 8.41 MiB 1.26 MiB
1d62dde+dirty 7.15 MiB 8.46 MiB 1.31 MiB
df83aad+dirty 43.94 MiB 48.82 MiB 4.88 MiB
8d20c8e+dirty 43.94 MiB 48.81 MiB 4.88 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 452.02 ms 523.54 ms 71.52 ms
Size 43.75 MiB 48.02 MiB 4.27 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
5ee3314+dirty 415.80 ms 426.14 ms 10.34 ms
55b77fc+dirty 411.87 ms 417.16 ms 5.29 ms
88890fe+dirty 350.94 ms 365.74 ms 14.80 ms
266bc7e+dirty 485.02 ms 551.94 ms 66.92 ms
64cd15c 439.02 ms 427.63 ms -11.39 ms
7be1f99 454.83 ms 461.36 ms 6.53 ms
df83aad+dirty 420.87 ms 434.98 ms 14.11 ms
4167e15+dirty 344.77 ms 367.78 ms 23.01 ms
8d20c8e+dirty 480.27 ms 503.35 ms 23.08 ms
136effd+dirty 398.57 ms 494.91 ms 96.34 ms

App size

Revision Plain With Sentry Diff
5ee3314+dirty 17.75 MiB 19.70 MiB 1.95 MiB
55b77fc+dirty 43.75 MiB 47.99 MiB 4.24 MiB
88890fe+dirty 17.75 MiB 19.71 MiB 1.96 MiB
266bc7e+dirty 43.75 MiB 47.99 MiB 4.24 MiB
64cd15c 17.75 MiB 20.15 MiB 2.41 MiB
7be1f99 17.75 MiB 20.15 MiB 2.41 MiB
df83aad+dirty 43.75 MiB 47.99 MiB 4.24 MiB
4167e15+dirty 17.75 MiB 19.74 MiB 1.99 MiB
8d20c8e+dirty 43.75 MiB 47.99 MiB 4.23 MiB
136effd+dirty 43.75 MiB 47.99 MiB 4.23 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1208.67 ms 1218.14 ms 9.47 ms
Size 3.41 MiB 4.64 MiB 1.22 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
9f211e3+dirty 1215.38 ms 1218.15 ms 2.77 ms
95aaf8a+dirty 1206.83 ms 1213.65 ms 6.81 ms
98f632c+dirty 1221.38 ms 1229.26 ms 7.88 ms
d1fd647+dirty 1218.16 ms 1225.82 ms 7.65 ms
49ef936+dirty 1221.27 ms 1221.60 ms 0.34 ms
46bd012+dirty 1231.78 ms 1212.30 ms -19.47 ms
6a70a7e+dirty 1231.40 ms 1239.49 ms 8.09 ms
07808fb+dirty 1240.76 ms 1251.00 ms 10.24 ms
ec14be7+dirty 1229.62 ms 1230.53 ms 0.91 ms
77061ed+dirty 1210.77 ms 1218.45 ms 7.68 ms

App size

Revision Plain With Sentry Diff
9f211e3+dirty 3.19 MiB 4.48 MiB 1.29 MiB
95aaf8a+dirty 3.19 MiB 4.44 MiB 1.25 MiB
98f632c+dirty 3.19 MiB 4.38 MiB 1.19 MiB
d1fd647+dirty 3.19 MiB 4.56 MiB 1.37 MiB
49ef936+dirty 3.19 MiB 4.54 MiB 1.36 MiB
46bd012+dirty 3.19 MiB 4.38 MiB 1.19 MiB
6a70a7e+dirty 3.19 MiB 4.54 MiB 1.36 MiB
07808fb+dirty 3.19 MiB 4.56 MiB 1.37 MiB
ec14be7+dirty 3.19 MiB 4.54 MiB 1.36 MiB
77061ed+dirty 3.19 MiB 4.54 MiB 1.36 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1222.55 ms 1224.45 ms 1.90 ms
Size 3.41 MiB 4.64 MiB 1.22 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
9f211e3+dirty 1218.80 ms 1233.88 ms 15.08 ms
95aaf8a+dirty 1234.78 ms 1241.94 ms 7.16 ms
98f632c+dirty 1236.40 ms 1241.62 ms 5.22 ms
d1fd647+dirty 1219.35 ms 1233.18 ms 13.83 ms
49ef936+dirty 1228.42 ms 1217.09 ms -11.33 ms
46bd012+dirty 1220.49 ms 1226.89 ms 6.40 ms
6a70a7e+dirty 1225.82 ms 1230.79 ms 4.98 ms
07808fb+dirty 1233.31 ms 1232.77 ms -0.54 ms
ec14be7+dirty 1234.64 ms 1245.54 ms 10.90 ms
77061ed+dirty 1233.16 ms 1234.88 ms 1.71 ms

App size

Revision Plain With Sentry Diff
9f211e3+dirty 2.63 MiB 3.91 MiB 1.28 MiB
95aaf8a+dirty 2.63 MiB 3.87 MiB 1.24 MiB
98f632c+dirty 2.63 MiB 3.81 MiB 1.18 MiB
d1fd647+dirty 2.63 MiB 3.99 MiB 1.36 MiB
49ef936+dirty 2.63 MiB 3.98 MiB 1.34 MiB
46bd012+dirty 2.63 MiB 3.81 MiB 1.18 MiB
6a70a7e+dirty 2.63 MiB 3.98 MiB 1.34 MiB
07808fb+dirty 2.63 MiB 3.99 MiB 1.36 MiB
ec14be7+dirty 2.63 MiB 3.98 MiB 1.34 MiB
77061ed+dirty 2.63 MiB 3.98 MiB 1.34 MiB

@antonis antonis changed the title chore(deps): update Cocoa SDK to v9.0.0-alpha.0 chore(deps): update Cocoa SDK to v9.0.0-alpha Nov 13, 2025
@antonis
Copy link
Contributor Author

antonis commented Nov 13, 2025

fix(podspec): Cocoa V9 allow any alpha version

Thank you @philipphofmann 🦸

antonis and others added 2 commits November 13, 2025 12:00
* fix: Fixes SentryScreenFrames use after being converted to Swift

* Fix build

* Use non-experimental enableLogs

* bump target

* remove enable tracing

* Update enableLogs tests

* chore: Fix RNSentry after SentryFramesTracker conversion to swift

* Run linter

* Remove unused debug image provider imports

* Remove imports

* chore: Remove deprecated user property

* chore: Remove use of deprecate integrations API (#5304)

* Remove use of integrations in tests

* chore: Remove use of deprecated prop (#5322)

* ref: Move options to wrapper

* chore(sample): Bump MacOS sample to macos 12.0 wich is the minimum for v9

---------

Co-authored-by: Itay Brenner <itay.brenner@sentry.io>
Co-authored-by: Itay Brenner <itaybrenner@hotmail.com>
Co-authored-by: Denis Andrasec <denrase@gmail.com>
Co-authored-by: Noah Martin <noahm444@gmail.com>
* fix: Fixes SentryScreenFrames use after being converted to Swift

* Fix build

* Use non-experimental enableLogs

* bump target

* remove enable tracing

* Update enableLogs tests

* chore: Fix RNSentry after SentryFramesTracker conversion to swift

* Run linter

* Remove unused debug image provider imports

* Remove imports

* chore: Remove deprecated user property

* chore: Remove use of deprecate integrations API (#5304)

* Remove use of integrations in tests

* chore: Remove use of deprecated prop (#5322)

* ref: Move options to wrapper

* chore(sample): Bump MacOS sample to macos 12.0 wich is the minimum for v9

* chore(e2e): Cocoa-v9: Bump E2E to iOS 15.0

* Bump to 15.1 due to RN 0.81.0 requirements

---------

Co-authored-by: Itay Brenner <itay.brenner@sentry.io>
Co-authored-by: Itay Brenner <itaybrenner@hotmail.com>
Co-authored-by: Denis Andrasec <denrase@gmail.com>
Co-authored-by: Noah Martin <noahm444@gmail.com>
@antonis antonis removed the ready-to-merge Triggers the full CI test suite label Nov 13, 2025
@antonis antonis added the ready-to-merge Triggers the full CI test suite label Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Blocked ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants