Skip to content

fix: skip bot members from stint inference and lfid enrichment (CM-1260) - #4227

Merged
skwowet merged 1 commit into
mainfrom
bugfix/CM-1260
Jun 17, 2026
Merged

fix: skip bot members from stint inference and lfid enrichment (CM-1260)#4227
skwowet merged 1 commit into
mainfrom
bugfix/CM-1260

Conversation

@skwowet

@skwowet skwowet commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Summary

Bot members were incorrectly enqueued for email-domain stint inference, which created memberOrganizations and triggered affiliation refreshes they should not have. LFID enrichment also did not exclude bot members, unlike the main enrichment pipeline.

Changes

  • Use existing db state to correctly detect bot members during updates
  • Skip bot members in stint inference processing
  • Exclude bot and organization members from LFID enrichment, consistent with fetchMembersForEnrichment

Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
@skwowet skwowet self-assigned this Jun 17, 2026
Copilot AI review requested due to automatic review settings June 17, 2026 09:40
@skwowet
skwowet merged commit 4aaeaf9 into main Jun 17, 2026
13 checks passed
@skwowet
skwowet deleted the bugfix/CM-1260 branch June 17, 2026 09:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens bot/organization-member exclusion across enrichment and email-domain stint inference to prevent bot members from being queued for stint inference (and related downstream affiliation refreshes), and to align LFID enrichment with the main enrichment pipeline’s exclusion rules.

Changes:

  • Exclude bot and organization members in fetchMembersForLFIDEnrichment via members.attributes flags.
  • Fix bot detection during member updates by relying on persisted DB state (original.attributes) instead of the potentially-empty update payload.
  • Prevent stint-inference buffering for bot members when assigning organizations by email domain.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
services/libs/data-access-layer/src/old/apps/members_enrichment_worker/index.ts Adds isBot/isOrganization filters to LFID enrichment selection query.
services/apps/data_sink_worker/src/service/member.service.ts Uses DB state to detect bot members during updates and avoids queuing stint inference for bots when assigning orgs by email domain.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants