ref(managed): Implement Managed::zip function

[pierceroberts]

Relates to #5985.
And implements: #6088

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

[pierceroberts]

@Dav1dde Please take a look at this implementation for the zip function. Thanks!
Also, Changelog failed again... This time I am pretty sure it's not the title, maybe it's the description?

[olksdr]

@pierceroberts you will have to fix the title, you can see the error

Error: PR title does not match Sentry conventions.
Please follow the Sentry commit message conventions: https://develop.sentry.dev/engineering-practices/commit-messages/

[pierceroberts]

@pierceroberts you will have to fix the title, you can see the error

Error: PR title does not match Sentry conventions.
Please follow the Sentry commit message conventions: https://develop.sentry.dev/engineering-practices/commit-messages/

Huh weird, that links to commit-message guidelines and only mentions the pr title once... I'll have to do some more digging to figure out what is wrong with the title.
Alright fixed the title, now looks like it's a changelog issue.

[pierceroberts]

@olksdr I fixed the failing pipeline. I did add the log to the unreleased section. Let me know if I need to move that. Thanks!

[pierceroberts]

MIght be a flaky test in the integration test suit:

FAILED tests/integration/test_metrics.py::test_transaction_metrics_extraction_external_relays[False] - AssertionError: assert 3 == 4
 +  where 3 = len({'discarded_events': [], 'filtered_sampling_events': [{'category': 'span_indexed', 'quantity': 2, 'reason': 'Sampled:0'}, {'category': 'transaction_indexed', 'quantity': 1, 'reason': 'Sampled:0'}], 'timestamp': 1783399550})
============ 1 failed, 966 passed, 59 skipped in 451.44s (0:07:31) =============
make: *** [Makefile:75: test-integration] Error 1
Error: Process completed with exit code 2.

If it passes, Should I create an issue for it?

[Dav1dde]

Can remove the changelog now, added the skip-changelog label

[Dav1dde]

This can be simplified to:

    Managed::zip(managed_err, items).try_map(|(_, items), _| {
        let event_id = common::event_id_from_items(&items)?.unwrap_or_default();
        let envelope = Envelope::from_request(Some(event_id), meta).with_items(items);
        Ok::<_, BadStoreRequest>(Box::new(envelope))
    })

Which then we might as well inline and we can drop the helper.