feat(enrichments): add LinkedIn profile lookup - #6926
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Providers cascade Findymail → Datagma → Prospeo → LeadMagic → PDL. Email reverse lookups run first; later providers use name + company when needed. Matches are normalized to The enrichment is registered in the catalog and gets a canvas sentence on the Data Enrichment block. Unit tests cover cascade order, param building, URL normalization, and Prospeo Reviewed by Cursor Bugbot for commit 0352193. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryAdds a LinkedIn profile enrichment that searches a hosted provider cascade and normalizes successful matches to canonical LinkedIn person URLs.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/enrichments/linkedin-profile/linkedin-profile.ts | Defines the provider cascade, request projections, failure handling, and canonical LinkedIn person-URL normalization. |
| apps/sim/enrichments/linkedin-profile/linkedin-profile.test.ts | Covers provider order, parameter construction, output normalization, and Prospeo failure projection. |
| apps/sim/enrichments/registry.ts | Registers the new LinkedIn profile enrichment. |
| apps/sim/blocks/blocks/enrichment.ts | Adds the LinkedIn profile operation’s canvas sentence. |
| packages/emcn/src/icons/linkedin.tsx | Adds the LinkedIn brand glyph used by the enrichment. |
Sequence Diagram
sequenceDiagram
participant W as Workflow
participant E as LinkedIn Enrichment
participant P as Provider Cascade
W->>E: Name, email, and company hints
loop Findymail → Datagma → Prospeo → LeadMagic → PDL
E->>P: Build provider-specific request
P-->>E: Provider response
E->>E: Normalize LinkedIn person URL
end
E-->>W: Canonical linkedinUrl or no match
Reviews (5): Last reviewed commit: "Merge remote-tracking branch 'origin/sta..." | Re-trigger Greptile
|
@cursor review |
|
@cursor review |
There was a problem hiding this comment.
✅ 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 088f8b7. Configure here.
|
@cursor review |
There was a problem hiding this comment.
✅ 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 7822eae. Configure here.
…ile-enrichment # Conflicts: # apps/sim/tools/generated/tool-outputs.ts
|
@cursor review |
There was a problem hiding this comment.
✅ 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 0352193. Configure here.
Summary
Type of Change
Testing
bun run test -- enrichments(44 tests)bun run type-checkbun run lintbun run check:auditsbun run docs-manifest:checkChecklist