@vibescope/mcp-server 0.5.0 → 0.5.1

package/src/tools/tools.test.ts

@@ -1,222 +1,222 @@
-/**
- * Tests for split tool definitions
- *
- * Validates that domain-specific tool files:
- * - Export valid Tool arrays
- * - Have unique tool names
- * - Have required inputSchema properties
- */
-
-import { describe, it, expect } from 'vitest';
-import { sessionTools } from './session.js';
-import { costTools } from './cost.js';
-import { discoveryTools } from './discovery.js';
-import { worktreeTools } from './worktrees.js';
-import { projectTools } from './project.js';
-import { blockerTools } from './blockers.js';
-import { decisionTools } from './decisions.js';
-import { ideaTools } from './ideas.js';
-import { buildToolList, toolCategories } from './index.js';
-
-describe('Split Tool Definitions', () => {
-	describe('sessionTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(sessionTools)).toBe(true);
-			expect(sessionTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected session tools', () => {
-			const toolNames = sessionTools.map(t => t.name);
-			expect(toolNames).toContain('start_work_session');
-			expect(toolNames).toContain('heartbeat');
-			expect(toolNames).toContain('end_work_session');
-			expect(toolNames).toContain('signal_idle');
-			expect(toolNames).toContain('get_token_usage');
-			expect(toolNames).toContain('report_token_usage');
-		});
-
-		it('should have valid inputSchema for all tools', () => {
-			for (const tool of sessionTools) {
-				expect(tool.inputSchema).toBeDefined();
-				expect(tool.inputSchema.type).toBe('object');
-				expect(tool.inputSchema.properties).toBeDefined();
-			}
-		});
-	});
-
-	describe('costTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(costTools)).toBe(true);
-			expect(costTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected cost tools', () => {
-			const toolNames = costTools.map(t => t.name);
-			expect(toolNames).toContain('get_cost_summary');
-			expect(toolNames).toContain('add_cost_alert');
-			expect(toolNames).toContain('update_cost_alert');
-			expect(toolNames).toContain('delete_cost_alert');
-		});
-	});
-
-	describe('discoveryTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(discoveryTools)).toBe(true);
-			expect(discoveryTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected discovery tools', () => {
-			const toolNames = discoveryTools.map(t => t.name);
-			expect(toolNames).toContain('query_knowledge_base');
-			expect(toolNames).toContain('discover_tools');
-			expect(toolNames).toContain('get_tool_info');
-		});
-	});
-
-	describe('worktreeTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(worktreeTools)).toBe(true);
-			expect(worktreeTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected worktree tools', () => {
-			const toolNames = worktreeTools.map(t => t.name);
-			expect(toolNames).toContain('get_stale_worktrees');
-			expect(toolNames).toContain('clear_worktree_path');
-		});
-	});
-
-	describe('projectTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(projectTools)).toBe(true);
-			expect(projectTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected project tools', () => {
-			const toolNames = projectTools.map(t => t.name);
-			expect(toolNames).toContain('get_project_context');
-			expect(toolNames).toContain('create_project');
-			expect(toolNames).toContain('update_project');
-			expect(toolNames).toContain('get_project_summary');
-		});
-	});
-
-	describe('blockerTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(blockerTools)).toBe(true);
-			expect(blockerTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected blocker tools', () => {
-			const toolNames = blockerTools.map(t => t.name);
-			expect(toolNames).toContain('add_blocker');
-			expect(toolNames).toContain('resolve_blocker');
-			expect(toolNames).toContain('get_blockers');
-			expect(toolNames).toContain('delete_blocker');
-		});
-	});
-
-	describe('decisionTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(decisionTools)).toBe(true);
-			expect(decisionTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected decision tools', () => {
-			const toolNames = decisionTools.map(t => t.name);
-			expect(toolNames).toContain('log_decision');
-			expect(toolNames).toContain('get_decisions');
-			expect(toolNames).toContain('delete_decision');
-		});
-	});
-
-	describe('ideaTools', () => {
-		it('should export an array of tools', () => {
-			expect(Array.isArray(ideaTools)).toBe(true);
-			expect(ideaTools.length).toBeGreaterThan(0);
-		});
-
-		it('should include expected idea tools', () => {
-			const toolNames = ideaTools.map(t => t.name);
-			expect(toolNames).toContain('add_idea');
-			expect(toolNames).toContain('update_idea');
-			expect(toolNames).toContain('get_ideas');
-			expect(toolNames).toContain('convert_idea_to_task');
-		});
-	});
-
-	describe('buildToolList', () => {
-		it('should combine all domain tools', () => {
-			const allTools = buildToolList();
-			expect(Array.isArray(allTools)).toBe(true);
-
-			// Should include tools from all domains
-			const toolNames = allTools.map(t => t.name);
-			expect(toolNames).toContain('start_work_session');
-			expect(toolNames).toContain('get_cost_summary');
-			expect(toolNames).toContain('discover_tools');
-			expect(toolNames).toContain('get_stale_worktrees');
-			expect(toolNames).toContain('get_project_context');
-			expect(toolNames).toContain('add_blocker');
-			expect(toolNames).toContain('log_decision');
-			expect(toolNames).toContain('add_idea');
-		});
-
-		it('should have unique tool names', () => {
-			const allTools = buildToolList();
-			const toolNames = allTools.map(t => t.name);
-			const uniqueNames = new Set(toolNames);
-			expect(uniqueNames.size).toBe(toolNames.length);
-		});
-	});
-
-	describe('toolCategories', () => {
-		it('should have categories for all domains', () => {
-			expect(toolCategories.session).toBeDefined();
-			expect(toolCategories.cost).toBeDefined();
-			expect(toolCategories.discovery).toBeDefined();
-			expect(toolCategories.worktrees).toBeDefined();
-			expect(toolCategories.project).toBeDefined();
-			expect(toolCategories.blockers).toBeDefined();
-			expect(toolCategories.decisions).toBeDefined();
-			expect(toolCategories.ideas).toBeDefined();
-		});
-
-		it('should have correct tool names per category', () => {
-			expect(toolCategories.session).toContain('start_work_session');
-			expect(toolCategories.cost).toContain('get_cost_summary');
-			expect(toolCategories.discovery).toContain('discover_tools');
-			expect(toolCategories.worktrees).toContain('get_stale_worktrees');
-			expect(toolCategories.project).toContain('get_project_context');
-			expect(toolCategories.blockers).toContain('add_blocker');
-			expect(toolCategories.decisions).toContain('log_decision');
-			expect(toolCategories.ideas).toContain('add_idea');
-		});
-	});
-
-	describe('tool structure validation', () => {
-		const allTools = buildToolList();
-
-		it('all tools should have a name', () => {
-			for (const tool of allTools) {
-				expect(tool.name).toBeDefined();
-				expect(typeof tool.name).toBe('string');
-				expect(tool.name.length).toBeGreaterThan(0);
-			}
-		});
-
-		it('all tools should have a description', () => {
-			for (const tool of allTools) {
-				expect(tool.description).toBeDefined();
-				expect(typeof tool.description).toBe('string');
-			}
-		});
-
-		it('all tools should have an inputSchema', () => {
-			for (const tool of allTools) {
-				expect(tool.inputSchema).toBeDefined();
-				expect(tool.inputSchema.type).toBe('object');
-			}
-		});
-	});
-});
+/**
+ * Tests for split tool definitions
+ *
+ * Validates that domain-specific tool files:
+ * - Export valid Tool arrays
+ * - Have unique tool names
+ * - Have required inputSchema properties
+ */
+
+import { describe, it, expect } from 'vitest';
+import { sessionTools } from './session.js';
+import { costTools } from './cost.js';
+import { discoveryTools } from './discovery.js';
+import { worktreeTools } from './worktrees.js';
+import { projectTools } from './project.js';
+import { blockerTools } from './blockers.js';
+import { decisionTools } from './decisions.js';
+import { ideaTools } from './ideas.js';
+import { buildToolList, toolCategories } from './index.js';
+
+describe('Split Tool Definitions', () => {
+	describe('sessionTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(sessionTools)).toBe(true);
+			expect(sessionTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected session tools', () => {
+			const toolNames = sessionTools.map(t => t.name);
+			expect(toolNames).toContain('start_work_session');
+			expect(toolNames).toContain('heartbeat');
+			expect(toolNames).toContain('end_work_session');
+			expect(toolNames).toContain('signal_idle');
+			expect(toolNames).toContain('get_token_usage');
+			expect(toolNames).toContain('report_token_usage');
+		});
+
+		it('should have valid inputSchema for all tools', () => {
+			for (const tool of sessionTools) {
+				expect(tool.inputSchema).toBeDefined();
+				expect(tool.inputSchema.type).toBe('object');
+				expect(tool.inputSchema.properties).toBeDefined();
+			}
+		});
+	});
+
+	describe('costTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(costTools)).toBe(true);
+			expect(costTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected cost tools', () => {
+			const toolNames = costTools.map(t => t.name);
+			expect(toolNames).toContain('get_cost_summary');
+			expect(toolNames).toContain('add_cost_alert');
+			expect(toolNames).toContain('update_cost_alert');
+			expect(toolNames).toContain('delete_cost_alert');
+		});
+	});
+
+	describe('discoveryTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(discoveryTools)).toBe(true);
+			expect(discoveryTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected discovery tools', () => {
+			const toolNames = discoveryTools.map(t => t.name);
+			expect(toolNames).toContain('query_knowledge_base');
+			expect(toolNames).toContain('discover_tools');
+			expect(toolNames).toContain('get_tool_info');
+		});
+	});
+
+	describe('worktreeTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(worktreeTools)).toBe(true);
+			expect(worktreeTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected worktree tools', () => {
+			const toolNames = worktreeTools.map(t => t.name);
+			expect(toolNames).toContain('get_stale_worktrees');
+			expect(toolNames).toContain('clear_worktree_path');
+		});
+	});
+
+	describe('projectTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(projectTools)).toBe(true);
+			expect(projectTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected project tools', () => {
+			const toolNames = projectTools.map(t => t.name);
+			expect(toolNames).toContain('get_project_context');
+			expect(toolNames).toContain('create_project');
+			expect(toolNames).toContain('update_project');
+			expect(toolNames).toContain('get_project_summary');
+		});
+	});
+
+	describe('blockerTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(blockerTools)).toBe(true);
+			expect(blockerTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected blocker tools', () => {
+			const toolNames = blockerTools.map(t => t.name);
+			expect(toolNames).toContain('add_blocker');
+			expect(toolNames).toContain('resolve_blocker');
+			expect(toolNames).toContain('get_blockers');
+			expect(toolNames).toContain('delete_blocker');
+		});
+	});
+
+	describe('decisionTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(decisionTools)).toBe(true);
+			expect(decisionTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected decision tools', () => {
+			const toolNames = decisionTools.map(t => t.name);
+			expect(toolNames).toContain('log_decision');
+			expect(toolNames).toContain('get_decisions');
+			expect(toolNames).toContain('delete_decision');
+		});
+	});
+
+	describe('ideaTools', () => {
+		it('should export an array of tools', () => {
+			expect(Array.isArray(ideaTools)).toBe(true);
+			expect(ideaTools.length).toBeGreaterThan(0);
+		});
+
+		it('should include expected idea tools', () => {
+			const toolNames = ideaTools.map(t => t.name);
+			expect(toolNames).toContain('add_idea');
+			expect(toolNames).toContain('update_idea');
+			expect(toolNames).toContain('get_ideas');
+			expect(toolNames).toContain('convert_idea_to_task');
+		});
+	});
+
+	describe('buildToolList', () => {
+		it('should combine all domain tools', () => {
+			const allTools = buildToolList();
+			expect(Array.isArray(allTools)).toBe(true);
+
+			// Should include tools from all domains
+			const toolNames = allTools.map(t => t.name);
+			expect(toolNames).toContain('start_work_session');
+			expect(toolNames).toContain('get_cost_summary');
+			expect(toolNames).toContain('discover_tools');
+			expect(toolNames).toContain('get_stale_worktrees');
+			expect(toolNames).toContain('get_project_context');
+			expect(toolNames).toContain('add_blocker');
+			expect(toolNames).toContain('log_decision');
+			expect(toolNames).toContain('add_idea');
+		});
+
+		it('should have unique tool names', () => {
+			const allTools = buildToolList();
+			const toolNames = allTools.map(t => t.name);
+			const uniqueNames = new Set(toolNames);
+			expect(uniqueNames.size).toBe(toolNames.length);
+		});
+	});
+
+	describe('toolCategories', () => {
+		it('should have categories for all domains', () => {
+			expect(toolCategories.session).toBeDefined();
+			expect(toolCategories.cost).toBeDefined();
+			expect(toolCategories.discovery).toBeDefined();
+			expect(toolCategories.worktrees).toBeDefined();
+			expect(toolCategories.project).toBeDefined();
+			expect(toolCategories.blockers).toBeDefined();
+			expect(toolCategories.decisions).toBeDefined();
+			expect(toolCategories.ideas).toBeDefined();
+		});
+
+		it('should have correct tool names per category', () => {
+			expect(toolCategories.session).toContain('start_work_session');
+			expect(toolCategories.cost).toContain('get_cost_summary');
+			expect(toolCategories.discovery).toContain('discover_tools');
+			expect(toolCategories.worktrees).toContain('get_stale_worktrees');
+			expect(toolCategories.project).toContain('get_project_context');
+			expect(toolCategories.blockers).toContain('add_blocker');
+			expect(toolCategories.decisions).toContain('log_decision');
+			expect(toolCategories.ideas).toContain('add_idea');
+		});
+	});
+
+	describe('tool structure validation', () => {
+		const allTools = buildToolList();
+
+		it('all tools should have a name', () => {
+			for (const tool of allTools) {
+				expect(tool.name).toBeDefined();
+				expect(typeof tool.name).toBe('string');
+				expect(tool.name.length).toBeGreaterThan(0);
+			}
+		});
+
+		it('all tools should have a description', () => {
+			for (const tool of allTools) {
+				expect(tool.description).toBeDefined();
+				expect(typeof tool.description).toBe('string');
+			}
+		});
+
+		it('all tools should have an inputSchema', () => {
+			for (const tool of allTools) {
+				expect(tool.inputSchema).toBeDefined();
+				expect(tool.inputSchema.type).toBe('object');
+			}
+		});
+	});
+});

package/src/tools/types.ts

@@ -1,9 +1,9 @@
-/**
- * Tool Definition Types
- *
- * Re-export the Tool type from the MCP SDK for use in domain-specific tool files.
- */
-
-import type { Tool } from '@modelcontextprotocol/sdk/types.js';
-
-export type { Tool };
+/**
+ * Tool Definition Types
+ *
+ * Re-export the Tool type from the MCP SDK for use in domain-specific tool files.
+ */
+
+import type { Tool } from '@modelcontextprotocol/sdk/types.js';
+
+export type { Tool };

package/src/tools/validation.ts

@@ -1,75 +1,75 @@
-/**
- * Validation Tool Definitions
- *
- * Tools for task validation workflow:
- * - get_tasks_awaiting_validation
- * - claim_validation
- * - validate_task
- */
-
-import type { Tool } from './types.js';
-
-export const validationTools: Tool[] = [
-	{
-		name: 'get_tasks_awaiting_validation',
-		description: `Get completed tasks not yet validated.`,
-		inputSchema: {
-			type: 'object',
-			properties: {
-				project_id: {
-					type: 'string',
-					description: 'Project UUID',
-				},
-			},
-			required: ['project_id'],
-		},
-	},
-	{
-		name: 'claim_validation',
-		description: 'Claim a completed task for review. Shows "being reviewed" on dashboard.',
-		inputSchema: {
-			type: 'object',
-			properties: {
-				task_id: {
-					type: 'string',
-					description: 'Task UUID to claim for review',
-				},
-			},
-			required: ['task_id'],
-		},
-	},
-	{
-		name: 'validate_task',
-		description: 'Validate a completed task. Include test results in validation_notes. For github-flow/git-flow projects, a PR must exist and checks must pass before approval.',
-		inputSchema: {
-			type: 'object',
-			properties: {
-				task_id: {
-					type: 'string',
-					description: 'Task UUID to validate',
-				},
-				validation_notes: {
-					type: 'string',
-					description: 'Notes about the validation (what was checked, any issues found)',
-				},
-				approved: {
-					type: 'boolean',
-					description: 'Whether the task passes validation (true = approved, false = needs more work)',
-				},
-				pr_checks_passing: {
-					type: 'boolean',
-					description: 'REQUIRED when approving tasks with PRs. Confirm you verified PR checks are passing via `gh pr view <PR_NUMBER> --json statusCheckRollup`. Set to true only if all required checks pass.',
-				},
-				skip_pr_check: {
-					type: 'boolean',
-					description: 'Skip PR existence check (use only for tasks that legitimately do not need a PR)',
-				},
-				create_fix_task: {
-					type: 'boolean',
-					description: 'When rejecting (approved: false), create a new fix task linked to the original. The fix task will contain the rejection reason and be assigned the same git branch.',
-				},
-			},
-			required: ['task_id', 'approved'],
-		},
-	},
-];
+/**
+ * Validation Tool Definitions
+ *
+ * Tools for task validation workflow:
+ * - get_tasks_awaiting_validation
+ * - claim_validation
+ * - validate_task
+ */
+
+import type { Tool } from './types.js';
+
+export const validationTools: Tool[] = [
+	{
+		name: 'get_tasks_awaiting_validation',
+		description: `Get completed tasks not yet validated.`,
+		inputSchema: {
+			type: 'object',
+			properties: {
+				project_id: {
+					type: 'string',
+					description: 'Project UUID',
+				},
+			},
+			required: ['project_id'],
+		},
+	},
+	{
+		name: 'claim_validation',
+		description: 'Claim a completed task for review. Shows "being reviewed" on dashboard.',
+		inputSchema: {
+			type: 'object',
+			properties: {
+				task_id: {
+					type: 'string',
+					description: 'Task UUID to claim for review',
+				},
+			},
+			required: ['task_id'],
+		},
+	},
+	{
+		name: 'validate_task',
+		description: 'Validate a completed task. Include test results in validation_notes. For github-flow/git-flow projects, a PR must exist and checks must pass before approval.',
+		inputSchema: {
+			type: 'object',
+			properties: {
+				task_id: {
+					type: 'string',
+					description: 'Task UUID to validate',
+				},
+				validation_notes: {
+					type: 'string',
+					description: 'Notes about the validation (what was checked, any issues found)',
+				},
+				approved: {
+					type: 'boolean',
+					description: 'Whether the task passes validation (true = approved, false = needs more work)',
+				},
+				pr_checks_passing: {
+					type: 'boolean',
+					description: 'REQUIRED when approving tasks with PRs. Confirm you verified PR checks are passing via `gh pr view <PR_NUMBER> --json statusCheckRollup`. Set to true only if all required checks pass.',
+				},
+				skip_pr_check: {
+					type: 'boolean',
+					description: 'Skip PR existence check (use only for tasks that legitimately do not need a PR)',
+				},
+				create_fix_task: {
+					type: 'boolean',
+					description: 'When rejecting (approved: false), create a new fix task linked to the original. The fix task will contain the rejection reason and be assigned the same git branch.',
+				},
+			},
+			required: ['task_id', 'approved'],
+		},
+	},
+];