Skip to content

fix: remove IGNORED_MESSAGE_TAGS filter so transactional contacts reach refined table#2746

Merged
malek10xdev merged 2 commits intomainfrom
fix/transactional-tag-filtering
Mar 26, 2026
Merged

fix: remove IGNORED_MESSAGE_TAGS filter so transactional contacts reach refined table#2746
malek10xdev merged 2 commits intomainfrom
fix/transactional-tag-filtering

Conversation

@malek10xdev
Copy link
Copy Markdown
Collaborator

Summary

  • Comment out the IGNORED_MESSAGE_TAGS filter in EmailMessage.extractContacts() that was filtering out transactional-tagged contacts before they reached the refined table
  • This bug caused transactional contacts to be missing from the refined table because the filter removed them during extraction, before the refine process could properly tag them

Bug Description

Contacts from emails tagged "transactional" were being filtered out at EmailMessage.ts:336-341. When the refine process ran, it couldn't find the "transactional" tag on these contacts (because they were already filtered out), so it counted them as "good" contacts instead of transactional ones.

Changes

  • backend/src/services/extractors/engines/EmailMessage.ts: Commented out lines 336-341 that filtered contacts with IGNORED_MESSAGE_TAGS (transactional, no-reply)

@github-actions
Copy link
Copy Markdown

Coverage Report

Passed

Commit: 3aab105

Summary

Name Stmts Branch Funcs Lines
🔴 Total 38.3% 36.1% 44.4% 38.4%
All files
Name Stmts Branch Funcs Lines
🟢 backend/src/services/extractors/engines/EmailMessage.ts 93.8% 66.7% 100% 93.7%
No coverage changes
Name Stmts Branch Funcs Lines
🟢 backend/src/services/extractors/engines/EmailMessage.ts 93.8% 66.7% 100% 93.7%

Generated by Test Coverage Reporter for commit 3aab105

@malek10xdev malek10xdev merged commit 452c757 into main Mar 26, 2026
14 of 15 checks passed
@malek10xdev malek10xdev deleted the fix/transactional-tag-filtering branch March 26, 2026 19:01
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.

1 participant