ci: Unify derivedData path resolution with realpath by antonis · Pull Request #6761 · getsentry/sentry-react-native

antonis · 2026-09-21T11:37:08Z

📢 Type of change

Bugfix
New feature
Enhancement
Refactoring

📜 Description

Replace derivedData="$(cd "DerivedData" ; pwd -P)" with derivedData="$(realpath DerivedData)" across all macOS Xcode build jobs for readability. Functionally equivalent, since mkdir -p "DerivedData" precedes each — the directory is guaranteed to exist and both idioms resolve to an absolute, canonical (symlink-resolved) path.

Unified all six occurrences for consistency (not a one-off):

File	Job(s)
`sample-application.yml`	build-ios, build-tvos, rntester, build-catalyst
`sample-application-expo.yml`	expo build
`size-analysis.yml`	size analysis

💡 Motivation and Context

Closes #6759.

💚 How did you test it?

CI-only change; verified by the sample-application / expo / size-analysis workflows building successfully.

📝 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.
Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.
No breaking changes.

🔮 Next steps

Replace `derivedData="$(cd "DerivedData" ; pwd -P)"` with `derivedData="$(realpath DerivedData)"` across all Mac Xcode build jobs for readability. Functionally equivalent since `mkdir -p "DerivedData"` precedes each. Closes #6759 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

github-actions · 2026-09-21T11:37:29Z

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

ci: Unify derivedData path resolution with realpath by antonis in #6761

fix(ios): Fix Mac Catalyst linking the wrong Sentry.xcframework slice by antonis in #6758
chore(deps): bump the codeql-action group across 1 directory with 3 updates by dependabot in #6756
chore(deps): bump ruby/setup-ruby from 1.321.0 to 1.324.0 by dependabot in #6757
fix(core): Declare optional peer dependencies to resolve phantom imports under strict package managers by antonis in #6729
Add missing exports and update public API report by antonis in #6731
test(ios): Reenable Session Replay E2E test by antonis in #6737
fix(ios): Honor shutdownTimeout on iOS by antonis in #6749

_{🤖 This preview updates automatically when you update the PR.}

sentry · 2026-09-21T11:50:31Z

📲 Install Builds

Android

🔗 App Name	App ID	Version	Configuration
Sentry RN	io.sentry.reactnative.sample	8.27.0 (107)	Release

⚙️ sentry-react-native Build Distribution Settings

github-actions · 2026-09-21T11:52:14Z

iOS (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	3864.87 ms	1245.02 ms	-2619.85 ms
Size	5.15 MiB	6.90 MiB	1.75 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
`ecf47a2`+dirty	3835.20 ms	1222.43 ms	-2612.77 ms
`3817909`+dirty	1183.90 ms	1187.50 ms	3.60 ms
`5ee78d6`+dirty	3847.52 ms	1224.27 ms	-2623.25 ms
`890d145`+dirty	1223.59 ms	1231.37 ms	7.78 ms
`e763471`+dirty	3841.16 ms	1218.06 ms	-2623.09 ms
`3d31fcf`+dirty	3838.09 ms	1223.46 ms	-2614.63 ms
`9210ae6`+dirty	3815.93 ms	1214.14 ms	-2601.79 ms
`26843eb`+dirty	3861.60 ms	1238.77 ms	-2622.82 ms
`7a89652`+dirty	3861.46 ms	1229.61 ms	-2631.85 ms
`9ad4522`+dirty	3850.40 ms	1222.40 ms	-2628.00 ms

App size

Revision	Plain	With Sentry	Diff
`ecf47a2`+dirty	4.98 MiB	6.46 MiB	1.49 MiB
`3817909`+dirty	3.38 MiB	4.73 MiB	1.35 MiB
`5ee78d6`+dirty	5.15 MiB	6.69 MiB	1.53 MiB
`890d145`+dirty	3.38 MiB	4.77 MiB	1.38 MiB
`e763471`+dirty	4.98 MiB	6.51 MiB	1.53 MiB
`3d31fcf`+dirty	4.98 MiB	6.56 MiB	1.58 MiB
`9210ae6`+dirty	5.15 MiB	6.68 MiB	1.53 MiB
`26843eb`+dirty	4.98 MiB	6.53 MiB	1.55 MiB
`7a89652`+dirty	5.15 MiB	6.70 MiB	1.55 MiB
`9ad4522`+dirty	4.98 MiB	6.56 MiB	1.58 MiB

github-actions · 2026-09-21T11:58:32Z

Android (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	419.35 ms	450.70 ms	31.35 ms
Size	50.56 MiB	56.49 MiB	5.93 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
`0d9949d`+dirty	403.57 ms	437.00 ms	33.43 ms
`64630e5`+dirty	419.18 ms	464.58 ms	45.40 ms
`7ac3378`+dirty	404.78 ms	439.84 ms	35.06 ms
`e763471`+dirty	426.57 ms	458.21 ms	31.64 ms
`4966363`+dirty	400.04 ms	431.08 ms	31.04 ms
`9474ead`+dirty	411.45 ms	446.80 ms	35.35 ms
`038a6d7`+dirty	524.82 ms	531.92 ms	7.10 ms
`37a2091`+dirty	407.82 ms	441.22 ms	33.40 ms
`7d6fd3a`+dirty	403.02 ms	422.56 ms	19.54 ms
`57e0069`+dirty	413.82 ms	453.47 ms	39.65 ms

App size

Revision	Plain	With Sentry	Diff
`0d9949d`+dirty	43.75 MiB	48.13 MiB	4.37 MiB
`64630e5`+dirty	49.74 MiB	54.82 MiB	5.07 MiB
`7ac3378`+dirty	43.75 MiB	48.13 MiB	4.37 MiB
`e763471`+dirty	49.74 MiB	54.85 MiB	5.11 MiB
`4966363`+dirty	48.30 MiB	53.54 MiB	5.24 MiB
`9474ead`+dirty	48.30 MiB	53.61 MiB	5.30 MiB
`038a6d7`+dirty	48.30 MiB	53.60 MiB	5.30 MiB
`37a2091`+dirty	48.30 MiB	53.58 MiB	5.28 MiB
`7d6fd3a`+dirty	43.75 MiB	48.14 MiB	4.39 MiB
`57e0069`+dirty	49.74 MiB	54.85 MiB	5.11 MiB

github-actions · 2026-09-21T12:08:58Z

Android (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	416.06 ms	468.80 ms	52.73 ms
Size	50.56 MiB	56.49 MiB	5.93 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
`f3215d3`+dirty	396.53 ms	436.66 ms	40.13 ms
`d0e3b3e`+dirty	443.19 ms	480.00 ms	36.81 ms
`a5d243c`+dirty	499.20 ms	525.62 ms	26.42 ms
`7fd0012`+dirty	422.41 ms	445.57 ms	23.17 ms
`0a147b2`+dirty	442.80 ms	522.24 ms	79.44 ms
`d7d54c6`+dirty	423.54 ms	498.56 ms	75.02 ms
`c823bb5`+dirty	468.26 ms	516.16 ms	47.90 ms
`5789645`+dirty	502.66 ms	594.19 ms	91.53 ms
`40c9884`+dirty	453.76 ms	478.98 ms	25.22 ms
`d771cd5`+dirty	432.29 ms	464.38 ms	32.09 ms

App size

Revision	Plain	With Sentry	Diff
`f3215d3`+dirty	48.30 MiB	53.49 MiB	5.19 MiB
`d0e3b3e`+dirty	49.74 MiB	55.09 MiB	5.34 MiB
`a5d243c`+dirty	48.30 MiB	53.54 MiB	5.23 MiB
`7fd0012`+dirty	50.56 MiB	56.46 MiB	5.90 MiB
`0a147b2`+dirty	49.74 MiB	55.08 MiB	5.34 MiB
`d7d54c6`+dirty	49.74 MiB	55.09 MiB	5.34 MiB
`c823bb5`+dirty	48.30 MiB	53.58 MiB	5.28 MiB
`5789645`+dirty	49.74 MiB	54.85 MiB	5.11 MiB
`40c9884`+dirty	49.74 MiB	55.09 MiB	5.34 MiB
`d771cd5`+dirty	49.74 MiB	55.36 MiB	5.62 MiB

github-actions · 2026-09-21T12:39:29Z

iOS (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	3809.33 ms	1205.15 ms	-2604.18 ms
Size	5.15 MiB	6.90 MiB	1.75 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
`68ae91b`+dirty	3836.36 ms	1222.04 ms	-2614.32 ms
`0b5a379`+dirty	3857.69 ms	1230.34 ms	-2627.35 ms
`bf168a4`+dirty	3846.83 ms	1221.33 ms	-2625.49 ms
`f170ec3`+dirty	3844.74 ms	1222.67 ms	-2622.07 ms
`68672fc`+dirty	3832.22 ms	1228.29 ms	-2603.93 ms
`84cba31`+dirty	3830.40 ms	1204.44 ms	-2625.96 ms
`2c735cc`+dirty	1223.33 ms	1224.38 ms	1.04 ms
`b9bebee`+dirty	3858.02 ms	1231.92 ms	-2626.11 ms
`a50b33d`+dirty	1207.11 ms	1212.10 ms	5.00 ms
`5569641`+dirty	3824.35 ms	1210.78 ms	-2613.57 ms

App size

Revision	Plain	With Sentry	Diff
`68ae91b`+dirty	4.98 MiB	6.46 MiB	1.48 MiB
`0b5a379`+dirty	5.15 MiB	6.70 MiB	1.54 MiB
`bf168a4`+dirty	4.98 MiB	6.47 MiB	1.49 MiB
`f170ec3`+dirty	5.15 MiB	6.69 MiB	1.53 MiB
`68672fc`+dirty	5.15 MiB	6.71 MiB	1.55 MiB
`84cba31`+dirty	4.98 MiB	6.55 MiB	1.57 MiB
`2c735cc`+dirty	3.38 MiB	4.74 MiB	1.35 MiB
`b9bebee`+dirty	5.15 MiB	6.68 MiB	1.53 MiB
`a50b33d`+dirty	3.38 MiB	4.73 MiB	1.35 MiB
`5569641`+dirty	5.15 MiB	6.67 MiB	1.51 MiB

alwx

Looks good.

antonis added the ready-to-merge Triggers the full CI test suite label Sep 21, 2026

antonis marked this pull request as ready for review September 21, 2026 12:03

antonis requested review from alwx and lucas-zimerman as code owners September 21, 2026 12:03

antonis mentioned this pull request Sep 21, 2026

fix(ios): Fix Mac Catalyst linking the wrong Sentry.xcframework slice #6758

Merged

11 tasks

alwx approved these changes Sep 22, 2026

View reviewed changes

antonis merged commit 38f2efb into main Sep 22, 2026
141 of 147 checks passed

antonis deleted the ci/unify-deriveddata-realpath branch September 22, 2026 09:37

Uh oh!

Conversation

antonis commented Sep 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📢 Type of change

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

🔮 Next steps

Uh oh!

github-actions Bot commented Sep 21, 2026

Semver Impact of This PR

Uh oh!

sentry Bot commented Sep 21, 2026

📲 Install Builds

Android

Uh oh!

github-actions Bot commented Sep 21, 2026

iOS (legacy) Performance metrics 🚀

Baseline results on branch: main

Startup times

App size

Uh oh!

github-actions Bot commented Sep 21, 2026

Android (legacy) Performance metrics 🚀

Baseline results on branch: main

Startup times

App size

Uh oh!

github-actions Bot commented Sep 21, 2026

Android (new) Performance metrics 🚀

Baseline results on branch: main

Startup times

App size

Uh oh!

github-actions Bot commented Sep 21, 2026

iOS (new) Performance metrics 🚀

Baseline results on branch: main

Startup times

App size

Uh oh!

alwx left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

antonis commented Sep 21, 2026 •

edited

Loading