Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Synthesise unmatched ANR reports #2113

Closed
wants to merge 11 commits into from
Closed

Synthesise unmatched ANR reports #2113

wants to merge 11 commits into from

Conversation

YYChen01988
Copy link
Contributor

Goal

Synthesise unmatched ANR reports

Changeset

Adding callback for empty events and synthesise the unmatched ANR reports

Testing

Introducing new tests and manual test

YYChen01988 and others added 10 commits September 11, 2024 09:36
* Buildkite analytics trial

* Integrate all jobs with Test Analytics
[full ci]

* Only capture test analytic data on next and master

* Correct escape character
[full ci]

* Skip empty buildkite jobs

* fix(startup): flexible timeout for EventStore.flushOnLaunch toa void startup ANRs in apps with slow startups

* fix(startup): log clear message when launch crashes will be sent asynchronously

* fix(launch): change the launch timeout calculation for synchronous delivery to handle cases where the app start has completed before Bugsnag.start is called

* fix(breadcrumb): work around NPEs from `Intent.getExtras` in ActivityBreadcrumbCollector

---------

Co-authored-by: Steve Kirkland-Walton <steve.kirkland-walton@SB-UK-NHLF0CT.fritz.box>
Co-authored-by: Steve Kirkland <twometresteve@users.noreply.github.com>
Co-authored-by: jason <lemnik@users.noreply.github.com>
* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* refactor(EventStore): reworked the EventStore callback to only be called once, and only when the store is completely empty (no files in queue or in storage)

* feat(ExitInfo): Track exitinfokey with event

* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

---------

Co-authored-by: jason <lemnik@users.noreply.github.com>
* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* feat(ExitInfo): Track exitinfokey with event

* feat(ExitInfo):new event with Anr exitInfo

* refactor(EventStore): reworked the EventStore callback to only be called once, and only when the store is completely empty (no files in queue or in storage)

* feat(ExitInfo):new event with Anr exitInfo

* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* refactor(EventStore): reworked the EventStore callback to only be called once, and only when the store is completely empty (no files in queue or in storage)

* refactor(EventStore): reworked the EventStore callback to only be called once, and only when the store is completely empty (no files in queue or in storage)

* feat(ExitInfo): ExitInfo setOnEventStoreEmptyCallback

* feat(ExitInfo):new event with Anr exitInfo

* feat(ExitInfo):register EventStoreEmptyCallback in exitinfo plugin

* feat(ExitInfo)synthesize native crash reports

* feat(ExitInfo)add exit infos at first run

---------

Co-authored-by: jason <lemnik@users.noreply.github.com>
…tive-crash-reports

# Conflicts:
#	.buildkite/pipeline.full.yml
#	.buildkite/pipeline.yml
#	bugsnag-android-core/detekt-baseline.xml
#	bugsnag-android-core/src/main/java/com/bugsnag/android/FileStore.kt
* feat(ExitInfo)removed disable config options for synthesising reports

* feat(OpenSSF) improve Pinned-Dependencies and Token-Permissions scores (#2103)

* feat(ExitInfo)added DiscardEventCallback

* test(mazerunner): include the mapping.txt file in the published end2end artifacts to aid test debugging

* Add signing workflow (#2109)

* feat(exitinfo): remove the unmatched native crash reporting and turn off the unmatched ANR handling by default to avoid unexpected behaviour in upgrades

* chore(exitinfo): cleanup consts making more use of MATCH_ALL and MAX_EXIT_INFO

* test(fixture): only copy mapping.txt of the test fixture if it exists

---------

Co-authored-by: jason <lemnik@users.noreply.github.com>
Co-authored-by: Alex Moinet <alex@bugsnag.com>
@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Dec 4, 2024

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1872.42 1675.92
arm64_v8a 651.52 450.82
armeabi_v7a 585.99 385.29
x86 725.23 528.62
x86_64 696.57 495.86

Generated by 🚫 Danger

@YYChen01988 YYChen01988 closed this Dec 4, 2024
@YYChen01988 YYChen01988 deleted the Synthesise branch December 4, 2024 10:09
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.

3 participants