@vibescope/mcp-server 0.2.8 → 0.3.0

package/src/handlers/project.test.ts

@@ -1,546 +1,546 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-	getProjectContext,
-	getGitWorkflow,
-	createProject,
-	updateProject,
-	updateProjectReadme,
-} from './project.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-const VALID_UUID = '123e4567-e89b-12d3-a456-426614174000';
-
-// ============================================================================
-// getProjectContext Tests
-// ============================================================================
-
-describe('getProjectContext', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should list all projects when no project_id or git_url', async () => {
-		const mockProjects = [
-			{ id: 'proj-1', name: 'Project 1', description: 'Desc 1', status: 'active', git_url: null },
-			{ id: 'proj-2', name: 'Project 2', description: 'Desc 2', status: 'active', git_url: 'https://github.com/test' },
-		];
-
-		mockApiClient.listProjects.mockResolvedValue({
-			ok: true,
-			data: { projects: mockProjects },
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectContext({}, ctx);
-
-		expect(result.result).toHaveProperty('projects');
-		expect((result.result as { projects: unknown[] }).projects).toEqual(mockProjects);
-		expect(mockApiClient.listProjects).toHaveBeenCalled();
-	});
-
-	it('should return found: false when project not found', async () => {
-		mockApiClient.getProject.mockResolvedValue({
-			ok: true,
-			data: {
-				found: false,
-				message: 'Project not found. Use create_project to create one.',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectContext(
-			{ project_id: VALID_UUID },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			found: false,
-			message: 'Project not found. Use create_project to create one.',
-		});
-	});
-
-	it('should find project by git_url', async () => {
-		mockApiClient.getProject.mockResolvedValue({
-			ok: true,
-			data: {
-				found: true,
-				project: {
-					id: 'proj-1',
-					name: 'Test Project',
-					description: null,
-					goal: null,
-					status: 'active',
-					git_url: 'https://github.com/test',
-					agent_instructions: null,
-					tech_stack: null,
-				},
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectContext(
-			{ git_url: 'https://github.com/test' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			found: true,
-		});
-		expect(mockApiClient.getProject).toHaveBeenCalledWith(
-			'by-git-url',
-			'https://github.com/test'
-		);
-	});
-
-	it('should include active_tasks in result', async () => {
-		mockApiClient.getProject.mockResolvedValue({
-			ok: true,
-			data: {
-				found: true,
-				project: {
-					id: 'proj-1',
-					name: 'Test Project',
-					description: null,
-					goal: null,
-					status: 'active',
-					git_url: null,
-					agent_instructions: null,
-					tech_stack: null,
-				},
-				active_tasks: [],
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectContext(
-			{ project_id: VALID_UUID },
-			ctx
-		);
-
-		expect(result.result).toHaveProperty('active_tasks');
-	});
-
-	it('should return error when API call fails', async () => {
-		mockApiClient.listProjects.mockResolvedValue({
-			ok: false,
-			error: 'Database error',
-		});
-		const ctx = createMockContext();
-
-		const result = await getProjectContext({}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Database error' });
-	});
-});
-
-// ============================================================================
-// getGitWorkflow Tests
-// ============================================================================
-
-describe('getGitWorkflow', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(getGitWorkflow({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			getGitWorkflow({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return workflow instructions for github-flow', async () => {
-		mockApiClient.getGitWorkflow.mockResolvedValue({
-			ok: true,
-			data: {
-				workflow: 'github-flow',
-				main_branch: 'main',
-				develop_branch: null,
-				auto_branch: true,
-				auto_tag: false,
-				git_url: 'https://github.com/test',
-				instructions: [
-					'Create feature branch from main',
-					'Make changes and commit',
-					'Create PR to main',
-				],
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getGitWorkflow(
-			{ project_id: VALID_UUID },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			workflow: 'github-flow',
-			main_branch: 'main',
-			auto_branch: true,
-		});
-		expect((result.result as { instructions: string[] }).instructions.length).toBeGreaterThan(0);
-	});
-
-	it('should return workflow instructions for git-flow', async () => {
-		mockApiClient.getGitWorkflow.mockResolvedValue({
-			ok: true,
-			data: {
-				workflow: 'git-flow',
-				main_branch: 'main',
-				develop_branch: 'develop',
-				auto_branch: false,
-				auto_tag: true,
-				git_url: 'https://github.com/test',
-				instructions: ['Create feature branch from develop'],
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getGitWorkflow(
-			{ project_id: VALID_UUID },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			workflow: 'git-flow',
-			main_branch: 'main',
-			develop_branch: 'develop',
-		});
-	});
-
-	it('should return no workflow instructions when workflow is none', async () => {
-		mockApiClient.getGitWorkflow.mockResolvedValue({
-			ok: true,
-			data: {
-				workflow: 'none',
-				main_branch: 'main',
-				develop_branch: null,
-				auto_branch: false,
-				auto_tag: false,
-				git_url: null,
-				instructions: ['No git workflow configured for this project.'],
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await getGitWorkflow(
-			{ project_id: VALID_UUID },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			workflow: 'none',
-		});
-		expect((result.result as { instructions: string[] }).instructions[0]).toContain('No git workflow configured');
-	});
-
-	it('should return error when project not found', async () => {
-		mockApiClient.getGitWorkflow.mockResolvedValue({
-			ok: false,
-			error: 'Project not found',
-		});
-		const ctx = createMockContext();
-
-		const result = await getGitWorkflow({ project_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Project not found' });
-	});
-});
-
-// ============================================================================
-// createProject Tests
-// ============================================================================
-
-describe('createProject', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should create project successfully with minimal args', async () => {
-		const mockProject = {
-			id: 'new-proj-1',
-			name: 'New Project',
-			description: null,
-			goal: null,
-			git_url: null,
-			tech_stack: null,
-		};
-
-		mockApiClient.createProject.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				project: mockProject,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await createProject({ name: 'New Project' }, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			project: mockProject,
-		});
-		expect(mockApiClient.createProject).toHaveBeenCalledWith({
-			name: 'New Project',
-			description: undefined,
-			goal: undefined,
-			git_url: undefined,
-			tech_stack: undefined,
-		});
-	});
-
-	it('should create project with all optional fields', async () => {
-		const mockProject = {
-			id: 'new-proj-2',
-			name: 'Full Project',
-			description: 'A test project',
-			goal: 'Complete the test',
-			git_url: 'https://github.com/test/repo',
-			tech_stack: ['TypeScript', 'Svelte'],
-		};
-
-		mockApiClient.createProject.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				project: mockProject,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await createProject(
-			{
-				name: 'Full Project',
-				description: 'A test project',
-				goal: 'Complete the test',
-				git_url: 'https://github.com/test/repo',
-				tech_stack: ['TypeScript', 'Svelte'],
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			project: mockProject,
-		});
-	});
-
-	it('should return error when insert fails', async () => {
-		mockApiClient.createProject.mockResolvedValue({
-			ok: false,
-			error: 'Failed to create project',
-		});
-		const ctx = createMockContext();
-
-		const result = await createProject({ name: 'Test Project' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to create project' });
-	});
-
-	it('should throw error when name is missing', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			createProject({}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-});
-
-// ============================================================================
-// updateProject Tests
-// ============================================================================
-
-describe('updateProject', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(updateProject({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateProject({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid status', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateProject({
-				project_id: VALID_UUID,
-				status: 'invalid_status',
-			}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should update project successfully', async () => {
-		mockApiClient.updateProject.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				project_id: VALID_UUID,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProject(
-			{
-				project_id: VALID_UUID,
-				name: 'Updated Name',
-				status: 'active',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			project_id: VALID_UUID,
-		});
-		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
-			VALID_UUID,
-			{
-				name: 'Updated Name',
-				status: 'active',
-			}
-		);
-	});
-
-	it('should update git workflow settings', async () => {
-		mockApiClient.updateProject.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				project_id: VALID_UUID,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProject(
-			{
-				project_id: VALID_UUID,
-				git_workflow: 'github-flow',
-				git_main_branch: 'main',
-				git_auto_branch: true,
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-		});
-		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
-			VALID_UUID,
-			{
-				git_workflow: 'github-flow',
-				git_main_branch: 'main',
-				git_auto_branch: true,
-			}
-		);
-	});
-
-	it('should return error when update fails', async () => {
-		mockApiClient.updateProject.mockResolvedValue({
-			ok: false,
-			error: 'Failed to update project',
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProject({
-			project_id: VALID_UUID,
-			name: 'Test',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to update project' });
-	});
-});
-
-// ============================================================================
-// updateProjectReadme Tests
-// ============================================================================
-
-describe('updateProjectReadme', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateProjectReadme({ readme_content: '# README' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing readme_content', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateProjectReadme({ project_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateProjectReadme({
-				project_id: 'invalid',
-				readme_content: '# README',
-			}, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should update readme successfully', async () => {
-		mockApiClient.updateProjectReadme.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				project_id: VALID_UUID,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProjectReadme(
-			{
-				project_id: VALID_UUID,
-				readme_content: '# My Project\n\nDescription here.',
-			},
-			ctx
-		);
-
-		expect(result.result).toMatchObject({
-			success: true,
-		});
-		expect(mockApiClient.updateProjectReadme).toHaveBeenCalledWith(
-			VALID_UUID,
-			'# My Project\n\nDescription here.'
-		);
-	});
-
-	it('should return error when update fails', async () => {
-		mockApiClient.updateProjectReadme.mockResolvedValue({
-			ok: false,
-			error: 'Failed to update README',
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProjectReadme({
-			project_id: VALID_UUID,
-			readme_content: '# README',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to update README' });
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import {
+	getProjectContext,
+	getGitWorkflow,
+	createProject,
+	updateProject,
+	updateProjectReadme,
+} from './project.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+const VALID_UUID = '123e4567-e89b-12d3-a456-426614174000';
+
+// ============================================================================
+// getProjectContext Tests
+// ============================================================================
+
+describe('getProjectContext', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should list all projects when no project_id or git_url', async () => {
+		const mockProjects = [
+			{ id: 'proj-1', name: 'Project 1', description: 'Desc 1', status: 'active', git_url: null },
+			{ id: 'proj-2', name: 'Project 2', description: 'Desc 2', status: 'active', git_url: 'https://github.com/test' },
+		];
+
+		mockApiClient.listProjects.mockResolvedValue({
+			ok: true,
+			data: { projects: mockProjects },
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectContext({}, ctx);
+
+		expect(result.result).toHaveProperty('projects');
+		expect((result.result as { projects: unknown[] }).projects).toEqual(mockProjects);
+		expect(mockApiClient.listProjects).toHaveBeenCalled();
+	});
+
+	it('should return found: false when project not found', async () => {
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: false,
+				message: 'Project not found. Use create_project to create one.',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectContext(
+			{ project_id: VALID_UUID },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			found: false,
+			message: 'Project not found. Use create_project to create one.',
+		});
+	});
+
+	it('should find project by git_url', async () => {
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: true,
+				project: {
+					id: 'proj-1',
+					name: 'Test Project',
+					description: null,
+					goal: null,
+					status: 'active',
+					git_url: 'https://github.com/test',
+					agent_instructions: null,
+					tech_stack: null,
+				},
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectContext(
+			{ git_url: 'https://github.com/test' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			found: true,
+		});
+		expect(mockApiClient.getProject).toHaveBeenCalledWith(
+			'by-git-url',
+			'https://github.com/test'
+		);
+	});
+
+	it('should include active_tasks in result', async () => {
+		mockApiClient.getProject.mockResolvedValue({
+			ok: true,
+			data: {
+				found: true,
+				project: {
+					id: 'proj-1',
+					name: 'Test Project',
+					description: null,
+					goal: null,
+					status: 'active',
+					git_url: null,
+					agent_instructions: null,
+					tech_stack: null,
+				},
+				active_tasks: [],
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectContext(
+			{ project_id: VALID_UUID },
+			ctx
+		);
+
+		expect(result.result).toHaveProperty('active_tasks');
+	});
+
+	it('should return error when API call fails', async () => {
+		mockApiClient.listProjects.mockResolvedValue({
+			ok: false,
+			error: 'Database error',
+		});
+		const ctx = createMockContext();
+
+		const result = await getProjectContext({}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Database error' });
+	});
+});
+
+// ============================================================================
+// getGitWorkflow Tests
+// ============================================================================
+
+describe('getGitWorkflow', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(getGitWorkflow({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			getGitWorkflow({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return workflow instructions for github-flow', async () => {
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'github-flow',
+				main_branch: 'main',
+				develop_branch: null,
+				auto_branch: true,
+				auto_tag: false,
+				git_url: 'https://github.com/test',
+				instructions: [
+					'Create feature branch from main',
+					'Make changes and commit',
+					'Create PR to main',
+				],
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getGitWorkflow(
+			{ project_id: VALID_UUID },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'github-flow',
+			main_branch: 'main',
+			auto_branch: true,
+		});
+		expect((result.result as { instructions: string[] }).instructions.length).toBeGreaterThan(0);
+	});
+
+	it('should return workflow instructions for git-flow', async () => {
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'git-flow',
+				main_branch: 'main',
+				develop_branch: 'develop',
+				auto_branch: false,
+				auto_tag: true,
+				git_url: 'https://github.com/test',
+				instructions: ['Create feature branch from develop'],
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getGitWorkflow(
+			{ project_id: VALID_UUID },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'git-flow',
+			main_branch: 'main',
+			develop_branch: 'develop',
+		});
+	});
+
+	it('should return no workflow instructions when workflow is none', async () => {
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: true,
+			data: {
+				workflow: 'none',
+				main_branch: 'main',
+				develop_branch: null,
+				auto_branch: false,
+				auto_tag: false,
+				git_url: null,
+				instructions: ['No git workflow configured for this project.'],
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await getGitWorkflow(
+			{ project_id: VALID_UUID },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			workflow: 'none',
+		});
+		expect((result.result as { instructions: string[] }).instructions[0]).toContain('No git workflow configured');
+	});
+
+	it('should return error when project not found', async () => {
+		mockApiClient.getGitWorkflow.mockResolvedValue({
+			ok: false,
+			error: 'Project not found',
+		});
+		const ctx = createMockContext();
+
+		const result = await getGitWorkflow({ project_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Project not found' });
+	});
+});
+
+// ============================================================================
+// createProject Tests
+// ============================================================================
+
+describe('createProject', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should create project successfully with minimal args', async () => {
+		const mockProject = {
+			id: 'new-proj-1',
+			name: 'New Project',
+			description: null,
+			goal: null,
+			git_url: null,
+			tech_stack: null,
+		};
+
+		mockApiClient.createProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project: mockProject,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await createProject({ name: 'New Project' }, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project: mockProject,
+		});
+		expect(mockApiClient.createProject).toHaveBeenCalledWith({
+			name: 'New Project',
+			description: undefined,
+			goal: undefined,
+			git_url: undefined,
+			tech_stack: undefined,
+		});
+	});
+
+	it('should create project with all optional fields', async () => {
+		const mockProject = {
+			id: 'new-proj-2',
+			name: 'Full Project',
+			description: 'A test project',
+			goal: 'Complete the test',
+			git_url: 'https://github.com/test/repo',
+			tech_stack: ['TypeScript', 'Svelte'],
+		};
+
+		mockApiClient.createProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project: mockProject,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await createProject(
+			{
+				name: 'Full Project',
+				description: 'A test project',
+				goal: 'Complete the test',
+				git_url: 'https://github.com/test/repo',
+				tech_stack: ['TypeScript', 'Svelte'],
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project: mockProject,
+		});
+	});
+
+	it('should return error when insert fails', async () => {
+		mockApiClient.createProject.mockResolvedValue({
+			ok: false,
+			error: 'Failed to create project',
+		});
+		const ctx = createMockContext();
+
+		const result = await createProject({ name: 'Test Project' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to create project' });
+	});
+
+	it('should throw error when name is missing', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			createProject({}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+});
+
+// ============================================================================
+// updateProject Tests
+// ============================================================================
+
+describe('updateProject', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(updateProject({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateProject({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid status', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateProject({
+				project_id: VALID_UUID,
+				status: 'invalid_status',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update project successfully', async () => {
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProject(
+			{
+				project_id: VALID_UUID,
+				name: 'Updated Name',
+				status: 'active',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			project_id: VALID_UUID,
+		});
+		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
+			VALID_UUID,
+			{
+				name: 'Updated Name',
+				status: 'active',
+			}
+		);
+	});
+
+	it('should update git workflow settings', async () => {
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProject(
+			{
+				project_id: VALID_UUID,
+				git_workflow: 'github-flow',
+				git_main_branch: 'main',
+				git_auto_branch: true,
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+		expect(mockApiClient.updateProject).toHaveBeenCalledWith(
+			VALID_UUID,
+			{
+				git_workflow: 'github-flow',
+				git_main_branch: 'main',
+				git_auto_branch: true,
+			}
+		);
+	});
+
+	it('should return error when update fails', async () => {
+		mockApiClient.updateProject.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update project',
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProject({
+			project_id: VALID_UUID,
+			name: 'Test',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to update project' });
+	});
+});
+
+// ============================================================================
+// updateProjectReadme Tests
+// ============================================================================
+
+describe('updateProjectReadme', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateProjectReadme({ readme_content: '# README' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing readme_content', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateProjectReadme({ project_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateProjectReadme({
+				project_id: 'invalid',
+				readme_content: '# README',
+			}, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update readme successfully', async () => {
+		mockApiClient.updateProjectReadme.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				project_id: VALID_UUID,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProjectReadme(
+			{
+				project_id: VALID_UUID,
+				readme_content: '# My Project\n\nDescription here.',
+			},
+			ctx
+		);
+
+		expect(result.result).toMatchObject({
+			success: true,
+		});
+		expect(mockApiClient.updateProjectReadme).toHaveBeenCalledWith(
+			VALID_UUID,
+			'# My Project\n\nDescription here.'
+		);
+	});
+
+	it('should return error when update fails', async () => {
+		mockApiClient.updateProjectReadme.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update README',
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProjectReadme({
+			project_id: VALID_UUID,
+			readme_content: '# README',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to update README' });
+	});
+});