@vibescope/mcp-server 0.4.5 → 0.4.6

package/src/handlers/chat.test.ts

@@ -1,185 +1,185 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { sendProjectMessage, getProjectMessages, autoPostActivity } from './chat.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-
-// vi.mock is hoisted above imports by Vitest — this intercepts the api-client module
-const mockSendProjectMessage = vi.fn();
-const mockGetProjectMessages = vi.fn();
-
-vi.mock('../api-client.js', () => ({
-	getApiClient: () => ({
-		sendProjectMessage: mockSendProjectMessage,
-		getProjectMessages: mockGetProjectMessages,
-	}),
-	initApiClient: vi.fn(),
-	VibescopeApiClient: vi.fn(),
-}));
-
-const VALID_PROJECT_ID = '123e4567-e89b-12d3-a456-426614174000';
-
-// ============================================================================
-// sendProjectMessage Tests
-// ============================================================================
-
-describe('sendProjectMessage', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
-		mockGetProjectMessages.mockResolvedValue({ ok: true, data: { messages: [], count: 0 } });
-	});
-
-	it('should throw for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			sendProjectMessage({ message: 'Hello' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-		await expect(
-			sendProjectMessage({ project_id: 'not-a-uuid', message: 'Hello' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw for missing message', async () => {
-		const ctx = createMockContext();
-		await expect(
-			sendProjectMessage({ project_id: VALID_PROJECT_ID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should send message successfully', async () => {
-		mockSendProjectMessage.mockResolvedValue({
-			ok: true,
-			data: { success: true, message_id: 'msg-1', sent_at: '2026-02-22T12:00:00Z' },
-		});
-		const ctx = createMockContext({ sessionId: 'session-abc' });
-
-		const result = await sendProjectMessage(
-			{ project_id: VALID_PROJECT_ID, message: 'Deployment complete' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ success: true, message_id: 'msg-1' });
-		expect(mockSendProjectMessage).toHaveBeenCalledWith(
-			VALID_PROJECT_ID,
-			'Deployment complete',
-			'session-abc'
-		);
-	});
-
-	it('should return error when API fails', async () => {
-		mockSendProjectMessage.mockResolvedValue({
-			ok: false,
-			error: 'Project not found',
-		});
-		const ctx = createMockContext();
-
-		const result = await sendProjectMessage(
-			{ project_id: VALID_PROJECT_ID, message: 'Hi' },
-			ctx
-		);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Project not found' });
-	});
-});
-
-// ============================================================================
-// getProjectMessages Tests
-// ============================================================================
-
-describe('getProjectMessages', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
-		mockGetProjectMessages.mockResolvedValue({ ok: true, data: { messages: [], count: 0 } });
-	});
-
-	it('should throw for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			getProjectMessages({}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should fetch messages successfully', async () => {
-		const mockMessages = [
-			{ id: 'msg-1', sender_type: 'agent', sender_name: 'Garath', content: 'Started session', created_at: '2026-02-22T12:00:00Z' },
-		];
-		mockGetProjectMessages.mockResolvedValue({
-			ok: true,
-			data: { messages: mockMessages, count: 1 },
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectMessages({ project_id: VALID_PROJECT_ID }, ctx);
-
-		expect(result.result).toMatchObject({ messages: mockMessages, count: 1 });
-		expect(mockGetProjectMessages).toHaveBeenCalledWith(VALID_PROJECT_ID, undefined);
-	});
-
-	it('should pass limit when provided', async () => {
-		const ctx = createMockContext();
-
-		await getProjectMessages({ project_id: VALID_PROJECT_ID, limit: 5 }, ctx);
-
-		expect(mockGetProjectMessages).toHaveBeenCalledWith(VALID_PROJECT_ID, 5);
-	});
-
-	it('should return error when API fails', async () => {
-		mockGetProjectMessages.mockResolvedValue({ ok: false, error: 'Unauthorized' });
-		const ctx = createMockContext();
-
-		const result = await getProjectMessages({ project_id: VALID_PROJECT_ID }, ctx);
-
-		expect(result.isError).toBe(true);
-	});
-});
-
-// ============================================================================
-// autoPostActivity Tests
-// ============================================================================
-
-describe('autoPostActivity', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
-	});
-
-	it('should call sendProjectMessage with correct args', async () => {
-		await autoPostActivity(VALID_PROJECT_ID, 'Agent started work', 'session-123');
-
-		expect(mockSendProjectMessage).toHaveBeenCalledWith(
-			VALID_PROJECT_ID,
-			'Agent started work',
-			'session-123'
-		);
-	});
-
-	it('should silently ignore errors (never throws)', async () => {
-		mockSendProjectMessage.mockRejectedValue(new Error('Network error'));
-
-		// Should not throw
-		await expect(
-			autoPostActivity(VALID_PROJECT_ID, 'message', 'session-123')
-		).resolves.toBeUndefined();
-	});
-
-	it('should do nothing when projectId is empty', async () => {
-		await autoPostActivity('', 'message', 'session-123');
-		expect(mockSendProjectMessage).not.toHaveBeenCalled();
-	});
-
-	it('should work without sessionId', async () => {
-		await autoPostActivity(VALID_PROJECT_ID, 'no session message');
-
-		expect(mockSendProjectMessage).toHaveBeenCalledWith(
-			VALID_PROJECT_ID,
-			'no session message',
-			undefined
-		);
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { sendProjectMessage, getProjectMessages, autoPostActivity } from './chat.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+
+// vi.mock is hoisted above imports by Vitest — this intercepts the api-client module
+const mockSendProjectMessage = vi.fn();
+const mockGetProjectMessages = vi.fn();
+
+vi.mock('../api-client.js', () => ({
+	getApiClient: () => ({
+		sendProjectMessage: mockSendProjectMessage,
+		getProjectMessages: mockGetProjectMessages,
+	}),
+	initApiClient: vi.fn(),
+	VibescopeApiClient: vi.fn(),
+}));
+
+const VALID_PROJECT_ID = '123e4567-e89b-12d3-a456-426614174000';
+
+// ============================================================================
+// sendProjectMessage Tests
+// ============================================================================
+
+describe('sendProjectMessage', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
+		mockGetProjectMessages.mockResolvedValue({ ok: true, data: { messages: [], count: 0 } });
+	});
+
+	it('should throw for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			sendProjectMessage({ message: 'Hello' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+		await expect(
+			sendProjectMessage({ project_id: 'not-a-uuid', message: 'Hello' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw for missing message', async () => {
+		const ctx = createMockContext();
+		await expect(
+			sendProjectMessage({ project_id: VALID_PROJECT_ID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should send message successfully', async () => {
+		mockSendProjectMessage.mockResolvedValue({
+			ok: true,
+			data: { success: true, message_id: 'msg-1', sent_at: '2026-02-22T12:00:00Z' },
+		});
+		const ctx = createMockContext({ sessionId: 'session-abc' });
+
+		const result = await sendProjectMessage(
+			{ project_id: VALID_PROJECT_ID, message: 'Deployment complete' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ success: true, message_id: 'msg-1' });
+		expect(mockSendProjectMessage).toHaveBeenCalledWith(
+			VALID_PROJECT_ID,
+			'Deployment complete',
+			'session-abc'
+		);
+	});
+
+	it('should return error when API fails', async () => {
+		mockSendProjectMessage.mockResolvedValue({
+			ok: false,
+			error: 'Project not found',
+		});
+		const ctx = createMockContext();
+
+		const result = await sendProjectMessage(
+			{ project_id: VALID_PROJECT_ID, message: 'Hi' },
+			ctx
+		);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Project not found' });
+	});
+});
+
+// ============================================================================
+// getProjectMessages Tests
+// ============================================================================
+
+describe('getProjectMessages', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
+		mockGetProjectMessages.mockResolvedValue({ ok: true, data: { messages: [], count: 0 } });
+	});
+
+	it('should throw for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			getProjectMessages({}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should fetch messages successfully', async () => {
+		const mockMessages = [
+			{ id: 'msg-1', sender_type: 'agent', sender_name: 'Garath', content: 'Started session', created_at: '2026-02-22T12:00:00Z' },
+		];
+		mockGetProjectMessages.mockResolvedValue({
+			ok: true,
+			data: { messages: mockMessages, count: 1 },
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectMessages({ project_id: VALID_PROJECT_ID }, ctx);
+
+		expect(result.result).toMatchObject({ messages: mockMessages, count: 1 });
+		expect(mockGetProjectMessages).toHaveBeenCalledWith(VALID_PROJECT_ID, undefined);
+	});
+
+	it('should pass limit when provided', async () => {
+		const ctx = createMockContext();
+
+		await getProjectMessages({ project_id: VALID_PROJECT_ID, limit: 5 }, ctx);
+
+		expect(mockGetProjectMessages).toHaveBeenCalledWith(VALID_PROJECT_ID, 5);
+	});
+
+	it('should return error when API fails', async () => {
+		mockGetProjectMessages.mockResolvedValue({ ok: false, error: 'Unauthorized' });
+		const ctx = createMockContext();
+
+		const result = await getProjectMessages({ project_id: VALID_PROJECT_ID }, ctx);
+
+		expect(result.isError).toBe(true);
+	});
+});
+
+// ============================================================================
+// autoPostActivity Tests
+// ============================================================================
+
+describe('autoPostActivity', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+		mockSendProjectMessage.mockResolvedValue({ ok: true, data: {} });
+	});
+
+	it('should call sendProjectMessage with correct args', async () => {
+		await autoPostActivity(VALID_PROJECT_ID, 'Agent started work', 'session-123');
+
+		expect(mockSendProjectMessage).toHaveBeenCalledWith(
+			VALID_PROJECT_ID,
+			'Agent started work',
+			'session-123'
+		);
+	});
+
+	it('should silently ignore errors (never throws)', async () => {
+		mockSendProjectMessage.mockRejectedValue(new Error('Network error'));
+
+		// Should not throw
+		await expect(
+			autoPostActivity(VALID_PROJECT_ID, 'message', 'session-123')
+		).resolves.toBeUndefined();
+	});
+
+	it('should do nothing when projectId is empty', async () => {
+		await autoPostActivity('', 'message', 'session-123');
+		expect(mockSendProjectMessage).not.toHaveBeenCalled();
+	});
+
+	it('should work without sessionId', async () => {
+		await autoPostActivity(VALID_PROJECT_ID, 'no session message');
+
+		expect(mockSendProjectMessage).toHaveBeenCalledWith(
+			VALID_PROJECT_ID,
+			'no session message',
+			undefined
+		);
+	});
+});

package/src/handlers/chat.ts

@@ -1,69 +1,101 @@
-/**
- * Chat handlers
- *
- * Handlers for project chat tools:
- * - send_project_message: Post a message to the project chat
- * - get_project_messages: Read recent messages from project chat
- *
- * Also exports autoPostActivity — a fire-and-forget helper used by other
- * handlers to log agent boot progress and key events to the chat window.
- */
-
-import type { Handler, HandlerRegistry } from './types.js';
-import { parseArgs, uuidValidator } from '../validators.js';
-import { getApiClient } from '../api-client.js';
-
-const sendProjectMessageSchema = {
-	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-	message: { type: 'string' as const, required: true as const },
-};
-
-const getProjectMessagesSchema = {
-	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
-	limit: { type: 'number' as const },
-};
-
-export const sendProjectMessage: Handler = async (args, ctx) => {
-	const { project_id, message } = parseArgs(args, sendProjectMessageSchema);
-	const api = getApiClient();
-	const response = await api.sendProjectMessage(project_id, message, ctx.session.currentSessionId || undefined);
-	if (!response.ok) {
-		return { result: { error: response.error || 'Failed to send message' }, isError: true };
-	}
-	return { result: response.data };
-};
-
-export const getProjectMessages: Handler = async (args, _ctx) => {
-	const { project_id, limit } = parseArgs(args, getProjectMessagesSchema);
-	const api = getApiClient();
-	const response = await api.getProjectMessages(project_id, limit);
-	if (!response.ok) {
-		return { result: { error: response.error || 'Failed to fetch messages' }, isError: true };
-	}
-	return { result: response.data };
-};
-
-/**
- * Auto-post an agent activity message to the project chat.
- *
- * Fire-and-forget: errors are silently swallowed so auto-posting never
- * interrupts or breaks the main tool response.
- */
-export async function autoPostActivity(
-	projectId: string,
-	message: string,
-	sessionId?: string | null
-): Promise<void> {
-	if (!projectId) return;
-	try {
-		const api = getApiClient();
-		await api.sendProjectMessage(projectId, message, sessionId ?? undefined);
-	} catch {
-		// Silently ignore — activity logging must never break main functionality
-	}
-}
-
-export const chatHandlers: HandlerRegistry = {
-	send_project_message: sendProjectMessage,
-	get_project_messages: getProjectMessages,
-};
+/**
+ * Chat handlers
+ *
+ * Handlers for project chat tools:
+ * - send_project_message: Post a message to the project chat
+ * - get_project_messages: Read recent messages from project chat
+ * - post_progress: Post a structured progress update with a type icon
+ *
+ * Also exports autoPostActivity — a fire-and-forget helper used by other
+ * handlers to log agent boot progress and key events to the chat window.
+ */
+
+import type { Handler, HandlerRegistry } from './types.js';
+import { parseArgs, uuidValidator } from '../validators.js';
+import { getApiClient } from '../api-client.js';
+
+const sendProjectMessageSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	message: { type: 'string' as const, required: true as const },
+};
+
+const getProjectMessagesSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	limit: { type: 'number' as const },
+};
+
+const postProgressSchema = {
+	project_id: { type: 'string' as const, required: true as const, validate: uuidValidator },
+	message: { type: 'string' as const, required: true as const },
+	type: { type: 'string' as const },
+};
+
+const PROGRESS_TYPE_ICONS: Record<string, string> = {
+	info: 'ℹ️',
+	milestone: '🏁',
+	blocker: '🚫',
+	question: '❓',
+};
+
+export const sendProjectMessage: Handler = async (args, ctx) => {
+	const { project_id, message } = parseArgs(args, sendProjectMessageSchema);
+	const api = getApiClient();
+	const response = await api.sendProjectMessage(project_id, message, ctx.session.currentSessionId || undefined);
+	if (!response.ok) {
+		return { result: { error: response.error || 'Failed to send message' }, isError: true };
+	}
+	return { result: response.data };
+};
+
+export const postProgress: Handler = async (args, ctx) => {
+	const { project_id, message, type } = parseArgs(args, postProgressSchema);
+	const progressType = (type as string) || 'info';
+	const icon = PROGRESS_TYPE_ICONS[progressType] ?? PROGRESS_TYPE_ICONS['info'];
+	const formattedMessage = `${icon} **[${progressType.toUpperCase()}]** ${message}`;
+	const api = getApiClient();
+	const response = await api.sendProjectMessage(
+		project_id,
+		formattedMessage,
+		ctx.session.currentSessionId || undefined
+	);
+	if (!response.ok) {
+		return { result: { error: response.error || 'Failed to post progress update' }, isError: true };
+	}
+	return { result: { ...response.data, type: progressType } };
+};
+
+export const getProjectMessages: Handler = async (args, _ctx) => {
+	const { project_id, limit } = parseArgs(args, getProjectMessagesSchema);
+	const api = getApiClient();
+	const response = await api.getProjectMessages(project_id, limit);
+	if (!response.ok) {
+		return { result: { error: response.error || 'Failed to fetch messages' }, isError: true };
+	}
+	return { result: response.data };
+};
+
+/**
+ * Auto-post an agent activity message to the project chat.
+ *
+ * Fire-and-forget: errors are silently swallowed so auto-posting never
+ * interrupts or breaks the main tool response.
+ */
+export async function autoPostActivity(
+	projectId: string,
+	message: string,
+	sessionId?: string | null
+): Promise<void> {
+	if (!projectId) return;
+	try {
+		const api = getApiClient();
+		await api.sendProjectMessage(projectId, message, sessionId ?? undefined);
+	} catch {
+		// Silently ignore — activity logging must never break main functionality
+	}
+}
+
+export const chatHandlers: HandlerRegistry = {
+	send_project_message: sendProjectMessage,
+	get_project_messages: getProjectMessages,
+	post_progress: postProgress,
+};