@bridge_gpt/mcp-server 0.2.68 → 0.2.69

package/build/executor/completion-contract.js

@@ -235,6 +235,8 @@ export const END_CAUSE_VALUES = [
     "worker_exit", "completion_window", "job_timeout", "executor_abort",
     "executor_shutdown", "server_stop", "lease_lost",
 ];
+/** BAPI-1421 (F4) — the end causes an executor may report (`lease_lost` is server-derived). */
+export const EXECUTOR_REPORTABLE_END_CAUSE_VALUES = END_CAUSE_VALUES.filter((value) => value !== "lease_lost");
 export const MAX_TASKS = 64;
 export const MAX_DENIALS = 32;
 export const MAX_IDENTIFIER_CHARS = 128;

package/build/executor/http-client.js

@@ -8,6 +8,7 @@
  * logged and never included in any error string.
  */
 import { RECONCILER_LIVENESS_VALUES } from "./types.js";
+import { EXECUTOR_REPORTABLE_END_CAUSE_VALUES, } from "./completion-contract.js";
 import { readEpicRunCompletionState, } from "../conductor/bridge-api-client.js";
 /** BAPI-1258 — the server's stable `error_code` for refused capacity evidence. */
 export const UNSUPPORTED_TERMINAL_CAPACITY_ERROR_CODE = "UNSUPPORTED_TERMINAL_CAPACITY";
@@ -92,6 +93,41 @@ export function normalizePriorAttemptHeadSha(value, onMalformed) {
     onMalformed();
     return null;
 }
+/**
+ * BAPI-1421 (F4) — normalize a claim's `end_cause_capability` to the recognized
+ * executor-reportable values. Absent, null, or malformed reads as an empty list
+ * (an older server), and an unknown string is dropped rather than trusted.
+ */
+export function normalizeEndCauseCapability(value) {
+    if (!Array.isArray(value))
+        return [];
+    const allowed = new Set(EXECUTOR_REPORTABLE_END_CAUSE_VALUES);
+    const out = [];
+    for (const entry of value) {
+        if (allowed.has(entry) && !out.includes(entry)) {
+            out.push(entry);
+        }
+    }
+    return out;
+}
+/**
+ * BAPI-1421 (F4) — the completion envelope a claim may carry. A copy that keeps
+ * `end_cause` only when the claim's advertised list holds that exact value; an
+ * unadvertised cause is removed, never sent as null or rewritten. Every other
+ * evidence field is unchanged. An older server's strict `extra="forbid"`
+ * envelope would 422 an unknown key, so this is what keeps a newer executor
+ * wire-compatible with it.
+ */
+export function envelopeForClaim(job, envelope) {
+    if (!("end_cause" in envelope))
+        return envelope;
+    const allowed = normalizeEndCauseCapability(job.end_cause_capability);
+    if (envelope.end_cause !== undefined && allowed.includes(envelope.end_cause)) {
+        return envelope;
+    }
+    const { end_cause: _omitted, ...rest } = envelope;
+    return rest;
+}
 /** Bound a server-provided diagnostic snippet included in error strings. */
 function boundedDetail(text) {
     const trimmed = (text ?? "").trim();
@@ -207,13 +243,13 @@ function toConductorGetFetch(fetchLike) {
  * `undefined`/`null`) keeps a legacy terminal body byte-identical, which an
  * older server's strict `extra="forbid"` schema depends on.
  */
-function completionContractFields(report) {
+function completionContractFields(job, report) {
     return {
         ...(report.completion_contract !== undefined
             ? { completion_contract: report.completion_contract }
             : {}),
         ...(report.completion_evidence !== undefined
-            ? { completion_evidence: report.completion_evidence }
+            ? { completion_evidence: envelopeForClaim(job, report.completion_evidence) }
             : {}),
     };
 }
@@ -289,6 +325,9 @@ export function createExecutorHttpClient(config) {
                     }
                     // BAPI-1367: a malformed prior head fails closed to null with one
                     // bounded warning naming only the job, never the raw value or a SHA.
+                    // BAPI-1421 (F4): only recognized executor-reportable end causes are
+                    // kept; an older server's absent key becomes an empty list.
+                    job.end_cause_capability = normalizeEndCauseCapability(job.end_cause_capability);
                     job.prior_attempt_head_sha = normalizePriorAttemptHeadSha(job.prior_attempt_head_sha, () => (config.warn ?? ((message) => console.error(message)))(`executor: ignoring malformed prior attempt head in the claim for job ${String(job.id)}; ` +
                         `treating it as a first dispatch.`));
                     return { kind: "claimed", job, reconcilerLiveness };
@@ -392,7 +431,7 @@ export function createExecutorHttpClient(config) {
                 local_commit_count: completion.local_commit_count,
                 last_commit_sha: completion.last_commit_sha,
                 telemetry: completion.telemetry,
-                ...completionContractFields(completion),
+                ...completionContractFields(job, completion),
             };
             return mutate(`/${job.id}/complete`, body, true, job.repo_name);
         },
@@ -409,7 +448,7 @@ export function createExecutorHttpClient(config) {
                 local_commit_count: failure.local_commit_count,
                 last_commit_sha: failure.last_commit_sha,
                 telemetry: failure.telemetry,
-                ...completionContractFields(failure),
+                ...completionContractFields(job, failure),
             };
             if (failure.capacity_evidence === undefined) {
                 return mutate(`/${job.id}/fail`, body, false, job.repo_name);