[Feat][SDK- 431] Add threads information to payload

[buongarzoni]

Description of the change

We will send information about the status of the threads in the application when sending an exception, whether it is caught or not.

• Add "threads" object to "body" in payload
• Add truncation strategy for Rollbar threads
  • For a given RollbarThread we use the same Strategy we have in a TraceChain, cutting the extra frames leaving 10 in head and 10 in tail.
• Add truncation strategy for Telemetry events
  • By default the telemetry events have a maximum capacity of 100. But when the payload is too big, we set the maximum to 10, leaving only the latest events.
• The structure of the new payload is:

{
   "trace": {
     "exception": {
       "description": "another thing to exist",
       "message": "this is an error",
       "class": "java.lang.Error"
     },
     "frames": [
       {
         "filename": "MainActivity.java",
         "lineno": 77,
         "method": "run",
         "class_name": "com.rollbar.example.android.MainActivity$2"
       }
     ]
   },
   "threads": [{
       "id": "2",
       "name": "main",
       "state": "RUNNABLE",
       "is_main": true
       "priority": "5",
       "group": [{
           "trace_chain": [{
               "frames": [{
                       "filename": "Thread.java",
                       "lineno": 1012,
                       "method": "run",
                       "class_name": "java.lang.Thread",
                   },
                   {
                       "filename": "Daemons.java",
                       "lineno": 145,
                       "method": "run",
                       "class_name": "java.lang.Daemons$Daemon",
                   },
               ],
               "exception": {
                   "message": "Uncaught exception",
                   "class": "java.lang.RuntimeException"
               }
           }]
       }]
   }]
}

Fields in a Body object

Field	Type	Mandatory	Description
trace	Trace	Yes (only 1 trace, trace_chain or message object is required)	Represent a Stack Trace
trace_chain	List < Trace >	Yes (only 1 trace, trace_chain or message object is required)	Represents a chain of errors (Traces)
message	Message	Yes (only 1 trace, trace_chain or message object is required)	Represents a message (text) sent to Rollbar, possible with additional information
--------	-------	-------	-------
telemetry	List < TelemetryEvents >	No
threads	List < Thread >	No

⚠️ whenever we send trace and trace_chain we are going to send threads, since these are going to be replaced by it. message continues to be sent as usual.

How to test

With any of the test examples we can see this new information in the payload.

Fields in a Thread object

Field	Type	Mandatory	Description
state	String	No	State of the Thread, could be: NEW, RUNNABLE, BLOCKED, WAITING, TIMED_WAITING or TERMINATED
priority	String	No	Priority level of a Thread, values would be [1, 10]
id	String	Yes	Positive long number generated when the Thread was created
name	String	No	Name of the Thread
is_main	Boolean	No	Boolean representing if this is a main thread
group	List < TraceChain >	Yes	This represent a Group of trace chains in a Thread. In Java we only send 1 TraceChain per Group, in other languages like Python (ExceptionGroup) or JS (AggregatorError), it may be more TraceChains per group.

Type of change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Maintenance
• New release

Related issues

Shortcut stories and GitHub issues (delete irrelevant)

• https://linear.app/rollbar-inc/issue/SDK-446/anr-implementation

Checklists

Development

• Lint rules pass locally
• The code changed/added as part of this pull request has been covered with tests
• All tests related to the changed code pass in development

Code review

• This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
• "Ready for review" label attached to the PR and reviewers assigned
• Issue from task tracker has a link to this pull request
• Changes have been reviewed by at least one other engineer

[linear]

SDK-431 ANR: where in the payload to put the data from the extra threads

[csaba-ilonka-rollbar]

Do you think we actually need the Thread object here, or would the information held by RollbarThread be sufficient?

[buongarzoni]

I think you're right, I could just use the RollbarThread here, I'll check it out, thanks!

[csaba-ilonka-rollbar]

What is the expected display value here?

[buongarzoni]

something like thread=Thread[main,5,main]
anyway with the other suggestion you made, I'm probably going to change this