fix(enrichment): wait for valid work email inputs by TheodoreSpeaks · Pull Request #6531 · simstudioai/sim

TheodoreSpeaks · 2026-08-11T04:14:35Z

Summary

skip work-email providers when row inputs cannot form a valid request
route PDL through LinkedIn profile or name + company and show Waiting for incomplete rows
re-arm skipped enrichment cells when mapped inputs change, without pre-emptively marking untouched manual cells as Waiting

Type of Change

Bug fix

Testing

focused Vitest suites: 54 tests passed
TypeScript check passed
bun run lint
bun run check:api-validation:strict

Checklist

Code follows project style guidelines
Self-reviewed my changes
Tests added/updated and passing
No new warnings introduced
I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CONTRIBUTING.md)

…nd enrichment cells

vercel · 2026-08-11T04:14:41Z

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 11, 2026 10:47pm

cursor · 2026-08-11T04:14:44Z

PR Summary

Medium Risk
Touches enrichment eligibility, cascade scheduling, cell rendering, and row-update invalidation across client and server. Incorrect readiness or provenance handling could skip runs or clear valid outputs.

Overview
Enrichment cells now wait until at least one provider can build a valid request, instead of running (or showing stale results) on incomplete inputs.

Adds shared readiness checks used by scheduling, workers, and the grid. Work-email’s PDL provider now requires LinkedIn or name+company. Incomplete rows show Waiting; in-flight reruns still keep the previous value until replacement.

Also tracks isManualRun through execution metadata/SSE so editing mapped inputs clears stale manual enrichment outputs (and re-arms dependents) without marking untouched manual cells as Waiting.

^{Reviewed by Cursor Bugbot for commit efbcebf. Bugbot is set up for automated code reviews on this repo. Configure here.}

greptile-apps · 2026-08-11T04:17:04Z

Greptile Summary

The PR prevents work-email enrichments from running until a provider can form a valid request and preserves manual-run provenance so readiness states remain accurate across dispatch, resume, persistence, and SSE updates.

Adds shared provider-level enrichment readiness checks and actionable Waiting labels.
Re-arms enrichment cells when mapped inputs change while leaving untouched manual cells idle.
Propagates manual-run provenance through queued, running, resumed, and terminal execution states.
Extends stale-output and execution invalidation to matching upserts and filtered bulk updates.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
apps/sim/enrichments/readiness.ts	Centralizes required-input and provider-request readiness, including actionable fallback labels when normalized values are rejected.
apps/sim/enrichments/work-email/work-email.ts	Tightens PDL request construction to LinkedIn or name-plus-company inputs and aligns it with readiness evaluation.
apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/data-row.tsx	Computes per-row readiness labels while requiring authoritative manual-run provenance for autoRun:false cells.
apps/sim/background/workflow-column-execution.ts	Gates enrichment execution on provider readiness and carries manual-run provenance through cascade state writes.
apps/sim/background/resume-execution.ts	Preserves the originating cell’s manual-run provenance across paused workflow resumes and subsequent cascade continuation.
apps/sim/lib/table/rows/service.ts	Applies stale enrichment invalidation and downstream cancellation consistently to upserts and filtered updates.
apps/sim/lib/table/rows/executions.ts	Persists and projects manual-run provenance in execution JSON metadata without loading full enrichment details into grid reads.
apps/sim/hooks/queries/tables.ts	Mirrors server readiness and invalidation behavior in optimistic row and manual-run cache updates.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Mapped row inputs change] --> B[Evaluate provider readiness]
  B -->|No valid request| C[Clear stale enrichment state]
  C --> D[Show Waiting with actionable inputs]
  B -->|Valid request| E[Dispatch enrichment]
  E --> F[Persist manual-run provenance]
  F --> G[Queued / Running / Resumed]
  G --> H[Terminal result and SSE update]

_{Reviews (18): Last reviewed commit: "fix(enrichment): cancel upsert dependent..." | Re-trigger Greptile}

# Conflicts: # apps/sim/background/workflow-column-execution.ts

TheodoreSpeaks · 2026-08-11T05:15:28Z

@greptile

TheodoreSpeaks · 2026-08-11T05:15:32Z

@cursor review

TheodoreSpeaks · 2026-08-11T05:55:55Z

@greptile

TheodoreSpeaks · 2026-08-11T05:55:59Z

@cursor review

TheodoreSpeaks · 2026-08-11T06:25:59Z

@greptile

TheodoreSpeaks · 2026-08-11T06:26:10Z

@cursor review

TheodoreSpeaks · 2026-08-11T06:29:12Z

@greptile

TheodoreSpeaks · 2026-08-11T06:29:27Z

@cursor review

TheodoreSpeaks · 2026-08-11T21:35:26Z

@greptile

TheodoreSpeaks · 2026-08-11T21:35:32Z

bugbot run

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 25c1978. Configure here.}

TheodoreSpeaks · 2026-08-11T22:18:42Z

@greptile

TheodoreSpeaks · 2026-08-11T22:18:46Z

bugbot run

TheodoreSpeaks · 2026-08-11T22:30:09Z

bugbot run

TheodoreSpeaks · 2026-08-11T22:30:11Z

@greptile

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit f4864e0. Configure here.}

TheodoreSpeaks · 2026-08-11T22:41:14Z

bugbot run

TheodoreSpeaks · 2026-08-11T22:41:15Z

@greptile

TheodoreSpeaks · 2026-08-11T22:47:46Z

bugbot run

TheodoreSpeaks · 2026-08-11T22:47:46Z

@greptile

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit efbcebf. Configure here.}

TheodoreSpeaks added 2 commits July 6, 2026 16:18

improvement(tables): show error message tooltip on errored workflow a…

4785802

…nd enrichment cells

fix(enrichment): wait for valid work email inputs

319969d

greptile-apps Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/data-row.tsx Outdated

Comment thread apps/sim/enrichments/readiness.ts

cursor Bot reviewed Aug 11, 2026

View reviewed changes

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

TheodoreSpeaks added 2 commits August 10, 2026 21:25

Merge remote-tracking branch 'origin/staging' into fix/table-error

3b92819

# Conflicts: # apps/sim/background/workflow-column-execution.ts

fix(enrichment): rearm waiting cells on input changes

6d26638

vercel Bot deployed to Preview August 11, 2026 04:36 View deployment

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/data-row.tsx Outdated

TheodoreSpeaks added 2 commits August 10, 2026 22:12

Merge remote-tracking branch 'origin/staging' into fix/table-error

0d0bcf5

fix(enrichment): align readiness input mapping

a30ed59

vercel Bot deployed to Preview August 11, 2026 05:20 View deployment

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/data-row.tsx

fix(enrichment): skip unready table dispatches

346e003

vercel Bot temporarily deployed to Preview August 11, 2026 05:55 Inactive

greptile-apps Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/data-row.tsx Outdated

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/utils.ts

fix(enrichment): track manual table attempts

3dfa07b

fix(tables): keep active cell status visible

4401581

vercel Bot temporarily deployed to Preview August 11, 2026 06:28 Inactive

vercel Bot temporarily deployed to Preview August 11, 2026 21:34 Inactive

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Merge remote-tracking branch 'origin/staging' into fix/table-error

e9bd79e

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/lib/table/deps.ts

vercel Bot deployed to Preview August 11, 2026 21:55 View deployment

Merge remote-tracking branch 'origin/staging' into fix/table-error

389c482

vercel Bot deployed to Preview August 11, 2026 22:10 View deployment

fix(enrichment): clear stale manual outputs

58618b0

vercel Bot temporarily deployed to Preview August 11, 2026 22:18 Inactive

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/lib/table/rows/service.ts Outdated

fix(enrichment): cancel stale downstream runs

f4864e0

vercel Bot temporarily deployed to Preview August 11, 2026 22:29 Inactive

cursor Bot reviewed Aug 11, 2026

View reviewed changes

greptile-apps Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/lib/table/rows/service.ts

fix(enrichment): invalidate all row mutation paths

e84015e

vercel Bot temporarily deployed to Preview August 11, 2026 22:40 Inactive

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Comment thread apps/sim/lib/table/rows/service.ts

fix(enrichment): cancel upsert dependents

efbcebf

vercel Bot temporarily deployed to Preview August 11, 2026 22:47 Inactive

cursor Bot reviewed Aug 11, 2026

View reviewed changes

Uh oh!

Conversation

TheodoreSpeaks commented Aug 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of Change

Testing

Checklist

Uh oh!

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

Uh oh!

Uh oh!

cursor Bot commented Aug 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

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

Uh oh!

Greptile Summary

Confidence Score: 5/5

Important Files Changed

Flowchart

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

TheodoreSpeaks commented Aug 11, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

TheodoreSpeaks commented Aug 11, 2026 •

edited

Loading

vercel Bot commented Aug 11, 2026 •

edited

Loading

cursor Bot commented Aug 11, 2026 •

edited

Loading

greptile-apps Bot commented Aug 11, 2026 •

edited

Loading