@commandable/integration-data 0.0.4 → 0.0.5

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

@@ -5,6 +5,9 @@ import { loadIntegrationTools } from '../../../../server/src/integrations/dataLo
 // LIVE Airtable integration tests using credentials
 // Required env vars:
 // - AIRTABLE_TOKEN
+// Optional (pins read tests to a specific base/table instead of auto-discovering):
+// - AIRTABLE_TEST_WRITE_BASE_ID
+// - AIRTABLE_TEST_WRITE_TABLE_ID
 
 interface Ctx {
   baseId?: string
@@ -53,24 +56,30 @@ suite('airtable read handlers (live)', () => {
       return build(integration) as (input: any) => Promise<any>
     }
 
-    // Discover base -> table -> record for tests
-    const list_bases = buildHandler('list_bases')
-    const bases = await list_bases({})
-    ctx.baseId = bases?.bases?.[0]?.id || bases?.[0]?.id
-
-    if (ctx.baseId) {
-      const list_tables = buildHandler('list_tables')
-      const tablesResp = await list_tables({ baseId: ctx.baseId })
-      const tables = tablesResp?.tables || tablesResp
-      ctx.tableId = tables?.[0]?.id
-
-      if (ctx.tableId) {
-        const list_records = buildHandler('list_records')
-        const recs = await list_records({ baseId: ctx.baseId, tableId: ctx.tableId, pageSize: 1 })
-        const records = recs?.records || recs
-        ctx.recordId = records?.[0]?.id
+    // Use explicit test base/table if provided, otherwise auto-discover from first base
+    if (env.AIRTABLE_TEST_WRITE_BASE_ID && env.AIRTABLE_TEST_WRITE_TABLE_ID) {
+      ctx.baseId = env.AIRTABLE_TEST_WRITE_BASE_ID
+      ctx.tableId = env.AIRTABLE_TEST_WRITE_TABLE_ID
+    }
+    else {
+      const list_bases = buildHandler('list_bases')
+      const bases = await list_bases({})
+      ctx.baseId = bases?.bases?.[0]?.id || bases?.[0]?.id
+
+      if (ctx.baseId) {
+        const list_tables = buildHandler('list_tables')
+        const tablesResp = await list_tables({ baseId: ctx.baseId })
+        const tables = tablesResp?.tables || tablesResp
+        ctx.tableId = tables?.[0]?.id
       }
     }
+
+    if (ctx.baseId && ctx.tableId) {
+      const list_records = buildHandler('list_records')
+      const recs = await list_records({ baseId: ctx.baseId, tableId: ctx.tableId, pageSize: 1 })
+      const records = recs?.records || recs
+      ctx.recordId = records?.[0]?.id
+    }
   }, 60000)
 
   it('list_bases returns bases', async () => {

package/integrations/github/__tests__/write_handlers.test.ts

@@ -66,11 +66,8 @@ suiteOrSkip('github write handlers (live)', () => {
 
       it('create_repo -> delete_repo lifecycle (classic_pat only)', async () => {
         if (!toolbox.hasTool('write', 'create_repo')) {
-          // This tool is not available for fine_grained_pat -- expected.
           return expect(true).toBe(true)
         }
-        if (!ctx.owner)
-          return expect(true).toBe(true)
 
         const repoName = `cmdtest-repo-${Date.now()}`
 
@@ -82,10 +79,17 @@ suiteOrSkip('github write handlers (live)', () => {
           auto_init: true,
         })
         expect(created?.name).toBe(repoName)
-        expect(created?.full_name).toBe(`${ctx.owner}/${repoName}`)
+
+        // /user/repos creates under the authenticated user, not necessarily ctx.owner
+        const createdOwner = created?.owner?.login
+        expect(createdOwner).toBeTruthy()
+        expect(created?.full_name).toBe(`${createdOwner}/${repoName}`)
+
+        // GitHub needs a moment to finish provisioning the repo before it can be deleted
+        await new Promise(resolve => setTimeout(resolve, 3000))
 
         const delete_repo = toolbox.write('delete_repo')
-        const deleted = await delete_repo({ owner: ctx.owner, repo: repoName })
+        const deleted = await delete_repo({ owner: createdOwner, repo: repoName })
         expect(deleted?.success).toBe(true)
         expect(deleted?.status).toBe(204)
       }, 90000)

package/integrations/github/handlers/create_commit.js

@@ -11,32 +11,17 @@ async (input) => {
   const commitData = await commitRes.json()
   const currentTreeSha = commitData.tree.sha
   
-  // 3. Create blobs for each file with content
+  // 3. Build tree entries — use inline content for creates/updates, sha null for deletions
   const tree = []
   for (const file of files) {
     if (file.content !== undefined && file.content !== null) {
-      // Create a blob for this file
-      const contentBase64 = typeof Buffer !== 'undefined'
-        ? Buffer.from(file.content).toString('base64')
-        : btoa(unescape(encodeURIComponent(file.content)))
-      
-      const blobRes = await integration.fetch(`/repos/${owner}/${repo}/git/blobs`, {
-        method: 'POST',
-        body: {
-          content: contentBase64,
-          encoding: 'base64',
-        },
-      })
-      const blobData = await blobRes.json()
-      
       tree.push({
         path: file.path,
         mode: file.mode || '100644',
         type: 'blob',
-        sha: blobData.sha,
+        content: file.content,
       })
     } else {
-      // File deletion (null sha means delete)
       tree.push({
         path: file.path,
         mode: '100644',

package/integrations/google-docs/__tests__/get_handlers.test.ts

@@ -16,7 +16,7 @@ interface VariantConfig {
 const variants: VariantConfig[] = [
   {
     key: 'service_account',
-    credentials: () => ({ serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '' }),
+    credentials: () => ({ serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '', subject: env.GOOGLE_IMPERSONATE_SUBJECT || '' }),
   },
   {
     key: 'oauth_token',

package/integrations/google-docs/__tests__/write_handlers.test.ts

@@ -16,7 +16,7 @@ interface VariantConfig {
 const variants: VariantConfig[] = [
   {
     key: 'service_account',
-    credentials: () => ({ serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '' }),
+    credentials: () => ({ serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '', subject: env.GOOGLE_IMPERSONATE_SUBJECT || '' }),
   },
   {
     key: 'oauth_token',
@@ -49,7 +49,9 @@ suiteOrSkip('google-docs write handlers (live)', () => {
           variant.key,
         )
 
-        const folder = await drive.write('create_folder')({ name: `CmdTest Docs Write ${Date.now()}` })
+        const folder = await drive.write('create_folder')({
+          name: `CmdTest Docs Write ${Date.now()}`,
+        })
         ctx.folderId = folder?.id
         expect(ctx.folderId).toBeTruthy()
 
@@ -172,9 +174,10 @@ suiteOrSkip('google-docs write handlers (live)', () => {
       }, 60000)
 
       it('insert_inline_image_after_first_match inserts an image when allowed', async () => {
-        if (!ctx.documentId || !process.env.GDOCS_TEST_IMAGE_URI)
+        if (!ctx.documentId)
           return expect(true).toBe(true)
         const documentId = ctx.documentId
+        const imageUri = 'https://www.google.com/images/branding/googlelogo/2x/googlelogo_color_272x92dp.png'
         const insert_inline_image_after_first_match = docs.write('insert_inline_image_after_first_match')
         const anchor = `ANCHOR_${Date.now()}`
         const appended = docs.write('append_text')
@@ -182,7 +185,7 @@ suiteOrSkip('google-docs write handlers (live)', () => {
         const before = await get_text({ documentId })
         if (!String(before?.text || '').includes(anchor))
           await appended({ documentId, text: `\n${anchor}\n` })
-        const res = await insert_inline_image_after_first_match({ documentId, findText: anchor, uri: process.env.GDOCS_TEST_IMAGE_URI!, altText: 'CmdTest' })
+        const res = await insert_inline_image_after_first_match({ documentId, findText: anchor, uri: imageUri })
         expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
       }, 60000)
 

package/integrations/google-docs/handlers/insert_inline_image_after_first_match.js

@@ -34,7 +34,7 @@ async (input) => {
     return { ok: true }
 
   const requests = []
-  requests.push({ insertInlineImage: { location: { index: baseIndex }, uri, altTextTitle: altText } })
+  requests.push({ insertInlineImage: { location: { index: baseIndex }, uri } })
   requests.push({ replaceAllText: { containsText: { text: marker, matchCase: true }, replaceText: findText } })
   const res = await integration.fetch(`/documents/${encodeURIComponent(documentId)}:batchUpdate`, { method: 'POST', body: { requests } })
   return await res.json()

package/integrations/google-docs/schemas/insert_inline_image_after_first_match.json

@@ -5,7 +5,6 @@
     "documentId": { "type": "string" },
     "findText": { "type": "string" },
     "uri": { "type": "string" },
-    "altText": { "type": "string" },
     "position": { "type": "string", "enum": ["after", "before"], "default": "after" }
   },
   "required": ["documentId", "findText", "uri"],

package/integrations/google-drive/__tests__/handlers.test.ts

@@ -0,0 +1,102 @@
+import { afterAll, beforeAll, describe, expect, it } from 'vitest'
+import { createCredentialStore, createIntegrationNode, createProxy, createToolbox, safeCleanup } from '../../__tests__/liveHarness.js'
+
+// LIVE Google Drive tests -- runs once per available credential variant.
+// Required env vars (at least one):
+// - GOOGLE_SERVICE_ACCOUNT_JSON  (service_account variant)
+// - GOOGLE_TOKEN                 (oauth_token variant)
+
+const env = process.env as Record<string, string | undefined>
+
+interface VariantConfig {
+  key: string
+  credentials: () => Record<string, string>
+}
+
+const variants: VariantConfig[] = [
+  {
+    key: 'service_account',
+    credentials: () => ({ serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '', subject: env.GOOGLE_IMPERSONATE_SUBJECT || '' }),
+  },
+  {
+    key: 'oauth_token',
+    credentials: () => ({ token: env.GOOGLE_TOKEN || '' }),
+  },
+].filter(v => Object.values(v.credentials()).some(val => val.trim().length > 0))
+
+const suiteOrSkip = variants.length > 0 ? describe : describe.skip
+
+suiteOrSkip('google-drive handlers (live)', () => {
+  for (const variant of variants) {
+    describe(`variant: ${variant.key}`, () => {
+      const ctx: { folderId?: string, fileId?: string, destFolderId?: string } = {}
+      let drive: ReturnType<typeof createToolbox>
+
+      beforeAll(async () => {
+        const credentialStore = createCredentialStore(async () => variant.credentials())
+        const proxy = createProxy(credentialStore)
+        drive = createToolbox(
+          'google-drive',
+          proxy,
+          createIntegrationNode('google-drive', { label: 'Google Drive', credentialId: 'google-drive-creds', credentialVariant: variant.key }),
+          variant.key,
+        )
+
+        const folder = await drive.write('create_folder')({ name: `CmdTest Drive ${Date.now()}` })
+        ctx.folderId = folder?.id
+        expect(ctx.folderId).toBeTruthy()
+
+        const destFolder = await drive.write('create_folder')({ name: `CmdTest Drive Dest ${Date.now()}` })
+        ctx.destFolderId = destFolder?.id
+        expect(ctx.destFolderId).toBeTruthy()
+
+        const file = await drive.write('create_file')({
+          name: `CmdTest File ${Date.now()}`,
+          mimeType: 'application/vnd.google-apps.document',
+          parentId: ctx.folderId,
+        })
+        ctx.fileId = file?.id
+        expect(ctx.fileId).toBeTruthy()
+      }, 60000)
+
+      afterAll(async () => {
+        await safeCleanup(async () => ctx.fileId ? drive.write('delete_file')({ fileId: ctx.fileId }) : Promise.resolve())
+        await safeCleanup(async () => ctx.folderId ? drive.write('delete_file')({ fileId: ctx.folderId }) : Promise.resolve())
+        await safeCleanup(async () => ctx.destFolderId ? drive.write('delete_file')({ fileId: ctx.destFolderId }) : Promise.resolve())
+      }, 60000)
+
+      it('get_file returns file metadata', async () => {
+        if (!ctx.fileId)
+          return expect(true).toBe(true)
+        const result = await drive.read('get_file')({ fileId: ctx.fileId })
+        expect(result?.id).toBe(ctx.fileId)
+        expect(typeof result?.name).toBe('string')
+        expect(typeof result?.mimeType).toBe('string')
+      }, 30000)
+
+      it('move_file moves the file to a different folder', async () => {
+        if (!ctx.fileId || !ctx.destFolderId || !ctx.folderId)
+          return expect(true).toBe(true)
+        const result = await drive.write('move_file')({
+          fileId: ctx.fileId,
+          addParents: ctx.destFolderId,
+          removeParents: ctx.folderId,
+        })
+        expect(result?.id).toBe(ctx.fileId)
+        const meta = await drive.read('get_file')({ fileId: ctx.fileId })
+        expect(meta?.parents).toContain(ctx.destFolderId)
+      }, 30000)
+
+      it('delete_file deletes a file permanently', async () => {
+        const tempFile = await drive.write('create_file')({
+          name: `CmdTest Temp ${Date.now()}`,
+          mimeType: 'application/vnd.google-apps.document',
+        })
+        const tempId = tempFile?.id
+        expect(tempId).toBeTruthy()
+        await drive.write('delete_file')({ fileId: tempId })
+        await expect(drive.read('get_file')({ fileId: tempId })).rejects.toThrow()
+      }, 30000)
+    })
+  }
+})

package/integrations/google-sheet/__tests__/get_handlers.test.ts

@@ -21,6 +21,7 @@ suite('google-sheet read handlers (live)', () => {
     const credentialStore = createCredentialStore(async () => ({
       token: env.GOOGLE_TOKEN || '',
       serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '',
+      subject: env.GOOGLE_IMPERSONATE_SUBJECT || '',
     }))
     const proxy = createProxy(credentialStore)
     sheets = createToolbox('google-sheet', proxy, createIntegrationNode('google-sheet', { label: 'Google Sheets', credentialId: 'google-sheet-creds' }))

package/integrations/google-sheet/__tests__/write_handlers.test.ts

@@ -27,6 +27,7 @@ suite('google-sheet write handlers (live)', () => {
     const credentialStore = createCredentialStore(async () => ({
       token: env.GOOGLE_TOKEN || '',
       serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '',
+      subject: env.GOOGLE_IMPERSONATE_SUBJECT || '',
     }))
     const proxy = createProxy(credentialStore)
     sheets = createToolbox('google-sheet', proxy, createIntegrationNode('google-sheet', { label: 'Google Sheets', credentialId: 'google-sheet-creds' }))

package/integrations/google-slides/__tests__/get_handlers.test.ts

@@ -20,6 +20,7 @@ suite('google-slides read handlers (live)', () => {
     const credentialStore = createCredentialStore(async () => ({
       token: env.GOOGLE_TOKEN || '',
       serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '',
+      subject: env.GOOGLE_IMPERSONATE_SUBJECT || '',
     }))
     const proxy = createProxy(credentialStore)
     slides = createToolbox('google-slides', proxy, createIntegrationNode('google-slides', { label: 'Google Slides', credentialId: 'google-slides-creds' }))

package/integrations/google-slides/__tests__/write_handlers.test.ts

@@ -21,6 +21,7 @@ suite('google-slides write handlers (live)', () => {
     const credentialStore = createCredentialStore(async () => ({
       token: env.GOOGLE_TOKEN || '',
       serviceAccountJson: env.GOOGLE_SERVICE_ACCOUNT_JSON || '',
+      subject: env.GOOGLE_IMPERSONATE_SUBJECT || '',
     }))
     const proxy = createProxy(credentialStore)
     slides = createToolbox('google-slides', proxy, createIntegrationNode('google-slides', { label: 'Google Slides', credentialId: 'google-slides-creds' }))
@@ -95,13 +96,11 @@ suite('google-slides write handlers (live)', () => {
   }, 60000)
 
   it('insert_image_after_first_match inserts an image when allowed', async () => {
-    const env = process.env as Record<string, string | undefined>
-    if (!ctx.presentationId || !env.GSLIDES_TEST_IMAGE_URI)
-      return expect(true).toBe(true)
-    if (!ctx.anchorText)
+    if (!ctx.presentationId || !ctx.anchorText)
       return expect(true).toBe(true)
+    const imageUri = 'https://www.google.com/images/branding/googlelogo/2x/googlelogo_color_272x92dp.png'
     const handler = slides.write('insert_image_after_first_match')
-    const res = await handler({ presentationId: ctx.presentationId, findText: ctx.anchorText, uri: env.GSLIDES_TEST_IMAGE_URI })
+    const res = await handler({ presentationId: ctx.presentationId, findText: ctx.anchorText, uri: imageUri })
     expect(res?.presentationId || Array.isArray(res?.replies)).toBeTruthy()
   }, 60000)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@commandable/integration-data",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "Integration data (manifests, schemas, handlers, credential configs) for Commandable integrations.",
   "license": "AGPL-3.0-only",
   "repository": {