feat(leads): record contacts on both research paths, and let the list out

[ralyodio]

outreach_contacts            9 rows
leads with an address       77
missing from contacts       70

A missing call, not a broken one

researchProspect has two branches. Only the unscanned one called recordContact. Almost every lead arrived through the scanning branch, which wrote the address onto the prospect and nowhere else — so the durable, org-scoped record the table exists to be was very nearly empty, while the data sat in project-scoped rows that have to be read one project at a time.

Both branches record now, and the scanning branch sets contact_id so the link works in both directions.

Contacts now carry a niche

They never did. The campaign that found someone is the only statement about them the pipeline can make without asking a model to guess — someone found by a campaign called "game designers" is a game designer. A list that can't be segmented is worth very little, and segment size is what decides that.

Backfill: the history the code fix can't reach

Idempotent — the unique index on (organization_id, identity_key) already holds every row it would add. Applied to prod:

contacts          9  →  80
with a niche             63
with a company           51
prospects linked         78
still missing             0

It had no surface at all

Which made it a table, not an asset. Now: a panel showing segment sizes, and a CSV export.

• Fields starting with =, +, - or @ are prefixed — a contact list is exactly the kind of file opened in a spreadsheet without a thought, and a leading = there is executable
• RFC 4180 quoting, CRLF, UTF-8 BOM so Excel doesn't mangle accented names
• do_not_contact rows are excluded from both the counts and the export, so a downloaded file can't reintroduce them somewhere else
• Viewers can read it — a read-only member being unable to open the list would make the export useless to the people most likely to want it

Checks

• tsc --noEmit clean · 1,028 tests pass (14 new) · build compiles
• Three of the new tests read the source: a unit test of upsertContact passed the whole time this was broken, because nothing called it

🤖 Generated with Claude Code

[github-actions]

vu1nz Security Review

0 finding(s) in PR #?

No security issues found.