gogcli-mcp 2.0.4 → 2.0.7

package/tests/tools/contacts.test.ts

@@ -120,103 +120,3 @@ describe('gog_contacts_run', () => {
     expect(result.content[0].text).toBe('Error: Run failed');
   });
 });
-
-describe('gog_people_me', () => {
-  it('calls run with people me', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{"resourceName":"people/me"}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_me')!({});
-    expect(runner.run).toHaveBeenCalledWith(['people', 'me'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Me failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_people_me')!({});
-    expect(result.content[0].text).toBe('Error: Me failed');
-  });
-});
-
-describe('gog_people_get', () => {
-  it('calls run with userId', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_get')!({ userId: 'people/c123' });
-    expect(runner.run).toHaveBeenCalledWith(['people', 'get', 'people/c123'], { account: undefined });
-  });
-});
-
-describe('gog_people_search', () => {
-  it('calls run with query', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_search')!({ query: 'alice' });
-    expect(runner.run).toHaveBeenCalledWith(['people', 'search', 'alice'], { account: undefined });
-  });
-
-  it('passes pagination flags', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_search')!({ query: 'alice', max: 100, page: 'tok', all: true });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['people', 'search', 'alice', '--max=100', '--page=tok', '--all'],
-      { account: undefined },
-    );
-  });
-
-  it('omits --all when false', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_search')!({ query: 'x', all: false });
-    expect(runner.run).toHaveBeenCalledWith(['people', 'search', 'x'], { account: undefined });
-  });
-});
-
-describe('gog_people_relations', () => {
-  it('calls run with no userId', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_relations')!({});
-    expect(runner.run).toHaveBeenCalledWith(['people', 'relations'], { account: undefined });
-  });
-
-  it('passes userId and --type when provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_relations')!({ userId: 'people/c123', type: 'manager' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['people', 'relations', 'people/c123', '--type=manager'],
-      { account: undefined },
-    );
-  });
-});
-
-describe('gog_people_raw', () => {
-  it('calls run with userId', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_raw')!({ userId: 'people/c123' });
-    expect(runner.run).toHaveBeenCalledWith(['people', 'raw', 'people/c123'], { account: undefined });
-  });
-
-  it('passes --person-fields and --pretty when provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_raw')!({
-      userId: 'people/c123',
-      personFields: 'names,emailAddresses',
-      pretty: true,
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['people', 'raw', 'people/c123', '--person-fields=names,emailAddresses', '--pretty'],
-      { account: undefined },
-    );
-  });
-
-  it('omits --pretty when false', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_people_raw')!({ userId: 'people/c123', pretty: false });
-    expect(runner.run).toHaveBeenCalledWith(['people', 'raw', 'people/c123'], { account: undefined });
-  });
-});

package/tests/tools/slides.test.ts

@@ -31,9 +31,9 @@ describe('gog_slides_export', () => {
   it('passes --out and --format when provided', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
     const handlers = setupHandlers();
-    await handlers.get('gog_slides_export')!({ presentationId: 'p1', out: '/tmp/out.pdf', format: 'pdf' });
+    await handlers.get('gog_slides_export')!({ presentationId: 'p1', out: '/tmp/deck.pdf', format: 'pdf' });
     expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'export', 'p1', '--out=/tmp/out.pdf', '--format=pdf'],
+      ['slides', 'export', 'p1', '--out=/tmp/deck.pdf', '--format=pdf'],
       { account: undefined },
     );
   });
@@ -66,16 +66,16 @@ describe('gog_slides_create', () => {
   it('calls run with title only', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
     const handlers = setupHandlers();
-    await handlers.get('gog_slides_create')!({ title: 'My Deck' });
-    expect(runner.run).toHaveBeenCalledWith(['slides', 'create', 'My Deck'], { account: undefined });
+    await handlers.get('gog_slides_create')!({ title: 'Deck' });
+    expect(runner.run).toHaveBeenCalledWith(['slides', 'create', 'Deck'], { account: undefined });
   });
 
   it('passes --parent and --template when provided', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
     const handlers = setupHandlers();
-    await handlers.get('gog_slides_create')!({ title: 'My Deck', parent: 'folder1', template: 'tpl1' });
+    await handlers.get('gog_slides_create')!({ title: 'Deck', parent: 'folder1', template: 'tpl1' });
     expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create', 'My Deck', '--parent=folder1', '--template=tpl1'],
+      ['slides', 'create', 'Deck', '--parent=folder1', '--template=tpl1'],
       { account: undefined },
     );
   });
@@ -88,134 +88,6 @@ describe('gog_slides_create', () => {
   });
 });
 
-describe('gog_slides_create_from_markdown', () => {
-  it('calls run with title only', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_markdown')!({ title: 'Deck' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create-from-markdown', 'Deck'],
-      { account: undefined },
-    );
-  });
-
-  it('passes all optional flags', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_markdown')!({
-      title: 'Deck',
-      content: '# Slide 1',
-      contentFile: '/tmp/deck.md',
-      parent: 'folder1',
-      debug: true,
-    });
-    expect(runner.run).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 () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_markdown')!({ title: 'Deck', debug: false });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create-from-markdown', 'Deck'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Markdown failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_create_from_markdown')!({ title: 'x' });
-    expect(result.content[0].text).toBe('Error: Markdown failed');
-  });
-});
-
-describe('gog_slides_create_from_template', () => {
-  it('calls run with templateId and title only', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_template')!({ templateId: 'tpl1', title: 'Deck' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create-from-template', 'tpl1', 'Deck'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --replace for a single replacement entry', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_template')!({
-      templateId: 'tpl1',
-      title: 'Deck',
-      replacements: { name: 'Alice' },
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create-from-template', 'tpl1', 'Deck', '--replace=name=Alice'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --replace for each entry in replacements', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_template')!({
-      templateId: 'tpl1',
-      title: 'Deck',
-      replacements: { name: 'Alice', company: 'Acme' },
-    });
-    const call = vi.mocked(runner.run).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 () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_template')!({
-      templateId: 'tpl1',
-      title: 'Deck',
-      replacementsFile: '/tmp/r.json',
-      parent: 'folder1',
-      exact: true,
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      [
-        'slides', 'create-from-template', 'tpl1', 'Deck',
-        '--replacements=/tmp/r.json',
-        '--parent=folder1',
-        '--exact',
-      ],
-      { account: undefined },
-    );
-  });
-
-  it('omits --exact when false', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_create_from_template')!({ templateId: 'tpl1', title: 'Deck', exact: false });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'create-from-template', 'tpl1', 'Deck'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Template failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_create_from_template')!({ templateId: 't', title: 'x' });
-    expect(result.content[0].text).toBe('Error: Template failed');
-  });
-});
-
 describe('gog_slides_copy', () => {
   it('calls run with presentationId and title', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
@@ -245,46 +117,6 @@ describe('gog_slides_copy', () => {
   });
 });
 
-describe('gog_slides_add_slide', () => {
-  it('calls run with presentationId and image', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_add_slide')!({ presentationId: 'p1', image: '/tmp/img.png' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'add-slide', 'p1', '/tmp/img.png'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --notes, --notes-file, and --before', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_add_slide')!({
-      presentationId: 'p1',
-      image: '/tmp/img.png',
-      notes: 'Speaker note',
-      notesFile: '/tmp/notes.txt',
-      before: 'slide5',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      [
-        'slides', 'add-slide', 'p1', '/tmp/img.png',
-        '--notes=Speaker note',
-        '--notes-file=/tmp/notes.txt',
-        '--before=slide5',
-      ],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Add failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_add_slide')!({ presentationId: 'p', image: 'i' });
-    expect(result.content[0].text).toBe('Error: Add failed');
-  });
-});
-
 describe('gog_slides_list_slides', () => {
   it('calls run with presentationId', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
@@ -301,25 +133,6 @@ describe('gog_slides_list_slides', () => {
   });
 });
 
-describe('gog_slides_delete_slide', () => {
-  it('calls run with presentationId and slideId', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_delete_slide')!({ presentationId: 'p1', slideId: 's1' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'delete-slide', 'p1', 's1'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_delete_slide')!({ presentationId: 'p', slideId: 's' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
 describe('gog_slides_read_slide', () => {
   it('calls run with presentationId and slideId', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');
@@ -339,91 +152,6 @@ describe('gog_slides_read_slide', () => {
   });
 });
 
-describe('gog_slides_update_notes', () => {
-  it('calls run with presentationId and slideId', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_update_notes')!({ presentationId: 'p1', slideId: 's1' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'update-notes', 'p1', 's1'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --notes and --notes-file when provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_update_notes')!({
-      presentationId: 'p1',
-      slideId: 's1',
-      notes: 'speak clearly',
-      notesFile: '/tmp/n.txt',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      [
-        'slides', 'update-notes', 'p1', 's1',
-        '--notes=speak clearly',
-        '--notes-file=/tmp/n.txt',
-      ],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Notes failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_update_notes')!({ presentationId: 'p', slideId: 's' });
-    expect(result.content[0].text).toBe('Error: Notes failed');
-  });
-});
-
-describe('gog_slides_replace_slide', () => {
-  it('calls run with presentationId, slideId, and image', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_replace_slide')!({
-      presentationId: 'p1',
-      slideId: 's1',
-      image: '/tmp/img.png',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['slides', 'replace-slide', 'p1', 's1', '/tmp/img.png'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --notes and --notes-file when provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_slides_replace_slide')!({
-      presentationId: 'p1',
-      slideId: 's1',
-      image: '/tmp/img.png',
-      notes: 'updated',
-      notesFile: '/tmp/n.txt',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      [
-        'slides', 'replace-slide', 'p1', 's1', '/tmp/img.png',
-        '--notes=updated',
-        '--notes-file=/tmp/n.txt',
-      ],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Replace failed'));
-    const handlers = setupHandlers();
-    const result = await handlers.get('gog_slides_replace_slide')!({
-      presentationId: 'p',
-      slideId: 's',
-      image: 'i',
-    });
-    expect(result.content[0].text).toBe('Error: Replace failed');
-  });
-});
-
 describe('gog_slides_run', () => {
   it('passes subcommand and args to runner', async () => {
     vi.mocked(runner.run).mockResolvedValue('{}');