@coldiq/mcp 0.2.4 → 0.2.6

package/tests/tools/verify-email.test.ts

@@ -1,6 +1,66 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { initClient } from '../../src/client.js'
-import { verifyEmailHandler } from '../../src/tools/verify-email.js'
+import { verifyEmailHandler, addNormalizedStatus } from '../../src/tools/verify-email.js'
+
+describe('addNormalizedStatus', () => {
+  it('findymail verified=true → status valid', () => {
+    const out = addNormalizedStatus({ email: 'michel@coldiq.com', verified: true, provider: 'Google' }, 'findymail') as Record<string, unknown>
+    expect(out.status).toBe('valid')
+    // Original fields preserved for back-compat.
+    expect(out.verified).toBe(true)
+    expect(out.provider).toBe('Google')
+  })
+
+  it('findymail verified=false → status invalid', () => {
+    const out = addNormalizedStatus({ email: 'bad@example.com', verified: false, provider: 'Other' }, 'findymail') as Record<string, unknown>
+    expect(out.status).toBe('invalid')
+  })
+
+  it('icypeas item.status=catch_all → catch_all', () => {
+    const out = addNormalizedStatus({ success: true, item: { status: 'catch_all', email: 'x@y.com' } }, 'icypeas') as Record<string, unknown>
+    expect(out.status).toBe('catch_all')
+  })
+
+  it('icypeas missing item → unknown', () => {
+    const out = addNormalizedStatus({ success: true }, 'icypeas') as Record<string, unknown>
+    expect(out.status).toBe('unknown')
+  })
+
+  it('upstream status="catchall" is normalized to catch_all', () => {
+    const out = addNormalizedStatus({ status: 'catchall', email: 'x@y.com' }, 'instantly') as Record<string, unknown>
+    expect(out.status).toBe('catch_all')
+  })
+
+  it('upstream status="risky" stays risky', () => {
+    const out = addNormalizedStatus({ status: 'risky' }, 'instantly') as Record<string, unknown>
+    expect(out.status).toBe('risky')
+  })
+
+  it('upstream status="disposable" stays disposable', () => {
+    const out = addNormalizedStatus({ status: 'disposable' }, 'instantly') as Record<string, unknown>
+    expect(out.status).toBe('disposable')
+  })
+
+  it('upstream status="VALID" is lowercased to valid', () => {
+    const out = addNormalizedStatus({ status: 'VALID' }, 'instantly') as Record<string, unknown>
+    expect(out.status).toBe('valid')
+  })
+
+  it('unknown provider id with no signal fields → unknown', () => {
+    const out = addNormalizedStatus({ foo: 'bar' }, 'mystery-provider') as Record<string, unknown>
+    expect(out.status).toBe('unknown')
+  })
+
+  it('null / non-object data is returned unchanged', () => {
+    expect(addNormalizedStatus(null, 'findymail')).toBe(null)
+    expect(addNormalizedStatus('a string', 'findymail')).toBe('a string')
+  })
+
+  it('linkupapi-validate verified=true → valid', () => {
+    const out = addNormalizedStatus({ verified: true }, 'linkupapi-validate') as Record<string, unknown>
+    expect(out.status).toBe('valid')
+  })
+})
 
 describe('verify_email handler', () => {
   const originalFetch = globalThis.fetch
@@ -13,9 +73,9 @@ describe('verify_email handler', () => {
     globalThis.fetch = originalFetch
   })
 
-  it('returns verification result from FindyMail', async () => {
+  it('normalizes findymail real upstream shape (verified boolean) to status=valid', async () => {
     globalThis.fetch = vi.fn(async () =>
-      new Response(JSON.stringify({ status: 'valid', email: 'michel@coldiq.com' }), { status: 200 })
+      new Response(JSON.stringify({ email: 'michel@coldiq.com', verified: true, provider: 'Google' }), { status: 200 })
     ) as typeof fetch
 
     const result = await verifyEmailHandler({ email: 'michel@coldiq.com' })
@@ -23,6 +83,20 @@ describe('verify_email handler', () => {
     const parsed = JSON.parse(result.content[0].text)
     expect(parsed._meta.provider).toBe('findymail')
     expect(parsed.data.status).toBe('valid')
+    // Raw fields still surfaced for callers that already depend on them.
+    expect(parsed.data.verified).toBe(true)
+    expect(parsed.data.provider).toBe('Google')
+  })
+
+  it('normalizes findymail verified=false to status=invalid', async () => {
+    globalThis.fetch = vi.fn(async () =>
+      new Response(JSON.stringify({ email: 'bad@example.com', verified: false, provider: 'Other' }), { status: 200 })
+    ) as typeof fetch
+
+    const result = await verifyEmailHandler({ email: 'bad@example.com' })
+
+    const parsed = JSON.parse(result.content[0].text)
+    expect(parsed.data.status).toBe('invalid')
   })
 
   it('falls back to IcyPeas on FindyMail failure', async () => {

package/tests/utils/compact-people.test.ts

@@ -35,12 +35,15 @@ describe('extractPeopleArray', () => {
     expect(extractPeopleArray({ people: [{ name: 'A' }] }, 'fullenrich-people-search')).toHaveLength(1)
   })
 
-  it('pdl / prospeo / companyenrich: top-level data[]', () => {
+  it('pdl / companyenrich: top-level data[]', () => {
     expect(extractPeopleArray({ data: [{ full_name: 'A' }] }, 'pdl')).toHaveLength(1)
-    expect(extractPeopleArray({ data: [{ name: 'B' }] }, 'prospeo-search-person')).toHaveLength(1)
     expect(extractPeopleArray({ data: [{ name: 'C' }] }, 'companyenrich')).toHaveLength(1)
   })
 
+  it('prospeo-search-person: top-level results[] (real upstream shape)', () => {
+    expect(extractPeopleArray({ results: [{ person: { full_name: 'B' } }] }, 'prospeo-search-person')).toHaveLength(1)
+  })
+
   it('ai-ark-people: reads content[]', () => {
     expect(extractPeopleArray({ content: [{ name: 'A' }] }, 'ai-ark-people')).toHaveLength(1)
   })
@@ -286,6 +289,28 @@ describe('normalizePerson — ai-ark profile-nested shape', () => {
   })
 })
 
+describe('normalizePerson — prospeo person-nested shape', () => {
+  // Prospeo /search-person returns { results: [{ person: { full_name, current_job_title, ... } }] }.
+  // Verified against the live api.prospeo.io/search-person response.
+  it('unwraps record.person and reads current_job_title as title', () => {
+    const p = normalizePerson({
+      person: {
+        person_id: 'aaaa07ad691a3a7f2ac039c6',
+        full_name: 'Catherine Powlett',
+        first_name: 'Catherine',
+        last_name: 'Powlett',
+        current_job_title: 'Marketing Manager',
+        linkedin_url: 'https://www.linkedin.com/in/catherinepowlett',
+      },
+    })!
+    expect(p.full_name).toBe('Catherine Powlett')
+    expect(p.first_name).toBe('Catherine')
+    expect(p.last_name).toBe('Powlett')
+    expect(p.title).toBe('Marketing Manager')
+    expect(p.linkedin_url).toBe('https://www.linkedin.com/in/catherinepowlett')
+  })
+})
+
 describe('normalizePerson — leadsfactory persona-level company sibling', () => {
   // LF persona items have shape { contact, company, persona_index, persona_job_title }.
   // Verified against src/providers/leadsfactory/schema.ts:134-141. The company sibling