@popoverai/dotrequirements 0.26.2 → 0.27.0

package/dist/mcp/handlers/push.js

@@ -1,232 +0,0 @@
-/**
- * Push handler for MCP tools
- *
- * Provides requirements push functionality:
- * - push_requirements: Push local requirements to cloud
- */
-import { existsSync } from "node:fs";
-import { basename, resolve } from "node:path";
-import { dryRunPush, executePush, parseFilesForPushIndividually, } 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
- *
- * Requirements covered:
- * - MCP-PUSH-1.0: When not confirmed, no changes are made and a preview of creates/updates is returned
- * - MCP-PUSH-1.1: When confirmed is true, the push executes and returns success message
- * - MCP-PUSH-1.2: When credentials are missing or invalid, an error explains how to authenticate
- * - MCP-PUSH-1.3: When filePath is provided, only that file is pushed (MCP-PUSH-1.3.0: missing file is an error)
- */
-export async function handlePushRequirements(args, context) {
-    const { filePath, confirmed = false, projectId } = args;
-    // MCP-PUSH-1.2: Get project credentials (throws if not configured)
-    let project;
-    try {
-        project = await context.getProjectFromDiscovery(projectId);
-    }
-    catch (error) {
-        return errorResponse(`Push requires project credentials. Run \`dotrequirements link\` to connect to cloud.\n\nError: ${error instanceof Error ? error.message : String(error)}`);
-    }
-    // Determine files to push
-    let filesToPush;
-    // MCP-PUSH-1.3: When filePath is provided, only that file is pushed
-    if (filePath) {
-        const fullPath = resolve(project.path, filePath);
-        if (!existsSync(fullPath)) {
-            return errorResponse(`File not found: ${filePath}`);
-        }
-        filesToPush = [fullPath];
-    }
-    else {
-        // 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.");
-        }
-    }
-    // Parse files. #45/SYNC-FAIL-4: one invalid file does not abort the whole
-    // push — its failure is surfaced in the preview's "Skipped - Invalid"
-    // section while valid files still push. Same helper as the CLI command.
-    const { parsedFiles, totalRequirements, parseFailures } = parseFilesForPushIndividually(filesToPush);
-    // Build credentials
-    const credentials = {
-        projectId: project.projectId,
-        projectSecret: project.projectSecret,
-        convexUrl: CONVEX_URL,
-    };
-    // Run dry run (always, even when confirmed - ensures fresh state).
-    // SYNC-FAIL-3: two files claiming the same document ID abort the push
-    // before any cloud write — surface that as a clean error result naming
-    // both files rather than an unhandled throw.
-    let dryRunResult;
-    try {
-        dryRunResult = await dryRunPush(parsedFiles, credentials);
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        if (message.includes("claim the same document ID")) {
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `✗ Push aborted: ${message}`,
-                    },
-                ],
-                isError: true,
-            };
-        }
-        throw error;
-    }
-    // #45: Merge dry-run invalids (missing document section, cloud validation)
-    // with local parse failures so both are surfaced together, each named by
-    // filename.
-    const allInvalid = [
-        ...dryRunResult.invalid.map(({ file, result }) => ({
-            fileName: basename(file.filePath),
-            error: result.error ?? "invalid",
-        })),
-        ...parseFailures.map(({ filePath, error }) => ({
-            fileName: basename(filePath),
-            error,
-        })),
-    ];
-    // Check if there's anything to push
-    const pushableCount = dryRunResult.updates.length +
-        dryRunResult.creates.length +
-        dryRunResult.notFound.length;
-    if (pushableCount === 0 && allInvalid.length > 0) {
-        // Only invalid files
-        const invalidList = allInvalid
-            .map(({ fileName, error }) => `- ${fileName}: ${error}`)
-            .join("\n");
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `✗ No valid documents to push.\n\n**Invalid files:**\n${invalidList}`,
-                },
-            ],
-            isError: true,
-        };
-    }
-    // MCP-PUSH-1.0: When not confirmed, show preview
-    if (!confirmed) {
-        let summary = `# Push Preview\n\n**Files:** ${parsedFiles.length}\n**Requirements:** ${totalRequirements}\n\n`;
-        if (dryRunResult.updates.length > 0) {
-            summary += `## Updates (${dryRunResult.updates.length})\n`;
-            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})` : "";
-                summary += `- ~ ${fileName}${conflictNote}${warningNote}\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})` : "";
-                summary += `- + ${fileName}${warningNote}\n`;
-            }
-            summary += "\n";
-        }
-        if (dryRunResult.notFound.length > 0) {
-            summary += `## Not Found in Cloud (${dryRunResult.notFound.length}) - will create new\n`;
-            for (const { file, result } of dryRunResult.notFound) {
-                const fileName = basename(file.filePath);
-                summary += `- ! ${fileName} (ID: ${result.documentId})\n`;
-            }
-            summary += "\n";
-        }
-        if (allInvalid.length > 0) {
-            summary += `## Skipped - Invalid (${allInvalid.length})\n`;
-            for (const { fileName, error } of allInvalid) {
-                summary += `- ✗ ${fileName}: ${error}\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`;
-        }
-        summary += `**To proceed:** Call this tool again with \`confirmed: true\``;
-        return textResponse(summary);
-    }
-    // MCP-PUSH-1.1: When confirmed, execute push
-    try {
-        const result = await executePush(dryRunResult, credentials);
-        // SYNC-FAIL-1: a failed push must not masquerade as success
-        const failed = result.errors.length;
-        const syncedCount = result.created + result.updated;
-        let output;
-        if (failed > 0 && syncedCount === 0) {
-            output = `✗ Push failed: ${failed} document(s) could not be saved.\n\n`;
-        }
-        else if (failed > 0) {
-            output = `⚠ Push incomplete: ${syncedCount} document(s) synced, ${failed} failed.\n\n`;
-        }
-        else {
-            output = "✓ Push complete!\n\n";
-        }
-        if (result.created > 0) {
-            output += `**Created:** ${result.created} document(s)\n`;
-        }
-        if (result.updated > 0) {
-            output += `**Updated:** ${result.updated} document(s)\n`;
-        }
-        // IMPORT-3: marker failures are loud but never fatal
-        if (result.importWarnings.length > 0) {
-            output += `\n**Import marker warnings:**\n`;
-            for (const { fileName, status } of result.importWarnings) {
-                output +=
-                    status === "already_used"
-                        ? `- ⚠ ${fileName}: import marker already used — requirements were saved as natively authored (they count toward the plan's requirement limit). Re-running codebase-to-spec produces a fresh marker.\n`
-                        : `- ⚠ ${fileName}: import marker not recognized — the CLI may need updating. Requirements were saved as natively authored (they count toward the plan's requirement limit).\n`;
-            }
-        }
-        if (result.errors.length > 0) {
-            output += `\n**Errors:**\n`;
-            for (const { fileName, error } of result.errors) {
-                output += `- ${fileName}: ${error}\n`;
-            }
-        }
-        // SYNC-FAIL-2.1: cloud save succeeded but the local file couldn't be
-        // updated — name the file, say the cloud is fine, and give the recovery
-        // step so a retry doesn't mint a duplicate document.
-        if (result.writeBackWarnings?.length > 0) {
-            output += `\n**Warnings:**\n`;
-            for (const warning of result.writeBackWarnings) {
-                output +=
-                    `- ⚠ ${warning.fileName}: saved to the cloud, but the local file could not be updated (${warning.error}). ` +
-                        `To avoid creating a duplicate, add "id: ${warning.documentId}" under "document:" in the frontmatter of ${warning.filePath}, then push again.\n`;
-            }
-        }
-        // SYNC-LAND-1: where each synced document lives in the web app
-        if (result.synced.length > 0) {
-            output += `\n**Review in dot•requirements:**\n`;
-            for (const doc of result.synced) {
-                output += `- ${doc.fileName}: ${doc.url}\n`;
-            }
-        }
-        // SYNC-FAIL-1.2: total failure is an error result, not a success story
-        if (failed > 0 && syncedCount === 0) {
-            return { content: [{ type: "text", text: output }], isError: true };
-        }
-        return textResponse(output);
-    }
-    catch (error) {
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: `✗ Push failed:\n\n${error instanceof Error ? error.message : String(error)}`,
-                },
-            ],
-            isError: true,
-        };
-    }
-}
-//# sourceMappingURL=push.js.map

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

@@ -1,16 +0,0 @@
-/**
- * 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

@@ -1,134 +0,0 @@
-/**
- * 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

@@ -1,52 +0,0 @@
-/**
- * Review handlers for MCP tools
- *
- * Provides AI-powered review functionality:
- * - 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";
-/**
- * Arguments for style_check tool
- */
-export interface StyleCheckArgs {
-    filePath: string;
-    requirementKeys?: string[];
-    model?: string;
-    projectId?: string;
-}
-/**
- * Arguments for review_test tool
- */
-export interface ReviewTestArgs {
-    testFilePath: string;
-    projectId?: string;
-}
-/**
- * Handler for style_check tool
- *
- * Requirements covered:
- * - MCP-REVIEW-1.0: For requirements files, feedback identifies clarity issues such as vague language and missing preconditions
- * - MCP-REVIEW-1.1: For test files, feedback identifies incorrect requirement() usage and missing test coverage
- * - MCP-REVIEW-1.2: Feedback is categorized by severity: must fix, should fix, could improve
- * - MCP-REVIEW-1.3: When cloud credentials are unavailable, an error explains how to authenticate
- */
-export declare function handleStyleCheck(args: StyleCheckArgs, context: HandlerContext, options?: {
-    useEnvAuth?: boolean;
-    apiBaseUrl?: string;
-}): Promise<ToolResponse>;
-/**
- * Handler for review_test tool
- *
- * Requirements covered:
- * - MCP-REVIEW-2.0: Feedback identifies test setup that does not match requirement preconditions
- * - MCP-REVIEW-2.1: Feedback identifies test actions that do not match requirement triggers
- * - MCP-REVIEW-2.2: Feedback identifies test assertions that do not match requirement outcomes
- * - MCP-REVIEW-2.3: Feedback identifies requirements without test coverage
- * - MCP-REVIEW-2.4: Feedback identifies tests that reference non-existent requirements
- * - MCP-REVIEW-2.5: When cloud credentials are unavailable, an error explains how to authenticate
- */
-export declare function handleReviewTest(args: ReviewTestArgs, context: HandlerContext, options?: {
-    apiBaseUrl?: string;
-}): Promise<ToolResponse>;
-//# sourceMappingURL=review.d.ts.map

package/dist/mcp/handlers/review.js

@@ -1,243 +0,0 @@
-/**
- * Review handlers for MCP tools
- *
- * Provides AI-powered review functionality:
- * - style_check: Check requirements or test files for style issues
- * - review_test: Comprehensively review tests for semantic correctness
- */
-import { existsSync, readFileSync } from "node:fs";
-import { dirname } from "node:path";
-import { fetchReviewTestFeedback, fetchStyleCheckFeedback, } from "../../requirements/cloud-ai.js";
-import { findRequirementsInFile } from "../../requirements/grep.js";
-import { filterRequirementsByKeys, formatRequirementTree, getRequirementTree, } from "../../requirements/index.js";
-import { discoverProjects, } from "../../utils/project-discovery.js";
-import { errorResponse, resolveProjectFilePath, textResponse, } from "./types.js";
-const STYLE_CHECK_GUIDANCE = `
-
----
-
-## How to use this feedback
-
-1. **This check is stateless.** It is a "new set of eyes" from any previous feedback rounds — it has no memory of prior suggestions or changes you've already made.
-
-2. **Act on feedback from the posture of "what will make the requirements better."** The severity categories (MUST FIX / SHOULD FIX / COULD IMPROVE) are prioritized for convenience, but unless the user has explicitly told you to ignore lower-priority feedback, go ahead and make any improvements — minor or otherwise. If you believe acting on a specific piece of feedback would make the requirements *worse*, surface that feedback to the user rather than silently ignoring it. Otherwise, just act.
-
-3. **Next steps:** Always run \`validate_requirements\` again after making updates. If you make significant changes based on this feedback, you can run \`style_check\` again — but be aware that will be a fresh set of eyes, not a continued conversation.`;
-/**
- * Handler for style_check tool
- *
- * Requirements covered:
- * - MCP-REVIEW-1.0: For requirements files, feedback identifies clarity issues such as vague language and missing preconditions
- * - MCP-REVIEW-1.1: For test files, feedback identifies incorrect requirement() usage and missing test coverage
- * - MCP-REVIEW-1.2: Feedback is categorized by severity: must fix, should fix, could improve
- * - MCP-REVIEW-1.3: When cloud credentials are unavailable, an error explains how to authenticate
- */
-export async function handleStyleCheck(args, context, options) {
-    const { filePath, requirementKeys, model, projectId } = args;
-    const { useEnvAuth = false, apiBaseUrl = "https://app.dotrequirements.io" } = options ?? {};
-    // #49: Resolve against the discovered project.path (matching push) with a
-    // workspaceRoot fallback, so a relative path that push accepts also works
-    // here in PROJ_*-based setups. Discovery may throw without credentials — that
-    // is fine, we then fall back to workspaceRoot.
-    let styleProjectPath;
-    try {
-        styleProjectPath = (await context.getProjectFromDiscovery(projectId)).path;
-    }
-    catch {
-        styleProjectPath = undefined;
-    }
-    const fullPath = resolveProjectFilePath(filePath, styleProjectPath, context.workspaceRoot);
-    if (!existsSync(fullPath)) {
-        return errorResponse(`File not found: ${filePath}`);
-    }
-    // Read file contents
-    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 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";
-    // If requirementKeys provided, filter the file to only those requirements
-    let fileContentsToCheck = fileContents;
-    let scopeNote = "";
-    if (requirementKeys && requirementKeys.length > 0) {
-        if (!isRequirementsFile) {
-            return errorResponse(`requirementKeys can only be used with requirements files (*.requirements.md), not test files.`);
-        }
-        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(", ")}`);
-            }
-            fileContentsToCheck = filteredContent;
-            if (missingKeys.length > 0) {
-                scopeNote = `\n\n> **Note:** Some specified keys were not found in the file: ${missingKeys.join(", ")}`;
-            }
-        }
-        catch (error) {
-            return errorResponse(`Failed to filter requirements: ${error instanceof Error ? error.message : String(error)}`);
-        }
-    }
-    // MCP-REVIEW-1.3: Get credentials - walk up from file's directory to find project
-    const fileDir = dirname(fullPath);
-    let project;
-    try {
-        if (projectId) {
-            // #48: PROJ-CONTEXT-6 — an explicit projectId disambiguates a
-            // multi-project workspace. Thread it straight through to discovery
-            // (matching review_test/push/report) instead of the file-directory
-            // auto-detect path, which cannot choose among multiple projects.
-            project = await context.getProjectFromDiscovery(projectId);
-        }
-        else if (useEnvAuth) {
-            // Using env auth - go straight to context
-            project = await context.getProjectFromDiscovery();
-        }
-        else {
-            // Try to find a project starting from the file's directory
-            const result = await discoverProjects(fileDir);
-            if (result.type === "none") {
-                // If no project found from file dir, try from context
-                project = await context.getProjectFromDiscovery();
-            }
-            else if (result.type === "single") {
-                project = result.project;
-            }
-            else {
-                // #48: Multiple projects — can't auto-detect. The remedy is to pass
-                // projectId (not `dotrequirements link`, which is already satisfied);
-                // list the available projects so the caller can choose.
-                const available = result.projects
-                    .map((p) => p.projectId)
-                    .filter(Boolean)
-                    .join(", ");
-                return errorResponse(`Multiple projects found. Specify the \`projectId\` parameter to choose one${available ? ` (available: ${available})` : ""}.`);
-            }
-        }
-    }
-    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 hosted endpoint via the shared library helper
-    try {
-        const feedback = await fetchStyleCheckFeedback({
-            apiBaseUrl,
-            projectId: project.projectId,
-            projectSecret: project.projectSecret,
-            fileContents: fileContentsToCheck,
-            fileType,
-            model,
-        });
-        const scopeLabel = requirementKeys && requirementKeys.length > 0
-            ? ` (${requirementKeys.join(", ")})`
-            : "";
-        // MCP-REVIEW-1.2: Feedback includes severity categories (in the API response)
-        return textResponse(`# Style Check Results for \`${filePath}\`${scopeLabel}\n\n${feedback}${scopeNote}${STYLE_CHECK_GUIDANCE}`);
-    }
-    catch (error) {
-        return errorResponse(`Style check failed: ${error instanceof Error ? error.message : String(error)}`);
-    }
-}
-/**
- * Handler for review_test tool
- *
- * Requirements covered:
- * - MCP-REVIEW-2.0: Feedback identifies test setup that does not match requirement preconditions
- * - MCP-REVIEW-2.1: Feedback identifies test actions that do not match requirement triggers
- * - MCP-REVIEW-2.2: Feedback identifies test assertions that do not match requirement outcomes
- * - MCP-REVIEW-2.3: Feedback identifies requirements without test coverage
- * - MCP-REVIEW-2.4: Feedback identifies tests that reference non-existent requirements
- * - MCP-REVIEW-2.5: When cloud credentials are unavailable, an error explains how to authenticate
- */
-export async function handleReviewTest(args, context, options) {
-    const { testFilePath, projectId } = args;
-    const { apiBaseUrl = "https://app.dotrequirements.io" } = options ?? {};
-    // #49: Resolve against the discovered project.path (matching push) with a
-    // workspaceRoot fallback. Discovery may throw without credentials — fall back
-    // to workspaceRoot; the credential error is reported later.
-    let reviewProjectPath;
-    try {
-        reviewProjectPath = (await context.getProjectFromDiscovery(projectId)).path;
-    }
-    catch {
-        reviewProjectPath = undefined;
-    }
-    const fullPath = resolveProjectFilePath(testFilePath, reviewProjectPath, context.workspaceRoot);
-    if (!existsSync(fullPath)) {
-        return errorResponse(`File not found: ${testFilePath}`);
-    }
-    // Verify it's a test file
-    const isTestFile = /\.(test|spec)\.(js|jsx|ts|tsx)$/.test(testFilePath);
-    if (!isTestFile) {
-        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");
-    // #43: Extract requirement IDs via the shared AST-based extractor so that
-    // multi-arg calls (`requirement("A", "B")`) and options-bearing calls
-    // (`requirement("A", { ... })`) are captured — the old single-string regex
-    // matched neither.
-    const references = await findRequirementsInFile(fullPath);
-    const requirementIds = new Set(references.map((ref) => ref.requirementId));
-    // MCP-REVIEW-2.5: Get project credentials
-    let project;
-    try {
-        project = await context.getProjectFromDiscovery(projectId);
-    }
-    catch (error) {
-        return errorResponse(`Test review requires a configured dotrequirements project.\n\nError: ${error instanceof Error ? error.message : String(error)}`);
-    }
-    // Load all requirements for this project
-    const allRequirements = await context.getRequirements(projectId);
-    // Group requirement IDs by their root to avoid sending duplicate trees
-    // e.g., NAV-3.0, NAV-3.1, NAV-3.2 all belong to root NAV-3
-    //
-    // #42: A reference may be a bare root (`AUTH-1`), an index child (`AUTH-1.0`),
-    // or a label path (`AUTH-1.given`). Normalize each reference to its root id
-    // (the substring before the first dot) before matching — the same approach
-    // getReferencedRequirementIds uses — so label-path refs resolve instead of
-    // being reported as non-existent.
-    const rootToTestedIds = new Map();
-    for (const reqId of requirementIds) {
-        const dotIndex = reqId.indexOf(".");
-        const refRootId = dotIndex > 0 ? reqId.substring(0, dotIndex) : reqId;
-        const req = allRequirements.find((r) => r.rootId === refRootId);
-        if (req) {
-            if (!rootToTestedIds.has(req.rootId)) {
-                rootToTestedIds.set(req.rootId, new Set());
-            }
-            rootToTestedIds.get(req.rootId).add(reqId);
-        }
-    }
-    // Build requirements array with one entry per unique root tree
-    // MCP-REVIEW-2.0, 2.1, 2.2: These validations happen in the API based on the tree structure
-    const requirements = [];
-    for (const [rootId, testedIds] of rootToTestedIds) {
-        const tree = getRequirementTree(allRequirements, rootId);
-        const formattedTree = formatRequirementTree(tree);
-        requirements.push({
-            id: rootId,
-            content: formattedTree,
-            testedIds: Array.from(testedIds).sort(),
-        });
-    }
-    // Call hosted endpoint via the shared library helper
-    // MCP-REVIEW-2.3 & 2.4: API identifies coverage gaps and non-existent references
-    try {
-        const feedback = await fetchReviewTestFeedback({
-            apiBaseUrl,
-            projectId: project.projectId,
-            projectSecret: project.projectSecret,
-            testFileContents,
-            requirements,
-        });
-        return textResponse(`# Test Review Results for \`${testFilePath}\`\n\n${feedback}`);
-    }
-    catch (error) {
-        return errorResponse(`Test review failed: ${error instanceof Error ? error.message : String(error)}`);
-    }
-}
-//# sourceMappingURL=review.js.map