@commandable/integration-data 0.0.1 → 0.0.4

package/integrations/notion/__tests__/usage_parity.test.ts

@@ -1,34 +1,9 @@
-import { existsSync, readdirSync, readFileSync } from 'node:fs'
-import { resolve } from 'node:path'
-import { fileURLToPath } from 'node:url'
 import { describe, expect, it } from 'vitest'
-import { loadIntegrationManifest } from '../../../src/integrations/dataLoader.js'
-
-function escapeRegExp(str: string): string {
-  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-}
+import { getMissingToolUsages } from '../../__tests__/usageParity.js'
 
 describe('notion static usage parity', () => {
-  it('every manifest tool is referenced in tests via build*(name)', () => {
-    const manifest = loadIntegrationManifest('notion')!
-    const toolNames = (manifest.tools as any[]).map(t => t.name)
-
-    const testsDir = fileURLToPath(new URL('.', import.meta.url))
-    expect(existsSync(testsDir)).toBe(true)
-    const testFiles = readdirSync(testsDir)
-      .filter(f => /\.test\.(t|j)s$/.test(f) && !f.includes('usage_parity.test'))
-      .map(f => resolve(testsDir, f))
-
-    const fileContents = testFiles.map(f => readFileSync(f, 'utf8'))
-
-    const missing: string[] = []
-    for (const name of toolNames) {
-      const nameRe = new RegExp(`build(?:Read|Write|Admin)?(?:Handler)?\\(\\s*['\"\`]${escapeRegExp(name)}['\"\`]\\s*\\)`, 'm')
-      const found = fileContents.some(src => nameRe.test(src))
-      if (!found)
-        missing.push(name)
-    }
-
+  it('every manifest tool is referenced in tests', () => {
+    const missing = getMissingToolUsages({ integrationName: 'notion', importMetaUrl: import.meta.url })
     expect(missing, `Missing handler usages in tests: ${missing.join(', ')}`).toEqual([])
   })
 })

package/integrations/notion/__tests__/write_and_admin_handlers.test.ts

@@ -1,73 +1,69 @@
-import { beforeAll, describe, expect, it } from 'vitest'
-import { IntegrationProxy } from '../../../src/integrations/proxy.js'
-import { loadIntegrationTools } from '../../../src/integrations/dataLoader.js'
+import { afterAll, beforeAll, describe, expect, it } from 'vitest'
+import { createCredentialStore, createIntegrationNode, createProxy, createToolbox, hasEnv, safeCleanup } from '../../__tests__/liveHarness.js'
 
 interface Ctx {
   createdPageId?: string
-  parentDatabaseId?: string
+  testDatabaseId?: string
   createdDatabaseId?: string
 }
 
-const env = process.env as Record<string, string>
-const hasEnv = (...keys: string[]) => keys.every(k => !!env[k] && env[k].trim().length > 0)
 const suite = hasEnv(
-  'COMMANDABLE_MANAGED_OAUTH_BASE_URL',
-  'COMMANDABLE_MANAGED_OAUTH_SECRET_KEY',
-  'NOTION_TEST_CONNECTION_ID',
+  'NOTION_TOKEN',
+  'NOTION_TEST_PARENT_PAGE_ID',
 )
   ? describe
   : describe.skip
 
 suite('notion write handlers (live)', () => {
   const ctx: Ctx = {}
-  let buildWrite: (name: string) => ((input: any) => Promise<any>)
-  let buildRead: (name: string) => ((input: any) => Promise<any>)
+  let notion: ReturnType<typeof createToolbox>
 
   beforeAll(async () => {
-    const { COMMANDABLE_MANAGED_OAUTH_BASE_URL, COMMANDABLE_MANAGED_OAUTH_SECRET_KEY, NOTION_TEST_CONNECTION_ID } = env
-
-    const proxy = new IntegrationProxy({
-      managedOAuthBaseUrl: COMMANDABLE_MANAGED_OAUTH_BASE_URL,
-      managedOAuthSecretKey: COMMANDABLE_MANAGED_OAUTH_SECRET_KEY,
+    const env = process.env as Record<string, string | undefined>
+    const credentialStore = createCredentialStore(async () => ({ token: env.NOTION_TOKEN || '' }))
+    const proxy = createProxy(credentialStore)
+    notion = createToolbox('notion', proxy, createIntegrationNode('notion', { label: 'Notion', credentialId: 'notion-creds' }))
+
+    // Create a dedicated database under a known parent page for this run
+    const create_database = notion.write('create_database')
+    const createdDb = await create_database({
+      parent: { page_id: env.NOTION_TEST_PARENT_PAGE_ID },
+      title: [{ type: 'text', text: { content: `CmdTest DB ${Date.now()}` } }],
+      properties: {
+        Name: { title: {} },
+        Status: { select: { options: [{ name: 'Open' }, { name: 'Done' }] } },
+      },
     })
-    const integrationNode = { id: 'node-notion', type: 'notion', label: 'Notion', connectionId: NOTION_TEST_CONNECTION_ID } as any
-
-    const tools = loadIntegrationTools('notion')
-    expect(tools).toBeTruthy()
-
-    buildWrite = (name: string) => {
-      const tool = tools!.write.find(t => t.name === name)
-      expect(tool, `write tool ${name} exists`).toBeTruthy()
-      const integration = { fetch: (path: string, init?: RequestInit) => proxy.call(integrationNode, path, init) }
-      const build = new Function('integration', `return (${tool!.handlerCode});`)
-      return build(integration) as (input: any) => Promise<any>
-    }
-
-    buildRead = (name: string) => {
-      const tool = tools!.read.find(t => t.name === name)
-      expect(tool, `read tool ${name} exists`).toBeTruthy()
-      const integration = { fetch: (path: string, init?: RequestInit) => proxy.call(integrationNode, path, init) }
-      const build = new Function('integration', `return (${tool!.handlerCode});`)
-      return build(integration) as (input: any) => Promise<any>
-    }
-
-    // Try to find a database to create a page in by searching
-    try {
-      const search = buildRead('search')
-      const res = await search({ query: '', filter: { value: 'database', property: 'object' }, page_size: 1 })
-      ctx.parentDatabaseId = res?.results?.[0]?.id
-    }
-    catch {}
+    ctx.testDatabaseId = createdDb?.id
+    expect(ctx.testDatabaseId).toBeTruthy()
   }, 60000)
 
+  afterAll(async () => {
+    // Archive any created page + any created databases from this run
+    await safeCleanup(async () => {
+      if (!ctx.createdPageId)
+        return
+      const update_page_properties = notion.write('update_page_properties')
+      await update_page_properties({ page_id: ctx.createdPageId, properties: {}, archived: true })
+    })
+
+    await safeCleanup(async () => {
+      const update_database = notion.write('update_database')
+      if (ctx.createdDatabaseId)
+        await update_database({ database_id: ctx.createdDatabaseId, archived: true })
+      if (ctx.testDatabaseId)
+        await update_database({ database_id: ctx.testDatabaseId, archived: true })
+    })
+  }, 60_000)
+
   it('create_page -> retrieve_page -> update_page_properties', async () => {
-    if (!ctx.parentDatabaseId)
+    if (!ctx.testDatabaseId)
       return expect(true).toBe(true)
 
-    const create_page = buildWrite('create_page')
+    const create_page = notion.write('create_page')
     const titleText = `CmdTest ${Date.now()}`
     const created = await create_page({
-      parent: { database_id: ctx.parentDatabaseId },
+      parent: { database_id: ctx.testDatabaseId },
       properties: {
         Name: { title: [{ type: 'text', text: { content: titleText } }] },
       },
@@ -76,11 +72,11 @@ suite('notion write handlers (live)', () => {
     expect(pageId).toBeTruthy()
     ctx.createdPageId = pageId
 
-    const retrieve_page = buildRead('retrieve_page')
+    const retrieve_page = notion.read('retrieve_page')
     const got = await retrieve_page({ page_id: pageId })
     expect(got?.id).toBe(pageId)
 
-    const update_page_properties = buildWrite('update_page_properties')
+    const update_page_properties = notion.write('update_page_properties')
     const newTitle = `${titleText} Updated`
     const updated = await update_page_properties({ page_id: pageId, properties: { Name: { title: [{ type: 'text', text: { content: newTitle } }] } } })
     expect(updated?.id).toBe(pageId)
@@ -94,7 +90,7 @@ suite('notion write handlers (live)', () => {
   it('append_block_children on created page', async () => {
     if (!ctx.createdPageId)
       return expect(true).toBe(true)
-    const append_block_children = buildWrite('append_block_children')
+    const append_block_children = notion.write('append_block_children')
     const contentText = 'Hello from test'
     const res = await append_block_children({
       block_id: ctx.createdPageId,
@@ -105,7 +101,7 @@ suite('notion write handlers (live)', () => {
     expect(res).toBeTruthy()
 
     // Verify via list_block_children
-    const list_block_children = buildRead('list_block_children')
+    const list_block_children = notion.read('list_block_children')
     const listed = await list_block_children({ block_id: ctx.createdPageId })
     const found = (listed?.results || listed || []).some((b: any) => b?.paragraph?.rich_text?.some((t: any) => (t?.plain_text || t?.text?.content) === contentText))
     expect(found).toBe(true)
@@ -114,13 +110,13 @@ suite('notion write handlers (live)', () => {
   it('create_comment on created page', async () => {
     if (!ctx.createdPageId)
       return expect(true).toBe(true)
-    const create_comment = buildWrite('create_comment')
+    const create_comment = notion.write('create_comment')
     const commentText = 'Test comment'
     const res = await create_comment({ parent: { block_id: ctx.createdPageId }, rich_text: [{ type: 'text', text: { content: commentText } }] })
     expect(res?.object === 'comment' || res?.results).toBeTruthy()
 
     // Verify via list_comments
-    const list_comments = buildRead('list_comments')
+    const list_comments = notion.read('list_comments')
     const comments = await list_comments({ block_id: ctx.createdPageId })
     const hasComment = (comments?.results || comments || []).some((c: any) => c?.rich_text?.some((t: any) => (t?.plain_text || t?.text?.content)?.includes(commentText)))
     expect(hasComment).toBe(true)
@@ -130,7 +126,7 @@ suite('notion write handlers (live)', () => {
     if (!ctx.createdPageId)
       return expect(true).toBe(true)
     // Create a specific block to edit
-    const append_block_children = buildWrite('append_block_children')
+    const append_block_children = notion.write('append_block_children')
     const appended = await append_block_children({
       block_id: ctx.createdPageId,
       children: [
@@ -141,18 +137,18 @@ suite('notion write handlers (live)', () => {
     if (!blockId)
       return expect(true).toBe(true)
 
-    const update_block = buildWrite('update_block')
+    const update_block = notion.write('update_block')
     const editedText = 'Edited'
     const updated = await update_block({ block_id: blockId, body: { paragraph: { rich_text: [{ type: 'text', text: { content: editedText } }] } } })
     expect(updated?.id).toBe(blockId)
 
     // Verify via retrieve_block
-    const retrieve_block = buildRead('retrieve_block')
+    const retrieve_block = notion.read('retrieve_block')
     const gotBlock = await retrieve_block({ block_id: blockId })
     const gotEdited = gotBlock?.paragraph?.rich_text?.some((t: any) => (t?.plain_text || t?.text?.content) === editedText)
     expect(gotEdited).toBe(true)
 
-    const delete_block = buildWrite('delete_block')
+    const delete_block = notion.write('delete_block')
     const del = await delete_block({ block_id: blockId })
     expect(del?.archived === true || del?.id === blockId).toBe(true)
 
@@ -164,7 +160,7 @@ suite('notion write handlers (live)', () => {
   it('create_database then update_database under created page (optional)', async () => {
     if (!ctx.createdPageId)
       return expect(true).toBe(true)
-    const create_database = buildWrite('create_database')
+    const create_database = notion.write('create_database')
     const created = await create_database({
       parent: { page_id: ctx.createdPageId },
       title: [{ type: 'text', text: { content: `CmdDB ${Date.now()}` } }],
@@ -177,12 +173,12 @@ suite('notion write handlers (live)', () => {
     ctx.createdDatabaseId = dbId
     expect(dbId).toBeTruthy()
 
-    const update_database = buildWrite('update_database')
+    const update_database = notion.write('update_database')
     const updated = await update_database({ database_id: dbId, title: [{ type: 'text', text: { content: 'CmdDB Updated' } }] })
     expect(updated?.id).toBe(dbId)
 
     // Verify via retrieve_database
-    const retrieve_database = buildRead('retrieve_database')
+    const retrieve_database = notion.read('retrieve_database')
     const gotDb = await retrieve_database({ database_id: dbId })
     const titleText = gotDb?.title?.[0]?.plain_text || gotDb?.title?.[0]?.text?.content
     expect(titleText?.includes('Updated')).toBe(true)

package/integrations/notion/credentials.json

@@ -1,21 +1,26 @@
 {
-  "schema": {
-    "type": "object",
-    "properties": {
-      "token": {
-        "type": "string",
-        "title": "Internal Integration Token",
-        "description": "Notion internal integration token (starts with \"secret_\")."
+  "variants": {
+    "internal_integration": {
+      "label": "Internal Integration Token",
+      "schema": {
+        "type": "object",
+        "properties": {
+          "token": {
+            "type": "string",
+            "title": "Internal Integration Token",
+            "description": "Notion internal integration token (starts with \"secret_\")."
+          }
+        },
+        "required": ["token"],
+        "additionalProperties": false
+      },
+      "injection": {
+        "headers": {
+          "Authorization": "Bearer {{token}}",
+          "Notion-Version": "2022-06-28"
+        }
       }
-    },
-    "required": ["token"],
-    "additionalProperties": false
-  },
-  "injection": {
-    "headers": {
-      "Authorization": "Bearer {{token}}",
-      "Notion-Version": "2022-06-28"
     }
-  }
+  },
+  "default": "internal_integration"
 }
-

package/integrations/trello/__tests__/get_handlers.test.ts

@@ -1,13 +1,10 @@
-import { beforeAll, describe, expect, it } from 'vitest'
-import { IntegrationProxy } from '../../../../server/src/integrations/proxy.js'
-import { loadIntegrationTools } from '../../../../server/src/integrations/dataLoader.js'
+import { afterAll, beforeAll, describe, expect, it } from 'vitest'
+import { createCredentialStore, createIntegrationNode, createProxy, createToolbox, hasEnv, safeCleanup } from '../../__tests__/liveHarness.js'
 
-// This is a LIVE integration test suite that hits Trello using managed OAuth.
+// This is a LIVE integration test suite that hits Trello using credentials.
 // Required env vars:
-// - COMMANDABLE_MANAGED_OAUTH_BASE_URL
-// - COMMANDABLE_MANAGED_OAUTH_SECRET_KEY
 // - TRELLO_API_KEY
-// - TRELLO_TEST_CONNECTION_ID (an existing managed OAuth connectionId for provider 'trello')
+// - TRELLO_API_TOKEN
 
 interface Ids {
   boardId?: string
@@ -16,68 +13,56 @@ interface Ids {
   orgId?: string
 }
 
-const env = process.env as Record<string, string>
-const hasEnv = (...keys: string[]) => keys.every(k => !!env[k] && env[k].trim().length > 0)
 const suite = hasEnv(
-  'COMMANDABLE_MANAGED_OAUTH_BASE_URL',
-  'COMMANDABLE_MANAGED_OAUTH_SECRET_KEY',
   'TRELLO_API_KEY',
-  'TRELLO_TEST_CONNECTION_ID',
+  'TRELLO_API_TOKEN',
 )
   ? describe
   : describe.skip
 
 suite('trello read handlers (live)', () => {
   const ids: Ids = {}
+  let boardId: string | undefined
+  let listId: string | undefined
 
-  let buildHandler: (name: string) => ((input: any) => Promise<any>)
+  let trello: ReturnType<typeof createToolbox>
 
   beforeAll(async () => {
-    const { COMMANDABLE_MANAGED_OAUTH_BASE_URL, COMMANDABLE_MANAGED_OAUTH_SECRET_KEY, TRELLO_API_KEY, TRELLO_TEST_CONNECTION_ID } = env
-
-    const proxy = new IntegrationProxy({
-      managedOAuthBaseUrl: COMMANDABLE_MANAGED_OAUTH_BASE_URL,
-      managedOAuthSecretKey: COMMANDABLE_MANAGED_OAUTH_SECRET_KEY,
-      trelloApiKey: TRELLO_API_KEY,
-    })
-    const integrationNode = { id: 'node-trello', type: 'trello', label: 'Trello', connectionId: TRELLO_TEST_CONNECTION_ID } as any
-
-    const tools = loadIntegrationTools('trello')
-    expect(tools).toBeTruthy()
-
-    buildHandler = (name: string) => {
-      const tool = tools!.read.find(t => t.name === name)
-      expect(tool, `tool ${name} exists`).toBeTruthy()
-      const integration = {
-        fetch: (path: string, init?: RequestInit) => proxy.call(integrationNode, path, init),
-      }
-      const build = new Function('integration', `return (${tool!.handlerCode});`)
-      return build(integration) as (input: any) => Promise<any>
-    }
-
-    // Discover a board, list, card, and org for subsequent tests
-    const get_member_boards = buildHandler('get_member_boards')
-    const boards = await get_member_boards({})
-    expect(Array.isArray(boards)).toBe(true)
-    ids.boardId = boards[0]?.id
-
-    if (ids.boardId) {
-      const get_board_lists = buildHandler('get_board_lists')
-      const lists = await get_board_lists({ boardId: ids.boardId })
-      ids.listId = lists[0]?.id
-
-      const get_board_cards = buildHandler('get_board_cards')
-      const cards = await get_board_cards({ boardId: ids.boardId })
-      ids.cardId = cards[0]?.id
-    }
-
-    const get_member_organizations = buildHandler('get_member_organizations')
+    const env = process.env as Record<string, string | undefined>
+    const credentialStore = createCredentialStore(async () => ({ apiKey: env.TRELLO_API_KEY || '', apiToken: env.TRELLO_API_TOKEN || '' }))
+    const proxy = createProxy(credentialStore)
+    const node = createIntegrationNode('trello', { label: 'Trello', credentialId: 'trello-creds' })
+    trello = createToolbox('trello', proxy, node)
+
+    // Create an isolated board/list/card for this run so tests don’t touch random user boards.
+    const board = await trello.write('create_board')({ name: `CmdTest Trello Read ${Date.now()}`, defaultLists: false })
+    boardId = board?.id
+    ids.boardId = boardId
+    expect(ids.boardId).toBeTruthy()
+
+    const list = await trello.write('create_list')({ idBoard: boardId, name: 'CmdTest List' })
+    listId = list?.id
+    ids.listId = listId
+    expect(ids.listId).toBeTruthy()
+
+    const card = await trello.write('create_card')({ idList: listId, name: `CmdTest Card ${Date.now()}` })
+    ids.cardId = card?.id
+    expect(ids.cardId).toBeTruthy()
+
+    const get_member_organizations = trello.read('get_member_organizations')
     const orgs = await get_member_organizations({})
     ids.orgId = orgs[0]?.id
   }, 60000)
 
+  afterAll(async () => {
+    if (!boardId)
+      return
+    await safeCleanup(async () => trello.write('close_board')({ boardId }))
+    await safeCleanup(async () => trello.write('delete_board')({ boardId }))
+  }, 60_000)
+
   it('get_member returns current member', async () => {
-    const handler = buildHandler('get_member')
+    const handler = trello.read('get_member')
     const result = await handler({})
     expect(result).toBeTruthy()
     expect(typeof result.id).toBe('string')
@@ -85,13 +70,13 @@ suite('trello read handlers (live)', () => {
   }, 30000)
 
   it('get_member_boards returns an array', async () => {
-    const handler = buildHandler('get_member_boards')
+    const handler = trello.read('get_member_boards')
     const result = await handler({})
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
 
   it('get_member_organizations returns an array', async () => {
-    const handler = buildHandler('get_member_organizations')
+    const handler = trello.read('get_member_organizations')
     const result = await handler({})
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -99,7 +84,7 @@ suite('trello read handlers (live)', () => {
   it('get_board works with boardId', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board')
+    const handler = trello.read('get_board')
     const result = await handler({ boardId: ids.boardId })
     expect(result?.id).toBe(ids.boardId)
   }, 30000)
@@ -107,7 +92,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_lists returns lists', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_lists')
+    const handler = trello.read('get_board_lists')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -115,7 +100,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_cards returns cards', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_cards')
+    const handler = trello.read('get_board_cards')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -123,7 +108,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_members returns members', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_members')
+    const handler = trello.read('get_board_members')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -131,7 +116,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_labels returns labels', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_labels')
+    const handler = trello.read('get_board_labels')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -139,7 +124,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_custom_fields returns custom fields', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_custom_fields')
+    const handler = trello.read('get_board_custom_fields')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -147,7 +132,7 @@ suite('trello read handlers (live)', () => {
   it('get_board_memberships returns memberships', async () => {
     if (!ids.boardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_board_memberships')
+    const handler = trello.read('get_board_memberships')
     const result = await handler({ boardId: ids.boardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -155,7 +140,7 @@ suite('trello read handlers (live)', () => {
   it('get_list returns a list', async () => {
     if (!ids.listId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_list')
+    const handler = trello.read('get_list')
     const result = await handler({ listId: ids.listId })
     expect(result?.id).toBe(ids.listId)
   }, 30000)
@@ -163,7 +148,7 @@ suite('trello read handlers (live)', () => {
   it('get_list_cards returns cards in a list', async () => {
     if (!ids.listId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_list_cards')
+    const handler = trello.read('get_list_cards')
     const result = await handler({ listId: ids.listId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -171,7 +156,7 @@ suite('trello read handlers (live)', () => {
   it('get_card returns a card', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card')
+    const handler = trello.read('get_card')
     const result = await handler({ cardId: ids.cardId })
     expect(result?.id).toBe(ids.cardId)
   }, 30000)
@@ -179,7 +164,7 @@ suite('trello read handlers (live)', () => {
   it('get_card_members returns members for a card', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card_members')
+    const handler = trello.read('get_card_members')
     const result = await handler({ cardId: ids.cardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -187,7 +172,7 @@ suite('trello read handlers (live)', () => {
   it('get_card_attachments returns attachments for a card', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card_attachments')
+    const handler = trello.read('get_card_attachments')
     const result = await handler({ cardId: ids.cardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -195,7 +180,7 @@ suite('trello read handlers (live)', () => {
   it('get_card_actions returns actions for a card', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card_actions')
+    const handler = trello.read('get_card_actions')
     const result = await handler({ cardId: ids.cardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -203,7 +188,7 @@ suite('trello read handlers (live)', () => {
   it('get_card_checklists returns checklists for a card', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card_checklists')
+    const handler = trello.read('get_card_checklists')
     const result = await handler({ cardId: ids.cardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -211,7 +196,7 @@ suite('trello read handlers (live)', () => {
   it('get_card_custom_field_items returns custom field items', async () => {
     if (!ids.cardId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_card_custom_field_items')
+    const handler = trello.read('get_card_custom_field_items')
     const result = await handler({ cardId: ids.cardId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
@@ -219,7 +204,7 @@ suite('trello read handlers (live)', () => {
   it('get_organization returns an organization', async () => {
     if (!ids.orgId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_organization')
+    const handler = trello.read('get_organization')
     const result = await handler({ orgId: ids.orgId })
     expect(result?.id).toBe(ids.orgId)
   }, 30000)
@@ -227,13 +212,13 @@ suite('trello read handlers (live)', () => {
   it('get_organization_boards returns boards in an org', async () => {
     if (!ids.orgId)
       return expect(true).toBe(true)
-    const handler = buildHandler('get_organization_boards')
+    const handler = trello.read('get_organization_boards')
     const result = await handler({ orgId: ids.orgId })
     expect(Array.isArray(result)).toBe(true)
   }, 30000)
 
   it('search returns results for a generic query', async () => {
-    const handler = buildHandler('search')
+    const handler = trello.read('search')
     const result = await handler({ query: 'test' })
     expect(result).toBeTruthy()
   }, 30000)

package/integrations/trello/__tests__/usage_parity.test.ts

@@ -1,34 +1,9 @@
-import { existsSync, readdirSync, readFileSync } from 'node:fs'
-import { resolve } from 'node:path'
-import { fileURLToPath } from 'node:url'
 import { describe, expect, it } from 'vitest'
-import { loadIntegrationManifest } from '../../../src/integrations/dataLoader.js'
-
-function escapeRegExp(str: string): string {
-  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-}
+import { getMissingToolUsages } from '../../__tests__/usageParity.js'
 
 describe('trello static usage parity', () => {
-  it('every manifest tool is referenced in tests via build*(name)', () => {
-    const manifest = loadIntegrationManifest('trello')!
-    const toolNames = (manifest.tools as any[]).map(t => t.name)
-
-    const testsDir = fileURLToPath(new URL('.', import.meta.url))
-    expect(existsSync(testsDir)).toBe(true)
-    const testFiles = readdirSync(testsDir)
-      .filter(f => /\.test\.(t|j)s$/.test(f) && !f.includes('usage_parity.test'))
-      .map(f => resolve(testsDir, f))
-
-    const fileContents = testFiles.map(f => readFileSync(f, 'utf8'))
-
-    const missing: string[] = []
-    for (const name of toolNames) {
-      const nameRe = new RegExp(`build(?:Read|Write|Admin)?(?:Handler)?\\(\\s*['\"\`]${escapeRegExp(name)}['\"\`]\\s*\\)`, 'm')
-      const found = fileContents.some(src => nameRe.test(src))
-      if (!found)
-        missing.push(name)
-    }
-
+  it('every manifest tool is referenced in tests', () => {
+    const missing = getMissingToolUsages({ integrationName: 'trello', importMetaUrl: import.meta.url })
     expect(missing, `Missing handler usages in tests: ${missing.join(', ')}`).toEqual([])
   })
 })