feat!: add support for crash-time attachments via hint

[jpnurmi]

Merge attachments and pass as hint to on_crash, then use the resulting hint attachments in the envelope.

Caution

BREAKING CHANGE: on_crash callbacks now have an additional sentry_hint_t *hint parameter.

Close: #2098

[github-actions]

	Fails
🚫	Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- add support for crash-time attachments via hint ([#2112](https://github.com/getsentry/sentry-native/pull/2112))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against d30860a

[cursor]

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

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

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

[cursor]

Crashpad ignores crash-time hint attachments

High Severity

Crashpad writes a __sentry-attachments manifest after on_crash/before_send but never registers or unregisters those files with the crashpad client. Crashpad only uploads attachments added via AddAttachment or the startup list, so hint changes never reach the minidump upload: new attachments are dropped and cleared ones still ship.

Additional Locations (1)

• src/backends/sentry_backend_crashpad.cpp#L290-L321

 

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

[cursor]

Scope flush overwrites crash hint attachments

Medium Severity

native_backend_flush_scope still writes the scope attachment list after a crash, even though native_backend_except already wrote the hint list. A later flush replaces that crash-time manifest with scope attachments, dropping hint-only files and restoring ones the callback cleared.

Additional Locations (2)

• src/backends/sentry_backend_native.c#L1181-L1186
• src/backends/sentry_backend_native.c#L1386-L1394

 

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

[cursor]

Recursive crash invokes before_send again

Medium Severity

The inproc handler used to skip before_send when skip_hooks was set for a recursive crash. The split now calls sentry__before_send whenever on_crash is unset, so a crash inside before_send can re-enter that callback.

 

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