@bbearai/mcp-server 0.3.3 → 0.5.0

package/dist/index-api.js

@@ -115,7 +115,7 @@ const tools = [
                     enum: ['new', 'reviewed', 'in_progress', 'resolved', 'closed'],
                     description: 'New status',
                 },
-                resolution: {
+                resolution_notes: {
                     type: 'string',
                     description: 'Resolution notes (optional)',
                 },
@@ -227,6 +227,33 @@ const tools = [
             required: ['test_key', 'title', 'steps', 'expected_result'],
         },
     },
+    {
+        name: 'delete_test_cases',
+        description: 'Delete one or more test cases. Supports single delete by ID or test_key, or bulk delete with arrays (max 50). WARNING: cascade-deletes associated test_assignments, test_feedback, and ai_test_runs.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                test_case_id: {
+                    type: 'string',
+                    description: 'UUID of a single test case to delete',
+                },
+                test_key: {
+                    type: 'string',
+                    description: 'Delete a single test case by test_key (e.g., TC-001)',
+                },
+                test_case_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of test case UUIDs to bulk delete (max 50)',
+                },
+                test_keys: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of test_keys to bulk delete (max 50)',
+                },
+            },
+        },
+    },
     {
         name: 'get_qa_tracks',
         description: 'Get QA tracks for the project',
@@ -267,7 +294,7 @@ async function handleTool(name, args) {
         case 'update_report_status':
             result = await apiRequest(`/reports/${args.report_id}`, 'PATCH', {
                 status: args.status,
-                resolution: args.resolution,
+                resolution_notes: args.resolution_notes,
             });
             break;
         case 'create_bug_report': {
@@ -319,6 +346,24 @@ async function handleTool(name, args) {
                 priority: args.priority || 'P2',
             });
             break;
+        case 'delete_test_cases': {
+            const ids = args.test_case_ids || (args.test_case_id ? [args.test_case_id] : undefined);
+            const keys = args.test_keys || (args.test_key ? [args.test_key] : undefined);
+            if (ids) {
+                const params = new URLSearchParams();
+                ids.forEach((id) => params.append('ids', id));
+                result = await apiRequest(`/test-cases?${params.toString()}`, 'DELETE');
+            }
+            else if (keys) {
+                const params = new URLSearchParams();
+                keys.forEach((k) => params.append('keys', k));
+                result = await apiRequest(`/test-cases?${params.toString()}`, 'DELETE');
+            }
+            else {
+                result = { error: 'Must provide test_case_id, test_key, test_case_ids, or test_keys' };
+            }
+            break;
+        }
         case 'get_qa_tracks':
             result = await apiRequest('/qa-tracks');
             break;

package/dist/index.js

@@ -127,7 +127,7 @@ const tools = [
                     enum: ['new', 'triaging', 'confirmed', 'in_progress', 'fixed', 'resolved', 'verified', 'wont_fix', 'duplicate'],
                     description: 'The new status for the report',
                 },
-                resolution: {
+                resolution_notes: {
                     type: 'string',
                     description: 'Optional resolution notes when marking as resolved',
                 },
@@ -275,6 +275,33 @@ const tools = [
             },
         },
     },
+    {
+        name: 'delete_test_cases',
+        description: 'Delete one or more test cases. Supports single delete by ID or test_key, or bulk delete with arrays (max 50). WARNING: cascade-deletes associated test_assignments, test_feedback, and ai_test_runs. Reports/qa_findings keep their records but test_case_id becomes null.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                test_case_id: {
+                    type: 'string',
+                    description: 'UUID of a single test case to delete',
+                },
+                test_key: {
+                    type: 'string',
+                    description: 'Delete a single test case by test_key (e.g., TC-001)',
+                },
+                test_case_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of test case UUIDs to bulk delete (max 50)',
+                },
+                test_keys: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of test_keys to bulk delete (max 50)',
+                },
+            },
+        },
+    },
     {
         name: 'list_test_cases',
         description: 'List all test cases in the project. Returns test_key, title, target_route, and other metadata. Use this to see existing tests before updating them.',
@@ -880,7 +907,7 @@ const tools = [
 async function listReports(args) {
     let query = supabase
         .from('reports')
-        .select('id, report_type, severity, status, description, app_context, created_at, tester:testers(name, email)')
+        .select('id, report_type, severity, status, description, app_context, created_at, reporter_name, reporter_email, tester:testers(name, email)')
         .eq('project_id', PROJECT_ID)
         .order('created_at', { ascending: false })
         .limit(Math.min(args.limit || 10, 50));
@@ -902,7 +929,7 @@ async function listReports(args) {
             status: r.status,
             description: r.description,
             route: r.app_context?.currentRoute,
-            reporter: r.tester?.name || 'Anonymous',
+            reporter: r.tester?.name || r.reporter_name || 'Anonymous',
             created_at: r.created_at,
         })),
         total: data?.length || 0,
@@ -937,7 +964,10 @@ async function getReport(args) {
             reporter: data.tester ? {
                 name: data.tester.name,
                 email: data.tester.email,
-            } : null,
+            } : (data.reporter_name ? {
+                name: data.reporter_name,
+                email: data.reporter_email,
+            } : null),
             track: data.track ? {
                 name: data.track.name,
                 icon: data.track.icon,
@@ -987,8 +1017,8 @@ async function updateReportStatus(args) {
         return { error: 'Invalid report_id format' };
     }
     const updates = { status: args.status };
-    if (args.resolution) {
-        updates.resolution = args.resolution;
+    if (args.resolution_notes) {
+        updates.resolution_notes = args.resolution_notes;
     }
     const { error } = await supabase
         .from('reports')
@@ -1194,6 +1224,110 @@ async function updateTestCase(args) {
         updatedFields: Object.keys(updates),
     };
 }
+async function deleteTestCases(args) {
+    // Validate: exactly one input mode
+    const modes = [
+        args.test_case_id,
+        args.test_key,
+        args.test_case_ids,
+        args.test_keys,
+    ].filter(v => v !== undefined && v !== null);
+    if (modes.length === 0) {
+        return { error: 'Must provide one of: test_case_id, test_key, test_case_ids, or test_keys' };
+    }
+    if (modes.length > 1) {
+        return { error: 'Provide only one of: test_case_id, test_key, test_case_ids, or test_keys' };
+    }
+    let idsToDelete = [];
+    // Single ID
+    if (args.test_case_id) {
+        if (!isValidUUID(args.test_case_id)) {
+            return { error: 'Invalid test_case_id format (must be UUID)' };
+        }
+        idsToDelete = [args.test_case_id];
+    }
+    // Single key
+    if (args.test_key) {
+        const { data: existing } = await supabase
+            .from('test_cases')
+            .select('id')
+            .eq('project_id', PROJECT_ID)
+            .eq('test_key', args.test_key)
+            .single();
+        if (!existing) {
+            return { error: `Test case with key "${args.test_key}" not found` };
+        }
+        idsToDelete = [existing.id];
+    }
+    // Bulk IDs
+    if (args.test_case_ids) {
+        if (!Array.isArray(args.test_case_ids) || args.test_case_ids.length === 0) {
+            return { error: 'test_case_ids must be a non-empty array' };
+        }
+        if (args.test_case_ids.length > 50) {
+            return { error: 'Cannot delete more than 50 test cases at once' };
+        }
+        const invalidIds = args.test_case_ids.filter(id => !isValidUUID(id));
+        if (invalidIds.length > 0) {
+            return { error: `Invalid UUID(s): ${invalidIds.join(', ')}` };
+        }
+        idsToDelete = args.test_case_ids;
+    }
+    // Bulk keys
+    if (args.test_keys) {
+        if (!Array.isArray(args.test_keys) || args.test_keys.length === 0) {
+            return { error: 'test_keys must be a non-empty array' };
+        }
+        if (args.test_keys.length > 50) {
+            return { error: 'Cannot delete more than 50 test cases at once' };
+        }
+        const { data: existing, error: lookupError } = await supabase
+            .from('test_cases')
+            .select('id, test_key')
+            .eq('project_id', PROJECT_ID)
+            .in('test_key', args.test_keys);
+        if (lookupError) {
+            return { error: lookupError.message };
+        }
+        const foundKeys = (existing || []).map((tc) => tc.test_key);
+        const notFound = args.test_keys.filter(k => !foundKeys.includes(k));
+        if (notFound.length > 0) {
+            return { error: `Test case(s) not found: ${notFound.join(', ')}` };
+        }
+        idsToDelete = (existing || []).map((tc) => tc.id);
+    }
+    // Pre-fetch details for the response
+    const { data: toDelete } = await supabase
+        .from('test_cases')
+        .select('id, test_key, title')
+        .eq('project_id', PROJECT_ID)
+        .in('id', idsToDelete);
+    if (!toDelete || toDelete.length === 0) {
+        return { error: 'No matching test cases found in this project' };
+    }
+    // Delete
+    const { error: deleteError } = await supabase
+        .from('test_cases')
+        .delete()
+        .eq('project_id', PROJECT_ID)
+        .in('id', idsToDelete);
+    if (deleteError) {
+        return { error: deleteError.message };
+    }
+    return {
+        success: true,
+        deletedCount: toDelete.length,
+        deleted: toDelete.map((tc) => ({
+            id: tc.id,
+            testKey: tc.test_key,
+            title: tc.title,
+        })),
+        message: toDelete.length === 1
+            ? `Test case ${toDelete[0].test_key} deleted successfully`
+            : `${toDelete.length} test cases deleted successfully`,
+        warning: 'Associated test_assignments, test_feedback, and ai_test_runs have been cascade-deleted. Reports and qa_findings referencing these tests now have null test_case_id.',
+    };
+}
 async function listTestCases(args) {
     let query = supabase
         .from('test_cases')
@@ -3157,7 +3291,7 @@ async function markFixedWithCommit(args) {
     const updates = {
         status: 'resolved',
         resolved_at: new Date().toISOString(),
-        resolution: args.resolution_notes || `Fixed in commit ${args.commit_sha.slice(0, 7)}`,
+        resolution_notes: args.resolution_notes || `Fixed in commit ${args.commit_sha.slice(0, 7)}`,
         code_context: {
             ...existingContext,
             fix: {
@@ -3387,7 +3521,7 @@ async function createRegressionTest(args) {
             {
                 stepNumber: 3,
                 action: 'Verify the fix is working',
-                expectedResult: report.resolution || 'Feature works as expected',
+                expectedResult: report.resolution_notes || 'Feature works as expected',
             },
         ],
         expected_result: `The bug "${report.title}" should not recur`,
@@ -3952,6 +4086,9 @@ async function main() {
                 case 'update_test_case':
                     result = await updateTestCase(args);
                     break;
+                case 'delete_test_cases':
+                    result = await deleteTestCases(args);
+                    break;
                 case 'list_test_cases':
                     result = await listTestCases(args);
                     break;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bbearai/mcp-server",
-  "version": "0.3.3",
+  "version": "0.5.0",
   "description": "MCP server for BugBear - allows Claude Code to query bug reports",
   "main": "dist/index.js",
   "bin": {

package/src/index-api.ts

@@ -40,7 +40,7 @@ function validateConfig() {
 // API helper
 async function apiRequest(
   endpoint: string,
-  method: 'GET' | 'POST' | 'PATCH' = 'GET',
+  method: 'GET' | 'POST' | 'PATCH' | 'DELETE' = 'GET',
   body?: Record<string, unknown>
 ): Promise<{ data?: unknown; error?: string }> {
   try {
@@ -129,7 +129,7 @@ const tools = [
           enum: ['new', 'reviewed', 'in_progress', 'resolved', 'closed'],
           description: 'New status',
         },
-        resolution: {
+        resolution_notes: {
           type: 'string',
           description: 'Resolution notes (optional)',
         },
@@ -241,6 +241,33 @@ const tools = [
       required: ['test_key', 'title', 'steps', 'expected_result'],
     },
   },
+  {
+    name: 'delete_test_cases',
+    description: 'Delete one or more test cases. Supports single delete by ID or test_key, or bulk delete with arrays (max 50). WARNING: cascade-deletes associated test_assignments, test_feedback, and ai_test_runs.',
+    inputSchema: {
+      type: 'object' as const,
+      properties: {
+        test_case_id: {
+          type: 'string',
+          description: 'UUID of a single test case to delete',
+        },
+        test_key: {
+          type: 'string',
+          description: 'Delete a single test case by test_key (e.g., TC-001)',
+        },
+        test_case_ids: {
+          type: 'array',
+          items: { type: 'string' },
+          description: 'Array of test case UUIDs to bulk delete (max 50)',
+        },
+        test_keys: {
+          type: 'array',
+          items: { type: 'string' },
+          description: 'Array of test_keys to bulk delete (max 50)',
+        },
+      },
+    },
+  },
   {
     name: 'get_qa_tracks',
     description: 'Get QA tracks for the project',
@@ -284,7 +311,7 @@ async function handleTool(
     case 'update_report_status':
       result = await apiRequest(`/reports/${args.report_id}`, 'PATCH', {
         status: args.status,
-        resolution: args.resolution,
+        resolution_notes: args.resolution_notes,
       });
       break;
 
@@ -340,6 +367,24 @@ async function handleTool(
       });
       break;
 
+    case 'delete_test_cases': {
+      const ids = args.test_case_ids as string[] || (args.test_case_id ? [args.test_case_id as string] : undefined);
+      const keys = args.test_keys as string[] || (args.test_key ? [args.test_key as string] : undefined);
+
+      if (ids) {
+        const params = new URLSearchParams();
+        ids.forEach((id: string) => params.append('ids', id));
+        result = await apiRequest(`/test-cases?${params.toString()}`, 'DELETE');
+      } else if (keys) {
+        const params = new URLSearchParams();
+        keys.forEach((k: string) => params.append('keys', k));
+        result = await apiRequest(`/test-cases?${params.toString()}`, 'DELETE');
+      } else {
+        result = { error: 'Must provide test_case_id, test_key, test_case_ids, or test_keys' };
+      }
+      break;
+    }
+
     case 'get_qa_tracks':
       result = await apiRequest('/qa-tracks');
       break;

package/src/index.ts

@@ -144,7 +144,7 @@ const tools = [
           enum: ['new', 'triaging', 'confirmed', 'in_progress', 'fixed', 'resolved', 'verified', 'wont_fix', 'duplicate'],
           description: 'The new status for the report',
         },
-        resolution: {
+        resolution_notes: {
           type: 'string',
           description: 'Optional resolution notes when marking as resolved',
         },
@@ -292,6 +292,33 @@ const tools = [
       },
     },
   },
+  {
+    name: 'delete_test_cases',
+    description: 'Delete one or more test cases. Supports single delete by ID or test_key, or bulk delete with arrays (max 50). WARNING: cascade-deletes associated test_assignments, test_feedback, and ai_test_runs. Reports/qa_findings keep their records but test_case_id becomes null.',
+    inputSchema: {
+      type: 'object' as const,
+      properties: {
+        test_case_id: {
+          type: 'string',
+          description: 'UUID of a single test case to delete',
+        },
+        test_key: {
+          type: 'string',
+          description: 'Delete a single test case by test_key (e.g., TC-001)',
+        },
+        test_case_ids: {
+          type: 'array',
+          items: { type: 'string' },
+          description: 'Array of test case UUIDs to bulk delete (max 50)',
+        },
+        test_keys: {
+          type: 'array',
+          items: { type: 'string' },
+          description: 'Array of test_keys to bulk delete (max 50)',
+        },
+      },
+    },
+  },
   {
     name: 'list_test_cases',
     description: 'List all test cases in the project. Returns test_key, title, target_route, and other metadata. Use this to see existing tests before updating them.',
@@ -1024,15 +1051,15 @@ async function searchReports(args: { query?: string; route?: string }) {
 async function updateReportStatus(args: {
   report_id: string;
   status: string;
-  resolution?: string;
+  resolution_notes?: string;
 }) {
   if (!isValidUUID(args.report_id)) {
     return { error: 'Invalid report_id format' };
   }
 
   const updates: Record<string, unknown> = { status: args.status };
-  if (args.resolution) {
-    updates.resolution = args.resolution;
+  if (args.resolution_notes) {
+    updates.resolution_notes = args.resolution_notes;
   }
 
   const { error } = await supabase
@@ -1281,6 +1308,130 @@ async function updateTestCase(args: {
   };
 }
 
+async function deleteTestCases(args: {
+  test_case_id?: string;
+  test_key?: string;
+  test_case_ids?: string[];
+  test_keys?: string[];
+}) {
+  // Validate: exactly one input mode
+  const modes = [
+    args.test_case_id,
+    args.test_key,
+    args.test_case_ids,
+    args.test_keys,
+  ].filter(v => v !== undefined && v !== null);
+
+  if (modes.length === 0) {
+    return { error: 'Must provide one of: test_case_id, test_key, test_case_ids, or test_keys' };
+  }
+  if (modes.length > 1) {
+    return { error: 'Provide only one of: test_case_id, test_key, test_case_ids, or test_keys' };
+  }
+
+  let idsToDelete: string[] = [];
+
+  // Single ID
+  if (args.test_case_id) {
+    if (!isValidUUID(args.test_case_id)) {
+      return { error: 'Invalid test_case_id format (must be UUID)' };
+    }
+    idsToDelete = [args.test_case_id];
+  }
+
+  // Single key
+  if (args.test_key) {
+    const { data: existing } = await supabase
+      .from('test_cases')
+      .select('id')
+      .eq('project_id', PROJECT_ID)
+      .eq('test_key', args.test_key)
+      .single();
+
+    if (!existing) {
+      return { error: `Test case with key "${args.test_key}" not found` };
+    }
+    idsToDelete = [existing.id];
+  }
+
+  // Bulk IDs
+  if (args.test_case_ids) {
+    if (!Array.isArray(args.test_case_ids) || args.test_case_ids.length === 0) {
+      return { error: 'test_case_ids must be a non-empty array' };
+    }
+    if (args.test_case_ids.length > 50) {
+      return { error: 'Cannot delete more than 50 test cases at once' };
+    }
+    const invalidIds = args.test_case_ids.filter(id => !isValidUUID(id));
+    if (invalidIds.length > 0) {
+      return { error: `Invalid UUID(s): ${invalidIds.join(', ')}` };
+    }
+    idsToDelete = args.test_case_ids;
+  }
+
+  // Bulk keys
+  if (args.test_keys) {
+    if (!Array.isArray(args.test_keys) || args.test_keys.length === 0) {
+      return { error: 'test_keys must be a non-empty array' };
+    }
+    if (args.test_keys.length > 50) {
+      return { error: 'Cannot delete more than 50 test cases at once' };
+    }
+    const { data: existing, error: lookupError } = await supabase
+      .from('test_cases')
+      .select('id, test_key')
+      .eq('project_id', PROJECT_ID)
+      .in('test_key', args.test_keys);
+
+    if (lookupError) {
+      return { error: lookupError.message };
+    }
+
+    const foundKeys = (existing || []).map((tc: any) => tc.test_key);
+    const notFound = args.test_keys.filter(k => !foundKeys.includes(k));
+    if (notFound.length > 0) {
+      return { error: `Test case(s) not found: ${notFound.join(', ')}` };
+    }
+    idsToDelete = (existing || []).map((tc: any) => tc.id);
+  }
+
+  // Pre-fetch details for the response
+  const { data: toDelete } = await supabase
+    .from('test_cases')
+    .select('id, test_key, title')
+    .eq('project_id', PROJECT_ID)
+    .in('id', idsToDelete);
+
+  if (!toDelete || toDelete.length === 0) {
+    return { error: 'No matching test cases found in this project' };
+  }
+
+  // Delete
+  const { error: deleteError } = await supabase
+    .from('test_cases')
+    .delete()
+    .eq('project_id', PROJECT_ID)
+    .in('id', idsToDelete);
+
+  if (deleteError) {
+    return { error: deleteError.message };
+  }
+
+  return {
+    success: true,
+    deletedCount: toDelete.length,
+    deleted: toDelete.map((tc: any) => ({
+      id: tc.id,
+      testKey: tc.test_key,
+      title: tc.title,
+    })),
+    message: toDelete.length === 1
+      ? `Test case ${toDelete[0].test_key} deleted successfully`
+      : `${toDelete.length} test cases deleted successfully`,
+    warning: 'Associated test_assignments, test_feedback, and ai_test_runs have been cascade-deleted. Reports and qa_findings referencing these tests now have null test_case_id.',
+  };
+}
+
 async function listTestCases(args: {
   track?: string;
   priority?: string;
@@ -3607,7 +3758,7 @@ async function markFixedWithCommit(args: {
   const updates = {
     status: 'resolved',
     resolved_at: new Date().toISOString(),
-    resolution: args.resolution_notes || `Fixed in commit ${args.commit_sha.slice(0, 7)}`,
+    resolution_notes: args.resolution_notes || `Fixed in commit ${args.commit_sha.slice(0, 7)}`,
     code_context: {
       ...existingContext,
       fix: {
@@ -3889,7 +4040,7 @@ async function createRegressionTest(args: {
       {
         stepNumber: 3,
         action: 'Verify the fix is working',
-        expectedResult: report.resolution || 'Feature works as expected',
+        expectedResult: report.resolution_notes || 'Feature works as expected',
       },
     ],
     expected_result: `The bug "${report.title}" should not recur`,
@@ -4527,6 +4678,9 @@ async function main() {
         case 'update_test_case':
           result = await updateTestCase(args as any);
           break;
+        case 'delete_test_cases':
+          result = await deleteTestCases(args as any);
+          break;
         case 'list_test_cases':
           result = await listTestCases(args as any);
           break;