7154 deleted event is not emitted when calling destroyone (#7159)

Closes #7154

packages/twenty-server/src/engine/api/graphql/graphql-query-runner/graphql-query-runner.service.ts

@@ -35,6 +35,7 @@ import {
 } from 'src/engine/api/graphql/workspace-query-runner/workspace-query-runner.exception';
 import { AuthContext } from 'src/engine/core-modules/auth/types/auth-context.type';
 import { ObjectRecordCreateEvent } from 'src/engine/core-modules/event-emitter/types/object-record-create.event';
+import { ObjectRecordDeleteEvent } from 'src/engine/core-modules/event-emitter/types/object-record-delete.event';
 import { InjectMessageQueue } from 'src/engine/core-modules/message-queue/decorators/message-queue.decorator';
 import { MessageQueue } from 'src/engine/core-modules/message-queue/message-queue.constants';
 import { MessageQueueService } from 'src/engine/core-modules/message-queue/services/message-queue.service';
@@ -284,10 +285,94 @@ export class GraphqlQueryRunnerService {
   async destroyOne<ObjectRecord extends IRecord = IRecord>(
     args: DestroyOneResolverArgs,
     options: WorkspaceQueryRunnerOptions,
-  ): Promise<ObjectRecord> {
+  ): Promise<ObjectRecord | undefined> {
     const graphqlQueryDestroyOneResolverService =
       new GraphqlQueryDestroyOneResolverService(this.twentyORMGlobalManager);
 
-    return graphqlQueryDestroyOneResolverService.destroyOne(args, options);
+    const { authContext, objectMetadataItem } = options;
+
+    assertMutationNotOnRemoteObject(objectMetadataItem);
+    assertIsValidUuid(args.id);
+
+    const hookedArgs =
+      await this.workspaceQueryHookService.executePreQueryHooks(
+        authContext,
+        objectMetadataItem.nameSingular,
+        'destroyOne',
+        args,
+      );
+
+    const computedArgs = (await this.queryRunnerArgsFactory.create(
+      hookedArgs,
+      options,
+      ResolverArgsType.DestroyOne,
+    )) as DestroyOneResolverArgs;
+
+    const result = (await graphqlQueryDestroyOneResolverService.destroyOne(
+      computedArgs,
+      options,
+    )) as ObjectRecord;
+
+    await this.workspaceQueryHookService.executePostQueryHooks(
+      authContext,
+      objectMetadataItem.nameSingular,
+      'destroyOne',
+      [result],
+    );
+
+    await this.triggerWebhooks<IRecord>(
+      [result],
+      CallWebhookJobsJobOperation.destroy,
+      options,
+    );
+
+    this.emitDestroyEvents<IRecord>([result], authContext, objectMetadataItem);
+
+    return result;
+  }
+
+  private emitDestroyEvents<BaseRecord extends IRecord = IRecord>(
+    records: BaseRecord[],
+    authContext: AuthContext,
+    objectMetadataItem: ObjectMetadataInterface,
+  ) {
+    this.workspaceEventEmitter.emit(
+      `${objectMetadataItem.nameSingular}.destroyed`,
+      records.map((record) => {
+        return {
+          userId: authContext.user?.id,
+          recordId: record.id,
+          objectMetadata: objectMetadataItem,
+          properties: {
+            before: this.removeNestedProperties(record),
+          },
+        } satisfies ObjectRecordDeleteEvent<any>;
+      }),
+      authContext.workspace.id,
+    );
+  }
+
+  private removeNestedProperties<Record extends IRecord = IRecord>(
+    record: Record,
+  ) {
+    if (!record) {
+      return;
+    }
+
+    const sanitizedRecord = {};
+
+    for (const [key, value] of Object.entries(record)) {
+      if (value && typeof value === 'object' && value['edges']) {
+        continue;
+      }
+
+      if (key === '__typename') {
+        continue;
+      }
+
+      sanitizedRecord[key] = value;
+    }
+
+    return sanitizedRecord;
   }
 }

packages/twenty-server/src/engine/api/graphql/workspace-query-runner/jobs/call-webhook-jobs.job.ts

@@ -20,6 +20,7 @@ export enum CallWebhookJobsJobOperation {
   create = 'create',
   update = 'update',
   delete = 'delete',
+  destroy = 'destroy',
 }
 
 export type CallWebhookJobsJobData = {

packages/twenty-server/src/engine/api/graphql/workspace-query-runner/listeners/entity-events-to-db.listener.ts

@@ -49,6 +49,13 @@ export class EntityEventsToDbListener {
     return this.handle(payload);
   }
 
+  @OnEvent('*.destroyed')
+  async handleDestroy(
+    payload: WorkspaceEventBatch<ObjectRecordUpdateEvent<any>>,
+  ) {
+    return this.handle(payload);
+  }
+
   private async handle(payload: WorkspaceEventBatch<ObjectRecordBaseEvent>) {
     const filteredEvents = payload.events.filter(
       (event) => event.objectMetadata?.isAuditLogged,

packages/twenty-server/src/engine/api/graphql/workspace-query-runner/workspace-query-hook/types/workspace-query-hook.type.ts

@@ -4,6 +4,7 @@ import {
   DeleteManyResolverArgs,
   DeleteOneResolverArgs,
   DestroyManyResolverArgs,
+  DestroyOneResolverArgs,
   FindDuplicatesResolverArgs,
   FindManyResolverArgs,
   FindOneResolverArgs,
@@ -39,4 +40,6 @@ export type WorkspacePreQueryHookPayload<T> = T extends 'createMany'
                     ? RestoreManyResolverArgs
                     : T extends 'destroyMany'
                       ? DestroyManyResolverArgs
-                      : never;
+                      : T extends 'destroyOne'
+                        ? DestroyOneResolverArgs
+                        : never;

packages/twenty-server/src/engine/api/graphql/workspace-resolver-builder/interfaces/workspace-resolvers-builder.interface.ts

@@ -22,6 +22,7 @@ export enum ResolverArgsType {
   DeleteMany = 'DeleteMany',
   RestoreMany = 'RestoreMany',
   DestroyMany = 'DestroyMany',
+  DestroyOne = 'DestroyOne',
 }
 
 export interface FindManyResolverArgs<

packages/twenty-server/src/engine/core-modules/event-emitter/types/object-record-destroy.event.ts

@@ -0,0 +1,7 @@
+import { ObjectRecordBaseEvent } from 'src/engine/core-modules/event-emitter/types/object-record.base.event';
+
+export class ObjectRecordDestroyEvent<T> extends ObjectRecordBaseEvent {
+  properties: {
+    before: T;
+  };
+}

packages/twenty-server/src/modules/calendar/calendar-event-cleaner/listeners/calendar-event-cleaner-connected-account.listener.ts

@@ -19,8 +19,8 @@ export class CalendarEventCleanerConnectedAccountListener {
     private readonly calendarQueueService: MessageQueueService,
   ) {}
 
-  @OnEvent('connectedAccount.deleted')
+  @OnEvent('connectedAccount.destroyed')
-  async handleDeletedEvent(
+  async handleDestroyedEvent(
     payload: WorkspaceEventBatch<
       ObjectRecordDeleteEvent<ConnectedAccountWorkspaceEntity>
     >,

packages/twenty-server/src/modules/connected-account/listeners/connected-account.listener.ts

@@ -15,8 +15,8 @@ export class ConnectedAccountListener {
     private readonly accountsToReconnectService: AccountsToReconnectService,
   ) {}
 
-  @OnEvent('connectedAccount.deleted')
+  @OnEvent('connectedAccount.destroyed')
-  async handleDeletedEvent(
+  async handleDestroyedEvent(
     payload: WorkspaceEventBatch<
       ObjectRecordDeleteEvent<ConnectedAccountWorkspaceEntity>
     >,

packages/twenty-server/src/modules/connected-account/query-hooks/connected-account-delete-one.pre-query.hook.ts

@@ -8,7 +8,7 @@ import { TwentyORMManager } from 'src/engine/twenty-orm/twenty-orm.manager';
 import { WorkspaceEventEmitter } from 'src/engine/workspace-event-emitter/workspace-event-emitter';
 import { MessageChannelWorkspaceEntity } from 'src/modules/messaging/common/standard-objects/message-channel.workspace-entity';
 
-@WorkspaceQueryHook(`connectedAccount.deleteOne`)
+@WorkspaceQueryHook(`connectedAccount.destroyOne`)
 export class ConnectedAccountDeleteOnePreQueryHook
   implements WorkspaceQueryHookInstance
 {
@@ -34,7 +34,7 @@ export class ConnectedAccountDeleteOnePreQueryHook
     });
 
     this.workspaceEventEmitter.emit(
-      'messageChannel.deleted',
+      'messageChannel.destroyed',
       messageChannels.map(
         (messageChannel) =>
           ({

packages/twenty-server/src/modules/messaging/message-cleaner/listeners/messaging-message-cleaner-connected-account.listener.ts

@@ -19,8 +19,8 @@ export class MessagingMessageCleanerConnectedAccountListener {
     private readonly messageQueueService: MessageQueueService,
   ) {}
 
-  @OnEvent('connectedAccount.deleted')
+  @OnEvent('connectedAccount.destroyed')
-  async handleDeletedEvent(
+  async handleDestroyedEvent(
     payload: WorkspaceEventBatch<
       ObjectRecordDeleteEvent<ConnectedAccountWorkspaceEntity>
     >,

packages/twenty-server/src/modules/messaging/message-import-manager/listeners/messaging-import-manager-message-channel.listener.ts

@@ -19,8 +19,8 @@ export class MessagingMessageImportManagerMessageChannelListener {
     private readonly messageQueueService: MessageQueueService,
   ) {}
 
-  @OnEvent('messageChannel.deleted')
+  @OnEvent('messageChannel.destroyed')
-  async handleDeletedEvent(
+  async handleDestroyedEvent(
     payload: WorkspaceEventBatch<
       ObjectRecordDeleteEvent<MessageChannelWorkspaceEntity>
     >,

packages/twenty-server/src/modules/workflow/workflow-trigger/database-event-trigger/listeners/database-event-trigger.listener.ts

@@ -1,11 +1,12 @@
 import { Injectable, Logger } from '@nestjs/common';
 import { OnEvent } from '@nestjs/event-emitter';
 
-import { FeatureFlagKey } from 'src/engine/core-modules/feature-flag/enums/feature-flag-key.enum';
-import { FeatureFlagService } from 'src/engine/core-modules/feature-flag/services/feature-flag.service';
 import { ObjectRecordCreateEvent } from 'src/engine/core-modules/event-emitter/types/object-record-create.event';
 import { ObjectRecordDeleteEvent } from 'src/engine/core-modules/event-emitter/types/object-record-delete.event';
+import { ObjectRecordDestroyEvent } from 'src/engine/core-modules/event-emitter/types/object-record-destroy.event';
 import { ObjectRecordUpdateEvent } from 'src/engine/core-modules/event-emitter/types/object-record-update.event';
+import { FeatureFlagKey } from 'src/engine/core-modules/feature-flag/enums/feature-flag-key.enum';
+import { FeatureFlagService } from 'src/engine/core-modules/feature-flag/services/feature-flag.service';
 import { InjectMessageQueue } from 'src/engine/core-modules/message-queue/decorators/message-queue.decorator';
 import { MessageQueue } from 'src/engine/core-modules/message-queue/message-queue.constants';
 import { MessageQueueService } from 'src/engine/core-modules/message-queue/services/message-queue.service';
@@ -49,11 +50,19 @@ export class DatabaseEventTriggerListener {
     await this.handleEvent(payload);
   }
 
+  @OnEvent('*.destroyed')
+  async handleObjectRecordDestroyEvent(
+    payload: WorkspaceEventBatch<ObjectRecordDestroyEvent<any>>,
+  ) {
+    await this.handleEvent(payload);
+  }
+
   private async handleEvent(
     payload: WorkspaceEventBatch<
       | ObjectRecordCreateEvent<any>
       | ObjectRecordUpdateEvent<any>
       | ObjectRecordDeleteEvent<any>
+      | ObjectRecordDestroyEvent<any>
     >,
   ) {
     const workspaceId = payload.workspaceId;