fix: event/sentry flush, event status code

Signed-off-by: Sam Gammon <[email protected]>

Author: sgammon

src/diagnostics.ts

@@ -143,7 +143,7 @@ async function transmitEvent(event: ActionEvent<any>): Promise<void> {
       encoded,
       headers
     )
-    if (res.message.statusCode !== 200 && res.message.statusCode !== 204) {
+    if (res.message.statusCode !== 200 && res.message.statusCode !== 202) {
       core.debug(`Event transmission failed: ${res.message.statusMessage}`)
     } else {
       core.debug(`Event transmission completed (ID: ${event.uuid})`)
@@ -209,6 +209,14 @@ export async function error(err: Error | unknown, fatal = true): Promise<void> {
     fatal
   })
   Sentry.captureException(err)
-  const flush = Sentry.flush()
-  await Promise.all([req, flush])
+  await Promise.all([req])
+}
+
+/**
+ * Exit hook for making sure Sentry finishes reporting.
+ */
+export async function onExit(): Promise<void> {
+  Sentry.close(2000).then(function () {
+    process.exit()
+  })
 }

src/main.ts

@@ -17,6 +17,7 @@ import {
 } from './agent'
 
 import {
+  onExit,
   ActionEventType,
   error as sendError,
   event as sendEvent
@@ -589,6 +590,8 @@ export async function entry(options?: Partial<Options>): Promise<void> {
     core.warning(
       'Buildless failed to install; this build may not be accelerated. Please see CI logs for more information.'
     )
+  } finally {
+    await onExit()
   }
 }
 
@@ -607,5 +610,7 @@ export async function cleanup(options?: Partial<Options>): Promise<void> {
     core.notice(
       'Cleanup stage for the Buildless action failed. Please see CI logs for more information.'
     )
+  } finally {
+    await onExit()
   }
 }