gogcli-mcp-slides 2.0.2 → 2.0.6

package/manifest.json

@@ -3,7 +3,7 @@
   "manifest_version": "0.3",
   "name": "gogcli-mcp-slides",
   "display_name": "gogcli (Slides)",
-  "version": "2.0.2",
+  "version": "2.0.6",
   "description": "Extended Google Slides for Claude via gogcli — auth + full Slides support (create, edit, export, templates, markdown)",
   "author": {
     "name": "Chris Hall",

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "gogcli-mcp-slides",
-  "version": "2.0.2",
+  "version": "2.0.6",
+  "mcpName": "io.github.chrischall/gogcli-mcp-slides",
   "description": "Extended Google Slides MCP server via gogcli — auth + full Slides support",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
   "repository": {
@@ -24,7 +25,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
-    "zod": "^4.3.6"
+    "zod": "^4.4.3"
   },
   "license": "MIT",
   "keywords": [

package/server.json

@@ -0,0 +1,35 @@
+{
+  "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
+  "name": "io.github.chrischall/gogcli-mcp-slides",
+  "description": "Google Slides via gogcli for Claude \u2014 deck and slide authoring",
+  "repository": {
+    "url": "https://github.com/chrischall/gogcli-mcp",
+    "source": "github",
+    "subfolder": "packages/gogcli-mcp-slides"
+  },
+  "version": "2.0.3",
+  "packages": [
+    {
+      "registryType": "npm",
+      "identifier": "gogcli-mcp-slides",
+      "version": "2.0.3",
+      "transport": {
+        "type": "stdio"
+      },
+      "environmentVariables": [
+        {
+          "name": "GOG_ACCOUNT",
+          "description": "Email address of the Google account to use (matches gogcli auth)",
+          "isRequired": false,
+          "format": "string"
+        },
+        {
+          "name": "GOG_PATH",
+          "description": "Override path to the gogcli binary (auto-discovered otherwise)",
+          "isRequired": false,
+          "format": "string"
+        }
+      ]
+    }
+  ]
+}

package/src/tools/slides-extra.ts

@@ -1,5 +1,113 @@
 import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { z } from 'zod';
+import { accountParam, runOrDiagnose } from '../../../gogcli-mcp/src/lib.js';
 
-export function registerExtraSlidesTools(_server: McpServer): void {
-  /* no extras yet */
+export function registerExtraSlidesTools(server: McpServer): void {
+  server.registerTool('gog_slides_create_from_markdown', {
+    description: 'Create a new Google Slides presentation from markdown content (inline or from a file).',
+    inputSchema: {
+      title: z.string().describe('Presentation title'),
+      content: z.string().optional().describe('Inline markdown content'),
+      contentFile: z.string().optional().describe('Path to a markdown file'),
+      parent: z.string().optional().describe('Destination folder ID'),
+      debug: z.boolean().optional().describe('Enable debug output'),
+      account: accountParam,
+    },
+  }, async ({ title, content, contentFile, parent, debug, account }) => {
+    const args = ['slides', 'create-from-markdown', title];
+    if (content) args.push(`--content=${content}`);
+    if (contentFile) args.push(`--content-file=${contentFile}`);
+    if (parent) args.push(`--parent=${parent}`);
+    if (debug) args.push('--debug');
+    return runOrDiagnose(args, { account });
+  });
+
+  server.registerTool('gog_slides_create_from_template', {
+    description: 'Create a new Google Slides presentation from a template, with optional placeholder replacements.',
+    inputSchema: {
+      templateId: z.string().describe('Template presentation ID'),
+      title: z.string().describe('New presentation title'),
+      replacements: z.record(z.string(), z.string()).optional().describe('Placeholder replacements as a key/value object (emitted as --replace=k=v for each entry)'),
+      replacementsFile: z.string().optional().describe('Path to a JSON file containing replacements'),
+      parent: z.string().optional().describe('Destination folder ID'),
+      exact: z.boolean().optional().describe('Require exact placeholder matches'),
+      account: accountParam,
+    },
+  }, async ({ templateId, title, replacements, replacementsFile, parent, exact, account }) => {
+    const args = ['slides', 'create-from-template', templateId, title];
+    if (replacements) {
+      for (const [k, v] of Object.entries(replacements)) {
+        args.push(`--replace=${k}=${v}`);
+      }
+    }
+    if (replacementsFile) args.push(`--replacements=${replacementsFile}`);
+    if (parent) args.push(`--parent=${parent}`);
+    if (exact) args.push('--exact');
+    return runOrDiagnose(args, { account });
+  });
+
+  server.registerTool('gog_slides_add_slide', {
+    description: 'Add a new slide to a presentation from a local image, with optional speaker notes.',
+    inputSchema: {
+      presentationId: z.string().describe('Presentation ID'),
+      image: z.string().describe('Path to the local image file'),
+      notes: z.string().optional().describe('Speaker notes text'),
+      notesFile: z.string().optional().describe('Path to a file containing speaker notes'),
+      before: z.string().optional().describe('Insert before this slide ID (default: append at end)'),
+      account: accountParam,
+    },
+  }, async ({ presentationId, image, notes, notesFile, before, account }) => {
+    const args = ['slides', 'add-slide', presentationId, image];
+    if (notes) args.push(`--notes=${notes}`);
+    if (notesFile) args.push(`--notes-file=${notesFile}`);
+    if (before) args.push(`--before=${before}`);
+    return runOrDiagnose(args, { account });
+  });
+
+  server.registerTool('gog_slides_delete_slide', {
+    description: 'Delete a slide from a Google Slides presentation.',
+    annotations: { destructiveHint: true },
+    inputSchema: {
+      presentationId: z.string().describe('Presentation ID'),
+      slideId: z.string().describe('Slide ID to delete'),
+      account: accountParam,
+    },
+  }, async ({ presentationId, slideId, account }) => {
+    return runOrDiagnose(['slides', 'delete-slide', presentationId, slideId], { account });
+  });
+
+  server.registerTool('gog_slides_update_notes', {
+    description: 'Update the speaker notes on a slide (inline text or from a file).',
+    annotations: { destructiveHint: true },
+    inputSchema: {
+      presentationId: z.string().describe('Presentation ID'),
+      slideId: z.string().describe('Slide ID'),
+      notes: z.string().optional().describe('New speaker notes text'),
+      notesFile: z.string().optional().describe('Path to a file containing new speaker notes'),
+      account: accountParam,
+    },
+  }, async ({ presentationId, slideId, notes, notesFile, account }) => {
+    const args = ['slides', 'update-notes', presentationId, slideId];
+    if (notes) args.push(`--notes=${notes}`);
+    if (notesFile) args.push(`--notes-file=${notesFile}`);
+    return runOrDiagnose(args, { account });
+  });
+
+  server.registerTool('gog_slides_replace_slide', {
+    description: 'Replace the image content of an existing slide, with optional speaker notes.',
+    annotations: { destructiveHint: true },
+    inputSchema: {
+      presentationId: z.string().describe('Presentation ID'),
+      slideId: z.string().describe('Slide ID to replace'),
+      image: z.string().describe('Path to the new local image file'),
+      notes: z.string().optional().describe('Speaker notes text'),
+      notesFile: z.string().optional().describe('Path to a file containing speaker notes'),
+      account: accountParam,
+    },
+  }, async ({ presentationId, slideId, image, notes, notesFile, account }) => {
+    const args = ['slides', 'replace-slide', presentationId, slideId, image];
+    if (notes) args.push(`--notes=${notes}`);
+    if (notesFile) args.push(`--notes-file=${notesFile}`);
+    return runOrDiagnose(args, { account });
+  });
 }

package/tests/tools/slides-extra.test.ts

@@ -1,10 +1,220 @@
-import { describe, it, expect } from 'vitest';
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { registerExtraSlidesTools } from '../../src/tools/slides-extra.js';
+import * as lib from '../../../gogcli-mcp/src/lib.js';
+import { setupExtrasHandlers, toText, type ToolHandler } from '../../../gogcli-mcp/tests/helpers/extras-harness.js';
 
-describe('registerExtraSlidesTools', () => {
-  it('does not throw when called (no extras yet)', () => {
-    const server = new McpServer({ name: 'test', version: '0.0.0' });
-    expect(() => registerExtraSlidesTools(server)).not.toThrow();
+vi.mock('../../../gogcli-mcp/src/lib.js', async (importOriginal) => {
+  const actual = await importOriginal<typeof lib>();
+  return {
+    ...actual,
+    runOrDiagnose: vi.fn(),
+  };
+});
+
+let handlers: Map<string, ToolHandler>;
+
+beforeEach(() => {
+  vi.clearAllMocks();
+  vi.mocked(lib.runOrDiagnose).mockResolvedValue(toText('{}'));
+  handlers = setupExtrasHandlers(registerExtraSlidesTools);
+});
+
+describe('gog_slides_create_from_markdown', () => {
+  it('calls runOrDiagnose with title only', async () => {
+    await handlers.get('gog_slides_create_from_markdown')!({ title: 'Deck' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'create-from-markdown', 'Deck'],
+      { account: undefined },
+    );
+  });
+
+  it('passes all optional flags', async () => {
+    await handlers.get('gog_slides_create_from_markdown')!({
+      title: 'Deck',
+      content: '# Slide 1',
+      contentFile: '/tmp/deck.md',
+      parent: 'folder1',
+      debug: true,
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      [
+        'slides', 'create-from-markdown', 'Deck',
+        '--content=# Slide 1',
+        '--content-file=/tmp/deck.md',
+        '--parent=folder1',
+        '--debug',
+      ],
+      { account: undefined },
+    );
+  });
+
+  it('omits --debug when false', async () => {
+    await handlers.get('gog_slides_create_from_markdown')!({ title: 'Deck', debug: false });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'create-from-markdown', 'Deck'],
+      { account: undefined },
+    );
+  });
+});
+
+describe('gog_slides_create_from_template', () => {
+  it('calls runOrDiagnose with templateId and title only', async () => {
+    await handlers.get('gog_slides_create_from_template')!({ templateId: 'tpl1', title: 'Deck' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'create-from-template', 'tpl1', 'Deck'],
+      { account: undefined },
+    );
+  });
+
+  it('passes --replace for a single replacement entry', async () => {
+    await handlers.get('gog_slides_create_from_template')!({
+      templateId: 'tpl1',
+      title: 'Deck',
+      replacements: { name: 'Alice' },
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'create-from-template', 'tpl1', 'Deck', '--replace=name=Alice'],
+      { account: undefined },
+    );
+  });
+
+  it('passes --replace for each entry in replacements', async () => {
+    await handlers.get('gog_slides_create_from_template')!({
+      templateId: 'tpl1',
+      title: 'Deck',
+      replacements: { name: 'Alice', company: 'Acme' },
+    });
+    const call = vi.mocked(lib.runOrDiagnose).mock.calls[0]!;
+    expect(call[0]).toEqual(expect.arrayContaining(['--replace=name=Alice', '--replace=company=Acme']));
+    expect(call[1]).toEqual({ account: undefined });
+  });
+
+  it('passes --replacements, --parent, and --exact', async () => {
+    await handlers.get('gog_slides_create_from_template')!({
+      templateId: 'tpl1',
+      title: 'Deck',
+      replacementsFile: '/tmp/r.json',
+      parent: 'folder1',
+      exact: true,
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      [
+        'slides', 'create-from-template', 'tpl1', 'Deck',
+        '--replacements=/tmp/r.json',
+        '--parent=folder1',
+        '--exact',
+      ],
+      { account: undefined },
+    );
+  });
+
+  it('omits --exact when false', async () => {
+    await handlers.get('gog_slides_create_from_template')!({
+      templateId: 'tpl1',
+      title: 'Deck',
+      exact: false,
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'create-from-template', 'tpl1', 'Deck'],
+      { account: undefined },
+    );
+  });
+});
+
+describe('gog_slides_add_slide', () => {
+  it('calls runOrDiagnose with presentationId and image', async () => {
+    await handlers.get('gog_slides_add_slide')!({ presentationId: 'p1', image: '/tmp/img.png' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'add-slide', 'p1', '/tmp/img.png'],
+      { account: undefined },
+    );
+  });
+
+  it('passes --notes, --notes-file, and --before', async () => {
+    await handlers.get('gog_slides_add_slide')!({
+      presentationId: 'p1',
+      image: '/tmp/img.png',
+      notes: 'Speaker note',
+      notesFile: '/tmp/notes.txt',
+      before: 'slide5',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      [
+        'slides', 'add-slide', 'p1', '/tmp/img.png',
+        '--notes=Speaker note',
+        '--notes-file=/tmp/notes.txt',
+        '--before=slide5',
+      ],
+      { account: undefined },
+    );
+  });
+});
+
+describe('gog_slides_delete_slide', () => {
+  it('calls runOrDiagnose with presentationId and slideId', async () => {
+    await handlers.get('gog_slides_delete_slide')!({ presentationId: 'p1', slideId: 's1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'delete-slide', 'p1', 's1'],
+      { account: undefined },
+    );
+  });
+});
+
+describe('gog_slides_update_notes', () => {
+  it('calls runOrDiagnose with presentationId and slideId', async () => {
+    await handlers.get('gog_slides_update_notes')!({ presentationId: 'p1', slideId: 's1' });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'update-notes', 'p1', 's1'],
+      { account: undefined },
+    );
+  });
+
+  it('passes --notes and --notes-file when provided', async () => {
+    await handlers.get('gog_slides_update_notes')!({
+      presentationId: 'p1',
+      slideId: 's1',
+      notes: 'speak clearly',
+      notesFile: '/tmp/n.txt',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      [
+        'slides', 'update-notes', 'p1', 's1',
+        '--notes=speak clearly',
+        '--notes-file=/tmp/n.txt',
+      ],
+      { account: undefined },
+    );
+  });
+});
+
+describe('gog_slides_replace_slide', () => {
+  it('calls runOrDiagnose with presentationId, slideId, and image', async () => {
+    await handlers.get('gog_slides_replace_slide')!({
+      presentationId: 'p1',
+      slideId: 's1',
+      image: '/tmp/img.png',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      ['slides', 'replace-slide', 'p1', 's1', '/tmp/img.png'],
+      { account: undefined },
+    );
+  });
+
+  it('passes --notes and --notes-file when provided', async () => {
+    await handlers.get('gog_slides_replace_slide')!({
+      presentationId: 'p1',
+      slideId: 's1',
+      image: '/tmp/img.png',
+      notes: 'updated',
+      notesFile: '/tmp/n.txt',
+    });
+    expect(lib.runOrDiagnose).toHaveBeenCalledWith(
+      [
+        'slides', 'replace-slide', 'p1', 's1', '/tmp/img.png',
+        '--notes=updated',
+        '--notes-file=/tmp/n.txt',
+      ],
+      { account: undefined },
+    );
   });
 });