@renaiss-shipflow/cli 0.9.0 → 0.11.0

package/dist/packet.js

@@ -0,0 +1,261 @@
+// Review-packet assembly (Superpowers-6 technique: pre-bake everything the
+// loop's PR reviewer needs into ONE command output, instead of the reviewer
+// spending tokens and wall time re-deriving it with a pile of gh calls).
+//
+// The packet deliberately leads with the SPEC/BRIEF section: a diff-only
+// packet tempts a reviewer into confidently redefining "spec" as whatever the
+// diff implies — the exact failure family Superpowers 6 measured (0/5 flagged
+// a missing brief). When no linked issue exists, the packet says so loudly.
+// Budgets mirror the server-side pr_review runner so both reviewers see
+// comparably sized diffs.
+export const PACKET_PER_FILE_CAP = 6000;
+export const PACKET_TOTAL_CAP = 60000;
+export const PACKET_BRIEF_CAP = 8000; // acceptance criteria sit at the BOTTOM of issue bodies — a tight cap silently drops the spec's checklist
+// Noise paths excluded from the packet diff — lockfiles, generated output,
+// vendored deps, binary-adjacent assets. Kept in sync (by convention) with the
+// server's featuremap noise lists.
+const NOISE_SUBSTRINGS = [
+    "node_modules/", "vendor/", "dist/", "build/", ".next/", "__generated__/",
+    "__snapshots__/", "coverage/", "storybook-static/", ".turbo/", "out/",
+];
+const NOISE_SUFFIXES = [
+    ".lock", ".sum", "package-lock.json", "bun.lockb", ".min.js", ".min.css",
+    ".map", ".snap", ".png", ".jpg", ".jpeg", ".gif", ".ico", ".svg",
+    ".woff", ".woff2", ".ttf", ".eot", ".pb.go", "_generated.go", ".gen.go",
+];
+export function isNoiseDiffPath(path) {
+    const p = path.toLowerCase();
+    return (NOISE_SUBSTRINGS.some((s) => p.includes(s)) ||
+        NOISE_SUFFIXES.some((s) => p.endsWith(s)));
+}
+/** Split a unified `gh pr diff` output into per-file sections. */
+export function splitUnifiedDiff(diff) {
+    const sections = [];
+    const lines = diff.split("\n");
+    let current = null;
+    for (const line of lines) {
+        const m = /^diff --git a\/(.+) b\/(.+)$/.exec(line);
+        if (m) {
+            if (current)
+                sections.push(current);
+            current = { path: m[2], body: line };
+        }
+        else if (current) {
+            current.body += "\n" + line;
+        }
+    }
+    if (current)
+        sections.push(current);
+    return sections;
+}
+/** Noise-filter and budget the diff (per-file + total caps). */
+export function filterDiffForPacket(diff) {
+    const out = [];
+    let shown = 0;
+    let omittedNoise = 0;
+    let omittedBudget = 0;
+    let truncatedFiles = 0;
+    let total = 0;
+    for (const s of splitUnifiedDiff(diff)) {
+        if (isNoiseDiffPath(s.path)) {
+            omittedNoise++;
+            continue;
+        }
+        if (total >= PACKET_TOTAL_CAP) {
+            omittedBudget++;
+            continue;
+        }
+        let body = s.body;
+        if (body.length > PACKET_PER_FILE_CAP) {
+            body = body.slice(0, PACKET_PER_FILE_CAP) + "\n… (file diff truncated)";
+            truncatedFiles++;
+        }
+        out.push(body);
+        total += body.length;
+        shown++;
+    }
+    return { text: out.join("\n"), shown, omittedNoise, omittedBudget, truncatedFiles };
+}
+/** Lines from PR comments that carry test evidence / health captions. */
+export function extractEvidenceLines(comments) {
+    const lines = [];
+    for (const c of comments) {
+        const body = c.body ?? "";
+        if (!body.includes("Test evidence") && !/health \d+/.test(body))
+            continue;
+        for (const line of body.split("\n")) {
+            if (line.includes("Test evidence") || /health \d+→\d+|health \d+ ?→/.test(line) || line.startsWith("Verified:") || line.trimStart().startsWith("- Verified")) {
+                lines.push(line.trim());
+            }
+        }
+    }
+    return lines.slice(0, 12);
+}
+/** Names of the features whose path prefixes match any non-noise diff path,
+ *  in feature-map order, deduped. */
+export function touchedFeatures(diffPaths, features) {
+    const changed = diffPaths.filter((p) => !isNoiseDiffPath(p));
+    const out = [];
+    for (const f of features) {
+        const prefixes = (f.paths ?? []).map((p) => p.replace(/^\.\//, ""));
+        // Directory-boundary match: "src/app/admin" owns "src/app/admin/…" and the
+        // exact path itself, but never "src/app/admin-shop/…".
+        const owns = (prefix, path) => path === prefix || path.startsWith(prefix.endsWith("/") ? prefix : prefix + "/");
+        if (prefixes.some((prefix) => prefix && changed.some((path) => owns(prefix, path)))) {
+            out.push(f.name || f.key);
+        }
+    }
+    return out;
+}
+/** The deterministic layer under the per-feature evidence rule: a PR that
+ *  touches more than one feature needs at least one proof per feature. The
+ *  semantic judgment (which proof covers which feature) stays with the
+ *  reviewer; this only counts — conservatively, by comment. */
+export function assessEvidenceCoverage(touched, comments) {
+    const evidenceItems = comments.filter((c) => {
+        const body = c.body ?? "";
+        // Anchored: a structured caption line ("- Verified: …") or the 🧪 bundle
+        // header at line start. A bare substring match would count prose like
+        // "not yet verified:" or quoted captions and silently suppress the warning.
+        return /^\s*🧪.*Test evidence/m.test(body) || /^\s*-?\s*\*{0,2}verified:/im.test(body);
+    }).length;
+    if (touched.length <= 1 || evidenceItems >= touched.length) {
+        return { evidenceItems, warning: null };
+    }
+    return {
+        evidenceItems,
+        warning: `⚠️ ${touched.length} features touched, ${evidenceItems} evidence item(s) — ` +
+            `need ≥1 proof per feature on a multi-feature PR; treat each unproven feature ` +
+            `as an unresolved thread (request_changes) unless every touched feature maps ` +
+            `to a proof.`,
+    };
+}
+/** The "### Deviations from brief" section of a PR body — where the worker
+ *  logs off-brief pivots (field-guide pattern: deviations must be visible to
+ *  review, not buried in commit messages). Empty string when absent. */
+export function extractDeviations(prBody) {
+    const m = /^###\s*Deviations from brief\s*$/im.exec(prBody);
+    if (!m)
+        return "";
+    const rest = prBody.slice(m.index + m[0].length);
+    const end = rest.search(/^#{1,3}\s/m);
+    return (end >= 0 ? rest.slice(0, end) : rest).trim();
+}
+/** Render the full review packet as markdown. */
+export function buildReviewPacket(input) {
+    const { pr, threads, diff, issue } = input;
+    const b = [];
+    b.push(`# Review packet — PR #${pr.number}: ${pr.title}`);
+    const meta = [];
+    if (pr.headRefName)
+        meta.push(`${pr.headRefName} → ${pr.baseRefName ?? "?"}`);
+    if (pr.isDraft)
+        meta.push("DRAFT");
+    if (pr.mergeable)
+        meta.push(`mergeable: ${pr.mergeable}`);
+    if (pr.labels?.length)
+        meta.push(`labels: ${pr.labels.map((l) => l.name).join(", ")}`);
+    b.push(meta.join(" · "));
+    // Spec first — the reviewer judges against THIS, not against the diff.
+    b.push("\n## Spec / acceptance brief");
+    if (issue) {
+        b.push(`Issue #${issue.number} (${issue.linkKind}): ${issue.title}`);
+        const body = (issue.body ?? "").trim();
+        b.push(body.length > PACKET_BRIEF_CAP ? body.slice(0, PACKET_BRIEF_CAP) + "\n… (brief truncated)" : body || "_(issue has no body)_");
+    }
+    else {
+        b.push("⚠️ **No linked issue/brief found.** Do NOT infer the spec from the diff — " +
+            "reviewing against a self-derived spec is a known silent failure. Flag the missing brief in your verdict.");
+    }
+    const prBody = (pr.body ?? "").trim();
+    if (prBody) {
+        b.push("\n## PR description");
+        b.push(prBody.length > PACKET_BRIEF_CAP ? prBody.slice(0, PACKET_BRIEF_CAP) + "\n… (truncated)" : prBody);
+    }
+    const deviations = extractDeviations(pr.body ?? "");
+    if (deviations) {
+        b.push("\n## Deviations from brief");
+        b.push(deviations);
+        b.push("_Verify each deviation: conservative? justified? does the spec still hold?_");
+    }
+    b.push("\n## CI");
+    const checks = pr.statusCheckRollup ?? [];
+    if (checks.length === 0) {
+        b.push("no checks reported");
+    }
+    else {
+        let pass = 0, fail = 0, pending = 0;
+        const failing = [];
+        for (const c of checks) {
+            const concl = (c.conclusion ?? c.state ?? "").toUpperCase();
+            if (concl === "SUCCESS" || concl === "NEUTRAL" || concl === "SKIPPED")
+                pass++;
+            else if (concl === "" || concl === "PENDING" || (c.status ?? "").toUpperCase() === "IN_PROGRESS")
+                pending++;
+            else {
+                fail++;
+                failing.push(c.name ?? "unnamed");
+            }
+        }
+        b.push(`${pass} passing · ${fail} failing · ${pending} pending${failing.length ? ` — failing: ${failing.join(", ")}` : ""}`);
+    }
+    const unresolved = threads.filter((t) => !t.isResolved);
+    b.push(`\n## External review threads (unresolved: ${unresolved.length})`);
+    if (unresolved.length === 0) {
+        b.push("none");
+    }
+    else {
+        for (const t of unresolved.slice(0, 20)) {
+            const anchor = t.path ? `${t.path}${t.line ? `:${t.line}` : ""}` : "(top-level)";
+            b.push(`- ${anchor} @${t.author || "unknown"} — ${t.body.replace(/\s+/g, " ").slice(0, 140)}`);
+        }
+    }
+    const evidence = extractEvidenceLines(pr.comments ?? []);
+    b.push("\n## Evidence / health");
+    if (input.features?.length) {
+        const touched = touchedFeatures(splitUnifiedDiff(diff).map((s) => s.path), input.features);
+        if (touched.length) {
+            b.push(`Features touched (${touched.length}): ${touched.join(", ")}`);
+            const cov = assessEvidenceCoverage(touched, pr.comments ?? []);
+            if (cov.warning)
+                b.push(cov.warning);
+        }
+    }
+    b.push(evidence.length ? evidence.join("\n") : "no evidence caption posted");
+    // Features (relevant slice) — kills the reviewer's mandatory full-map pull:
+    // touched features in detail, same-layer neighbors by name, full map only
+    // on demand (issue #83).
+    if (input.features?.length) {
+        const diffPaths = splitUnifiedDiff(diff).map((s) => s.path);
+        const touchedNames = new Set(touchedFeatures(diffPaths, input.features));
+        if (touchedNames.size) {
+            const touched = input.features.filter((f) => touchedNames.has(f.name || f.key)).slice(0, 12);
+            b.push("\n## Features (relevant slice)");
+            for (const f of touched) {
+                const layer = f.layer ? ` [${f.layer}]` : "";
+                const tp = f.testPriority ? ` · test_priority: ${f.testPriority}` : "";
+                const desc = f.description ? ` — ${f.description}` : "";
+                b.push(`- ${f.name || f.key}${layer}${tp}${desc}`);
+            }
+            const layers = new Set(touched.map((f) => f.layer).filter(Boolean));
+            const neighbors = input.features
+                .filter((f) => !touchedNames.has(f.name || f.key) && f.layer && layers.has(f.layer))
+                .map((f) => f.name || f.key)
+                .slice(0, 15);
+            if (neighbors.length)
+                b.push(`Same-layer neighbors: ${neighbors.join(", ")}`);
+            b.push("_This slice replaces the full map for most reviews — run `renaiss-shipflow features --json` only if you need beyond it._");
+        }
+    }
+    const filtered = filterDiffForPacket(diff);
+    b.push(`\n## Diff (${filtered.shown} file(s) shown` +
+        (filtered.omittedNoise ? `, ${filtered.omittedNoise} noise file(s) omitted` : "") +
+        (filtered.omittedBudget ? `, ${filtered.omittedBudget} over budget` : "") +
+        (filtered.truncatedFiles ? `, ${filtered.truncatedFiles} truncated` : "") +
+        ")");
+    b.push("```diff");
+    b.push(filtered.text);
+    b.push("```");
+    return b.join("\n");
+}
+//# sourceMappingURL=packet.js.map

package/dist/packet.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"packet.js","sourceRoot":"","sources":["../src/packet.ts"],"names":[],"mappings":"AAAA,2EAA2E;AAC3E,4EAA4E;AAC5E,yEAAyE;AACzE,EAAE;AACF,yEAAyE;AACzE,8EAA8E;AAC9E,8EAA8E;AAC9E,4EAA4E;AAI5E,wEAAwE;AACxE,0BAA0B;AAC1B,MAAM,CAAC,MAAM,mBAAmB,GAAG,IAAI,CAAC;AACxC,MAAM,CAAC,MAAM,gBAAgB,GAAG,KAAK,CAAC;AACtC,MAAM,CAAC,MAAM,gBAAgB,GAAG,IAAI,CAAC,CAAC,0GAA0G;AAEhJ,2EAA2E;AAC3E,+EAA+E;AAC/E,mCAAmC;AACnC,MAAM,gBAAgB,GAAG;IACvB,eAAe,EAAE,SAAS,EAAE,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,gBAAgB;IACzE,gBAAgB,EAAE,WAAW,EAAE,mBAAmB,EAAE,SAAS,EAAE,MAAM;CACtE,CAAC;AACF,MAAM,cAAc,GAAG;IACrB,OAAO,EAAE,MAAM,EAAE,mBAAmB,EAAE,WAAW,EAAE,SAAS,EAAE,UAAU;IACxE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM;IAChE,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,eAAe,EAAE,SAAS;CACxE,CAAC;AAEF,MAAM,UAAU,eAAe,CAAC,IAAY;IAC1C,MAAM,CAAC,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAC7B,OAAO,CACL,gBAAgB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC;QAC3C,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAC1C,CAAC;AACJ,CAAC;AAOD,kEAAkE;AAClE,MAAM,UAAU,gBAAgB,CAAC,IAAY;IAC3C,MAAM,QAAQ,GAAkB,EAAE,CAAC;IACnC,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,IAAI,OAAO,GAAuB,IAAI,CAAC;IACvC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,CAAC,GAAG,8BAA8B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACpD,IAAI,CAAC,EAAE,CAAC;YACN,IAAI,OAAO;gBAAE,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YACpC,OAAO,GAAG,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,CAAC;QACvC,CAAC;aAAM,IAAI,OAAO,EAAE,CAAC;YACnB,OAAO,CAAC,IAAI,IAAI,IAAI,GAAG,IAAI,CAAC;QAC9B,CAAC;IACH,CAAC;IACD,IAAI,OAAO;QAAE,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IACpC,OAAO,QAAQ,CAAC;AAClB,CAAC;AAUD,gEAAgE;AAChE,MAAM,UAAU,mBAAmB,CAAC,IAAY;IAC9C,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,YAAY,GAAG,CAAC,CAAC;IACrB,IAAI,aAAa,GAAG,CAAC,CAAC;IACtB,IAAI,cAAc,GAAG,CAAC,CAAC;IACvB,IAAI,KAAK,GAAG,CAAC,CAAC;IAEd,KAAK,MAAM,CAAC,IAAI,gBAAgB,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,IAAI,eAAe,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC;YAC5B,YAAY,EAAE,CAAC;YACf,SAAS;QACX,CAAC;QACD,IAAI,KAAK,IAAI,gBAAgB,EAAE,CAAC;YAC9B,aAAa,EAAE,CAAC;YAChB,SAAS;QACX,CAAC;QACD,IAAI,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC;QAClB,IAAI,IAAI,CAAC,MAAM,GAAG,mBAAmB,EAAE,CAAC;YACtC,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,mBAAmB,CAAC,GAAG,2BAA2B,CAAC;YACxE,cAAc,EAAE,CAAC;QACnB,CAAC;QACD,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACf,KAAK,IAAI,IAAI,CAAC,MAAM,CAAC;QACrB,KAAK,EAAE,CAAC;IACV,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,YAAY,EAAE,aAAa,EAAE,cAAc,EAAE,CAAC;AACtF,CAAC;AAED,yEAAyE;AACzE,MAAM,UAAU,oBAAoB,CAAC,QAAqB;IACxD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,MAAM,IAAI,GAAG,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC;QAC1B,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,eAAe,CAAC,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC;YAAE,SAAS;QAC1E,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACpC,IAAI,IAAI,CAAC,QAAQ,CAAC,eAAe,CAAC,IAAI,8BAA8B,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC,UAAU,CAAC,YAAY,CAAC,EAAE,CAAC;gBAC7J,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;YAC1B,CAAC;QACH,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;AAC5B,CAAC;AAYD;qCACqC;AACrC,MAAM,UAAU,eAAe,CAAC,SAAmB,EAAE,QAA6B;IAChF,MAAM,OAAO,GAAG,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,CAAC;IAC7D,MAAM,GAAG,GAAa,EAAE,CAAC;IACzB,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,MAAM,QAAQ,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,CAAC;QACpE,2EAA2E;QAC3E,uDAAuD;QACvD,MAAM,IAAI,GAAG,CAAC,MAAc,EAAE,IAAY,EAAE,EAAE,CAC5C,IAAI,KAAK,MAAM,IAAI,IAAI,CAAC,UAAU,CAAC,MAAM,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,GAAG,GAAG,CAAC,CAAC;QACnF,IAAI,QAAQ,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,MAAM,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC;YACpF,GAAG,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;QAC5B,CAAC;IACH,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAWD;;;+DAG+D;AAC/D,MAAM,UAAU,sBAAsB,CAAC,OAAiB,EAAE,QAAkC;IAC1F,MAAM,aAAa,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;QAC1C,MAAM,IAAI,GAAG,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC;QAC1B,yEAAyE;QACzE,sEAAsE;QACtE,4EAA4E;QAC5E,OAAO,wBAAwB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,6BAA6B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACzF,CAAC,CAAC,CAAC,MAAM,CAAC;IACV,IAAI,OAAO,CAAC,MAAM,IAAI,CAAC,IAAI,aAAa,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;QAC3D,OAAO,EAAE,aAAa,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;IAC1C,CAAC;IACD,OAAO;QACL,aAAa;QACb,OAAO,EACL,MAAM,OAAO,CAAC,MAAM,sBAAsB,aAAa,sBAAsB;YAC7E,+EAA+E;YAC/E,8EAA8E;YAC9E,aAAa;KAChB,CAAC;AACJ,CAAC;AAED;;wEAEwE;AACxE,MAAM,UAAU,iBAAiB,CAAC,MAAc;IAC9C,MAAM,CAAC,GAAG,oCAAoC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAC5D,IAAI,CAAC,CAAC;QAAE,OAAO,EAAE,CAAC;IAClB,MAAM,IAAI,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IACjD,MAAM,GAAG,GAAG,IAAI,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC;IACtC,OAAO,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC;AACvD,CAAC;AAaD,iDAAiD;AACjD,MAAM,UAAU,iBAAiB,CAAC,KAAkB;IAClD,MAAM,EAAE,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,GAAG,KAAK,CAAC;IAC3C,MAAM,CAAC,GAAa,EAAE,CAAC;IAEvB,CAAC,CAAC,IAAI,CAAC,yBAAyB,EAAE,CAAC,MAAM,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC,CAAC;IAC1D,MAAM,IAAI,GAAa,EAAE,CAAC;IAC1B,IAAI,EAAE,CAAC,WAAW;QAAE,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,WAAW,MAAM,EAAE,CAAC,WAAW,IAAI,GAAG,EAAE,CAAC,CAAC;IAC9E,IAAI,EAAE,CAAC,OAAO;QAAE,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;IACnC,IAAI,EAAE,CAAC,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,cAAc,EAAE,CAAC,SAAS,EAAE,CAAC,CAAC;IAC1D,IAAI,EAAE,CAAC,MAAM,EAAE,MAAM;QAAE,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACvF,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;IAEzB,uEAAuE;IACvE,CAAC,CAAC,IAAI,CAAC,8BAA8B,CAAC,CAAC;IACvC,IAAI,KAAK,EAAE,CAAC;QACV,CAAC,CAAC,IAAI,CAAC,UAAU,KAAK,CAAC,MAAM,KAAK,KAAK,CAAC,QAAQ,MAAM,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC;QACrE,MAAM,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;QACvC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM,GAAG,gBAAgB,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,GAAG,uBAAuB,CAAC,CAAC,CAAC,IAAI,IAAI,uBAAuB,CAAC,CAAC;IACvI,CAAC;SAAM,CAAC;QACN,CAAC,CAAC,IAAI,CACJ,4EAA4E;YAC1E,0GAA0G,CAC7G,CAAC;IACJ,CAAC;IAED,MAAM,MAAM,GAAG,CAAC,EAAE,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;IACtC,IAAI,MAAM,EAAE,CAAC;QACX,CAAC,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QAC9B,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,gBAAgB,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,gBAAgB,CAAC,GAAG,iBAAiB,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAC5G,CAAC;IAED,MAAM,UAAU,GAAG,iBAAiB,CAAC,EAAE,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC;IACpD,IAAI,UAAU,EAAE,CAAC;QACf,CAAC,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;QACrC,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACnB,CAAC,CAAC,IAAI,CAAC,6EAA6E,CAAC,CAAC;IACxF,CAAC;IAED,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAClB,MAAM,MAAM,GAAG,EAAE,CAAC,iBAAiB,IAAI,EAAE,CAAC;IAC1C,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACxB,CAAC,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAC;IAC/B,CAAC;SAAM,CAAC;QACN,IAAI,IAAI,GAAG,CAAC,EAAE,IAAI,GAAG,CAAC,EAAE,OAAO,GAAG,CAAC,CAAC;QACpC,MAAM,OAAO,GAAa,EAAE,CAAC;QAC7B,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;YACvB,MAAM,KAAK,GAAG,CAAC,CAAC,CAAC,UAAU,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC;YAC5D,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,SAAS;gBAAE,IAAI,EAAE,CAAC;iBACzE,IAAI,KAAK,KAAK,EAAE,IAAI,KAAK,KAAK,SAAS,IAAI,CAAC,CAAC,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC,WAAW,EAAE,KAAK,aAAa;gBAAE,OAAO,EAAE,CAAC;iBACvG,CAAC;gBAAC,IAAI,EAAE,CAAC;gBAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,IAAI,SAAS,CAAC,CAAC;YAAC,CAAC;QACrD,CAAC;QACD,CAAC,CAAC,IAAI,CAAC,GAAG,IAAI,cAAc,IAAI,cAAc,OAAO,WAAW,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,eAAe,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;IAC/H,CAAC;IAED,MAAM,UAAU,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;IACxD,CAAC,CAAC,IAAI,CAAC,6CAA6C,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC;IAC1E,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC5B,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IACjB,CAAC;SAAM,CAAC;QACN,KAAK,MAAM,CAAC,IAAI,UAAU,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC;YACxC,MAAM,MAAM,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC;YACjF,CAAC,CAAC,IAAI,CAAC,KAAK,MAAM,KAAK,CAAC,CAAC,MAAM,IAAI,SAAS,MAAM,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;QACjG,CAAC;IACH,CAAC;IAED,MAAM,QAAQ,GAAG,oBAAoB,CAAC,EAAE,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC;IACzD,CAAC,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAC;IACjC,IAAI,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,CAAC;QAC3B,MAAM,OAAO,GAAG,eAAe,CAAC,gBAAgB,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,EAAE,KAAK,CAAC,QAAQ,CAAC,CAAC;QAC3F,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;YACnB,CAAC,CAAC,IAAI,CAAC,qBAAqB,OAAO,CAAC,MAAM,MAAM,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACtE,MAAM,GAAG,GAAG,sBAAsB,CAAC,OAAO,EAAE,EAAE,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC;YAC/D,IAAI,GAAG,CAAC,OAAO;gBAAE,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC;QACvC,CAAC;IACH,CAAC;IACD,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,4BAA4B,CAAC,CAAC;IAE7E,4EAA4E;IAC5E,0EAA0E;IAC1E,yBAAyB;IACzB,IAAI,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,CAAC;QAC3B,MAAM,SAAS,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC5D,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,eAAe,CAAC,SAAS,EAAE,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC;QACzE,IAAI,YAAY,CAAC,IAAI,EAAE,CAAC;YACtB,MAAM,OAAO,GAAG,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAC7F,CAAC,CAAC,IAAI,CAAC,gCAAgC,CAAC,CAAC;YACzC,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;gBACxB,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;gBAC7C,MAAM,EAAE,GAAG,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,qBAAqB,CAAC,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBACvE,MAAM,IAAI,GAAG,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;gBACxD,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,GAAG,GAAG,KAAK,GAAG,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC;YACrD,CAAC;YACD,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC;YACpE,MAAM,SAAS,GAAG,KAAK,CAAC,QAAQ;iBAC7B,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,KAAK,IAAI,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;iBACnF,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,GAAG,CAAC;iBAC3B,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAChB,IAAI,SAAS,CAAC,MAAM;gBAAE,CAAC,CAAC,IAAI,CAAC,yBAAyB,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YAC9E,CAAC,CAAC,IAAI,CAAC,0HAA0H,CAAC,CAAC;QACrI,CAAC;IACH,CAAC;IAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,IAAI,CAAC,CAAC;IAC3C,CAAC,CAAC,IAAI,CACJ,cAAc,QAAQ,CAAC,KAAK,gBAAgB;QAC1C,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,KAAK,QAAQ,CAAC,YAAY,wBAAwB,CAAC,CAAC,CAAC,EAAE,CAAC;QACjF,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC,CAAC,KAAK,QAAQ,CAAC,aAAa,cAAc,CAAC,CAAC,CAAC,EAAE,CAAC;QACzE,CAAC,QAAQ,CAAC,cAAc,CAAC,CAAC,CAAC,KAAK,QAAQ,CAAC,cAAc,YAAY,CAAC,CAAC,CAAC,EAAE,CAAC;QACzE,GAAG,CACN,CAAC;IACF,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IAClB,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;IACtB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAEd,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACtB,CAAC"}

package/dist/review-contract.d.ts

@@ -0,0 +1,59 @@
+/** Unified verdict vocabulary. The loop reviewer's `approve`/`request_changes`/
+ *  `reject` plus the server's advisory `comment`. */
+export type Verdict = "approve" | "comment" | "request_changes" | "reject";
+/** Severity badge — byte-identical to the server's severityBadge(). */
+export declare function severityBadge(severity: string): string;
+/** CodeRabbit-style fix-size tag after the badge — mirrors the server. */
+export declare function effortTag(effort?: string): string;
+/** Verdict header, shared by the inline review body and any fallback comment. */
+export declare function verdictHeader(verdict: Verdict): string;
+export interface ReviewFinding {
+    path: string;
+    line: number;
+    severity: string;
+    issue: string;
+    why?: string;
+    fix?: string;
+    effort?: string;
+    suggestion?: string;
+    code?: string;
+}
+/** One finding's comment body — mirrors renderInlineFinding(): badge + effort +
+ *  issue, why, fix, and a committable ```suggestion block when present. */
+export declare function renderFindingBody(f: ReviewFinding): string;
+/** Repo-relative path in one canonical shape: forward slashes, no leading
+ *  "./". Applied to both diff keys and finding paths so `./src/x` / `src\\x`
+ *  still anchor. NOT a filesystem realpath — diff paths aren't fs paths. */
+export declare function normPath(p: string): string;
+/** Map of path → set of new-file line numbers that GitHub will accept a
+ *  RIGHT-side inline comment on (context + added lines). Mirrors the server's
+ *  annotatePatch line numbering so a bad anchor can't 422 the whole review. */
+export declare function diffAnchors(diff: string): Map<string, Set<number>>;
+/** Partition findings into ones that anchor to a real diff line (postable
+ *  inline) and ones that must fold into the summary body (bad line / file). */
+export declare function splitAnchorable(findings: ReviewFinding[], anchors: Map<string, Set<number>>): {
+    inline: ReviewFinding[];
+    body: ReviewFinding[];
+};
+export interface ReviewComment {
+    path: string;
+    line: number;
+    side: "RIGHT";
+    body: string;
+}
+export interface ReviewPayload {
+    event: "COMMENT";
+    body: string;
+    comments: ReviewComment[];
+}
+/** Build the GitHub "create review" payload: a COMMENT-event review (ShipFlow
+ *  reviews are advisory — approval authority stays with the label gate, so the
+ *  event is never APPROVE/REQUEST_CHANGES; the verdict lives in the header),
+ *  with on-diff findings as inline comments and the rest folded into the body. */
+export declare function buildReviewPayload(opts: {
+    summary: string;
+    verdict: Verdict;
+    findings: ReviewFinding[];
+    anchors: Map<string, Set<number>>;
+}): ReviewPayload;
+//# sourceMappingURL=review-contract.d.ts.map

package/dist/review-contract.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-contract.d.ts","sourceRoot":"","sources":["../src/review-contract.ts"],"names":[],"mappings":"AAKA;qDACqD;AACrD,MAAM,MAAM,OAAO,GAAG,SAAS,GAAG,SAAS,GAAG,iBAAiB,GAAG,QAAQ,CAAC;AAE3E,uEAAuE;AACvE,wBAAgB,aAAa,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAOtD;AAED,0EAA0E;AAC1E,wBAAgB,SAAS,CAAC,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,CAMjD;AAED,iFAAiF;AACjF,wBAAgB,aAAa,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAOtD;AAED,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,MAAM,CAAC;IACd,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAID;2EAC2E;AAC3E,wBAAgB,iBAAiB,CAAC,CAAC,EAAE,aAAa,GAAG,MAAM,CAS1D;AAED;;4EAE4E;AAC5E,wBAAgB,QAAQ,CAAC,CAAC,EAAE,MAAM,GAAG,MAAM,CAE1C;AAKD;;+EAE+E;AAC/E,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,GAAG,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAkBlE;AAED;+EAC+E;AAC/E,wBAAgB,eAAe,CAC7B,QAAQ,EAAE,aAAa,EAAE,EACzB,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,GAChC;IAAE,MAAM,EAAE,aAAa,EAAE,CAAC;IAAC,IAAI,EAAE,aAAa,EAAE,CAAA;CAAE,CAQpD;AAED,MAAM,WAAW,aAAa;IAAG,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,OAAO,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;CAAE;AAC3F,MAAM,WAAW,aAAa;IAAG,KAAK,EAAE,SAAS,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,aAAa,EAAE,CAAC;CAAE;AAE7F;;;kFAGkF;AAClF,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,OAAO,CAAC;IACjB,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,OAAO,EAAE,GAAG,CAAC,MAAM,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC;CACnC,GAAG,aAAa,CAqBhB"}

package/dist/review-contract.js

@@ -0,0 +1,129 @@
+// Canonical review contract shared by the loop reviewer (this CLI) — the
+// single source of truth for verdict vocabulary, severity badges, finding
+// rendering, and diff anchoring, mirroring the server's runner_pr_review.go so
+// the two reviewers post CONSISTENTLY (epic #96). Pure + testable; no I/O.
+/** Severity badge — byte-identical to the server's severityBadge(). */
+export function severityBadge(severity) {
+    switch (severity.trim().toLowerCase()) {
+        case "critical": return "🔴 **critical**";
+        case "high": return "🟠 **high**";
+        case "low": return "🔵 **low**";
+        default: return "🟡 **medium**";
+    }
+}
+/** CodeRabbit-style fix-size tag after the badge — mirrors the server. */
+export function effortTag(effort) {
+    switch ((effort ?? "").trim().toLowerCase()) {
+        case "quick": return " · ⚡ quick fix";
+        case "involved": return " · 🔨 involved";
+        default: return "";
+    }
+}
+/** Verdict header, shared by the inline review body and any fallback comment. */
+export function verdictHeader(verdict) {
+    switch (verdict) {
+        case "approve": return "**✅ APPROVE — ShipFlow review**";
+        case "request_changes": return "**🔴 CHANGES REQUESTED — ShipFlow review**";
+        case "reject": return "**⛔ REJECT — ShipFlow review**";
+        default: return "**💬 ShipFlow review — comments**";
+    }
+}
+const REVIEW_MARKER = "<!-- shipflow:loop-review -->";
+/** One finding's comment body — mirrors renderInlineFinding(): badge + effort +
+ *  issue, why, fix, and a committable ```suggestion block when present. */
+export function renderFindingBody(f) {
+    let b = `${severityBadge(f.severity)}${effortTag(f.effort)} ${f.issue}`;
+    if (f.why?.trim())
+        b += `\n\n${f.why.trim()}`;
+    if (f.fix?.trim())
+        b += `\n\n**Fix:** ${f.fix.trim()}`;
+    const sug = (f.suggestion ?? "").replace(/\n+$/, "");
+    const code = (f.code ?? "").replace(/\n+$/, "");
+    if (sug.trim())
+        b += `\n\`\`\`suggestion\n${sug}\n\`\`\``;
+    else if (code.trim())
+        b += `\n\`\`\`\n${code}\n\`\`\``;
+    return `${b}\n\n${REVIEW_MARKER}`;
+}
+/** Repo-relative path in one canonical shape: forward slashes, no leading
+ *  "./". Applied to both diff keys and finding paths so `./src/x` / `src\\x`
+ *  still anchor. NOT a filesystem realpath — diff paths aren't fs paths. */
+export function normPath(p) {
+    return p.replace(/\\/g, "/").replace(/^\.\//, "").trim();
+}
+const HUNK_RE = /^@@ -\d+(?:,\d+)? \+(\d+)(?:,\d+)? @@/;
+const FILE_RE = /^\+\+\+ b\/(.+)$/;
+/** Map of path → set of new-file line numbers that GitHub will accept a
+ *  RIGHT-side inline comment on (context + added lines). Mirrors the server's
+ *  annotatePatch line numbering so a bad anchor can't 422 the whole review. */
+export function diffAnchors(diff) {
+    const out = new Map();
+    let path = "";
+    let ln = 0;
+    for (const raw of diff.split(/\r?\n/)) {
+        const fm = FILE_RE.exec(raw);
+        if (fm) {
+            path = normPath(fm[1]);
+            ln = 0;
+            continue;
+        }
+        const hm = HUNK_RE.exec(raw);
+        if (hm) {
+            ln = parseInt(hm[1], 10);
+            continue;
+        }
+        if (!path || ln === 0)
+            continue;
+        if (raw.startsWith("-"))
+            continue; // removed: no RIGHT-side line
+        if (raw.startsWith("+") || raw.startsWith(" ")) {
+            if (!out.has(path))
+                out.set(path, new Set());
+            out.get(path).add(ln);
+            ln++;
+        }
+    }
+    return out;
+}
+/** Partition findings into ones that anchor to a real diff line (postable
+ *  inline) and ones that must fold into the summary body (bad line / file). */
+export function splitAnchorable(findings, anchors) {
+    const inline = [];
+    const body = [];
+    for (const f of findings) {
+        if (f.line > 0 && anchors.get(normPath(f.path))?.has(f.line))
+            inline.push(f);
+        else
+            body.push(f);
+    }
+    return { inline, body };
+}
+/** Build the GitHub "create review" payload: a COMMENT-event review (ShipFlow
+ *  reviews are advisory — approval authority stays with the label gate, so the
+ *  event is never APPROVE/REQUEST_CHANGES; the verdict lives in the header),
+ *  with on-diff findings as inline comments and the rest folded into the body. */
+export function buildReviewPayload(opts) {
+    const { inline, body: unanchored } = splitAnchorable(opts.findings, opts.anchors);
+    const lines = [verdictHeader(opts.verdict)];
+    if (opts.summary.trim())
+        lines.push("", opts.summary.trim());
+    if (unanchored.length) {
+        lines.push("", "**Further findings (outside the annotated diff lines):**");
+        for (const f of unanchored) {
+            const anchor = f.line > 0 ? `${f.path}:${f.line}` : f.path;
+            const fbLines = renderFindingBody(f).split("\n\n" + REVIEW_MARKER)[0].split("\n");
+            lines.push(`- \`${anchor}\` — ${fbLines[0]}`);
+            // Indent why/fix/suggestion continuation two spaces so GitHub keeps them
+            // nested under the bullet instead of rendering as top-level paragraphs.
+            for (let i = 1; i < fbLines.length; i++)
+                lines.push(fbLines[i] ? `  ${fbLines[i]}` : "");
+        }
+    }
+    lines.push("", REVIEW_MARKER);
+    return {
+        event: "COMMENT",
+        body: lines.join("\n"),
+        comments: inline.map((f) => ({ path: f.path, line: f.line, side: "RIGHT", body: renderFindingBody(f) })),
+    };
+}
+//# sourceMappingURL=review-contract.js.map

package/dist/review-contract.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-contract.js","sourceRoot":"","sources":["../src/review-contract.ts"],"names":[],"mappings":"AAAA,yEAAyE;AACzE,0EAA0E;AAC1E,+EAA+E;AAC/E,2EAA2E;AAM3E,uEAAuE;AACvE,MAAM,UAAU,aAAa,CAAC,QAAgB;IAC5C,QAAQ,QAAQ,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,EAAE,CAAC;QACtC,KAAK,UAAU,CAAC,CAAC,OAAO,iBAAiB,CAAC;QAC1C,KAAK,MAAM,CAAC,CAAC,OAAO,aAAa,CAAC;QAClC,KAAK,KAAK,CAAC,CAAC,OAAO,YAAY,CAAC;QAChC,OAAO,CAAC,CAAC,OAAO,eAAe,CAAC;IAClC,CAAC;AACH,CAAC;AAED,0EAA0E;AAC1E,MAAM,UAAU,SAAS,CAAC,MAAe;IACvC,QAAQ,CAAC,MAAM,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,EAAE,CAAC;QAC5C,KAAK,OAAO,CAAC,CAAC,OAAO,gBAAgB,CAAC;QACtC,KAAK,UAAU,CAAC,CAAC,OAAO,gBAAgB,CAAC;QACzC,OAAO,CAAC,CAAC,OAAO,EAAE,CAAC;IACrB,CAAC;AACH,CAAC;AAED,iFAAiF;AACjF,MAAM,UAAU,aAAa,CAAC,OAAgB;IAC5C,QAAQ,OAAO,EAAE,CAAC;QAChB,KAAK,SAAS,CAAC,CAAC,OAAO,iCAAiC,CAAC;QACzD,KAAK,iBAAiB,CAAC,CAAC,OAAO,4CAA4C,CAAC;QAC5E,KAAK,QAAQ,CAAC,CAAC,OAAO,gCAAgC,CAAC;QACvD,OAAO,CAAC,CAAC,OAAO,mCAAmC,CAAC;IACtD,CAAC;AACH,CAAC;AAcD,MAAM,aAAa,GAAG,+BAA+B,CAAC;AAEtD;2EAC2E;AAC3E,MAAM,UAAU,iBAAiB,CAAC,CAAgB;IAChD,IAAI,CAAC,GAAG,GAAG,aAAa,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,CAAC;IACxE,IAAI,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE;QAAE,CAAC,IAAI,OAAO,CAAC,CAAC,GAAG,CAAC,IAAI,EAAE,EAAE,CAAC;IAC9C,IAAI,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE;QAAE,CAAC,IAAI,gBAAgB,CAAC,CAAC,GAAG,CAAC,IAAI,EAAE,EAAE,CAAC;IACvD,MAAM,GAAG,GAAG,CAAC,CAAC,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IACrD,MAAM,IAAI,GAAG,CAAC,CAAC,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IAChD,IAAI,GAAG,CAAC,IAAI,EAAE;QAAE,CAAC,IAAI,uBAAuB,GAAG,UAAU,CAAC;SACrD,IAAI,IAAI,CAAC,IAAI,EAAE;QAAE,CAAC,IAAI,aAAa,IAAI,UAAU,CAAC;IACvD,OAAO,GAAG,CAAC,OAAO,aAAa,EAAE,CAAC;AACpC,CAAC;AAED;;4EAE4E;AAC5E,MAAM,UAAU,QAAQ,CAAC,CAAS;IAChC,OAAO,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;AAC3D,CAAC;AAED,MAAM,OAAO,GAAG,uCAAuC,CAAC;AACxD,MAAM,OAAO,GAAG,kBAAkB,CAAC;AAEnC;;+EAE+E;AAC/E,MAAM,UAAU,WAAW,CAAC,IAAY;IACtC,MAAM,GAAG,GAAG,IAAI,GAAG,EAAuB,CAAC;IAC3C,IAAI,IAAI,GAAG,EAAE,CAAC;IACd,IAAI,EAAE,GAAG,CAAC,CAAC;IACX,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,EAAE,CAAC;QACtC,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC7B,IAAI,EAAE,EAAE,CAAC;YAAC,IAAI,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;YAAC,EAAE,GAAG,CAAC,CAAC;YAAC,SAAS;QAAC,CAAC;QACrD,MAAM,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC7B,IAAI,EAAE,EAAE,CAAC;YAAC,EAAE,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YAAC,SAAS;QAAC,CAAC;QAC/C,IAAI,CAAC,IAAI,IAAI,EAAE,KAAK,CAAC;YAAE,SAAS;QAChC,IAAI,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC;YAAE,SAAS,CAAe,8BAA8B;QAC/E,IAAI,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC,UAAU,CAAC,GAAG,CAAC,EAAE,CAAC;YAC/C,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC;gBAAE,GAAG,CAAC,GAAG,CAAC,IAAI,EAAE,IAAI,GAAG,EAAE,CAAC,CAAC;YAC7C,GAAG,CAAC,GAAG,CAAC,IAAI,CAAE,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;YACvB,EAAE,EAAE,CAAC;QACP,CAAC;IACH,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC;AAED;+EAC+E;AAC/E,MAAM,UAAU,eAAe,CAC7B,QAAyB,EACzB,OAAiC;IAEjC,MAAM,MAAM,GAAoB,EAAE,CAAC;IACnC,MAAM,IAAI,GAAoB,EAAE,CAAC;IACjC,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QACzB,IAAI,CAAC,CAAC,IAAI,GAAG,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC;YAAE,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;;YACxE,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IACpB,CAAC;IACD,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;AAC1B,CAAC;AAKD;;;kFAGkF;AAClF,MAAM,UAAU,kBAAkB,CAAC,IAKlC;IACC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,UAAU,EAAE,GAAG,eAAe,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IAClF,MAAM,KAAK,GAAa,CAAC,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC;IACtD,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE;QAAE,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC;IAC7D,IAAI,UAAU,CAAC,MAAM,EAAE,CAAC;QACtB,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,0DAA0D,CAAC,CAAC;QAC3E,KAAK,MAAM,CAAC,IAAI,UAAU,EAAE,CAAC;YAC3B,MAAM,MAAM,GAAG,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YAC3D,MAAM,OAAO,GAAG,iBAAiB,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,MAAM,GAAG,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;YAClF,KAAK,CAAC,IAAI,CAAC,OAAO,MAAM,QAAQ,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YAC9C,yEAAyE;YACzE,wEAAwE;YACxE,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,EAAE;gBAAE,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QAC3F,CAAC;IACH,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,aAAa,CAAC,CAAC;IAC9B,OAAO;QACL,KAAK,EAAE,SAAS;QAChB,IAAI,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC;QACtB,QAAQ,EAAE,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,iBAAiB,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;KACzG,CAAC;AACJ,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@renaiss-shipflow/cli",
-  "version": "0.9.0",
+  "version": "0.11.0",
   "description": "CLI tool for RenaissShipFlow - AI-powered project management automation",
   "type": "module",
   "bin": {