@vibescope/mcp-server 0.2.4 → 0.2.5

package/src/handlers/progress.test.ts

@@ -1,269 +1,269 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { logProgress, getActivityFeed } from './progress.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// logProgress Tests
-// ============================================================================
-
-describe('logProgress', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-	});
-
-	it('should log progress successfully', async () => {
-		mockApiClient.logProgress.mockResolvedValue({
-			ok: true,
-			data: { progress_id: 'prog-123' },
-		});
-		const ctx = createMockContext();
-
-		const result = await logProgress(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				summary: 'Completed initial setup',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			progress_id: 'prog-123',
-		});
-	});
-
-	it('should log progress with task_id and details', async () => {
-		mockApiClient.logProgress.mockResolvedValue({
-			ok: true,
-			data: { progress_id: 'prog-456' },
-		});
-		const ctx = createMockContext();
-
-		await logProgress(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				task_id: '456e4567-e89b-12d3-a456-426614174000',
-				summary: 'Working on feature',
-				details: 'Added the main component and started styling',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.logProgress).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			{
-				summary: 'Working on feature',
-				details: 'Added the main component and started styling',
-				task_id: '456e4567-e89b-12d3-a456-426614174000',
-				session_id: 'session-123',
-			}
-		);
-	});
-
-	it('should include session_id in request', async () => {
-		mockApiClient.logProgress.mockResolvedValue({
-			ok: true,
-			data: { progress_id: 'prog-789' },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await logProgress(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				summary: 'Progress update',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.logProgress).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.objectContaining({
-				session_id: 'my-session',
-			})
-		);
-	});
-
-	it('should call API client logProgress', async () => {
-		mockApiClient.logProgress.mockResolvedValue({
-			ok: true,
-			data: { progress_id: 'prog-abc' },
-		});
-		const ctx = createMockContext();
-
-		await logProgress(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				summary: 'Progress',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.logProgress).toHaveBeenCalled();
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.logProgress.mockResolvedValue({
-			ok: false,
-			error: 'Insert failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await logProgress({
-			project_id: '123e4567-e89b-12d3-a456-426614174000',
-			summary: 'Progress',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Insert failed',
-		});
-	});
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logProgress({ summary: 'Progress' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logProgress({ project_id: 'invalid', summary: 'Progress' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing summary', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			logProgress({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-});
-
-// ============================================================================
-// getActivityFeed Tests
-// ============================================================================
-
-describe('getActivityFeed', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-	});
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getActivityFeed({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getActivityFeed({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty activities when no data', async () => {
-		mockApiClient.getActivityFeed.mockResolvedValue({
-			ok: true,
-			data: { activities: [] },
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityFeed(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			activities: [],
-		});
-	});
-
-	it('should return activities from API', async () => {
-		const mockActivities = [
-			{ id: 'a1', type: 'task', created_at: '2025-01-14T10:00:00Z' },
-			{ id: 'a2', type: 'progress', created_at: '2025-01-14T11:00:00Z' },
-		];
-		mockApiClient.getActivityFeed.mockResolvedValue({
-			ok: true,
-			data: { activities: mockActivities },
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityFeed(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			activities: mockActivities,
-		});
-	});
-
-	it('should pass limit parameter to API', async () => {
-		mockApiClient.getActivityFeed.mockResolvedValue({
-			ok: true,
-			data: { activities: [] },
-		});
-		const ctx = createMockContext();
-
-		await getActivityFeed(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				limit: 10,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.getActivityFeed).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.objectContaining({ limit: 10 })
-		);
-	});
-
-	it('should cap limit at 200', async () => {
-		mockApiClient.getActivityFeed.mockResolvedValue({
-			ok: true,
-			data: { activities: [] },
-		});
-		const ctx = createMockContext();
-
-		await getActivityFeed(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				limit: 500,
-			},
-			ctx
-		);
-
-		expect(mockApiClient.getActivityFeed).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.objectContaining({ limit: 200 })
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getActivityFeed.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getActivityFeed(
-			{ 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 { logProgress, getActivityFeed } from './progress.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// logProgress Tests
+// ============================================================================
+
+describe('logProgress', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should log progress successfully', async () => {
+		mockApiClient.logProgress.mockResolvedValue({
+			ok: true,
+			data: { progress_id: 'prog-123' },
+		});
+		const ctx = createMockContext();
+
+		const result = await logProgress(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				summary: 'Completed initial setup',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			progress_id: 'prog-123',
+		});
+	});
+
+	it('should log progress with task_id and details', async () => {
+		mockApiClient.logProgress.mockResolvedValue({
+			ok: true,
+			data: { progress_id: 'prog-456' },
+		});
+		const ctx = createMockContext();
+
+		await logProgress(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				task_id: '456e4567-e89b-12d3-a456-426614174000',
+				summary: 'Working on feature',
+				details: 'Added the main component and started styling',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.logProgress).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			{
+				summary: 'Working on feature',
+				details: 'Added the main component and started styling',
+				task_id: '456e4567-e89b-12d3-a456-426614174000',
+				session_id: 'session-123',
+			}
+		);
+	});
+
+	it('should include session_id in request', async () => {
+		mockApiClient.logProgress.mockResolvedValue({
+			ok: true,
+			data: { progress_id: 'prog-789' },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await logProgress(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				summary: 'Progress update',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.logProgress).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.objectContaining({
+				session_id: 'my-session',
+			})
+		);
+	});
+
+	it('should call API client logProgress', async () => {
+		mockApiClient.logProgress.mockResolvedValue({
+			ok: true,
+			data: { progress_id: 'prog-abc' },
+		});
+		const ctx = createMockContext();
+
+		await logProgress(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				summary: 'Progress',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.logProgress).toHaveBeenCalled();
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.logProgress.mockResolvedValue({
+			ok: false,
+			error: 'Insert failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await logProgress({
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+			summary: 'Progress',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Insert failed',
+		});
+	});
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logProgress({ summary: 'Progress' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logProgress({ project_id: 'invalid', summary: 'Progress' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing summary', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			logProgress({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+});
+
+// ============================================================================
+// getActivityFeed Tests
+// ============================================================================
+
+describe('getActivityFeed', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getActivityFeed({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getActivityFeed({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty activities when no data', async () => {
+		mockApiClient.getActivityFeed.mockResolvedValue({
+			ok: true,
+			data: { activities: [] },
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityFeed(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			activities: [],
+		});
+	});
+
+	it('should return activities from API', async () => {
+		const mockActivities = [
+			{ id: 'a1', type: 'task', created_at: '2025-01-14T10:00:00Z' },
+			{ id: 'a2', type: 'progress', created_at: '2025-01-14T11:00:00Z' },
+		];
+		mockApiClient.getActivityFeed.mockResolvedValue({
+			ok: true,
+			data: { activities: mockActivities },
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityFeed(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			activities: mockActivities,
+		});
+	});
+
+	it('should pass limit parameter to API', async () => {
+		mockApiClient.getActivityFeed.mockResolvedValue({
+			ok: true,
+			data: { activities: [] },
+		});
+		const ctx = createMockContext();
+
+		await getActivityFeed(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				limit: 10,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.getActivityFeed).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.objectContaining({ limit: 10 })
+		);
+	});
+
+	it('should cap limit at 200', async () => {
+		mockApiClient.getActivityFeed.mockResolvedValue({
+			ok: true,
+			data: { activities: [] },
+		});
+		const ctx = createMockContext();
+
+		await getActivityFeed(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				limit: 500,
+			},
+			ctx
+		);
+
+		expect(mockApiClient.getActivityFeed).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.objectContaining({ limit: 200 })
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getActivityFeed.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getActivityFeed(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});