@bbearai/mcp-server 0.5.1 → 0.7.0

package/dist/index.js

@@ -12,26 +12,30 @@ const supabase_js_1 = require("@supabase/supabase-js");
 // Configuration from environment
 const SUPABASE_URL = process.env.SUPABASE_URL || 'https://kyxgzjnqgvapvlnvqawz.supabase.co';
 const SUPABASE_ANON_KEY = process.env.SUPABASE_ANON_KEY || '';
-const PROJECT_ID = process.env.BUGBEAR_PROJECT_ID || '';
+// Active project — set from env var, switchable via switch_project tool
+let currentProjectId = process.env.BUGBEAR_currentProjectId || '';
 // Initialize Supabase client
 let supabase;
 function validateConfig() {
-    const errors = [];
     if (!SUPABASE_ANON_KEY) {
-        errors.push('SUPABASE_ANON_KEY environment variable is required');
+        console.error('BugBear MCP Server: SUPABASE_ANON_KEY environment variable is required');
+        process.exit(1);
     }
-    if (!PROJECT_ID) {
-        errors.push('BUGBEAR_PROJECT_ID environment variable is required');
+    // Validate project ID format if provided
+    if (currentProjectId && !/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(currentProjectId)) {
+        console.error('BugBear MCP Server: BUGBEAR_currentProjectId must be a valid UUID');
+        process.exit(1);
     }
-    // Basic UUID format validation for project ID
-    if (PROJECT_ID && !/^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(PROJECT_ID)) {
-        errors.push('BUGBEAR_PROJECT_ID must be a valid UUID');
+    if (!currentProjectId) {
+        console.error('BugBear MCP Server: No BUGBEAR_currentProjectId set. Use list_projects + switch_project to select one.');
     }
-    if (errors.length > 0) {
-        console.error('BugBear MCP Server configuration errors:');
-        errors.forEach(e => console.error(`  - ${e}`));
-        process.exit(1);
+}
+/** Guard for tools that require a project to be selected */
+function requireProject() {
+    if (!currentProjectId) {
+        throw new Error('No project selected. Use list_projects to see available projects, then switch_project to select one.');
     }
+    return currentProjectId;
 }
 function initSupabase() {
     validateConfig();
@@ -124,7 +128,7 @@ const tools = [
                 },
                 status: {
                     type: 'string',
-                    enum: ['new', 'triaging', 'confirmed', 'in_progress', 'fixed', 'resolved', 'verified', 'wont_fix', 'duplicate'],
+                    enum: ['new', 'triaging', 'confirmed', 'in_progress', 'fixed', 'ready_to_test', 'verified', 'resolved', 'reviewed', 'closed', 'wont_fix', 'duplicate'],
                     description: 'The new status for the report',
                 },
                 resolution_notes: {
@@ -137,7 +141,47 @@ const tools = [
     },
     {
         name: 'get_report_context',
-        description: 'Get the full debugging context for a report including console logs, network requests, and navigation history',
+        description: 'Get the full debugging context for a report including console logs, network requests, and navigation history. Use compact=true for app_context summary only (no console/network/navigation).',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                report_id: {
+                    type: 'string',
+                    description: 'The UUID of the report',
+                },
+                compact: {
+                    type: 'boolean',
+                    description: 'Compact mode: returns app_context only, skips console logs, network requests, and navigation history. (default: false)',
+                },
+            },
+            required: ['report_id'],
+        },
+    },
+    {
+        name: 'add_report_comment',
+        description: 'Add a comment/note to a bug report thread without changing its status. Use this for follow-up questions, investigation notes, or developer-tester communication.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                report_id: {
+                    type: 'string',
+                    description: 'The UUID of the report to comment on',
+                },
+                message: {
+                    type: 'string',
+                    description: 'The comment/note content',
+                },
+                author: {
+                    type: 'string',
+                    description: 'Optional author name (defaults to "Claude Code")',
+                },
+            },
+            required: ['report_id', 'message'],
+        },
+    },
+    {
+        name: 'get_report_comments',
+        description: 'Get all comments/notes on a bug report in chronological order. Returns the full discussion thread.',
         inputSchema: {
             type: 'object',
             properties: {
@@ -304,7 +348,7 @@ const tools = [
     },
     {
         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.',
+        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. Use compact=true for id, test_key, title, and priority only (saves tokens).',
         inputSchema: {
             type: 'object',
             properties: {
@@ -329,6 +373,10 @@ const tools = [
                     type: 'number',
                     description: 'Offset for pagination (default 0)',
                 },
+                compact: {
+                    type: 'boolean',
+                    description: 'Compact mode: returns id, test_key, title, and priority only. (default: false)',
+                },
             },
         },
     },
@@ -437,7 +485,7 @@ const tools = [
                 },
                 notify_tester: {
                     type: 'boolean',
-                    description: 'If true, notify the original tester about the fix with a message and verification task. Default: false (silent resolve).',
+                    description: 'Notify the original tester about the fix with a message and verification task. Default: true. Set to false for silent resolve.',
                 },
             },
             required: ['report_id', 'commit_sha'],
@@ -679,17 +727,17 @@ const tools = [
     },
     {
         name: 'get_coverage_matrix',
-        description: 'Get a comprehensive Route × Track coverage matrix showing test counts, pass rates, and execution data. Use this for a complete view of test coverage.',
+        description: 'Get a comprehensive Route × Track coverage matrix showing test counts, pass rates, and execution data. Use this for a complete view of test coverage. Execution data and bug counts are opt-in to save tokens.',
         inputSchema: {
             type: 'object',
             properties: {
                 include_execution_data: {
                     type: 'boolean',
-                    description: 'Include pass/fail rates and last execution times (default: true)',
+                    description: 'Include pass/fail rates and last execution times (default: false). Set true when you need execution history.',
                 },
                 include_bug_counts: {
                     type: 'boolean',
-                    description: 'Include open/critical bug counts per route (default: true)',
+                    description: 'Include open/critical bug counts per route (default: false). Set true when you need bug context.',
                 },
             },
         },
@@ -923,6 +971,11 @@ const tools = [
                     enum: ['ios', 'android', 'web'],
                     description: 'Filter by platform support',
                 },
+                role: {
+                    type: 'string',
+                    enum: ['tester', 'feedback'],
+                    description: 'Filter by role: "tester" for QA testers, "feedback" for feedback-only users (default: all)',
+                },
             },
         },
     },
@@ -1011,6 +1064,21 @@ const tools = [
             required: ['tester_id', 'test_case_ids'],
         },
     },
+    {
+        name: 'unassign_tests',
+        description: 'Remove one or more test assignments by assignment ID. Preserves the test case and its history — only the assignment link is deleted. Use list_test_assignments first to find assignment IDs. Max 50 per call.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                assignment_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of test assignment UUIDs to remove (required, max 50)',
+                },
+            },
+            required: ['assignment_ids'],
+        },
+    },
     {
         name: 'get_tester_workload',
         description: 'View a specific tester\'s current workload — assignment counts by status and recent assignments.',
@@ -1025,13 +1093,863 @@ const tools = [
             required: ['tester_id'],
         },
     },
+    // === NEW TESTER & ANALYTICS TOOLS ===
+    {
+        name: 'create_tester',
+        description: 'Add a new QA tester to the project without opening the dashboard.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                name: {
+                    type: 'string',
+                    description: 'Full name of the tester (required)',
+                },
+                email: {
+                    type: 'string',
+                    description: 'Email address of the tester (required, must be unique per project)',
+                },
+                platforms: {
+                    type: 'array',
+                    items: { type: 'string', enum: ['ios', 'android', 'web'] },
+                    description: 'Platforms the tester can test on (default: ["ios", "web"])',
+                },
+                tier: {
+                    type: 'number',
+                    description: 'Tester tier 1-3 (default: 1)',
+                },
+                notes: {
+                    type: 'string',
+                    description: 'Optional notes about the tester',
+                },
+                role: {
+                    type: 'string',
+                    enum: ['tester', 'feedback'],
+                    description: 'Role: "tester" for QA testers (default), "feedback" for feedback-only users',
+                },
+            },
+            required: ['name', 'email'],
+        },
+    },
+    {
+        name: 'update_tester',
+        description: 'Update an existing tester\'s status, platforms, tier, or notes.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                tester_id: {
+                    type: 'string',
+                    description: 'UUID of the tester to update (required)',
+                },
+                status: {
+                    type: 'string',
+                    enum: ['active', 'inactive', 'invited'],
+                    description: 'New status for the tester',
+                },
+                platforms: {
+                    type: 'array',
+                    items: { type: 'string', enum: ['ios', 'android', 'web'] },
+                    description: 'Updated platforms array',
+                },
+                tier: {
+                    type: 'number',
+                    description: 'Updated tier (1-3)',
+                },
+                notes: {
+                    type: 'string',
+                    description: 'Updated notes',
+                },
+                name: {
+                    type: 'string',
+                    description: 'Updated name',
+                },
+            },
+            required: ['tester_id'],
+        },
+    },
+    {
+        name: 'bulk_update_reports',
+        description: 'Update the status of multiple bug reports at once. Useful after a fix session to close many bugs.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                report_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Array of report UUIDs to update (required, max 50)',
+                },
+                status: {
+                    type: 'string',
+                    enum: ['new', 'triaging', 'confirmed', 'in_progress', 'fixed', 'resolved', 'verified', 'wont_fix', 'duplicate', 'closed'],
+                    description: 'New status for all reports (required)',
+                },
+                resolution_notes: {
+                    type: 'string',
+                    description: 'Optional resolution notes applied to all reports',
+                },
+            },
+            required: ['report_ids', 'status'],
+        },
+    },
+    {
+        name: 'get_bug_trends',
+        description: 'Get bug report trends over time — grouped by week, severity, category, or status. Useful for spotting patterns.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                group_by: {
+                    type: 'string',
+                    enum: ['week', 'severity', 'category', 'status'],
+                    description: 'How to group the trends (default: week)',
+                },
+                days: {
+                    type: 'number',
+                    description: 'Number of days to look back (default: 30, max: 180)',
+                },
+            },
+        },
+    },
+    {
+        name: 'get_tester_leaderboard',
+        description: 'Rank testers by testing activity — bugs found, tests completed, pass rate, and average test duration.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                days: {
+                    type: 'number',
+                    description: 'Number of days to look back (default: 30, max: 180)',
+                },
+                sort_by: {
+                    type: 'string',
+                    enum: ['bugs_found', 'tests_completed', 'pass_rate'],
+                    description: 'Sort metric (default: tests_completed)',
+                },
+            },
+        },
+    },
+    {
+        name: 'export_test_results',
+        description: 'Export test results for a specific test run as structured JSON — includes every assignment, tester, result, and duration. Use compact=true for summary only (no assignments array). Use limit to cap assignments returned.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                test_run_id: {
+                    type: 'string',
+                    description: 'UUID of the test run to export (required)',
+                },
+                compact: {
+                    type: 'boolean',
+                    description: 'Compact mode: returns test run info + summary only, no assignments array. (default: false)',
+                },
+                limit: {
+                    type: 'number',
+                    description: 'Max assignments to return in full mode (default: 100, max: 500). Ignored when compact=true.',
+                },
+            },
+            required: ['test_run_id'],
+        },
+    },
+    {
+        name: 'get_testing_velocity',
+        description: 'Get a rolling average of test completions per day over the specified window. Shows daily completion counts and trend direction.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                days: {
+                    type: 'number',
+                    description: 'Number of days to analyze (default: 14, max: 90)',
+                },
+            },
+        },
+    },
+    // === PROJECT MANAGEMENT TOOLS ===
+    {
+        name: 'list_projects',
+        description: 'List all BugBear projects accessible with the current credentials. Use this to find project IDs for switch_project.',
+        inputSchema: {
+            type: 'object',
+            properties: {},
+        },
+    },
+    {
+        name: 'switch_project',
+        description: 'Switch the active project. All subsequent tool calls will use this project. Use list_projects first to find the project ID.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                project_id: {
+                    type: 'string',
+                    description: 'UUID of the project to switch to (required)',
+                },
+            },
+            required: ['project_id'],
+        },
+    },
+    {
+        name: 'get_current_project',
+        description: 'Show which project is currently active.',
+        inputSchema: {
+            type: 'object',
+            properties: {},
+        },
+    },
+    // === TEST EXECUTION INTELLIGENCE ===
+    {
+        name: 'get_test_impact',
+        description: 'Given changed files, identify which test cases are affected by mapping file paths to test case target routes.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                changed_files: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'List of changed file paths (relative to project root)',
+                },
+            },
+            required: ['changed_files'],
+        },
+    },
+    {
+        name: 'get_flaky_tests',
+        description: 'Analyze test run history to identify tests with intermittent failure rates above a threshold.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                threshold: {
+                    type: 'number',
+                    description: 'Minimum flakiness rate to report (0-100, default: 5)',
+                },
+                limit: {
+                    type: 'number',
+                    description: 'Maximum results to return (default: 20)',
+                },
+            },
+        },
+    },
+    {
+        name: 'assess_test_quality',
+        description: 'Analyze test case steps for weak patterns: vague assertions, missing edge cases, no negative testing, generic descriptions.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                test_case_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Specific test case IDs to assess. If omitted, assesses recent test cases.',
+                },
+                limit: {
+                    type: 'number',
+                    description: 'Maximum test cases to assess (default: 20)',
+                },
+            },
+        },
+    },
+    {
+        name: 'get_test_execution_summary',
+        description: 'Aggregate test execution metrics: pass rate, completion rate, most-failed tests, fastest/slowest tests.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                days: {
+                    type: 'number',
+                    description: 'Number of days to analyze (default: 30)',
+                },
+            },
+        },
+    },
+    {
+        name: 'check_test_freshness',
+        description: 'Identify test cases that have not been updated since their target code was modified.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                limit: {
+                    type: 'number',
+                    description: 'Maximum results to return (default: 20)',
+                },
+            },
+        },
+    },
+    {
+        name: 'get_untested_changes',
+        description: 'Given recent commits or changed files, find code changes with no corresponding test coverage in BugBear.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                changed_files: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'List of changed file paths. If omitted, uses git diff against main.',
+                },
+            },
+        },
+    },
+    // === AUTO-MONITORING TOOLS ===
+    {
+        name: 'get_auto_detected_issues',
+        description: 'Get auto-detected monitoring issues grouped by error fingerprint. Shows recurring crashes, API failures, and rage clicks with frequency and user impact.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                source: {
+                    type: 'string',
+                    enum: ['auto_crash', 'auto_api', 'auto_rage_click'],
+                    description: 'Filter by source type',
+                },
+                min_occurrences: {
+                    type: 'number',
+                    description: 'Min occurrence count (default: 1)',
+                },
+                since: {
+                    type: 'string',
+                    description: 'ISO date — only issues after this date (default: 7 days ago)',
+                },
+                limit: {
+                    type: 'number',
+                    description: 'Max results (default: 20)',
+                },
+                compact: {
+                    type: 'boolean',
+                    description: 'Compact mode: fingerprint, source, count only',
+                },
+            },
+        },
+    },
+    {
+        name: 'generate_tests_from_errors',
+        description: 'Suggest QA test cases from auto-detected error patterns. Returns structured suggestions — does NOT auto-create test cases.',
+        inputSchema: {
+            type: 'object',
+            properties: {
+                report_ids: {
+                    type: 'array',
+                    items: { type: 'string' },
+                    description: 'Specific report IDs. If omitted, uses top uncovered errors.',
+                },
+                limit: {
+                    type: 'number',
+                    description: 'Max suggestions (default: 5)',
+                },
+            },
+        },
+    },
 ];
+// === TEST EXECUTION INTELLIGENCE ===
+async function getTestImpact(args) {
+    const projectId = requireProject();
+    const changedFiles = args.changed_files || [];
+    if (changedFiles.length === 0) {
+        return { affectedTests: [], message: 'No changed files provided.' };
+    }
+    // Get all test cases for the project with their target routes
+    const { data: testCases, error } = await supabase
+        .from('test_cases')
+        .select('id, title, target_route, qa_track, priority')
+        .eq('project_id', projectId);
+    if (error)
+        return { error: error.message };
+    if (!testCases || testCases.length === 0) {
+        return { affectedTests: [], message: 'No test cases found for this project.' };
+    }
+    // Map changed files to affected test cases
+    const affected = [];
+    for (const tc of testCases) {
+        const route = tc.target_route || '';
+        const matchedFiles = changedFiles.filter(f => {
+            // Match file path to route (e.g., src/app/api/tasks/route.ts -> /api/tasks)
+            const normalized = f.replace(/\\/g, '/');
+            const routeParts = route.split('/').filter(Boolean);
+            return routeParts.some((part) => normalized.includes(part)) || normalized.includes(route.replace(/\//g, '/'));
+        });
+        if (matchedFiles.length > 0) {
+            affected.push({
+                testId: tc.id,
+                title: tc.title,
+                targetRoute: route,
+                matchedFiles,
+                qaTrack: tc.qa_track,
+            });
+        }
+    }
+    return {
+        affectedTests: affected,
+        totalTestCases: testCases.length,
+        affectedCount: affected.length,
+        changedFileCount: changedFiles.length,
+    };
+}
+async function getFlakyTests(args) {
+    const projectId = requireProject();
+    const threshold = args.threshold || 5;
+    const limit = args.limit || 20;
+    // Get test results grouped by test case
+    const { data: results, error } = await supabase
+        .from('test_results')
+        .select('test_case_id, status, test_cases!inner(title, target_route, qa_track)')
+        .eq('test_cases.project_id', projectId)
+        .order('created_at', { ascending: false })
+        .limit(5000);
+    if (error)
+        return { error: error.message };
+    if (!results || results.length === 0) {
+        return { flakyTests: [], message: 'No test results found.' };
+    }
+    // Group by test case and calculate flakiness
+    const testStats = {};
+    for (const r of results) {
+        const id = r.test_case_id;
+        if (!testStats[id]) {
+            const tc = r.test_cases;
+            testStats[id] = { passes: 0, fails: 0, total: 0, title: tc?.title || '', route: tc?.target_route || '', track: tc?.qa_track || '' };
+        }
+        testStats[id].total++;
+        if (r.status === 'pass')
+            testStats[id].passes++;
+        else if (r.status === 'fail')
+            testStats[id].fails++;
+    }
+    // Find flaky tests (have both passes and fails, with fail rate above threshold)
+    const flaky = Object.entries(testStats)
+        .filter(([, stats]) => {
+        if (stats.total < 3)
+            return false; // Need enough data
+        const failRate = (stats.fails / stats.total) * 100;
+        const passRate = (stats.passes / stats.total) * 100;
+        return failRate >= threshold && passRate > 0; // Has both passes and fails
+    })
+        .map(([id, stats]) => ({
+        testCaseId: id,
+        title: stats.title,
+        targetRoute: stats.route,
+        qaTrack: stats.track,
+        totalRuns: stats.total,
+        failRate: Math.round((stats.fails / stats.total) * 100),
+        passRate: Math.round((stats.passes / stats.total) * 100),
+    }))
+        .sort((a, b) => b.failRate - a.failRate)
+        .slice(0, limit);
+    return {
+        flakyTests: flaky,
+        totalAnalyzed: Object.keys(testStats).length,
+        flakyCount: flaky.length,
+        threshold,
+    };
+}
+async function assessTestQuality(args) {
+    const projectId = requireProject();
+    const limit = args.limit || 20;
+    let query = supabase
+        .from('test_cases')
+        .select('id, title, steps, target_route, qa_track, priority')
+        .eq('project_id', projectId)
+        .limit(limit);
+    if (args.test_case_ids && args.test_case_ids.length > 0) {
+        query = query.in('id', args.test_case_ids);
+    }
+    const { data: testCases, error } = await query;
+    if (error)
+        return { error: error.message };
+    if (!testCases || testCases.length === 0) {
+        return { assessments: [], message: 'No test cases found.' };
+    }
+    const assessments = testCases.map(tc => {
+        const issues = [];
+        const steps = tc.steps || [];
+        // Check for weak patterns
+        if (steps.length < 2) {
+            issues.push('Too few steps — test may not cover the full flow');
+        }
+        const allStepsText = steps.map((s) => (typeof s === 'string' ? s : s.action || s.description || '')).join(' ');
+        // Vague assertions
+        if (/should work|looks good|is correct|verify it works/i.test(allStepsText)) {
+            issues.push('Vague assertions detected — use specific expected outcomes');
+        }
+        // Missing edge cases
+        if (!/error|invalid|empty|missing|unauthorized|forbidden|404|500/i.test(allStepsText)) {
+            issues.push('No negative/error test cases — add edge case testing');
+        }
+        // Generic descriptions
+        if (/test the|check the|verify the/i.test(tc.title) && tc.title.length < 30) {
+            issues.push('Generic test title — be more specific about what is being tested');
+        }
+        // No specific UI elements referenced
+        if (!/button|input|form|modal|dropdown|select|click|type|enter|submit/i.test(allStepsText)) {
+            issues.push('No specific UI elements referenced — steps may be too abstract');
+        }
+        const quality = issues.length === 0 ? 'good' : issues.length <= 2 ? 'needs-improvement' : 'poor';
+        return {
+            testCaseId: tc.id,
+            title: tc.title,
+            targetRoute: tc.target_route,
+            stepCount: steps.length,
+            quality,
+            issues,
+        };
+    });
+    const qualityCounts = {
+        good: assessments.filter(a => a.quality === 'good').length,
+        needsImprovement: assessments.filter(a => a.quality === 'needs-improvement').length,
+        poor: assessments.filter(a => a.quality === 'poor').length,
+    };
+    return {
+        assessments,
+        summary: qualityCounts,
+        totalAssessed: assessments.length,
+    };
+}
+async function getTestExecutionSummary(args) {
+    const projectId = requireProject();
+    const days = args.days || 30;
+    const since = new Date(Date.now() - days * 24 * 60 * 60 * 1000).toISOString();
+    // Get test results
+    const { data: results, error } = await supabase
+        .from('test_results')
+        .select('test_case_id, status, duration_ms, created_at, test_cases!inner(title, target_route)')
+        .eq('test_cases.project_id', projectId)
+        .gte('created_at', since)
+        .order('created_at', { ascending: false });
+    if (error)
+        return { error: error.message };
+    if (!results || results.length === 0) {
+        return { message: `No test results found in the last ${days} days.` };
+    }
+    const totalRuns = results.length;
+    const passed = results.filter(r => r.status === 'pass').length;
+    const failed = results.filter(r => r.status === 'fail').length;
+    const blocked = results.filter(r => r.status === 'blocked').length;
+    // Most failed tests
+    const failCounts = {};
+    for (const r of results.filter(r => r.status === 'fail')) {
+        const id = r.test_case_id;
+        const tc = r.test_cases;
+        if (!failCounts[id]) {
+            failCounts[id] = { count: 0, title: tc?.title || '', route: tc?.target_route || '' };
+        }
+        failCounts[id].count++;
+    }
+    const mostFailed = Object.entries(failCounts)
+        .sort((a, b) => b[1].count - a[1].count)
+        .slice(0, 5)
+        .map(([id, data]) => ({ testCaseId: id, ...data }));
+    // Duration stats
+    const durations = results.filter(r => r.duration_ms).map(r => r.duration_ms);
+    const avgDuration = durations.length > 0 ? Math.round(durations.reduce((a, b) => a + b, 0) / durations.length) : 0;
+    const maxDuration = durations.length > 0 ? Math.max(...durations) : 0;
+    return {
+        period: `${days} days`,
+        totalRuns,
+        passRate: Math.round((passed / totalRuns) * 100),
+        failRate: Math.round((failed / totalRuns) * 100),
+        blockedCount: blocked,
+        averageDurationMs: avgDuration,
+        maxDurationMs: maxDuration,
+        mostFailed,
+        uniqueTestsCovered: new Set(results.map(r => r.test_case_id)).size,
+    };
+}
+async function checkTestFreshness(args) {
+    const projectId = requireProject();
+    const limit = args.limit || 20;
+    // Get test cases with their last update and last result
+    const { data: testCases, error } = await supabase
+        .from('test_cases')
+        .select('id, title, target_route, updated_at, created_at')
+        .eq('project_id', projectId)
+        .order('updated_at', { ascending: true })
+        .limit(limit);
+    if (error)
+        return { error: error.message };
+    if (!testCases || testCases.length === 0) {
+        return { staleTests: [], message: 'No test cases found.' };
+    }
+    const thirtyDaysAgo = new Date(Date.now() - 30 * 24 * 60 * 60 * 1000).toISOString();
+    const stale = testCases
+        .filter(tc => tc.updated_at < thirtyDaysAgo)
+        .map(tc => ({
+        testCaseId: tc.id,
+        title: tc.title,
+        targetRoute: tc.target_route,
+        lastUpdated: tc.updated_at,
+        daysSinceUpdate: Math.round((Date.now() - new Date(tc.updated_at).getTime()) / (24 * 60 * 60 * 1000)),
+    }));
+    return {
+        staleTests: stale,
+        totalTestCases: testCases.length,
+        staleCount: stale.length,
+        stalenessThreshold: '30 days',
+    };
+}
+async function getUntestedChanges(args) {
+    const projectId = requireProject();
+    // Get all test cases to understand what's covered
+    const { data: testCases, error } = await supabase
+        .from('test_cases')
+        .select('id, title, target_route')
+        .eq('project_id', projectId);
+    if (error)
+        return { error: error.message };
+    const coveredRoutes = new Set((testCases || []).map(tc => tc.target_route).filter(Boolean));
+    // If changed_files provided, check coverage
+    const changedFiles = args.changed_files || [];
+    if (changedFiles.length === 0) {
+        return {
+            message: 'No changed files provided. Pass changed_files to check coverage.',
+            totalCoveredRoutes: coveredRoutes.size,
+        };
+    }
+    // Map changed files to routes and check coverage
+    const untested = [];
+    for (const file of changedFiles) {
+        const normalized = file.replace(/\\/g, '/');
+        // Extract route-like path from file
+        let inferredRoute = '';
+        // Next.js app router: app/api/tasks/route.ts -> /api/tasks
+        const appRouterMatch = normalized.match(/app\/(api\/[^/]+(?:\/[^/]+)*?)\/route\.\w+$/);
+        if (appRouterMatch) {
+            inferredRoute = '/' + appRouterMatch[1];
+        }
+        // Pages router: pages/api/tasks.ts -> /api/tasks
+        const pagesMatch = normalized.match(/pages\/(api\/[^.]+)\.\w+$/);
+        if (!inferredRoute && pagesMatch) {
+            inferredRoute = '/' + pagesMatch[1];
+        }
+        // Component files
+        const componentMatch = normalized.match(/(?:components|screens|pages)\/([^.]+)\.\w+$/);
+        if (!inferredRoute && componentMatch) {
+            inferredRoute = '/' + componentMatch[1].replace(/\\/g, '/');
+        }
+        if (inferredRoute && !coveredRoutes.has(inferredRoute)) {
+            untested.push({
+                file,
+                inferredRoute,
+                reason: 'No test cases cover this route',
+            });
+        }
+        else if (!inferredRoute) {
+            // Can't map to a route — flag as potentially untested
+            untested.push({
+                file,
+                inferredRoute: 'unknown',
+                reason: 'Could not map file to a testable route',
+            });
+        }
+    }
+    return {
+        untestedChanges: untested,
+        changedFileCount: changedFiles.length,
+        untestedCount: untested.length,
+        coveredRoutes: coveredRoutes.size,
+    };
+}
+// === AUTO-MONITORING HANDLERS ===
+async function getAutoDetectedIssues(args) {
+    const projectId = requireProject();
+    const since = args.since || new Date(Date.now() - 7 * 24 * 60 * 60 * 1000).toISOString();
+    const limit = args.limit || 20;
+    let query = supabase
+        .from('reports')
+        .select('id, error_fingerprint, report_source, title, severity, reporter_id, sentry_event_id, created_at, app_context')
+        .eq('project_id', projectId)
+        .neq('report_source', 'manual')
+        .not('error_fingerprint', 'is', null)
+        .gte('created_at', since)
+        .order('created_at', { ascending: false });
+    if (args.source) {
+        query = query.eq('report_source', args.source);
+    }
+    const { data, error } = await query;
+    if (error)
+        return { error: error.message };
+    if (!data || data.length === 0)
+        return { issues: [], total: 0 };
+    // Group by fingerprint
+    const grouped = new Map();
+    for (const report of data) {
+        const fp = report.error_fingerprint;
+        if (!grouped.has(fp))
+            grouped.set(fp, []);
+        grouped.get(fp).push(report);
+    }
+    // Build issue summaries
+    const issues = Array.from(grouped.entries())
+        .map(([fingerprint, reports]) => {
+        const uniqueReporters = new Set(reports.map(r => r.reporter_id));
+        const sorted = reports.sort((a, b) => new Date(a.created_at).getTime() - new Date(b.created_at).getTime());
+        const first = sorted[0];
+        const last = sorted[sorted.length - 1];
+        const route = first.app_context?.currentRoute || 'unknown';
+        return {
+            fingerprint,
+            source: first.report_source,
+            message: first.title,
+            route,
+            occurrence_count: reports.length,
+            affected_users: uniqueReporters.size,
+            first_seen: first.created_at,
+            last_seen: last.created_at,
+            severity: first.severity,
+            has_sentry_link: reports.some(r => r.sentry_event_id != null),
+            sample_report_id: first.id,
+        };
+    })
+        .filter(issue => issue.occurrence_count >= (args.min_occurrences || 1))
+        .sort((a, b) => b.occurrence_count - a.occurrence_count)
+        .slice(0, limit);
+    if (args.compact) {
+        return {
+            issues: issues.map(i => ({
+                fingerprint: i.fingerprint,
+                source: i.source,
+                count: i.occurrence_count,
+                users: i.affected_users,
+                severity: i.severity,
+            })),
+            total: issues.length,
+        };
+    }
+    return { issues, total: issues.length };
+}
+async function generateTestsFromErrors(args) {
+    const projectId = requireProject();
+    const limit = args.limit || 5;
+    let reports;
+    if (args.report_ids?.length) {
+        // Validate all UUIDs
+        for (const id of args.report_ids) {
+            if (!isValidUUID(id)) {
+                return { error: `Invalid report_id format: ${id}` };
+            }
+        }
+        const { data, error } = await supabase
+            .from('reports')
+            .select('id, title, report_source, severity, app_context, error_fingerprint, description')
+            .eq('project_id', projectId)
+            .in('id', args.report_ids);
+        if (error)
+            return { error: error.message };
+        reports = data;
+    }
+    else {
+        // Get top uncovered auto-detected errors
+        const { data, error } = await supabase
+            .from('reports')
+            .select('id, title, report_source, severity, app_context, error_fingerprint, description')
+            .eq('project_id', projectId)
+            .neq('report_source', 'manual')
+            .not('error_fingerprint', 'is', null)
+            .order('created_at', { ascending: false })
+            .limit(50);
+        if (error)
+            return { error: error.message };
+        // Deduplicate by fingerprint, keep first occurrence
+        const seen = new Set();
+        reports = (data || []).filter(r => {
+            if (!r.error_fingerprint || seen.has(r.error_fingerprint))
+                return false;
+            seen.add(r.error_fingerprint);
+            return true;
+        }).slice(0, limit);
+    }
+    if (!reports?.length)
+        return { suggestions: [] };
+    const suggestions = reports.map(report => {
+        const route = report.app_context?.currentRoute || '/unknown';
+        const source = report.report_source;
+        const priority = report.severity === 'critical' ? 'P1' : report.severity === 'high' ? 'P1' : 'P2';
+        let suggestedSteps;
+        if (source === 'auto_crash') {
+            suggestedSteps = [
+                `Navigate to ${route}`,
+                'Reproduce the action that triggered the crash',
+                'Verify the page does not throw an unhandled error',
+                'Verify error boundary displays a user-friendly message if error occurs',
+            ];
+        }
+        else if (source === 'auto_api') {
+            const statusCode = report.app_context?.custom?.statusCode || 'error';
+            const method = report.app_context?.custom?.requestMethod || 'API';
+            suggestedSteps = [
+                `Navigate to ${route}`,
+                `Trigger the ${method} request that returned ${statusCode}`,
+                'Verify the request succeeds or displays an appropriate error message',
+                'Verify no data corruption occurs on failure',
+            ];
+        }
+        else {
+            // rage_click or sentry_sync
+            const target = report.app_context?.custom?.targetSelector || 'the element';
+            suggestedSteps = [
+                `Navigate to ${route}`,
+                `Click on ${target}`,
+                'Verify the element responds to interaction',
+                'Verify loading state is shown if action takes time',
+            ];
+        }
+        return {
+            title: `Test: ${report.title?.replace('[Auto] ', '') || 'Auto-detected issue'}`,
+            track: source === 'auto_crash' ? 'Stability' : source === 'auto_api' ? 'API' : 'UX',
+            priority,
+            rationale: `Auto-detected ${source?.replace('auto_', '')} on ${route}`,
+            suggested_steps: suggestedSteps,
+            source_report_id: report.id,
+            route,
+        };
+    });
+    return { suggestions };
+}
+// === Project management handlers ===
+async function listProjects() {
+    const { data, error } = await supabase
+        .from('projects')
+        .select('id, name, slug, is_qa_enabled, created_at')
+        .order('name');
+    if (error) {
+        return { error: error.message };
+    }
+    return {
+        currentProjectId: currentProjectId || null,
+        projects: data?.map(p => ({
+            id: p.id,
+            name: p.name,
+            slug: p.slug,
+            isQAEnabled: p.is_qa_enabled,
+            isActive: p.id === currentProjectId,
+            createdAt: p.created_at,
+        })) || [],
+    };
+}
+async function switchProject(args) {
+    if (!isValidUUID(args.project_id)) {
+        return { error: 'Invalid project_id format — must be a UUID' };
+    }
+    // Verify the project exists and is accessible
+    const { data, error } = await supabase
+        .from('projects')
+        .select('id, name, slug')
+        .eq('id', args.project_id)
+        .single();
+    if (error || !data) {
+        return { error: 'Project not found or not accessible' };
+    }
+    currentProjectId = data.id;
+    return {
+        success: true,
+        message: `Switched to project "${data.name}" (${data.slug})`,
+        projectId: data.id,
+        projectName: data.name,
+    };
+}
+function getCurrentProject() {
+    if (!currentProjectId) {
+        return { message: 'No project selected. Use list_projects to see available projects, then switch_project to select one.' };
+    }
+    return { projectId: currentProjectId };
+}
 // Tool handlers
 async function listReports(args) {
     let query = supabase
         .from('reports')
         .select('id, report_type, severity, status, description, app_context, created_at, reporter_name, tester:testers(name)')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('created_at', { ascending: false })
         .limit(Math.min(args.limit || 10, 50));
     if (args.status)
@@ -1067,7 +1985,7 @@ async function getReport(args) {
         .from('reports')
         .select('*, tester:testers(*), track:qa_tracks(*)')
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure report belongs to this project
         .single();
     if (error) {
         return { error: error.message };
@@ -1082,7 +2000,7 @@ async function getReport(args) {
             app_context: data.app_context,
             device_info: data.device_info,
             navigation_history: data.navigation_history,
-            screenshots: data.screenshots,
+            screenshot_urls: data.screenshot_urls,
             created_at: data.created_at,
             reporter: data.tester ? {
                 name: data.tester.name,
@@ -1102,7 +2020,7 @@ async function searchReports(args) {
     let query = supabase
         .from('reports')
         .select('id, report_type, severity, status, description, app_context, created_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('created_at', { ascending: false })
         .limit(20);
     if (sanitizedQuery) {
@@ -1145,7 +2063,7 @@ async function updateReportStatus(args) {
         .from('reports')
         .update(updates)
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID); // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId); // Security: ensure report belongs to this project
     if (error) {
         return { error: error.message };
     }
@@ -1157,28 +2075,92 @@ async function getReportContext(args) {
     }
     const { data, error } = await supabase
         .from('reports')
-        .select('app_context, device_info, navigation_history, enhanced_context')
+        .select('app_context, device_info, navigation_history, enhanced_context, screenshot_urls')
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure report belongs to this project
         .single();
     if (error) {
         return { error: error.message };
     }
+    // Compact: return app_context only (skip console/network/navigation)
+    if (args.compact === true) {
+        return {
+            context: {
+                app_context: data.app_context,
+                screenshot_urls: data.screenshot_urls,
+            },
+        };
+    }
     return {
         context: {
             app_context: data.app_context,
             device_info: data.device_info,
             navigation_history: data.navigation_history,
             enhanced_context: data.enhanced_context || {},
+            screenshot_urls: data.screenshot_urls,
         },
     };
 }
+async function addReportComment(args) {
+    if (!isValidUUID(args.report_id))
+        return { error: 'Invalid report_id format' };
+    if (!args.message?.trim())
+        return { error: 'Message is required' };
+    // Verify report exists
+    const { data: report } = await supabase
+        .from('reports').select('id').eq('id', args.report_id).eq('project_id', currentProjectId).single();
+    if (!report)
+        return { error: 'Report not found' };
+    // Find or create a discussion thread for this report
+    const { data: existingThread } = await supabase
+        .from('discussion_threads').select('id')
+        .eq('project_id', currentProjectId).eq('report_id', args.report_id).eq('thread_type', 'report')
+        .limit(1).single();
+    let threadId;
+    if (existingThread) {
+        threadId = existingThread.id;
+    }
+    else {
+        const newId = crypto.randomUUID();
+        const { error: threadErr } = await supabase
+            .from('discussion_threads').insert({
+            id: newId, project_id: currentProjectId, report_id: args.report_id,
+            thread_type: 'report', subject: 'Bug Report Discussion', audience: 'all',
+            priority: 'normal', created_by_admin: true, last_message_at: new Date().toISOString(),
+        });
+        if (threadErr)
+            return { error: `Failed to create thread: ${threadErr.message}` };
+        threadId = newId;
+    }
+    const { data: msg, error: msgErr } = await supabase
+        .from('discussion_messages').insert({
+        thread_id: threadId, sender_type: 'admin', sender_name: args.author || 'Claude Code', content: args.message.trim(), content_type: 'text',
+    }).select('id, content, created_at').single();
+    if (msgErr)
+        return { error: `Failed to add comment: ${msgErr.message}` };
+    return { success: true, comment: { id: msg.id, thread_id: threadId, content: msg.content, author: args.author || 'Claude Code', created_at: msg.created_at }, message: 'Comment added to report' };
+}
+async function getReportComments(args) {
+    if (!isValidUUID(args.report_id))
+        return { error: 'Invalid report_id format' };
+    const { data: threads } = await supabase
+        .from('discussion_threads').select('id')
+        .eq('project_id', currentProjectId).eq('report_id', args.report_id).order('created_at', { ascending: true });
+    if (!threads || threads.length === 0)
+        return { comments: [], total: 0, message: 'No comments on this report' };
+    const { data: messages, error } = await supabase
+        .from('discussion_messages').select('id, thread_id, sender_type, content, content_type, created_at, attachments')
+        .in('thread_id', threads.map(t => t.id)).order('created_at', { ascending: true });
+    if (error)
+        return { error: error.message };
+    return { comments: (messages || []).map(m => ({ id: m.id, sender_type: m.sender_type, content: m.content, created_at: m.created_at, attachments: m.attachments })), total: (messages || []).length };
+}
 async function getProjectInfo() {
     // Get project details
     const { data: project, error: projectError } = await supabase
         .from('projects')
         .select('id, name, slug, is_qa_enabled')
-        .eq('id', PROJECT_ID)
+        .eq('id', currentProjectId)
         .single();
     if (projectError) {
         return { error: projectError.message };
@@ -1187,17 +2169,17 @@ async function getProjectInfo() {
     const { data: tracks } = await supabase
         .from('qa_tracks')
         .select('id, name, icon, test_template')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     // Get test case count
     const { count: testCaseCount } = await supabase
         .from('test_cases')
         .select('id', { count: 'exact', head: true })
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     // Get open bug count
     const { count: openBugCount } = await supabase
         .from('reports')
         .select('id', { count: 'exact', head: true })
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .in('status', ['new', 'confirmed', 'in_progress']);
     return {
@@ -1224,7 +2206,7 @@ async function getQaTracks() {
     const { data, error } = await supabase
         .from('qa_tracks')
         .select('*')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('sort_order');
     if (error) {
         return { error: error.message };
@@ -1252,14 +2234,14 @@ async function createTestCase(args) {
             const { data: trackData } = await supabase
                 .from('qa_tracks')
                 .select('id')
-                .eq('project_id', PROJECT_ID)
+                .eq('project_id', currentProjectId)
                 .ilike('name', `%${sanitizedTrack}%`)
                 .single();
             trackId = trackData?.id || null;
         }
     }
     const testCase = {
-        project_id: PROJECT_ID,
+        project_id: currentProjectId,
         test_key: args.test_key,
         title: args.title,
         description: args.description || '',
@@ -1299,7 +2281,7 @@ async function updateTestCase(args) {
         const { data: existing } = await supabase
             .from('test_cases')
             .select('id')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('test_key', args.test_key)
             .single();
         if (!existing) {
@@ -1330,7 +2312,7 @@ async function updateTestCase(args) {
         .from('test_cases')
         .update(updates)
         .eq('id', testCaseId)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .select('id, test_key, title, target_route')
         .single();
     if (error) {
@@ -1375,7 +2357,7 @@ async function deleteTestCases(args) {
         const { data: existing } = await supabase
             .from('test_cases')
             .select('id')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('test_key', args.test_key)
             .single();
         if (!existing) {
@@ -1408,7 +2390,7 @@ async function deleteTestCases(args) {
         const { data: existing, error: lookupError } = await supabase
             .from('test_cases')
             .select('id, test_key')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .in('test_key', args.test_keys);
         if (lookupError) {
             return { error: lookupError.message };
@@ -1424,7 +2406,7 @@ async function deleteTestCases(args) {
     const { data: toDelete } = await supabase
         .from('test_cases')
         .select('id, test_key, title')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .in('id', idsToDelete);
     if (!toDelete || toDelete.length === 0) {
         return { error: 'No matching test cases found in this project' };
@@ -1433,7 +2415,7 @@ async function deleteTestCases(args) {
     const { error: deleteError } = await supabase
         .from('test_cases')
         .delete()
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .in('id', idsToDelete);
     if (deleteError) {
         return { error: deleteError.message };
@@ -1467,7 +2449,7 @@ async function listTestCases(args) {
       steps,
       track:qa_tracks(id, name, icon, color)
     `)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('test_key', { ascending: true });
     // Apply filters
     if (args.priority) {
@@ -1489,6 +2471,19 @@ async function listTestCases(args) {
     if (args.track) {
         testCases = testCases.filter((tc) => tc.track?.name?.toLowerCase().includes(args.track.toLowerCase()));
     }
+    // Compact: return minimal fields only
+    if (args.compact === true) {
+        return {
+            count: testCases.length,
+            testCases: testCases.map((tc) => ({
+                id: tc.id,
+                testKey: tc.test_key,
+                title: tc.title,
+                priority: tc.priority,
+            })),
+            pagination: { limit, offset, hasMore: testCases.length === limit },
+        };
+    }
     return {
         count: testCases.length,
         testCases: testCases.map((tc) => ({
@@ -1514,7 +2509,7 @@ async function getBugPatterns(args) {
     let query = supabase
         .from('reports')
         .select('app_context, severity, status, created_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .order('created_at', { ascending: false })
         .limit(100);
@@ -1566,7 +2561,7 @@ async function suggestTestCases(args) {
     const { data: existingTests } = await supabase
         .from('test_cases')
         .select('test_key, title')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('test_key', { ascending: false })
         .limit(1);
     // Calculate next test key number
@@ -1601,7 +2596,7 @@ async function suggestTestCases(args) {
     const { data: relatedBugs } = await supabase
         .from('reports')
         .select('id, description, severity')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .limit(10);
     const routeBugs = (relatedBugs || []).filter(bug => {
@@ -1635,7 +2630,7 @@ async function getTestPriorities(args) {
     const minScore = args.min_score || 0;
     const includeFactors = args.include_factors !== false;
     // First, refresh the route stats
-    const { error: refreshError } = await supabase.rpc('refresh_route_test_stats', { p_project_id: PROJECT_ID });
+    const { error: refreshError } = await supabase.rpc('refresh_route_test_stats', { p_project_id: currentProjectId });
     if (refreshError) {
         // Non-fatal: proceed with potentially stale data but warn
         console.warn('Failed to refresh route stats:', refreshError.message);
@@ -1644,7 +2639,7 @@ async function getTestPriorities(args) {
     const { data: routes, error } = await supabase
         .from('route_test_stats')
         .select('*')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .gte('priority_score', minScore)
         .order('priority_score', { ascending: false })
         .limit(limit);
@@ -1761,7 +2756,7 @@ async function getCoverageGaps(args) {
     const { data: routesFromReports } = await supabase
         .from('reports')
         .select('app_context')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .not('app_context->currentRoute', 'is', null);
     const allRoutes = new Set();
     (routesFromReports || []).forEach(r => {
@@ -1773,7 +2768,7 @@ async function getCoverageGaps(args) {
     const { data: testCases } = await supabase
         .from('test_cases')
         .select('target_route, category, track_id')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     const coveredRoutes = new Set();
     const routeTrackCoverage = {};
     (testCases || []).forEach(tc => {
@@ -1790,13 +2785,13 @@ async function getCoverageGaps(args) {
     const { data: tracks } = await supabase
         .from('qa_tracks')
         .select('id, name')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     const trackMap = new Map((tracks || []).map(t => [t.id, t.name]));
     // Get route stats for staleness
     const { data: routeStats } = await supabase
         .from('route_test_stats')
         .select('route, last_tested_at, open_bugs, critical_bugs')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     const routeStatsMap = new Map((routeStats || []).map(r => [r.route, r]));
     // Find untested routes
     if (gapType === 'all' || gapType === 'untested_routes') {
@@ -1893,14 +2888,14 @@ async function getRegressions(args) {
     const { data: resolvedBugs } = await supabase
         .from('reports')
         .select('id, description, severity, app_context, resolved_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .in('status', ['resolved', 'fixed', 'verified', 'closed'])
         .gte('resolved_at', new Date(Date.now() - days * 24 * 60 * 60 * 1000).toISOString());
     const { data: newBugs } = await supabase
         .from('reports')
         .select('id, description, severity, app_context, created_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .in('status', ['new', 'triaging', 'confirmed', 'in_progress', 'reviewed'])
         .gte('created_at', new Date(Date.now() - days * 24 * 60 * 60 * 1000).toISOString());
@@ -1998,26 +2993,26 @@ async function getRegressions(args) {
     };
 }
 async function getCoverageMatrix(args) {
-    const includeExecution = args.include_execution_data !== false;
-    const includeBugs = args.include_bug_counts !== false;
+    const includeExecution = args.include_execution_data === true;
+    const includeBugs = args.include_bug_counts === true;
     // Get tracks
     const { data: tracks } = await supabase
         .from('qa_tracks')
         .select('id, name, icon, color')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('sort_order');
     // Get test cases with track info
     const { data: testCases } = await supabase
         .from('test_cases')
         .select('id, target_route, category, track_id')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     // Get test assignments for execution data
     let assignments = [];
     if (includeExecution) {
         const { data } = await supabase
             .from('test_assignments')
             .select('test_case_id, status, completed_at')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .in('status', ['passed', 'failed'])
             .order('completed_at', { ascending: false })
             .limit(2000);
@@ -2029,7 +3024,7 @@ async function getCoverageMatrix(args) {
         const { data } = await supabase
             .from('route_test_stats')
             .select('route, open_bugs, critical_bugs')
-            .eq('project_id', PROJECT_ID);
+            .eq('project_id', currentProjectId);
         routeStats = data || [];
     }
     const routeStatsMap = new Map(routeStats.map(r => [r.route, r]));
@@ -2168,7 +3163,7 @@ async function getStaleCoverage(args) {
     const daysThreshold = args.days_threshold || 14;
     const limit = args.limit || 20;
     // Refresh stats first
-    const { error: refreshError } = await supabase.rpc('refresh_route_test_stats', { p_project_id: PROJECT_ID });
+    const { error: refreshError } = await supabase.rpc('refresh_route_test_stats', { p_project_id: currentProjectId });
     if (refreshError) {
         // Non-fatal: proceed with potentially stale data but warn
         console.warn('Failed to refresh route stats:', refreshError.message);
@@ -2177,7 +3172,7 @@ async function getStaleCoverage(args) {
     const { data: routes, error } = await supabase
         .from('route_test_stats')
         .select('route, last_tested_at, open_bugs, critical_bugs, test_case_count, priority_score')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('last_tested_at', { ascending: true, nullsFirst: true })
         .limit(limit * 2); // Get extra to filter
     if (error) {
@@ -2264,12 +3259,12 @@ async function generateDeployChecklist(args) {
         supabase
             .from('test_cases')
             .select('id, test_key, title, target_route, category, priority, track:qa_tracks(name)')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .in('target_route', safeRoutes),
         supabase
             .from('test_cases')
             .select('id, test_key, title, target_route, category, priority, track:qa_tracks(name)')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .in('category', safeRoutes),
     ]);
     // Deduplicate by id
@@ -2284,7 +3279,7 @@ async function generateDeployChecklist(args) {
     const { data: routeStats } = await supabase
         .from('route_test_stats')
         .select('*')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .in('route', Array.from(allRoutes));
     const routeStatsMap = new Map((routeStats || []).map(r => [r.route, r]));
     // Categorize tests
@@ -2383,30 +3378,30 @@ async function getQAHealth(args) {
     const { data: currentTests } = await supabase
         .from('test_assignments')
         .select('id, status, completed_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .gte('completed_at', periodStart.toISOString())
         .in('status', ['passed', 'failed']);
     const { data: currentBugs } = await supabase
         .from('reports')
         .select('id, severity, status, created_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .gte('created_at', periodStart.toISOString());
     const { data: resolvedBugs } = await supabase
         .from('reports')
         .select('id, created_at, resolved_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .in('status', ['resolved', 'fixed', 'verified', 'closed'])
         .gte('resolved_at', periodStart.toISOString());
     const { data: testers } = await supabase
         .from('testers')
         .select('id, status')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     const { data: routeStats } = await supabase
         .from('route_test_stats')
         .select('route, test_case_count')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     // Get previous period data for comparison
     let previousTests = [];
     let previousBugs = [];
@@ -2415,7 +3410,7 @@ async function getQAHealth(args) {
         const { data: pt } = await supabase
             .from('test_assignments')
             .select('id, status')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .gte('completed_at', previousStart.toISOString())
             .lt('completed_at', periodStart.toISOString())
             .in('status', ['passed', 'failed']);
@@ -2423,7 +3418,7 @@ async function getQAHealth(args) {
         const { data: pb } = await supabase
             .from('reports')
             .select('id, severity')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('report_type', 'bug')
             .gte('created_at', previousStart.toISOString())
             .lt('created_at', periodStart.toISOString());
@@ -2431,7 +3426,7 @@ async function getQAHealth(args) {
         const { data: pr } = await supabase
             .from('reports')
             .select('id')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .in('status', ['resolved', 'fixed', 'verified', 'closed'])
             .gte('resolved_at', previousStart.toISOString())
             .lt('resolved_at', periodStart.toISOString());
@@ -2585,7 +3580,7 @@ async function getQASessions(args) {
       findings_count, bugs_filed, created_at,
       tester:testers(id, name, email)
     `)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('started_at', { ascending: false })
         .limit(limit);
     if (status !== 'all') {
@@ -2635,12 +3630,12 @@ async function getQAAlerts(args) {
     const status = args.status || 'active';
     // Optionally refresh alerts
     if (args.refresh) {
-        await supabase.rpc('detect_all_alerts', { p_project_id: PROJECT_ID });
+        await supabase.rpc('detect_all_alerts', { p_project_id: currentProjectId });
     }
     let query = supabase
         .from('qa_alerts')
         .select('*')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('severity', { ascending: true }) // critical first
         .order('created_at', { ascending: false });
     if (severity !== 'all') {
@@ -2693,7 +3688,7 @@ async function getDeploymentAnalysis(args) {
             .from('deployments')
             .select('*')
             .eq('id', args.deployment_id)
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .single();
         if (error) {
             return { error: error.message };
@@ -2704,7 +3699,7 @@ async function getDeploymentAnalysis(args) {
     let query = supabase
         .from('deployments')
         .select('*')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('deployed_at', { ascending: false })
         .limit(limit);
     if (args.environment && args.environment !== 'all') {
@@ -2778,7 +3773,7 @@ async function analyzeCommitForTesting(args) {
     const { data: mappings } = await supabase
         .from('file_route_mapping')
         .select('file_pattern, route, feature, confidence')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     const affectedRoutes = [];
     for (const mapping of mappings || []) {
         const matchedFiles = filesChanged.filter(file => {
@@ -2812,7 +3807,7 @@ async function analyzeCommitForTesting(args) {
         const { data: bugs } = await supabase
             .from('reports')
             .select('id, severity, description, route, created_at')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('report_type', 'bug')
             .in('route', routes)
             .gte('created_at', new Date(Date.now() - 30 * 24 * 60 * 60 * 1000).toISOString());
@@ -2844,7 +3839,7 @@ async function analyzeCommitForTesting(args) {
     // Optionally record as deployment
     if (args.record_deployment) {
         await supabase.rpc('record_deployment', {
-            p_project_id: PROJECT_ID,
+            p_project_id: currentProjectId,
             p_environment: 'production',
             p_commit_sha: args.commit_sha || null,
             p_commit_message: args.commit_message || null,
@@ -2933,12 +3928,12 @@ async function analyzeChangesForTests(args) {
     const { data: existingTests } = await supabase
         .from('test_cases')
         .select('test_key, title, target_route, description')
-        .eq('project_id', PROJECT_ID);
+        .eq('project_id', currentProjectId);
     // Get next test key
     const { data: lastTest } = await supabase
         .from('test_cases')
         .select('test_key')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('test_key', { ascending: false })
         .limit(1);
     const lastKey = lastTest?.[0]?.test_key || 'TC-000';
@@ -2950,7 +3945,7 @@ async function analyzeChangesForTests(args) {
         const { data: bugs } = await supabase
             .from('reports')
             .select('id, description, severity, app_context')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('report_type', 'bug')
             .limit(50);
         relatedBugs = (bugs || []).filter(bug => {
@@ -3295,7 +4290,7 @@ async function createBugReport(args) {
     const { data: project } = await supabase
         .from('projects')
         .select('owner_id')
-        .eq('id', PROJECT_ID)
+        .eq('id', currentProjectId)
         .single();
     if (project?.owner_id) {
         reporterId = project.owner_id;
@@ -3305,14 +4300,14 @@ async function createBugReport(args) {
         const { data: testers } = await supabase
             .from('testers')
             .select('id')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .limit(1);
         if (testers && testers.length > 0) {
             reporterId = testers[0].id;
         }
     }
     const report = {
-        project_id: PROJECT_ID,
+        project_id: currentProjectId,
         report_type: 'bug',
         title: args.title,
         description: args.description,
@@ -3376,7 +4371,7 @@ async function getBugsForFile(args) {
     let query = supabase
         .from('reports')
         .select('id, title, description, severity, status, created_at, code_context')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug');
     if (!args.include_resolved) {
         query = query.in('status', ['new', 'confirmed', 'in_progress', 'reviewed']);
@@ -3442,7 +4437,7 @@ async function markFixedWithCommit(args) {
         .from('reports')
         .select('code_context')
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure report belongs to this project
         .single();
     if (fetchError) {
         return { error: fetchError.message };
@@ -3452,7 +4447,7 @@ async function markFixedWithCommit(args) {
         status: 'resolved',
         resolved_at: new Date().toISOString(),
         resolution_notes: args.resolution_notes || `Fixed in commit ${args.commit_sha.slice(0, 7)}`,
-        notify_tester: args.notify_tester === true, // Opt-in: only notify if explicitly requested
+        notify_tester: args.notify_tester !== false, // Default: notify tester. Pass false to silently resolve.
         code_context: {
             ...existingContext,
             fix: {
@@ -3468,11 +4463,12 @@ async function markFixedWithCommit(args) {
         .from('reports')
         .update(updates)
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID); // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId); // Security: ensure report belongs to this project
     if (error) {
         return { error: error.message };
     }
-    const notificationStatus = args.notify_tester
+    const notifyTester = args.notify_tester !== false;
+    const notificationStatus = notifyTester
         ? 'The original tester will be notified and assigned a verification task.'
         : 'No notification sent (silent resolve). A verification task was created.';
     return {
@@ -3480,7 +4476,7 @@ async function markFixedWithCommit(args) {
         message: `Bug marked as fixed in commit ${args.commit_sha.slice(0, 7)}. ${notificationStatus}`,
         report_id: args.report_id,
         commit: args.commit_sha,
-        tester_notified: args.notify_tester === true,
+        tester_notified: notifyTester,
         next_steps: [
             'Consider running create_regression_test to prevent this bug from recurring',
             'Push your changes to trigger CI/CD',
@@ -3492,7 +4488,7 @@ async function getBugsAffectingCode(args) {
     const { data, error } = await supabase
         .from('reports')
         .select('id, title, description, severity, status, code_context, app_context')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('report_type', 'bug')
         .in('status', ['new', 'confirmed', 'in_progress', 'reviewed'])
         .order('severity', { ascending: true });
@@ -3597,7 +4593,7 @@ async function linkBugToCode(args) {
         .from('reports')
         .select('code_context')
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure report belongs to this project
         .single();
     if (fetchError) {
         return { error: fetchError.message };
@@ -3618,7 +4614,7 @@ async function linkBugToCode(args) {
         .from('reports')
         .update(updates)
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID); // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId); // Security: ensure report belongs to this project
     if (error) {
         return { error: error.message };
     }
@@ -3637,7 +4633,7 @@ async function createRegressionTest(args) {
         .from('reports')
         .select('*')
         .eq('id', args.report_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure report belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure report belongs to this project
         .single();
     if (fetchError) {
         return { error: fetchError.message };
@@ -3654,7 +4650,7 @@ async function createRegressionTest(args) {
     const { data: existingTests } = await supabase
         .from('test_cases')
         .select('test_key')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('test_key', { ascending: false })
         .limit(1);
     const lastKey = existingTests?.[0]?.test_key || 'TC-000';
@@ -3665,7 +4661,7 @@ async function createRegressionTest(args) {
     const targetRoute = appContext?.currentRoute;
     // Generate test case from bug
     const testCase = {
-        project_id: PROJECT_ID,
+        project_id: currentProjectId,
         test_key: newKey,
         title: `Regression: ${report.title}`,
         description: `Regression test to prevent recurrence of bug #${args.report_id.slice(0, 8)}\n\nOriginal bug: ${report.description}`,
@@ -3741,7 +4737,7 @@ async function getPendingFixes(args) {
       created_at,
       report:reports(id, title, severity, description)
     `)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('created_at', { ascending: true })
         .limit(limit);
     if (!args.include_claimed) {
@@ -3791,7 +4787,7 @@ async function claimFixRequest(args) {
         .from('fix_requests')
         .select('id, status, claimed_by, prompt, title')
         .eq('id', args.fix_request_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure fix request belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure fix request belongs to this project
         .single();
     if (checkError) {
         return { error: checkError.message };
@@ -3818,7 +4814,7 @@ async function claimFixRequest(args) {
         claimed_by: claimedBy,
     })
         .eq('id', args.fix_request_id)
-        .eq('project_id', PROJECT_ID) // Security: ensure fix request belongs to this project
+        .eq('project_id', currentProjectId) // Security: ensure fix request belongs to this project
         .eq('status', 'pending'); // Only claim if still pending (race condition protection)
     if (updateError) {
         return { error: updateError.message };
@@ -3853,7 +4849,7 @@ async function completeFixRequest(args) {
         .from('fix_requests')
         .update(updates)
         .eq('id', args.fix_request_id)
-        .eq('project_id', PROJECT_ID); // Security: ensure fix request belongs to this project
+        .eq('project_id', currentProjectId); // Security: ensure fix request belongs to this project
     if (error) {
         return { error: error.message };
     }
@@ -3936,7 +4932,7 @@ async function generatePromptContent(name, args) {
           created_at,
           report:reports(id, title, severity)
         `)
-                .eq('project_id', PROJECT_ID)
+                .eq('project_id', currentProjectId)
                 .eq('status', 'pending')
                 .order('created_at', { ascending: true })
                 .limit(5);
@@ -3944,7 +4940,7 @@ async function generatePromptContent(name, args) {
             let query = supabase
                 .from('reports')
                 .select('id, title, description, severity, status, code_context, created_at')
-                .eq('project_id', PROJECT_ID)
+                .eq('project_id', currentProjectId)
                 .eq('report_type', 'bug')
                 .in('status', ['new', 'confirmed', 'in_progress']);
             if (severity !== 'all') {
@@ -4094,7 +5090,7 @@ Would you like me to generate test cases for these files?`;
             const { data: resolvedBugs } = await supabase
                 .from('reports')
                 .select('id, title, description, severity, resolved_at, code_context')
-                .eq('project_id', PROJECT_ID)
+                .eq('project_id', currentProjectId)
                 .eq('report_type', 'bug')
                 .eq('status', 'resolved')
                 .order('resolved_at', { ascending: false })
@@ -4205,12 +5201,15 @@ Which files or areas would you like me to analyze?`;
 async function listTesters(args) {
     let query = supabase
         .from('testers')
-        .select('id, name, email, status, platforms, tier, assigned_count, completed_count, notes, created_at')
-        .eq('project_id', PROJECT_ID)
+        .select('id, name, email, status, platforms, tier, assigned_count, completed_count, notes, role, created_at')
+        .eq('project_id', currentProjectId)
         .order('name', { ascending: true });
     if (args.status) {
         query = query.eq('status', args.status);
     }
+    if (args.role) {
+        query = query.eq('role', args.role);
+    }
     const { data, error } = await query;
     if (error) {
         return { error: error.message };
@@ -4232,6 +5231,7 @@ async function listTesters(args) {
             assignedCount: t.assigned_count,
             completedCount: t.completed_count,
             notes: t.notes,
+            role: t.role,
         })),
     };
 }
@@ -4240,7 +5240,7 @@ async function listTestRuns(args) {
     let query = supabase
         .from('test_runs')
         .select('id, name, description, status, total_tests, passed_tests, failed_tests, started_at, completed_at, created_at')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('created_at', { ascending: false })
         .limit(limit);
     if (args.status) {
@@ -4274,7 +5274,7 @@ async function createTestRun(args) {
     const { data, error } = await supabase
         .from('test_runs')
         .insert({
-        project_id: PROJECT_ID,
+        project_id: currentProjectId,
         name: args.name.trim(),
         description: args.description?.trim() || null,
         status: 'draft',
@@ -4319,7 +5319,7 @@ async function listTestAssignments(args) {
       tester:testers(id, name, email),
       test_run:test_runs(id, name)
     `)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .order('assigned_at', { ascending: false })
         .limit(limit);
     if (args.tester_id) {
@@ -4389,7 +5389,7 @@ async function assignTests(args) {
         .from('testers')
         .select('id, name, email, status')
         .eq('id', args.tester_id)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .single();
     if (testerErr || !tester) {
         return { error: 'Tester not found in this project' };
@@ -4401,7 +5401,7 @@ async function assignTests(args) {
     const { data: testCases, error: tcErr } = await supabase
         .from('test_cases')
         .select('id, test_key, title')
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .in('id', args.test_case_ids);
     if (tcErr) {
         return { error: tcErr.message };
@@ -4419,7 +5419,7 @@ async function assignTests(args) {
             .from('test_runs')
             .select('id')
             .eq('id', args.test_run_id)
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .single();
         if (runErr || !run) {
             return { error: 'Test run not found in this project' };
@@ -4427,12 +5427,28 @@ async function assignTests(args) {
     }
     // Build assignment rows
     const rows = args.test_case_ids.map(tcId => ({
-        project_id: PROJECT_ID,
+        project_id: currentProjectId,
         test_case_id: tcId,
         tester_id: args.tester_id,
         test_run_id: args.test_run_id || null,
         status: 'pending',
     }));
+    // Helper: after assignments change, sync the test run's total_tests counter
+    async function syncRunCounter() {
+        if (!args.test_run_id)
+            return;
+        const { count } = await supabase
+            .from('test_assignments')
+            .select('id', { count: 'exact', head: true })
+            .eq('test_run_id', args.test_run_id)
+            .eq('project_id', currentProjectId);
+        if (count !== null) {
+            await supabase
+                .from('test_runs')
+                .update({ total_tests: count })
+                .eq('id', args.test_run_id);
+        }
+    }
     // Insert — use upsert-like approach: insert and handle conflicts
     const { data: inserted, error: insertErr } = await supabase
         .from('test_assignments')
@@ -4458,6 +5474,7 @@ async function assignTests(args) {
                     created.push(single);
                 }
             }
+            await syncRunCounter();
             return {
                 success: true,
                 created: created.length,
@@ -4469,6 +5486,7 @@ async function assignTests(args) {
         }
         return { error: insertErr.message };
     }
+    await syncRunCounter();
     return {
         success: true,
         created: (inserted || []).length,
@@ -4478,6 +5496,71 @@ async function assignTests(args) {
         message: `Assigned ${(inserted || []).length} test(s) to ${tester.name}.`,
     };
 }
+async function unassignTests(args) {
+    if (!args.assignment_ids || args.assignment_ids.length === 0) {
+        return { error: 'At least one assignment_id is required' };
+    }
+    if (args.assignment_ids.length > 50) {
+        return { error: 'Maximum 50 assignments per unassign batch' };
+    }
+    const invalidIds = args.assignment_ids.filter(id => !isValidUUID(id));
+    if (invalidIds.length > 0) {
+        return { error: `Invalid UUID(s): ${invalidIds.join(', ')}` };
+    }
+    // Verify assignments exist and belong to this project
+    const { data: existing, error: lookupErr } = await supabase
+        .from('test_assignments')
+        .select('id, test_run_id, test_case:test_cases(test_key, title), tester:testers(name)')
+        .eq('project_id', currentProjectId)
+        .in('id', args.assignment_ids);
+    if (lookupErr)
+        return { error: lookupErr.message };
+    if (!existing || existing.length === 0) {
+        return { error: 'No matching assignments found in this project' };
+    }
+    const foundIds = new Set(existing.map((a) => a.id));
+    const notFound = args.assignment_ids.filter(id => !foundIds.has(id));
+    // Delete the assignments
+    const { error: deleteErr } = await supabase
+        .from('test_assignments')
+        .delete()
+        .eq('project_id', currentProjectId)
+        .in('id', args.assignment_ids);
+    if (deleteErr)
+        return { error: deleteErr.message };
+    // Sync run counters for any affected test runs
+    const affectedRunIds = [...new Set(existing.filter((a) => a.test_run_id).map((a) => a.test_run_id))];
+    for (const runId of affectedRunIds) {
+        const { count } = await supabase
+            .from('test_assignments')
+            .select('id', { count: 'exact', head: true })
+            .eq('test_run_id', runId)
+            .eq('project_id', currentProjectId);
+        if (count !== null) {
+            await supabase.from('test_runs').update({ total_tests: count }).eq('id', runId);
+        }
+    }
+    const deleted = existing.map((a) => {
+        const tc = a.test_case;
+        const tester = a.tester;
+        return {
+            id: a.id,
+            testKey: tc?.test_key || null,
+            testTitle: tc?.title || null,
+            testerName: tester?.name || null,
+        };
+    });
+    const firstKey = deleted[0]?.testKey;
+    return {
+        success: true,
+        deletedCount: existing.length,
+        deleted,
+        notFound: notFound.length > 0 ? notFound : undefined,
+        message: existing.length === 1
+            ? `Removed 1 assignment${firstKey ? ` (${firstKey})` : ''}`
+            : `Removed ${existing.length} assignment(s)`,
+    };
+}
 async function getTesterWorkload(args) {
     if (!isValidUUID(args.tester_id)) {
         return { error: 'Invalid tester_id format' };
@@ -4487,7 +5570,7 @@ async function getTesterWorkload(args) {
         .from('testers')
         .select('id, name, email, status, platforms, tier')
         .eq('id', args.tester_id)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .single();
     if (testerErr || !tester) {
         return { error: 'Tester not found in this project' };
@@ -4503,7 +5586,7 @@ async function getTesterWorkload(args) {
       test_case:test_cases(test_key, title, priority),
       test_run:test_runs(name)
     `)
-        .eq('project_id', PROJECT_ID)
+        .eq('project_id', currentProjectId)
         .eq('tester_id', args.tester_id)
         .order('assigned_at', { ascending: false });
     if (assignErr) {
@@ -4548,6 +5631,448 @@ async function getTesterWorkload(args) {
         })),
     };
 }
+// === NEW TESTER & ANALYTICS HANDLERS ===
+async function createTester(args) {
+    if (!args.name || args.name.trim().length === 0) {
+        return { error: 'Tester name is required' };
+    }
+    if (!args.email || !args.email.includes('@')) {
+        return { error: 'A valid email address is required' };
+    }
+    if (args.tier !== undefined && (args.tier < 1 || args.tier > 3)) {
+        return { error: 'Tier must be 1, 2, or 3' };
+    }
+    const validPlatforms = ['ios', 'android', 'web'];
+    if (args.platforms) {
+        for (const p of args.platforms) {
+            if (!validPlatforms.includes(p)) {
+                return { error: `Invalid platform "${p}". Must be one of: ${validPlatforms.join(', ')}` };
+            }
+        }
+    }
+    const { data, error } = await supabase
+        .from('testers')
+        .insert({
+        project_id: currentProjectId,
+        name: args.name.trim(),
+        email: args.email.trim().toLowerCase(),
+        platforms: args.platforms || ['ios', 'web'],
+        tier: args.tier ?? 1,
+        notes: args.notes?.trim() || null,
+        status: 'active',
+        role: args.role || 'tester',
+    })
+        .select('id, name, email, status, platforms, tier, notes, role, created_at')
+        .single();
+    if (error) {
+        if (error.message.includes('duplicate') || error.message.includes('unique')) {
+            return { error: `A tester with email "${args.email}" already exists in this project` };
+        }
+        return { error: error.message };
+    }
+    return {
+        success: true,
+        tester: {
+            id: data.id,
+            name: data.name,
+            email: data.email,
+            status: data.status,
+            platforms: data.platforms,
+            tier: data.tier,
+            notes: data.notes,
+            role: data.role,
+            createdAt: data.created_at,
+        },
+        message: `Tester "${data.name}" added to the project. Use assign_tests to give them test cases.`,
+    };
+}
+async function updateTester(args) {
+    if (!isValidUUID(args.tester_id)) {
+        return { error: 'Invalid tester_id format' };
+    }
+    const updates = {};
+    if (args.status)
+        updates.status = args.status;
+    if (args.platforms)
+        updates.platforms = args.platforms;
+    if (args.tier !== undefined) {
+        if (args.tier < 1 || args.tier > 3) {
+            return { error: 'Tier must be 1, 2, or 3' };
+        }
+        updates.tier = args.tier;
+    }
+    if (args.notes !== undefined)
+        updates.notes = args.notes.trim() || null;
+    if (args.name)
+        updates.name = args.name.trim();
+    if (Object.keys(updates).length === 0) {
+        return { error: 'No fields to update. Provide at least one of: status, platforms, tier, notes, name' };
+    }
+    const { data, error } = await supabase
+        .from('testers')
+        .update(updates)
+        .eq('id', args.tester_id)
+        .eq('project_id', currentProjectId)
+        .select('id, name, email, status, platforms, tier, notes')
+        .single();
+    if (error) {
+        return { error: error.message };
+    }
+    if (!data) {
+        return { error: 'Tester not found in this project' };
+    }
+    return {
+        success: true,
+        tester: {
+            id: data.id,
+            name: data.name,
+            email: data.email,
+            status: data.status,
+            platforms: data.platforms,
+            tier: data.tier,
+            notes: data.notes,
+        },
+        updatedFields: Object.keys(updates),
+    };
+}
+async function bulkUpdateReports(args) {
+    if (!args.report_ids || args.report_ids.length === 0) {
+        return { error: 'At least one report_id is required' };
+    }
+    if (args.report_ids.length > 50) {
+        return { error: 'Maximum 50 reports per bulk update' };
+    }
+    for (const id of args.report_ids) {
+        if (!isValidUUID(id)) {
+            return { error: `Invalid report_id format: ${id}` };
+        }
+    }
+    const updates = { status: args.status };
+    if (args.resolution_notes) {
+        updates.resolution_notes = args.resolution_notes;
+    }
+    // Set resolved_at timestamp for terminal statuses
+    if (['fixed', 'resolved', 'verified', 'wont_fix', 'duplicate', 'closed'].includes(args.status)) {
+        updates.resolved_at = new Date().toISOString();
+    }
+    const { data, error } = await supabase
+        .from('reports')
+        .update(updates)
+        .eq('project_id', currentProjectId)
+        .in('id', args.report_ids)
+        .select('id, status, description');
+    if (error) {
+        return { error: error.message };
+    }
+    const updated = data || [];
+    const updatedIds = new Set(updated.map((r) => r.id));
+    const notFound = args.report_ids.filter(id => !updatedIds.has(id));
+    return {
+        success: true,
+        updatedCount: updated.length,
+        requestedCount: args.report_ids.length,
+        notFound: notFound.length > 0 ? notFound : undefined,
+        status: args.status,
+        reports: updated.map((r) => ({
+            id: r.id,
+            status: r.status,
+            description: r.description?.slice(0, 80),
+        })),
+        message: `Updated ${updated.length} report(s) to "${args.status}".${notFound.length > 0 ? ` ${notFound.length} report(s) not found.` : ''}`,
+    };
+}
+async function getBugTrends(args) {
+    const days = Math.min(args.days || 30, 180);
+    const groupBy = args.group_by || 'week';
+    const since = new Date(Date.now() - days * 86400000).toISOString();
+    const { data, error } = await supabase
+        .from('reports')
+        .select('id, severity, category, status, report_type, created_at')
+        .eq('project_id', currentProjectId)
+        .gte('created_at', since)
+        .order('created_at', { ascending: true });
+    if (error) {
+        return { error: error.message };
+    }
+    const reports = data || [];
+    if (groupBy === 'week') {
+        const weeks = {};
+        for (const r of reports) {
+            const d = new Date(r.created_at);
+            // Get Monday of that week
+            const day = d.getDay();
+            const diff = d.getDate() - day + (day === 0 ? -6 : 1);
+            const monday = new Date(d.setDate(diff));
+            const weekKey = monday.toISOString().slice(0, 10);
+            if (!weeks[weekKey])
+                weeks[weekKey] = { count: 0, critical: 0, high: 0, medium: 0, low: 0 };
+            weeks[weekKey].count++;
+            const sev = (r.severity || 'low');
+            weeks[weekKey][sev]++;
+        }
+        return {
+            period: `${days} days`,
+            groupBy: 'week',
+            totalReports: reports.length,
+            weeks: Object.entries(weeks).map(([week, data]) => ({ week, ...data })),
+        };
+    }
+    if (groupBy === 'severity') {
+        const groups = { critical: 0, high: 0, medium: 0, low: 0 };
+        for (const r of reports)
+            groups[r.severity || 'low']++;
+        return { period: `${days} days`, groupBy: 'severity', totalReports: reports.length, breakdown: groups };
+    }
+    if (groupBy === 'category') {
+        const groups = {};
+        for (const r of reports) {
+            const cat = r.category || 'uncategorized';
+            groups[cat] = (groups[cat] || 0) + 1;
+        }
+        return { period: `${days} days`, groupBy: 'category', totalReports: reports.length, breakdown: groups };
+    }
+    if (groupBy === 'status') {
+        const groups = {};
+        for (const r of reports) {
+            groups[r.status] = (groups[r.status] || 0) + 1;
+        }
+        return { period: `${days} days`, groupBy: 'status', totalReports: reports.length, breakdown: groups };
+    }
+    return { error: `Invalid group_by: ${groupBy}. Must be one of: week, severity, category, status` };
+}
+async function getTesterLeaderboard(args) {
+    const days = Math.min(args.days || 30, 180);
+    const sortBy = args.sort_by || 'tests_completed';
+    const since = new Date(Date.now() - days * 86400000).toISOString();
+    // Get all testers for the project
+    const { data: testers, error: testerErr } = await supabase
+        .from('testers')
+        .select('id, name, email, status, platforms, tier')
+        .eq('project_id', currentProjectId)
+        .eq('status', 'active');
+    if (testerErr)
+        return { error: testerErr.message };
+    // Get completed assignments in the period
+    const { data: assignments, error: assignErr } = await supabase
+        .from('test_assignments')
+        .select('tester_id, status, completed_at, duration_seconds')
+        .eq('project_id', currentProjectId)
+        .gte('completed_at', since)
+        .in('status', ['passed', 'failed']);
+    if (assignErr)
+        return { error: assignErr.message };
+    // Get bugs filed in the period
+    const { data: bugs, error: bugErr } = await supabase
+        .from('reports')
+        .select('tester_id, severity')
+        .eq('project_id', currentProjectId)
+        .gte('created_at', since)
+        .not('tester_id', 'is', null);
+    if (bugErr)
+        return { error: bugErr.message };
+    // Aggregate per tester
+    const testerMap = new Map();
+    for (const t of testers || []) {
+        testerMap.set(t.id, {
+            id: t.id,
+            name: t.name,
+            email: t.email,
+            tier: t.tier,
+            testsCompleted: 0,
+            testsPassed: 0,
+            testsFailed: 0,
+            bugsFound: 0,
+            criticalBugs: 0,
+            avgDurationSeconds: 0,
+            totalDuration: 0,
+        });
+    }
+    for (const a of assignments || []) {
+        const entry = testerMap.get(a.tester_id);
+        if (!entry)
+            continue;
+        entry.testsCompleted++;
+        if (a.status === 'passed')
+            entry.testsPassed++;
+        if (a.status === 'failed')
+            entry.testsFailed++;
+        if (a.duration_seconds)
+            entry.totalDuration += a.duration_seconds;
+    }
+    for (const b of bugs || []) {
+        const entry = testerMap.get(b.tester_id);
+        if (!entry)
+            continue;
+        entry.bugsFound++;
+        if (b.severity === 'critical')
+            entry.criticalBugs++;
+    }
+    let leaderboard = Array.from(testerMap.values()).map(t => ({
+        ...t,
+        passRate: t.testsCompleted > 0 ? Math.round((t.testsPassed / t.testsCompleted) * 100) : 0,
+        avgDurationSeconds: t.testsCompleted > 0 ? Math.round(t.totalDuration / t.testsCompleted) : 0,
+        totalDuration: undefined,
+    }));
+    // Sort
+    if (sortBy === 'bugs_found') {
+        leaderboard.sort((a, b) => b.bugsFound - a.bugsFound);
+    }
+    else if (sortBy === 'pass_rate') {
+        leaderboard.sort((a, b) => b.passRate - a.passRate);
+    }
+    else {
+        leaderboard.sort((a, b) => b.testsCompleted - a.testsCompleted);
+    }
+    return {
+        period: `${days} days`,
+        sortedBy: sortBy,
+        leaderboard,
+    };
+}
+async function exportTestResults(args) {
+    if (!isValidUUID(args.test_run_id)) {
+        return { error: 'Invalid test_run_id format' };
+    }
+    // Get the test run
+    const { data: run, error: runErr } = await supabase
+        .from('test_runs')
+        .select('id, name, description, status, total_tests, passed_tests, failed_tests, started_at, completed_at, created_at')
+        .eq('id', args.test_run_id)
+        .eq('project_id', currentProjectId)
+        .single();
+    if (runErr || !run) {
+        return { error: 'Test run not found in this project' };
+    }
+    // Get all assignments for this run
+    const { data: assignments, error: assignErr } = await supabase
+        .from('test_assignments')
+        .select(`
+      id,
+      status,
+      assigned_at,
+      started_at,
+      completed_at,
+      duration_seconds,
+      is_verification,
+      notes,
+      skip_reason,
+      test_result,
+      feedback_rating,
+      feedback_note,
+      test_case:test_cases(id, test_key, title, priority, description, target_route),
+      tester:testers(id, name, email)
+    `)
+        .eq('test_run_id', args.test_run_id)
+        .eq('project_id', currentProjectId)
+        .order('assigned_at', { ascending: true });
+    if (assignErr) {
+        return { error: assignErr.message };
+    }
+    const all = assignments || [];
+    const passCount = all.filter(a => a.status === 'passed').length;
+    const failCount = all.filter(a => a.status === 'failed').length;
+    const testRunInfo = {
+        id: run.id,
+        name: run.name,
+        description: run.description,
+        status: run.status,
+        startedAt: run.started_at,
+        completedAt: run.completed_at,
+        createdAt: run.created_at,
+    };
+    const summaryInfo = {
+        totalAssignments: all.length,
+        passed: passCount,
+        failed: failCount,
+        blocked: all.filter(a => a.status === 'blocked').length,
+        skipped: all.filter(a => a.status === 'skipped').length,
+        pending: all.filter(a => a.status === 'pending').length,
+        inProgress: all.filter(a => a.status === 'in_progress').length,
+        passRate: all.length > 0 ? Math.round((passCount / all.length) * 100) : 0,
+    };
+    // Compact: return test run info + summary only, no assignments array
+    if (args.compact === true) {
+        return { testRun: testRunInfo, summary: summaryInfo };
+    }
+    // Apply limit (default: 100, max: 500)
+    const assignmentLimit = Math.min(Math.max(args.limit ?? 100, 1), 500);
+    const limitedAssignments = all.slice(0, assignmentLimit);
+    return {
+        testRun: testRunInfo,
+        summary: summaryInfo,
+        assignmentsReturned: limitedAssignments.length,
+        assignmentsTotal: all.length,
+        assignments: limitedAssignments.map((a) => ({
+            id: a.id,
+            status: a.status,
+            assignedAt: a.assigned_at,
+            startedAt: a.started_at,
+            completedAt: a.completed_at,
+            durationSeconds: a.duration_seconds,
+            isVerification: a.is_verification,
+            notes: a.notes,
+            skipReason: a.skip_reason,
+            testResult: a.test_result,
+            feedbackRating: a.feedback_rating,
+            feedbackNote: a.feedback_note,
+            testCase: a.test_case ? {
+                id: a.test_case.id,
+                testKey: a.test_case.test_key,
+                title: a.test_case.title,
+                priority: a.test_case.priority,
+                description: a.test_case.description,
+                targetRoute: a.test_case.target_route,
+            } : null,
+            tester: a.tester ? {
+                id: a.tester.id,
+                name: a.tester.name,
+                email: a.tester.email,
+            } : null,
+        })),
+    };
+}
+async function getTestingVelocity(args) {
+    const days = Math.min(args.days || 14, 90);
+    const since = new Date(Date.now() - days * 86400000).toISOString();
+    const { data, error } = await supabase
+        .from('test_assignments')
+        .select('completed_at, status')
+        .eq('project_id', currentProjectId)
+        .gte('completed_at', since)
+        .in('status', ['passed', 'failed'])
+        .order('completed_at', { ascending: true });
+    if (error) {
+        return { error: error.message };
+    }
+    const completions = data || [];
+    // Group by day
+    const dailyCounts = {};
+    for (let i = 0; i < days; i++) {
+        const d = new Date(Date.now() - (days - 1 - i) * 86400000);
+        dailyCounts[d.toISOString().slice(0, 10)] = 0;
+    }
+    for (const c of completions) {
+        const day = new Date(c.completed_at).toISOString().slice(0, 10);
+        if (dailyCounts[day] !== undefined) {
+            dailyCounts[day]++;
+        }
+    }
+    const dailyArray = Object.entries(dailyCounts).map(([date, count]) => ({ date, count }));
+    const totalCompleted = completions.length;
+    const avgPerDay = days > 0 ? Math.round((totalCompleted / days) * 10) / 10 : 0;
+    // Trend: compare first half to second half
+    const mid = Math.floor(dailyArray.length / 2);
+    const firstHalf = dailyArray.slice(0, mid).reduce((sum, d) => sum + d.count, 0);
+    const secondHalf = dailyArray.slice(mid).reduce((sum, d) => sum + d.count, 0);
+    const trend = secondHalf > firstHalf ? 'increasing' : secondHalf < firstHalf ? 'decreasing' : 'stable';
+    return {
+        period: `${days} days`,
+        totalCompleted,
+        averagePerDay: avgPerDay,
+        trend,
+        daily: dailyArray,
+    };
+}
 // Main server setup
 async function main() {
     initSupabase();
@@ -4570,6 +6095,11 @@ async function main() {
         const { name, arguments: args } = request.params;
         try {
             let result;
+            // Project management tools don't require a project to be selected
+            const projectFreeTools = ['list_projects', 'switch_project', 'get_current_project'];
+            if (!projectFreeTools.includes(name)) {
+                requireProject();
+            }
             switch (name) {
                 case 'list_reports':
                     result = await listReports(args);
@@ -4586,6 +6116,12 @@ async function main() {
                 case 'get_report_context':
                     result = await getReportContext(args);
                     break;
+                case 'add_report_comment':
+                    result = await addReportComment(args);
+                    break;
+                case 'get_report_comments':
+                    result = await getReportComments(args);
+                    break;
                 case 'get_project_info':
                     result = await getProjectInfo();
                     break;
@@ -4699,9 +6235,70 @@ async function main() {
                 case 'assign_tests':
                     result = await assignTests(args);
                     break;
+                case 'unassign_tests':
+                    result = await unassignTests(args);
+                    break;
                 case 'get_tester_workload':
                     result = await getTesterWorkload(args);
                     break;
+                // === NEW TESTER & ANALYTICS TOOLS ===
+                case 'create_tester':
+                    result = await createTester(args);
+                    break;
+                case 'update_tester':
+                    result = await updateTester(args);
+                    break;
+                case 'bulk_update_reports':
+                    result = await bulkUpdateReports(args);
+                    break;
+                case 'get_bug_trends':
+                    result = await getBugTrends(args);
+                    break;
+                case 'get_tester_leaderboard':
+                    result = await getTesterLeaderboard(args);
+                    break;
+                case 'export_test_results':
+                    result = await exportTestResults(args);
+                    break;
+                case 'get_testing_velocity':
+                    result = await getTestingVelocity(args);
+                    break;
+                // === PROJECT MANAGEMENT ===
+                case 'list_projects':
+                    result = await listProjects();
+                    break;
+                case 'switch_project':
+                    result = await switchProject(args);
+                    break;
+                case 'get_current_project':
+                    result = getCurrentProject();
+                    break;
+                // === TEST EXECUTION INTELLIGENCE ===
+                case 'get_test_impact':
+                    result = await getTestImpact(args);
+                    break;
+                case 'get_flaky_tests':
+                    result = await getFlakyTests(args);
+                    break;
+                case 'assess_test_quality':
+                    result = await assessTestQuality(args);
+                    break;
+                case 'get_test_execution_summary':
+                    result = await getTestExecutionSummary(args);
+                    break;
+                case 'check_test_freshness':
+                    result = await checkTestFreshness(args);
+                    break;
+                case 'get_untested_changes':
+                    result = await getUntestedChanges(args);
+                    break;
+                // === AUTO-MONITORING TOOLS ===
+                case 'get_auto_detected_issues':
+                    result = await getAutoDetectedIssues(args);
+                    break;
+                case 'generate_tests_from_errors':
+                    result = await generateTestsFromErrors(args);
+                    break;
                 default:
                     return {
                         content: [{ type: 'text', text: `Unknown tool: ${name}` }],
@@ -4724,7 +6321,7 @@ async function main() {
         const { data, error } = await supabase
             .from('reports')
             .select('id, description, report_type, severity')
-            .eq('project_id', PROJECT_ID)
+            .eq('project_id', currentProjectId)
             .eq('status', 'new')
             .order('created_at', { ascending: false })
             .limit(10);