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

JavascriptException: com.facebook.react.modules.core.ExceptionsManagerModule in reportException #6416

Closed
fluiddot opened this issue Nov 28, 2023 · 3 comments

Comments

@fluiddot
Copy link
Contributor

fluiddot commented Nov 28, 2023

Intro

On Android, all exceptions/crashes originated in the editor are currently being grouped under a single Sentry event. This GitHub issue aims to track the ones that are occurring to users in the latest versions.

ℹ️ When a new version is released, generate a new list and post the previous one as comment in the issue.

How to generate the list ⤵️
  1. Navigate to Discover page of the Sentry event.
  2. Set 90 days range.
  3. Click on "Export All" and follow the instructions to download the CSV file.
  4. Open the CSV file.
  5. Remove all occurrences of string , js engine: hermes, stack:.
  6. Remove all occurrences of string TaskQueue: Error with task : .
  7. Open the spreadsheet and navigate to the Data tab.
  8. Import the CSV by using the following parameters: Import location: Replace current sheet
  9. Remove columns environment, user.display and timestamp.
  10. Select release column and set plain text format (Format => Number => Plain text)
  11. Duplicate the first sheet.
  12. Change the version in cell A1.
  13. Select cells from C3 to I3.
  14. Copy the formula to the rest of the cells by dragging the range.
  15. Edit the description of this GitHub issue.
  16. Create a new entry with the following markdown code.
## <VERSION> (Jetpack app)
_Data collected on <DATE>_

| Title | Events | Issue | Assignee | Status |
|--------|--------|--------|--------|--------|
  1. For each item in the list, create the list item with the following format:
| <TITLE> | <Number of events/Sample Count> | <Link to GitHub issue> (create one if it doesn't exist) | <Assignee> or Pending if no DRI | <Status> |
  1. If there's no GitHub associated, create one with the following template:
**Sentry Issue:** https://a8c.sentry.io/share/issue/9a338d040674404dad0055ce6249c2de/

**⚠️ NOTE:** On Android, all Javascript exceptions are being grouped in a single Sentry issue. Hence, we can't reference the specific crash here. Navigate to "All Events" in order to find events related to this crash.

**Symbolicated stack trace:**
```
<SYMBOLICATED_STACKTRACE>
```
  1. For the past version, move its content to the disclosure tag details:
<details><summary>PREVIOUS_VERSION</summary> 

<PREVIOUS_VERSION_CONTENT>

</details>

23.8 (Jetpack app)

Data collected on Jan 15, 2024

Title Events Issue Assignee Status
TypeError: false is not a function 81 #6359 - ℹ️ issue already ungrouped @dcalhoun 🔍 Investigating
TypeError: Cannot read property 'width' of undefined 43 #6542 @derekblank 🔍 Investigating
TypeError: Cannot read property 'replace' of undefined 23 #6432 (source 2) @fluiddot ✋ Pending
RangeError: Maximum call stack size exceeded (native stack depth) 22 #6433 - ℹ️ issue already ungrouped @fluiddot ✋ Pending
TypeError: Cannot read property 'name' of null 16 #6128 - ℹ️ issue already ungrouped @dcalhoun ✅ Will be solved in 24.0
TypeError: Cannot read property 'scrollToEnd' of null 9 #6593 - ℹ️ issue already ungrouped Pending ✋ Pending
TypeError: Cannot set property 'block' of undefined 9 N/A Pending ✋ Pending
TypeError: Cannot read property 'length' of undefined 7 #6600 - ℹ️ issue already ungrouped Pending ✋ Pending
Error: 49.9%, 50%): Expected color definition 3 N/A Pending ✅ Solved in 23.9
Error: Block type 'core/other' is not registered. 3 #6443 @geriux ✅ Will be solved in 24.0
SyntaxError: 1:175:')' expected at end of 'if' condition 2 N/A Pending ✋ Pending
TypeError: Cannot read property 'getEditWrapperProps' of undefined 2 #6410 @derekblank ✅ Will be solved in 24.0
Error: Text strings must be rendered within a <Text> component. 1 N/A Pending ✋ Pending
TypeError: undefined is not a function 1 https://github.com/wordpress-mobile/gutenberg-mobile/issues/65438 Pending ✋ Pending

Past versions

23.6 (Jetpack app)

23.6 (Jetpack app)

Total events: 200 events (Data collected on Nov 28, 2023)

Title Events Issue Assignee Status
RangeError: Maximum call stack size exceeded 40 wordpress-mobile/WordPress-Android#18750 @fluiddot ✅ Solved in 23.7
TypeError: undefined is not a function 40 #6358 @fluiddot ✅ Solved in 23.8
Cannot read property 'attributes' of undefined 33 #6084 @derekblank @fluiddot ✅ Solved in 23.8
TypeError: false is not a function 22 #6359 @dcalhoun 🔍 Investigating
TypeError: Cannot read property 'props' of undefined 13 #6189 @fluiddot ✅ Solved in 23.8
Block type 'core/other' is not registered 4 #6443 @geriux ✅ Will be solved in 24.0
Cannot read property 'getEditWrapperProps' of undefined 3 #6410 @derekblank ✅ Will be solved in 24.0
TypeError: Cannot set property 'block' of undefined 2 N/A Pending ✋ Pending
TypeError: Cannot read property 'scrollToEnd' of null 2 N/A Pending ✋ Pending
TypeError: Cannot read property 'replace' of undefined 2 #6432 @fluiddot ✅ Solved in 23.8
Cannot read property 'name' of null 2 #6128 @dcalhoun ✅ Will be solved in 24.0
Cannot read property 'length' of undefined 2 N/A Pending ✋ Pending
Text strings must be rendered within a <Text> component 2 N/A Pending ✋ Pending
Error: 49.9%, 50%): Expected color definition 2 N/A Pending ✋ Pending
TypeError: Cannot set property 'index' of undefined 1 N/A Pending ✋ Pending
Exception in HostFunction: java.lang.OutOfMemoryError 1 N/A Pending ✋ Pending
@fluiddot
Copy link
Contributor Author

ℹ️ Heads up that I've updated the issue with data from version 23.8. Additionally, I updated the instructions for generating the list.

@fluiddot
Copy link
Contributor Author

Since yesterday (Jan 30, 2024), the JavaScript exceptions are no longer being grouped in a single Sentry event. Hence, we won't need to maintain this GitHub issue. Nevertheless, I'd like to propose keeping it until at least the top 5 issues are reported in Sentry with their associated event.

Internal ref: pdnsEh-1uR-p2

@fluiddot
Copy link
Contributor Author

fluiddot commented Feb 2, 2024

Following #6416 (comment) I'm closing this GitHub issue as we already received multiple Sentry events, so we no longer need to maintain this issue. We'll track the progress of each crash in its individual GitHub issue.

@fluiddot fluiddot closed this as completed Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant