@coldiq/mcp 5.4.6 → 5.4.8

package/tests/tools/list-website-visitors-sites.test.ts

@@ -0,0 +1,142 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { initClient } from '../../src/client.js'
+import {
+  listWebsiteVisitorsSitesDescription,
+  listWebsiteVisitorsSitesHandler,
+  listWebsiteVisitorsSitesSchema,
+  listWebsiteVisitorsSitesName,
+} from '../../src/tools/list-website-visitors-sites.js'
+
+describe('list_website_visitors_sites', () => {
+  const originalFetch = globalThis.fetch
+
+  beforeEach(() => {
+    initClient('http://test-api.local', 'test-key')
+  })
+
+  afterEach(() => {
+    globalThis.fetch = originalFetch
+    vi.restoreAllMocks()
+  })
+
+  type Call = { path: string; method: string }
+
+  /** Route each request by pathname; record every call in order. */
+  function captureByPath(routes: Record<string, { payload: unknown; status?: number }>) {
+    const calls: Call[] = []
+    globalThis.fetch = vi.fn(async (url: string | URL | Request, init?: RequestInit) => {
+      const path = new URL(url.toString()).pathname
+      calls.push({ path, method: init?.method ?? 'GET' })
+      const route = routes[path]
+      if (!route) return new Response(JSON.stringify({ error: 'unexpected path' }), { status: 500 })
+      return new Response(JSON.stringify(route.payload), { status: route.status ?? 200 })
+    }) as typeof fetch
+    return calls
+  }
+
+  it('is named list_website_visitors_sites', () => {
+    expect(listWebsiteVisitorsSitesName).toBe('list_website_visitors_sites')
+  })
+
+  it('GETs /visitor-id/websites when no website_id is given', async () => {
+    const list = { websites: [{ id: 7, domain: 'coldiq.com', status: 'active' }] }
+    const calls = captureByPath({ '/v1/visitor-id/websites': { payload: list } })
+
+    const result = await listWebsiteVisitorsSitesHandler({})
+
+    expect(calls).toEqual([{ path: '/v1/visitor-id/websites', method: 'GET' }])
+    expect(result.isError).toBeFalsy()
+    expect(JSON.parse(result.content[0].text)).toEqual({ data: list })
+  })
+
+  it('with website_id GETs the website and its usage and merges them as { website, usage }', async () => {
+    const website = { id: 7, domain: 'coldiq.com', embed_html: '<script></script>', install: { is_installed: true } }
+    const usage = { people: 3, companies: 12, credits: 30 }
+    const calls = captureByPath({
+      '/v1/visitor-id/websites/7': { payload: website },
+      '/v1/visitor-id/websites/7/usage': { payload: usage },
+    })
+
+    const result = await listWebsiteVisitorsSitesHandler({ website_id: 7 })
+
+    // Both reads are issued together (Promise.all), so only the set is asserted, not the order.
+    expect(calls.map((call) => call.path).sort()).toEqual([
+      '/v1/visitor-id/websites/7',
+      '/v1/visitor-id/websites/7/usage',
+    ])
+    expect(calls.every((call) => call.method === 'GET')).toBe(true)
+    expect(result.isError).toBeFalsy()
+    expect(JSON.parse(result.content[0].text)).toEqual({ data: { website, usage } })
+  })
+
+  it('shapes a list failure as isError with status and data', async () => {
+    captureByPath({ '/v1/visitor-id/websites': { payload: { error: 'Unauthorized' }, status: 401 } })
+
+    const result = await listWebsiteVisitorsSitesHandler({})
+
+    expect(result.isError).toBe(true)
+    expect(JSON.parse(result.content[0].text)).toEqual({
+      error: 'Failed to list tracked websites',
+      status: 401,
+      data: { error: 'Unauthorized' },
+    })
+  })
+
+  it('shapes a website 404 as isError with status and data', async () => {
+    captureByPath({
+      '/v1/visitor-id/websites/999': { payload: { error: 'Website not found' }, status: 404 },
+    })
+
+    const result = await listWebsiteVisitorsSitesHandler({ website_id: 999 })
+
+    expect(result.isError).toBe(true)
+    expect(JSON.parse(result.content[0].text)).toEqual({
+      error: 'Failed to read tracked website',
+      status: 404,
+      data: { error: 'Website not found' },
+    })
+  })
+
+  it('keeps the website when the usage read fails: { website, usage: null, usage_error } (T13b)', async () => {
+    const website = { id: 7, domain: 'coldiq.com', embed_html: '<script></script>', install: null }
+    captureByPath({
+      '/v1/visitor-id/websites/7': { payload: website },
+      '/v1/visitor-id/websites/7/usage': { payload: { error: 'Provider temporarily unavailable' }, status: 502 },
+    })
+
+    const result = await listWebsiteVisitorsSitesHandler({ website_id: 7 })
+
+    expect(result.isError).toBeFalsy()
+    expect(JSON.parse(result.content[0].text)).toEqual({
+      data: {
+        website,
+        usage: null,
+        usage_error: { status: 502, data: { error: 'Provider temporarily unavailable' } },
+      },
+    })
+  })
+
+  it('describes the resilient shape, the live rate, the 409 split, install null and the member rule (T13b)', () => {
+    expect(listWebsiteVisitorsSitesDescription).toContain('usage_error')
+    expect(listWebsiteVisitorsSitesDescription).toContain('6× a company')
+    expect(listWebsiteVisitorsSitesDescription).toContain('credits_per_person')
+    expect(listWebsiteVisitorsSitesDescription).toMatch(/never numbers from memory/)
+    expect(listWebsiteVisitorsSitesDescription).not.toMatch(/\d+(\.\d+)? credits/)
+    expect(listWebsiteVisitorsSitesDescription).not.toMatch(/about \d/)
+    expect(listWebsiteVisitorsSitesDescription).toMatch(/409 there whose message mentions a deactivated workspace/)
+    expect(listWebsiteVisitorsSitesDescription).toContain('do not retry')
+    expect(listWebsiteVisitorsSitesDescription).toMatch(/any other 409/)
+    expect(listWebsiteVisitorsSitesDescription).not.toContain('400')
+    expect(listWebsiteVisitorsSitesDescription).toMatch(/`install` null = state unknown/)
+    expect(listWebsiteVisitorsSitesDescription).toContain('POST/PATCH/PUT/DELETE')
+    expect(listWebsiteVisitorsSitesDescription).toContain('403')
+  })
+
+  it('accepts only an optional integer website_id at the schema layer', () => {
+    expect(Object.keys(listWebsiteVisitorsSitesSchema)).toEqual(['website_id'])
+    expect(listWebsiteVisitorsSitesSchema.website_id.safeParse(undefined).success).toBe(true)
+    expect(listWebsiteVisitorsSitesSchema.website_id.safeParse(7).success).toBe(true)
+    expect(listWebsiteVisitorsSitesSchema.website_id.safeParse(7.5).success).toBe(false)
+    expect(listWebsiteVisitorsSitesSchema.website_id.safeParse('7').success).toBe(false)
+  })
+})

package/tests/tools/setup-website-visitors.test.ts

@@ -0,0 +1,106 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { initClient } from '../../src/client.js'
+import {
+  setupWebsiteVisitorsDescription,
+  setupWebsiteVisitorsHandler,
+  setupWebsiteVisitorsSchema,
+  setupWebsiteVisitorsName,
+} from '../../src/tools/setup-website-visitors.js'
+
+describe('setup_website_visitors', () => {
+  const originalFetch = globalThis.fetch
+
+  beforeEach(() => {
+    initClient('http://test-api.local', 'test-key')
+  })
+
+  afterEach(() => {
+    globalThis.fetch = originalFetch
+    vi.restoreAllMocks()
+  })
+
+  function capture(payload: unknown, status = 201) {
+    const seen: { url: string; method: string; body: unknown } = { url: '', method: '', body: undefined }
+    globalThis.fetch = vi.fn(async (url: string | URL | Request, init?: RequestInit) => {
+      seen.url = url.toString()
+      seen.method = init?.method ?? 'GET'
+      seen.body = init?.body ? JSON.parse(String(init.body)) : undefined
+      return new Response(JSON.stringify(payload), { status })
+    }) as typeof fetch
+    return seen
+  }
+
+  it('is named setup_website_visitors', () => {
+    expect(setupWebsiteVisitorsName).toBe('setup_website_visitors')
+  })
+
+  it('quotes the live rate and the 6× person/company ratio, never an absolute credit number (T13b)', () => {
+    expect(setupWebsiteVisitorsDescription).toContain('6× a company')
+    expect(setupWebsiteVisitorsDescription).toContain('credits_per_person')
+    expect(setupWebsiteVisitorsDescription).toContain('credits_per_company')
+    expect(setupWebsiteVisitorsDescription).toContain('list_website_visitors_sites')
+    expect(setupWebsiteVisitorsDescription).toMatch(/never numbers from memory/)
+    // No literal price: "N credits", "about N", "N per identified".
+    expect(setupWebsiteVisitorsDescription).not.toMatch(/\d+(\.\d+)? credits/)
+    expect(setupWebsiteVisitorsDescription).not.toMatch(/about \d/)
+    expect(setupWebsiteVisitorsDescription).not.toMatch(/\d+ per identified/)
+  })
+
+  it('tells the agent what a deactivated-workspace 409 and any other 409 mean (T13b)', () => {
+    expect(setupWebsiteVisitorsDescription).toMatch(/409 whose message mentions a deactivated workspace/)
+    expect(setupWebsiteVisitorsDescription).toContain('do not retry')
+    expect(setupWebsiteVisitorsDescription).toContain('contact ColdIQ support')
+    expect(setupWebsiteVisitorsDescription).toMatch(/any other 409/)
+    expect(setupWebsiteVisitorsDescription).not.toContain('400')
+  })
+
+  it('explains install: null as unknown and the member 403 rule (T13b)', () => {
+    expect(setupWebsiteVisitorsDescription).toMatch(/`install` can be null/)
+    expect(setupWebsiteVisitorsDescription).toMatch(/never report "not installed" from null/)
+    expect(setupWebsiteVisitorsDescription).toContain('POST/PATCH/PUT/DELETE')
+    expect(setupWebsiteVisitorsDescription).toContain('403')
+    expect(setupWebsiteVisitorsDescription).toContain('workspace owner or an admin')
+  })
+
+  it('POSTs the domain to /visitor-id/websites and returns the website', async () => {
+    const website = { id: 7, domain: 'coldiq.com', embed_html: '<script></script>', install: { is_installed: false } }
+    const seen = capture(website)
+
+    const result = await setupWebsiteVisitorsHandler({ domain: 'coldiq.com' })
+
+    expect(seen.method).toBe('POST')
+    expect(new URL(seen.url).pathname).toBe('/v1/visitor-id/websites')
+    expect(seen.body).toEqual({ domain: 'coldiq.com' })
+    expect(result.isError).toBeFalsy()
+    expect(JSON.parse(result.content[0].text)).toEqual({ data: website })
+  })
+
+  it('forwards the optional name', async () => {
+    const seen = capture({ id: 7 })
+
+    await setupWebsiteVisitorsHandler({ domain: 'coldiq.com', name: 'ColdIQ' })
+
+    expect(seen.body).toEqual({ domain: 'coldiq.com', name: 'ColdIQ' })
+  })
+
+  it('shapes a 409 (domain already tracked) as isError with status and data', async () => {
+    capture({ error: 'coldiq.com already has a pixel on this account.' }, 409)
+
+    const result = await setupWebsiteVisitorsHandler({ domain: 'coldiq.com' })
+
+    expect(result.isError).toBe(true)
+    const parsed = JSON.parse(result.content[0].text)
+    expect(parsed).toEqual({
+      error: 'Failed to set up website visitor identification',
+      status: 409,
+      data: { error: 'coldiq.com already has a pixel on this account.' },
+    })
+  })
+
+  it('requires a non-empty domain at the schema layer and mirrors the API body fields', () => {
+    expect(Object.keys(setupWebsiteVisitorsSchema).sort()).toEqual(['domain', 'name'])
+    expect(setupWebsiteVisitorsSchema.domain.safeParse('').success).toBe(false)
+    expect(setupWebsiteVisitorsSchema.domain.safeParse('coldiq.com').success).toBe(true)
+    expect(setupWebsiteVisitorsSchema.name.safeParse(undefined).success).toBe(true)
+  })
+})