@shiyuphan/follow-me 0.5.1 → 0.7.1

package/dist/src/git.js

@@ -1,13 +1,16 @@
 import { execFile } from "node:child_process";
 import { link, mkdir, mkdtemp, readdir, rm } from "node:fs/promises";
 import { join } from "node:path";
-import { isSha, neededBlobs, parseDiff } from "./diff.js";
+import { isRepoPath, isSha, neededBlobs, parseDiff } from "./diff.js";
 export class GitError extends Error {
     name = "GitError";
     args;
-    constructor(args, stderr) {
+    /** git's exit status, or null when it did not run or was killed. */
+    status;
+    constructor(args, stderr, status = null) {
         super(`git ${args.join(" ")}: ${stderr}`);
         this.args = args;
+        this.status = status;
     }
 }
 function git(repo, args, opts = {}) {
@@ -18,8 +21,10 @@ function git(repo, args, opts = {}) {
             maxBuffer: 1 << 30,
             env: { ...process.env, ...opts.env },
         }, (error, stdout, stderr) => {
-            if (error)
-                reject(new GitError(args, stderr.toString("utf8").trim() || error.message));
+            if (error) {
+                const status = typeof error.code === "number" ? error.code : null;
+                reject(new GitError(args, stderr.toString("utf8").trim() || error.message, status));
+            }
             else
                 resolve(stdout);
         });
@@ -97,25 +102,94 @@ export async function defaultBaseRef(repo) {
     }
     throw new Error("neither main nor master exists; pass --base <ref>");
 }
-async function catBlobs(repo, shas) {
-    const blobs = new Map();
-    if (shas.length === 0)
-        return blobs;
-    const out = await git(repo, ["cat-file", "--batch"], { input: `${shas.join("\n")}\n` });
+/** The blob named by each object spec, in order, as latin1; throws when one is not a blob. */
+async function catBatch(repo, specs) {
+    if (specs.length === 0)
+        return [];
+    const input = Buffer.from(`${specs.join("\n")}\n`, "latin1");
+    const out = await git(repo, ["cat-file", "--batch"], { input });
+    const blobs = [];
     let at = 0;
-    for (const sha of shas) {
+    for (const spec of specs) {
         const nl = out.indexOf(0x0a, at);
         const header = out.subarray(at, nl).toString("latin1");
         const m = /^([0-9a-f]+) blob (\d+)$/.exec(header);
-        if (nl === -1 || m === null || m[1] !== sha) {
-            throw new Error(`cat-file returned "${header}" for ${sha}`);
+        if (nl === -1 || m === null || (isSha(spec) && m[1] !== spec)) {
+            throw new Error(`cat-file returned "${header}" for ${spec}`);
         }
         const size = Number(m[2]);
-        blobs.set(sha, out.subarray(nl + 1, nl + 1 + size).toString("latin1"));
+        blobs.push(out.subarray(nl + 1, nl + 1 + size).toString("latin1"));
         at = nl + 1 + size + 1;
     }
     return blobs;
 }
+async function catBlobs(repo, shas) {
+    const blobs = await catBatch(repo, shas);
+    return new Map(shas.map((sha, i) => [sha, blobs[i] ?? ""]));
+}
+/** Each path's bytes at the commit, as latin1. */
+export async function blobsAt(repo, commit, paths) {
+    const blobs = await catBatch(repo, paths.map((p) => `${commit}:${p}`));
+    return new Map(paths.map((p, i) => [p, blobs[i] ?? ""]));
+}
+function pathList(out) {
+    return new Set(out
+        .toString("latin1")
+        .split("\0")
+        .filter((p) => isRepoPath(p)));
+}
+/** Every file path in the commit's tree. */
+export async function pathsAt(repo, commit) {
+    return pathList(await git(repo, ["ls-tree", "-r", "-z", "--name-only", commit]));
+}
+/**
+ * Paths whose content differs between the commit and `to`: another commit, or with null the
+ * working tree and index. Untracked files are not listed.
+ */
+export async function pathsChanged(repo, from, to) {
+    const target = to === null ? [] : [to];
+    return pathList(await git(repo, ["diff", "--name-only", "-z", ...DIFF_FLAGS, from, ...target, "--"]));
+}
+/**
+ * Lines of the commit's tree, not the working tree, that match any of the POSIX extended
+ * patterns, in files matching the pathspecs. Binary files are skipped.
+ */
+export async function grepTip(repo, commit, patterns, pathspecs) {
+    const args = [
+        "grep",
+        "-n",
+        "-E",
+        "-I",
+        "-z",
+        "--no-color",
+        "--no-column",
+        ...patterns.flatMap((p) => ["-e", p]),
+        commit,
+        "--",
+        ...pathspecs,
+    ];
+    let out;
+    try {
+        out = await git(repo, args);
+    }
+    catch (error) {
+        // grep exits 1 when nothing matched.
+        if (error instanceof GitError && error.status === 1)
+            return [];
+        throw error;
+    }
+    const prefix = `${commit}:`;
+    return out
+        .toString("latin1")
+        .split("\n")
+        .flatMap((record) => {
+        const [name = "", line = "", ...text] = record.split("\0");
+        const path = name.slice(prefix.length);
+        if (!name.startsWith(prefix) || !isRepoPath(path) || !/^\d+$/.test(line))
+            return [];
+        return [{ path, line: Number(line), text: text.join("\0") }];
+    });
+}
 const DIFF_FLAGS = ["--no-renames", "--full-index", "--no-color", "--no-ext-diff", "--no-textconv"];
 export async function loadDiff(repo, baseRef, tipRef) {
     const [tip, baseCommit] = await Promise.all([

package/dist/src/host.js

@@ -373,6 +373,10 @@ export async function postComment(port, summary, comment) {
     }
     return id;
 }
+/** Tells the hosted page that coverage.json changed, so it reloads its marks. */
+export async function postCoverage(port, summary) {
+    await call(port, "POST", `${summary.url}api/coverage`);
+}
 /** The consolidated report of the hosted session as it stands. */
 export async function fetchReport(port, summary, format = "markdown") {
     const res = await fetch(`${sessionUrl(port, summary)}api/report?format=${format}`);

package/dist/src/plan.js

@@ -17,6 +17,10 @@ const STATUSES = ["addressed", "declined", "deferred", "answered"];
 function isResponseStatus(s) {
     return STATUSES.some((x) => x === s);
 }
+const SECTIONS = ["story", "steps", "details", "wiring"];
+function isCommitSection(s) {
+    return SECTIONS.some((x) => x === s);
+}
 const SEVERITIES = ["blocker", "major", "minor"];
 const CONCERN_KINDS = ["bug", "missing-test", "naming", "risk"];
 export function isSeverity(s) {
@@ -223,6 +227,10 @@ export function parsePlanFile(draft) {
             }
             if (typeof c.message !== "string")
                 bad(`${at}.message`, "expected a string");
+            const section = c.section ?? null;
+            if (section !== null && !isCommitSection(section)) {
+                bad(`${at}.section`, `expected one of ${SECTIONS.join(", ")}`);
+            }
             const extras = parseExtras(c, false);
             if ("error" in extras)
                 bad(`${at}.${extras.field}`, extras.error);
@@ -239,7 +247,8 @@ export function parsePlanFile(draft) {
                 }
             }
             if (typeof c.message === "string" && !("error" in extras)) {
-                commits.push({ message: c.message, ...extras, units });
+                const placed = isCommitSection(section) ? { section } : {};
+                commits.push({ message: c.message, ...placed, ...extras, units });
             }
         }
     }
@@ -287,6 +296,7 @@ export function serializePlanFile(file) {
         tip: file.tip,
         commits: file.commits.map((c) => ({
             message: c.message,
+            ...(c.section === undefined ? {} : { section: c.section }),
             ...serializeExtras(c),
             units: c.units.map((ref) => ref.narration === undefined && ref.diagram === undefined && ref.concerns === undefined
                 ? formatUnitRef(ref)
@@ -578,6 +588,7 @@ function project(diff, file, claims, issues) {
         return {
             index: k,
             message: commit.message,
+            ...(commit.section === undefined ? {} : { section: commit.section }),
             ...withText(commit.narration ?? null, commit.diagram ?? null),
             ...(mine.length === 0 ? {} : { concerns: mine }),
             refs,

package/dist/src/prompt.js

@@ -1,3 +1,4 @@
+import { changedLines } from "./coverage.js";
 import { tipStartOf } from "./diff.js";
 import { anchorOn, concernOn } from "./evolve.js";
 /** Consecutive runs collapse: [0,1,2,3,7] becomes "0-3,7". */
@@ -581,14 +582,14 @@ function itemHeading(item, level) {
     parts.push(kept === null ? "raised by the reviewer" : "raised by the agent, kept by the reviewer");
     return `${"#".repeat(level)} ${parts.join(" · ")}`;
 }
-function renderItems(items, units, level) {
+function renderItems(items, units, level, tallies) {
     const out = [];
     const sorted = sortItems(items, units);
     const paths = [...new Set(sorted.flatMap((i) => (i.where.path === null ? [] : [i.where.path])))];
-    const group = (title, members) => {
+    const group = (title, members, lead = []) => {
         if (members.length === 0)
             return;
-        out.push(`${"#".repeat(level)} ${title}`, "");
+        out.push(`${"#".repeat(level)} ${title}`, "", ...lead);
         for (const item of members) {
             out.push(itemHeading(item, level + 1), "", ...quoteBlock(item.where.quote));
             if (item.where.quote.length > 0)
@@ -596,8 +597,12 @@ function renderItems(items, units, level) {
             out.push(...voices(item), "");
         }
     };
-    for (const path of paths)
-        group(path, sorted.filter((i) => i.where.path === path));
+    for (const path of paths) {
+        const tally = tallies.get(path);
+        group(path, sorted.filter((i) => i.where.path === path), tally === undefined
+            ? []
+            : [`coverage: ${tally.covered} of ${tally.executable} changed lines`, ""]);
+    }
     group("General", sorted.filter((i) => i.where.path === null));
     return out;
 }
@@ -719,24 +724,43 @@ function spanOf(rows) {
         startOldLine: range && first.op !== "+" ? first.base : null,
     };
 }
-function commentSpan(ctx, c) {
+function commentRows(ctx, c) {
     const { anchor } = c;
     if (anchor.kind !== "hunk")
-        return NO_SPAN;
+        return null;
     const hunk = ctx.units.get(anchor.unit);
     if (hunk?.kind !== "hunk")
-        return NO_SPAN;
+        return null;
     const { offsets, changes } = anchor;
-    if (offsets !== undefined)
-        return spanOf(numberedRows(hunk, ctx.units, (i) => offsets.includes(i)));
-    return spanOf(numberedRows(hunk, ctx.units, (_, n) => n !== null && (changes === null || changes.includes(n))));
+    const want = offsets !== undefined
+        ? (i) => offsets.includes(i)
+        : (_, n) => n !== null && (changes === null || changes.includes(n));
+    return { path: hunk.path, rows: numberedRows(hunk, ctx.units, want) };
 }
-function concernSpan(ctx, concern) {
+function concernRows(ctx, concern) {
     const hunk = concern.unit === undefined ? undefined : ctx.units.get(concern.unit);
     if (hunk?.kind !== "hunk")
-        return NO_SPAN;
+        return null;
     const lines = concern.lines;
-    return spanOf(numberedRows(hunk, ctx.units, (_, n) => n !== null && (lines === undefined || lines.includes(n))));
+    const want = (_, n) => n !== null && (lines === undefined || lines.includes(n));
+    return { path: hunk.path, rows: numberedRows(hunk, ctx.units, want) };
+}
+function spanAt(at) {
+    return at === null ? NO_SPAN : spanOf(at.rows);
+}
+/**
+ * Whether the test run ran every executable added row the item points at. Absent without a run
+ * that still fits the file, when it picks no such row, or when its code changed after it was said.
+ */
+function coveredOf(ctx, id, at) {
+    const file = at === null ? undefined : ctx.coverage?.get(at.path);
+    if (at === null || file === undefined || ctx.marks?.has(id))
+        return {};
+    const hits = at.rows.flatMap((r) => {
+        const n = r.op === "+" ? file.lines.get(r.tip) : undefined;
+        return n === undefined ? [] : [n];
+    });
+    return hits.length === 0 ? {} : { covered: hits.every((n) => n > 0) };
 }
 function threadOf(c) {
     return (c.replies ?? []).map((r) => ({ author: r.author, body: r.body }));
@@ -761,6 +785,7 @@ function collectItems(review, ctx, responses) {
         const { kept } = it;
         const first = it.comments[0];
         if (kept !== null) {
+            const at = concernRows(ctx, kept.concern);
             const note = kept.note === null ? "" : `\n\nReviewer: ${kept.note}`;
             const thread = it.comments.flatMap((c) => [
                 { author: "reviewer", body: c.body },
@@ -770,7 +795,7 @@ function collectItems(review, ctx, responses) {
                 where: it.where,
                 item: {
                     id: kept.concern.id,
-                    ...concernSpan(ctx, kept.concern),
+                    ...spanAt(at),
                     path: it.where.path,
                     oldPath: it.where.path,
                     severity: it.severity,
@@ -780,16 +805,18 @@ function collectItems(review, ctx, responses) {
                     resolved: false,
                     excluded: noConcerns.has(kept.concern.id),
                     ...changedOf(ctx, kept.concern.id),
+                    ...coveredOf(ctx, kept.concern.id, at),
                 },
             };
         }
         if (first === undefined)
             throw new Error("an item with neither a concern nor a comment");
+        const at = commentRows(ctx, first);
         return {
             where: it.where,
             item: {
                 id: first.id,
-                ...commentSpan(ctx, first),
+                ...spanAt(at),
                 path: it.where.path,
                 oldPath: it.where.path,
                 severity: it.severity,
@@ -799,6 +826,7 @@ function collectItems(review, ctx, responses) {
                 resolved: resolved.has(first.id),
                 excluded: noComments.has(first.id),
                 ...changedOf(ctx, first.id),
+                ...coveredOf(ctx, first.id, at),
             },
         };
     });
@@ -812,7 +840,7 @@ function collectItems(review, ctx, responses) {
             where,
             item: {
                 id: q.id,
-                ...commentSpan(ctx, q),
+                ...spanAt(commentRows(ctx, q)),
                 path: where.path,
                 oldPath: where.path,
                 severity: q.severity ?? null,
@@ -832,7 +860,7 @@ function collectItems(review, ctx, responses) {
  * Excluded ones are present and marked; whoever posts them skips those.
  */
 export function reportItems(review, commits, units, opts) {
-    const session = folded(review, commits, units, opts.earlier);
+    const session = folded(review, commits, units, opts);
     return collectItems(session.review, session.ctx, opts.responses).map((c) => c.item);
 }
 /** A short list of what the reviewer left out, so nothing vanishes without a trace. */
@@ -855,9 +883,11 @@ function renderExcluded(collected) {
  * taking their unit, and a mark on each whose code changed. Each id appears once; this
  * version's own copy wins.
  */
-function folded(review, commits, units, earlier) {
+function folded(review, commits, units, opts) {
+    const { earlier } = opts;
+    const coverage = opts.coverage === undefined ? {} : { coverage: opts.coverage };
     if (earlier === undefined)
-        return { review, ctx: { commits, units } };
+        return { review, ctx: { commits, units, ...coverage } };
     const marks = new Map();
     const mark = (id, at) => {
         if (at.since !== null)
@@ -895,6 +925,7 @@ function folded(review, commits, units, earlier) {
         ctx: {
             units,
             marks,
+            ...coverage,
             commits: commits.map((commit, k) => {
                 const mine = lost.filter((c) => owner(c.unit, c.commit) === k);
                 if (mine.length === 0)
@@ -915,7 +946,7 @@ function folded(review, commits, units, earlier) {
  * Concerns the reviewer has not decided are left out, as are those dismissed.
  */
 export function renderReport(full, commits, units, opts) {
-    const session = folded(full, commits, units, opts.earlier);
+    const session = folded(full, commits, units, opts);
     const { ctx } = session;
     const excluded = session.review.excluded;
     if (excluded === undefined)
@@ -942,12 +973,14 @@ function renderIncluded(review, ctx, opts) {
     const issues = walk ? rest : rest.filter((c) => c.severity !== undefined);
     const feedback = walk ? [] : rest.filter((c) => c.severity === undefined);
     const items = buildItems(ctx, keptConcerns(review, commits), issues);
+    // Keyed by path as the items name it, so a file's heading finds its count.
+    const tallies = new Map(ctx.coverage === undefined ? [] : changedLines(units, ctx.coverage));
     const out = [];
     if (walk) {
         out.push(`# Code review: ${opts.branch} (${commits.length === 1 ? "1 commit" : `${commits.length} commits`}, ${summarizeIssues(items)})`, "");
         if (items.length === 0 && questions.length === 0)
             out.push("The reviewer raised no issues.", "");
-        out.push(...renderItems(items, units, 2));
+        out.push(...renderItems(items, units, 2, tallies));
     }
     else {
         out.push(`# Review of plan v${review.planVersion}`, "");
@@ -960,7 +993,7 @@ function renderIncluded(review, ctx, opts) {
         }
         out.push(...renderPlanFeedback(ctx, feedback));
         if (items.length > 0) {
-            out.push(`## Code issues (${summarizeIssues(items)})`, "", ...renderItems(items, units, 3));
+            out.push(`## Code issues (${summarizeIssues(items)})`, "", ...renderItems(items, units, 3, tallies));
         }
     }
     if (questions.length > 0)
@@ -1016,6 +1049,8 @@ function renderEvent(event, ctx) {
             return [`- ${event.commentId} ${event.resolved ? "resolved" : "reopened"}`];
         case "version":
             return [`- plan v${event.version} is open`];
+        case "coverage":
+            return [`- the page shows coverage from ${event.tip.slice(0, 12)}`];
         case "submitted":
             return ["- the reviewer submitted the review"];
         case "ended":
@@ -1065,3 +1100,25 @@ export function renderShownConcern(concern, commit, ctx) {
         "",
     ].join("\n");
 }
+function utf8(latin1) {
+    return Buffer.from(latin1, "latin1").toString("utf8");
+}
+/** Where a name is defined, each with the commit it lands in and its first lines at the tip. */
+export function renderShownDefinitions(name, candidates) {
+    if (candidates.length === 0)
+        return `no definition of ${name} at the tip\n`;
+    return `${candidates
+        .flatMap((c) => {
+        const where = c.commit !== null
+            ? `in commit ${c.commit + 1}`
+            : c.unit !== null
+                ? `in ${c.unit}, which no commit of the plan takes`
+                : "unchanged by the branch";
+        const width = String(c.newLine + c.preview.length - 1).length;
+        return [
+            `${utf8(c.path)}:${c.newLine} ${c.kind}, ${where}`,
+            ...c.preview.map((l, i) => `    ${String(c.newLine + i).padStart(width)}  ${utf8(l)}`),
+        ];
+    })
+        .join("\n")}\n`;
+}

package/dist/src/server.js

@@ -5,10 +5,13 @@ import { homedir } from "node:os";
 import { basename, isAbsolute, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { isReplyId, parseComment, parseDraft, parseNewDecision, parseNewReply, } from "./comments.js";
+import { serializeCoverage, staleFiles } from "./coverage.js";
+import { definitionIndex, isIdentifier } from "./definition.js";
+import { isRepoPath } from "./diff.js";
 import { carryConcerns, carrySession } from "./evolve.js";
 import { isConcernId, isPlanVersion } from "./plan.js";
 import { renderReport, reportItems } from "./prompt.js";
-import { isLockToken } from "./session.js";
+import { coverageAt, isLockToken, readCoverageIn, SessionFileError } from "./session.js";
 /** The requested port is taken; host.ts decides whether that is our server or a stranger. */
 export class PortInUseError extends Error {
     name = "PortInUseError";
@@ -255,6 +258,7 @@ function viewOf(registration, loaded, draft, earlier = loaded.earlier) {
         unitIds: new Set(state.units.map((u) => u.id)),
         hunkRows: new Map(state.units.flatMap((u) => u.kind === "hunk" ? [[u.id, u.body.length]] : [])),
         concernIds: new Set(state.commits.flatMap((c) => (c.concerns ?? []).map((k) => k.id))),
+        definitions: definitionIndex(loaded.repo, state.anchor.tip, new Map(state.units.map((u) => [u.id, u])), state.commits),
         draftFile: draftPath(reviewsDir, state.version),
         reviewFile: join(reviewsDir, `v${state.version}.json`),
     };
@@ -468,25 +472,44 @@ function hostSession(id, registration, loaded, draft, load) {
             ...carriedRefs(view),
         };
     };
-    const reportOptions = () => {
+    const reportOptions = async () => {
         const comments = sessionOf(view);
+        const { repo, state } = view.loaded;
+        const coverage = await coverageAt(repo, view.registration.dir, state.anchor.tip);
         return {
-            branch: view.loaded.state.anchor.branch,
+            branch: state.anchor.branch,
             responses: [],
             ...(comments === null ? {} : { earlier: { comments, concerns: view.earlierConcerns } }),
+            ...(coverage === null ? {} : { coverage }),
         };
     };
     const unitsOf = () => new Map(view.loaded.state.units.map((u) => [u.id, u]));
-    const report = (finished) => renderReport(finished, view.loaded.state.commits, unitsOf(), reportOptions());
+    const report = async (finished) => renderReport(finished, view.loaded.state.commits, unitsOf(), await reportOptions());
     /** Writes reviews/vN.json and its report once, and removes the autosave. */
     const writeReview = async (finished) => {
         await writes;
         await mkdir(view.loaded.reviewsDir, { recursive: true });
         await writeAtomic(view.reviewFile, `${JSON.stringify(finished, null, 2)}\n`);
-        await writeAtomic(view.reviewFile.replace(/\.json$/, ".md"), report(finished));
+        await writeAtomic(view.reviewFile.replace(/\.json$/, ".md"), await report(finished));
         await rm(view.draftFile, { force: true });
         return view.reviewFile;
     };
+    /** coverage.json as the agent last wrote it, or a 404 before its first run. */
+    const coverageFile = () => {
+        let coverage;
+        try {
+            coverage = readCoverageIn(view.registration.dir);
+        }
+        catch (error) {
+            if (error instanceof SessionFileError)
+                throw new HttpError(500, error.message);
+            throw error;
+        }
+        if (coverage === null) {
+            throw new HttpError(404, "no coverage yet; the agent records it with follow-me coverage");
+        }
+        return coverage;
+    };
     const hasWork = () => view.comments.size > 0 || view.decisions.size > 0 || view.earlierReplies.size > 0;
     return {
         id,
@@ -544,7 +567,28 @@ function hostSession(id, registration, loaded, draft, load) {
             return waitForEvents(since, timeoutMs, signal);
         },
         report: () => report(review()),
-        reportItems: () => reportItems(review(), view.loaded.state.commits, unitsOf(), reportOptions()),
+        /**
+         * The agent's last test run, read from disk so a restarted server serves it too, with the
+         * covered files whose content changed since its tip. A new plan for the same tip stales nothing.
+         */
+        coverage: async () => {
+            const coverage = coverageFile();
+            const stale = await staleFiles(view.loaded.repo, coverage, view.loaded.state.anchor.tip);
+            return { ...serializeCoverage(coverage), stale };
+        },
+        /** The agent wrote coverage.json; the page hears it as one event and reloads its marks. */
+        coverageTaken: () => {
+            open();
+            const { tip, takenAt } = coverageFile();
+            emit({ type: "coverage", tip, takenAt });
+            return { tip, takenAt };
+        },
+        /** The page reads a definition the plan shows from its hunk, so only others carry a preview. */
+        define: async (name, from) => {
+            const found = await view.definitions.lookup(name, from !== null && isRepoPath(from) ? from : null);
+            return found.map((c) => (c.unit === null ? c : { ...c, preview: [] }));
+        },
+        reportItems: async () => reportItems(review(), view.loaded.state.commits, unitsOf(), await reportOptions()),
         add: async (raw) => {
             open();
             if (!isRecord(raw))
@@ -943,12 +987,25 @@ export function startServer(opts) {
             const timeout = queryInt(url, "timeout", 0, MAX_POLL_MS, DEFAULT_POLL_MS);
             return sendJson(res, 200, await session.poll(since, timeout, abort.signal));
         }
+        if (method === "GET" && rest === "/api/definition") {
+            const name = url.searchParams.get("name") ?? "";
+            if (!isIdentifier(name))
+                throw new HttpError(400, "name must be an identifier");
+            const candidates = await session.define(name, url.searchParams.get("from"));
+            return sendJson(res, 200, { candidates });
+        }
+        if (method === "GET" && rest === "/api/coverage") {
+            return sendJson(res, 200, await session.coverage());
+        }
+        if (method === "POST" && rest === "/api/coverage") {
+            return sendJson(res, 200, session.coverageTaken());
+        }
         if (method === "GET" && rest === "/api/report" && url.searchParams.get("format") === "json") {
-            return sendJson(res, 200, session.reportItems());
+            return sendJson(res, 200, await session.reportItems());
         }
         if (method === "GET" && rest === "/api/report") {
             res.writeHead(200, { "content-type": "text/markdown; charset=utf-8" });
-            res.end(session.report());
+            res.end(await session.report());
             return;
         }
         if (method === "POST" && rest === "/api/version") {

package/dist/src/session.js

@@ -2,6 +2,7 @@ import { randomUUID } from "node:crypto";
 import { existsSync, linkSync, mkdirSync, readdirSync, readFileSync, renameSync, rmSync, statSync, writeFileSync, } from "node:fs";
 import { basename, dirname, join } from "node:path";
 import { parseDraft, parseReview } from "./comments.js";
+import { freshFiles, parseCoverage, serializeCoverage, staleFiles } from "./coverage.js";
 import { isSha } from "./diff.js";
 import { evolvePlan } from "./evolve.js";
 import { branchExists, currentBranch, defaultBaseRef, GitError, loadDiff, objectsDir, QUARANTINE_PREFIX, repoRoot, resolveCommit, } from "./git.js";
@@ -200,6 +201,36 @@ function parseApplied(v) {
         return null;
     return { schema, version, branch, head, hash, signed };
 }
+function coveragePath(dir) {
+    return join(dir, "coverage.json");
+}
+/** The coverage.json of the session in `dir`, or null when there is none. */
+export function readCoverageIn(dir) {
+    const path = coveragePath(dir);
+    const raw = readJson(path);
+    if (raw === undefined)
+        return null;
+    const coverage = parseCoverage(raw);
+    if (typeof coverage === "string")
+        throw new SessionFileError(`${path}: ${coverage}`);
+    return coverage;
+}
+/**
+ * What the report counts: the files of the session's test run that still describe `tip`. Null
+ * without a run, and when coverage.json cannot be read, so a report never fails on it.
+ */
+export async function coverageAt(repo, dir, tip) {
+    let coverage;
+    try {
+        coverage = readCoverageIn(dir);
+    }
+    catch (error) {
+        if (error instanceof SessionFileError)
+            return null;
+        throw error;
+    }
+    return coverage === null ? null : freshFiles(coverage, await staleFiles(repo, coverage, tip));
+}
 function makeSession(root, dir, anchor, diff) {
     const plansDir = join(dir, "plans");
     const splitReviews = join(dir, "reviews");
@@ -310,6 +341,8 @@ function makeSession(root, dir, anchor, diff) {
             writeJson(join(plansDir, `v${version}.json`), serializePlanFile(plan.file));
             return version;
         },
+        coverage: () => readCoverageIn(dir),
+        writeCoverage: (coverage) => writeJson(coveragePath(dir), serializeCoverage(coverage)),
         lock: (command) => acquireLock(dir, anchor.branch, command),
         adoptLock: (from, command) => adoptLock(dir, from, command),
         lockState: () => readLock(join(dir, "lock")),

package/dist/src/types.js

@@ -10,6 +10,10 @@
  *   session.ts  the .follow-me/<slug>/ layout: session anchor, plans, reviews, draft.
  *   comments.ts the wire formats of review state: comments, replies, concern decisions, the
  *               review file, the autosaved draft and the server's events. Pure parsers.
+ *   definition.ts where an identifier is defined at the branch tip, and where that lands in the
+ *               plan: per-language patterns, ranking, placement and a per-tip cache.
+ *   coverage.ts lcov from the agent's test run: parse, place at the tip, merge runs, count the
+ *               changed lines covered, and which files went stale since.
  *   prompt.ts   the planner prompt for an agent, the feedback and the consolidated report.
  *   server.ts   the review server: a registry of hosted sessions, their JSON API, and ui/.
  *   host.ts     the one server per user: its record in the state dir, attach, start, stop.

package/dist/ui/dist/assets/{abnfDiagram-WEYOIA3C-8UsLPznO.js → abnfDiagram-WEYOIA3C-cHEJvcBn.js}

@@ -1 +1 @@
-import{m as e,t}from"./mermaid-parser.core-DxvaorTa.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-BZ3kG7lO.js";import"./chunk-DSNLSHJZ-YiMuxsb-.js";import{n as i,r as a,t as o}from"./chunk-2E7WHVMD-BiHSeJMu.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-ujYTsRZz.js";var c=e().RailroadAbnf.parser.LangiumParser,l=n(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=n(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=n(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=n(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=n(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=n(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=n(e=>{s(e,o),e.title&&o.setTitle(e.title),e.rules.map(e=>o.addRule(m(e)))},`populateDb`),g={parser:{parse:n(e=>{o.clear(),r.debug(`[ABNF Parser] Starting Langium parse`);let n=c.parse(e);if(n.lexerErrors.length>0||n.parserErrors.length>0)throw new t(n);let i=n.value;r.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),r.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:o}},db:o,renderer:a,styles:i};export{g as diagram};
+import{m as e,t}from"./mermaid-parser.core-CMPNc-u6.js";import{n}from"./chunk-Y2CYZVJY-DsF7k-Jl.js";import{m as r}from"./src-BiC9XNcI.js";import"./chunk-DSNLSHJZ-Bk8WO5JF.js";import{n as i,r as a,t as o}from"./chunk-2E7WHVMD-BTet9STn.js";import{t as s}from"./chunk-JWPE2WC7-DVXcaiue.js";import"./mermaid.core-Nd_xOg2y.js";var c=e().RailroadAbnf.parser.LangiumParser,l=n(e=>{let t=e.alternatives.map(u);return t.length===1?t[0]:{type:`choice`,alternatives:t}},`transformAlternation`),u=n(e=>{let t=e.elements.map(f);return t.length===1?t[0]:{type:`sequence`,elements:t}},`transformConcatenation`),d=n(e=>{if(e.includes(`*`)){let[t,n]=e.split(`*`);return{min:t?parseInt(t,10):0,max:n?parseInt(n,10):1/0}}let t=parseInt(e,10);return{min:t,max:t}},`parseRepeat`),f=n(e=>{let t=p(e.primary);if(!e.repeat)return t;let{min:n,max:r}=d(e.repeat);return n===0&&r===1?{type:`optional`,element:t}:{type:`repetition`,element:t,min:n,max:r}},`transformElement`),p=n(e=>{switch(e.$type){case`AbnfStringLiteral`:return{type:`terminal`,value:e.value};case`AbnfNumVal`:return{type:`terminal`,value:e.value};case`AbnfRuleName`:return{type:`nonterminal`,name:e.name};case`AbnfGroup`:return l(e.element);case`AbnfOptionalGroup`:return{type:`optional`,element:l(e.element)};default:throw Error(`Unsupported ABNF primary node: ${e.$type}`)}},`transformPrimary`),m=n(e=>({name:e.name,definition:l(e.definition)}),`transformRule`),h=n(e=>{s(e,o),e.title&&o.setTitle(e.title),e.rules.map(e=>o.addRule(m(e)))},`populateDb`),g={parser:{parse:n(e=>{o.clear(),r.debug(`[ABNF Parser] Starting Langium parse`);let n=c.parse(e);if(n.lexerErrors.length>0||n.parserErrors.length>0)throw new t(n);let i=n.value;r.debug(`[ABNF Parser] Parsed rules:`,i.rules.length),h(i),r.debug(`[ABNF Parser] Parse complete`)},`parse`),parser:{yy:o}},db:o,renderer:a,styles:i};export{g as diagram};

package/dist/ui/dist/assets/{arc-vovjGalz.js → arc-xolFyc0v.js}

@@ -1 +1 @@
-import{n as e,t}from"./path-fybaL0A-.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-D3j-DODP.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i){if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};
+import{n as e,t}from"./path-fybaL0A-.js";import{a as n,c as r,d as i,f as a,i as o,l as s,m as c,n as l,o as u,p as d,r as f,u as p}from"./dist-CHZ9mrl_.js";function m(e){return e.innerRadius}function h(e){return e.outerRadius}function g(e){return e.startAngle}function _(e){return e.endAngle}function v(e){return e&&e.padAngle}function y(e,t,n,r,i,a,o,s){var c=n-e,l=r-t,u=o-i,d=s-a,f=d*c-u*l;if(!(f*f<1e-12))return f=(u*(t-a)-d*(e-i))/f,[e+f*c,t+f*l]}function b(e,t,n,r,i,a,o){var c=e-n,l=t-r,u=(o?a:-a)/d(c*c+l*l),f=u*l,p=-u*c,m=e+f,h=t+p,g=n+f,_=r+p,v=(m+g)/2,y=(h+_)/2,b=g-m,x=_-h,S=b*b+x*x,C=i-a,w=m*_-g*h,T=(x<0?-1:1)*d(s(0,C*C*S-w*w)),E=(w*x-b*T)/S,D=(-w*b-x*T)/S,O=(w*x+b*T)/S,k=(-w*b+x*T)/S,A=E-v,j=D-y,M=O-v,N=k-y;return A*A+j*j>M*M+N*N&&(E=O,D=k),{cx:E,cy:D,x01:-f,y01:-p,x11:E*(i/C-1),y11:D*(i/C-1)}}function x(){var s=m,x=h,S=e(0),C=null,w=g,T=_,E=v,D=null,O=t(k);function k(){var e,t,m=+s.apply(this,arguments),h=+x.apply(this,arguments),g=w.apply(this,arguments)-r,_=T.apply(this,arguments)-r,v=l(_-g),k=_>g;if(D||=e=O(),h<m&&(t=h,h=m,m=t),!(h>1e-12))D.moveTo(0,0);else if(v>c-1e-12)D.moveTo(h*u(g),h*a(g)),D.arc(0,0,h,g,_,!k),m>1e-12&&(D.moveTo(m*u(_),m*a(_)),D.arc(0,0,m,_,g,k));else{var A=g,j=_,M=g,N=_,P=v,F=v,I=E.apply(this,arguments)/2,L=I>1e-12&&(C?+C.apply(this,arguments):d(m*m+h*h)),R=p(l(h-m)/2,+S.apply(this,arguments)),z=R,B=R,V,H;if(L>1e-12){var U=o(L/m*a(I)),W=o(L/h*a(I));(P-=U*2)>1e-12?(U*=k?1:-1,M+=U,N-=U):(P=0,M=N=(g+_)/2),(F-=W*2)>1e-12?(W*=k?1:-1,A+=W,j-=W):(F=0,A=j=(g+_)/2)}var G=h*u(A),K=h*a(A),q=m*u(N),J=m*a(N);if(R>1e-12){var Y=h*u(j),X=h*a(j),Z=m*u(M),Q=m*a(M),$;if(v<i){if($=y(G,K,Z,Q,Y,X,q,J)){var ee=G-$[0],te=K-$[1],ne=Y-$[0],re=X-$[1],ie=1/a(f((ee*ne+te*re)/(d(ee*ee+te*te)*d(ne*ne+re*re)))/2),ae=d($[0]*$[0]+$[1]*$[1]);z=p(R,(m-ae)/(ie-1)),B=p(R,(h-ae)/(ie+1))}else z=B=0}}F>1e-12?B>1e-12?(V=b(Z,Q,G,K,h,B,k),H=b(Y,X,q,J,h,B,k),D.moveTo(V.cx+V.x01,V.cy+V.y01),B<R?D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,B,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,h,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),!k),D.arc(H.cx,H.cy,B,n(H.y11,H.x11),n(H.y01,H.x01),!k))):(D.moveTo(G,K),D.arc(0,0,h,A,j,!k)):D.moveTo(G,K),!(m>1e-12)||!(P>1e-12)?D.lineTo(q,J):z>1e-12?(V=b(q,J,Y,X,m,-z,k),H=b(G,K,Z,Q,m,-z,k),D.lineTo(V.cx+V.x01,V.cy+V.y01),z<R?D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(H.y01,H.x01),!k):(D.arc(V.cx,V.cy,z,n(V.y01,V.x01),n(V.y11,V.x11),!k),D.arc(0,0,m,n(V.cy+V.y11,V.cx+V.x11),n(H.cy+H.y11,H.cx+H.x11),k),D.arc(H.cx,H.cy,z,n(H.y11,H.x11),n(H.y01,H.x01),!k))):D.arc(0,0,m,N,M,k)}if(D.closePath(),e)return D=null,e+``||null}return k.centroid=function(){var e=(+s.apply(this,arguments)+ +x.apply(this,arguments))/2,t=(+w.apply(this,arguments)+ +T.apply(this,arguments))/2-i/2;return[u(t)*e,a(t)*e]},k.innerRadius=function(t){return arguments.length?(s=typeof t==`function`?t:e(+t),k):s},k.outerRadius=function(t){return arguments.length?(x=typeof t==`function`?t:e(+t),k):x},k.cornerRadius=function(t){return arguments.length?(S=typeof t==`function`?t:e(+t),k):S},k.padRadius=function(t){return arguments.length?(C=t==null?null:typeof t==`function`?t:e(+t),k):C},k.startAngle=function(t){return arguments.length?(w=typeof t==`function`?t:e(+t),k):w},k.endAngle=function(t){return arguments.length?(T=typeof t==`function`?t:e(+t),k):T},k.padAngle=function(t){return arguments.length?(E=typeof t==`function`?t:e(+t),k):E},k.context=function(e){return arguments.length?(D=e??null,k):D},k}export{x as t};