Skip to content

feat(native): attach WER report#1777

Closed
jpnurmi wants to merge 30 commits into
masterfrom
jpnurmi/feat/native/wer
Closed

feat(native): attach WER report#1777
jpnurmi wants to merge 30 commits into
masterfrom
jpnurmi/feat/native/wer

Conversation

@jpnurmi

@jpnurmi jpnurmi commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Warning

WIP    🚧🔨⏳⛔

Open questions

Configuration

sentry_options_set_attach_wer_report(options, true);

Crash reporter

image

Sentry

image

See also: https://sentry-sdks.sentry.io/issues/7524261038

Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_wer.c Outdated
Comment thread src/backends/native/sentry_wer.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread tests/test_integration_native.py
Comment thread src/backends/native/sentry_wer.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_wer.c
Comment thread src/backends/native/sentry_wer.c
Comment thread tests/test_integration_native.py
Comment thread src/backends/native/sentry_wer.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5c812aa. Configure here.

Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
Comment thread src/backends/native/sentry_crash_daemon.c
Comment thread src/backends/native/sentry_crash_daemon.c Outdated
@jpnurmi
jpnurmi force-pushed the jpnurmi/feat/native/wer branch from 90a93a6 to 84a4568 Compare June 3, 2026 14:17
@linear-code

linear-code Bot commented Jun 3, 2026

Copy link
Copy Markdown

TGDX-58

Comment thread src/backends/native/sentry_crash_daemon.c
@jpnurmi jpnurmi changed the title wip(native): WER report feat(native): attach WER report Jun 4, 2026
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against bd2e8d3

Comment thread CHANGELOG.md
- Add a `transfer_timeout` option for SDK-managed HTTP transports. ([#1741](https://github.com/getsentry/sentry-native/pull/1741))
- Apple: use `os_sync_wait_on_address` for the level-triggered waitable flag in the batcher on modern macOS(14.4+) and iOS(17.4+). ([#1765](https://github.com/getsentry/sentry-native/pull/1765))
- Native/macOS: add thread names. ([#1766](https://github.com/getsentry/sentry-native/pull/1766))
- Native/Windows: add `attach_wer_report` option for attaching `Report.wer` generated by WER. ([#1777](https://github.com/getsentry/sentry-native/pull/1777))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • 🚫 The changelog entry seems to be part of an already released section ## 0.15.4.
    Consider moving the entry to the ## Unreleased section, please.

@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 22.58065% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.71%. Comparing base (0f4163a) to head (bd2e8d3).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1777      +/-   ##
==========================================
- Coverage   75.78%   75.71%   -0.08%     
==========================================
  Files          92       92              
  Lines       21989    22019      +30     
  Branches     3918     3923       +5     
==========================================
+ Hits        16664    16671       +7     
- Misses       4443     4464      +21     
- Partials      882      884       +2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpnurmi

jpnurmi commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator Author

@jpnurmi jpnurmi closed this Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant