@bridge_gpt/mcp-server 0.2.58 → 0.2.59

package/build/review-tickets.js

@@ -12,8 +12,8 @@
  *
  * Optionally supports a supervisor-driven epic-dispatch path: when an
  * `EpicDispatchIdentity` is threaded through `ReviewTicketsOptions.epic`, each
- * spawned tab receives an epic-identity env block (BAPI_CONDUCTOR_EPIC_KEY /
- * BAPI_CONDUCTOR_EPIC_RUN_ID / BAPI_CONDUCTOR_PLAN_VERSION / BAPI_CONDUCTOR_RUN_ID)
+ * spawned tab receives an epic-identity env block (BAPI_EPIC_IMPLEMENTER_EPIC_KEY /
+ * BAPI_EPIC_IMPLEMENTER_EPIC_RUN_ID / BAPI_EPIC_IMPLEMENTER_PLAN_VERSION / BAPI_EPIC_IMPLEMENTER_RUN_ID)
  * and dispatch is made idempotent via an injected `claimEpicDispatch` callback.
  * This path is worktree-free and base-branch-free; the human CLI path leaves
  * `epic` unset and retains byte-for-byte identical behavior.
@@ -518,7 +518,7 @@ export function buildReviewPlanRows(deps, options, agent, status, overrides = {}
         if (options.epic) {
             const mint = overrides.mintRunId ?? mintStartTicketsRunId;
             runId = mint([key]);
-            const env = { ...buildEpicIdentityEnv(options.epic), BAPI_CONDUCTOR_RUN_ID: runId };
+            const env = { ...buildEpicIdentityEnv(options.epic), BAPI_EPIC_IMPLEMENTER_RUN_ID: runId };
             command = injectConductorEnvIntoShellCommand(deps.platform, baseCommand, env);
         }
         return {
@@ -599,7 +599,7 @@ export async function orchestrateReviewTickets(deps, options, overrides = {}) {
         if (options.epic) {
             const mint = overrides.mintRunId ?? mintStartTicketsRunId;
             runId = mint([key]);
-            const env = { ...buildEpicIdentityEnv(options.epic), BAPI_CONDUCTOR_RUN_ID: runId };
+            const env = { ...buildEpicIdentityEnv(options.epic), BAPI_EPIC_IMPLEMENTER_RUN_ID: runId };
             shellCommand = injectConductorEnvIntoShellCommand(deps.platform, baseShellCommand, env);
             if (options.epic.dispatch_key) {
                 if (!overrides.claimEpicDispatch) {

package/build/setup-epic.js

@@ -89,8 +89,8 @@ import { validateBranchName } from "./base-ref.js";
 // typed clients the pilot already has are the v2 path's clients too, so neither
 // conductor can drift on a field name or an endpoint shape. No MCP tool, no
 // input-schema field, and no description changes.
-import { getConfigFieldBaseBranch, getIndexScopeLifecycle, getParseStatus, } from "./implement-epic/bridge-client.js";
-import { boundScopeLifecycleLabel, createExecFileRunCommand, ensureCommitResolvableLocally, normalizeCommitSha, performExactIndexScopeCut, pollIndexScopeLifecycle, readRemoteBranchHead, runGit, SCOPE_BOOTSTRAP_POLL_INTERVAL_MS, SCOPE_LIFECYCLE_LABELS, } from "./implement-epic/cut-protocol.js";
+import { getConfigFieldBaseBranch, getIndexScopeLifecycle, getParseStatus, } from "./epic-shared/bridge-client.js";
+import { boundScopeLifecycleLabel, createExecFileRunCommand, ensureCommitResolvableLocally, normalizeCommitSha, performExactIndexScopeCut, pollIndexScopeLifecycle, readRemoteBranchHead, runGit, SCOPE_BOOTSTRAP_POLL_INTERVAL_MS, SCOPE_LIFECYCLE_LABELS, } from "./epic-shared/cut-protocol.js";
 import { hashPlan } from "./conductor/plan.js";
 import { BRING_UP_REFUSAL_MESSAGES, firstBringUpRefusal, } from "./conductor/bring-up-facts.js";
 import { MCP_PACKAGE_NAME } from "./mcp-identity.js";
@@ -111,7 +111,7 @@ export const SETUP_EPIC_UNATTENDED_POSTURE = "unattended";
 /**
  * The server's stable error code for an unattended-posture refusal (BAPI-734).
  *
- * Mirrors `api.library.epic_conductor.unattended_posture.UNATTENDED_POSTURE_REFUSED_CODE`.
+ * Mirrors `api.library.epic_implementer.unattended_posture.UNATTENDED_POSTURE_REFUSED_CODE`.
  * Matched by EQUALITY on the structured `error_code`, never by searching the
  * message text: the message is prose that may be rewritten, the code is the
  * contract.
@@ -239,7 +239,7 @@ export function getSetupEpicUsage() {
         "                         10799; remediate / ci_fix 3599; rebase / smoke 2699;",
         "                         merge 1799); the server rejects a higher value with a",
         "                         422 naming the offending field. Full table:",
-        "                         docs/claude/epic-conductor-v2-operator-runbook.md §4.",
+        "                         docs/claude/epic-implementer-v2-operator-runbook.md §4.",
         "  --replace-policy       Authorize replacing a LIVE run's stored policy with the",
         "                         policy file, as a complete replacement. Without it, a",
         "                         divergent policy on a reused run is refused with a",
@@ -1130,7 +1130,7 @@ export function resolveFeatureBranchSelection(opts, planNodeCount, policyDeclare
  * refused every unattended run, its own named remedy included.
  *
  * The reason CODES are the server's bounded vocabulary, spelled exactly as
- * `api/library/epic_conductor/unattended_posture.py` spells them. The client adds
+ * `api/library/epic_implementer/unattended_posture.py` spells them. The client adds
  * remediation TEXT around them and invents no aliases — an alias would drift from
  * the server's refusal the first time either side was edited, and an operator
  * comparing a CLI message with an API response would have no way to tell they
@@ -1145,7 +1145,7 @@ export function resolveFeatureBranchSelection(opts, planNodeCount, policyDeclare
  * Does this policy declare a notify webhook the server would probe?
  *
  * BAPI-1102. The shape check mirrors `_extract_webhook_url` in
- * `api/library/epic_conductor/webhook_verification.py` exactly as far as
+ * `api/library/epic_implementer/webhook_verification.py` exactly as far as
  * PRESENCE goes — a `notify` object carrying a non-blank string `webhook_url` —
  * and stops there on purpose. Scheme, host, and reachability are the server's to
  * judge at activation, where the probe actually happens; re-deciding them here
@@ -1492,7 +1492,7 @@ export const SETUP_EPIC_SCOPE_RECOVERY_COMMAND = "re-run this same setup-epic co
  * The canonical one-line recovery for a scope collision (BAPI-1009).
  *
  * Repeated VERBATIM as the epic-run gate snapshot's shadow-hold recovery hint
- * (``api/library/epic_conductor/gate_snapshot_service.py``), so an operator who
+ * (``api/library/epic_implementer/gate_snapshot_service.py``), so an operator who
  * reads the diagnosis on the status surface and an operator who reads it from
  * the CLI are told to do the same thing in the same words.
  */
@@ -2256,7 +2256,7 @@ async function executeSetupEpic(argv, overrides, runIdSink) {
                 : `  - feature branch: ${effectiveFeatureBranch} (validated server-side at approval; no request made in dry-run)`);
             if (needsCut && cutCommitSha !== null && scopeBaseBranch !== null) {
                 say(`  - would open a draft pull request ${effectiveFeatureBranch} → ${scopeBaseBranch} ` +
-                    "for conductor-ci / gate (BAPI-951; nothing opened in dry-run)");
+                    "for epic-implementer-ci / gate (BAPI-951; nothing opened in dry-run)");
             }
         }
         if (opts.reviewPolicy !== undefined) {

package/build/start-tickets-conductor.js

@@ -62,20 +62,20 @@ export function mintStartTicketsWorkerId(ticketKey, agentName, fragment = random
  */
 export function buildEpicIdentityEnv(epic) {
     const env = {
-        BAPI_CONDUCTOR_EPIC_KEY: epic.epic_key,
-        BAPI_CONDUCTOR_EPIC_RUN_ID: epic.epic_run_id,
-        BAPI_CONDUCTOR_PLAN_VERSION: String(epic.plan_version),
+        BAPI_EPIC_IMPLEMENTER_EPIC_KEY: epic.epic_key,
+        BAPI_EPIC_IMPLEMENTER_EPIC_RUN_ID: epic.epic_run_id,
+        BAPI_EPIC_IMPLEMENTER_PLAN_VERSION: String(epic.plan_version),
     };
     const declared = normalizeDeclaredTouchedFiles(epic.declared_touched_files);
     if (declared.length > 0) {
-        env.BAPI_CONDUCTOR_DECLARED_TOUCHED_FILES_JSON = JSON.stringify(declared);
+        env.BAPI_EPIC_IMPLEMENTER_DECLARED_TOUCHED_FILES_JSON = JSON.stringify(declared);
     }
     return env;
 }
 // ---------------------------------------------------------------------------
 // Conductor context
 // ---------------------------------------------------------------------------
-/** Default gate name when no `BAPI_CONDUCTOR_GATE_NAME` override is present. */
+/** Default gate name when no `BAPI_EPIC_IMPLEMENTER_GATE_NAME` override is present. */
 export const DEFAULT_CONDUCTOR_GATE_NAME = "implement-ticket";
 /**
  * Resolve a packaged build artifact path relative to this compiled module.
@@ -110,11 +110,11 @@ export async function createStartTicketsConductorContext(options, agent, deps) {
         // Repo resolution must never abort conductor setup; fall back to null.
         repoName = null;
     }
-    const gateName = nonEmpty(deps.env.BAPI_CONDUCTOR_GATE_NAME)
-        ? deps.env.BAPI_CONDUCTOR_GATE_NAME.trim()
+    const gateName = nonEmpty(deps.env.BAPI_EPIC_IMPLEMENTER_GATE_NAME)
+        ? deps.env.BAPI_EPIC_IMPLEMENTER_GATE_NAME.trim()
         : DEFAULT_CONDUCTOR_GATE_NAME;
-    const supervisorMode = nonEmpty(deps.env.BAPI_CONDUCTOR_SUPERVISOR_MODE)
-        ? deps.env.BAPI_CONDUCTOR_SUPERVISOR_MODE.trim()
+    const supervisorMode = nonEmpty(deps.env.BAPI_EPIC_IMPLEMENTER_SUPERVISOR_MODE)
+        ? deps.env.BAPI_EPIC_IMPLEMENTER_SUPERVISOR_MODE.trim()
         : options.autoApprove
             ? "auto"
             : "interactive";
@@ -142,12 +142,12 @@ export async function createStartTicketsConductorContext(options, agent, deps) {
 // ---------------------------------------------------------------------------
 /** Conductor store-tuning env keys forwarded only when already set upstream. */
 const CONDUCTOR_TUNING_ENV_KEYS = [
-    "BAPI_CONDUCTOR_BUSY_TIMEOUT_MS",
-    "BAPI_CONDUCTOR_RETENTION_DAYS",
-    "BAPI_CONDUCTOR_RETENTION_MAX_ROWS",
+    "BAPI_EPIC_IMPLEMENTER_BUSY_TIMEOUT_MS",
+    "BAPI_EPIC_IMPLEMENTER_RETENTION_DAYS",
+    "BAPI_EPIC_IMPLEMENTER_RETENTION_MAX_ROWS",
     // BAPI-397: per-type relay cooldown — forwarded only when explicitly set so
     // the supervisor's send config is honoured by the conductor store it writes to.
-    "BAPI_CONDUCTOR_MESSAGE_COOLDOWN_MS",
+    "BAPI_EPIC_IMPLEMENTER_MESSAGE_COOLDOWN_MS",
 ];
 /** Truthy predicate for opt-in conductor boolean flags (`1` / `true`). */
 export function isConductorFlagEnabled(value) {
@@ -164,30 +164,30 @@ export function isConductorFlagEnabled(value) {
  * upstream.
  */
 export function buildConductorWorkerEnv(context, worker, parentEnv) {
-    // Merge "conductor" into the parent project's profile rather than overwriting it.
+    // Merge "epic_implementer" into the parent project's profile rather than overwriting it.
     // This preserves groups like "sfcc" that the project already has, while adding
     // the conductor-specific tools. Only validated group tokens from resolveProfiles
     // are threaded — arbitrary parentEnv keys never leak through.
     const parentActiveGroups = new Set(resolveProfiles(parentEnv.BRIDGE_MCP_PROFILE));
-    parentActiveGroups.add("conductor");
+    parentActiveGroups.add("epic_implementer");
     const mergedProfile = Array.from(parentActiveGroups).join(",");
     const env = {
-        BAPI_CONDUCTOR_ENABLED: "1",
+        BAPI_EPIC_IMPLEMENTER_ENABLED: "1",
         BRIDGE_MCP_PROFILE: mergedProfile,
-        BAPI_CONDUCTOR_RUN_ID: context.runId,
-        BAPI_CONDUCTOR_WORKER_ID: worker.workerId,
-        BAPI_CONDUCTOR_TICKET_KEY: worker.ticketKey,
-        BAPI_CONDUCTOR_WORKTREE_PATH: worker.worktreePath,
-        BAPI_CONDUCTOR_GATE_NAME: context.gateName,
-        BAPI_CONDUCTOR_SUPERVISOR_MODE: context.supervisorMode,
-        BAPI_CONDUCTOR_CLI_FILE: context.cliFile,
+        BAPI_EPIC_IMPLEMENTER_RUN_ID: context.runId,
+        BAPI_EPIC_IMPLEMENTER_WORKER_ID: worker.workerId,
+        BAPI_EPIC_IMPLEMENTER_TICKET_KEY: worker.ticketKey,
+        BAPI_EPIC_IMPLEMENTER_WORKTREE_PATH: worker.worktreePath,
+        BAPI_EPIC_IMPLEMENTER_GATE_NAME: context.gateName,
+        BAPI_EPIC_IMPLEMENTER_SUPERVISOR_MODE: context.supervisorMode,
+        BAPI_EPIC_IMPLEMENTER_CLI_FILE: context.cliFile,
         // BAPI-527: the CLI file + Node runtime are a PAIR. The worker runs
         // conductor-bin.js under this captured conductor Node for every worker-side
         // ledger op, so the worker Node never loads better-sqlite3. Non-secret.
-        CONDUCTOR_NODE_PATH: context.conductorNodePath,
+        EPIC_IMPLEMENTER_NODE_PATH: context.conductorNodePath,
     };
     if (context.repoName) {
-        env.BAPI_CONDUCTOR_REPO_NAME = context.repoName;
+        env.BAPI_EPIC_IMPLEMENTER_REPO_NAME = context.repoName;
     }
     if (context.epic) {
         const epicEnv = buildEpicIdentityEnv(context.epic);
@@ -195,8 +195,8 @@ export function buildConductorWorkerEnv(context, worker, parentEnv) {
             env[k] = v;
         }
     }
-    if (isConductorFlagEnabled(parentEnv.BAPI_CONDUCTOR_ENABLE_PRE_TOOL_USE)) {
-        env.BAPI_CONDUCTOR_ENABLE_PRE_TOOL_USE = "1";
+    if (isConductorFlagEnabled(parentEnv.BAPI_EPIC_IMPLEMENTER_ENABLE_PRE_TOOL_USE)) {
+        env.BAPI_EPIC_IMPLEMENTER_ENABLE_PRE_TOOL_USE = "1";
     }
     for (const key of CONDUCTOR_TUNING_ENV_KEYS) {
         if (nonEmpty(parentEnv[key])) {
@@ -234,13 +234,13 @@ function shellQuotePath(value) {
     return `'${value.replace(/'/g, "'\\''")}'`;
 }
 /**
- * Resolve the hook command string. Prefers `BAPI_CONDUCTOR_HOOK_COMMAND`
+ * Resolve the hook command string. Prefers `BAPI_EPIC_IMPLEMENTER_HOOK_COMMAND`
  * verbatim; otherwise runs the packaged hook bin via the current Node
  * executable.
  */
 export function resolveConductorHookCommand(env, hookBinPath, execPath = process.execPath) {
-    if (nonEmpty(env.BAPI_CONDUCTOR_HOOK_COMMAND)) {
-        return env.BAPI_CONDUCTOR_HOOK_COMMAND;
+    if (nonEmpty(env.BAPI_EPIC_IMPLEMENTER_HOOK_COMMAND)) {
+        return env.BAPI_EPIC_IMPLEMENTER_HOOK_COMMAND;
     }
     return `${shellQuotePath(execPath)} ${shellQuotePath(hookBinPath)}`;
 }
@@ -279,7 +279,7 @@ export async function provisionConductorHookForWorktree(worktreePath, command, o
  */
 export async function provisionConductorHooksForRows(rows, context, deps) {
     const isClaude = context.agentName === "claude";
-    const enablePreToolUse = isConductorFlagEnabled(deps.env.BAPI_CONDUCTOR_ENABLE_PRE_TOOL_USE);
+    const enablePreToolUse = isConductorFlagEnabled(deps.env.BAPI_EPIC_IMPLEMENTER_ENABLE_PRE_TOOL_USE);
     const command = resolveConductorHookCommand(deps.env, context.hookBinPath, deps.execPath);
     const fragment = deps.workerFragment;
     const out = [];
@@ -451,7 +451,7 @@ export function buildSupervisorTabCommand(context, platform, nodeExecPath = proc
 /**
  * Decide whether the supervisor peer tab should launch. Enabled by default for a
  * normal packaged conductor context; disabled only when the resolved supervisor
- * mode is `off` (i.e. `BAPI_CONDUCTOR_SUPERVISOR_MODE=off`).
+ * mode is `off` (i.e. `BAPI_EPIC_IMPLEMENTER_SUPERVISOR_MODE=off`).
  */
 export function isSupervisorLaunchEnabled(context) {
     return context.supervisorMode.trim().toLowerCase() !== "off";

package/build/start-tickets-prereqs.js

@@ -52,7 +52,7 @@ export const START_TICKETS_DOCTOR_COMMAND = `npx -y ${MCP_PACKAGE_NAME} doctor`;
 /**
  * BAPI-527 live-source checkout guard (conductor durable execution).
  *
- * `BAPI_CONDUCTOR_LIVE_SOURCE_PATH` — OPTIONAL operator-set path to the checkout
+ * `BAPI_EPIC_IMPLEMENTER_LIVE_SOURCE_PATH` — OPTIONAL operator-set path to the checkout
  * an operator's live dev server (e.g. `uvicorn main:app --reload`) is running
  * from. When set, unattended conductor/epic/non-mutating start-tickets dispatch
  * refuses to run if the conductor base repo it would operate on resolves to the
@@ -62,14 +62,14 @@ export const START_TICKETS_DOCTOR_COMMAND = `npx -y ${MCP_PACKAGE_NAME} doctor`;
  * discovery, so this local env contract is introduced here. Read only from the
  * injected `deps.env` (never global `process.env`) so it stays mockable.
  *
- * `BAPI_CONDUCTOR_ALLOW_LIVE_SOURCE_DISPATCH` — explicit escape hatch. When set
+ * `BAPI_EPIC_IMPLEMENTER_ALLOW_LIVE_SOURCE_DISPATCH` — explicit escape hatch. When set
  * to a truthy value (`1`/`true`/`yes`/`on`, case-insensitive) a detected
  * collision is DOWNGRADED from fatal to a loud warning rather than silently
  * ignored, so an operator who genuinely wants same-checkout dispatch can opt in
  * with eyes open.
  */
-export const CONDUCTOR_LIVE_SOURCE_PATH_ENV = "BAPI_CONDUCTOR_LIVE_SOURCE_PATH";
-export const CONDUCTOR_ALLOW_LIVE_SOURCE_DISPATCH_ENV = "BAPI_CONDUCTOR_ALLOW_LIVE_SOURCE_DISPATCH";
+export const CONDUCTOR_LIVE_SOURCE_PATH_ENV = "BAPI_EPIC_IMPLEMENTER_LIVE_SOURCE_PATH";
+export const CONDUCTOR_ALLOW_LIVE_SOURCE_DISPATCH_ENV = "BAPI_EPIC_IMPLEMENTER_ALLOW_LIVE_SOURCE_DISPATCH";
 /** Stable descriptor id for the live-source checkout guard (used in messages + doctor). */
 export const LIVE_SOURCE_GUARD_ID = "conductor-live-source";
 /** True only for the three first-class supported platforms. */
@@ -566,11 +566,6 @@ const CLAUDE_MCP_SHADOWING_INSTALL_HINTS = {
  *   - a CONFIRMED foreign registration                      → `found: true` + advisory detail
  *   - unreadable / malformed config, or an unresolved main
  *     worktree path                                          → `found: true` + detail
- *
- * Reporting stays deliberately independent of the retired
- * `BAPI_CONDUCTOR_ALLOW_CLAUDE_MCP_SHADOWING` override: whether that variable is
- * exported has never had any bearing on whether the collision exists, and now it
- * has no bearing on anything at all.
  */
 export function claudeMcpShadowingDescriptor() {
     return {