feat: Read `sentry.options.json` during cocoa init #4447

krystofwoldrich · 2025-01-15T10:26:37Z

📢 Type of change

New feature

⛓️ PR Chain

📜 Description

This PR adds [RNSentrySDK start] function (exported by RNSentry/RNSentry.h) which allows users to initialize Sentry Cocoa in RN applications without any additional params because the options are loaded from sentry.options.json.

This PR adds automatic copy of sentry.options.json resource from root of the RN project. This works as long as users setuped sentry-xcode.sh on their Bundle RN build phase.

💡 Motivation and Context

Initialize native SDKs before JS #3608

💚 How did you test it?

Only in Sample app for now, tests will follow.

📝 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

… on iOS

…platforms`

github-actions · 2025-01-15T10:27:04Z

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

Generated by 🚫 dangerJS against 6d81a10

github-actions · 2025-01-15T10:52:33Z

Android (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	650.96 ms	736.00 ms	85.04 ms
Size	7.15 MiB	8.38 MiB	1.23 MiB

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	412.92 ms	538.19 ms	125.27 ms

App size

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	7.15 MiB	8.38 MiB	1.23 MiB

Previous results on branch: kw-add-read-ios-options-file

Startup times

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	424.00 ms	469.60 ms	45.60 ms

App size

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	7.15 MiB	8.38 MiB	1.23 MiB

github-actions · 2025-01-15T10:57:59Z

iOS (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1219.41 ms	1228.23 ms	8.82 ms
Size	2.63 MiB	3.69 MiB	1.05 MiB

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	1230.51 ms	1223.85 ms	-6.66 ms

App size

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	2.63 MiB	3.69 MiB	1.05 MiB

Previous results on branch: kw-add-read-ios-options-file

Startup times

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	1224.98 ms	1225.46 ms	0.48 ms

App size

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	2.63 MiB	3.69 MiB	1.05 MiB

github-actions · 2025-01-15T11:04:07Z

iOS (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1221.30 ms	1225.33 ms	4.03 ms
Size	3.19 MiB	4.25 MiB	1.07 MiB

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	1220.98 ms	1219.06 ms	-1.92 ms

App size

Revision	Plain	With Sentry	Diff
`a1ab76a`+dirty	3.19 MiB	4.25 MiB	1.06 MiB

Previous results on branch: kw-add-read-ios-options-file

Startup times

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	1227.17 ms	1235.08 ms	7.91 ms

App size

Revision	Plain	With Sentry	Diff
`eda229b`+dirty	3.19 MiB	4.25 MiB	1.07 MiB

…to kw-add-read-ios-options-file

github-actions · 2025-01-15T13:47:28Z

Android (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	398.02 ms	424.46 ms	26.44 ms
Size	17.75 MiB	20.11 MiB	2.37 MiB

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

Revision	Plain	With Sentry	Diff
`a1ab76a`	339.87 ms	333.56 ms	-6.31 ms

App size

Revision	Plain	With Sentry	Diff
`a1ab76a`	17.75 MiB	20.11 MiB	2.37 MiB

Previous results on branch: kw-add-read-ios-options-file

Startup times

Revision	Plain	With Sentry	Diff
`eda229b`	461.02 ms	578.98 ms	117.96 ms

App size

Revision	Plain	With Sentry	Diff
`eda229b`	17.75 MiB	20.11 MiB	2.37 MiB

brustolin

ObjC code looks good.

…l-init

…os-options-file

antonis

The code looks good and the sample app worked as expected for me with [RNSentrySDK start] 🚀
Thank you for the added tests 🙇

krystofwoldrich added 16 commits January 14, 2025 13:47

chore: Initialize RNSentryTimeToDisplay during native module init…

4328298

… on iOS

ref(ios): Extract Cocoa SDK init into standalone file

27d76de

fix lint

5a28c03

rename rnsentrysdk to rnsentrystart

8b67bf1

rename class name also to rnsentrystart, fix tests

05cef29

explicitly import only used classes

778478f

fix call cocoa start not itself

3d3d0b4

also extract didBecomeActive notification

e2e40fa

add explanation comment to native crash handling

83106a7

explain notification purpose

742f28c

fix sdk name

4897aa3

fix native sdk not react native package

d37ebdd

fix lint sentry start

13a0b89

add changelog

d7270b1

feat(experimental): Add native startWithConfigureOptions for Apple …

580590b

…platforms`

feat: Read sentry.options.json during cocoa init

2cc92f2

Update CHANGELOG.md

592b903

krystofwoldrich added 2 commits January 15, 2025 14:33

remove global processor

a2717bd

Merge remote-tracking branch 'origin/kw-add-read-ios-options-file' in…

5ecd6b2

…to kw-add-read-ios-options-file

krystofwoldrich requested review from philipphofmann, brustolin and philprime January 15, 2025 13:44

Update code docs for start methods

890133a

brustolin reviewed Jan 17, 2025

View reviewed changes

Merge branch 'main' into kw-ref-ios-sdk-init

2d021e8

krystofwoldrich added 4 commits January 20, 2025 15:48

fix lint

cfc6293

remove unused rnSentry

f2a041e

fix changelog

fc4ff7e

Merge branch 'kw-ref-ios-sdk-init' into kw-add-react-native-ios-manua…

879044f

…l-init

krystofwoldrich mentioned this pull request Jan 20, 2025

feat(experimental): Add native startWithConfigureOptions for Apple platforms #4444

Merged

7 tasks

krystofwoldrich added 7 commits January 20, 2025 18:57

wip! add manual init tests

dec9869

add tests

22e88c7

Merge branch 'kw-add-react-native-ios-manual-init' into kw-add-read-i…

4ac4896

…os-options-file

add tests

1baac18

remove unavailable and unused global events processors import

f92edd9

Merge branch 'kw-add-react-native-ios-manual-init' into kw-add-read-i…

2662dbd

…os-options-file

fix c-format

7baad41

krystofwoldrich marked this pull request as ready for review January 21, 2025 16:16

krystofwoldrich requested review from lucas-zimerman and antonis as code owners January 21, 2025 16:16

krystofwoldrich requested a review from brustolin January 21, 2025 16:16

antonis approved these changes Jan 22, 2025

View reviewed changes

remove old sentry-cocoa init code

6d81a10

Base automatically changed from kw-add-react-native-ios-manual-init to capture-app-start-errors January 22, 2025 13:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Read `sentry.options.json` during cocoa init #4447

feat: Read `sentry.options.json` during cocoa init #4447

krystofwoldrich commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 •

edited

Loading

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 •

edited

Loading

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 •

edited

Loading

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

brustolin left a comment

antonis left a comment

feat: Read sentry.options.json during cocoa init #4447

Are you sure you want to change the base?

feat: Read sentry.options.json during cocoa init #4447

Conversation

krystofwoldrich commented Jan 15, 2025 • edited Loading

📢 Type of change

⛓️ PR Chain

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

🔮 Next steps

github-actions bot commented Jan 15, 2025 • edited Loading

github-actions bot commented Jan 15, 2025 • edited Loading

Android (new) Performance metrics 🚀

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 • edited Loading

iOS (legacy) Performance metrics 🚀

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 • edited Loading

iOS (new) Performance metrics 🚀

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

github-actions bot commented Jan 15, 2025 • edited Loading

Android (legacy) Performance metrics 🚀

Baseline results on branch: kw-add-react-native-ios-manual-init

Startup times

App size

Previous results on branch: kw-add-read-ios-options-file

Startup times

App size

brustolin left a comment

Choose a reason for hiding this comment

antonis left a comment

Choose a reason for hiding this comment

feat: Read `sentry.options.json` during cocoa init #4447

feat: Read `sentry.options.json` during cocoa init #4447

krystofwoldrich commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading

github-actions bot commented Jan 15, 2025 •

edited

Loading