sentry-cocoa icon indicating copy to clipboard operation
sentry-cocoa copied to clipboard

Log Integrations: Add `SentrySwiftLog` Package

Open denrase opened this issue 1 month ago • 3 comments

#skip-changelog

:scroll: Description

  • Adds the SentrySwiftLog integration to the integrations/logs/sentry-swift-log folder
  • Skipping changelog as this is not available to users through this repo

:bulb: Motivation and Context

Closes #5372 Relates to #6902

:green_heart: How did you test it?

  • Runs pure swift tests against the current Package.swift version
  • This loads pre-build artefacts, so we always test against already released versions.
  • Should we rather test against current source?

:pencil: Checklist

You have to check all boxes before merging:

  • [x] I added tests to verify the changes.
  • [x] 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.
  • [ ] Review from the native team if needed.
  • [x] No breaking change or entry added to the changelog.
  • [x] No breaking change for hybrid SDKs or communicated to hybrid SDKs.

denrase avatar Dec 01 '25 15:12 denrase

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 84.936%. Comparing base (3acb318) to head (18e2765). :warning: Report is 12 commits behind head on main. :white_check_mark: All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #6941       +/-   ##
=============================================
- Coverage   85.041%   84.936%   -0.105%     
=============================================
  Files          454       457        +3     
  Lines        27703     27604       -99     
  Branches     12150     12141        -9     
=============================================
- Hits         23559     23446      -113     
- Misses        4100      4118       +18     
+ Partials        44        40        -4     

see 50 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3acb318...18e2765. Read the comment docs.

codecov[bot] avatar Dec 03 '25 10:12 codecov[bot]

Performance metrics :rocket:

  Plain With Sentry Diff
Startup time 1210.29 ms 1246.00 ms 35.71 ms
Size 24.14 KiB 1.03 MiB 1.00 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
7969533b517f4c480b3a2c6decc90d4afb44dac2 1216.16 ms 1248.22 ms 32.05 ms
df3872ad0d4069e829177540936ef3a84363b8ea 1234.37 ms 1268.72 ms 34.35 ms
1c4c700b85dc0e7a2a79a2f452d74a283f1b4441 1193.28 ms 1218.88 ms 25.61 ms
3067c237a11ea5f4c7698408fd3925a54c6ba67c 1230.48 ms 1257.90 ms 27.42 ms
0ecea77a73dfa48ec94e47b3941a6478cf9cc31e 1191.47 ms 1216.12 ms 24.65 ms
f262e101660a2e489a414e4d231e26e8de7a0b05 1226.88 ms 1250.39 ms 23.51 ms
62799929787f2fedf64bbf922cda9e40ff7232b1 1213.60 ms 1241.38 ms 27.79 ms
73c97128227b5caa1eb72ab32bb9b7d6d2eac458 1238.57 ms 1260.38 ms 21.80 ms
c8dd5e412091848b01e8c4342a7b02ce8f90cb9f 1217.67 ms 1242.90 ms 25.23 ms
7d236393f3ac50d31f94c4e5a01995a615a6e1f7 1237.93 ms 1243.04 ms 5.11 ms

App size

Revision Plain With Sentry Diff
7969533b517f4c480b3a2c6decc90d4afb44dac2 24.14 KiB 1.02 MiB 1016.90 KiB
df3872ad0d4069e829177540936ef3a84363b8ea 23.74 KiB 1.02 MiB 1016.77 KiB
1c4c700b85dc0e7a2a79a2f452d74a283f1b4441 24.14 KiB 1.02 MiB 1017.25 KiB
3067c237a11ea5f4c7698408fd3925a54c6ba67c 23.75 KiB 928.15 KiB 904.40 KiB
0ecea77a73dfa48ec94e47b3941a6478cf9cc31e 24.15 KiB 1.01 MiB 1014.90 KiB
f262e101660a2e489a414e4d231e26e8de7a0b05 23.75 KiB 913.62 KiB 889.87 KiB
62799929787f2fedf64bbf922cda9e40ff7232b1 23.75 KiB 891.03 KiB 867.28 KiB
73c97128227b5caa1eb72ab32bb9b7d6d2eac458 23.75 KiB 908.01 KiB 884.26 KiB
c8dd5e412091848b01e8c4342a7b02ce8f90cb9f 23.75 KiB 913.48 KiB 889.72 KiB
7d236393f3ac50d31f94c4e5a01995a615a6e1f7 23.75 KiB 891.01 KiB 867.26 KiB

Previous results on branch: feat/add-swift-log-integration

Startup times

Revision Plain With Sentry Diff
ec6df71c04340a939daeaa477b5848a0df48ff9e 1210.16 ms 1240.33 ms 30.16 ms
b78f1c73666d871e5d39a428b55fba74d7131ad6 1213.30 ms 1242.79 ms 29.49 ms
5c126cfac332c273e3f28d2634162523069ee96c 1210.42 ms 1237.67 ms 27.25 ms
587a9b5e67f38e3c5a942623b034e861f5ba4197 1213.80 ms 1250.94 ms 37.14 ms
e39cfe2be6619f965eaef5991988162864a9a365 1196.87 ms 1216.73 ms 19.87 ms
b06a36606c54dde547956b4c7fcb79639bebd82a 1212.46 ms 1242.20 ms 29.74 ms
213440f4effdb15bea3aef1e2cf8f65715eb3b31 1228.80 ms 1260.73 ms 31.93 ms

App size

Revision Plain With Sentry Diff
ec6df71c04340a939daeaa477b5848a0df48ff9e 24.14 KiB 1.02 MiB 1019.40 KiB
b78f1c73666d871e5d39a428b55fba74d7131ad6 24.14 KiB 1.02 MiB 1019.59 KiB
5c126cfac332c273e3f28d2634162523069ee96c 24.14 KiB 1.02 MiB 1019.58 KiB
587a9b5e67f38e3c5a942623b034e861f5ba4197 24.14 KiB 1.02 MiB 1016.89 KiB
e39cfe2be6619f965eaef5991988162864a9a365 24.14 KiB 1.02 MiB 1016.90 KiB
b06a36606c54dde547956b4c7fcb79639bebd82a 24.14 KiB 1.02 MiB 1016.90 KiB
213440f4effdb15bea3aef1e2cf8f65715eb3b31 24.14 KiB 1.02 MiB 1019.58 KiB

github-actions[bot] avatar Dec 03 '25 11:12 github-actions[bot]

🚨 Detected changes in high risk code 🚨

High-risk code can easily blow up and is hard to test. We had severe bugs in the past. Be extra careful when changing these files, and have an extra careful look at these:

  • [ ] .github/file-filters.yml

github-actions[bot] avatar Dec 15 '25 10:12 github-actions[bot]