Exclude bots from automations (#1414)

Author: gitstart-crowddev

backend/src/serverless/microservices/nodejs/automation/workers/__tests__/newActivityWorker.test.ts

@@ -65,6 +65,9 @@ describe('New Activity Automation Worker tests', () => {
             default: true,
             custom: true,
           },
+          isBot: {
+            default: false,
+          },
         },
       },
 
@@ -86,7 +89,6 @@ describe('New Activity Automation Worker tests', () => {
       id: '1234',
       type: 'comment',
       platform: PlatformType.DISCORD,
-
       body: 'Crowd.dev is awesome!',
     }
 
@@ -129,6 +131,35 @@ describe('New Activity Automation Worker tests', () => {
     expect(await shouldProcessActivity(activity, automation)).toBeFalsy()
   })
 
+  it("Shouldn't process an activity that belongs to a bot", async () => {
+    const automation = createAutomationData({
+      platforms: [PlatformType.DEVTO],
+      types: ['comment'],
+      keywords: ['Crowd.dev'],
+      teamMemberActivities: false,
+    })
+
+    const activity = {
+      id: '1234',
+      type: 'comment',
+      platform: PlatformType.DEVTO,
+      member: {
+        attributes: {
+          isTeamMember: {
+            default: false,
+            custom: true,
+          },
+          isBot: {
+            default: true,
+          },
+        },
+      },
+      body: 'Crowd.dev all awesome!',
+    }
+
+    expect(await shouldProcessActivity(activity, automation)).toBeFalsy()
+  })
+
   it("Shouldn't process an activity that belongs to a team member", async () => {
     const automation = createAutomationData({
       platforms: [PlatformType.DEVTO],
@@ -147,6 +178,9 @@ describe('New Activity Automation Worker tests', () => {
             default: true,
             custom: true,
           },
+          isBot: {
+            default: false,
+          },
         },
       },
       body: 'Crowd.dev all awesome!',

backend/src/serverless/microservices/nodejs/automation/workers/newActivityWorker.ts

@@ -79,6 +79,13 @@ export const shouldProcessActivity = async (
     process = false
   }
 
+  if (activity?.member?.attributes?.isBot && activity?.member?.attributes?.isBot.default) {
+    log.warn(
+      `Ignoring automation ${automation.id} - Activity ${activity.id} belongs to a bot, cannot be processed automaticaly!`,
+    )
+    process = false
+  }
+
   if (process) {
     const userContext = await SequelizeRepository.getDefaultIRepositoryOptions()
     const repo = new AutomationExecutionRepository(userContext)