@sagentlab/navarch-runtime 0.1.55 → 0.1.56

package/dist/exit-conditions.cjs

@@ -6,6 +6,7 @@ exports.parseCodexJsonEvents = parseCodexJsonEvents;
 exports.extractUsageFromCodexEvents = extractUsageFromCodexEvents;
 exports.extractFinalMessageFromCodexEvents = extractFinalMessageFromCodexEvents;
 exports.leadingVerificationVerdict = leadingVerificationVerdict;
+exports.leadWithReviewFixMarker = leadWithReviewFixMarker;
 exports.mapExitCondition = mapExitCondition;
 /**
  * Best-effort parse of `claude -p --output-format json` stdout into the
@@ -201,6 +202,26 @@ function leadingVerificationVerdict(report) {
     const match = /^Verification verdict:\s*(PASS|FAIL|BLOCKED)\b/i.exec(firstLine ?? "");
     return match?.[1] ? match[1].toLowerCase() : null;
 }
+const REVIEW_FIX_MARKER_LINE = /^[ \t]*Review fix delivered: [a-f0-9]{40}(?=\s|$)/im;
+/**
+ * Moves a `Review fix delivered: <sha>` line to the top of the report. The
+ * control plane (lib/navarch/review-fix-marker.ts) accepts the marker at the
+ * start of any line, but summarize() collapses whitespace and truncates, so a
+ * marker below a sentence of preamble ends up mid-sentence or cut off and the
+ * review fix is rejected. Kept in step with leadWithReviewFixMarker there;
+ * this package cannot import it. A report without a marker is unchanged.
+ */
+function leadWithReviewFixMarker(report) {
+    const marker = REVIEW_FIX_MARKER_LINE.exec(report);
+    if (!marker || !report.slice(0, marker.index).trim())
+        return report;
+    const lineEnd = report.indexOf("\n", marker.index);
+    const line = report.slice(marker.index, lineEnd === -1 ? undefined : lineEnd).trim();
+    const rest = (report.slice(0, marker.index) + (lineEnd === -1 ? "" : report.slice(lineEnd + 1)))
+        .replace(/\n{3,}/g, "\n\n")
+        .trim();
+    return rest ? `${line}\n\n${rest}` : line;
+}
 function summarize(text, maxLen = 500) {
     const trimmed = text.trim();
     if (!trimmed)
@@ -292,7 +313,7 @@ function mapExitCondition(result) {
     // falls back to raw stdout when neither is available (see
     // parseClaudeJsonResult's doc comment).
     const reportText = result.reportText ?? parsedJson?.result ?? result.stdout;
-    const reportSummary = summarize(reportText) || "Adapter completed with no report text.";
+    const reportSummary = summarize(leadWithReviewFixMarker(reportText)) || "Adapter completed with no report text.";
     // A verify agent's FAIL/BLOCKED verdict is a successful agent run whose
     // conclusion is that the delivery did not pass. The completion API only
     // accepts those verdicts with a failed outcome, so map them to a failed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sagentlab/navarch-runtime",
-  "version": "0.1.55",
+  "version": "0.1.56",
   "description": "Navarch machine-side session manager: claims delivery tasks and runs them through local coding agents.",
   "type": "commonjs",
   "license": "MIT",