@bridge_gpt/mcp-server 0.2.34 → 0.2.37

package/build/executor/http-client.js

@@ -12,6 +12,28 @@ function isOkBody(bodyText) {
         return false;
     }
 }
+/**
+ * The server-published run-level stop field on a heartbeat response (BAPI-731,
+ * WS-B). Owned by the reconciler safety-plane ticket; until it ships the field is
+ * simply never present.
+ */
+export const HEARTBEAT_STOP_FIELD = "stop_requested";
+/**
+ * True ONLY for a literal `true` on the stop field. Anything else — absent,
+ * `false`, a string, a truthy non-boolean, or an unparseable body — means "no
+ * stop was requested", so an ambiguous body can never cancel a healthy run.
+ */
+function parseStopRequested(bodyText) {
+    try {
+        const parsed = JSON.parse(bodyText);
+        if (!parsed || typeof parsed !== "object")
+            return false;
+        return parsed[HEARTBEAT_STOP_FIELD] === true;
+    }
+    catch {
+        return false;
+    }
+}
 /**
  * Map a mutation (heartbeat/complete/fail) HTTP response onto the fencing
  * outcome. `allowInvalidResult` maps 422 to `invalid_job_result` (complete only).
@@ -92,14 +114,33 @@ export function createExecutorHttpClient(config) {
             // 5xx and any other unexpected status are retryable.
             return { kind: "retryable", error: `claim failed (HTTP ${status}): ${boundedDetail(text)}` };
         },
-        heartbeat(job, payload) {
-            return mutate(`/${job.id}/heartbeat`, {
-                repo_name: job.repo_name,
-                claim_token: job.claim_token,
-                local_commit_count: payload.local_commit_count,
-                last_commit_sha: payload.last_commit_sha,
-                telemetry: payload.telemetry,
-            }, false, job.repo_name);
+        // Heartbeat does NOT go through `mutate()`: it is the one mutation whose
+        // response BODY carries information beyond the outcome (the BAPI-731 stop
+        // request), so it maps the response itself instead of discarding the body.
+        async heartbeat(job, payload) {
+            let status;
+            let text;
+            try {
+                const res = await post(`/${job.id}/heartbeat`, {
+                    repo_name: job.repo_name,
+                    claim_token: job.claim_token,
+                    local_commit_count: payload.local_commit_count,
+                    last_commit_sha: payload.last_commit_sha,
+                    telemetry: payload.telemetry,
+                }, job.repo_name);
+                status = res.status;
+                text = res.text;
+            }
+            catch {
+                // Transient network error — identical to the previous behavior; the
+                // caller's dead-man logic decides whether to keep the worker alive.
+                return { outcome: "fatal_http_error" };
+            }
+            const outcome = mapMutationResponse(status, text, false);
+            const result = { outcome };
+            if (parseStopRequested(text))
+                result.stop_requested = true;
+            return result;
         },
         complete(job, completion) {
             return mutate(`/${job.id}/complete`, {

package/build/executor/index.js

@@ -7,4 +7,8 @@
  */
 export { runExecutorCli, getExecutorUsage, parseExecutorArgs } from "./cli.js";
 export { runExecutor } from "./runner.js";
+// Persistent-service packaging (BAPI-688). The CLI entry plus the PURE helpers
+// doctor needs to enumerate and inspect generated units — doctor must share this
+// module's naming/format contract instead of maintaining a second parser.
+export { runExecutorInstallServiceCli, getExecutorInstallServiceUsage, buildExecutorServiceArtifact, writeExecutorServiceArtifact, inspectExecutorServiceArtifact, executorLaunchdDirForHome, executorSystemdDirForHome, executorIdFromLaunchdFilename, executorIdFromSystemdFilename, EXECUTOR_SERVICE_BASE_URL_ENV, EXECUTOR_SERVICE_PATH_ENV, } from "./service-unit.js";
 export { runExecutorWatchCli, getExecutorWatchUsage, parseExecutorWatchArgs, } from "./watch-cli.js";

package/build/executor/job-errors.js

@@ -4,6 +4,10 @@ export const WorktreeLostBeforePush = "WorktreeLostBeforePush";
 export const BranchMismatch = "BranchMismatch";
 export const WorkerFinalizationMissingRemoteBranchAndPr = "WorkerFinalizationMissingRemoteBranchAndPr";
 export const WorkerFinalizationPrBaseMismatch = "WorkerFinalizationPrBaseMismatch";
+export const WorkerFinalizationSavedButUnfinalized = "WorkerFinalizationSavedButUnfinalized";
+export const WorkerConfigIsolation = "ContractError.WorkerConfigIsolation";
+export const PreSpawnVerification = "ContractError.PreSpawnVerification";
+export const BridgeCredentialUnavailable = "ContractError.BridgeCredentialUnavailable";
 /** Bound a failure message so no unbounded/secret-bearing text is posted. */
 const ERROR_MESSAGE_MAX_CHARS = 300;
 /**