feat(sentry): include message into title (#541)

* feat(sentry): include message into title

* fix(sentry): message is included in title if exception is missing

Author: akulistus

workers/sentry/src/utils/converter.ts

@@ -54,7 +54,13 @@ function flattenObject(obj: unknown, prefix = ''): string[] {
  * @param eventPayload - Sentry event payload
  */
 export function composeTitle(eventPayload: SentryEvent): string {
-  return `${eventPayload.exception?.values?.[0]?.type || 'Unknown'}: ${eventPayload.exception?.values?.[0]?.value || ''}`;
+  const exception = eventPayload.exception?.values?.[0];
+
+  if (exception) {
+    return `${exception.type || 'Unknown'}: ${exception.value || ''}`;
+  }
+
+  return eventPayload.message || 'Unknown: ';
 }
 
 /**

workers/sentry/tests/converter.test.ts

@@ -21,6 +21,28 @@ describe('converter utils', () => {
 
       expect(composeTitle(event)).toBe('Unknown: ');
     });
+
+    it('should compose title from message if exception is missing', () => {
+      const event: SentryEvent = {
+        message: 'message'
+      };
+
+      expect(composeTitle(event)).toBe('message');
+    });
+
+    it('should compose title from exception type and value even if message is present', () => {
+      const event: SentryEvent = {
+        exception: {
+          values: [ {
+            type: 'Error',
+            value: 'Something went wrong',
+          } ],
+        },
+        message: 'message'
+      };
+
+      expect(composeTitle(event)).toBe('Error: Something went wrong');
+    });
   });
 
   describe('composeBacktrace()', () => {

workers/sentry/tests/index.test.ts

@@ -304,7 +304,7 @@ describe('SentryEventWorker', () => {
               },
             },
             catcherVersion: '1.0.1',
-            title: 'Unknown: ',
+            title: 'Test timestamp',
             type: 'error',
           },
         }));