fix(enrichments): reduce cost, increase accuracy by waleedlatif1 · Pull Request #8936 · simstudioai/sim

waleedlatif1 · 2026-10-12T19:42:29Z

Summary

Work email waterfall prefers mailbox-verified emails; an unverified hit (catch-all guess, risky) is held and only used if nothing verified turns up
People Data Labs moved to the end of work email (least accurate, most expensive) and skipped once a fallback is held
Prospeo asks for verified emails/mobiles only; Wiza excludes personal emails
Stop billing ZeroBounce unknown and MillionVerifier catch-all/unknown results the vendors don't charge for
Mark an enrichment cell errored instead of completing it blank when every provider errored

Type of Change

Bug fix

Testing

enrichments/run.test.ts, background/enrichment-capability-subject.test.ts (new tests fail on the pre-fix code)
Replayed old vs new waterfall against live providers on 92 contacts: same fill rate, equal or better accuracy, ~60% cheaper on rows the top providers miss

Checklist

Code follows project style guidelines
Self-reviewed my changes
Tests added/updated and passing (new tests pass the test-audit authoring gate)
No new warnings introduced
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

vercel · 2026-10-12T19:42:32Z

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 12, 2026 8:02pm UTC

cubic-dev-ai

All reported issues were addressed across 11 files

_{Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic}

greptile-apps · 2026-10-12T19:49:44Z

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; the latest change fixes the remaining Dropcontact label mismatch.

Summary

The PR prefers verified work emails, reduces unnecessary provider charges, and marks all-provider failures as errors.

Work-email lookups prefer verified addresses and keep a fallback.
Phone lookups use stronger matches and prefer mobile numbers.
Email checks move past unclear results and count only charged checks.
A cell shows an error when every enrichment provider fails.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Try next provider] --> B{Result found?}
  B -->|No| C{Providers remain?}
  B -->|Yes| D{Verified?}
  D -->|Yes| E[Return verified result]
  D -->|No| F[Keep first fallback]
  F --> C
  C -->|Yes| A
  C -->|No| G{Fallback held?}
  G -->|Yes| H[Return fallback]
  G -->|No| I{Every attempted provider errored?}
  I -->|Yes| J[Blank outputs and mark error]
  I -->|No| K[Blank outputs and mark completed]

_{Reviews (3) · Last reviewed commit: "fix(enrichments): accept both Dropcontac..." · Reviewed by Greptile}

…ror state for outages - Work email: hold unverified hits (Hunter accept_all, Wiza risky, Dropcontact non-nominative) as a fallback while later providers look for a verified one; the held hit still fills the cell if none verifies. Ask Prospeo for verified emails only, keep personal addresses out of Wiza results, and move People Data Labs (unverified, $0.28 per hit) to the end, skipping it once a fallback is held. Order is otherwise unchanged. - Phone: skip PDL on a bare name (not a valid match), pass the LinkedIn profile to PDL, prefer its mobile, and ask Prospeo for verified mobiles only (no billed person match without one). - Email verification: unverified/error verdicts fall through; bill nothing for ZeroBounce unknown or MillionVerifier catch-all/unknown, which those vendors do not charge for. - Table cells: when every provider errored, mark the cell errored instead of completing it blank, so an outage no longer silently finalizes empty rows.

… LeadMagic verified check, toArray

waleedlatif1 · 2026-10-12T19:53:31Z

@greptile

waleedlatif1 · 2026-10-12T19:53:32Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-12T19:53:42Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai

No issues found across 11 files

Confidence score: 5/5

Automated review surfaced no issues in the provided summaries.
No files require special attention.

_{Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic}

waleedlatif1 · 2026-10-12T20:02:45Z

@greptile

waleedlatif1 · 2026-10-12T20:02:46Z

@cubic-dev-ai review this PR

cubic-dev-ai · 2026-10-12T20:02:52Z

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai

No issues found across 11 files

Confidence score: 5/5

Automated review surfaced no issues in the provided summaries.
No files require special attention.

_{Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

Turn on auto-fix | Re-trigger cubic}

waleedlatif1 changed the title ~~enrichment fix~~ fix(enrichments): reduce cost Oct 12, 2026

waleedlatif1 changed the title ~~fix(enrichments): reduce cost~~ fix(enrichments): reduce cost, increase accuracy Oct 12, 2026

cubic-dev-ai Bot reviewed Oct 12, 2026

View reviewed changes

Comment thread .agents/skills/add-enrichment/SKILL.md Outdated

Comment thread apps/sim/enrichments/work-email/work-email.ts

vercel Bot deployed to Preview October 12, 2026 19:47 View deployment

greptile-apps Bot reviewed Oct 12, 2026

View reviewed changes

Comment thread apps/sim/background/workflow-column-execution.ts Outdated

Comment thread apps/sim/enrichments/work-email/work-email.ts Outdated

waleedlatif1 added 2 commits October 12, 2026 12:52

fix(enrichments): address review — blank outputs on an all-error run,…

2ba6ad9

… LeadMagic verified check, toArray

waleedlatif1 force-pushed the fix/enrichment-waterfall-cost-quality branch from 1b808c7 to 2ba6ad9 Compare October 12, 2026 19:53

cubic-dev-ai Bot reviewed Oct 12, 2026

View reviewed changes

greptile-apps Bot reviewed Oct 12, 2026

View reviewed changes

Comment thread apps/sim/enrichments/work-email/work-email.ts Outdated

vercel Bot deployed to Preview October 12, 2026 19:59 View deployment

fix(enrichments): accept both Dropcontact verified-address labels

ce571b4

vercel Bot temporarily deployed to Preview October 12, 2026 20:02 Inactive

cubic-dev-ai Bot reviewed Oct 12, 2026

View reviewed changes

waleedlatif1 merged commit ba998b6 into staging Oct 12, 2026
38 checks passed

waleedlatif1 deleted the fix/enrichment-waterfall-cost-quality branch October 12, 2026 20:10

TheodoreSpeaks mentioned this pull request Oct 13, 2026

v0.9.20: project membership enforcement, db and perf improvements, realtime and auth hardening, CI and test fixes #8963

Merged

Conversation

waleedlatif1 commented Oct 12, 2026

Summary

Type of Change

Testing

Checklist

Uh oh!

vercel Bot commented Oct 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cubic-dev-ai Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

greptile-apps Bot commented Oct 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Confidence Score: 5/5

Summary

Diagram

Uh oh!

Uh oh!

Uh oh!

waleedlatif1 commented Oct 12, 2026

Uh oh!

waleedlatif1 commented Oct 12, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 12, 2026

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

waleedlatif1 commented Oct 12, 2026

Uh oh!

waleedlatif1 commented Oct 12, 2026

Uh oh!

cubic-dev-ai Bot commented Oct 12, 2026

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

This branch was previously deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

vercel Bot commented Oct 12, 2026 •

edited

Loading

cubic-dev-ai Bot left a comment •

edited

Loading

greptile-apps Bot commented Oct 12, 2026 •

edited

Loading