Skip to content

Conversation

@alwx
Copy link
Contributor

@alwx alwx commented Dec 15, 2025

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Fixes #5457

💡 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

@alwx alwx self-assigned this Dec 15, 2025
@antonis antonis added the ready-to-merge Triggers the full CI test suite label Dec 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 361.54 ms 373.70 ms 12.16 ms
Size 43.94 MiB 48.90 MiB 4.96 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
46bd012+dirty 333.76 ms 359.24 ms 25.48 ms
8ece263+dirty 369.44 ms 414.65 ms 45.21 ms
36841a6+dirty 404.22 ms 450.06 ms 45.84 ms
c7f264b+dirty 356.98 ms 407.46 ms 50.48 ms
1853710+dirty 360.67 ms 396.28 ms 35.61 ms
21c9e75+dirty 356.73 ms 381.06 ms 24.33 ms
46da307+dirty 356.62 ms 415.02 ms 58.40 ms
e2fa43d+dirty 326.56 ms 372.88 ms 46.32 ms
c08359e+dirty 406.04 ms 428.87 ms 22.83 ms
05bef0e+dirty 451.63 ms 533.90 ms 82.27 ms

App size

Revision Plain With Sentry Diff
46bd012+dirty 7.15 MiB 8.42 MiB 1.27 MiB
8ece263+dirty 7.15 MiB 8.41 MiB 1.26 MiB
36841a6+dirty 43.94 MiB 48.90 MiB 4.96 MiB
c7f264b+dirty 7.15 MiB 8.41 MiB 1.26 MiB
1853710+dirty 7.15 MiB 8.41 MiB 1.26 MiB
21c9e75+dirty 7.15 MiB 8.42 MiB 1.27 MiB
46da307+dirty 7.15 MiB 8.41 MiB 1.26 MiB
e2fa43d+dirty 7.15 MiB 8.42 MiB 1.27 MiB
c08359e+dirty 7.15 MiB 8.42 MiB 1.27 MiB
05bef0e+dirty 7.15 MiB 8.43 MiB 1.28 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 406.29 ms 426.00 ms 19.71 ms
Size 43.75 MiB 48.07 MiB 4.32 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
7be1f99 454.83 ms 461.36 ms 6.53 ms
3099014+dirty 439.20 ms 468.40 ms 29.20 ms
36841a6+dirty 502.96 ms 547.14 ms 44.18 ms
20d5eaa 377.62 ms 406.50 ms 28.88 ms
a31630c+dirty 408.91 ms 416.80 ms 7.89 ms
b3b5b0d 399.82 ms 419.20 ms 19.38 ms
6fee48d+dirty 407.42 ms 437.71 ms 30.29 ms
69602ce 417.47 ms 443.52 ms 26.05 ms
eb07ba3 470.04 ms 473.35 ms 3.31 ms
5c16cdc+dirty 423.48 ms 452.35 ms 28.88 ms

App size

Revision Plain With Sentry Diff
7be1f99 17.75 MiB 20.15 MiB 2.41 MiB
3099014+dirty 17.75 MiB 19.70 MiB 1.95 MiB
36841a6+dirty 43.75 MiB 48.07 MiB 4.32 MiB
20d5eaa 17.75 MiB 20.15 MiB 2.41 MiB
a31630c+dirty 17.75 MiB 19.68 MiB 1.94 MiB
b3b5b0d 17.75 MiB 19.68 MiB 1.94 MiB
6fee48d+dirty 17.75 MiB 19.68 MiB 1.94 MiB
69602ce 17.75 MiB 19.68 MiB 1.94 MiB
eb07ba3 17.75 MiB 20.15 MiB 2.41 MiB
5c16cdc+dirty 17.75 MiB 19.68 MiB 1.94 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1214.41 ms 1214.84 ms 0.44 ms
Size 3.41 MiB 4.58 MiB 1.17 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2adbd1e+dirty 1220.65 ms 1230.20 ms 9.56 ms
4604da9+dirty 1208.67 ms 1208.12 ms -0.54 ms
9b50d32+dirty 1210.36 ms 1218.41 ms 8.05 ms
f234eb4+dirty 1233.52 ms 1235.78 ms 2.26 ms
90afdd3+dirty 1216.17 ms 1225.55 ms 9.38 ms
161947d+dirty 1203.45 ms 1217.83 ms 14.38 ms
1ef8a04+dirty 1184.42 ms 1186.44 ms 2.02 ms
6416d6c+dirty 1222.83 ms 1222.04 ms -0.79 ms
ebf60f9+dirty 1218.85 ms 1212.53 ms -6.32 ms
ea3e26e+dirty 1216.61 ms 1214.15 ms -2.47 ms

App size

Revision Plain With Sentry Diff
2adbd1e+dirty 3.19 MiB 4.56 MiB 1.38 MiB
4604da9+dirty 3.19 MiB 4.58 MiB 1.39 MiB
9b50d32+dirty 3.41 MiB 4.59 MiB 1.18 MiB
f234eb4+dirty 3.19 MiB 4.57 MiB 1.38 MiB
90afdd3+dirty 3.19 MiB 4.55 MiB 1.37 MiB
161947d+dirty 3.19 MiB 4.56 MiB 1.37 MiB
1ef8a04+dirty 3.41 MiB 4.59 MiB 1.18 MiB
6416d6c+dirty 3.41 MiB 4.59 MiB 1.18 MiB
ebf60f9+dirty 3.41 MiB 4.67 MiB 1.25 MiB
ea3e26e+dirty 3.41 MiB 4.58 MiB 1.17 MiB

@github-actions
Copy link
Contributor

github-actions bot commented Dec 15, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1224.66 ms 1223.61 ms -1.05 ms
Size 3.41 MiB 4.58 MiB 1.17 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
2adbd1e+dirty 1207.51 ms 1218.98 ms 11.47 ms
4604da9+dirty 1232.59 ms 1232.26 ms -0.33 ms
9b50d32+dirty 1216.53 ms 1221.64 ms 5.10 ms
f234eb4+dirty 1227.22 ms 1228.45 ms 1.22 ms
90afdd3+dirty 1233.90 ms 1240.90 ms 7.00 ms
161947d+dirty 1203.43 ms 1217.00 ms 13.57 ms
1ef8a04+dirty 1226.14 ms 1223.15 ms -2.99 ms
6416d6c+dirty 1220.38 ms 1222.98 ms 2.60 ms
ebf60f9+dirty 1217.66 ms 1214.82 ms -2.84 ms
ea3e26e+dirty 1229.13 ms 1228.46 ms -0.67 ms

App size

Revision Plain With Sentry Diff
2adbd1e+dirty 2.63 MiB 4.00 MiB 1.36 MiB
4604da9+dirty 2.63 MiB 4.01 MiB 1.38 MiB
9b50d32+dirty 3.41 MiB 4.59 MiB 1.18 MiB
f234eb4+dirty 2.63 MiB 4.00 MiB 1.37 MiB
90afdd3+dirty 2.63 MiB 3.99 MiB 1.35 MiB
161947d+dirty 2.63 MiB 3.99 MiB 1.36 MiB
1ef8a04+dirty 3.41 MiB 4.59 MiB 1.18 MiB
6416d6c+dirty 3.41 MiB 4.59 MiB 1.18 MiB
ebf60f9+dirty 3.41 MiB 4.67 MiB 1.25 MiB
ea3e26e+dirty 3.41 MiB 4.58 MiB 1.17 MiB

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.

Thank you for the PR!
Lets merge it once all tests are passing

Comment on lines 280 to 288
- uses: ruby/setup-ruby@v1
if: ${{ matrix.platform == 'ios' }}
with:
working-directory: dev-packages/e2e-tests/react-native-versions/${{ matrix.rn-version }}/RnDiffApp
ruby-version: '3.3.0' # based on what is used in the sample
bundler-cache: true # runs 'bundle install' and caches installed gems automatically
cache-version: 1 # cache the installed gems

- name: Setup Plain RN ${{ matrix.rn-version }} App
run: ./dev-packages/e2e-tests/cli.mjs ${{ matrix.platform }} --create

This comment was marked as outdated.

Comment on lines 289 to 292

- name: Build Plain RN ${{ matrix.rn-version }} App
run: ./dev-packages/e2e-tests/cli.mjs ${{ matrix.platform }} --build

This comment was marked as outdated.

Comment on lines 4 to 10
prepare_react_native_project!

target 'sentry-react-native-sample-macOS' do
platform :macos, '10.15'
platform :macos, '11'
use_native_modules!

# Flags change depending on the env values.
Copy link

Choose a reason for hiding this comment

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

Bug: The Podfile specifies platform :macos, '11', but the Xcode project's MACOSX_DEPLOYMENT_TARGET is still 10.15. This inconsistency will cause build failures.
Severity: CRITICAL | Confidence: High

🔍 Detailed Analysis

The pull request updates the macOS platform target in the Podfile to 11 but fails to update the corresponding MACOSX_DEPLOYMENT_TARGET in the Xcode project file, which remains 10.15. This discrepancy creates a conflict. CocoaPods will resolve dependencies based on macOS 11, but the project is configured to build for macOS 10.15. This will lead to build failures, likely during dependency resolution or the linking phase, as frameworks compiled for macOS 11 may not be compatible with a macOS 10.15 target.

💡 Suggested Fix

Update the MACOSX_DEPLOYMENT_TARGET setting in the project.pbxproj file for both the Debug and Release configurations to 11 to match the platform version specified in the Podfile. This will ensure consistency between the project settings and CocoaPods dependencies.

🤖 Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: samples/react-native-macos/macos/Podfile#L4-L10

Potential issue: The pull request updates the macOS platform target in the `Podfile` to
`11` but fails to update the corresponding `MACOSX_DEPLOYMENT_TARGET` in the Xcode
project file, which remains `10.15`. This discrepancy creates a conflict. CocoaPods will
resolve dependencies based on macOS 11, but the project is configured to build for macOS
10.15. This will lead to build failures, likely during dependency resolution or the
linking phase, as frameworks compiled for macOS 11 may not be compatible with a macOS
10.15 target.

Did we get this right? 👍 / 👎 to inform future reviews.
Reference ID: 7525881

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bump E2E tests and sample apps to xcode 26

4 participants