@vibescope/mcp-server 0.5.0 → 0.5.1

package/src/handlers/organizations.test.ts

@@ -1,826 +1,826 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import {
-	listOrganizations,
-	createOrganization,
-	updateOrganization,
-	deleteOrganization,
-	listOrgMembers,
-	inviteMember,
-	updateMemberRole,
-	removeMember,
-	leaveOrganization,
-	shareProjectWithOrg,
-	updateProjectShare,
-	unshareProject,
-	listProjectShares,
-} from './organizations.js';
-import { ValidationError } from '../validators.js';
-import { createMockContext } from './__test-utils__.js';
-import { mockApiClient } from './__test-setup__.js';
-
-// ============================================================================
-// Test Utilities
-// ============================================================================
-
-const VALID_UUID = '123e4567-e89b-12d3-a456-426614174000';
-const OTHER_UUID = '223e4567-e89b-12d3-a456-426614174001';
-
-// ============================================================================
-// listOrganizations Tests
-// ============================================================================
-
-describe('listOrganizations', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should return empty array when no organizations', async () => {
-		mockApiClient.listOrganizations.mockResolvedValue({
-			ok: true,
-			data: { organizations: [], count: 0 },
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrganizations({}, ctx);
-
-		expect(result.result).toMatchObject({
-			organizations: [],
-			count: 0,
-		});
-	});
-
-	it('should return organizations with role and joined_at', async () => {
-		mockApiClient.listOrganizations.mockResolvedValue({
-			ok: true,
-			data: {
-				organizations: [{
-					id: 'org-1',
-					name: 'Test Org',
-					slug: 'test-org',
-					role: 'admin',
-					joined_at: '2025-01-14T12:00:00Z',
-				}],
-				count: 1,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrganizations({}, ctx);
-
-		expect(result.result).toMatchObject({ count: 1 });
-	});
-
-	it('should return error when query fails', async () => {
-		mockApiClient.listOrganizations.mockResolvedValue({
-			ok: false,
-			error: 'Failed to list organizations',
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrganizations({}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to list organizations' });
-	});
-});
-
-// ============================================================================
-// createOrganization Tests
-// ============================================================================
-
-describe('createOrganization', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing name', async () => {
-		const ctx = createMockContext();
-		await expect(createOrganization({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when slug is taken', async () => {
-		mockApiClient.createOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Organization slug "test-org" is already taken',
-		});
-		const ctx = createMockContext();
-
-		const result = await createOrganization({ name: 'Test Org' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Organization slug "test-org" is already taken' });
-	});
-
-	it('should create organization with auto-generated slug', async () => {
-		mockApiClient.createOrganization.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				organization_id: 'org-1',
-				slug: 'test-org',
-				message: 'Organization "Test Org" created. You are the owner.',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await createOrganization({ name: 'Test Org' }, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Organization "Test Org" created. You are the owner.',
-		});
-	});
-
-	it('should create organization with custom slug', async () => {
-		mockApiClient.createOrganization.mockResolvedValue({
-			ok: true,
-			data: { success: true, organization_id: 'org-1', slug: 'my-custom-slug' },
-		});
-		const ctx = createMockContext();
-
-		const result = await createOrganization(
-			{ name: 'Test Org', slug: 'my-custom-slug' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should return error when insert fails', async () => {
-		mockApiClient.createOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Failed to create organization',
-		});
-		const ctx = createMockContext();
-
-		const result = await createOrganization({ name: 'Test Org' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to create organization' });
-	});
-});
-
-// ============================================================================
-// updateOrganization Tests
-// ============================================================================
-
-describe('updateOrganization', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateOrganization({ name: 'New Name' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid organization_id UUID', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateOrganization({ organization_id: 'invalid', name: 'New Name' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error when no updates provided', async () => {
-		const ctx = createMockContext();
-
-		await expect(
-			updateOrganization({ organization_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should update organization successfully', async () => {
-		mockApiClient.updateOrganization.mockResolvedValue({
-			ok: true,
-			data: { success: true, organization_id: VALID_UUID },
-		});
-		const ctx = createMockContext();
-
-		const result = await updateOrganization(
-			{ organization_id: VALID_UUID, name: 'Updated Org', description: 'New desc' },
-			ctx
-		);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should return error when update fails', async () => {
-		mockApiClient.updateOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Failed to update organization',
-		});
-		const ctx = createMockContext();
-
-		const result = await updateOrganization({ organization_id: VALID_UUID, name: 'New Name' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to update organization' });
-	});
-});
-
-// ============================================================================
-// deleteOrganization Tests
-// ============================================================================
-
-describe('deleteOrganization', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(deleteOrganization({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid organization_id UUID', async () => {
-		const ctx = createMockContext();
-		await expect(
-			deleteOrganization({ organization_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should delete organization successfully', async () => {
-		mockApiClient.deleteOrganization.mockResolvedValue({
-			ok: true,
-			data: { success: true, message: 'Organization deleted. All shares have been removed.' },
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteOrganization({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Organization deleted. All shares have been removed.',
-		});
-	});
-
-	it('should return error when delete fails', async () => {
-		mockApiClient.deleteOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Failed to delete organization',
-		});
-		const ctx = createMockContext();
-
-		const result = await deleteOrganization({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to delete organization' });
-	});
-});
-
-// ============================================================================
-// listOrgMembers Tests
-// ============================================================================
-
-describe('listOrgMembers', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(listOrgMembers({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid organization_id UUID', async () => {
-		const ctx = createMockContext();
-		await expect(
-			listOrgMembers({ organization_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty array when no members', async () => {
-		mockApiClient.listOrgMembers.mockResolvedValue({
-			ok: true,
-			data: { members: [], count: 0 },
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({ members: [], count: 0 });
-	});
-
-	it('should return members list', async () => {
-		mockApiClient.listOrgMembers.mockResolvedValue({
-			ok: true,
-			data: {
-				members: [
-					{ id: 'm-1', user_id: 'u-1', role: 'owner' },
-					{ id: 'm-2', user_id: 'u-2', role: 'member' },
-				],
-				count: 2,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({ count: 2 });
-	});
-
-	it('should return error when query fails', async () => {
-		mockApiClient.listOrgMembers.mockResolvedValue({
-			ok: false,
-			error: 'Failed to list members',
-		});
-		const ctx = createMockContext();
-
-		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to list members' });
-	});
-});
-
-// ============================================================================
-// inviteMember Tests
-// ============================================================================
-
-describe('inviteMember', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			inviteMember({ email: 'test@example.com' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing email', async () => {
-		const ctx = createMockContext();
-		await expect(
-			inviteMember({ organization_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when invite already exists', async () => {
-		mockApiClient.inviteMember.mockResolvedValue({
-			ok: false,
-			error: 'A pending invite already exists for test@example.com',
-		});
-		const ctx = createMockContext();
-
-		const result = await inviteMember({ organization_id: VALID_UUID, email: 'test@example.com' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'A pending invite already exists for test@example.com' });
-	});
-
-	it('should create invite successfully with default role', async () => {
-		mockApiClient.inviteMember.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				invite_id: 'invite-1',
-				message: 'Invite sent to test@example.com with role "member"',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await inviteMember({
-			organization_id: VALID_UUID,
-			email: 'test@example.com',
-		}, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Invite sent to test@example.com with role "member"',
-		});
-	});
-
-	it('should return error when insert fails', async () => {
-		mockApiClient.inviteMember.mockResolvedValue({
-			ok: false,
-			error: 'Failed to create invite',
-		});
-		const ctx = createMockContext();
-
-		const result = await inviteMember({ organization_id: VALID_UUID, email: 'test@example.com' }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to create invite' });
-	});
-});
-
-// ============================================================================
-// updateMemberRole Tests
-// ============================================================================
-
-describe('updateMemberRole', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateMemberRole({ user_id: VALID_UUID, role: 'admin' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing user_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateMemberRole({ organization_id: VALID_UUID, role: 'admin' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing role', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateMemberRole({ organization_id: VALID_UUID, user_id: OTHER_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when changing own role', async () => {
-		mockApiClient.updateMemberRole.mockResolvedValue({
-			ok: false,
-			error: 'Cannot change your own role',
-		});
-		const ctx = createMockContext({ userId: VALID_UUID });
-
-		const result = await updateMemberRole({
-			organization_id: OTHER_UUID,
-			user_id: VALID_UUID,
-			role: 'admin',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Cannot change your own role' });
-	});
-
-	it('should update member role successfully', async () => {
-		mockApiClient.updateMemberRole.mockResolvedValue({
-			ok: true,
-			data: { success: true, user_id: OTHER_UUID, role: 'admin' },
-		});
-		const ctx = createMockContext();
-
-		const result = await updateMemberRole({
-			organization_id: VALID_UUID,
-			user_id: OTHER_UUID,
-			role: 'admin',
-		}, ctx);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should return error when update fails', async () => {
-		mockApiClient.updateMemberRole.mockResolvedValue({
-			ok: false,
-			error: 'Failed to update member role',
-		});
-		const ctx = createMockContext();
-
-		const result = await updateMemberRole({
-			organization_id: VALID_UUID,
-			user_id: OTHER_UUID,
-			role: 'admin',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to update member role' });
-	});
-});
-
-// ============================================================================
-// removeMember Tests
-// ============================================================================
-
-describe('removeMember', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			removeMember({ user_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing user_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			removeMember({ organization_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should remove member successfully', async () => {
-		mockApiClient.removeMember.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				message: 'Member removed. Their org-scoped API keys have been invalidated.',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await removeMember({
-			organization_id: VALID_UUID,
-			user_id: OTHER_UUID,
-		}, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Member removed. Their org-scoped API keys have been invalidated.',
-		});
-	});
-
-	it('should return error when delete fails', async () => {
-		mockApiClient.removeMember.mockResolvedValue({
-			ok: false,
-			error: 'Failed to remove member',
-		});
-		const ctx = createMockContext();
-
-		const result = await removeMember({ organization_id: VALID_UUID, user_id: OTHER_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to remove member' });
-	});
-});
-
-// ============================================================================
-// leaveOrganization Tests
-// ============================================================================
-
-describe('leaveOrganization', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(leaveOrganization({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when user is owner', async () => {
-		mockApiClient.leaveOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Owner cannot leave. Transfer ownership first or delete the organization.',
-		});
-		const ctx = createMockContext();
-
-		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Owner cannot leave. Transfer ownership first or delete the organization.' });
-	});
-
-	it('should leave organization successfully', async () => {
-		mockApiClient.leaveOrganization.mockResolvedValue({
-			ok: true,
-			data: { success: true, message: 'You have left the organization.' },
-		});
-		const ctx = createMockContext();
-
-		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'You have left the organization.',
-		});
-	});
-
-	it('should return error when delete fails', async () => {
-		mockApiClient.leaveOrganization.mockResolvedValue({
-			ok: false,
-			error: 'Failed to leave organization',
-		});
-		const ctx = createMockContext();
-
-		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to leave organization' });
-	});
-});
-
-// ============================================================================
-// shareProjectWithOrg Tests
-// ============================================================================
-
-describe('shareProjectWithOrg', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			shareProjectWithOrg({ organization_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			shareProjectWithOrg({ project_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return error when project not found or not owned', async () => {
-		mockApiClient.shareProjectWithOrg.mockResolvedValue({
-			ok: false,
-			error: 'Project not found or you are not the owner',
-		});
-		const ctx = createMockContext();
-
-		const result = await shareProjectWithOrg({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Project not found or you are not the owner' });
-	});
-
-	it('should share project successfully with default permission', async () => {
-		mockApiClient.shareProjectWithOrg.mockResolvedValue({
-			ok: true,
-			data: { success: true, share_id: 'share-1', permission: 'read' },
-		});
-		const ctx = createMockContext();
-
-		const result = await shareProjectWithOrg({
-			project_id: VALID_UUID,
-			organization_id: OTHER_UUID,
-		}, ctx);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should return error when share already exists', async () => {
-		mockApiClient.shareProjectWithOrg.mockResolvedValue({
-			ok: false,
-			error: 'Project is already shared with this organization',
-		});
-		const ctx = createMockContext();
-
-		const result = await shareProjectWithOrg({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Project is already shared with this organization' });
-	});
-});
-
-// ============================================================================
-// updateProjectShare Tests
-// ============================================================================
-
-describe('updateProjectShare', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateProjectShare({ organization_id: VALID_UUID, permission: 'write' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateProjectShare({ project_id: VALID_UUID, permission: 'write' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing permission', async () => {
-		const ctx = createMockContext();
-		await expect(
-			updateProjectShare({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should update share permission successfully', async () => {
-		mockApiClient.updateProjectShare.mockResolvedValue({
-			ok: true,
-			data: { success: true, share_id: 'share-1', permission: 'write' },
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProjectShare({
-			project_id: VALID_UUID,
-			organization_id: OTHER_UUID,
-			permission: 'write',
-		}, ctx);
-
-		expect(result.result).toMatchObject({ success: true });
-	});
-
-	it('should return error when update fails', async () => {
-		mockApiClient.updateProjectShare.mockResolvedValue({
-			ok: false,
-			error: 'Failed to update share',
-		});
-		const ctx = createMockContext();
-
-		const result = await updateProjectShare({
-			project_id: VALID_UUID,
-			organization_id: OTHER_UUID,
-			permission: 'write',
-		}, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to update share' });
-	});
-});
-
-// ============================================================================
-// unshareProject Tests
-// ============================================================================
-
-describe('unshareProject', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			unshareProject({ organization_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for missing organization_id', async () => {
-		const ctx = createMockContext();
-		await expect(
-			unshareProject({ project_id: VALID_UUID }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should unshare project successfully', async () => {
-		mockApiClient.unshareProject.mockResolvedValue({
-			ok: true,
-			data: {
-				success: true,
-				message: 'Project share removed. Org members can no longer access this project.',
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await unshareProject({
-			project_id: VALID_UUID,
-			organization_id: OTHER_UUID,
-		}, ctx);
-
-		expect(result.result).toMatchObject({
-			success: true,
-			message: 'Project share removed. Org members can no longer access this project.',
-		});
-	});
-
-	it('should return error when delete fails', async () => {
-		mockApiClient.unshareProject.mockResolvedValue({
-			ok: false,
-			error: 'Failed to unshare project',
-		});
-		const ctx = createMockContext();
-
-		const result = await unshareProject({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to unshare project' });
-	});
-});
-
-// ============================================================================
-// listProjectShares Tests
-// ============================================================================
-
-describe('listProjectShares', () => {
-	beforeEach(() => vi.clearAllMocks());
-
-	it('should throw error for missing project_id', async () => {
-		const ctx = createMockContext();
-		await expect(listProjectShares({}, ctx)).rejects.toThrow(ValidationError);
-	});
-
-	it('should throw error for invalid project_id UUID', async () => {
-		const ctx = createMockContext();
-		await expect(
-			listProjectShares({ project_id: 'invalid' }, ctx)
-		).rejects.toThrow(ValidationError);
-	});
-
-	it('should return empty array when no shares', async () => {
-		mockApiClient.listProjectShares.mockResolvedValue({
-			ok: true,
-			data: { shares: [], count: 0 },
-		});
-		const ctx = createMockContext();
-
-		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({ shares: [], count: 0 });
-	});
-
-	it('should return shares list', async () => {
-		mockApiClient.listProjectShares.mockResolvedValue({
-			ok: true,
-			data: {
-				shares: [{ id: 'share-1', permission: 'read', organization: { name: 'Test Org' } }],
-				count: 1,
-			},
-		});
-		const ctx = createMockContext();
-
-		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
-
-		expect(result.result).toMatchObject({ count: 1 });
-	});
-
-	it('should return error when query fails', async () => {
-		mockApiClient.listProjectShares.mockResolvedValue({
-			ok: false,
-			error: 'Failed to list shares',
-		});
-		const ctx = createMockContext();
-
-		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
-
-		expect(result.isError).toBe(true);
-		expect(result.result).toMatchObject({ error: 'Failed to list shares' });
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import {
+	listOrganizations,
+	createOrganization,
+	updateOrganization,
+	deleteOrganization,
+	listOrgMembers,
+	inviteMember,
+	updateMemberRole,
+	removeMember,
+	leaveOrganization,
+	shareProjectWithOrg,
+	updateProjectShare,
+	unshareProject,
+	listProjectShares,
+} from './organizations.js';
+import { ValidationError } from '../validators.js';
+import { createMockContext } from './__test-utils__.js';
+import { mockApiClient } from './__test-setup__.js';
+
+// ============================================================================
+// Test Utilities
+// ============================================================================
+
+const VALID_UUID = '123e4567-e89b-12d3-a456-426614174000';
+const OTHER_UUID = '223e4567-e89b-12d3-a456-426614174001';
+
+// ============================================================================
+// listOrganizations Tests
+// ============================================================================
+
+describe('listOrganizations', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should return empty array when no organizations', async () => {
+		mockApiClient.listOrganizations.mockResolvedValue({
+			ok: true,
+			data: { organizations: [], count: 0 },
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrganizations({}, ctx);
+
+		expect(result.result).toMatchObject({
+			organizations: [],
+			count: 0,
+		});
+	});
+
+	it('should return organizations with role and joined_at', async () => {
+		mockApiClient.listOrganizations.mockResolvedValue({
+			ok: true,
+			data: {
+				organizations: [{
+					id: 'org-1',
+					name: 'Test Org',
+					slug: 'test-org',
+					role: 'admin',
+					joined_at: '2025-01-14T12:00:00Z',
+				}],
+				count: 1,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrganizations({}, ctx);
+
+		expect(result.result).toMatchObject({ count: 1 });
+	});
+
+	it('should return error when query fails', async () => {
+		mockApiClient.listOrganizations.mockResolvedValue({
+			ok: false,
+			error: 'Failed to list organizations',
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrganizations({}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to list organizations' });
+	});
+});
+
+// ============================================================================
+// createOrganization Tests
+// ============================================================================
+
+describe('createOrganization', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing name', async () => {
+		const ctx = createMockContext();
+		await expect(createOrganization({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when slug is taken', async () => {
+		mockApiClient.createOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Organization slug "test-org" is already taken',
+		});
+		const ctx = createMockContext();
+
+		const result = await createOrganization({ name: 'Test Org' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Organization slug "test-org" is already taken' });
+	});
+
+	it('should create organization with auto-generated slug', async () => {
+		mockApiClient.createOrganization.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				organization_id: 'org-1',
+				slug: 'test-org',
+				message: 'Organization "Test Org" created. You are the owner.',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await createOrganization({ name: 'Test Org' }, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Organization "Test Org" created. You are the owner.',
+		});
+	});
+
+	it('should create organization with custom slug', async () => {
+		mockApiClient.createOrganization.mockResolvedValue({
+			ok: true,
+			data: { success: true, organization_id: 'org-1', slug: 'my-custom-slug' },
+		});
+		const ctx = createMockContext();
+
+		const result = await createOrganization(
+			{ name: 'Test Org', slug: 'my-custom-slug' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should return error when insert fails', async () => {
+		mockApiClient.createOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Failed to create organization',
+		});
+		const ctx = createMockContext();
+
+		const result = await createOrganization({ name: 'Test Org' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to create organization' });
+	});
+});
+
+// ============================================================================
+// updateOrganization Tests
+// ============================================================================
+
+describe('updateOrganization', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateOrganization({ name: 'New Name' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid organization_id UUID', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateOrganization({ organization_id: 'invalid', name: 'New Name' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error when no updates provided', async () => {
+		const ctx = createMockContext();
+
+		await expect(
+			updateOrganization({ organization_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update organization successfully', async () => {
+		mockApiClient.updateOrganization.mockResolvedValue({
+			ok: true,
+			data: { success: true, organization_id: VALID_UUID },
+		});
+		const ctx = createMockContext();
+
+		const result = await updateOrganization(
+			{ organization_id: VALID_UUID, name: 'Updated Org', description: 'New desc' },
+			ctx
+		);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should return error when update fails', async () => {
+		mockApiClient.updateOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update organization',
+		});
+		const ctx = createMockContext();
+
+		const result = await updateOrganization({ organization_id: VALID_UUID, name: 'New Name' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to update organization' });
+	});
+});
+
+// ============================================================================
+// deleteOrganization Tests
+// ============================================================================
+
+describe('deleteOrganization', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(deleteOrganization({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid organization_id UUID', async () => {
+		const ctx = createMockContext();
+		await expect(
+			deleteOrganization({ organization_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should delete organization successfully', async () => {
+		mockApiClient.deleteOrganization.mockResolvedValue({
+			ok: true,
+			data: { success: true, message: 'Organization deleted. All shares have been removed.' },
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteOrganization({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Organization deleted. All shares have been removed.',
+		});
+	});
+
+	it('should return error when delete fails', async () => {
+		mockApiClient.deleteOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Failed to delete organization',
+		});
+		const ctx = createMockContext();
+
+		const result = await deleteOrganization({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to delete organization' });
+	});
+});
+
+// ============================================================================
+// listOrgMembers Tests
+// ============================================================================
+
+describe('listOrgMembers', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(listOrgMembers({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid organization_id UUID', async () => {
+		const ctx = createMockContext();
+		await expect(
+			listOrgMembers({ organization_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty array when no members', async () => {
+		mockApiClient.listOrgMembers.mockResolvedValue({
+			ok: true,
+			data: { members: [], count: 0 },
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({ members: [], count: 0 });
+	});
+
+	it('should return members list', async () => {
+		mockApiClient.listOrgMembers.mockResolvedValue({
+			ok: true,
+			data: {
+				members: [
+					{ id: 'm-1', user_id: 'u-1', role: 'owner' },
+					{ id: 'm-2', user_id: 'u-2', role: 'member' },
+				],
+				count: 2,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({ count: 2 });
+	});
+
+	it('should return error when query fails', async () => {
+		mockApiClient.listOrgMembers.mockResolvedValue({
+			ok: false,
+			error: 'Failed to list members',
+		});
+		const ctx = createMockContext();
+
+		const result = await listOrgMembers({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to list members' });
+	});
+});
+
+// ============================================================================
+// inviteMember Tests
+// ============================================================================
+
+describe('inviteMember', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			inviteMember({ email: 'test@example.com' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing email', async () => {
+		const ctx = createMockContext();
+		await expect(
+			inviteMember({ organization_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when invite already exists', async () => {
+		mockApiClient.inviteMember.mockResolvedValue({
+			ok: false,
+			error: 'A pending invite already exists for test@example.com',
+		});
+		const ctx = createMockContext();
+
+		const result = await inviteMember({ organization_id: VALID_UUID, email: 'test@example.com' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'A pending invite already exists for test@example.com' });
+	});
+
+	it('should create invite successfully with default role', async () => {
+		mockApiClient.inviteMember.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				invite_id: 'invite-1',
+				message: 'Invite sent to test@example.com with role "member"',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await inviteMember({
+			organization_id: VALID_UUID,
+			email: 'test@example.com',
+		}, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Invite sent to test@example.com with role "member"',
+		});
+	});
+
+	it('should return error when insert fails', async () => {
+		mockApiClient.inviteMember.mockResolvedValue({
+			ok: false,
+			error: 'Failed to create invite',
+		});
+		const ctx = createMockContext();
+
+		const result = await inviteMember({ organization_id: VALID_UUID, email: 'test@example.com' }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to create invite' });
+	});
+});
+
+// ============================================================================
+// updateMemberRole Tests
+// ============================================================================
+
+describe('updateMemberRole', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateMemberRole({ user_id: VALID_UUID, role: 'admin' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing user_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateMemberRole({ organization_id: VALID_UUID, role: 'admin' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing role', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateMemberRole({ organization_id: VALID_UUID, user_id: OTHER_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when changing own role', async () => {
+		mockApiClient.updateMemberRole.mockResolvedValue({
+			ok: false,
+			error: 'Cannot change your own role',
+		});
+		const ctx = createMockContext({ userId: VALID_UUID });
+
+		const result = await updateMemberRole({
+			organization_id: OTHER_UUID,
+			user_id: VALID_UUID,
+			role: 'admin',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Cannot change your own role' });
+	});
+
+	it('should update member role successfully', async () => {
+		mockApiClient.updateMemberRole.mockResolvedValue({
+			ok: true,
+			data: { success: true, user_id: OTHER_UUID, role: 'admin' },
+		});
+		const ctx = createMockContext();
+
+		const result = await updateMemberRole({
+			organization_id: VALID_UUID,
+			user_id: OTHER_UUID,
+			role: 'admin',
+		}, ctx);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should return error when update fails', async () => {
+		mockApiClient.updateMemberRole.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update member role',
+		});
+		const ctx = createMockContext();
+
+		const result = await updateMemberRole({
+			organization_id: VALID_UUID,
+			user_id: OTHER_UUID,
+			role: 'admin',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to update member role' });
+	});
+});
+
+// ============================================================================
+// removeMember Tests
+// ============================================================================
+
+describe('removeMember', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			removeMember({ user_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing user_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			removeMember({ organization_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should remove member successfully', async () => {
+		mockApiClient.removeMember.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				message: 'Member removed. Their org-scoped API keys have been invalidated.',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await removeMember({
+			organization_id: VALID_UUID,
+			user_id: OTHER_UUID,
+		}, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Member removed. Their org-scoped API keys have been invalidated.',
+		});
+	});
+
+	it('should return error when delete fails', async () => {
+		mockApiClient.removeMember.mockResolvedValue({
+			ok: false,
+			error: 'Failed to remove member',
+		});
+		const ctx = createMockContext();
+
+		const result = await removeMember({ organization_id: VALID_UUID, user_id: OTHER_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to remove member' });
+	});
+});
+
+// ============================================================================
+// leaveOrganization Tests
+// ============================================================================
+
+describe('leaveOrganization', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(leaveOrganization({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when user is owner', async () => {
+		mockApiClient.leaveOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Owner cannot leave. Transfer ownership first or delete the organization.',
+		});
+		const ctx = createMockContext();
+
+		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Owner cannot leave. Transfer ownership first or delete the organization.' });
+	});
+
+	it('should leave organization successfully', async () => {
+		mockApiClient.leaveOrganization.mockResolvedValue({
+			ok: true,
+			data: { success: true, message: 'You have left the organization.' },
+		});
+		const ctx = createMockContext();
+
+		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'You have left the organization.',
+		});
+	});
+
+	it('should return error when delete fails', async () => {
+		mockApiClient.leaveOrganization.mockResolvedValue({
+			ok: false,
+			error: 'Failed to leave organization',
+		});
+		const ctx = createMockContext();
+
+		const result = await leaveOrganization({ organization_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to leave organization' });
+	});
+});
+
+// ============================================================================
+// shareProjectWithOrg Tests
+// ============================================================================
+
+describe('shareProjectWithOrg', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			shareProjectWithOrg({ organization_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			shareProjectWithOrg({ project_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return error when project not found or not owned', async () => {
+		mockApiClient.shareProjectWithOrg.mockResolvedValue({
+			ok: false,
+			error: 'Project not found or you are not the owner',
+		});
+		const ctx = createMockContext();
+
+		const result = await shareProjectWithOrg({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Project not found or you are not the owner' });
+	});
+
+	it('should share project successfully with default permission', async () => {
+		mockApiClient.shareProjectWithOrg.mockResolvedValue({
+			ok: true,
+			data: { success: true, share_id: 'share-1', permission: 'read' },
+		});
+		const ctx = createMockContext();
+
+		const result = await shareProjectWithOrg({
+			project_id: VALID_UUID,
+			organization_id: OTHER_UUID,
+		}, ctx);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should return error when share already exists', async () => {
+		mockApiClient.shareProjectWithOrg.mockResolvedValue({
+			ok: false,
+			error: 'Project is already shared with this organization',
+		});
+		const ctx = createMockContext();
+
+		const result = await shareProjectWithOrg({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Project is already shared with this organization' });
+	});
+});
+
+// ============================================================================
+// updateProjectShare Tests
+// ============================================================================
+
+describe('updateProjectShare', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateProjectShare({ organization_id: VALID_UUID, permission: 'write' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateProjectShare({ project_id: VALID_UUID, permission: 'write' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing permission', async () => {
+		const ctx = createMockContext();
+		await expect(
+			updateProjectShare({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should update share permission successfully', async () => {
+		mockApiClient.updateProjectShare.mockResolvedValue({
+			ok: true,
+			data: { success: true, share_id: 'share-1', permission: 'write' },
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProjectShare({
+			project_id: VALID_UUID,
+			organization_id: OTHER_UUID,
+			permission: 'write',
+		}, ctx);
+
+		expect(result.result).toMatchObject({ success: true });
+	});
+
+	it('should return error when update fails', async () => {
+		mockApiClient.updateProjectShare.mockResolvedValue({
+			ok: false,
+			error: 'Failed to update share',
+		});
+		const ctx = createMockContext();
+
+		const result = await updateProjectShare({
+			project_id: VALID_UUID,
+			organization_id: OTHER_UUID,
+			permission: 'write',
+		}, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to update share' });
+	});
+});
+
+// ============================================================================
+// unshareProject Tests
+// ============================================================================
+
+describe('unshareProject', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			unshareProject({ organization_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for missing organization_id', async () => {
+		const ctx = createMockContext();
+		await expect(
+			unshareProject({ project_id: VALID_UUID }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should unshare project successfully', async () => {
+		mockApiClient.unshareProject.mockResolvedValue({
+			ok: true,
+			data: {
+				success: true,
+				message: 'Project share removed. Org members can no longer access this project.',
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await unshareProject({
+			project_id: VALID_UUID,
+			organization_id: OTHER_UUID,
+		}, ctx);
+
+		expect(result.result).toMatchObject({
+			success: true,
+			message: 'Project share removed. Org members can no longer access this project.',
+		});
+	});
+
+	it('should return error when delete fails', async () => {
+		mockApiClient.unshareProject.mockResolvedValue({
+			ok: false,
+			error: 'Failed to unshare project',
+		});
+		const ctx = createMockContext();
+
+		const result = await unshareProject({ project_id: VALID_UUID, organization_id: OTHER_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to unshare project' });
+	});
+});
+
+// ============================================================================
+// listProjectShares Tests
+// ============================================================================
+
+describe('listProjectShares', () => {
+	beforeEach(() => vi.clearAllMocks());
+
+	it('should throw error for missing project_id', async () => {
+		const ctx = createMockContext();
+		await expect(listProjectShares({}, ctx)).rejects.toThrow(ValidationError);
+	});
+
+	it('should throw error for invalid project_id UUID', async () => {
+		const ctx = createMockContext();
+		await expect(
+			listProjectShares({ project_id: 'invalid' }, ctx)
+		).rejects.toThrow(ValidationError);
+	});
+
+	it('should return empty array when no shares', async () => {
+		mockApiClient.listProjectShares.mockResolvedValue({
+			ok: true,
+			data: { shares: [], count: 0 },
+		});
+		const ctx = createMockContext();
+
+		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({ shares: [], count: 0 });
+	});
+
+	it('should return shares list', async () => {
+		mockApiClient.listProjectShares.mockResolvedValue({
+			ok: true,
+			data: {
+				shares: [{ id: 'share-1', permission: 'read', organization: { name: 'Test Org' } }],
+				count: 1,
+			},
+		});
+		const ctx = createMockContext();
+
+		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
+
+		expect(result.result).toMatchObject({ count: 1 });
+	});
+
+	it('should return error when query fails', async () => {
+		mockApiClient.listProjectShares.mockResolvedValue({
+			ok: false,
+			error: 'Failed to list shares',
+		});
+		const ctx = createMockContext();
+
+		const result = await listProjectShares({ project_id: VALID_UUID }, ctx);
+
+		expect(result.isError).toBe(true);
+		expect(result.result).toMatchObject({ error: 'Failed to list shares' });
+	});
+});