@coldiq/mcp 0.1.13 → 0.1.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@coldiq/mcp",
-  "version": "0.1.13",
+  "version": "0.1.14",
   "type": "module",
   "main": "dist/index.js",
   "bin": {

package/src/registry.ts

@@ -1045,7 +1045,9 @@ const findPeopleProviders: ProviderEntry[] = [
       },
       extractId: (response) => {
         const d = response as Record<string, unknown>
-        return (d._id as string) ?? (d.search_id as string)
+        // LeadsFactory's upstream returns the ObjectId as `id`; the short `search_id`
+        // cannot be used as the GET param (upstream rejects it as "not a valid ObjectId").
+        return (d.id as string) ?? (d.search_id as string)
       },
     },
   },

package/tests/live/leadsfactory-upstream-probe.ts

@@ -0,0 +1,97 @@
+// Direct upstream probe — bypasses the marketplace proxy. Goal: figure out why
+// GET /api/v1/find-people/searches/{id} returns errors from our proxy. Probe with
+// both id formats (mongo _id and short search_id) and both path forms (singular vs plural).
+//
+// Run:  LEADSFACTORY_API_KEY=… npx tsx mcp/tests/live/leadsfactory-upstream-probe.ts
+
+const KEY = process.env.LEADSFACTORY_API_KEY
+if (!KEY) { console.error('LEADSFACTORY_API_KEY required'); process.exit(1) }
+const BASE = 'https://apiv2.leadsfactory.io'
+
+async function call(method: 'GET' | 'POST', path: string, body?: unknown) {
+  const t0 = Date.now()
+  try {
+    const res = await fetch(`${BASE}${path}`, {
+      method,
+      headers: {
+        'X-API-Key': KEY!,
+        ...(body ? { 'Content-Type': 'application/json' } : {}),
+      },
+      body: body ? JSON.stringify(body) : undefined,
+      signal: AbortSignal.timeout(30_000),
+    })
+    let data: any
+    try { data = await res.json() } catch { data = await res.text().catch(() => '<no body>') }
+    return { ok: res.ok, status: res.status, ms: Date.now() - t0, data }
+  } catch (err) {
+    return { ok: false, status: 0, ms: Date.now() - t0, data: { error: String(err) } }
+  }
+}
+
+function summary(r: any, max = 300) {
+  const s = typeof r.data === 'string' ? r.data : JSON.stringify(r.data)
+  return `status=${r.status} ms=${r.ms} body=${s.slice(0, max)}`
+}
+
+async function main() {
+  console.log('=== LeadsFactory upstream probe ===')
+  console.log(`BASE=${BASE}\n`)
+
+  // 1. Create a small search to get fresh IDs to test against
+  console.log('▸ Step 1: POST to create a small search (1 contact at ColdIQ)')
+  const post = await call('POST', '/api/v1/find-people/search', {
+    company_domains: ['coldiq.com'],
+    search: {
+      max_persona_results: 1,
+      personas: [{ job_title: 'CEO' }],
+    },
+  })
+  console.log(`  ${summary(post, 800)}`)
+
+  if (!post.ok) {
+    console.log('  POST failed; aborting GET probes.')
+    return
+  }
+
+  const mongoId = post.data?._id
+  const searchId = post.data?.search_id
+  console.log(`\n  mongoId=${mongoId}`)
+  console.log(`  searchId=${searchId}\n`)
+
+  // 2. Try the GET in every shape we can think of
+  console.log('▸ Step 2: GET probes (try every path/id combo)')
+
+  for (const id of [mongoId, searchId].filter(Boolean)) {
+    const idType = id === mongoId ? 'mongoId' : 'searchId'
+    for (const path of [
+      `/api/v1/find-people/searches/${id}`,       // current marketplace shape (plural)
+      `/api/v1/find-people/search/${id}`,         // singular — matches POST path style
+      `/api/v1/find-people/searches/${id}/`,      // trailing slash
+      `/api/v1/find-people/${id}`,                // no segment
+      `/api/v1/find-people/search?id=${id}`,      // query param
+      `/api/v1/find-people/search?search_id=${id}`,
+    ]) {
+      const r = await call('GET', path)
+      console.log(`  [${idType}] GET ${path}`)
+      console.log(`    ${summary(r)}`)
+    }
+  }
+
+  // 3. Try the GET via different auth shapes (in case X-API-Key vs Bearer matters for GET)
+  console.log('\n▸ Step 3: alternate auth headers on the canonical path (plural + mongoId)')
+  const probePath = `/api/v1/find-people/searches/${mongoId}`
+  for (const [label, headers] of [
+    ['X-API-Key', { 'X-API-Key': KEY! }],
+    ['Authorization Bearer', { Authorization: `Bearer ${KEY}` }],
+    ['Authorization raw', { Authorization: KEY! }],
+    ['Api-Key', { 'Api-Key': KEY! }],
+  ] as const) {
+    const t0 = Date.now()
+    const res = await fetch(`${BASE}${probePath}`, { headers, signal: AbortSignal.timeout(20_000) })
+    let body: any
+    try { body = await res.json() } catch { body = await res.text() }
+    console.log(`  [${label}] status=${res.status} ms=${Date.now() - t0} body=${JSON.stringify(body).slice(0, 200)}`)
+  }
+}
+
+main().catch((e) => { console.error('FATAL', e); process.exit(1) })

package/tests/tools/find-people.test.ts

@@ -21,7 +21,7 @@ describe('find_people handler', () => {
 
       // LeadsFactory create search — returns an ID
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && callCount === 1) {
-        return new Response(JSON.stringify({ _id: 'abc123', search_id: 'abc', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', search_id: 'abc', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
       }
 
       // LeadsFactory poll — always RUNNING (will timeout)
@@ -70,7 +70,7 @@ describe('find_people handler', () => {
 
       // LeadsFactory create search
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && !urlStr.includes('abc123')) {
-        return new Response(JSON.stringify({ _id: 'abc123', search_id: 'abc', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', search_id: 'abc', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
       }
 
       // LeadsFactory poll — first RUNNING, then SUCCESSFUL with contacts
@@ -121,7 +121,7 @@ describe('find_people handler', () => {
 
       // LeadsFactory create
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && !urlStr.includes('abc123')) {
-        return new Response(JSON.stringify({ _id: 'abc123', search_id: 'abc', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', search_id: 'abc', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
       }
 
       // LeadsFactory poll — SUCCESSFUL with one miss
@@ -179,7 +179,7 @@ describe('find_people handler', () => {
       const urlStr = url.toString()
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && !urlStr.includes('abc123')) {
-        return new Response(JSON.stringify({ _id: 'abc123', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
       }
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches/abc123')) {
@@ -231,7 +231,7 @@ describe('find_people handler', () => {
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && !urlStr.includes('abc123')) {
         capturedBody = JSON.parse((opts?.body as string) ?? '{}')
-        return new Response(JSON.stringify({ _id: 'abc123', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', nb_jobs_total: 2, progress_percentage: 0 }), { status: 201 })
       }
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches/abc123')) {
@@ -277,7 +277,7 @@ describe('find_people handler', () => {
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches') && !urlStr.includes('abc123')) {
         capturedBody = JSON.parse((opts?.body as string) ?? '{}')
-        return new Response(JSON.stringify({ _id: 'abc123', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
+        return new Response(JSON.stringify({ id: 'abc123', nb_jobs_total: 1, progress_percentage: 0 }), { status: 201 })
       }
 
       if (urlStr.includes('/leadsfactory/contact-finder/searches/abc123')) {