@popoverai/dotrequirements 0.22.0 → 0.24.0

package/dist/mcp/handlers/list.js

@@ -1,65 +1,43 @@
-/**
- * list_all_requirements and list_untested_requirements handlers
- *
- * Lists requirements in a project with optional filtering for untested requirements.
- */
-import { textResponse } from './types.js';
-import { getReferencedRequirementIds } from '../grep.js';
-/**
- * Handler for list_all_requirements tool
- *
- * Requirements covered:
- * - MCP-LIST-1.0: Response includes count of total requirements and total nodes
- * - MCP-LIST-1.1: Each requirement displays its key, label, and child count
- * - MCP-LIST-1.2: Requirements are listed from all files discovered in the project
- */
-export async function handleListAllRequirements(args, context) {
-    const { projectId } = args;
+import { getReferencedRequirementIds } from "../../requirements/grep.js";
+import { textResponse } from "./types.js";
+export async function handleList(args, context) {
+    const { untested = false, projectId } = args;
     const project = await context.getProjectFromDiscovery(projectId);
     const requirements = await context.getRequirements(projectId);
     if (requirements.length === 0) {
         return textResponse(`No requirements found in project: ${project.path}\n\nMake sure .requirements/ directory exists with Markdown files (*.requirements.md).`);
     }
-    // MCP-LIST-1.2: Group requirements from all discovered files by root ID
+    if (untested) {
+        return formatUntested(requirements, project.path);
+    }
+    return formatAll(requirements);
+}
+function formatAll(requirements) {
     const grouped = new Map();
     for (const req of requirements) {
         const existing = grouped.get(req.rootId) || [];
         existing.push(req);
         grouped.set(req.rootId, existing);
     }
-    // MCP-LIST-1.1: Each requirement displays key, label (from content), and child count
     const formatted = Array.from(grouped.entries())
         .map(([rootId, reqs]) => {
         const root = reqs.find((r) => r.path.length === 0);
         const childCount = reqs.length - 1;
-        return `- **${rootId}**: ${root?.content || '(no content)'} (${childCount} children)`;
+        return `- **${rootId}**: ${root?.content || "(no content)"} (${childCount} children)`;
     })
-        .join('\n');
-    // MCP-LIST-1.0: Response includes count of total requirements and total nodes
+        .join("\n");
     return textResponse(`Found ${grouped.size} requirement(s) with ${requirements.length} total nodes:\n\n${formatted}`);
 }
-/**
- * Handler for list_untested_requirements tool
- *
- * Requirements covered:
- * - MCP-LIST-2.0: Only requirements with no test references are included
- * - MCP-LIST-2.1: Response indicates count of requirements lacking coverage
- */
-export async function handleListUntestedRequirements(args, context) {
-    const { projectId } = args;
-    const project = await context.getProjectFromDiscovery(projectId);
-    const requirements = await context.getRequirements(projectId);
-    const referencedIds = await getReferencedRequirementIds(project.path);
-    // MCP-LIST-2.0: Filter to root requirements without test references
+async function formatUntested(requirements, projectPath) {
+    const referencedIds = await getReferencedRequirementIds(projectPath);
     const rootRequirements = requirements.filter((r) => r.path.length === 0);
     const untested = rootRequirements.filter((r) => !referencedIds.has(r.id));
     if (untested.length === 0) {
-        return textResponse(`All ${rootRequirements.length} requirements have test references!`);
+        return textResponse(`All ${rootRequirements.length} requirement(s) have test references — coverage is complete.`);
     }
     const formatted = untested
         .map((r) => `- **${r.id}**: ${r.content}`)
-        .join('\n');
-    // MCP-LIST-2.1: Response indicates count of requirements lacking coverage
+        .join("\n");
     return textResponse(`Found ${untested.length} untested requirement(s) out of ${rootRequirements.length} total:\n\n${formatted}`);
 }
 //# sourceMappingURL=list.js.map

package/dist/mcp/handlers/push.d.ts

@@ -4,7 +4,7 @@
  * Provides requirements push functionality:
  * - push_requirements: Push local requirements to cloud
  */
-import type { HandlerContext, ToolResponse } from './types.js';
+import type { HandlerContext, ToolResponse } from "./types.js";
 /**
  * Arguments for push_requirements tool
  */

package/dist/mcp/handlers/push.js

@@ -4,12 +4,12 @@
  * Provides requirements push functionality:
  * - push_requirements: Push local requirements to cloud
  */
-import { textResponse, errorResponse } from './types.js';
-import { existsSync } from 'fs';
-import { resolve, basename } from 'path';
-import { CONVEX_URL } from '../convexClient.js';
-import { findRequirementsFiles } from '../requirements.js';
-import { parseFilesForPush, dryRunPush, executePush, } from '../../push/index.js';
+import { existsSync } from "node:fs";
+import { basename, resolve } from "node:path";
+import { dryRunPush, executePush, parseFilesForPush, } from "../../push/index.js";
+import { findRequirementsFiles } from "../../requirements/index.js";
+import { CONVEX_URL } from "../convexClient.js";
+import { errorResponse, textResponse } from "./types.js";
 /**
  * Handler for push_requirements tool
  *
@@ -43,7 +43,7 @@ export async function handlePushRequirements(args, context) {
         // Use same file discovery as CLI - searches entire workspace
         filesToPush = await findRequirementsFiles(project.path);
         if (filesToPush.length === 0) {
-            return textResponse('No *.requirements.md files found. Nothing to push.');
+            return textResponse("No *.requirements.md files found. Nothing to push.");
         }
     }
     // Parse files
@@ -64,9 +64,14 @@ export async function handlePushRequirements(args, context) {
         // Only invalid files
         const invalidList = dryRunResult.invalid
             .map(({ file, result }) => `- ${basename(file.filePath)}: ${result.error}`)
-            .join('\n');
+            .join("\n");
         return {
-            content: [{ type: 'text', text: `✗ No valid documents to push.\n\n**Invalid files:**\n${invalidList}` }],
+            content: [
+                {
+                    type: "text",
+                    text: `✗ No valid documents to push.\n\n**Invalid files:**\n${invalidList}`,
+                },
+            ],
             isError: true,
         };
     }
@@ -78,20 +83,20 @@ export async function handlePushRequirements(args, context) {
             for (const { file, result } of dryRunResult.updates) {
                 const fileName = basename(file.filePath);
                 const hasConflict = dryRunResult.conflicts.some((c) => c.item.file === file);
-                const conflictNote = hasConflict ? ' ⚠️ [cloud changed since pull]' : '';
-                const warningNote = result.warning ? ` (${result.warning})` : '';
+                const conflictNote = hasConflict ? " ⚠️ [cloud changed since pull]" : "";
+                const warningNote = result.warning ? ` (${result.warning})` : "";
                 summary += `- ~ ${fileName}${conflictNote}${warningNote}\n`;
             }
-            summary += '\n';
+            summary += "\n";
         }
         if (dryRunResult.creates.length > 0) {
             summary += `## New Documents (${dryRunResult.creates.length})\n`;
             for (const { file, result } of dryRunResult.creates) {
                 const fileName = basename(file.filePath);
-                const warningNote = result.warning ? ` (${result.warning})` : '';
+                const warningNote = result.warning ? ` (${result.warning})` : "";
                 summary += `- + ${fileName}${warningNote}\n`;
             }
-            summary += '\n';
+            summary += "\n";
         }
         if (dryRunResult.notFound.length > 0) {
             summary += `## Not Found in Cloud (${dryRunResult.notFound.length}) - will create new\n`;
@@ -99,7 +104,7 @@ export async function handlePushRequirements(args, context) {
                 const fileName = basename(file.filePath);
                 summary += `- ! ${fileName} (ID: ${result.documentId})\n`;
             }
-            summary += '\n';
+            summary += "\n";
         }
         if (dryRunResult.invalid.length > 0) {
             summary += `## Skipped - Invalid (${dryRunResult.invalid.length})\n`;
@@ -107,7 +112,7 @@ export async function handlePushRequirements(args, context) {
                 const fileName = basename(file.filePath);
                 summary += `- ✗ ${fileName}: ${result.error}\n`;
             }
-            summary += '\n';
+            summary += "\n";
         }
         if (dryRunResult.conflicts.length > 0) {
             summary += `⚠️ **Warning:** ${dryRunResult.conflicts.length} file(s) have cloud changes since last pull. Pushing will overwrite those changes.\n\n`;
@@ -118,7 +123,7 @@ export async function handlePushRequirements(args, context) {
     // MCP-PUSH-1.1: When confirmed, execute push
     try {
         const result = await executePush(dryRunResult, credentials);
-        let output = '✓ Push complete!\n\n';
+        let output = "✓ Push complete!\n\n";
         if (result.created > 0) {
             output += `**Created:** ${result.created} document(s)\n`;
         }
@@ -135,7 +140,12 @@ export async function handlePushRequirements(args, context) {
     }
     catch (error) {
         return {
-            content: [{ type: 'text', text: `✗ Push failed:\n\n${error instanceof Error ? error.message : String(error)}` }],
+            content: [
+                {
+                    type: "text",
+                    text: `✗ Push failed:\n\n${error instanceof Error ? error.message : String(error)}`,
+                },
+            ],
             isError: true,
         };
     }

package/dist/mcp/handlers/report.d.ts

@@ -0,0 +1,16 @@
+/**
+ * Unified coverage handler for the MCP `report_coverage` tool. Replaces
+ * `get_requirement_coverage` and `get_project_coverage_summary` with a
+ * single tool that takes a `source: "local" | "cloud"` argument and
+ * mirrors the CLI `dotreq report` semantics.
+ */
+import type { HandlerContext, ToolResponse } from "./types.js";
+export interface ReportArgs {
+    source?: "local" | "cloud";
+    requirementKey?: string;
+    branch?: string;
+    sinceTimestamp?: number;
+    projectId?: string;
+}
+export declare function handleReport(args: ReportArgs, context: HandlerContext): Promise<ToolResponse>;
+//# sourceMappingURL=report.d.ts.map

package/dist/mcp/handlers/report.js

@@ -0,0 +1,134 @@
+/**
+ * Unified coverage handler for the MCP `report_coverage` tool. Replaces
+ * `get_requirement_coverage` and `get_project_coverage_summary` with a
+ * single tool that takes a `source: "local" | "cloud"` argument and
+ * mirrors the CLI `dotreq report` semantics.
+ */
+import { getProjectCoverage as queryProjectCoverage, getRequirementCoverage as queryRequirementCoverage, } from "../../requirements/cloud-coverage.js";
+import { buildLocalReport, LocalReportCacheMissingError, } from "../../requirements/coverage.js";
+import { CONVEX_URL } from "../convexClient.js";
+import { errorResponse, textResponse } from "./types.js";
+export async function handleReport(args, context) {
+    const source = args.source ?? "local";
+    if (source === "local") {
+        if (args.branch || args.sinceTimestamp !== undefined) {
+            return errorResponse("branch and sinceTimestamp require source=cloud. Local reports cover only the most recent test run on this machine.");
+        }
+        try {
+            const report = buildLocalReport({
+                projectRoot: context.workspaceRoot,
+                requirementFilter: args.requirementKey,
+            });
+            return textResponse(formatLocalReport(report));
+        }
+        catch (error) {
+            if (error instanceof LocalReportCacheMissingError) {
+                return errorResponse(error.message);
+            }
+            throw error;
+        }
+    }
+    // source === "cloud"
+    let project;
+    try {
+        project = await context.getProjectFromDiscovery(args.projectId);
+    }
+    catch (error) {
+        return errorResponse(`Cloud report requires cloud credentials. Run \`dotrequirements link\` to connect to cloud.\n\nError: ${error instanceof Error ? error.message : String(error)}`);
+    }
+    if (args.requirementKey) {
+        const record = await queryRequirementCoverage(args.requirementKey, project.projectId, project.projectSecret, CONVEX_URL);
+        return textResponse(formatCloudRequirement(record));
+    }
+    const record = await queryProjectCoverage(project.projectId, project.projectSecret, CONVEX_URL, {
+        branch: args.branch,
+        sinceTimestamp: args.sinceTimestamp,
+    });
+    return textResponse(formatCloudProject(record, {
+        branch: args.branch,
+        sinceTimestamp: args.sinceTimestamp,
+    }));
+}
+function formatLocalReport(report) {
+    const { total, tested, untested, coveragePercent } = report.summary;
+    let out = `# Coverage Report (local)\n\n`;
+    if (report.warnings.length > 0) {
+        for (const w of report.warnings) {
+            out += `> ⚠️ ${w}\n`;
+        }
+        out += "\n";
+    }
+    out += `**Total:** ${total} | **Tested:** ${tested} | **Untested:** ${untested} | **Coverage:** ${coveragePercent}%\n\n`;
+    const testedEntries = report.entries.filter((c) => c.tested);
+    if (testedEntries.length > 0) {
+        out += `## Tested (${testedEntries.length})\n\n`;
+        for (const c of testedEntries) {
+            out += `- **${c.key}**: ${c.content}\n`;
+            for (const loc of c.locations) {
+                out += `  - \`${loc}\`\n`;
+            }
+        }
+        out += "\n";
+    }
+    const untestedEntries = report.entries.filter((c) => !c.tested);
+    if (untestedEntries.length > 0) {
+        out += `## Untested (${untestedEntries.length})\n\n`;
+        for (const c of untestedEntries) {
+            out += `- **${c.key}**: ${c.content}\n`;
+        }
+    }
+    return out;
+}
+function formatCloudRequirement(record) {
+    if (!record.lastTestedAt) {
+        return `**${record.requirementKey}** has no cloud coverage records.`;
+    }
+    const lastTested = new Date(record.lastTestedAt).toISOString();
+    let out = `# Coverage for ${record.requirementKey} (cloud)\n\n`;
+    out += `**Last tested:** ${lastTested}\n`;
+    if (record.branch)
+        out += `**Branch:** ${record.branch}\n`;
+    if (record.testFile) {
+        out += `**Test location:** \`${record.testFile}${record.testLine ? `:${record.testLine}` : ""}\`\n`;
+    }
+    const otherBranches = record.allBranches.filter((b) => b.branch !== record.branch);
+    if (otherBranches.length > 0) {
+        out += `\n**Also tested on:**\n`;
+        for (const b of otherBranches) {
+            out += `- ${b.branch}: ${new Date(b.lastTestedAt).toISOString()}\n`;
+        }
+    }
+    return out;
+}
+function formatCloudProject(record, filters) {
+    const tested = record.tested.length;
+    const untested = record.untested.length;
+    const total = tested + untested;
+    const coveragePercent = total > 0 ? ((tested / total) * 100).toFixed(1) : "0.0";
+    let filterNote = "";
+    if (filters.branch)
+        filterNote += `\n**Branch:** ${filters.branch}`;
+    if (filters.sinceTimestamp) {
+        filterNote += `\n**Since:** ${new Date(filters.sinceTimestamp).toISOString()}`;
+    }
+    let out = `# Project Coverage Report (cloud)\n\n`;
+    out += `**Total Requirements:** ${total}\n**Coverage:** ${coveragePercent}%${filterNote}\n`;
+    if (record.tested.length > 0) {
+        out += `\n## Tested Requirements (${record.tested.length})\n\n`;
+        for (const t of record.tested) {
+            const date = new Date(t.lastTestedAt).toISOString();
+            const location = t.testFile
+                ? ` (\`${t.testFile}${t.testLine ? `:${t.testLine}` : ""}\`)`
+                : "";
+            out += `- **${t.requirementKey}** [${t.branch}]: ${date}${location}\n`;
+        }
+    }
+    if (record.untested.length > 0) {
+        out += `\n## Untested Requirements (${record.untested.length})\n\n`;
+        for (const key of record.untested) {
+            out += `- ${key}\n`;
+        }
+    }
+    return out;
+}
+//# sourceMappingURL=report.js.map

package/dist/mcp/handlers/review.d.ts

@@ -5,7 +5,7 @@
  * - style_check: Check requirements or test files for style issues
  * - review_test: Comprehensively review tests for semantic correctness
  */
-import type { HandlerContext, ToolResponse } from './types.js';
+import type { HandlerContext, ToolResponse } from "./types.js";
 /**
  * Arguments for style_check tool
  */

package/dist/mcp/handlers/review.js

@@ -5,11 +5,12 @@
  * - style_check: Check requirements or test files for style issues
  * - review_test: Comprehensively review tests for semantic correctness
  */
-import { textResponse, errorResponse } from './types.js';
-import { existsSync, readFileSync } from 'fs';
-import { resolve, dirname } from 'path';
-import { filterRequirementsByKeys, getRequirementTree, formatRequirementTree, } from '../requirements.js';
-import { discoverProjects, } from '../../utils/project-discovery.js';
+import { existsSync, readFileSync } from "node:fs";
+import { dirname, resolve } from "node:path";
+import { fetchReviewTestFeedback, fetchStyleCheckFeedback, } from "../../requirements/cloud-ai.js";
+import { filterRequirementsByKeys, formatRequirementTree, getRequirementTree, } from "../../requirements/index.js";
+import { discoverProjects, } from "../../utils/project-discovery.js";
+import { errorResponse, textResponse } from "./types.js";
 const STYLE_CHECK_GUIDANCE = `
 
 ---
@@ -32,24 +33,24 @@ const STYLE_CHECK_GUIDANCE = `
  */
 export async function handleStyleCheck(args, context, options) {
     const { filePath, requirementKeys, model } = args;
-    const { useEnvAuth = false, apiBaseUrl = 'https://app.dotrequirements.io' } = options ?? {};
+    const { useEnvAuth = false, apiBaseUrl = "https://app.dotrequirements.io" } = options ?? {};
     const fullPath = resolve(context.workspaceRoot, filePath);
     if (!existsSync(fullPath)) {
         return errorResponse(`File not found: ${filePath}`);
     }
     // Read file contents
-    const fileContents = readFileSync(fullPath, 'utf-8');
+    const fileContents = readFileSync(fullPath, "utf-8");
     // Detect file type
     // MCP-REVIEW-1.0 & MCP-REVIEW-1.1: Different feedback for different file types
-    const isRequirementsFile = filePath.endsWith('.requirements.md');
+    const isRequirementsFile = filePath.endsWith(".requirements.md");
     const isTestFile = /\.(test|spec)\.(js|jsx|ts|tsx)$/.test(filePath);
     if (!isRequirementsFile && !isTestFile) {
         return errorResponse(`Unsupported file type. File must be:\n- Requirements file: *.requirements.md\n- Test file: *.test.{js,jsx,ts,tsx} or *.spec.{js,jsx,ts,tsx}`);
     }
-    const fileType = isRequirementsFile ? 'requirements' : 'test';
+    const fileType = isRequirementsFile ? "requirements" : "test";
     // If requirementKeys provided, filter the file to only those requirements
     let fileContentsToCheck = fileContents;
-    let scopeNote = '';
+    let scopeNote = "";
     if (requirementKeys && requirementKeys.length > 0) {
         if (!isRequirementsFile) {
             return errorResponse(`requirementKeys can only be used with requirements files (*.requirements.md), not test files.`);
@@ -57,11 +58,11 @@ export async function handleStyleCheck(args, context, options) {
         try {
             const { filteredContent, foundKeys, missingKeys } = filterRequirementsByKeys(fileContents, requirementKeys);
             if (foundKeys.length === 0) {
-                return errorResponse(`None of the specified requirement keys were found in ${filePath}: ${requirementKeys.join(', ')}`);
+                return errorResponse(`None of the specified requirement keys were found in ${filePath}: ${requirementKeys.join(", ")}`);
             }
             fileContentsToCheck = filteredContent;
             if (missingKeys.length > 0) {
-                scopeNote = `\n\n> **Note:** Some specified keys were not found in the file: ${missingKeys.join(', ')}`;
+                scopeNote = `\n\n> **Note:** Some specified keys were not found in the file: ${missingKeys.join(", ")}`;
             }
         }
         catch (error) {
@@ -79,50 +80,40 @@ export async function handleStyleCheck(args, context, options) {
         else {
             // Try to find a project starting from the file's directory
             const result = await discoverProjects(fileDir);
-            if (result.type === 'none') {
+            if (result.type === "none") {
                 // If no project found from file dir, try from context
                 project = await context.getProjectFromDiscovery();
             }
-            else if (result.type === 'single') {
+            else if (result.type === "single") {
                 project = result.project;
             }
             else {
                 // Multiple projects - can't auto-detect which one to use
-                throw new Error('Multiple projects found - cannot auto-detect for this file');
+                throw new Error("Multiple projects found - cannot auto-detect for this file");
             }
         }
     }
     catch (error) {
         return errorResponse(`Style check requires project credentials. Run \`dotrequirements link\` to connect to cloud.\n\nError: ${error instanceof Error ? error.message : String(error)}`);
     }
-    // Call API endpoint for style checking
+    // Call hosted endpoint via the shared library helper
     try {
-        const response = await fetch(`${apiBaseUrl}/api/style-check`, {
-            method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-            },
-            body: JSON.stringify({
-                projectId: project.projectId,
-                projectSecret: project.projectSecret,
-                fileContents: fileContentsToCheck,
-                fileType,
-                model,
-            }),
+        const feedback = await fetchStyleCheckFeedback({
+            apiBaseUrl,
+            projectId: project.projectId,
+            projectSecret: project.projectSecret,
+            fileContents: fileContentsToCheck,
+            fileType,
+            model,
         });
-        if (!response.ok) {
-            const errorData = await response.json();
-            return errorResponse(`Style check failed: ${errorData.error || response.statusText}`);
-        }
-        const data = await response.json();
         const scopeLabel = requirementKeys && requirementKeys.length > 0
-            ? ` (${requirementKeys.join(', ')})`
-            : '';
+            ? ` (${requirementKeys.join(", ")})`
+            : "";
         // MCP-REVIEW-1.2: Feedback includes severity categories (in the API response)
-        return textResponse(`# Style Check Results for \`${filePath}\`${scopeLabel}\n\n${data.feedback}${scopeNote}${STYLE_CHECK_GUIDANCE}`);
+        return textResponse(`# Style Check Results for \`${filePath}\`${scopeLabel}\n\n${feedback}${scopeNote}${STYLE_CHECK_GUIDANCE}`);
     }
     catch (error) {
-        return errorResponse(`Style check error: ${error instanceof Error ? error.message : String(error)}`);
+        return errorResponse(`Style check failed: ${error instanceof Error ? error.message : String(error)}`);
     }
 }
 /**
@@ -138,7 +129,7 @@ export async function handleStyleCheck(args, context, options) {
  */
 export async function handleReviewTest(args, context, options) {
     const { testFilePath, projectId } = args;
-    const { apiBaseUrl = 'https://app.dotrequirements.io' } = options ?? {};
+    const { apiBaseUrl = "https://app.dotrequirements.io" } = options ?? {};
     const fullPath = resolve(context.workspaceRoot, testFilePath);
     if (!existsSync(fullPath)) {
         return errorResponse(`File not found: ${testFilePath}`);
@@ -149,7 +140,7 @@ export async function handleReviewTest(args, context, options) {
         return errorResponse(`File must be a test file: *.test.{js,jsx,ts,tsx} or *.spec.{js,jsx,ts,tsx}`);
     }
     // Read test file contents
-    const testFileContents = readFileSync(fullPath, 'utf-8');
+    const testFileContents = readFileSync(fullPath, "utf-8");
     // Extract requirement IDs from the test file using regex
     const requirementIdMatches = testFileContents.matchAll(/requirement\s*\(\s*['"`]([^'"`]+)['"`]\s*\)/g);
     const requirementIds = new Set();
@@ -170,7 +161,7 @@ export async function handleReviewTest(args, context, options) {
     // e.g., NAV-3.0, NAV-3.1, NAV-3.2 all belong to root NAV-3
     const rootToTestedIds = new Map();
     for (const reqId of requirementIds) {
-        const req = allRequirements.find(r => r.id === reqId || r.rootId === reqId);
+        const req = allRequirements.find((r) => r.id === reqId || r.rootId === reqId);
         if (req) {
             if (!rootToTestedIds.has(req.rootId)) {
                 rootToTestedIds.set(req.rootId, new Set());
@@ -190,30 +181,20 @@ export async function handleReviewTest(args, context, options) {
             testedIds: Array.from(testedIds).sort(),
         });
     }
-    // Call API endpoint for test review
+    // Call hosted endpoint via the shared library helper
     // MCP-REVIEW-2.3 & 2.4: API identifies coverage gaps and non-existent references
     try {
-        const response = await fetch(`${apiBaseUrl}/api/review-test`, {
-            method: 'POST',
-            headers: {
-                'Content-Type': 'application/json',
-            },
-            body: JSON.stringify({
-                projectId: project.projectId,
-                projectSecret: project.projectSecret,
-                testFileContents,
-                requirements,
-            }),
+        const feedback = await fetchReviewTestFeedback({
+            apiBaseUrl,
+            projectId: project.projectId,
+            projectSecret: project.projectSecret,
+            testFileContents,
+            requirements,
         });
-        if (!response.ok) {
-            const errorData = await response.json();
-            return errorResponse(`Test review failed: ${errorData.error || response.statusText}`);
-        }
-        const data = await response.json();
-        return textResponse(`# Test Review Results for \`${testFilePath}\`\n\n${data.feedback}`);
+        return textResponse(`# Test Review Results for \`${testFilePath}\`\n\n${feedback}`);
     }
     catch (error) {
-        return errorResponse(`Test review error: ${error instanceof Error ? error.message : String(error)}`);
+        return errorResponse(`Test review failed: ${error instanceof Error ? error.message : String(error)}`);
     }
 }
 //# sourceMappingURL=review.js.map

package/dist/mcp/handlers/search.d.ts

@@ -4,7 +4,7 @@
  * Searches requirements by text or regex query across IDs, content, and labels.
  * Returns matching requirements with their full tree context.
  */
-import type { HandlerContext, ToolResponse } from './types.js';
+import type { HandlerContext, ToolResponse } from "./types.js";
 /**
  * Arguments for search_requirements tool
  */

package/dist/mcp/handlers/search.js

@@ -4,8 +4,8 @@
  * Searches requirements by text or regex query across IDs, content, and labels.
  * Returns matching requirements with their full tree context.
  */
-import { textResponse, errorResponse } from './types.js';
-import { searchRequirements, getRequirementTree, formatRequirementTree, } from '../requirements.js';
+import { formatRequirementTree, getRequirementTree, searchRequirements, } from "../../requirements/index.js";
+import { errorResponse, textResponse } from "./types.js";
 /**
  * Handler for search_requirements tool
  *
@@ -27,10 +27,8 @@ export async function handleSearchRequirements(args, context) {
         // MCP-SEARCH-1.3: Interpret query as case-insensitive regex
         // MCP-SEARCH-1.4: Return error for invalid regex
         try {
-            const regex = new RegExp(query, 'i');
-            results = requirements.filter((r) => regex.test(r.id) ||
-                regex.test(r.content) ||
-                regex.test(r.label));
+            const regex = new RegExp(query, "i");
+            results = requirements.filter((r) => regex.test(r.id) || regex.test(r.content) || regex.test(r.label));
         }
         catch (error) {
             return errorResponse(`Invalid regular expression: ${error instanceof Error ? error.message : String(error)}`);
@@ -45,7 +43,7 @@ export async function handleSearchRequirements(args, context) {
     const rootResults = requirements.filter((r) => r.path.length === 0 && matchedRootIds.has(r.rootId));
     // MCP-SEARCH-1.5: No matches returns informative message
     if (rootResults.length === 0) {
-        return textResponse(`No requirements found matching "${query}"${useRegex ? ' (regex)' : ''}`);
+        return textResponse(`No requirements found matching "${query}"${useRegex ? " (regex)" : ""}`);
     }
     // MCP-SEARCH-1.7: Each result includes full tree as code block
     const formatted = rootResults
@@ -54,7 +52,7 @@ export async function handleSearchRequirements(args, context) {
         const treeFormatted = formatRequirementTree(tree);
         return `**${r.id}** (${r.label})\n${r.content}\n_Source: ${r.documentTitle}_\n\n\`\`\`\n${treeFormatted}\n\`\`\``;
     })
-        .join('\n\n---\n\n');
-    return textResponse(`Found ${rootResults.length} requirement(s) matching "${query}"${useRegex ? ' (regex)' : ''}:\n\n${formatted}`);
+        .join("\n\n---\n\n");
+    return textResponse(`Found ${rootResults.length} requirement(s) matching "${query}"${useRegex ? " (regex)" : ""}:\n\n${formatted}`);
 }
 //# sourceMappingURL=search.js.map

package/dist/mcp/handlers/test-mapping.d.ts

@@ -3,7 +3,7 @@
  *
  * Maps between test files and requirements in both directions.
  */
-import type { HandlerContext, ToolResponse } from './types.js';
+import type { HandlerContext, ToolResponse } from "./types.js";
 /**
  * Arguments for get_requirements_by_test tool
  */