gogcli-mcp 2.0.4 → 2.0.7

package/tests/runner.test.ts

@@ -113,6 +113,69 @@ describe('run', () => {
     }
   });
 
+  it('falls back to "gog" on PATH when GOG_PATH is unset', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_PATH;
+    delete process.env.GOG_PATH;
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      expect(spawner).toHaveBeenCalledWith('gog', expect.any(Array), expect.any(Object));
+    } finally {
+      if (originalEnv !== undefined) {
+        process.env.GOG_PATH = originalEnv;
+      }
+    }
+  });
+
+  it('falls back to "gog" on PATH when GOG_PATH is set to empty string', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_PATH;
+    process.env.GOG_PATH = '';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      expect(spawner).toHaveBeenCalledWith('gog', expect.any(Array), expect.any(Object));
+    } finally {
+      if (originalEnv === undefined) {
+        delete process.env.GOG_PATH;
+      } else {
+        process.env.GOG_PATH = originalEnv;
+      }
+    }
+  });
+
+  it('falls back to "gog" on PATH when GOG_PATH is an unresolved .mcpb placeholder', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_PATH;
+    process.env.GOG_PATH = '${user_config.gog_path}';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      expect(spawner).toHaveBeenCalledWith('gog', expect.any(Array), expect.any(Object));
+    } finally {
+      if (originalEnv === undefined) {
+        delete process.env.GOG_PATH;
+      } else {
+        process.env.GOG_PATH = originalEnv;
+      }
+    }
+  });
+
+  it('omits --account when GOG_ACCOUNT is an unresolved .mcpb placeholder', async () => {
+    const spawner = makeSpawner(0, '{}');
+    const originalEnv = process.env.GOG_ACCOUNT;
+    process.env.GOG_ACCOUNT = '${user_config.gog_account}';
+    try {
+      await run(['sheets', 'metadata', 'id1'], { spawner });
+      const callArgs = (spawner as ReturnType<typeof vi.fn>).mock.calls[0][1] as string[];
+      expect(callArgs).not.toContain('--account');
+    } finally {
+      if (originalEnv === undefined) {
+        delete process.env.GOG_ACCOUNT;
+      } else {
+        process.env.GOG_ACCOUNT = originalEnv;
+      }
+    }
+  });
+
   it('returns stdout on exit code 0', async () => {
     const spawner = makeSpawner(0, '{"values":[["hello"]]}');
     const result = await run(['sheets', 'get', 'id1', 'A1'], { spawner });

package/tests/tools/calendar.test.ts

@@ -228,140 +228,3 @@ describe('gog_calendar_run', () => {
   });
 });
 
-describe('gog_meet_create', () => {
-  it('calls run with no flags', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_create')!({});
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'create'], { account: undefined });
-  });
-
-  it('passes --access and --open when provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_create')!({ access: 'open', open: true });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['meet', 'create', '--access=open', '--open'],
-      { account: undefined },
-    );
-  });
-
-  it('omits --open when false', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_create')!({ open: false });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'create'], { account: undefined });
-  });
-});
-
-describe('gog_meet_get', () => {
-  it('calls run with meetingCode', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_get')!({ meetingCode: 'abc-defg-hij' });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'get', 'abc-defg-hij'], { account: undefined });
-  });
-});
-
-describe('gog_meet_update', () => {
-  it('calls run with meetingCode and --access', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_update')!({ meetingCode: 'abc-defg-hij', access: 'restricted' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['meet', 'update', 'abc-defg-hij', '--access=restricted'],
-      { account: undefined },
-    );
-  });
-
-  it('omits --access when not provided', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_update')!({ meetingCode: 'abc-defg-hij' });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'update', 'abc-defg-hij'], { account: undefined });
-  });
-});
-
-describe('gog_meet_end', () => {
-  it('calls run with meetingCode', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_end')!({ meetingCode: 'abc-defg-hij' });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'end', 'abc-defg-hij'], { account: undefined });
-  });
-});
-
-describe('gog_meet_history', () => {
-  it('calls run with meetingCode', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_history')!({ meetingCode: 'abc-defg-hij' });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'history', 'abc-defg-hij'], { account: undefined });
-  });
-
-  it('passes pagination flags', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_history')!({
-      meetingCode: 'abc-defg-hij',
-      max: 50,
-      page: 'tok',
-      all: true,
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['meet', 'history', 'abc-defg-hij', '--max=50', '--page=tok', '--all'],
-      { account: undefined },
-    );
-  });
-
-  it('omits --all when false', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_history')!({ meetingCode: 'abc-defg-hij', all: false });
-    expect(runner.run).toHaveBeenCalledWith(['meet', 'history', 'abc-defg-hij'], { account: undefined });
-  });
-});
-
-describe('gog_meet_participants', () => {
-  it('calls run with meetingCode', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_participants')!({ meetingCode: 'abc-defg-hij' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['meet', 'participants', 'abc-defg-hij'],
-      { account: undefined },
-    );
-  });
-
-  it('passes --conference and pagination flags', async () => {
-    vi.mocked(runner.run).mockResolvedValue('{}');
-    const handlers = setupHandlers();
-    await handlers.get('gog_meet_participants')!({
-      meetingCode: 'abc-defg-hij',
-      conference: 'conf123',
-      max: 100,
-      page: 'tok',
-      all: true,
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      [
-        'meet', 'participants', 'abc-defg-hij',
-        '--conference=conf123',
-        '--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_meet_participants')!({ meetingCode: 'abc-defg-hij', all: false });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['meet', 'participants', 'abc-defg-hij'],
-      { account: undefined },
-    );
-  });
-});

package/tests/tools/classroom.test.ts

@@ -72,107 +72,6 @@ describe('gog_classroom_courses_get', () => {
   });
 });
 
-describe('gog_classroom_courses_create', () => {
-  it('calls run with required name only', async () => {
-    await handlers.get('gog_classroom_courses_create')!({ name: 'Math 101' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'courses', 'create', '--name=Math 101'],
-      { account: undefined },
-    );
-  });
-
-  it('passes all optional flags', async () => {
-    await handlers.get('gog_classroom_courses_create')!({
-      name: 'Math 101',
-      owner: 'me',
-      section: 'Section A',
-      descriptionHeading: 'Welcome',
-      description: 'Algebra',
-      room: 'R101',
-      state: 'ACTIVE',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'courses', 'create', '--name=Math 101', '--owner=me', '--section=Section A', '--description-heading=Welcome', '--description=Algebra', '--room=R101', '--state=ACTIVE'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Create failed'));
-    const result = await handlers.get('gog_classroom_courses_create')!({ name: 'X' });
-    expect(result.content[0].text).toBe('Error: Create failed');
-  });
-});
-
-describe('gog_classroom_courses_update', () => {
-  it('calls run with courseId only', async () => {
-    await handlers.get('gog_classroom_courses_update')!({ courseId: 'c1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'courses', 'update', 'c1'], { account: undefined });
-  });
-
-  it('passes all optional flags', async () => {
-    await handlers.get('gog_classroom_courses_update')!({
-      courseId: 'c1',
-      name: 'New Name',
-      owner: 'me',
-      section: 'B',
-      descriptionHeading: 'Heading',
-      description: 'Desc',
-      room: 'R2',
-      state: 'ARCHIVED',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'courses', 'update', 'c1', '--name=New Name', '--owner=me', '--section=B', '--description-heading=Heading', '--description=Desc', '--room=R2', '--state=ARCHIVED'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Update failed'));
-    const result = await handlers.get('gog_classroom_courses_update')!({ courseId: 'bad' });
-    expect(result.content[0].text).toBe('Error: Update failed');
-  });
-});
-
-describe('gog_classroom_courses_delete', () => {
-  it('calls run with courseId', async () => {
-    await handlers.get('gog_classroom_courses_delete')!({ courseId: 'c1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'courses', 'delete', 'c1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const result = await handlers.get('gog_classroom_courses_delete')!({ courseId: 'bad' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
-describe('gog_classroom_courses_archive', () => {
-  it('calls run with courseId', async () => {
-    await handlers.get('gog_classroom_courses_archive')!({ courseId: 'c1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'courses', 'archive', 'c1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Archive failed'));
-    const result = await handlers.get('gog_classroom_courses_archive')!({ courseId: 'x' });
-    expect(result.content[0].text).toBe('Error: Archive failed');
-  });
-});
-
-describe('gog_classroom_courses_unarchive', () => {
-  it('calls run with courseId', async () => {
-    await handlers.get('gog_classroom_courses_unarchive')!({ courseId: 'c1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'courses', 'unarchive', 'c1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Unarchive failed'));
-    const result = await handlers.get('gog_classroom_courses_unarchive')!({ courseId: 'x' });
-    expect(result.content[0].text).toBe('Error: Unarchive failed');
-  });
-});
-
 describe('gog_classroom_students_list', () => {
   it('calls run with courseId only', async () => {
     await handlers.get('gog_classroom_students_list')!({ courseId: 'c1' });
@@ -212,40 +111,6 @@ describe('gog_classroom_students_get', () => {
   });
 });
 
-describe('gog_classroom_students_add', () => {
-  it('calls run with courseId and userId', async () => {
-    await handlers.get('gog_classroom_students_add')!({ courseId: 'c1', userId: 'u1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'students', 'add', 'c1', 'u1'], { account: undefined });
-  });
-
-  it('passes --enrollment-code when provided', async () => {
-    await handlers.get('gog_classroom_students_add')!({ courseId: 'c1', userId: 'u1', enrollmentCode: 'abc123' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'students', 'add', 'c1', 'u1', '--enrollment-code=abc123'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Add failed'));
-    const result = await handlers.get('gog_classroom_students_add')!({ courseId: 'c1', userId: 'u1' });
-    expect(result.content[0].text).toBe('Error: Add failed');
-  });
-});
-
-describe('gog_classroom_students_remove', () => {
-  it('calls run with courseId and userId', async () => {
-    await handlers.get('gog_classroom_students_remove')!({ courseId: 'c1', userId: 'u1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'students', 'remove', 'c1', 'u1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Remove failed'));
-    const result = await handlers.get('gog_classroom_students_remove')!({ courseId: 'c1', userId: 'u1' });
-    expect(result.content[0].text).toBe('Error: Remove failed');
-  });
-});
-
 describe('gog_classroom_teachers_list', () => {
   it('calls run with courseId only', async () => {
     await handlers.get('gog_classroom_teachers_list')!({ courseId: 'c1' });
@@ -285,32 +150,6 @@ describe('gog_classroom_teachers_get', () => {
   });
 });
 
-describe('gog_classroom_teachers_add', () => {
-  it('calls run with courseId and userId', async () => {
-    await handlers.get('gog_classroom_teachers_add')!({ courseId: 'c1', userId: 'u1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'teachers', 'add', 'c1', 'u1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Add failed'));
-    const result = await handlers.get('gog_classroom_teachers_add')!({ courseId: 'c1', userId: 'u1' });
-    expect(result.content[0].text).toBe('Error: Add failed');
-  });
-});
-
-describe('gog_classroom_teachers_remove', () => {
-  it('calls run with courseId and userId', async () => {
-    await handlers.get('gog_classroom_teachers_remove')!({ courseId: 'c1', userId: 'u1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'teachers', 'remove', 'c1', 'u1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Remove failed'));
-    const result = await handlers.get('gog_classroom_teachers_remove')!({ courseId: 'c1', userId: 'u1' });
-    expect(result.content[0].text).toBe('Error: Remove failed');
-  });
-});
-
 describe('gog_classroom_roster', () => {
   it('calls run with courseId only', async () => {
     await handlers.get('gog_classroom_roster')!({ courseId: 'c1' });
@@ -385,89 +224,6 @@ describe('gog_classroom_coursework_get', () => {
   });
 });
 
-describe('gog_classroom_coursework_create', () => {
-  it('calls run with required title only', async () => {
-    await handlers.get('gog_classroom_coursework_create')!({ courseId: 'c1', title: 'HW1' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'coursework', 'create', 'c1', '--title=HW1'],
-      { account: undefined },
-    );
-  });
-
-  it('passes all optional flags', async () => {
-    await handlers.get('gog_classroom_coursework_create')!({
-      courseId: 'c1',
-      title: 'HW1',
-      description: 'Chapter 1',
-      type: 'ASSIGNMENT',
-      state: 'PUBLISHED',
-      maxPoints: 100,
-      due: '2026-05-01T23:59',
-      dueDate: '2026-05-01',
-      dueTime: '23:59',
-      scheduled: '2026-04-30T12:00',
-      topic: 't1',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'coursework', 'create', 'c1', '--title=HW1', '--description=Chapter 1', '--type=ASSIGNMENT', '--state=PUBLISHED', '--max-points=100', '--due=2026-05-01T23:59', '--due-date=2026-05-01', '--due-time=23:59', '--scheduled=2026-04-30T12:00', '--topic=t1'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Create failed'));
-    const result = await handlers.get('gog_classroom_coursework_create')!({ courseId: 'c1', title: 'HW1' });
-    expect(result.content[0].text).toBe('Error: Create failed');
-  });
-});
-
-describe('gog_classroom_coursework_update', () => {
-  it('calls run with ids only', async () => {
-    await handlers.get('gog_classroom_coursework_update')!({ courseId: 'c1', courseworkId: 'w1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'coursework', 'update', 'c1', 'w1'], { account: undefined });
-  });
-
-  it('passes all optional flags', async () => {
-    await handlers.get('gog_classroom_coursework_update')!({
-      courseId: 'c1',
-      courseworkId: 'w1',
-      title: 'New Title',
-      description: 'Desc',
-      type: 'SHORT_ANSWER_QUESTION',
-      state: 'DRAFT',
-      maxPoints: 50,
-      due: '2026-05-01T23:59',
-      dueDate: '2026-05-01',
-      dueTime: '23:59',
-      scheduled: '2026-04-30T12:00',
-      topic: 't1',
-    });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'coursework', 'update', 'c1', 'w1', '--title=New Title', '--description=Desc', '--type=SHORT_ANSWER_QUESTION', '--state=DRAFT', '--max-points=50', '--due=2026-05-01T23:59', '--due-date=2026-05-01', '--due-time=23:59', '--scheduled=2026-04-30T12:00', '--topic=t1'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Update failed'));
-    const result = await handlers.get('gog_classroom_coursework_update')!({ courseId: 'c1', courseworkId: 'w1' });
-    expect(result.content[0].text).toBe('Error: Update failed');
-  });
-});
-
-describe('gog_classroom_coursework_delete', () => {
-  it('calls run with ids', async () => {
-    await handlers.get('gog_classroom_coursework_delete')!({ courseId: 'c1', courseworkId: 'w1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'coursework', 'delete', 'c1', 'w1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const result = await handlers.get('gog_classroom_coursework_delete')!({ courseId: 'c1', courseworkId: 'w1' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
 describe('gog_classroom_submissions_list', () => {
   it('calls run with ids only', async () => {
     await handlers.get('gog_classroom_submissions_list')!({ courseId: 'c1', courseworkId: 'w1' });
@@ -639,40 +395,6 @@ describe('gog_classroom_announcements_create', () => {
   });
 });
 
-describe('gog_classroom_announcements_update', () => {
-  it('calls run with ids only', async () => {
-    await handlers.get('gog_classroom_announcements_update')!({ courseId: 'c1', announcementId: 'a1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'announcements', 'update', 'c1', 'a1'], { account: undefined });
-  });
-
-  it('passes all optional flags', async () => {
-    await handlers.get('gog_classroom_announcements_update')!({ courseId: 'c1', announcementId: 'a1', text: 'edited', state: 'PUBLISHED', scheduled: '2026-05-01T12:00' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'announcements', 'update', 'c1', 'a1', '--text=edited', '--state=PUBLISHED', '--scheduled=2026-05-01T12:00'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Update failed'));
-    const result = await handlers.get('gog_classroom_announcements_update')!({ courseId: 'c1', announcementId: 'a1' });
-    expect(result.content[0].text).toBe('Error: Update failed');
-  });
-});
-
-describe('gog_classroom_announcements_delete', () => {
-  it('calls run with ids', async () => {
-    await handlers.get('gog_classroom_announcements_delete')!({ courseId: 'c1', announcementId: 'a1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'announcements', 'delete', 'c1', 'a1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const result = await handlers.get('gog_classroom_announcements_delete')!({ courseId: 'c1', announcementId: 'a1' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
 describe('gog_classroom_topics_list', () => {
   it('calls run with courseId only', async () => {
     await handlers.get('gog_classroom_topics_list')!({ courseId: 'c1' });
@@ -712,51 +434,6 @@ describe('gog_classroom_topics_get', () => {
   });
 });
 
-describe('gog_classroom_topics_create', () => {
-  it('calls run with courseId and name', async () => {
-    await handlers.get('gog_classroom_topics_create')!({ courseId: 'c1', name: 'Week 1' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'topics', 'create', 'c1', '--name=Week 1'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Create failed'));
-    const result = await handlers.get('gog_classroom_topics_create')!({ courseId: 'c1', name: 'x' });
-    expect(result.content[0].text).toBe('Error: Create failed');
-  });
-});
-
-describe('gog_classroom_topics_update', () => {
-  it('calls run with ids and name', async () => {
-    await handlers.get('gog_classroom_topics_update')!({ courseId: 'c1', topicId: 't1', name: 'Week 2' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'topics', 'update', 'c1', 't1', '--name=Week 2'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Update failed'));
-    const result = await handlers.get('gog_classroom_topics_update')!({ courseId: 'c1', topicId: 't1', name: 'x' });
-    expect(result.content[0].text).toBe('Error: Update failed');
-  });
-});
-
-describe('gog_classroom_topics_delete', () => {
-  it('calls run with ids', async () => {
-    await handlers.get('gog_classroom_topics_delete')!({ courseId: 'c1', topicId: 't1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'topics', 'delete', 'c1', 't1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const result = await handlers.get('gog_classroom_topics_delete')!({ courseId: 'c1', topicId: 't1' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
 describe('gog_classroom_invitations_list', () => {
   it('calls run with no flags', async () => {
     await handlers.get('gog_classroom_invitations_list')!({});
@@ -796,22 +473,6 @@ describe('gog_classroom_invitations_get', () => {
   });
 });
 
-describe('gog_classroom_invitations_create', () => {
-  it('calls run with courseId, userId, role', async () => {
-    await handlers.get('gog_classroom_invitations_create')!({ courseId: 'c1', userId: 'u1', role: 'STUDENT' });
-    expect(runner.run).toHaveBeenCalledWith(
-      ['classroom', 'invitations', 'create', 'c1', 'u1', '--role=STUDENT'],
-      { account: undefined },
-    );
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Create failed'));
-    const result = await handlers.get('gog_classroom_invitations_create')!({ courseId: 'c1', userId: 'u1', role: 'TEACHER' });
-    expect(result.content[0].text).toBe('Error: Create failed');
-  });
-});
-
 describe('gog_classroom_invitations_accept', () => {
   it('calls run with invitationId', async () => {
     await handlers.get('gog_classroom_invitations_accept')!({ invitationId: 'i1' });
@@ -825,19 +486,6 @@ describe('gog_classroom_invitations_accept', () => {
   });
 });
 
-describe('gog_classroom_invitations_delete', () => {
-  it('calls run with invitationId', async () => {
-    await handlers.get('gog_classroom_invitations_delete')!({ invitationId: 'i1' });
-    expect(runner.run).toHaveBeenCalledWith(['classroom', 'invitations', 'delete', 'i1'], { account: undefined });
-  });
-
-  it('returns error text on failure', async () => {
-    vi.mocked(runner.run).mockRejectedValue(new Error('Delete failed'));
-    const result = await handlers.get('gog_classroom_invitations_delete')!({ invitationId: 'bad' });
-    expect(result.content[0].text).toBe('Error: Delete failed');
-  });
-});
-
 describe('gog_classroom_profile_get', () => {
   it('calls run with no userId (self)', async () => {
     await handlers.get('gog_classroom_profile_get')!({});