@vibescope/mcp-server 0.0.1 → 0.2.0

package/src/handlers/findings.ts

@@ -3,13 +3,15 @@
  *
  * Handles audit findings and knowledge base:
  * - add_finding
- * - get_findings
+ * - get_findings (supports summary_only for reduced tokens)
+ * - get_findings_stats (aggregate counts for minimal tokens)
  * - update_finding
  * - delete_finding
  */
 
 import type { Handler, HandlerRegistry } from './types.js';
 import { validateRequired, validateUUID } from '../validators.js';
+import { getApiClient } from '../api-client.js';
 
 type FindingCategory = 'performance' | 'security' | 'code_quality' | 'accessibility' | 'documentation' | 'architecture' | 'testing' | 'other';
 type FindingSeverity = 'info' | 'low' | 'medium' | 'high' | 'critical';
@@ -32,60 +34,78 @@ export const addFinding: Handler = async (args, ctx) => {
 	validateRequired(title, 'title');
 	if (related_task_id) validateUUID(related_task_id, 'related_task_id');
 
-	const { supabase, session } = ctx;
-
-	const { data, error } = await supabase
-		.from('findings')
-		.insert({
-			project_id,
-			category: category || 'other',
-			title,
-			description: description || null,
-			severity: severity || 'info',
-			file_path: file_path || null,
-			line_number: line_number || null,
-			related_task_id: related_task_id || null,
-			created_by: 'agent',
-			created_by_session_id: session.currentSessionId,
-		})
-		.select('id')
-		.single();
-
-	if (error) throw new Error(`Failed to add finding: ${error.message}`);
-
-	return { result: { success: true, finding_id: data.id, title } };
+	const apiClient = getApiClient();
+	const response = await apiClient.addFinding(project_id, {
+		title,
+		description,
+		category,
+		severity,
+		file_path,
+		line_number,
+		related_task_id
+	}, ctx.session.currentSessionId || undefined);
+
+	if (!response.ok) {
+		throw new Error(response.error || 'Failed to add finding');
+	}
+
+	return { result: response.data };
 };
 
 export const getFindings: Handler = async (args, ctx) => {
-	const { project_id, category, severity, status, limit } = args as {
+	const { project_id, category, severity, status, limit = 50, offset = 0, search_query, summary_only = false } = args as {
 		project_id: string;
 		category?: FindingCategory;
 		severity?: FindingSeverity;
 		status?: FindingStatus;
 		limit?: number;
+		offset?: number;
+		search_query?: string;
+		summary_only?: boolean;
 	};
 
 	validateRequired(project_id, 'project_id');
 	validateUUID(project_id, 'project_id');
 
-	const { supabase } = ctx;
+	const apiClient = getApiClient();
+	const response = await apiClient.getFindings(project_id, {
+		category,
+		severity,
+		status,
+		limit,
+		offset,
+		search_query,
+		summary_only
+	});
+
+	if (!response.ok) {
+		throw new Error(response.error || 'Failed to get findings');
+	}
 
-	let query = supabase
-		.from('findings')
-		.select('id, title, category, severity, status, file_path, created_at')
-		.eq('project_id', project_id)
-		.order('created_at', { ascending: false })
-		.limit(limit || 50);
+	return { result: response.data };
+};
 
-	if (category) query = query.eq('category', category);
-	if (severity) query = query.eq('severity', severity);
-	if (status) query = query.eq('status', status);
+/**
+ * Get aggregate statistics about findings for a project.
+ * Returns counts by category, severity, and status without the actual finding data.
+ * This is much more token-efficient than get_findings for understanding the overall state.
+ */
+export const getFindingsStats: Handler = async (args, ctx) => {
+	const { project_id } = args as {
+		project_id: string;
+	};
 
-	const { data, error } = await query;
+	validateRequired(project_id, 'project_id');
+	validateUUID(project_id, 'project_id');
 
-	if (error) throw new Error(`Failed to get findings: ${error.message}`);
+	const apiClient = getApiClient();
+	const response = await apiClient.getFindingsStats(project_id);
+
+	if (!response.ok) {
+		throw new Error(response.error || 'Failed to get findings stats');
+	}
 
-	return { result: { findings: data } };
+	return { result: response.data };
 };
 
 export const updateFinding: Handler = async (args, ctx) => {
@@ -101,29 +121,20 @@ export const updateFinding: Handler = async (args, ctx) => {
 	validateRequired(finding_id, 'finding_id');
 	validateUUID(finding_id, 'finding_id');
 
-	const { supabase, session } = ctx;
-
-	const updates: Record<string, unknown> = { updated_at: new Date().toISOString() };
-	if (title) updates.title = title;
-	if (description) updates.description = description;
-	if (severity) updates.severity = severity;
-	if (status) {
-		updates.status = status;
-		if (status === 'addressed' || status === 'dismissed' || status === 'wontfix') {
-			updates.addressed_at = new Date().toISOString();
-			updates.addressed_by_session_id = session.currentSessionId;
-		}
+	const apiClient = getApiClient();
+	const response = await apiClient.updateFinding(finding_id, {
+		title,
+		description,
+		severity,
+		status,
+		resolution_note
+	});
+
+	if (!response.ok) {
+		throw new Error(response.error || 'Failed to update finding');
 	}
-	if (resolution_note) updates.resolution_note = resolution_note;
 
-	const { error } = await supabase
-		.from('findings')
-		.update(updates)
-		.eq('id', finding_id);
-
-	if (error) throw new Error(`Failed to update finding: ${error.message}`);
-
-	return { result: { success: true, finding_id } };
+	return { result: response.data };
 };
 
 export const deleteFinding: Handler = async (args, ctx) => {
@@ -132,14 +143,14 @@ export const deleteFinding: Handler = async (args, ctx) => {
 	validateRequired(finding_id, 'finding_id');
 	validateUUID(finding_id, 'finding_id');
 
-	const { error } = await ctx.supabase
-		.from('findings')
-		.delete()
-		.eq('id', finding_id);
+	const apiClient = getApiClient();
+	const response = await apiClient.deleteFinding(finding_id);
 
-	if (error) throw new Error(`Failed to delete finding: ${error.message}`);
+	if (!response.ok) {
+		throw new Error(response.error || 'Failed to delete finding');
+	}
 
-	return { result: { success: true } };
+	return { result: response.data };
 };
 
 /**
@@ -148,6 +159,7 @@ export const deleteFinding: Handler = async (args, ctx) => {
 export const findingHandlers: HandlerRegistry = {
 	add_finding: addFinding,
 	get_findings: getFindings,
+	get_findings_stats: getFindingsStats,
 	update_finding: updateFinding,
 	delete_finding: deleteFinding,
 };