Skip to content

Commit ba48bdb

Browse files
antonisclaude
antonis
and
claude
committed
Merge antonis/oxlint into antonis/oxfmt and apply formatting
Resolve merge conflicts, apply oxfmt formatting to new/changed files. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2 parents b9994b1 + 8d1eb87 commit ba48bdb

69 files changed

Lines changed: 1384 additions & 435 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.github/actions/platform-check/action.yml‎

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
name: 'Platform Check'
2+
description: 'Decides whether to skip a build/test based on detect-changes outputs'
3+
4+
inputs:
5+
platform:
6+
description: 'Target platform (ios, macos, android, web)'
7+
required: true
8+
sample_changed:
9+
description: 'Whether the sample app itself changed (true/false)'
10+
required: true
11+
needs_ios:
12+
description: 'detect-changes needs_ios output'
13+
required: false
14+
default: 'false'
15+
needs_android:
16+
description: 'detect-changes needs_android output'
17+
required: false
18+
default: 'false'
19+
needs_web:
20+
description: 'detect-changes needs_web output'
21+
required: false
22+
default: 'false'
23+
24+
outputs:
25+
skip:
26+
description: 'Whether to skip this platform (true/false)'
27+
value: ${{ steps.check.outputs.skip }}
28+
29+
runs:
30+
using: 'composite'
31+
steps:
32+
- name: Check if platform is needed
33+
id: check
34+
shell: bash
35+
run: |
36+
PLATFORM="${{ inputs.platform }}"
37+
SAMPLE_CHANGED="${{ inputs.sample_changed }}"
38+
39+
if [[ "$SAMPLE_CHANGED" == "true" ]]; then
40+
echo "skip=false" >> "$GITHUB_OUTPUT"
41+
echo "Sample app changed — building/testing $PLATFORM."
42+
exit 0
43+
fi
44+
45+
# macOS uses the iOS change-detection flag
46+
case "$PLATFORM" in
47+
ios|macos) NEEDS="${{ inputs.needs_ios }}" ;;
48+
android) NEEDS="${{ inputs.needs_android }}" ;;
49+
web) NEEDS="${{ inputs.needs_web }}" ;;
50+
*)
51+
echo "::warning::Unknown platform '$PLATFORM' — not skipping."
52+
echo "skip=false" >> "$GITHUB_OUTPUT"
53+
exit 0
54+
;;
55+
esac
56+
57+
if [[ "$NEEDS" != "true" ]]; then
58+
echo "skip=true" >> "$GITHUB_OUTPUT"
59+
echo "Skipping $PLATFORM — no relevant changes detected."
60+
else
61+
echo "skip=false" >> "$GITHUB_OUTPUT"
62+
fi

‎.github/workflows/sample-application-expo.yml‎

Lines changed: 9 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -59,27 +59,17 @@ jobs:
5959
- platform: 'android'
6060
ios-use-frameworks: 'dynamic-frameworks'
6161
steps:
62+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
63+
6264
- name: Check if platform is needed
6365
id: platform-check
64-
run: |
65-
SAMPLE_CHANGED="${{ needs.detect-changes.outputs.sample_expo }}"
66-
67-
if [[ "$SAMPLE_CHANGED" == "true" ]]; then
68-
echo "skip=false" >> "$GITHUB_OUTPUT"
69-
echo "Sample app changed — building ${{ matrix.platform }}."
70-
elif [[ "${{ matrix.platform }}" == "ios" && "${{ needs.detect-changes.outputs.needs_ios }}" != "true" ]]; then
71-
echo "skip=true" >> "$GITHUB_OUTPUT"
72-
echo "Skipping iOS — no relevant changes detected."
73-
elif [[ "${{ matrix.platform }}" == "android" && "${{ needs.detect-changes.outputs.needs_android }}" != "true" ]]; then
74-
echo "skip=true" >> "$GITHUB_OUTPUT"
75-
echo "Skipping Android — no relevant changes detected."
76-
elif [[ "${{ matrix.platform }}" == "web" && "${{ needs.detect-changes.outputs.needs_web }}" != "true" ]]; then
77-
echo "skip=true" >> "$GITHUB_OUTPUT"
78-
echo "Skipping Web — no relevant changes detected."
79-
fi
80-
81-
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
82-
if: ${{ steps.platform-check.outputs.skip != 'true' }}
66+
uses: ./.github/actions/platform-check
67+
with:
68+
platform: ${{ matrix.platform }}
69+
sample_changed: ${{ needs.detect-changes.outputs.sample_expo }}
70+
needs_ios: ${{ needs.detect-changes.outputs.needs_ios }}
71+
needs_android: ${{ needs.detect-changes.outputs.needs_android }}
72+
needs_web: ${{ needs.detect-changes.outputs.needs_web }}
8373

8474
- name: Enable Corepack (NPM)
8575
if: ${{ steps.platform-check.outputs.skip != 'true' && matrix.platform != 'ios' }}

‎.github/workflows/sample-application.yml‎

Lines changed: 16 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -74,32 +74,16 @@ jobs:
7474
- ios-use-frameworks: 'dynamic-frameworks'
7575
platform: 'macos'
7676
steps:
77+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
78+
7779
- name: Check if platform is needed
7880
id: platform-check
79-
run: |
80-
# Sample app changes should always build all platforms (that's the
81-
# whole point of this workflow). The needs_ios/needs_android flags
82-
# only track SDK source & native code — not sample app files.
83-
SAMPLE_CHANGED="${{ needs.detect-changes.outputs.sample_react_native }}"
84-
85-
if [[ "$SAMPLE_CHANGED" == "true" ]]; then
86-
echo "skip=false" >> "$GITHUB_OUTPUT"
87-
echo "Sample app changed — building ${{ matrix.platform }}."
88-
elif [[ "${{ matrix.platform }}" == "ios" && "${{ needs.detect-changes.outputs.needs_ios }}" != "true" ]]; then
89-
echo "skip=true" >> "$GITHUB_OUTPUT"
90-
echo "Skipping iOS — no relevant changes detected."
91-
elif [[ "${{ matrix.platform }}" == "macos" && "${{ needs.detect-changes.outputs.needs_ios }}" != "true" ]]; then
92-
echo "skip=true" >> "$GITHUB_OUTPUT"
93-
echo "Skipping macOS — no relevant changes detected."
94-
elif [[ "${{ matrix.platform }}" == "android" && "${{ needs.detect-changes.outputs.needs_android }}" != "true" ]]; then
95-
echo "skip=true" >> "$GITHUB_OUTPUT"
96-
echo "Skipping Android — no relevant changes detected."
97-
else
98-
echo "skip=false" >> "$GITHUB_OUTPUT"
99-
fi
100-
101-
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
102-
if: ${{ steps.platform-check.outputs.skip != 'true' }}
81+
uses: ./.github/actions/platform-check
82+
with:
83+
platform: ${{ matrix.platform }}
84+
sample_changed: ${{ needs.detect-changes.outputs.sample_react_native }}
85+
needs_ios: ${{ needs.detect-changes.outputs.needs_ios }}
86+
needs_android: ${{ needs.detect-changes.outputs.needs_android }}
10387

10488
- name: Enable Corepack (NPM)
10589
if: ${{ steps.platform-check.outputs.skip != 'true' && matrix.platform == 'android' }}
@@ -264,26 +248,16 @@ jobs:
264248
build-type: 'production'
265249

266250
steps:
251+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
252+
267253
- name: Check if platform is needed
268254
id: platform-check
269-
run: |
270-
SAMPLE_CHANGED="${{ needs.detect-changes.outputs.sample_react_native }}"
271-
272-
if [[ "$SAMPLE_CHANGED" == "true" ]]; then
273-
echo "skip=false" >> "$GITHUB_OUTPUT"
274-
echo "Sample app changed — testing ${{ matrix.platform }}."
275-
elif [[ "${{ matrix.platform }}" == "ios" && "${{ needs.detect-changes.outputs.needs_ios }}" != "true" ]]; then
276-
echo "skip=true" >> "$GITHUB_OUTPUT"
277-
echo "Skipping iOS — no relevant changes detected."
278-
elif [[ "${{ matrix.platform }}" == "android" && "${{ needs.detect-changes.outputs.needs_android }}" != "true" ]]; then
279-
echo "skip=true" >> "$GITHUB_OUTPUT"
280-
echo "Skipping Android — no relevant changes detected."
281-
else
282-
echo "skip=false" >> "$GITHUB_OUTPUT"
283-
fi
284-
285-
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
286-
if: ${{ steps.platform-check.outputs.skip != 'true' }}
255+
uses: ./.github/actions/platform-check
256+
with:
257+
platform: ${{ matrix.platform }}
258+
sample_changed: ${{ needs.detect-changes.outputs.sample_react_native }}
259+
needs_ios: ${{ needs.detect-changes.outputs.needs_ios }}
260+
needs_android: ${{ needs.detect-changes.outputs.needs_android }}
287261

288262
- name: Install Maestro
289263
if: ${{ steps.platform-check.outputs.skip != 'true' }}

‎CHANGELOG.md‎

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,12 @@
66
> make sure you follow our [migration guide](https://docs.sentry.io/platforms/react-native/migration/) first.
77
<!-- prettier-ignore-end -->
88
9-
## Unreleased
9+
## 8.6.0
1010

1111
### Fixes
1212

13+
- Session replay will no longer start recording when an event was ignored or dropped. ([#5885](https://github.com/getsentry/sentry-react-native/pull/5885))
14+
- Capture native exceptions consumed by Expo's bridgeless error handling on Android ([#5871](https://github.com/getsentry/sentry-react-native/pull/5871))
1315
- Fix SIGABRT crash on launch when `mobileReplayIntegration` is not configured and iOS deployment target >= 16.0 ([#5858](https://github.com/getsentry/sentry-react-native/pull/5858))
1416
- Reduce `reactNavigationIntegration` performance overhead ([#5840](https://github.com/getsentry/sentry-react-native/pull/5840), [#5842](https://github.com/getsentry/sentry-react-native/pull/5842), [#5849](https://github.com/getsentry/sentry-react-native/pull/5849))
1517
- Fix duplicated breadcrumbs on Android ([#5841](https://github.com/getsentry/sentry-react-native/pull/5841))
@@ -19,9 +21,12 @@
1921
- Bump Cocoa SDK from v9.7.0 to v9.8.0 ([#5847](https://github.com/getsentry/sentry-react-native/pull/5847))
2022
- [changelog](https://github.com/getsentry/sentry-cocoa/blob/main/CHANGELOG.md#980)
2123
- [diff](https://github.com/getsentry/sentry-cocoa/compare/9.7.0...9.8.0)
22-
- Bump JavaScript SDK from v10.44.0 to v10.45.0 ([#5848](https://github.com/getsentry/sentry-react-native/pull/5848))
23-
- [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#10450)
24-
- [diff](https://github.com/getsentry/sentry-javascript/compare/10.44.0...10.45.0)
24+
- Bump JavaScript SDK from v10.44.0 to v10.46.0 ([#5848](https://github.com/getsentry/sentry-react-native/pull/5848), [#5890](https://github.com/getsentry/sentry-react-native/pull/5890))
25+
- [changelog](https://github.com/getsentry/sentry-javascript/blob/develop/CHANGELOG.md#10460)
26+
- [diff](https://github.com/getsentry/sentry-javascript/compare/10.44.0...10.46.0)
27+
- Bump CLI from v3.3.3 to v3.3.4 ([#5891](https://github.com/getsentry/sentry-react-native/pull/5891))
28+
- [changelog](https://github.com/getsentry/sentry-cli/blob/master/CHANGELOG.md#334)
29+
- [diff](https://github.com/getsentry/sentry-cli/compare/3.3.3...3.3.4)
2530

2631
## 8.5.0
2732

‎SDK-VERSIONS.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ To manually update the table with the current version, run `./scripts/update-sdk
1212

1313
| React Native SDK | Android SDK | Cocoa SDK | JavaScript SDK |
1414
| ---------------- | ----------- | --------- | -------------- |
15+
| [8.6.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.6.0) | [8.36.0](https://github.com/getsentry/sentry-java/releases/tag/8.36.0) | [9.8.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.8.0) | [10.46.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.46.0) |
1516
| [8.5.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.5.0) | [8.36.0](https://github.com/getsentry/sentry-java/releases/tag/8.36.0) | [9.7.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.7.0) | [10.44.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.44.0) |
1617
| [8.4.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.4.0) | [8.33.0](https://github.com/getsentry/sentry-java/releases/tag/8.33.0) | [9.7.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.7.0) | [10.43.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.43.0) |
1718
| [8.3.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.3.0) | [8.33.0](https://github.com/getsentry/sentry-java/releases/tag/8.33.0) | [9.6.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.6.0) | [10.42.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.42.0) |

‎dev-packages/e2e-tests/package.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "sentry-react-native-e2e-tests",
3-
"version": "8.5.0",
3+
"version": "8.6.0",
44
"private": true,
55
"description": "Sentry React Native End to End Tests Library",
66
"main": "dist/index.js",
@@ -13,8 +13,8 @@
1313
"devDependencies": {
1414
"@babel/preset-env": "^7.25.3",
1515
"@babel/preset-typescript": "^7.18.6",
16-
"@sentry/core": "10.45.0",
17-
"@sentry/react-native": "8.5.0",
16+
"@sentry/core": "10.46.0",
17+
"@sentry/react-native": "8.6.0",
1818
"@types/node": "^20.9.3",
1919
"@types/react": "^18.2.64",
2020
"appium": "3.2.2",

‎dev-packages/type-check/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "sentry-react-native-type-check",
33
"private": true,
4-
"version": "8.5.0",
4+
"version": "8.6.0",
55
"scripts": {
66
"type-check": "./run-type-check.sh"
77
}

‎dev-packages/utils/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "sentry-react-native-samples-utils",
3-
"version": "8.5.0",
3+
"version": "8.6.0",
44
"description": "Internal Samples Utils",
55
"main": "index.js",
66
"license": "MIT",

‎lerna.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"$schema": "node_modules/lerna/schemas/lerna-schema.json",
3-
"version": "8.5.0",
3+
"version": "8.6.0",
44
"packages": [
55
"packages/*",
66
"dev-packages/*",

‎package.json‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
},
3030
"devDependencies": {
3131
"@naturalcycles/ktlint": "^1.13.0",
32-
"@sentry/cli": "3.3.3",
32+
"@sentry/cli": "3.3.4",
3333
"downlevel-dts": "^0.11.0",
3434
"google-java-format": "^1.4.0",
3535
"lerna": "^8.1.8",
@@ -67,7 +67,6 @@
6767
"front-matter@npm:4.0.2/js-yaml": "^3.14.2",
6868
"js-yaml": "^4.1.1",
6969
"detox@npm:20.46.0/ajv": "^8.18.0",
70-
"express@npm:4.19.2/path-to-regexp": "0.1.12",
7170
"@expo/fingerprint@npm:0.6.1/minimatch": "^3.1.3",
7271
"@lerna/create@npm:8.1.8/minimatch": "^3.1.3",
7372
"glob@npm:6.0.4/minimatch": "^3.1.3",
@@ -92,7 +91,6 @@
9291
"@tufjs/models@npm:2.0.1/minimatch": "^9.0.7",
9392
"@typescript-eslint/typescript-estree@npm:6.21.0/minimatch": "^9.0.7",
9493
"editorconfig@npm:1.0.4/minimatch": "^9.0.7",
95-
"glob@npm:10.4.1/minimatch": "^9.0.7",
9694
"glob@npm:10.4.5/minimatch": "^9.0.7",
9795
"ignore-walk@npm:6.0.5/minimatch": "^9.0.7",
9896
"npm-run-all2@npm:6.2.2/minimatch": "^9.0.7",
@@ -108,8 +106,7 @@
108106
"on-headers": "^1.1.0",
109107
"diff": "^5.2.2",
110108
"tar": "^7.5.11",
111-
"tmp": "^0.2.4",
112-
"@appium/support@npm:6.1.1/yauzl": "^3.2.1"
109+
"tmp": "^0.2.4"
113110
},
114111
"version": "0.0.0",
115112
"name": "sentry-react-native",

0 commit comments

Comments
 (0)