Fixed TS errors by adding ERROR_GROUP to cases where it was missing

Author: matt-aitken

apps/webapp/app/presenters/v3/ApiAlertChannelPresenter.server.ts

@@ -17,6 +17,7 @@ export const ApiAlertType = z.enum([
   "attempt_failure",
   "deployment_failure",
   "deployment_success",
+  "error_group",
 ]);
 
 export type ApiAlertType = z.infer<typeof ApiAlertType>;
@@ -85,6 +86,8 @@ export class ApiAlertChannelPresenter {
         return "deployment_failure";
       case "DEPLOYMENT_SUCCESS":
         return "deployment_success";
+      case "ERROR_GROUP":
+        return "error_group";
       default:
         assertNever(alertType);
     }
@@ -100,6 +103,8 @@ export class ApiAlertChannelPresenter {
         return "DEPLOYMENT_FAILURE";
       case "deployment_success":
         return "DEPLOYMENT_SUCCESS";
+      case "error_group":
+        return "ERROR_GROUP";
       default:
         assertNever(alertType);
     }

apps/webapp/app/presenters/v3/ErrorGroupPresenter.server.ts

@@ -289,6 +289,20 @@ export class ErrorGroupPresenter extends BasePresenter {
       firstSeen: parseClickHouseDateTime(record.first_seen),
       lastSeen: parseClickHouseDateTime(record.last_seen),
       affectedVersions: [],
+      state: {
+        status: "UNRESOLVED" as const,
+        resolvedAt: null,
+        resolvedInVersion: null,
+        resolvedBy: null,
+        ignoredAt: null,
+        ignoredUntil: null,
+        ignoredReason: null,
+        ignoredByUserId: null,
+        ignoredByUserDisplayName: null,
+        ignoredUntilOccurrenceRate: null,
+        ignoredUntilTotalOccurrences: null,
+        ignoredAtOccurrenceCount: null,
+      },
     };
   }
 

apps/webapp/app/v3/services/alerts/deliverAlert.server.ts

@@ -319,6 +319,9 @@ export class DeliverAlertService extends BaseService {
 
         break;
       }
+      case "ERROR_GROUP": {
+        break;
+      }
       default: {
         assertNever(alert.type);
       }
@@ -657,6 +660,9 @@ export class DeliverAlertService extends BaseService {
 
         break;
       }
+      case "ERROR_GROUP": {
+        break;
+      }
       default: {
         assertNever(alert.type);
       }
@@ -913,6 +919,9 @@ export class DeliverAlertService extends BaseService {
           return;
         }
       }
+      case "ERROR_GROUP": {
+        break;
+      }
       default: {
         assertNever(alert.type);
       }

apps/webapp/app/v3/services/alerts/deliverErrorGroupAlert.server.ts

@@ -193,7 +193,7 @@ export class DeliverErrorGroupAlertService {
     await this.#postSlackMessage(integration, {
       channel: slackProperties.data.channelId,
       ...message,
-    });
+    } as ChatPostMessageArguments);
   }
 
   async #sendWebhook(
@@ -300,7 +300,7 @@ export class DeliverErrorGroupAlertService {
     payload: ErrorAlertPayload,
     errorLink: string,
     projectName: string
-  ): Pick<ChatPostMessageArguments, "text" | "blocks" | "attachments"> {
+  ): { text: string; blocks: object[]; attachments: object[] } {
     const label = this.#classificationLabel(payload.classification);
     const errorType = payload.error.errorType || "Error";
     const task = payload.error.taskIdentifier;