@vibescope/mcp-server 0.5.0 → 0.5.1

package/src/handlers/blockers.test.ts

@@ -1,468 +1,468 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-	addBlocker,
-	resolveBlocker,
-	getBlockers,
-	getBlockersStats,
-	deleteBlocker,
-} from './blockers.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// addBlocker Tests
-// ============================================================================
-
-describe('addBlocker', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addBlocker({ description: 'Some blocker' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addBlocker({ project_id: 'invalid', description: 'Some blocker' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing description', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			addBlocker({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should add blocker successfully', async () => {
-		mockApiClient.addBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true, blocker_id: 'blocker-1' },
-		});
-		const ctx = createMockContext();
-
-		const result = await addBlocker(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				description: 'API is down',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			blocker_id: 'blocker-1',
-		});
-	});
-
-	it('should include session_id in API call', async () => {
-		mockApiClient.addBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true, blocker_id: 'blocker-1' },
-		});
-		const ctx = createMockContext({ sessionId: 'my-session' });
-
-		await addBlocker(
-			{
-				project_id: '123e4567-e89b-12d3-a456-426614174000',
-				description: 'Blocked by dependency',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.addBlocker).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'Blocked by dependency',
-			'my-session'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.addBlocker.mockResolvedValue({
-			ok: false,
-			error: 'Insert failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await addBlocker({
-			project_id: '123e4567-e89b-12d3-a456-426614174000',
-			description: 'Test blocker',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Insert failed',
-		});
-	});
-});
-
-// ============================================================================
-// resolveBlocker Tests
-// ============================================================================
-
-describe('resolveBlocker', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing blocker_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(resolveBlocker({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid blocker_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			resolveBlocker({ blocker_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should resolve blocker successfully', async () => {
-		mockApiClient.resolveBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-		});
-		const ctx = createMockContext();
-
-		const result = await resolveBlocker(
-			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			blocker_id: '123e4567-e89b-12d3-a456-426614174000',
-		});
-	});
-
-	it('should resolve blocker with resolution note', async () => {
-		mockApiClient.resolveBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-		});
-		const ctx = createMockContext();
-
-		await resolveBlocker(
-			{
-				blocker_id: '123e4567-e89b-12d3-a456-426614174000',
-				resolution_note: 'Fixed by upgrading dependency',
-			},
-			ctx
-		);
-
-		expect(mockApiClient.resolveBlocker).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			'Fixed by upgrading dependency'
-		);
-	});
-
-	it('should call API client resolveBlocker', async () => {
-		mockApiClient.resolveBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-		});
-		const ctx = createMockContext();
-
-		await resolveBlocker(
-			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.resolveBlocker).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			undefined
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.resolveBlocker.mockResolvedValue({
-			ok: false,
-			error: 'Update failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await resolveBlocker({ blocker_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Update failed',
-		});
-	});
-});
-
-// ============================================================================
-// getBlockers Tests
-// ============================================================================
-
-describe('getBlockers', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getBlockers({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getBlockers({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty list when no blockers', async () => {
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: true,
-			data: { blockers: [] },
-		});
-		const ctx = createMockContext();
-
-		const result = await getBlockers(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			blockers: [],
-		});
-	});
-
-	it('should return blockers list', async () => {
-		const mockBlockers = [
-			{ id: 'b1', description: 'Blocker 1', status: 'open', created_at: '2025-01-14T10:00:00Z' },
-			{ id: 'b2', description: 'Blocker 2', status: 'open', created_at: '2025-01-14T11:00:00Z' },
-		];
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: true,
-			data: { blockers: mockBlockers },
-		});
-		const ctx = createMockContext();
-
-		const result = await getBlockers(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect((result.result as { blockers: unknown[] }).blockers).toHaveLength(2);
-	});
-
-	it('should pass status parameter to API (default: open)', async () => {
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: true,
-			data: { blockers: [] },
-		});
-		const ctx = createMockContext();
-
-		await getBlockers(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getBlockers).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.objectContaining({ status: 'open' })
-		);
-	});
-
-	it('should pass custom status to API', async () => {
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: true,
-			data: { blockers: [] },
-		});
-		const ctx = createMockContext();
-
-		await getBlockers(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'resolved' },
-			ctx
-		);
-
-		expect(mockApiClient.getBlockers).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000',
-			expect.objectContaining({ status: 'resolved' })
-		);
-	});
-
-	it('should call API client getBlockers', async () => {
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: true,
-			data: { blockers: [] },
-		});
-		const ctx = createMockContext();
-
-		await getBlockers(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getBlockers).toHaveBeenCalled();
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getBlockers.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getBlockers({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Query failed',
-		});
-	});
-});
-
-// ============================================================================
-// deleteBlocker Tests
-// ============================================================================
-
-describe('deleteBlocker', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing blocker_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(deleteBlocker({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid blocker_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			deleteBlocker({ blocker_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should delete blocker successfully', async () => {
-		mockApiClient.deleteBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteBlocker(
-			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-		});
-	});
-
-	it('should call API client deleteBlocker', async () => {
-		mockApiClient.deleteBlocker.mockResolvedValue({
-			ok: true,
-			data: { success: true },
-		});
-		const ctx = createMockContext();
-
-		await deleteBlocker(
-			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.deleteBlocker).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.deleteBlocker.mockResolvedValue({
-			ok: false,
-			error: 'Delete failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteBlocker({ blocker_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({
-			error: 'Delete failed',
-		});
-	});
-});
-
-// ============================================================================
-// getBlockersStats Tests
-// ============================================================================
-
-describe('getBlockersStats', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getBlockersStats({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getBlockersStats({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return blockers stats', async () => {
-		mockApiClient.getBlockersStats.mockResolvedValue({
-			ok: true,
-			data: {
-				total: 5,
-				by_status: { open: 3, resolved: 2 },
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getBlockersStats(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			total: 5,
-			by_status: { open: 3, resolved: 2 },
-		});
-	});
-
-	it('should call API client getBlockersStats', async () => {
-		mockApiClient.getBlockersStats.mockResolvedValue({
-			ok: true,
-			data: { total: 0, by_status: {} },
-		});
-		const ctx = createMockContext();
-
-		await getBlockersStats(
-			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
-			ctx
-		);
-
-		expect(mockApiClient.getBlockersStats).toHaveBeenCalledWith(
-			'123e4567-e89b-12d3-a456-426614174000'
-		);
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.getBlockersStats.mockResolvedValue({
-			ok: false,
-			error: 'Query failed',
-		});
-		const ctx = createMockContext();
-
-		const result = await getBlockersStats({ 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 {
+	addBlocker,
+	resolveBlocker,
+	getBlockers,
+	getBlockersStats,
+	deleteBlocker,
+} from './blockers.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// addBlocker Tests
+// ============================================================================
+
+describe('addBlocker', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addBlocker({ description: 'Some blocker' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addBlocker({ project_id: 'invalid', description: 'Some blocker' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing description', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			addBlocker({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should add blocker successfully', async () => {
+		mockApiClient.addBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true, blocker_id: 'blocker-1' },
+		});
+		const ctx = createMockContext();
+
+		const result = await addBlocker(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				description: 'API is down',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			blocker_id: 'blocker-1',
+		});
+	});
+
+	it('should include session_id in API call', async () => {
+		mockApiClient.addBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true, blocker_id: 'blocker-1' },
+		});
+		const ctx = createMockContext({ sessionId: 'my-session' });
+
+		await addBlocker(
+			{
+				project_id: '123e4567-e89b-12d3-a456-426614174000',
+				description: 'Blocked by dependency',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.addBlocker).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'Blocked by dependency',
+			'my-session'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.addBlocker.mockResolvedValue({
+			ok: false,
+			error: 'Insert failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await addBlocker({
+			project_id: '123e4567-e89b-12d3-a456-426614174000',
+			description: 'Test blocker',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Insert failed',
+		});
+	});
+});
+
+// ============================================================================
+// resolveBlocker Tests
+// ============================================================================
+
+describe('resolveBlocker', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing blocker_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(resolveBlocker({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid blocker_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			resolveBlocker({ blocker_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should resolve blocker successfully', async () => {
+		mockApiClient.resolveBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+		});
+		const ctx = createMockContext();
+
+		const result = await resolveBlocker(
+			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			blocker_id: '123e4567-e89b-12d3-a456-426614174000',
+		});
+	});
+
+	it('should resolve blocker with resolution note', async () => {
+		mockApiClient.resolveBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+		});
+		const ctx = createMockContext();
+
+		await resolveBlocker(
+			{
+				blocker_id: '123e4567-e89b-12d3-a456-426614174000',
+				resolution_note: 'Fixed by upgrading dependency',
+			},
+			ctx
+		);
+
+		expect(mockApiClient.resolveBlocker).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			'Fixed by upgrading dependency'
+		);
+	});
+
+	it('should call API client resolveBlocker', async () => {
+		mockApiClient.resolveBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true, blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+		});
+		const ctx = createMockContext();
+
+		await resolveBlocker(
+			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.resolveBlocker).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			undefined
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.resolveBlocker.mockResolvedValue({
+			ok: false,
+			error: 'Update failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await resolveBlocker({ blocker_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Update failed',
+		});
+	});
+});
+
+// ============================================================================
+// getBlockers Tests
+// ============================================================================
+
+describe('getBlockers', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getBlockers({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getBlockers({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty list when no blockers', async () => {
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: true,
+			data: { blockers: [] },
+		});
+		const ctx = createMockContext();
+
+		const result = await getBlockers(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			blockers: [],
+		});
+	});
+
+	it('should return blockers list', async () => {
+		const mockBlockers = [
+			{ id: 'b1', description: 'Blocker 1', status: 'open', created_at: '2025-01-14T10:00:00Z' },
+			{ id: 'b2', description: 'Blocker 2', status: 'open', created_at: '2025-01-14T11:00:00Z' },
+		];
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: true,
+			data: { blockers: mockBlockers },
+		});
+		const ctx = createMockContext();
+
+		const result = await getBlockers(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect((result.result as { blockers: unknown[] }).blockers).toHaveLength(2);
+	});
+
+	it('should pass status parameter to API (default: open)', async () => {
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: true,
+			data: { blockers: [] },
+		});
+		const ctx = createMockContext();
+
+		await getBlockers(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getBlockers).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.objectContaining({ status: 'open' })
+		);
+	});
+
+	it('should pass custom status to API', async () => {
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: true,
+			data: { blockers: [] },
+		});
+		const ctx = createMockContext();
+
+		await getBlockers(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000', status: 'resolved' },
+			ctx
+		);
+
+		expect(mockApiClient.getBlockers).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000',
+			expect.objectContaining({ status: 'resolved' })
+		);
+	});
+
+	it('should call API client getBlockers', async () => {
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: true,
+			data: { blockers: [] },
+		});
+		const ctx = createMockContext();
+
+		await getBlockers(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getBlockers).toHaveBeenCalled();
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getBlockers.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getBlockers({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});
+
+// ============================================================================
+// deleteBlocker Tests
+// ============================================================================
+
+describe('deleteBlocker', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing blocker_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(deleteBlocker({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid blocker_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			deleteBlocker({ blocker_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should delete blocker successfully', async () => {
+		mockApiClient.deleteBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteBlocker(
+			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+	});
+
+	it('should call API client deleteBlocker', async () => {
+		mockApiClient.deleteBlocker.mockResolvedValue({
+			ok: true,
+			data: { success: true },
+		});
+		const ctx = createMockContext();
+
+		await deleteBlocker(
+			{ blocker_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.deleteBlocker).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.deleteBlocker.mockResolvedValue({
+			ok: false,
+			error: 'Delete failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteBlocker({ blocker_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Delete failed',
+		});
+	});
+});
+
+// ============================================================================
+// getBlockersStats Tests
+// ============================================================================
+
+describe('getBlockersStats', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getBlockersStats({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getBlockersStats({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return blockers stats', async () => {
+		mockApiClient.getBlockersStats.mockResolvedValue({
+			ok: true,
+			data: {
+				total: 5,
+				by_status: { open: 3, resolved: 2 },
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getBlockersStats(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			total: 5,
+			by_status: { open: 3, resolved: 2 },
+		});
+	});
+
+	it('should call API client getBlockersStats', async () => {
+		mockApiClient.getBlockersStats.mockResolvedValue({
+			ok: true,
+			data: { total: 0, by_status: {} },
+		});
+		const ctx = createMockContext();
+
+		await getBlockersStats(
+			{ project_id: '123e4567-e89b-12d3-a456-426614174000' },
+			ctx
+		);
+
+		expect(mockApiClient.getBlockersStats).toHaveBeenCalledWith(
+			'123e4567-e89b-12d3-a456-426614174000'
+		);
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.getBlockersStats.mockResolvedValue({
+			ok: false,
+			error: 'Query failed',
+		});
+		const ctx = createMockContext();
+
+		const result = await getBlockersStats({ project_id: '123e4567-e89b-12d3-a456-426614174000' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({
+			error: 'Query failed',
+		});
+	});
+});