@bridge_gpt/mcp-server 0.2.62 → 0.2.65

package/build/conductor/bridge-api-client.js

@@ -16,6 +16,7 @@ import { readEpicLaunchProfile, resolveEpicLaunchTarget, } from "../epic-launch-
 import { resolveStartTicketsRepoName } from "../start-tickets-repo.js";
 import { normalizePrNumber, normalizeSha } from "./git-ci-types.js";
 import { ConductorValidationError } from "./errors.js";
+import { parseContinuationStatusResult, parseContinueParkedChildResult, } from "./epic-child-continuation.js";
 import { resolveDeclaredRunBaseBranch } from "./run-branch.js";
 /** Default Bridge API base URL when `BAPI_BASE_URL` is unset. */
 export const CONDUCTOR_DEFAULT_BASE_URL = "https://bridgegpt-api.com";
@@ -70,12 +71,15 @@ export async function resolveConductorBridgeApiAccess(deps = {}) {
  *
  *   1. the selected profile is read and validated (host-local, read-only);
  *   2. the TARGET is decided from `--base-url`, the profile, and `BAPI_BASE_URL`
- *      — conflicts refuse, and an absent target is refused, never defaulted to
- *      {@link CONDUCTOR_DEFAULT_BASE_URL};
+ *      — conflicts refuse. With none of them set, the target is
+ *      {@link CONDUCTOR_DEFAULT_BASE_URL} (BAPI-1272);
  *   3. the repository is decided (profile binding, then `--repo`, then the
  *      shared resolver);
  *   4. only then is a credential looked up, coupled to that target AND that
- *      repository ({@link resolveEpicLaunchCredential}).
+ *      repository ({@link resolveEpicLaunchCredential}). A stored key with no
+ *      binding is accepted only when the target is production; a key bound to
+ *      another endpoint, or an ambient key without a matching `BAPI_BASE_URL`,
+ *      is still refused.
  *
  * Nothing is sent anywhere by this function. {@link resolveConductorBridgeApiAccess}
  * is deliberately left as it is for every other client.
@@ -104,6 +108,7 @@ export async function resolveEpicLaunchBridgeApiAccess(selection, deps = {}) {
         flagRepo: deps.repoName,
         profile,
         env,
+        defaultBaseUrl: CONDUCTOR_DEFAULT_BASE_URL,
     });
     if (!target.ok)
         return { ok: false, kind: target.kind, error: target.message };
@@ -119,13 +124,7 @@ export async function resolveEpicLaunchBridgeApiAccess(selection, deps = {}) {
     }
     let cred;
     try {
-        cred = await resolveEpicLaunchCredential(repoName, target.baseUrl, {
-            env,
-            homedir,
-            platform,
-            readFile: readFileImpl,
-            stat: statImpl,
-        });
+        cred = await resolveEpicLaunchCredential(repoName, target.baseUrl, { env, homedir, platform, readFile: readFileImpl, stat: statImpl }, { productionBaseUrl: CONDUCTOR_DEFAULT_BASE_URL });
     }
     catch {
         return { ok: false, kind: "credentials-unavailable", error: "failed to resolve Bridge API credentials" };
@@ -750,6 +749,18 @@ export async function remediateEpicTicket(access, request, fetchImpl = globalThi
         throw err;
     }
 }
+/**
+ * BAPI-1310 (E1) — the server's refusal codes for the unexplained review-head
+ * stop. Each is an HTTP 409 that leaves the ticket parked.
+ */
+export const REVIEW_HEAD_REFUSAL_CODES = [
+    "REVIEW_HEAD_ADOPTION_REQUIRED",
+    "REVIEW_HEAD_EPISODE_CONFLICT",
+    "REVIEW_HEAD_STATE_INVALID",
+    "REVIEW_HEAD_BINDING_CHANGED",
+    "REVIEW_HEAD_LIVE_WORK",
+    "REVIEW_HEAD_UNAVAILABLE",
+];
 // ---------------------------------------------------------------------------
 // Local boundary validators (reject before any fetch call)
 // ---------------------------------------------------------------------------
@@ -1251,6 +1262,108 @@ export async function unparkEpicTicket(access, request, fetchImpl = globalThis.f
     });
     return postUnparkLikeRequest(url, conductorPostHeaders(access), body, fetchImpl);
 }
+const TYPED_CONTINUATION_STATUSES = new Set([200, 201, 409, 503]);
+const SAFE_ERROR_CODE_RE = /^[A-Za-z0-9_.:-]{1,64}$/;
+function continuationChildPath(epicRunId, ticketKey) {
+    return `${epicRunApiPath(epicRunId)}/tickets/${encodeURIComponent(ticketKey)}`;
+}
+function safeDetailErrorCode(body) {
+    if (body === null || typeof body !== "object")
+        return null;
+    const detail = body.detail;
+    if (detail === null || typeof detail !== "object")
+        return null;
+    const code = detail.error_code;
+    return typeof code === "string" && SAFE_ERROR_CODE_RE.test(code) ? code : null;
+}
+/**
+ * POST `.../tickets/{ticket_key}/continue` exactly once and classify the answer.
+ *
+ * Never throws for an HTTP or transport outcome — every one of them is returned
+ * as a {@link ContinuationPostOutcome}, so no caller can accidentally turn a
+ * timeout into "refused" by catching an exception. Never falls back to unpark.
+ * The API key travels only in the header; the body carries no credential and no
+ * caller-asserted authority.
+ */
+export async function postContinueParkedChild(access, request, fetchImpl = globalThis.fetch, timeoutMs = CONDUCTOR_FETCH_TIMEOUT_MS) {
+    const b = request.bindings;
+    requireNonEmptyString(b.epicRunId);
+    requireNonEmptyString(b.ticketKey);
+    const url = buildConductorJiraUrl(access.baseUrl, `${continuationChildPath(b.epicRunId, b.ticketKey)}/continue`);
+    const body = JSON.stringify({
+        repo_name: b.repoName,
+        request_id: request.requestId,
+        parked_gate: b.parkedGate,
+        expected_row_version: b.expectedRowVersion,
+        expected_head_sha: b.expectedHeadSha,
+        source_park_generation: b.sourceParkGeneration,
+        request_fingerprint: request.requestFingerprint,
+        ...(request.reason ? { reason: request.reason } : {}),
+    });
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(), timeoutMs);
+    try {
+        let resp;
+        try {
+            resp = await fetchImpl(url, {
+                method: "POST",
+                headers: conductorPostHeaders(access),
+                body,
+                signal: controller.signal,
+            });
+        }
+        catch {
+            return { kind: "uncertain", reason: controller.signal.aborted ? "timeout" : "network" };
+        }
+        let parsed;
+        let parsedOk = true;
+        try {
+            parsed = await resp.json();
+        }
+        catch {
+            parsedOk = false;
+        }
+        if (TYPED_CONTINUATION_STATUSES.has(resp.status) && parsedOk) {
+            try {
+                return { kind: "result", status: resp.status, result: parseContinueParkedChildResult(parsed) };
+            }
+            catch {
+                // Fall through: an untyped body on these statuses is classified below.
+            }
+        }
+        if (resp.status >= 200 && resp.status < 300) {
+            return { kind: "uncertain", reason: "malformed", status: resp.status };
+        }
+        if (resp.status >= 500) {
+            return { kind: "uncertain", reason: "server", status: resp.status };
+        }
+        // A 4xx without a typed body is an authoritative pre-mutation rejection.
+        return {
+            kind: "refused_http",
+            status: resp.status,
+            errorCode: parsedOk ? safeDetailErrorCode(parsed) : null,
+        };
+    }
+    finally {
+        clearTimeout(timer);
+    }
+}
+/**
+ * GET `.../tickets/{ticket_key}/continuation` — the child's current episode and,
+ * with `requestId`, that request's receipt. Read-only; bounded by the standard
+ * conductor timeout. Throws {@link ConductorBridgeApiError} on transport or HTTP
+ * failure and {@link ContinuationParseError} on a malformed body.
+ */
+export async function fetchChildContinuationStatus(access, query, fetchImpl = globalThis.fetch) {
+    requireNonEmptyString(query.epicRunId);
+    requireNonEmptyString(query.ticketKey);
+    const params = { repo_name: access.repoName };
+    if (query.requestId)
+        params.request_id = query.requestId;
+    const url = buildConductorJiraUrl(access.baseUrl, `${continuationChildPath(query.epicRunId, query.ticketKey)}/continuation`, params);
+    const parsed = await fetchConductorJsonWithTimeout(url, conductorGetHeaders(access), CONDUCTOR_FETCH_TIMEOUT_MS, fetchImpl);
+    return parseContinuationStatusResult(parsed);
+}
 /**
  * POST `/jira/epic-runs/runs/{epic_run_id}/tickets/{ticket_key}/adopt-current-head-and-unpark`
  * — the recovery lane for a human/external push that drifted the PR head off the
@@ -1441,8 +1554,13 @@ const REPAIR_TOKEN_RE = /^[a-z0-9_.:-]{1,120}$/;
 const POLICY_DIGEST_TOKEN_RE = /^policy:[0-9a-f]{12}$/;
 /** A dotted policy-path NAME. Never a value. */
 const POLICY_PATH_RE = /^[A-Za-z0-9_-]{1,100}(?:\.[A-Za-z0-9_-]{1,100}){0,15}$/;
-/** `resume-run`'s closed run-control state vocabulary. */
-const RESUME_RUN_STATES = ["paused", "quarantined", "none"];
+/**
+ * `resume-run`'s closed PRIOR-state vocabulary. `blocked` is the qualifying
+ * integration block the run-addressed route may lift; it is never a resulting state.
+ */
+const RESUME_RUN_BEFORE_STATES = ["blocked", "paused", "quarantined", "none"];
+/** `resume-run`'s closed RESULTING-state vocabulary: a resume always leaves no control. */
+const RESUME_RUN_AFTER_STATES = ["none"];
 function requireOnlyRepairKeys(p, extraKeys) {
     for (const key of Object.keys(p)) {
         if (!REPAIR_BASE_KEYS.includes(key) && !extraKeys.includes(key)) {
@@ -1509,18 +1627,29 @@ function parseResumeRunResult(parsed) {
     requireOnlyRepairKeys(p, []);
     const before = p["before"];
     const after = p["after"];
+    const committed = p["committed"];
+    const eventRecorded = p["event_recorded"];
     if (typeof before !== "string" ||
-        !RESUME_RUN_STATES.includes(before) ||
+        !RESUME_RUN_BEFORE_STATES.includes(before) ||
         typeof after !== "string" ||
-        !RESUME_RUN_STATES.includes(after)) {
+        !RESUME_RUN_AFTER_STATES.includes(after)) {
+        throw new ConductorBridgeApiError("server");
+    }
+    // Exactly two families: an applied resume (`blocked`/`paused`/`quarantined` →
+    // `none`, committed, either audit outcome) or the already-current no-op
+    // (`none` → `none`, nothing committed, no event). Any other combination is
+    // unconfirmed, never coerced into a result.
+    const applied = before !== "none" && committed;
+    const noOp = before === "none" && !committed && !eventRecorded;
+    if (!applied && !noOp) {
         throw new ConductorBridgeApiError("server");
     }
     return {
         epic_run_id: p["epic_run_id"],
         before,
         after,
-        committed: p["committed"],
-        event_recorded: p["event_recorded"],
+        committed,
+        event_recorded: eventRecorded,
     };
 }
 /**
@@ -2333,6 +2462,20 @@ function requireNullableString(o, key) {
         throw readinessShapeError();
     return v;
 }
+/**
+ * BAPI-1279 — the executor coverage fields. Each is omitted when the server did
+ * not send it (an older deploy), and a present non-boolean fails CLOSED to
+ * `false`: coverage that cannot be read is coverage that is not confirmed.
+ */
+function parseOptionalExecutorCoverage(o) {
+    const out = {};
+    for (const key of ["process_heartbeat_fresh", "repo_wide_executor_live"]) {
+        if (!(key in o))
+            continue;
+        out[key] = o[key] === true;
+    }
+    return out;
+}
 function requireNullableBool(o, key) {
     const v = o[key];
     if (v === null || v === undefined)
@@ -2519,6 +2662,7 @@ export function parseConductorReadinessResponse(body) {
             last_seen_at: requireNullableString(exec, "last_seen_at"),
             last_seen_age_seconds: requireNullableInt(exec, "last_seen_age_seconds"),
             ready: requireNullableBool(exec, "ready"),
+            ...parseOptionalExecutorCoverage(exec),
         },
         thresholds: {
             reconciler_stale_after_seconds: requireInt(thr, "reconciler_stale_after_seconds"),
@@ -2704,8 +2848,9 @@ function parseConductorCiWorkflow(body) {
  *
  * `notify_webhook_default_verified` is the one tri-state: absent is coerced to
  * `null` rather than to `false`, because "not verified here" and "verified as
- * broken" are different statements and only one of them is evidence. Every other
- * field is a strict boolean and a wrong type throws.
+ * broken" are different statements and only one of them is evidence.
+ * `escalation_email_available` (BAPI-1274) is lenient: anything but `true` is
+ * `false`. Every other field is a strict boolean and a wrong type throws.
  */
 function parseUnattended(body) {
     const raw = body.unattended;
@@ -2719,6 +2864,10 @@ function parseUnattended(body) {
             ? null
             : requireNullableBool(o, "notify_webhook_default_verified"),
         repository_readiness_confirmed: requireBool(o, "repository_readiness_confirmed"),
+        // BAPI-1274 — optional and lenient: absent, null, or any non-boolean
+        // normalizes to `false` rather than throwing, because a server older than
+        // BAPI-1274 omits it and must keep today's behavior.
+        escalation_email_available: o.escalation_email_available === true,
     };
 }
 /**
@@ -2735,6 +2884,15 @@ export async function fetchConductorReadiness(access, fetchImpl = globalThis.fet
     const body = await fetchConductorJsonWithTimeout(url, conductorGetHeaders(access), CONDUCTOR_FETCH_TIMEOUT_MS, fetchImpl);
     return parseConductorReadinessResponse(body);
 }
+const ROW_OUTCOMES = new Set(["created", "changed", "preserved"]);
+function optionalRowOutcome(o, key) {
+    const v = o[key];
+    if (v === undefined || v === null)
+        return null;
+    if (typeof v !== "string" || !ROW_OUTCOMES.has(v))
+        throw readinessShapeError();
+    return v;
+}
 /** Validate a raw bootstrap body, fail-closed (see {@link parseConductorReadinessResponse}). */
 export function parseConductorSupervisorBootstrapResponse(body) {
     const o = requireObject(body);
@@ -2749,6 +2907,8 @@ export function parseConductorSupervisorBootstrapResponse(body) {
         repo_name: repoName,
         setup_written: requireBool(o, "setup_written"),
         config_written: requireBool(o, "config_written"),
+        setup_outcome: optionalRowOutcome(o, "setup_outcome"),
+        config_outcome: optionalRowOutcome(o, "config_outcome"),
         setup_source: requireEnum(o, "setup_source", READINESS_SOURCES),
         config_source: requireEnum(o, "config_source", READINESS_SOURCES),
         setup_created_at: requireNullableString(o, "setup_created_at"),
@@ -2762,14 +2922,167 @@ export function parseConductorSupervisorBootstrapResponse(body) {
 /**
  * POST `/jira/epic-runs/supervisor-bootstrap?repo_name=<repo>`.
  *
- * The ONLY write this client performs. Since BAPI-694 it may also carry the
- * optional repository-default `review_policy`; omitting that field stores no
- * default rather than a neutral one.
+ * Since BAPI-694 it may also carry the optional repository-default
+ * `review_policy`; omitting that field keeps the stored default. Since BAPI-1288
+ * a relaxation is refused with 409 unless `approval` carries the exact token a
+ * {@link previewConductorSupervisorBootstrap} of the SAME request returned.
+ * A 409 is never retried here — see {@link classifySupervisorBootstrapFailure}.
  */
 export async function bootstrapConductorSupervisorDefaults(access, request, fetchImpl) {
     const url = buildConductorJiraUrl(access.baseUrl, `${EPIC_RUNS_API_PREFIX}/supervisor-bootstrap`, {
         repo_name: access.repoName,
     });
-    const body = await fetchConductorJsonPostWithTimeout(url, conductorPostHeaders(access), JSON.stringify(request), CONDUCTOR_FETCH_TIMEOUT_MS, fetchImpl);
+    const body = await fetchConductorJsonWithMethodAndTimeout("POST", url, conductorPostHeaders(access), JSON.stringify(request), CONDUCTOR_FETCH_TIMEOUT_MS, fetchImpl, { readErrorField: true });
     return parseConductorSupervisorBootstrapResponse(body);
 }
+// ---------------------------------------------------------------------------
+// BAPI-1288 — preview of a protected project-default change
+// ---------------------------------------------------------------------------
+/** Closed vocabularies of a server-authored gate-change effect. */
+const GATE_CHANGE_REQUIREMENTS = new Set([
+    "done_gate",
+    "required_ci_checks",
+    "done_gate_review",
+    "review_policy_default",
+]);
+const GATE_CHANGE_KINDS = new Set(["removed", "weakened", "replaced"]);
+const GATE_OBLIGATIONS = new Set([
+    "ci_check_green",
+    "sticky_verdict_approved",
+    "native_review_approved",
+    "approval_minimum",
+    "verdictless_review_parks",
+]);
+const CHECK_REF_RE = /^ci-[0-9a-f]{12}$/;
+/** Bounds the client enforces on server text before it may be displayed. */
+const MAX_EFFECTS = 16;
+const MAX_DESCRIPTION_LENGTH = 300;
+const MAX_CHECK_REFS = 50;
+const MAX_APPROVAL_TOKEN_LENGTH = 1024;
+function requireObligationList(o, key) {
+    const v = o[key];
+    if (!Array.isArray(v) || v.length > GATE_OBLIGATIONS.size)
+        throw readinessShapeError();
+    for (const item of v) {
+        if (typeof item !== "string" || !GATE_OBLIGATIONS.has(item))
+            throw readinessShapeError();
+    }
+    return v;
+}
+function parseGateChangeEffect(raw) {
+    const o = requireObject(raw);
+    const refs = o.removed_ci_check_refs;
+    if (!Array.isArray(refs) ||
+        refs.length > MAX_CHECK_REFS ||
+        refs.some((r) => typeof r !== "string" || !CHECK_REF_RE.test(r))) {
+        throw readinessShapeError();
+    }
+    const description = o.description;
+    if (typeof description !== "string" ||
+        description.length === 0 ||
+        description.length > MAX_DESCRIPTION_LENGTH ||
+        /[\u0000-\u001F\u007F]/.test(description)) {
+        throw readinessShapeError();
+    }
+    const count = requireInt(o, "removed_ci_check_count");
+    if (count < 0)
+        throw readinessShapeError();
+    return {
+        row: requireEnum(o, "row", new Set(["setup", "config"])),
+        requirement: requireEnum(o, "requirement", GATE_CHANGE_REQUIREMENTS),
+        change: requireEnum(o, "change", GATE_CHANGE_KINDS),
+        removed_obligations: requireObligationList(o, "removed_obligations"),
+        replacement_obligations: requireObligationList(o, "replacement_obligations"),
+        removed_ci_check_count: count,
+        removed_ci_check_refs: refs,
+        stored_min_approvals: requireNullableInt(o, "stored_min_approvals"),
+        proposed_min_approvals: requireNullableInt(o, "proposed_min_approvals"),
+        description,
+    };
+}
+/**
+ * Validate a raw preview body, fail-closed.
+ *
+ * Every field is checked against the server's closed contract; anything else —
+ * an unknown outcome, an effect category this client does not know, an
+ * over-long or control-character description, a token without
+ * `approval_required`, a writer or repository that does not match the request —
+ * is a shape error, so an unbounded server string can never reach the operator's
+ * terminal as if it were a vetted description.
+ */
+export function parseConductorSupervisorBootstrapPreview(body, expectedRepoName) {
+    const o = requireObject(body);
+    if (o.repo_name !== expectedRepoName)
+        throw readinessShapeError();
+    if (o.writer !== "bootstrap")
+        throw readinessShapeError();
+    const names = o.changed_field_names;
+    if (!Array.isArray(names) || names.some((n) => typeof n !== "string" || n.length > 100)) {
+        throw readinessShapeError();
+    }
+    const effectsRaw = o.effects;
+    if (!Array.isArray(effectsRaw) || effectsRaw.length > MAX_EFFECTS)
+        throw readinessShapeError();
+    const effects = effectsRaw.map(parseGateChangeEffect);
+    const approvalRequired = requireBool(o, "approval_required");
+    const token = requireNullableString(o, "approval_token");
+    if (approvalRequired !== effects.length > 0)
+        throw readinessShapeError();
+    if (approvalRequired !== (token !== null))
+        throw readinessShapeError();
+    if (token !== null && (token.length === 0 || token.length > MAX_APPROVAL_TOKEN_LENGTH)) {
+        throw readinessShapeError();
+    }
+    return {
+        repo_name: expectedRepoName,
+        writer: "bootstrap",
+        setup_outcome: requireEnum(o, "setup_outcome", ROW_OUTCOMES),
+        config_outcome: requireEnum(o, "config_outcome", ROW_OUTCOMES),
+        changed_field_names: names,
+        approval_required: approvalRequired,
+        effects,
+        approval_token: token,
+    };
+}
+/**
+ * POST `/jira/epic-runs/supervisor-bootstrap/preview?repo_name=<repo>` — what a
+ * bootstrap with `request` WOULD do. Read-only on the server.
+ *
+ * Send the same body to {@link bootstrapConductorSupervisorDefaults} afterwards
+ * (plus `approval` when the preview required it); any difference makes the
+ * server's token check fail with a fresh-preview conflict.
+ */
+export async function previewConductorSupervisorBootstrap(access, request, fetchImpl) {
+    const url = buildConductorJiraUrl(access.baseUrl, `${EPIC_RUNS_API_PREFIX}/supervisor-bootstrap/preview`, { repo_name: access.repoName });
+    const body = await fetchConductorJsonWithMethodAndTimeout("POST", url, conductorPostHeaders(access), JSON.stringify(request), CONDUCTOR_FETCH_TIMEOUT_MS, fetchImpl, { readErrorField: true });
+    return parseConductorSupervisorBootstrapPreview(body, access.repoName);
+}
+const GATE_CHANGE_CODE_TO_KIND = Object.freeze({
+    APPROVAL_REQUIRED: "approval-required",
+    STALE_PREVIEW: "stale-preview",
+    UNKNOWN_STORED_STATE: "unknown-stored-state",
+    NON_RELAXABLE: "non-relaxable",
+    INVALID_PROPOSAL: "invalid-proposal",
+    STATE_UNAVAILABLE: "state-unavailable",
+});
+export function classifySupervisorBootstrapFailure(err) {
+    if (!(err instanceof ConductorBridgeApiError))
+        return "network";
+    const mapped = err.errorCode ? GATE_CHANGE_CODE_TO_KIND[err.errorCode] : undefined;
+    if (mapped)
+        return mapped;
+    if (err.kind === "invalid-input")
+        return "unsupported-protocol";
+    if (err.kind === "unauthorized")
+        return "unauthorized";
+    if (err.kind === "timeout")
+        return "timeout";
+    if (err.kind === "network")
+        return "network";
+    if (err.status === 404 || err.status === 405 || err.status === 422) {
+        return "unsupported-protocol";
+    }
+    if (err.kind === "server")
+        return "server";
+    return "rejected";
+}

package/build/conductor/bring-up-facts.js

@@ -35,7 +35,9 @@ export const WORKER_NOT_PRESENT_REMEDIATION = "start a worker (`python worker.py
 export const SECOND_RECONCILER_REMEDIATION = "find and stop the extra reconciler: check `ps aux | grep worker.py` for a reparented " +
     `survivor of an earlier plane (${BRING_UP_RUNBOOK} §6), and note that a dev server started ` +
     "without DISABLE_SCHEDULER=true runs a reconciler too. A reconciler stopped in the last 120 " +
-    "seconds still counts — heartbeat rows are not deleted on shutdown — so wait and retry.";
+    "seconds still counts — heartbeat rows are not deleted on shutdown — so wait and retry. " +
+    "On a hosted deployment, check that exactly one worker process is scaled (for example one " +
+    "Heroku `worker` dyno): each worker runs its own reconciler.";
 // ---------------------------------------------------------------------------
 // Classification
 // ---------------------------------------------------------------------------

package/build/conductor/doctor.js

@@ -82,8 +82,9 @@ export const MCP_PROFILE_UNSET = "(unset)";
 /**
  * The CLI-only verbs that call the Bridge API from a shell-spawned process and
  * therefore resolve credentials themselves (CLAUDE.md "Shell-Spawned Credential
- * Resolution"): the four BAPI-872 recovery verbs and the six run-addressed repair
- * verbs (BAPI-1154's three and BAPI-1176's three). `.mcp.json` env is invisible to
+ * Resolution"): the four BAPI-872 recovery verbs, the six run-addressed repair
+ * verbs (BAPI-1154's three and BAPI-1176's three), and BAPI-1300's parked-child
+ * continuation pair. `.mcp.json` env is invisible to
  * them — only `BAPI_API_KEY` in the parent env and the user-scoped credential
  * store count.
  */
@@ -98,6 +99,9 @@ export const CREDENTIALED_RECOVERY_COMMANDS = [
     "scope-catch-up",
     "patch-policy",
     "resume-run",
+    // BAPI-1300: parked-child continuation and its read-only receipt lookup.
+    "continue-child",
+    "continuation-receipt",
 ];
 /** Remediation for recovery/repair verbs that cannot resolve Bridge credentials. */
 export const REPAIR_CREDENTIAL_REMEDIATION = "export BAPI_API_KEY in the shell that runs `conductor`, or store a credential for this " +
@@ -803,11 +807,14 @@ export function formatConductorDoctorReport(report) {
     lines.push(`hooks dir:             ${git_hooks.hooks_dir ?? "n/a"}`);
     lines.push(`degraded:              ${git_hooks.degraded}`);
     for (const hook of git_hooks.hooks) {
-        // BAPI-772: the embedded target is shown alongside the structural fields —
-        // a managed block whose binary no longer exists is a DEAD hook, and this line
-        // is where that becomes visible. Advisory only: never changes the exit code.
+        // BAPI-772 made a dead embedded target visible here; BAPI-1290 replaces the
+        // raw path with the shared per-hook verdict, because a file that EXISTS can
+        // still be a prunable cache entry or a runtime whose native binding does not
+        // load. The embedded path stays in `--json` output only. Advisory only:
+        // never changes the exit code. Read defensively — a literal fixture may
+        // predate `state`/`usable`, and unknown is never healthy.
         lines.push(`  ${hook.name}: exists=${hook.exists} executable=${hook.executable} managed=${hook.managed_block_present} ` +
-            `bin=${hook.embedded_bin_path ?? "n/a"} bin_is_file=${hook.embedded_bin_is_file}`);
+            `state=${hook.state ?? "unknown"} usable=${hook.usable === true}`);
     }
     if (git_hooks.warnings.length > 0) {
         lines.push("git hook warnings:");
@@ -962,8 +969,8 @@ export const CONDUCTOR_READINESS_DESCRIPTORS = [
 export const CONDUCTOR_LEDGER_REMEDIATION = "run `epic-implementer doctor` for the ledger detail, then re-create the local ledger directory with " +
     "owner-only permissions; the ledger is local observability and never blocks a run.";
 /** Remediation for degraded or missing managed git hooks. */
-export const CONDUCTOR_GIT_HOOKS_REMEDIATION = "run `epic-implementer install-git-hooks` to install the managed post-commit and reference-transaction " +
-    "hooks; they are local, opportunistic, and bypassable.";
+export const CONDUCTOR_GIT_HOOKS_REMEDIATION = "run `epic-implementer install-git-hooks` from a persistent install to install the managed post-commit and " +
+    "reference-transaction hooks; they are local, opportunistic, and bypassable, and never gate engine admission.";
 /** Remediation for a Claude login this host cannot confirm. */
 export const CONDUCTOR_CLAUDE_LOGIN_REMEDIATION = "run `claude login` on the host that will run conductor workers; Bridge stores no Anthropic " +
     "credential and can only advise here.";
@@ -1014,22 +1021,24 @@ function gitHooksOutcome(report) {
         };
     }
     // Hook NAMES and booleans only — `hooks_dir` is absolute and never projected.
-    const rendered = hooks.hooks
-        .map((hook) => `${hook.name}=${hook.exists && hook.managed_block_present}`)
-        .join(", ");
-    return hooks.degraded
+    // BAPI-1290: the boolean is the shared per-hook `usable` verdict (qualified
+    // persistent runtime, loadable native binding), not mere file presence; a
+    // hook without that verdict is unknown, and unknown is never healthy.
+    const rendered = hooks.hooks.map((hook) => `${hook.name}=${hook.usable === true}`).join(", ");
+    const allUsable = hooks.hooks.length > 0 && hooks.hooks.every((hook) => hook.usable === true);
+    return hooks.degraded || !allUsable
         ? {
             id: "git-hooks",
             label: "Conductor git hooks",
             status: "warn",
-            detail: `managed hooks installed: ${rendered}`,
+            detail: `managed hooks usable: ${rendered}`,
             remediation: CONDUCTOR_GIT_HOOKS_REMEDIATION,
         }
         : {
             id: "git-hooks",
             label: "Conductor git hooks",
             status: "pass",
-            detail: `managed hooks installed: ${rendered}`,
+            detail: `managed hooks usable: ${rendered}`,
         };
 }
 /**