gogcli-mcp-slides 2.0.4 → 2.0.7

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
   },
   "metadata": {
     "description": "Extended Google Slides for Claude via gogcli — auth + full Slides support (create, edit, export, templates, markdown)",
-    "version": "2.0.4"
+    "version": "2.0.7"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "gogcli (Slides)",
       "source": "./",
       "description": "Extended Google Slides for Claude via gogcli — auth + full Slides support (create, edit, export, templates, markdown)",
-      "version": "2.0.4",
+      "version": "2.0.7",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "gogcli-mcp-slides",
   "displayName": "gogcli (Slides)",
-  "version": "2.0.4",
+  "version": "2.0.7",
   "description": "Extended Google Slides for Claude via gogcli — auth + full Slides support (create, edit, export, templates, markdown)",
   "author": {
     "name": "Chris Hall",

package/dist/index.js

@@ -30864,6 +30864,11 @@ var EMPTY_COMPLETION_RESULT = {
 // ../gogcli-mcp/src/runner.ts
 import { spawn } from "node:child_process";
 var TIMEOUT_MS = 3e4;
+function envOrUndefined(key) {
+  const value = process.env[key];
+  if (!value || value.startsWith("${")) return void 0;
+  return value;
+}
 function formatTimeout(ms) {
   const seconds = Math.round(ms / 1e3);
   if (seconds >= 60) {
@@ -30874,7 +30879,7 @@ function formatTimeout(ms) {
 }
 async function run(args, options = {}) {
   const { account, spawner = spawn, interactive = false, timeout } = options;
-  const effectiveAccount = account ?? process.env.GOG_ACCOUNT;
+  const effectiveAccount = account ?? envOrUndefined("GOG_ACCOUNT");
   const fullArgs = ["--json", "--color=never"];
   if (!interactive) {
     fullArgs.push("--no-input");
@@ -30886,7 +30891,7 @@ async function run(args, options = {}) {
   const effectiveTimeout = timeout ?? TIMEOUT_MS;
   return new Promise((resolve, reject) => {
     const { GOG_ACCESS_TOKEN: _, ...cleanEnv } = process.env;
-    const child = spawner(process.env.GOG_PATH ?? "gog", fullArgs, { env: cleanEnv });
+    const child = spawner(envOrUndefined("GOG_PATH") ?? "gog", fullArgs, { env: cleanEnv });
     const stdoutChunks = [];
     const stderrChunks = [];
     let settled = false;
@@ -31068,6 +31073,64 @@ function registerSlidesTools(server2) {
     if (template) args.push(`--template=${template}`);
     return runOrDiagnose(args, { account });
   });
+  server2.registerTool("gog_slides_copy", {
+    description: "Copy a Google Slides presentation to a new presentation with the given title.",
+    inputSchema: {
+      presentationId: external_exports.string().describe("Presentation ID to copy"),
+      title: external_exports.string().describe("Title for the new copy"),
+      parent: external_exports.string().optional().describe("Destination folder ID"),
+      account: accountParam
+    }
+  }, async ({ presentationId, title, parent, account }) => {
+    const args = ["slides", "copy", presentationId, title];
+    if (parent) args.push(`--parent=${parent}`);
+    return runOrDiagnose(args, { account });
+  });
+  server2.registerTool("gog_slides_list_slides", {
+    description: "List slides in a Google Slides presentation.",
+    annotations: { readOnlyHint: true },
+    inputSchema: {
+      presentationId: external_exports.string().describe("Presentation ID"),
+      account: accountParam
+    }
+  }, async ({ presentationId, account }) => {
+    return runOrDiagnose(["slides", "list-slides", presentationId], { account });
+  });
+  server2.registerTool("gog_slides_read_slide", {
+    description: "Read the content of a slide (text, shapes, speaker notes).",
+    annotations: { readOnlyHint: true },
+    inputSchema: {
+      presentationId: external_exports.string().describe("Presentation ID"),
+      slideId: external_exports.string().describe("Slide ID to read"),
+      account: accountParam
+    }
+  }, async ({ presentationId, slideId, account }) => {
+    return runOrDiagnose(["slides", "read-slide", presentationId, slideId], { account });
+  });
+  server2.registerTool("gog_slides_run", {
+    description: "Run any gog slides subcommand not covered by the other tools. Run `gog slides --help` for the full list of subcommands, or `gog slides <subcommand> --help` for flags on a specific subcommand.",
+    annotations: { destructiveHint: true },
+    inputSchema: {
+      subcommand: external_exports.string().describe("The gog slides subcommand to run"),
+      args: external_exports.array(external_exports.string()).describe("Additional positional args and flags"),
+      account: accountParam
+    }
+  }, async ({ subcommand, args, account }) => {
+    return runOrDiagnose(["slides", subcommand, ...args], { account });
+  });
+}
+
+// ../gogcli-mcp/src/server.ts
+var VERSION = true ? "2.0.7" : "0.0.0";
+function createServer(options) {
+  return new McpServer({
+    name: options?.name ?? "gogcli",
+    version: options?.version ?? VERSION
+  });
+}
+
+// src/tools/slides-extra.ts
+function registerExtraSlidesTools(server2) {
   server2.registerTool("gog_slides_create_from_markdown", {
     description: "Create a new Google Slides presentation from markdown content (inline or from a file).",
     inputSchema: {
@@ -31109,19 +31172,6 @@ function registerSlidesTools(server2) {
     if (exact) args.push("--exact");
     return runOrDiagnose(args, { account });
   });
-  server2.registerTool("gog_slides_copy", {
-    description: "Copy a Google Slides presentation to a new presentation with the given title.",
-    inputSchema: {
-      presentationId: external_exports.string().describe("Presentation ID to copy"),
-      title: external_exports.string().describe("Title for the new copy"),
-      parent: external_exports.string().optional().describe("Destination folder ID"),
-      account: accountParam
-    }
-  }, async ({ presentationId, title, parent, account }) => {
-    const args = ["slides", "copy", presentationId, title];
-    if (parent) args.push(`--parent=${parent}`);
-    return runOrDiagnose(args, { account });
-  });
   server2.registerTool("gog_slides_add_slide", {
     description: "Add a new slide to a presentation from a local image, with optional speaker notes.",
     inputSchema: {
@@ -31139,16 +31189,6 @@ function registerSlidesTools(server2) {
     if (before) args.push(`--before=${before}`);
     return runOrDiagnose(args, { account });
   });
-  server2.registerTool("gog_slides_list_slides", {
-    description: "List slides in a Google Slides presentation.",
-    annotations: { readOnlyHint: true },
-    inputSchema: {
-      presentationId: external_exports.string().describe("Presentation ID"),
-      account: accountParam
-    }
-  }, async ({ presentationId, account }) => {
-    return runOrDiagnose(["slides", "list-slides", presentationId], { account });
-  });
   server2.registerTool("gog_slides_delete_slide", {
     description: "Delete a slide from a Google Slides presentation.",
     annotations: { destructiveHint: true },
@@ -31160,17 +31200,6 @@ function registerSlidesTools(server2) {
   }, async ({ presentationId, slideId, account }) => {
     return runOrDiagnose(["slides", "delete-slide", presentationId, slideId], { account });
   });
-  server2.registerTool("gog_slides_read_slide", {
-    description: "Read the content of a slide (text, shapes, speaker notes).",
-    annotations: { readOnlyHint: true },
-    inputSchema: {
-      presentationId: external_exports.string().describe("Presentation ID"),
-      slideId: external_exports.string().describe("Slide ID to read"),
-      account: accountParam
-    }
-  }, async ({ presentationId, slideId, account }) => {
-    return runOrDiagnose(["slides", "read-slide", presentationId, slideId], { account });
-  });
   server2.registerTool("gog_slides_update_notes", {
     description: "Update the speaker notes on a slide (inline text or from a file).",
     annotations: { destructiveHint: true },
@@ -31204,29 +31233,6 @@ function registerSlidesTools(server2) {
     if (notesFile) args.push(`--notes-file=${notesFile}`);
     return runOrDiagnose(args, { account });
   });
-  server2.registerTool("gog_slides_run", {
-    description: "Run any gog slides subcommand not covered by the other tools. Run `gog slides --help` for the full list of subcommands, or `gog slides <subcommand> --help` for flags on a specific subcommand.",
-    inputSchema: {
-      subcommand: external_exports.string().describe("The gog slides subcommand to run"),
-      args: external_exports.array(external_exports.string()).describe("Additional positional args and flags"),
-      account: accountParam
-    }
-  }, async ({ subcommand, args, account }) => {
-    return runOrDiagnose(["slides", subcommand, ...args], { account });
-  });
-}
-
-// ../gogcli-mcp/src/server.ts
-var VERSION = true ? "2.0.4" : "0.0.0";
-function createServer(options) {
-  return new McpServer({
-    name: options?.name ?? "gogcli",
-    version: options?.version ?? VERSION
-  });
-}
-
-// src/tools/slides-extra.ts
-function registerExtraSlidesTools(_server) {
 }
 
 // src/index.ts

package/manifest.json

@@ -3,7 +3,7 @@
   "manifest_version": "0.3",
   "name": "gogcli-mcp-slides",
   "display_name": "gogcli (Slides)",
-  "version": "2.0.4",
+  "version": "2.0.7",
   "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,6 @@
 {
   "name": "gogcli-mcp-slides",
-  "version": "2.0.4",
+  "version": "2.0.7",
   "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>",

package/server.json

@@ -7,12 +7,12 @@
     "source": "github",
     "subfolder": "packages/gogcli-mcp-slides"
   },
-  "version": "2.0.4",
+  "version": "2.0.7",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "gogcli-mcp-slides",
-      "version": "2.0.4",
+      "version": "2.0.7",
       "transport": {
         "type": "stdio"
       },

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 },
+    );
   });
 });