@vibescope/mcp-server 0.5.0 → 0.5.2

package/src/handlers/fallback.test.ts

@@ -1,537 +1,537 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-	startFallbackActivity,
-	stopFallbackActivity,
-	getActivityHistory,
-	getActivitySchedules,
-} from './fallback.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// startFallbackActivity Tests
-// ============================================================================
-
-describe('startFallbackActivity', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			startFallbackActivity({ activity: 'code_review' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			startFallbackActivity({ project_id: 'invalid', activity: 'code_review' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing activity', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			startFallbackActivity({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid activity type', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			startFallbackActivity({
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity: 'invalid_activity',
-			}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should start fallback activity successfully', async () => {
-		mockApiClient.startFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: { message: 'Started code_review' },
-		});
-		const ctx = createMockContext();
-
-		const result = await startFallbackActivity(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity: 'code_review',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			activity: 'code_review',
-		});
-		expect((result.result as { title: string }).title).toBeDefined();
-	});
-
-	it('should call API client with correct parameters', async () => {
-		mockApiClient.startFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: { message: 'Started' },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await startFallbackActivity(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity: 'security_review',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.startFallbackActivity).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'security_review',
-			'my-session'
-		);
-	});
-
-	it('should accept all valid activity types', async () => {
-		const validActivities = [
-			'feature_ideation',
-			'code_review',
-			'performance_audit',
-			'ux_review',
-			'cost_analysis',
-			'security_review',
-			'test_coverage',
-			'documentation_review',
-			'dependency_audit',
-			'validate_completed_tasks',
-		];
-
-		for (const activity of validActivities) {
-			mockApiClient.startFallbackActivity.mockResolvedValue({
-				ok: true,
-				data: { message: `Started ${activity}` },
-			});
-			const ctx = createMockContext();
-
-			const result = await startFallbackActivity(
-				{
-					project_id: '123e4567-e89b-12d3-a456-426614174000',
-					activity,
-				},
-				ctx
-			);
-
-			expect(result.result).toMatchObject({
-				success: true,
-				activity,
-			});
-		}
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.startFallbackActivity.mockResolvedValue({
-			ok: false,
-			error: 'Failed to start activity',
-		});
-		const ctx = createMockContext();
-
-		const result = await startFallbackActivity({
-			project_id: '123e4567-e89b-12d3-a456-426614174000',
-			activity: 'code_review',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to start activity' });
-	});
-
-	it('should pass through worktree guidance when API returns it', async () => {
-		mockApiClient.startFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				activity: 'code_review',
-				message: 'Started code_review',
-				git_workflow: {
-					workflow: 'git-flow',
-					base_branch: 'develop',
-					worktree_recommended: true,
-					note: 'Fallback activities use the base branch directly (read-only).',
-				},
-				worktree_setup: {
-					message: 'RECOMMENDED: Create a worktree to avoid conflicts.',
-					commands: [
-						'git checkout develop',
-						'git pull origin develop',
-						'git worktree add ../Project-code-review develop',
-						'cd ../Project-code-review',
-					],
-					worktree_path: '../Project-code-review',
-					branch_name: 'develop',
-					cleanup_command: 'git worktree remove ../Project-code-review',
-					report_worktree: 'heartbeat(current_worktree_path: "../Project-code-review")',
-				},
-				next_step: 'After setting up worktree: call heartbeat to report your location.',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await startFallbackActivity(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity: 'code_review',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			activity: 'code_review',
-		});
-		expect((result.result as { git_workflow?: unknown }).git_workflow).toBeDefined();
-		expect((result.result as { git_workflow: { workflow: string } }).git_workflow.workflow).toBe('git-flow');
-		expect((result.result as { worktree_setup?: unknown }).worktree_setup).toBeDefined();
-		expect((result.result as { worktree_setup: { worktree_path: string } }).worktree_setup.worktree_path).toBe('../Project-code-review');
-		expect((result.result as { next_step?: string }).next_step).toContain('heartbeat');
-	});
-
-	it('should not include worktree guidance when API does not return it', async () => {
-		mockApiClient.startFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				activity: 'code_review',
-				message: 'Started code_review',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await startFallbackActivity(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity: 'code_review',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			activity: 'code_review',
-		});
-		expect((result.result as { git_workflow?: unknown }).git_workflow).toBeUndefined();
-		expect((result.result as { worktree_setup?: unknown }).worktree_setup).toBeUndefined();
-		expect((result.result as { next_step?: string }).next_step).toBeUndefined();
-	});
-});
-
-// ============================================================================
-// stopFallbackActivity Tests
-// ============================================================================
-
-describe('stopFallbackActivity', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(stopFallbackActivity({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			stopFallbackActivity({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should stop fallback activity successfully', async () => {
-		mockApiClient.stopFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await stopFallbackActivity(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Fallback activity stopped',
-		});
-	});
-
-	it('should call API client with summary when provided', async () => {
-		mockApiClient.stopFallbackActivity.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await stopFallbackActivity(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				summary: 'Reviewed 5 files, found 3 issues',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.stopFallbackActivity).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'Reviewed 5 files, found 3 issues',
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.stopFallbackActivity.mockResolvedValue({
-			ok: false,
-			error: 'Failed to stop activity',
-		});
-		const ctx = createMockContext();
-
-		const result = await stopFallbackActivity({
-			project_id: '123e4567-e89b-12d3-a456-426614174000',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to stop activity' });
-	});
-});
-
-// ============================================================================
-// getActivityHistory Tests
-// ============================================================================
-
-describe('getActivityHistory', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getActivityHistory({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getActivityHistory({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty history when no activities', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				history: [],
-				latest_by_type: {},
-				count: 0,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityHistory(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			history: [],
-			count: 0,
-		});
-	});
-
-	it('should return activity history from API', async () => {
-		const mockHistory = [
-			{
-				id: 'h1',
-				activity_type: 'code_review',
-				completed_at: '2025-01-14T10:00:00Z',
-				summary: 'Reviewed auth module',
-			},
-			{
-				id: 'h2',
-				activity_type: 'security_review',
-				completed_at: '2025-01-14T09:00:00Z',
-				summary: null,
-			},
-		];
-
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				history: mockHistory,
-				latest_by_type: { code_review: mockHistory[0] },
-				count: 2,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityHistory(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect((result.result as { history: unknown[] }).history).toHaveLength(2);
-		expect((result.result as { count: number }).count).toBe(2);
-	});
-
-	it('should call API proxy with correct parameters', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: { history: [], count: 0 },
-		});
-		const ctx = createMockContext();
-
-		await getActivityHistory(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity_type: 'security_review',
-				limit: 10,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.proxy).toHaveBeenCalledWith(
-			'get_activity_history',
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				activity_type: 'security_review',
-				limit: 10,
-				offset: 0,
-			}
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityHistory({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Query failed' });
-	});
-});
-
-// ============================================================================
-// getActivitySchedules Tests
-// ============================================================================
-
-describe('getActivitySchedules', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getActivitySchedules({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getActivitySchedules({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty schedules when none exist', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				schedules: [],
-				total_count: 0,
-				has_more: false,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivitySchedules(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			schedules: [],
-			total_count: 0,
-			has_more: false,
-		});
-	});
-
-	it('should return activity schedules from API', async () => {
-		const mockSchedules = [
-			{
-				id: 's1',
-				activity_type: 'code_review',
-				schedule_type: 'weekly',
-				enabled: true,
-				next_run_at: '2025-01-20T10:00:00Z',
-			},
-			{
-				id: 's2',
-				activity_type: 'security_review',
-				schedule_type: 'monthly',
-				enabled: true,
-				next_run_at: '2025-02-14T10:00:00Z',
-			},
-		];
-
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: {
-				schedules: mockSchedules,
-				total_count: 2,
-				has_more: false,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivitySchedules(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect((result.result as { schedules: unknown[] }).schedules).toHaveLength(2);
-		expect((result.result as { total_count: number }).total_count).toBe(2);
-		expect((result.result as { has_more: boolean }).has_more).toBe(false);
-	});
-
-	it('should call API proxy with correct parameters', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: true,
-			data: { schedules: [], total_count: 0, has_more: false },
-		});
-		const ctx = createMockContext();
-
-		await getActivitySchedules(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.proxy).toHaveBeenCalledWith(
-			'get_activity_schedules',
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000', limit: 50, offset: 0 }
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.proxy.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivitySchedules({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Query failed' });
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import {
+	startFallbackActivity,
+	stopFallbackActivity,
+	getActivityHistory,
+	getActivitySchedules,
+} from './fallback.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// startFallbackActivity Tests
+// ============================================================================
+
+describe('startFallbackActivity', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			startFallbackActivity({ activity: 'code_review' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			startFallbackActivity({ project_id: 'invalid', activity: 'code_review' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing activity', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			startFallbackActivity({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid activity type', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			startFallbackActivity({
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity: 'invalid_activity',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should start fallback activity successfully', async () => {
+		mockApiClient.startFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: { message: 'Started code_review' },
+		});
+		const ctx = createMockContext();
+
+		const result = await startFallbackActivity(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity: 'code_review',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			activity: 'code_review',
+		});
+		expect((result.result as { title: string }).title).toBeDefined();
+	});
+
+	it('should call API client with correct parameters', async () => {
+		mockApiClient.startFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: { message: 'Started' },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await startFallbackActivity(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity: 'security_review',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.startFallbackActivity).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'security_review',
+			'my-session'
+		);
+	});
+
+	it('should accept all valid activity types', async () => {
+		const validActivities = [
+			'feature_ideation',
+			'code_review',
+			'performance_audit',
+			'ux_review',
+			'cost_analysis',
+			'security_review',
+			'test_coverage',
+			'documentation_review',
+			'dependency_audit',
+			'validate_completed_tasks',
+		];
+
+		for (const activity of validActivities) {
+			mockApiClient.startFallbackActivity.mockResolvedValue({
+				ok: true,
+				data: { message: `Started ${activity}` },
+			});
+			const ctx = createMockContext();
+
+			const result = await startFallbackActivity(
+				{
+					project_id: '123e4567-e89b-12d3-a456-426614174000',
+					activity,
+				},
+				ctx
+			);
+
+			expect(result.result).toMatchObject({
+				success: true,
+				activity,
+			});
+		}
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.startFallbackActivity.mockResolvedValue({
+			ok: false,
+			error: 'Failed to start activity',
+		});
+		const ctx = createMockContext();
+
+		const result = await startFallbackActivity({
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+			activity: 'code_review',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to start activity' });
+	});
+
+	it('should pass through worktree guidance when API returns it', async () => {
+		mockApiClient.startFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				activity: 'code_review',
+				message: 'Started code_review',
+				git_workflow: {
+					workflow: 'git-flow',
+					base_branch: 'develop',
+					worktree_recommended: true,
+					note: 'Fallback activities use the base branch directly (read-only).',
+				},
+				worktree_setup: {
+					message: 'RECOMMENDED: Create a worktree to avoid conflicts.',
+					commands: [
+						'git checkout develop',
+						'git pull origin develop',
+						'git worktree add ../Project-code-review develop',
+						'cd ../Project-code-review',
+					],
+					worktree_path: '../Project-code-review',
+					branch_name: 'develop',
+					cleanup_command: 'git worktree remove ../Project-code-review',
+					report_worktree: 'heartbeat(current_worktree_path: "../Project-code-review")',
+				},
+				next_step: 'After setting up worktree: call heartbeat to report your location.',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await startFallbackActivity(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity: 'code_review',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			activity: 'code_review',
+		});
+		expect((result.result as { git_workflow?: unknown }).git_workflow).toBeDefined();
+		expect((result.result as { git_workflow: { workflow: string } }).git_workflow.workflow).toBe('git-flow');
+		expect((result.result as { worktree_setup?: unknown }).worktree_setup).toBeDefined();
+		expect((result.result as { worktree_setup: { worktree_path: string } }).worktree_setup.worktree_path).toBe('../Project-code-review');
+		expect((result.result as { next_step?: string }).next_step).toContain('heartbeat');
+	});
+
+	it('should not include worktree guidance when API does not return it', async () => {
+		mockApiClient.startFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				activity: 'code_review',
+				message: 'Started code_review',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await startFallbackActivity(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity: 'code_review',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			activity: 'code_review',
+		});
+		expect((result.result as { git_workflow?: unknown }).git_workflow).toBeUndefined();
+		expect((result.result as { worktree_setup?: unknown }).worktree_setup).toBeUndefined();
+		expect((result.result as { next_step?: string }).next_step).toBeUndefined();
+	});
+});
+
+// ============================================================================
+// stopFallbackActivity Tests
+// ============================================================================
+
+describe('stopFallbackActivity', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(stopFallbackActivity({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			stopFallbackActivity({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should stop fallback activity successfully', async () => {
+		mockApiClient.stopFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await stopFallbackActivity(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Fallback activity stopped',
+		});
+	});
+
+	it('should call API client with summary when provided', async () => {
+		mockApiClient.stopFallbackActivity.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await stopFallbackActivity(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				summary: 'Reviewed 5 files, found 3 issues',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.stopFallbackActivity).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'Reviewed 5 files, found 3 issues',
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.stopFallbackActivity.mockResolvedValue({
+			ok: false,
+			error: 'Failed to stop activity',
+		});
+		const ctx = createMockContext();
+
+		const result = await stopFallbackActivity({
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to stop activity' });
+	});
+});
+
+// ============================================================================
+// getActivityHistory Tests
+// ============================================================================
+
+describe('getActivityHistory', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getActivityHistory({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getActivityHistory({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty history when no activities', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				history: [],
+				latest_by_type: {},
+				count: 0,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityHistory(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			history: [],
+			count: 0,
+		});
+	});
+
+	it('should return activity history from API', async () => {
+		const mockHistory = [
+			{
+				id: 'h1',
+				activity_type: 'code_review',
+				completed_at: '2025-01-14T10:00:00Z',
+				summary: 'Reviewed auth module',
+			},
+			{
+				id: 'h2',
+				activity_type: 'security_review',
+				completed_at: '2025-01-14T09:00:00Z',
+				summary: null,
+			},
+		];
+
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				history: mockHistory,
+				latest_by_type: { code_review: mockHistory[0] },
+				count: 2,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityHistory(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect((result.result as { history: unknown[] }).history).toHaveLength(2);
+		expect((result.result as { count: number }).count).toBe(2);
+	});
+
+	it('should call API proxy with correct parameters', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: { history: [], count: 0 },
+		});
+		const ctx = createMockContext();
+
+		await getActivityHistory(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity_type: 'security_review',
+				limit: 10,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.proxy).toHaveBeenCalledWith(
+			'get_activity_history',
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				activity_type: 'security_review',
+				limit: 10,
+				offset: 0,
+			}
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityHistory({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Query failed' });
+	});
+});
+
+// ============================================================================
+// getActivitySchedules Tests
+// ============================================================================
+
+describe('getActivitySchedules', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getActivitySchedules({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getActivitySchedules({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty schedules when none exist', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				schedules: [],
+				total_count: 0,
+				has_more: false,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivitySchedules(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			schedules: [],
+			total_count: 0,
+			has_more: false,
+		});
+	});
+
+	it('should return activity schedules from API', async () => {
+		const mockSchedules = [
+			{
+				id: 's1',
+				activity_type: 'code_review',
+				schedule_type: 'weekly',
+				enabled: true,
+				next_run_at: '2025-01-20T10:00:00Z',
+			},
+			{
+				id: 's2',
+				activity_type: 'security_review',
+				schedule_type: 'monthly',
+				enabled: true,
+				next_run_at: '2025-02-14T10:00:00Z',
+			},
+		];
+
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: {
+				schedules: mockSchedules,
+				total_count: 2,
+				has_more: false,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivitySchedules(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect((result.result as { schedules: unknown[] }).schedules).toHaveLength(2);
+		expect((result.result as { total_count: number }).total_count).toBe(2);
+		expect((result.result as { has_more: boolean }).has_more).toBe(false);
+	});
+
+	it('should call API proxy with correct parameters', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: true,
+			data: { schedules: [], total_count: 0, has_more: false },
+		});
+		const ctx = createMockContext();
+
+		await getActivitySchedules(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.proxy).toHaveBeenCalledWith(
+			'get_activity_schedules',
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', limit: 50, offset: 0 }
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.proxy.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivitySchedules({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Query failed' });
+	});
+});