feat!: Add hint support to scope-level event capture

[jpnurmi]

Accept hints in sentry_scope_capture_event and pass them to before_send. Keep sentry_capture_event unchanged.

Merge hint and scope attachments before before_send and before_send_feedback, then use the resulting hint attachments in the envelope. Add sentry_hint_get_attachments, sentry_hint_remove_attachment and sentry_hint_clear_attachments to allow filtering attachments without modifying scopes.

Caution

BREAKING CHANGE: before_send callbacks now take sentry_hint_t * instead of void * for the hint parameter. sentry_scope_capture_event now requires a hint argument; pass NULL when no hint is needed. Scope attachment changes inside either callback no longer affect the current event; modify the hint instead.

Close: #2098

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

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

^{Reviewed by Cursor Bugbot for commit 5f5a9a8. Configure here.}

[codecov]

Codecov Report

❌ Patch coverage is 92.13483% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.33%. Comparing base (5f3b00e) to head (98b9bfe).

Additional details and impacted files

@@                         Coverage Diff                         @@
##           jpnurmi/ref/attachment-manifest    #2099      +/-   ##
===================================================================
+ Coverage                            75.32%   75.33%   +0.01%     
===================================================================
  Files                                  103      103              
  Lines                                27922    27939      +17     
  Branches                              5100     5103       +3     
===================================================================
+ Hits                                 21031    21049      +18     
- Misses                                5556     5559       +3     
+ Partials                              1335     1331       -4     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[limbonaut]

Looks good!

[tustanivsky]

Just to clarify - do we support hints in before_send for crash events as well? All four backends seem to pass NULL as the hint there 🤔

[jpnurmi]

Just to clarify - do we support hints in before_send for crash events as well? All four backends seem to pass NULL as the hint there 🤔

Hmm, but sentry-unreal sets on_crash which replaces before_send for crash events. 😕

EDIT: Which means, do we then need hint for on_crash, too?

[tustanivsky]

Hmm, but sentry-unreal sets on_crash which replaces before_send for crash events

Right, for Unreal it’s on_crash that matters so a before_send fix alone wouldn’t help there. But it still makes sense for users who rely solely on before_send.

Which means, do we then need hint for on_crash, too?

Yes, I think it would be good to align what both callbacks carry so downstream SDKs get the same capabilities regardless of which hook handles the event.