@popoverai/dotrequirements 0.26.2 → 0.27.1

package/dist/mcp/handlers/test-mapping.js

@@ -1,168 +0,0 @@
-/**
- * get_requirements_by_test and get_tests_by_requirement handlers
- *
- * Maps between test files and requirements in both directions.
- */
-import { existsSync } from "node:fs";
-import { relative, resolve } from "node:path";
-import { findAllTestReferences, findRequirementsInFile, } from "../../requirements/grep.js";
-import { getRequirementById } from "../../requirements/index.js";
-import { errorResponse, resolveProjectFilePath, textResponse, } from "./types.js";
-/**
- * Handler for get_requirements_by_test tool
- *
- * Requirements covered:
- * - MCP-MAP-1.0: The response lists each requirement() call found in the test file
- * - MCP-MAP-1.1: For each reference, the requirement content and line number are shown
- * - MCP-MAP-1.2: When the test file does not exist, an error is returned
- */
-export async function handleGetRequirementsByTest(args, context) {
-    const { testFile, projectId } = args;
-    // #49: Resolve against the discovered project.path (matching push) with a
-    // workspaceRoot fallback, so a relative path works in PROJ_*-based setups.
-    let projectPath;
-    try {
-        projectPath = (await context.getProjectFromDiscovery(projectId)).path;
-    }
-    catch {
-        projectPath = undefined;
-    }
-    // MCP-MAP-1.2: Check if file exists before processing
-    const fullPath = resolveProjectFilePath(testFile, projectPath, context.workspaceRoot);
-    if (!existsSync(fullPath)) {
-        return errorResponse(`Test file not found: ${testFile}`);
-    }
-    const requirements = await context.getRequirements(projectId);
-    // MCP-MAP-1.0: Find all requirement references in the test file.
-    // #44: Pass the already-resolved absolute path — findRequirementsInFile
-    // resolves relative paths against process.cwd(), which is not the workspace
-    // root when the MCP server runs from a different directory.
-    const refs = await findRequirementsInFile(fullPath);
-    if (refs.length === 0) {
-        return textResponse(`No requirement references found in "${testFile}"`);
-    }
-    // MCP-MAP-1.1: For each reference, get the requirement content and show line number
-    const entries = refs.map((ref) => {
-        const req = getRequirementById(requirements, ref.requirementId);
-        const reqContent = req
-            ? `**${req.label}:** ${req.content}`
-            : `(requirement not found)`;
-        return `**Line ${ref.line}:** \`${ref.requirementId}\` - ${reqContent}`;
-    });
-    return textResponse(`Requirements referenced in "${testFile}":\n\n${entries.join("\n\n")}`);
-}
-/**
- * Handler for get_tests_by_requirement tool
- *
- * Requirements covered:
- * - MCP-MAP-2.0: The response lists which requirements have test references
- * - MCP-MAP-2.1: For each covered requirement, the test file and line number are shown
- * - MCP-MAP-2.2: Requirements without test references are listed separately
- */
-export async function handleGetTestsByRequirement(args, context) {
-    const { requirementsFile, projectId } = args;
-    const { workspaceRoot } = context;
-    // #49: Resolve against the discovered project.path (matching push) with a
-    // workspaceRoot fallback, so a relative path works in PROJ_*-based setups.
-    // The same base root anchors sourceFile comparison, test scanning, and the
-    // relative paths in the output.
-    let projectPath;
-    try {
-        projectPath = (await context.getProjectFromDiscovery(projectId)).path;
-    }
-    catch {
-        projectPath = undefined;
-    }
-    const fullPath = resolveProjectFilePath(requirementsFile, projectPath, workspaceRoot);
-    // Verify file exists
-    if (!existsSync(fullPath)) {
-        return errorResponse(`Requirements file not found: ${requirementsFile}`);
-    }
-    // Verify it's a requirements file
-    if (!requirementsFile.endsWith(".requirements.md")) {
-        return errorResponse(`File must be a requirements file: *.requirements.md`);
-    }
-    // Anchor all subsequent path work to the root that actually contains the
-    // requirements file, so sourceFile comparison and test discovery agree.
-    const baseRoot = projectPath ?? workspaceRoot;
-    // Load all requirements from this file
-    const requirements = await context.getRequirements(projectId);
-    const fileRequirements = requirements.filter((req) => resolve(baseRoot, req.sourceFile) === fullPath);
-    if (fileRequirements.length === 0) {
-        return textResponse(`No requirements found in "${requirementsFile}"`);
-    }
-    // Get unique root requirement IDs from this file
-    const rootIds = new Set(fileRequirements.map((req) => req.rootId));
-    // Find all test references in the workspace
-    const allTestRefs = await findAllTestReferences(baseRoot);
-    // Build coverage map keyed by ROOT id. A reference may be a bare root
-    // (`AUTH-1`), an index child (`AUTH-1.0`), or a label path (`AUTH-1.given`).
-    // #47: Normalize each reference to its root id (the substring before the
-    // first dot) before keying — the same normalization as the tests-for fix —
-    // so label-path refs match numeric requirement ids instead of rendering as
-    // "(requirement not found)".
-    const coverageMap = new Map();
-    for (const ref of allTestRefs) {
-        const dotIndex = ref.requirementId.indexOf(".");
-        const refRootId = dotIndex > 0
-            ? ref.requirementId.substring(0, dotIndex)
-            : ref.requirementId;
-        if (!coverageMap.has(refRootId)) {
-            coverageMap.set(refRootId, []);
-        }
-        coverageMap.get(refRootId).push(ref);
-    }
-    // MCP-MAP-2.0: Categorize requirements as covered or not covered
-    const covered = [];
-    const notCovered = [];
-    for (const rootId of rootIds) {
-        // Check if this requirement ID or any of its children are referenced
-        const reqAndChildren = fileRequirements.filter((r) => r.rootId === rootId);
-        const allIds = reqAndChildren.map((r) => r.id);
-        const testsForThisReq = [];
-        for (const id of allIds) {
-            const refs = coverageMap.get(id);
-            if (refs) {
-                testsForThisReq.push(...refs);
-            }
-        }
-        if (testsForThisReq.length > 0) {
-            covered.push({ id: rootId, tests: testsForThisReq });
-        }
-        else {
-            notCovered.push(rootId);
-        }
-    }
-    // Format output
-    let output = `# Test Coverage for \`${requirementsFile}\`\n\n`;
-    // MCP-MAP-2.1: For each covered requirement, show test file and line number
-    if (covered.length > 0) {
-        output += `## Covered (${covered.length})\n\n`;
-        for (const { id, tests } of covered) {
-            // Group tests by file
-            const testsByFile = new Map();
-            for (const test of tests) {
-                if (!testsByFile.has(test.file)) {
-                    testsByFile.set(test.file, []);
-                }
-                testsByFile.get(test.file).push(test.line);
-            }
-            output += `- **${id}**: ${tests.length} test${tests.length === 1 ? "" : "s"}\n`;
-            for (const [file, lines] of testsByFile.entries()) {
-                const relPath = relative(baseRoot, file);
-                const lineList = lines.sort((a, b) => a - b).join(", ");
-                output += `  - \`${relPath}\` (lines ${lineList})\n`;
-            }
-        }
-        output += "\n";
-    }
-    // MCP-MAP-2.2: Requirements without test references are listed separately
-    if (notCovered.length > 0) {
-        output += `## Not Covered (${notCovered.length})\n\n`;
-        for (const id of notCovered) {
-            output += `- **${id}**: No tests found\n`;
-        }
-    }
-    return textResponse(output);
-}
-//# sourceMappingURL=test-mapping.js.map

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

@@ -1,89 +0,0 @@
-/**
- * Handler types for MCP tool handlers
- *
- * Each handler is a pure function that takes arguments and context,
- * and returns an MCP tool response. This enables unit testing without
- * the full MCP server infrastructure.
- */
-import type { FlattenedRequirement } from "../../requirements/index.js";
-import type { DotreqProject } from "../../utils/project-discovery.js";
-/**
- * MCP tool response content item
- */
-export interface ToolContentItem {
-    type: "text";
-    text: string;
-}
-/**
- * MCP tool response
- *
- * Compatible with MCP SDK's CallToolResult type.
- * The index signature allows for SDK extensibility (e.g., _meta, task).
- */
-export interface ToolResponse {
-    [key: string]: unknown;
-    content: ToolContentItem[];
-    isError?: boolean;
-}
-/**
- * Context provided to all handlers
- *
- * This abstraction enables:
- * - Unit testing with mock context
- * - Dependency injection
- * - Isolation from global state
- */
-export interface HandlerContext {
-    /**
-     * Get flattened requirements for a project
-     * @param projectId - Optional project ID for multi-project scenarios
-     */
-    getRequirements: (projectId?: string) => Promise<FlattenedRequirement[]>;
-    /**
-     * Get project from discovery (path, credentials)
-     * @param projectId - Optional project ID for multi-project scenarios
-     */
-    getProjectFromDiscovery: (projectId?: string) => Promise<DotreqProject>;
-    /**
-     * Workspace root directory
-     */
-    workspaceRoot: string;
-    /**
-     * Project paths from environment variables (PROJ_* pattern)
-     */
-    projectPaths: Map<string, string>;
-    /**
-     * Environment variables (for debug_mcp_environment)
-     */
-    env: NodeJS.ProcessEnv;
-}
-/**
- * Handler function signature
- *
- * All MCP tool handlers implement this interface.
- * The args parameter contains tool-specific arguments.
- */
-export type ToolHandler<TArgs = Record<string, unknown>> = (args: TArgs, context: HandlerContext) => Promise<ToolResponse>;
-/**
- * Resolve a user-supplied relative file path to an absolute path.
- *
- * #49: The MCP server's workspaceRoot is its process cwd, but in PROJ_*-based
- * (Antigravity) setups the discovered project.path is the real project root and
- * differs from cwd. push_requirements resolves against project.path; the other
- * file-taking handlers historically resolved against workspaceRoot, so the same
- * relative path worked for push but not for validate/style/review/mapping.
- *
- * This resolves against project.path first (matching push) and falls back to
- * workspaceRoot when the file is not present there, so both layouts work.
- * Absolute paths are returned unchanged.
- */
-export declare function resolveProjectFilePath(filePath: string, projectPath: string | undefined, workspaceRoot: string): string;
-/**
- * Helper to create a text response
- */
-export declare function textResponse(text: string): ToolResponse;
-/**
- * Helper to create an error response
- */
-export declare function errorResponse(message: string): ToolResponse;
-//# sourceMappingURL=types.d.ts.map

package/dist/mcp/handlers/types.js

@@ -1,52 +0,0 @@
-/**
- * Handler types for MCP tool handlers
- *
- * Each handler is a pure function that takes arguments and context,
- * and returns an MCP tool response. This enables unit testing without
- * the full MCP server infrastructure.
- */
-import { existsSync } from "node:fs";
-import { isAbsolute, resolve } from "node:path";
-/**
- * Resolve a user-supplied relative file path to an absolute path.
- *
- * #49: The MCP server's workspaceRoot is its process cwd, but in PROJ_*-based
- * (Antigravity) setups the discovered project.path is the real project root and
- * differs from cwd. push_requirements resolves against project.path; the other
- * file-taking handlers historically resolved against workspaceRoot, so the same
- * relative path worked for push but not for validate/style/review/mapping.
- *
- * This resolves against project.path first (matching push) and falls back to
- * workspaceRoot when the file is not present there, so both layouts work.
- * Absolute paths are returned unchanged.
- */
-export function resolveProjectFilePath(filePath, projectPath, workspaceRoot) {
-    if (isAbsolute(filePath)) {
-        return filePath;
-    }
-    if (projectPath) {
-        const fromProject = resolve(projectPath, filePath);
-        if (existsSync(fromProject)) {
-            return fromProject;
-        }
-    }
-    return resolve(workspaceRoot, filePath);
-}
-/**
- * Helper to create a text response
- */
-export function textResponse(text) {
-    return {
-        content: [{ type: "text", text }],
-    };
-}
-/**
- * Helper to create an error response
- */
-export function errorResponse(message) {
-    return {
-        content: [{ type: "text", text: message }],
-        isError: true,
-    };
-}
-//# sourceMappingURL=types.js.map

package/dist/mcp/index.d.ts

@@ -1,45 +0,0 @@
-#!/usr/bin/env node
-import { Server } from "@modelcontextprotocol/sdk/server/index.js";
-export declare function invalidateCache(): void;
-export declare const server: Server<{
-    method: string;
-    params?: {
-        [x: string]: unknown;
-        task?: {
-            [x: string]: unknown;
-            ttl?: number | null | undefined;
-            pollInterval?: number | undefined;
-        } | undefined;
-        _meta?: {
-            [x: string]: unknown;
-            progressToken?: string | number | undefined;
-            "io.modelcontextprotocol/related-task"?: {
-                [x: string]: unknown;
-                taskId: string;
-            } | undefined;
-        } | undefined;
-    } | undefined;
-}, {
-    method: string;
-    params?: {
-        [x: string]: unknown;
-        _meta?: {
-            [x: string]: unknown;
-            "io.modelcontextprotocol/related-task"?: {
-                [x: string]: unknown;
-                taskId: string;
-            } | undefined;
-        } | undefined;
-    } | undefined;
-}, {
-    [x: string]: unknown;
-    _meta?: {
-        [x: string]: unknown;
-        "io.modelcontextprotocol/related-task"?: {
-            [x: string]: unknown;
-            taskId: string;
-        } | undefined;
-    } | undefined;
-}>;
-export declare function main(): Promise<void>;
-//# sourceMappingURL=index.d.ts.map