fix(eap): Remove integer attribute values larger than I64::MAX

[elramen]

Remove integer attribute values larger than I64::MAX as EAP only supports signed 64 bit ints. Add an error along with the removed value to the attribute's metadata.

Fixes #5612

[Dav1dde]

As discussed, next steps to add an integration test and verify code behaves as desired.

[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.}

[Copilot]

Pull request overview

This PR enforces EAP’s signed 64-bit integer limitation by removing integer attribute values that exceed i64::MAX, while preserving the removed original value and an invalid_data error in _meta (per #5612).

Changes:

• Drop integer attributes whose JSON numbers deserialize as u64 and exceed i64::MAX, and record the original value in metadata.
• Extend Rust unit tests and Python integration tests to cover the overflow case and the boundary value (i64::MAX).
• Add a changelog entry documenting the behavior change.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
relay-event-normalization/src/eap/mod.rs	Removes overflowing u64 integer attributes and records invalid_data + original value in _meta; adds unit test coverage.
tests/integration/test_spansv2.py	Adds integration coverage for i64::MAX (kept) and i64::MAX + 1 (removed with _meta).
CHANGELOG.md	Documents the bugfix (but currently has a broken Markdown link).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Dav1dde]

I think I would write this as i64::try_from(*u).is_ok()

[Dav1dde]

Since the code in this branch is equivalent to the default error branch, you can also write this as:

Suggested change

	(Annotated(Some(Integer), _), Annotated(Some(Value::U64(u)), _)) => {
	(Annotated(Some(Integer), _), Annotated(Some(Value::U64(u)), _))
	if i64::try_from(*u).is_ok() => {}

[Dav1dde]

Suggested change

	- Remove EAP integer attributes larger than I64::MAX. ([#5621](https://github.com/getsentry/relay/pull/5621))
	- Remove EAP integer attributes larger than `i64::MAX`. ([#5621](https://github.com/getsentry/relay/pull/5621))

Or maybe:

Suggested change

	- Remove EAP integer attributes larger than I64::MAX. ([#5621](https://github.com/getsentry/relay/pull/5621))
	- Validate log/span/metric attribute integers to fit within an `i64`. ([#5621](https://github.com/getsentry/relay/pull/5621))

[Dav1dde]

Oops, meant to approve before, it's really just nits.