Skip to content

Commit 23d9fd1

Browse files
committed
fix: add rule to prevent bot merges with normal profiles
Signed-off-by: Joana Maia <jmaia@contractor.linuxfoundation.org>
1 parent f05aec3 commit 23d9fd1

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

services/apps/merge_suggestions_worker/src/workflows/mergeMembersWithLLM.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,11 @@ export async function mergeMembersWithLLM(
5252
- If the values are DIFFERENT, immediately return 'false' - these are definitely different people
5353
- This rule applies REGARDLESS of how similar other fields appear.
5454
This check must be performed FIRST before evaluating any other similarities. Only do such labeling if both members have identities in the same platform. If they don't have identities in the same platform, ignore the rule.
55+
BOT CHECKS - NEVER MERGE IF ONE PROFILE IS A BOT AND THE OTHER IS NOT
56+
- Check the bot status in attributes.isBot.default for each member
57+
- If one member has attributes.isBot.default === true and the other has attributes.isBot.default === false (or undefined), return 'false'
58+
- Bots and humans are never the same entity
59+
- This check must be performed before evaluating any other similarities
5560
Print 'true' if they are the same member, 'false' otherwise. No explanation required. Don't print anything else.`
5661

5762
const suggestions = await memberActivitiesProxy.getRawMemberMergeSuggestions(

0 commit comments

Comments
 (0)