agentic-qe 3.15.4 → 3.15.6

package/dist/coordination/handlers/coverage-runner.js

@@ -0,0 +1,462 @@
+/**
+ * Run a JS/TS project's test runner with coverage — safely (issue #995).
+ *
+ * Properties, each learned the hard way:
+ *
+ *   1. **Run from the project root, write to a private temp dir.** The coverage
+ *      report and the runner's JSON result go to `reportDir`, never into the
+ *      project tree (#995 wrote `src/coverage/`). The result file is set with
+ *      plain `--outputFile=` — vitest ignores `--outputFile.json=` when the
+ *      user's config sets `outputFile` as a string.
+ *   2. **No shell, no download.** The runner's own JS entry point is resolved
+ *      from the project's `node_modules` and started with `process.execPath`.
+ *      Arguments are an array, so `$(…)`, spaces, `& ^ %` in a path are data on
+ *      every platform. A runner that isn't installed is reported, never fetched
+ *      (`npx` would silently download one).
+ *   3. **Kill the whole process tree.** Runner → workers is several levels deep.
+ *      The child is started in its own process group (POSIX) and the group is
+ *      killed on timeout (`taskkill /T /F` on Windows).
+ *   4. **Die with the host.** Every live run is registered; a synchronous
+ *      `process.on('exit')` hook kills each group and removes its temp dir. That
+ *      covers every path that ends in `process.exit()` or a natural exit — the
+ *      MCP server's shutdown on SIGTERM/SIGINT/stdin EOF routes there. It cannot
+ *      cover SIGKILL of the host (or a signal with no handler installed): no
+ *      code runs in that case, and the group is left to its own timeout-less
+ *      fate. Hosts should shut down via a handled signal.
+ */
+import { spawn, spawnSync } from 'child_process';
+import * as fs from 'fs';
+import * as os from 'os';
+import * as path from 'path';
+import { safeJsonParse } from '../../shared/safe-json.js';
+import { loadCoverageReportFile } from './coverage-target.js';
+/**
+ * Outcome of a coverage run's tests (#1027 NB2): 'failed' when the runner's
+ * own result reports a failed test or test suite, 'passed' when it reports
+ * none — whatever the exit code (a coverage-threshold failure exits 1 with
+ * every test passing). Without counts (nyc), the exit code decides.
+ */
+export function testRunStatus(run) {
+    if (typeof run.numFailedTests === 'number' || typeof run.numFailedTestSuites === 'number') {
+        return (run.numFailedTests ?? 0) > 0 || (run.numFailedTestSuites ?? 0) > 0 ? 'failed' : 'passed';
+    }
+    return run.exitedCleanly ? 'passed' : 'failed';
+}
+const RESULTS_FILE = 'test-results.json';
+const STDERR_TAIL_BYTES = 2048;
+const STDERR_HEAD_BYTES = 4096;
+const STDERR_PREFIX_BYTES = 512;
+const ERROR_LINE_RE = /^.*(?:error|\[[A-Z][A-Z_]{2,}\]|ERR_[A-Z_]+).*$/im;
+/**
+ * Summarise a runner's stderr for a caller: the FIRST error-looking line (or
+ * the first ~512 bytes) plus the last ~2 KB (#995 review NB-C). A tail alone
+ * loses the cause when a long stack follows it — a vitest config syntax error
+ * prints `[PARSE_ERROR] …` first and then kilobytes of frames.
+ */
+export function summarizeStderr(head, tail, totalBytes) {
+    if (totalBytes <= STDERR_TAIL_BYTES)
+        return tail.trim();
+    const errorLine = ERROR_LINE_RE.exec(head)?.[0]?.trim();
+    const prefix = (errorLine ?? head.slice(0, STDERR_PREFIX_BYTES)).trim();
+    if (prefix && tail.includes(prefix))
+        return tail.trim();
+    return `${prefix}\n…\n${tail.trim()}`;
+}
+// ============================================================================
+// Locating the runner
+// ============================================================================
+/**
+ * Find an installed package's bin script, walking up `node_modules` from the
+ * project root (hoisted monorepo installs live above the package).
+ */
+export function findPackageBin(projectRoot, pkg, binName = pkg, maxDepth = 10) {
+    let dir = path.resolve(projectRoot);
+    for (let i = 0; i <= maxDepth; i++) {
+        const pkgDir = path.join(dir, 'node_modules', pkg);
+        try {
+            const manifest = JSON.parse(fs.readFileSync(path.join(pkgDir, 'package.json'), 'utf-8'));
+            const rel = typeof manifest.bin === 'string' ? manifest.bin : manifest.bin?.[binName];
+            if (rel) {
+                const binPath = path.join(pkgDir, rel);
+                if (fs.existsSync(binPath))
+                    return binPath;
+            }
+        }
+        catch {
+            // not installed at this level
+        }
+        const parent = path.dirname(dir);
+        if (parent === dir)
+            break;
+        dir = parent;
+    }
+    return null;
+}
+function readDeps(projectRoot) {
+    try {
+        const pkg = safeJsonParse(fs.readFileSync(path.join(projectRoot, 'package.json'), 'utf-8'));
+        return {
+            ...(pkg.devDependencies || {}),
+            ...(pkg.dependencies || {}),
+        };
+    }
+    catch {
+        return {};
+    }
+}
+const RUNNER_DEPS = ['jest', '@jest/core', 'mocha', 'nyc', 'vitest'];
+/**
+ * The project and its ancestors up to the repository / workspace root: the
+ * project itself, then each parent, stopping after the directory holding
+ * `.git` (or a `workspaces` manifest), at $HOME, or after `maxDepth` levels.
+ */
+function projectAndAncestors(projectRoot, maxDepth = 10) {
+    const dirs = [];
+    let home = null;
+    try {
+        home = fs.realpathSync(os.homedir());
+    }
+    catch {
+        home = null;
+    }
+    let dir = path.resolve(projectRoot);
+    for (let i = 0; i <= maxDepth; i++) {
+        if (i > 0 && home !== null && dir === home)
+            break;
+        dirs.push(dir);
+        if (fs.existsSync(path.join(dir, '.git')) || isWorkspaceRoot(dir))
+            break;
+        const parent = path.dirname(dir);
+        if (parent === dir)
+            break;
+        dir = parent;
+    }
+    return dirs;
+}
+function isWorkspaceRoot(dir) {
+    try {
+        const pkg = JSON.parse(fs.readFileSync(path.join(dir, 'package.json'), 'utf-8'));
+        return pkg.workspaces !== undefined;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Dependencies that decide the runner (#1012): the package's own when it
+ * declares a test runner, otherwise those of the nearest ancestor (workspace
+ * root) that does. A package of a monorepo whose jest is declared only at the
+ * root runs jest, not the vitest default.
+ */
+function readRunnerDeps(projectRoot) {
+    for (const dir of projectAndAncestors(projectRoot)) {
+        const deps = readDeps(dir);
+        if (RUNNER_DEPS.some(d => deps[d]))
+            return deps;
+    }
+    return {};
+}
+/**
+ * Every `node_modules/.bin` from the project up to the workspace root,
+ * nearest first. Runners that shell out to sibling tools (nyc → mocha) need
+ * the hoisted ones too (#1012).
+ */
+export function runnerBinDirs(projectRoot) {
+    return projectAndAncestors(projectRoot)
+        .map(dir => path.join(dir, 'node_modules', '.bin'))
+        .filter(dir => fs.existsSync(dir));
+}
+/** Is the project installed with Yarn Plug'n'Play (no node_modules to resolve from)? */
+function usesPlugNPlay(projectRoot) {
+    return projectAndAncestors(projectRoot).some(dir => fs.existsSync(path.join(dir, '.pnp.cjs')) || fs.existsSync(path.join(dir, '.pnp.js')));
+}
+/**
+ * Pick the runner and its argv. No shell is involved: `command` is the Node
+ * binary and `args[0]` the runner's JS entry, so paths are passed verbatim.
+ */
+export function buildJsCoverageInvocation(projectRoot, reportDir) {
+    const resultsFile = path.join(reportDir, RESULTS_FILE);
+    const deps = readRunnerDeps(projectRoot);
+    let runner = 'vitest';
+    let runnerArgs;
+    let results = resultsFile;
+    if (deps['jest'] || deps['@jest/core']) {
+        runner = 'jest';
+        runnerArgs = ['--coverage', `--coverageDirectory=${reportDir}`, '--coverageReporters=json',
+            '--json', `--outputFile=${resultsFile}`];
+    }
+    else if (deps['mocha'] || deps['nyc']) {
+        runner = 'nyc';
+        runnerArgs = [`--report-dir=${reportDir}`, `--temp-dir=${path.join(reportDir, '.nyc_output')}`,
+            '--reporter=json', 'mocha'];
+        results = null;
+    }
+    else {
+        // vitest is the default — covers vitest, @vitest/coverage-v8, etc.
+        // Plain `--outputFile=` (not `.json=`): it overrides a string, object, or
+        // absent `outputFile` in the user's config alike.
+        // #1027 N4: vitest writes no coverage when a test fails unless asked;
+        // coverage from a failing suite is still a measurement (runStatus 'failed').
+        runnerArgs = ['run', '--coverage', `--coverage.reportsDirectory=${reportDir}`,
+            '--coverage.reporter=json', '--coverage.reportOnFailure=true', '--reporter=json', `--outputFile=${resultsFile}`];
+    }
+    const bin = findPackageBin(projectRoot, runner === 'jest' ? 'jest' : runner);
+    return {
+        runner,
+        command: bin ? process.execPath : null,
+        args: bin ? [bin, ...runnerArgs] : runnerArgs,
+        resultsFile: results,
+    };
+}
+const NO_COUNTS = {
+    numTotalTests: null, numPassedTests: null, numFailedTests: null, numPendingTests: null, numFailedTestSuites: null,
+};
+/** Test counts from the runner's own JSON result (vitest/jest share the shape). */
+function readRunCounts(resultsFile) {
+    if (!resultsFile)
+        return NO_COUNTS;
+    try {
+        const parsed = JSON.parse(fs.readFileSync(resultsFile, 'utf-8'));
+        const n = (k) => (typeof parsed[k] === 'number' ? parsed[k] : null);
+        return {
+            numTotalTests: n('numTotalTests'), numPassedTests: n('numPassedTests'), numFailedTests: n('numFailedTests'),
+            numPendingTests: n('numPendingTests'), numFailedTestSuites: n('numFailedTestSuites'),
+        };
+    }
+    catch {
+        return NO_COUNTS;
+    }
+}
+/**
+ * Read the istanbul report the runner wrote. Distinguishes an EMPTY report
+ * (`{}` — the run instrumented nothing) from NO report.
+ */
+async function readRunReport(file) {
+    const data = await loadCoverageReportFile(file);
+    if (data)
+        return data;
+    try {
+        const parsed = JSON.parse(fs.readFileSync(file, 'utf-8'));
+        if (parsed && typeof parsed === 'object' && !Array.isArray(parsed) && Object.keys(parsed).length === 0) {
+            return { files: [], summary: { line: 0, branch: 0, function: 0, statement: 0, files: 0 } };
+        }
+    }
+    catch {
+        // absent or unparseable
+    }
+    return null;
+}
+// ============================================================================
+// Process-tree lifecycle
+// ============================================================================
+/** Live runs: process-group leader pid → temp dir to remove. */
+const liveRuns = new Map();
+let exitHookInstalled = false;
+function killTree(pid, signal) {
+    if (process.platform === 'win32') {
+        // Windows has no process groups; taskkill /T walks the tree.
+        try {
+            spawnSync('taskkill', ['/pid', String(pid), '/T', '/F'], { stdio: 'ignore' });
+        }
+        catch { /* gone */ }
+        return;
+    }
+    try {
+        process.kill(-pid, signal);
+    }
+    catch { /* group already gone */ }
+}
+/**
+ * Synchronously kill every live coverage run and remove its temp dir. Runs from
+ * `process.on('exit')`, where only synchronous work is possible.
+ */
+export function killAllLiveRunsSync() {
+    for (const [pid, dir] of liveRuns) {
+        killTree(pid, 'SIGKILL');
+        if (dir) {
+            try {
+                fs.rmSync(dir, { recursive: true, force: true });
+            }
+            catch { /* best-effort */ }
+        }
+    }
+    liveRuns.clear();
+}
+/** Number of runs currently registered (for tests and diagnostics). */
+export function liveRunCount() {
+    return liveRuns.size;
+}
+function ensureExitHook() {
+    if (exitHookInstalled)
+        return;
+    process.on('exit', killAllLiveRunsSync);
+    exitHookInstalled = true;
+}
+/** Is any process of this group still alive? */
+function groupAlive(pgid) {
+    try {
+        process.kill(-pgid, 0);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
+async function waitForGroupExit(pid, maxMs) {
+    if (process.platform === 'win32')
+        return;
+    const deadline = Date.now() + maxMs;
+    while (groupAlive(pid) && Date.now() < deadline) {
+        await new Promise(r => setTimeout(r, 50));
+    }
+}
+/**
+ * Spawn `command args` in `cwd` (no shell) as the leader of a new process
+ * group, wait for it, and on timeout kill the entire group. Resolves only once
+ * no process of the group remains (bounded), so the caller can safely delete
+ * its temp dir. While running, the group and `cleanupDir` are registered with
+ * the exit hook so they never outlive the host process.
+ */
+export async function runProcessTree(command, args, options) {
+    const graceMs = options.graceMs ?? 3000;
+    const isWindows = process.platform === 'win32';
+    ensureExitHook();
+    return new Promise(resolve => {
+        let timedOut = false;
+        let settled = false;
+        let stderrTail = '';
+        let stderrHead = '';
+        let stderrBytes = 0;
+        const child = spawn(command, args, {
+            cwd: options.cwd,
+            env: options.env ?? process.env,
+            // Own process group (POSIX) so the whole tree can be signalled at once.
+            detached: !isWindows,
+            shell: false,
+            stdio: ['ignore', 'ignore', 'pipe'],
+            windowsHide: true,
+        });
+        if (child.pid !== undefined)
+            liveRuns.set(child.pid, options.cleanupDir);
+        child.stderr?.on('data', (chunk) => {
+            const text = chunk.toString('utf-8');
+            stderrBytes += chunk.length;
+            if (stderrHead.length < STDERR_HEAD_BYTES)
+                stderrHead = (stderrHead + text).slice(0, STDERR_HEAD_BYTES);
+            stderrTail = (stderrTail + text).slice(-STDERR_TAIL_BYTES);
+        });
+        // A grandchild holding the pipe open must not keep the host alive.
+        child.stderr?.on('error', () => { });
+        const finish = async (exitCode, spawnError) => {
+            if (settled)
+                return;
+            settled = true;
+            clearTimeout(timer);
+            if (child.pid !== undefined && !isWindows && groupAlive(child.pid)) {
+                // Anything left in the group after the runner exited is orphaned work
+                // (stray workers, a hung watcher). Sweep it before the caller cleans up.
+                killTree(child.pid, 'SIGTERM');
+                await waitForGroupExit(child.pid, graceMs);
+                if (groupAlive(child.pid))
+                    killTree(child.pid, 'SIGKILL');
+                await waitForGroupExit(child.pid, graceMs);
+            }
+            child.stderr?.destroy();
+            if (child.pid !== undefined)
+                liveRuns.delete(child.pid);
+            resolve({
+                exitCode, timedOut, spawnError, pid: child.pid,
+                stderrTail: summarizeStderr(stderrHead, stderrTail, stderrBytes),
+            });
+        };
+        const timer = setTimeout(() => {
+            timedOut = true;
+            if (child.pid === undefined)
+                return;
+            killTree(child.pid, 'SIGTERM');
+            setTimeout(() => { if (!settled && child.pid !== undefined)
+                killTree(child.pid, 'SIGKILL'); }, graceMs).unref();
+        }, options.timeoutMs);
+        child.on('error', () => { void finish(null, true); });
+        child.on('exit', code => { void finish(code, false); });
+    });
+}
+/**
+ * Run the project's test runner with coverage from `projectRoot`.
+ *
+ * Everything the runner writes goes to a fresh temp dir that is removed before
+ * this returns — after the whole process group is gone — or by the exit hook if
+ * the host exits first.
+ */
+export async function runJsCoverage(projectRoot, options = {}) {
+    const timeoutMs = options.timeoutMs ?? 120000;
+    const reportDir = fs.mkdtempSync(path.join(os.tmpdir(), 'aqe-js-cov-'));
+    try {
+        const inv = buildJsCoverageInvocation(projectRoot, reportDir);
+        if (!inv.command) {
+            return {
+                runner: inv.runner, missing: true, exitCode: null, exitedCleanly: false, timedOut: false,
+                timeoutMs, data: null, numTotalTests: null, stderrTail: '',
+            };
+        }
+        // Runners that shell out to sibling tools (nyc → mocha) find them here,
+        // including hoisted workspace-root installs (#1012).
+        const binDirs = runnerBinDirs(projectRoot);
+        // The runner's own temp files (vite's SSR cache, etc.) go inside reportDir
+        // too, so a killed run leaves nothing behind in the system temp dir.
+        const runnerTmp = path.join(reportDir, 'tmp');
+        fs.mkdirSync(runnerTmp);
+        const env = {
+            ...process.env,
+            PATH: [...binDirs, process.env.PATH ?? ''].join(path.delimiter),
+            TMPDIR: runnerTmp, TMP: runnerTmp, TEMP: runnerTmp,
+        };
+        const run = await runProcessTree(inv.command, inv.args, {
+            cwd: projectRoot, timeoutMs, cleanupDir: reportDir, env,
+        });
+        const data = run.timedOut ? null : await readRunReport(path.join(reportDir, 'coverage-final.json'));
+        return {
+            runner: inv.runner,
+            missing: false,
+            exitCode: run.exitCode,
+            exitedCleanly: run.exitCode === 0 && !run.timedOut,
+            timedOut: run.timedOut,
+            timeoutMs,
+            data,
+            ...(run.timedOut ? NO_COUNTS : readRunCounts(inv.resultsFile)),
+            stderrTail: run.stderrTail,
+        };
+    }
+    finally {
+        try {
+            fs.rmSync(reportDir, { recursive: true, force: true });
+        }
+        catch { /* best-effort */ }
+    }
+}
+/**
+ * One-line, caller-facing explanation of why a run produced no usable report.
+ * Never "No instrumentation ran" when the runner did run and failed.
+ */
+export function describeRunFailure(run, projectRoot) {
+    if (run.missing && usesPlugNPlay(projectRoot)) {
+        return `${run.runner} could not be resolved for ${projectRoot}: this project uses Yarn Plug'n'Play, ` +
+            'and agentic-qe resolves test runners only from node_modules. Nothing was run and nothing was downloaded. ' +
+            `Generate a coverage report (e.g. \`yarn ${run.runner} --coverage\`) and pass it as the target, ` +
+            'or use the node-modules linker.';
+    }
+    if (run.missing) {
+        return `${run.runner} is not installed in ${projectRoot} (no node_modules/${run.runner} found); ` +
+            'nothing was run and nothing was downloaded. Install it, or generate a coverage report and pass it as the target.';
+    }
+    if (run.timedOut) {
+        return `The coverage run timed out after ${Math.round(run.timeoutMs / 1000)}s and was stopped.`;
+    }
+    if (run.exitCode !== 0) {
+        const tail = run.stderrTail ? `: ${run.stderrTail}` : '';
+        return `Test runner (${run.runner}) exited with code ${run.exitCode ?? 'unknown'} without a coverage report${tail}`;
+    }
+    return `Test runner (${run.runner}) exited cleanly but wrote no coverage report. ` +
+        'Ensure a coverage provider is configured (e.g., @vitest/coverage-v8, istanbul).';
+}
+//# sourceMappingURL=coverage-runner.js.map

package/dist/coordination/handlers/coverage-target.d.ts

@@ -0,0 +1,145 @@
+/**
+ * Coverage target resolution (issue #995).
+ *
+ * A coverage target can be the project root, a subfolder of it, a single source
+ * file, or an existing coverage report. Before #995 every target was treated as
+ * if it were the project root: `target: "src/"` ran vitest FROM `src/` (which
+ * found no tests, reported 0%, and wrote `src/coverage/` into the source tree),
+ * and `target: "coverage/lcov.info"` was searched for a `coverage/` folder
+ * inside a file and came back empty.
+ *
+ * This module answers the three questions every collector needs first:
+ *   1. Which project does the target belong to?  (`findCoverageProjectRoot`)
+ *   2. Is the target itself a coverage report?   (`isCoverageReportFile` /
+ *      `loadCoverageReportFile`)
+ *   3. Which entries of a project-wide report belong to the target?
+ *      (`scopeCoverageToTarget`)
+ *
+ * Kept free of task-executor and child-process imports so both the MCP handler
+ * and the CLI `coverage` command can share it.
+ */
+import type { CoverageData, FileCoverage } from '../../domains/coverage-analysis/interfaces';
+/**
+ * Walk up from a target to the nearest directory that holds a project marker.
+ *
+ * NEAREST rather than topmost: in a monorepo, `packages/core/src` belongs to
+ * `packages/core` (its own package.json and test config), not the workspace
+ * root. This intentionally differs from `kernel/findProjectRoot`, which locates
+ * the AQE data root (honouring AQE_PROJECT_ROOT and `.agentic-qe`).
+ *
+ * Returns `null` when the target does not exist (a typo must not silently
+ * resolve to — and run tests for — an ancestor project) or when no marker is
+ * found below the home directory / filesystem root within `maxDepth` levels.
+ */
+export declare function findCoverageProjectRoot(target: string, maxDepth?: number): string | null;
+/** Is this path (by name) a coverage report rather than source code? */
+export declare function isCoverageReportFile(filePath: string): boolean;
+/**
+ * Parse a coverage report file of any supported format.
+ *
+ * Returns `null` — never an empty "0%" report — when the file can't be read,
+ * can't be parsed, or lists no files.
+ */
+export declare function loadCoverageReportFile(filePath: string): Promise<CoverageData | null>;
+/**
+ * Keep only the report entries that lie under `target`.
+ *
+ * Report paths come in two shapes: absolute (istanbul `coverage-final.json`)
+ * and project-root-relative (vitest/jest `lcov.info` writes `SF:src/math.ts`).
+ * Relative entries are resolved against `root` before comparing. Entry paths are
+ * returned unchanged.
+ */
+export declare function scopeCoverageToTarget(data: CoverageData, root: string, target: string): FileCoverage[];
+/** Why there are no coverage figures for a target. */
+export type NoDataReason = 
+/** The target path does not exist. */
+'target-missing'
+/** A coverage report was given but cannot be parsed (malformed, raw V8, empty). */
+ | 'unreadable-report'
+/** The report lists only test files. */
+ | 'test-only-report'
+/** The report has no entries under the target — it did not measure it. */
+ | 'target-not-covered'
+/** The test runner itself reports running zero tests. */
+ | 'no-tests-ran';
+export type CoverageEvaluation = {
+    status: 'measured';
+    files: FileCoverage[];
+} | {
+    status: 'no-data';
+    reason: NoDataReason;
+};
+/**
+ * Decide what a report says about `target`. Shared by the MCP handler and the
+ * CLI so the two cannot disagree (#995 review B2).
+ *
+ * "Unmeasured" is decided ONLY from evidence that nothing was measured:
+ *   - the runner's own result says it ran 0 tests (`numTotalTests === 0`), or
+ *   - the report has no entry under the target.
+ * Hit counts are never used: a file listed with 0 hits is a measured 0%,
+ * whether the report came from our run or from the user (#995 review B3).
+ */
+export declare function evaluateCoverageForTarget(report: CoverageData, projectRoot: string, target: string, options?: {
+    numTotalTests?: number | null;
+}): CoverageEvaluation;
+export type ExistingCoverage = {
+    status: 'target-missing';
+    target: string;
+} | {
+    status: 'no-report';
+    target: string;
+    projectRoot: string;
+} | {
+    status: 'evaluated';
+    target: string;
+    projectRoot: string;
+    source: 'report-file' | 'project-report';
+    evaluation: CoverageEvaluation;
+};
+/**
+ * Resolve a target to coverage that already exists on disk, without running
+ * anything:
+ *   - a coverage report passed as the target is parsed and analysed whole;
+ *   - otherwise the PROJECT ROOT's report (not the cwd's, not the subfolder's)
+ *     is scoped to the target;
+ *   - `no-report` tells the caller it may run tooling from `projectRoot`.
+ */
+export declare function resolveExistingCoverage(targetPath: string): Promise<ExistingCoverage>;
+/**
+ * Threshold verdict over the metrics that were actually collected. Shared by
+ * MCP and CLI (#995 review NB5): an uncollected metric (no branch data in the
+ * report) is null, and a null must not drag the average down as a fake 0.
+ */
+export declare function thresholdVerdict(summary: {
+    line: number;
+    branch: number;
+    function: number;
+    statement: number;
+}, threshold: number, collected: {
+    branch: boolean;
+    function: boolean;
+    statement?: boolean;
+}): {
+    overall: number;
+    meetsThreshold: boolean;
+};
+/**
+ * Make the analyzer's recommendations agree with `thresholdVerdict`
+ * (#995 review NB-A). The analyzer computes its "Coverage is X% below
+ * threshold" line from a 4-metric average that counts an uncollected metric as
+ * 0 — so a line-only report at 90% got "Met" next to "35.0% below threshold".
+ * Here the shortfall line is recomputed from the collected-metrics verdict,
+ * and advice about a metric that was never collected is dropped.
+ */
+export declare function reconcileRecommendations(recommendations: string[], verdict: {
+    overall: number;
+    meetsThreshold: boolean;
+}, threshold: number, collected: {
+    branch: boolean;
+    function: boolean;
+}): string[];
+/** Is this verdict "no measurement exists" (as opposed to an error or test-only report)? */
+export declare function isUnmeasured(reason: NoDataReason): boolean;
+/** Caller-facing explanation of a no-data verdict. Shared by MCP and CLI. */
+export declare function describeNoData(reason: NoDataReason, target: string, projectRoot: string | null): string;
+//# sourceMappingURL=coverage-target.d.ts.map