@commandable/integration-data 0.0.1 → 0.0.5

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

@@ -1,249 +1,255 @@
-import { beforeAll, describe, expect, it } from 'vitest'
-import { IntegrationProxy } from '../../../src/integrations/proxy.js'
-import { loadIntegrationTools } from '../../../src/integrations/dataLoader.js'
-
-// LIVE Google Docs write tests using managed OAuth
-// Required env vars:
-// - COMMANDABLE_MANAGED_OAUTH_BASE_URL
-// - COMMANDABLE_MANAGED_OAUTH_SECRET_KEY
-// - GDOCS_TEST_CONNECTION_ID (managed OAuth connection for provider 'google-docs')
-// - GDOCS_TEST_DOCUMENT_ID (target document ID with write access) OR GDOCS_ALLOW_CREATE to create
-
-interface Ctx {
-  documentId?: string
+import { afterAll, beforeAll, describe, expect, it } from 'vitest'
+import { createCredentialStore, createIntegrationNode, createProxy, createToolbox, safeCleanup } from '../../__tests__/liveHarness.js'
+
+// LIVE Google Docs write 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 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',
-  'GDOCS_TEST_CONNECTION_ID',
-)
-  ? describe
-  : describe.skip
-
-suite('google-docs write handlers (live)', () => {
-  const ctx: Ctx = {}
-  let buildWriteHandler: (name: string) => ((input: any) => Promise<any>)
-  let buildReadHandler: (name: string) => ((input: any) => Promise<any>)
-
-  beforeAll(async () => {
-    const { COMMANDABLE_MANAGED_OAUTH_BASE_URL, COMMANDABLE_MANAGED_OAUTH_SECRET_KEY, GDOCS_TEST_CONNECTION_ID, GDOCS_TEST_DOCUMENT_ID } = env
-
-    const proxy = new IntegrationProxy({
-      managedOAuthBaseUrl: COMMANDABLE_MANAGED_OAUTH_BASE_URL,
-      managedOAuthSecretKey: COMMANDABLE_MANAGED_OAUTH_SECRET_KEY,
+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-docs write handlers (live)', () => {
+  for (const variant of variants) {
+    describe(`variant: ${variant.key}`, () => {
+      const ctx: { documentId?: string, folderId?: string } = {}
+      let docs: ReturnType<typeof createToolbox>
+      let drive: ReturnType<typeof createToolbox>
+
+      beforeAll(async () => {
+        const credentialStore = createCredentialStore(async () => variant.credentials())
+        const proxy = createProxy(credentialStore)
+        docs = createToolbox(
+          'google-docs',
+          proxy,
+          createIntegrationNode('google-docs', { label: 'Google Docs', credentialId: 'google-docs-creds', credentialVariant: variant.key }),
+          variant.key,
+        )
+        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 Docs Write ${Date.now()}`,
+        })
+        ctx.folderId = folder?.id
+        expect(ctx.folderId).toBeTruthy()
+
+        const doc = await drive.write('create_file')({
+          name: `CmdTest Doc ${Date.now()}`,
+          mimeType: 'application/vnd.google-apps.document',
+          parentId: ctx.folderId,
+        })
+        ctx.documentId = doc?.id
+        expect(ctx.documentId).toBeTruthy()
+      }, 60000)
+
+      afterAll(async () => {
+        await safeCleanup(async () => ctx.documentId ? drive.write('delete_file')({ fileId: ctx.documentId }) : Promise.resolve())
+        await safeCleanup(async () => ctx.folderId ? drive.write('delete_file')({ fileId: ctx.folderId }) : Promise.resolve())
+      }, 60000)
+
+      it('batch_update can perform a trivial replaceAllText no-op', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const batch_update = docs.write('batch_update')
+        const res = await batch_update({ documentId, requests: [
+          { replaceAllText: { containsText: { text: '___unlikely___', matchCase: true }, replaceText: '___unlikely___' } },
+        ] })
+        expect(Array.isArray(res?.replies) || res?.documentId).toBeTruthy()
+      }, 60000)
+
+      it('append_text appends content', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const append_text = docs.write('append_text')
+        const marker = `CmdTest ${Date.now()}`
+        const res = await append_text({ documentId, text: marker })
+        expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
+        const get_text = docs.read('get_document_text')
+        const after = await get_text({ documentId })
+        expect(String(after?.text || '')).toContain(marker)
+      }, 60000)
+
+      it('insert_text_after_first_match inserts text near target', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const insert_text_after_first_match = docs.write('insert_text_after_first_match')
+        const get_text = docs.read('get_document_text')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_text({ documentId })
+        if (!String(before?.text || '').includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const insertSnippet = ` CmdTest ${Date.now()} `
+        const res = await insert_text_after_first_match({ documentId, findText: anchor, insertText: insertSnippet, position: 'after' })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+        const after = await get_text({ documentId })
+        const text = String(after?.text || '')
+        expect(text).toContain(anchor)
+        expect(text).toContain(insertSnippet)
+      }, 60000)
+
+      it('replace_all_text replaces occurrences', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const replace_all_text = docs.write('replace_all_text')
+        const res = await replace_all_text({ documentId, findText: '___unlikely___', replaceText: '___unlikely___', matchCase: true })
+        expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
+      }, 60000)
+
+      it('style_first_match applies style to first match', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const style_first_match = docs.write('style_first_match')
+        const get_struct = docs.read('get_document_structured')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_struct({ documentId })
+        if (!JSON.stringify(before?.body || {}).includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const res = await style_first_match({ documentId, findText: anchor, textStyle: { bold: true } })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+      }, 60000)
+
+      it('insert_table_after_first_match inserts a table near target', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const insert_table_after_first_match = docs.write('insert_table_after_first_match')
+        const get_struct = docs.read('get_document_structured')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_struct({ documentId })
+        if (!JSON.stringify(before?.body || {}).includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const res = await insert_table_after_first_match({ documentId, findText: anchor, rows: 1, columns: 1 })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+        const after = await get_struct({ documentId })
+        const hasTable = (after?.body?.content || []).some((el: any) => Boolean(el.table))
+        expect(hasTable).toBe(true)
+      }, 60000)
+
+      it('insert_page_break_after_first_match inserts a break near target', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const insert_page_break_after_first_match = docs.write('insert_page_break_after_first_match')
+        const get_struct = docs.read('get_document_structured')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_struct({ documentId })
+        if (!JSON.stringify(before?.body || {}).includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const res = await insert_page_break_after_first_match({ documentId, findText: anchor })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+        const after = await get_struct({ documentId })
+        const hasBreak = (after?.body?.content || []).some((el: any) => Boolean(el.sectionBreak))
+        expect(hasBreak).toBe(true)
+      }, 60000)
+
+      it('insert_inline_image_after_first_match inserts an image when allowed', async () => {
+        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')
+        const get_text = docs.read('get_document_text')
+        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: imageUri })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+      }, 60000)
+
+      it('delete_first_match deletes a small span (no-op ok)', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const delete_first_match = docs.write('delete_first_match')
+        const get_text = docs.read('get_document_text')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_text({ documentId })
+        if (!String(before?.text || '').includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const res = await delete_first_match({ documentId, findText: anchor })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+        const after = await get_text({ documentId })
+        expect(String(after?.text || '')).not.toContain(anchor)
+      }, 60000)
+
+      it('update_paragraph_style_for_first_match updates paragraph style near target', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const update_paragraph_style_for_first_match = docs.write('update_paragraph_style_for_first_match')
+        const get_struct = docs.read('get_document_structured')
+        const anchor = `ANCHOR_${Date.now()}`
+        const appended = docs.write('append_text')
+        const before = await get_struct({ documentId })
+        if (!JSON.stringify(before?.body || {}).includes(anchor))
+          await appended({ documentId, text: `\n${anchor}\n` })
+        const res = await update_paragraph_style_for_first_match({ documentId, findText: anchor, paragraphStyle: { alignment: 'CENTER' } })
+        expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
+        const after = await get_struct({ documentId })
+        let foundAligned = false
+        for (const el of (after?.body?.content || [])) {
+          if (!el.paragraph)
+            continue
+          const p = el.paragraph
+          const text = (p.elements || []).map((e: any) => e?.textRun?.content || '').join('')
+          if (text.includes(anchor)) {
+            if (p.paragraphStyle?.alignment === 'CENTER')
+              foundAligned = true
+            break
+          }
+        }
+        expect(foundAligned).toBe(true)
+      }, 60000)
+
+      it('update_document_style updates doc style with no-op', async () => {
+        const documentId = ctx.documentId
+        if (!documentId)
+          return expect(true).toBe(true)
+        const update_document_style = docs.write('update_document_style')
+        const res = await update_document_style({ documentId, documentStyle: { useFirstPageHeaderFooter: false } })
+        expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
+      }, 60000)
+
+      it('create_document creates a document (self-cleaning)', async () => {
+        const created = await docs.write('create_document')({ title: `CmdTest Doc Tool ${Date.now()}` })
+        const id = created?.documentId
+        expect(typeof id).toBe('string')
+        await safeCleanup(async () => id ? drive.write('delete_file')({ fileId: id }) : Promise.resolve())
+      }, 60000)
     })
-    const integrationNode = { id: 'node-gdocs', type: 'google-docs', label: 'Google Docs', connectionId: GDOCS_TEST_CONNECTION_ID } as any
-
-    const tools = loadIntegrationTools('google-docs')
-    expect(tools).toBeTruthy()
-
-    buildWriteHandler = (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>
-    }
-
-    buildReadHandler = (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>
-    }
-
-    ctx.documentId = GDOCS_TEST_DOCUMENT_ID
-  }, 60000)
-
-  it('create_document creates a document when allowed', async () => {
-    if (!process.env.GDOCS_ALLOW_CREATE)
-      return expect(true).toBe(true)
-    const create_document = buildWriteHandler('create_document')
-    const res = await create_document({ title: `CmdTest Doc ${Date.now()}` })
-    expect(res?.documentId).toBeTruthy()
-  }, 60000)
-
-  it('batch_update can perform a trivial replaceAllText no-op', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const batch_update = buildWriteHandler('batch_update')
-    const res = await batch_update({ documentId, requests: [
-      { replaceAllText: { containsText: { text: '___unlikely___', matchCase: true }, replaceText: '___unlikely___' } },
-    ] })
-    expect(Array.isArray(res?.replies) || res?.documentId).toBeTruthy()
-  }, 60000)
-
-  it('append_text appends content', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const append_text = buildWriteHandler('append_text')
-    const marker = `CmdTest ${Date.now()}`
-    const res = await append_text({ documentId, text: marker })
-    expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
-    const get_text = buildReadHandler('get_document_text')
-    const after = await get_text({ documentId })
-    expect(String(after?.text || '')).toContain(marker)
-  }, 60000)
-
-  it('insert_text_after_first_match inserts text near target', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const insert_text_after_first_match = buildWriteHandler('insert_text_after_first_match')
-    const get_text = buildReadHandler('get_document_text')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_text({ documentId })
-    if (!String(before?.text || '').includes(anchor)) {
-      await appended({ documentId, text: `\n${anchor}\n` })
-    }
-    const insertSnippet = ` CmdTest ${Date.now()} `
-    const res = await insert_text_after_first_match({ documentId, findText: anchor, insertText: insertSnippet, position: 'after' })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-    const after = await get_text({ documentId })
-    const text = String(after?.text || '')
-    expect(text).toContain(anchor)
-    expect(text).toContain(insertSnippet)
-  }, 60000)
-
-  it('replace_all_text replaces occurrences', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const replace_all_text = buildWriteHandler('replace_all_text')
-    const res = await replace_all_text({ documentId, findText: '___unlikely___', replaceText: '___unlikely___', matchCase: true })
-    expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
-  }, 60000)
-
-  it('style_first_match applies style to first match', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const style_first_match = buildWriteHandler('style_first_match')
-    const get_struct = buildReadHandler('get_document_structured')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_struct({ documentId })
-    const hasAnchorBefore = JSON.stringify(before?.body || {}).includes(anchor)
-    if (!hasAnchorBefore)
-      await appended({ documentId, text: `\n${anchor}\n` })
-    const res = await style_first_match({ documentId, findText: anchor, textStyle: { bold: true } })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-  }, 60000)
-
-  it('insert_table_after_first_match inserts a table near target', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const insert_table_after_first_match = buildWriteHandler('insert_table_after_first_match')
-    const get_struct = buildReadHandler('get_document_structured')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_struct({ documentId })
-    const hasAnchorBefore = JSON.stringify(before?.body || {}).includes(anchor)
-    if (!hasAnchorBefore)
-      await appended({ documentId, text: `\n${anchor}\n` })
-    const res = await insert_table_after_first_match({ documentId, findText: anchor, rows: 1, columns: 1 })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-    const after = await get_struct({ documentId })
-    const hasTable = (after?.body?.content || []).some((el: any) => Boolean(el.table))
-    expect(hasTable).toBe(true)
-  }, 60000)
-
-  it('insert_page_break_after_first_match inserts a break near target', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const insert_page_break_after_first_match = buildWriteHandler('insert_page_break_after_first_match')
-    const get_struct = buildReadHandler('get_document_structured')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_struct({ documentId })
-    const hasAnchorBefore = JSON.stringify(before?.body || {}).includes(anchor)
-    if (!hasAnchorBefore)
-      await appended({ documentId, text: `\n${anchor}\n` })
-    const res = await insert_page_break_after_first_match({ documentId, findText: anchor })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-    const after = await get_struct({ documentId })
-    const hasBreak = (after?.body?.content || []).some((el: any) => Boolean(el.sectionBreak))
-    expect(hasBreak).toBe(true)
-  }, 60000)
-
-  it('insert_inline_image_after_first_match inserts an image when allowed', async () => {
-    if (!process.env.GDOCS_TEST_DOCUMENT_ID || !process.env.GDOCS_TEST_IMAGE_URI)
-      return expect(true).toBe(true)
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    const insert_inline_image_after_first_match = buildWriteHandler('insert_inline_image_after_first_match')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const get_text = buildReadHandler('get_document_text')
-    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' })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-  }, 60000)
-
-  it('delete_first_match deletes a small span (no-op ok)', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const delete_first_match = buildWriteHandler('delete_first_match')
-    const get_text = buildReadHandler('get_document_text')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_text({ documentId })
-    if (!String(before?.text || '').includes(anchor))
-      await appended({ documentId, text: `\n${anchor}\n` })
-    const res = await delete_first_match({ documentId, findText: anchor })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-    const after = await get_text({ documentId })
-    expect(String(after?.text || '')).not.toContain(anchor)
-  }, 60000)
-
-  it('update_paragraph_style_for_first_match updates paragraph style near target', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const update_paragraph_style_for_first_match = buildWriteHandler('update_paragraph_style_for_first_match')
-    const get_struct = buildReadHandler('get_document_structured')
-    const anchor = `ANCHOR_${Date.now()}`
-    const appended = buildWriteHandler('append_text')
-    const before = await get_struct({ documentId })
-    const hasAnchorBefore = JSON.stringify(before?.body || {}).includes(anchor)
-    if (!hasAnchorBefore)
-      await appended({ documentId, text: `\n${anchor}\n` })
-    const res = await update_paragraph_style_for_first_match({ documentId, findText: anchor, paragraphStyle: { alignment: 'CENTER' } })
-    expect(res?.applied === true || Array.isArray(res?.replies)).toBeTruthy()
-    const after = await get_struct({ documentId })
-    // find the paragraph containing anchor and verify alignment
-    let foundAligned = false
-    for (const el of (after?.body?.content || [])) {
-      if (!el.paragraph)
-        continue
-      const p = el.paragraph
-      const text = (p.elements || []).map((e: any) => e?.textRun?.content || '').join('')
-      if (text.includes(anchor)) {
-        if (p.paragraphStyle?.alignment === 'CENTER')
-          foundAligned = true
-        break
-      }
-    }
-    expect(foundAligned).toBe(true)
-  }, 60000)
-
-  it('update_document_style updates doc style with no-op', async () => {
-    const documentId = process.env.GDOCS_TEST_DOCUMENT_ID
-    if (!documentId)
-      return expect(true).toBe(true)
-    const update_document_style = buildWriteHandler('update_document_style')
-    const res = await update_document_style({ documentId, documentStyle: { useFirstPageHeaderFooter: false } })
-    expect(res?.documentId || Array.isArray(res?.replies)).toBeTruthy()
-  }, 60000)
+  }
 })

package/integrations/google-docs/credentials.json

@@ -1,36 +1,57 @@
 {
-  "schema": {
-    "type": "object",
-    "properties": {
-      "token": {
-        "type": "string",
-        "title": "OAuth Access Token (optional)",
-        "description": "Google OAuth access token to use as a Bearer token (typically short-lived)."
+  "variants": {
+    "service_account": {
+      "label": "Service Account (recommended)",
+      "schema": {
+        "type": "object",
+        "properties": {
+          "serviceAccountJson": {
+            "type": "string",
+            "title": "Service Account JSON",
+            "description": "Full service account key JSON (contents of the downloaded JSON file from Google Cloud)."
+          },
+          "subject": {
+            "type": "string",
+            "title": "Subject / impersonated user (optional)",
+            "description": "Optional user email to impersonate when using Google Workspace domain-wide delegation."
+          },
+          "scopes": {
+            "type": "array",
+            "title": "OAuth scopes (optional)",
+            "description": "Optional override for OAuth scopes. Defaults to documents + drive.",
+            "items": { "type": "string" }
+          }
+        },
+        "required": ["serviceAccountJson"],
+        "additionalProperties": false
       },
-      "serviceAccountJson": {
-        "type": "string",
-        "title": "Service Account JSON (recommended)",
-        "description": "Full service account key JSON (contents of the downloaded JSON file)."
+      "injection": {
+        "headers": {
+          "Authorization": "Bearer {{token}}"
+        }
       },
-      "subject": {
-        "type": "string",
-        "title": "Subject / impersonated user (optional)",
-        "description": "Optional user email to impersonate when using Google Workspace domain-wide delegation."
+      "preprocess": "google_service_account"
+    },
+    "oauth_token": {
+      "label": "OAuth Access Token (short-lived)",
+      "schema": {
+        "type": "object",
+        "properties": {
+          "token": {
+            "type": "string",
+            "title": "OAuth Access Token",
+            "description": "Short-lived Google OAuth access token with documents and drive scopes."
+          }
+        },
+        "required": ["token"],
+        "additionalProperties": false
       },
-      "scopes": {
-        "type": "array",
-        "title": "OAuth scopes (optional)",
-        "description": "Optional override for OAuth scopes.",
-        "items": { "type": "string" }
+      "injection": {
+        "headers": {
+          "Authorization": "Bearer {{token}}"
+        }
       }
-    },
-    "required": [],
-    "additionalProperties": false
-  },
-  "injection": {
-    "headers": {
-      "Authorization": "Bearer {{token}}"
     }
-  }
+  },
+  "default": "service_account"
 }
-

package/integrations/google-docs/credentials_hint_oauth_token.md

@@ -0,0 +1,8 @@
+Obtain a short-lived Google OAuth access token:
+
+1. Use the Google OAuth 2.0 Playground (`https://developers.google.com/oauthplayground/`) or your own OAuth flow
+2. Select the scopes: `https://www.googleapis.com/auth/documents` and `https://www.googleapis.com/auth/drive`
+3. Exchange the authorization code for an access token
+4. Paste the access token here
+
+Note: OAuth access tokens are short-lived (typically 1 hour). For long-running use, prefer the Service Account variant.

package/integrations/google-docs/credentials_hint_service_account.md

@@ -0,0 +1,10 @@
+Set up a Google Cloud Service Account:
+
+1. Open the [Google Cloud Console](https://console.cloud.google.com/)
+2. Enable the **Google Docs API** and **Google Drive API** for your project
+3. Go to **IAM & Admin → Service Accounts** and create a new service account
+4. Under **Keys**, click **Add Key → Create new key → JSON** and download the file
+5. Paste the full contents of the JSON file here
+6. Share your target Google Docs documents with the service account's `client_email`
+
+For Google Workspace users: if you need to access user documents, configure domain-wide delegation and set the `subject` field to the user's email.

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"],