Skip to content

ref: type KSCrash installer monitors - #8792

Merged
NinjaLikesCheez merged 1 commit into
chore/remove-kscrash-imports-restoredfrom
feat/use-kscrash-api-restored
Aug 13, 2026
Merged

ref: type KSCrash installer monitors#8792
NinjaLikesCheez merged 1 commit into
chore/remove-kscrash-imports-restoredfrom
feat/use-kscrash-api-restored

Conversation

@NinjaLikesCheez

Copy link
Copy Markdown
Member

Replacement for incorrectly auto-merged #8787.

Restores the intended top layer of the KSCrash V10 stack.

#skip-changelog

@NinjaLikesCheez NinjaLikesCheez self-assigned this Aug 12, 2026
@NinjaLikesCheez NinjaLikesCheez added the run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request label Aug 12, 2026
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1211.68 ms 1252.91 ms 41.23 ms
Size 24.14 KiB 1.29 MiB 1.27 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
df67624 1225.12 ms 1259.90 ms 34.78 ms
6e35062 1232.19 ms 1262.91 ms 30.73 ms
220d7ef 1238.77 ms 1267.42 ms 28.65 ms
4b5a1d7 1235.27 ms 1264.49 ms 29.22 ms
9319185 1223.24 ms 1249.98 ms 26.73 ms
d0b52d1 1227.31 ms 1258.32 ms 31.01 ms
b6b0d1d 1236.69 ms 1270.22 ms 33.52 ms
5b1c31e 1221.35 ms 1246.14 ms 24.79 ms
9787f05 1233.67 ms 1269.00 ms 35.33 ms
29d1b7f 1220.58 ms 1254.14 ms 33.56 ms

App size

Revision Plain With Sentry Diff
df67624 24.14 KiB 1.14 MiB 1.12 MiB
6e35062 24.14 KiB 1.22 MiB 1.20 MiB
220d7ef 24.14 KiB 1.17 MiB 1.14 MiB
4b5a1d7 24.14 KiB 1.22 MiB 1.20 MiB
9319185 24.14 KiB 1.16 MiB 1.13 MiB
d0b52d1 24.14 KiB 1.15 MiB 1.13 MiB
b6b0d1d 24.14 KiB 1.18 MiB 1.16 MiB
5b1c31e 24.14 KiB 1.24 MiB 1.22 MiB
9787f05 24.14 KiB 1.18 MiB 1.15 MiB
29d1b7f 24.14 KiB 1.26 MiB 1.23 MiB

Previous results on branch: feat/use-kscrash-api-restored

Startup times

Revision Plain With Sentry Diff
62e6022 1207.29 ms 1241.81 ms 34.53 ms
5cccfb1 1245.00 ms 1269.30 ms 24.30 ms

App size

Revision Plain With Sentry Diff
62e6022 24.14 KiB 1.29 MiB 1.27 MiB
5cccfb1 24.14 KiB 1.29 MiB 1.27 MiB

@NinjaLikesCheez
NinjaLikesCheez force-pushed the feat/use-kscrash-api-restored branch from 9dfde85 to 3cf4397 Compare August 13, 2026 06:37
@sentry

sentry Bot commented Aug 13, 2026

Copy link
Copy Markdown

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
SDK-Size io.sentry.sample.SDK-Size 9.26.0 (1) Release

⚙️ sentry-cocoa Build Distribution Settings

@NinjaLikesCheez
NinjaLikesCheez force-pushed the feat/use-kscrash-api-restored branch from 3cf4397 to e5c6995 Compare August 13, 2026 08:30

@philprime philprime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@NinjaLikesCheez
NinjaLikesCheez force-pushed the feat/use-kscrash-api-restored branch from e5c6995 to cbb0451 Compare August 13, 2026 11:04
@NinjaLikesCheez
NinjaLikesCheez force-pushed the feat/use-kscrash-api-restored branch from cbb0451 to d100e61 Compare August 13, 2026 11:11
Use KSCrash's MonitorType through the installer interface instead of passing an untyped UInt mask. This removes the raw-value conversion at configuration and keeps the integration and its test mock type-safe.
@NinjaLikesCheez
NinjaLikesCheez force-pushed the feat/use-kscrash-api-restored branch from d100e61 to 53ae414 Compare August 13, 2026 11:32
@NinjaLikesCheez
NinjaLikesCheez merged commit fa2b13f into main Aug 13, 2026
242 checks passed
@NinjaLikesCheez
NinjaLikesCheez deleted the feat/use-kscrash-api-restored branch August 13, 2026 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved run-full-ci Allows gated GitHub Action workflows to run for a labelled pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants