@exaudeus/workrail 3.33.0 → 3.34.0

package/dist/mcp-server.js

@@ -1,60 +1,19 @@
 #!/usr/bin/env node
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.composeServer = exports.detectHealthyPrimary = exports.startBridgeServer = exports.startHttpServer = exports.startStdioServer = void 0;
-exports.waitForStdinReadable = waitForStdinReadable;
+exports.composeServer = exports.startHttpServer = exports.startStdioServer = void 0;
 const transport_mode_js_1 = require("./mcp/transports/transport-mode.js");
 const stdio_entry_js_1 = require("./mcp/transports/stdio-entry.js");
 const http_entry_js_1 = require("./mcp/transports/http-entry.js");
-const bridge_entry_js_1 = require("./mcp/transports/bridge-entry.js");
 const assert_never_js_1 = require("./runtime/assert-never.js");
 var stdio_entry_js_2 = require("./mcp/transports/stdio-entry.js");
 Object.defineProperty(exports, "startStdioServer", { enumerable: true, get: function () { return stdio_entry_js_2.startStdioServer; } });
 var http_entry_js_2 = require("./mcp/transports/http-entry.js");
 Object.defineProperty(exports, "startHttpServer", { enumerable: true, get: function () { return http_entry_js_2.startHttpServer; } });
-var bridge_entry_js_2 = require("./mcp/transports/bridge-entry.js");
-Object.defineProperty(exports, "startBridgeServer", { enumerable: true, get: function () { return bridge_entry_js_2.startBridgeServer; } });
-Object.defineProperty(exports, "detectHealthyPrimary", { enumerable: true, get: function () { return bridge_entry_js_2.detectHealthyPrimary; } });
 var server_js_1 = require("./mcp/server.js");
 Object.defineProperty(exports, "composeServer", { enumerable: true, get: function () { return server_js_1.composeServer; } });
-const DEFAULT_MCP_PORT = 3100;
-const STDIO_CLIENT_PROBE_MS = 150;
-function waitForStdinReadable(timeoutMs, stdin = process.stdin) {
-    return new Promise((resolve) => {
-        stdin.pause();
-        const timer = setTimeout(() => {
-            stdin.removeListener('readable', onReadable);
-            resolve(false);
-        }, timeoutMs);
-        const onReadable = () => {
-            clearTimeout(timer);
-            stdin.removeListener('readable', onReadable);
-            resolve(true);
-        };
-        stdin.once('readable', onReadable);
-    });
-}
 async function main() {
     const mode = (0, transport_mode_js_1.resolveTransportMode)(process.env);
-    if (mode.kind === 'stdio') {
-        const primaryDetected = await (0, bridge_entry_js_1.detectHealthyPrimary)(DEFAULT_MCP_PORT);
-        if (primaryDetected != null) {
-            const hasClient = await waitForStdinReadable(STDIO_CLIENT_PROBE_MS, process.stdin);
-            if (!hasClient) {
-                console.error(`[Startup] Primary on :${primaryDetected.port}, no stdio client within ${STDIO_CLIENT_PROBE_MS}ms — exiting`);
-                process.exit(0);
-            }
-            console.error(`[Startup] Primary detected on :${primaryDetected.port} — starting in bridge mode`);
-            try {
-                await (0, bridge_entry_js_1.startBridgeServer)(primaryDetected.port, undefined, { originalPrimaryPid: primaryDetected.pid });
-            }
-            catch (error) {
-                console.error('[Bridge] Fatal error, falling back to full stdio server:', error);
-                await (0, stdio_entry_js_1.startStdioServer)();
-            }
-            return;
-        }
-    }
     switch (mode.kind) {
         case 'stdio':
             await (0, stdio_entry_js_1.startStdioServer)();

package/dist/v2/durable-core/domain/observation-builder.d.ts

@@ -10,6 +10,9 @@ export interface ObservationEventData {
     } | {
         readonly type: 'sha256';
         readonly value: string;
+    } | {
+        readonly type: 'path';
+        readonly value: string;
     };
     readonly confidence: 'low' | 'med' | 'high';
 }

package/dist/v2/durable-core/domain/observation-builder.js

@@ -34,11 +34,11 @@ function anchorsToObservations(anchors) {
                 });
                 break;
             case 'repo_root':
-                if (anchor.value.length > constants_js_1.MAX_OBSERVATION_SHORT_STRING_LENGTH)
+                if (anchor.value.length > constants_js_1.MAX_OBSERVATION_PATH_LENGTH)
                     break;
                 observations.push({
                     key: 'repo_root',
-                    value: { type: 'short_string', value: anchor.value },
+                    value: { type: 'path', value: anchor.value },
                     confidence: 'high',
                 });
                 break;

package/dist/v2/durable-core/domain/prompt-renderer.d.ts

@@ -1,6 +1,6 @@
 import type { Result } from 'neverthrow';
 import type { Workflow } from '../../../types/workflow.js';
-import type { PromptFragment } from '../../../types/workflow-definition.js';
+import type { AssessmentDefinition, PromptFragment } from '../../../types/workflow-definition.js';
 import type { LoadedSessionTruthV2 } from '../../ports/session-event-log-store.port.js';
 import type { LoopPathFrameV1 } from '../schemas/execution-snapshot/index.js';
 import type { NodeId, RunId } from '../ids/index.js';
@@ -8,6 +8,7 @@ export type PromptRenderError = {
     readonly code: 'RENDER_FAILED';
     readonly message: string;
 };
+export declare function formatAssessmentRequirementsForTest(assessments: readonly Pick<AssessmentDefinition, 'id' | 'purpose' | 'dimensions'>[]): readonly string[];
 export declare function assembleFragmentedPrompt(fragments: readonly PromptFragment[], context: Record<string, unknown>): string;
 export interface StepMetadata {
     readonly stepId: string;

package/dist/v2/durable-core/domain/prompt-renderer.js

@@ -1,5 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.formatAssessmentRequirementsForTest = formatAssessmentRequirementsForTest;
 exports.assembleFragmentedPrompt = assembleFragmentedPrompt;
 exports.renderPendingPrompt = renderPendingPrompt;
 const neverthrow_1 = require("neverthrow");
@@ -174,6 +175,9 @@ function formatOutputContractRequirements(outputContract) {
             ];
     }
 }
+function formatAssessmentRequirementsForTest(assessments) {
+    return formatAssessmentRequirements(assessments);
+}
 function formatAssessmentRequirements(assessments) {
     if (assessments.length === 0)
         return [];
@@ -190,6 +194,12 @@ function formatAssessmentRequirements(assessments) {
         for (const dimension of assessment.dimensions) {
             requirements.push(`  ${dimension.id} (${dimension.levels.join(' | ')}): ${dimension.purpose}`);
         }
+        const firstDimension = assessment.dimensions[0];
+        const exampleDimValue = firstDimension ? `"${firstDimension.levels[0] ?? 'high'}"` : '"high"';
+        const exampleDimKey = firstDimension ? `"${firstDimension.id}"` : '"dimensionId"';
+        requirements.push(`Canonical format:\n\`\`\`json\n` +
+            `{ "artifacts": [{ "kind": "wr.assessment", "assessmentId": "${assessment.id}", "dimensions": { ${exampleDimKey}: ${exampleDimValue} } }] }\n` +
+            `\`\`\``);
         requirements.push('Use only canonical dimension levels. If the engine rejects the artifact, correct the submitted levels instead of inventing new ones.');
     }
     return requirements;

package/dist/v2/usecases/console-service.js

@@ -60,6 +60,49 @@ const AUTONOMOUS_HEARTBEAT_THRESHOLD_MS = 10 * 60 * 1000;
 const LIVE_ACTIVITY_MAX_ENTRIES = 5;
 const DAEMON_EVENT_LOG_READ_LIMIT_BYTES = 100 * 1024;
 const DAEMON_EVENTS_DIR = path.join(os.homedir(), '.workrail', 'events', 'daemon');
+async function isSessionLiveFromEventLog(workrailSessionId) {
+    const date = new Date().toISOString().slice(0, 10);
+    const filePath = path.join(DAEMON_EVENTS_DIR, `${date}.jsonl`);
+    try {
+        let raw;
+        const stat = await fs.stat(filePath);
+        if (stat.size > DAEMON_EVENT_LOG_READ_LIMIT_BYTES) {
+            const fd = await fs.open(filePath, 'r');
+            const offset = stat.size - DAEMON_EVENT_LOG_READ_LIMIT_BYTES;
+            const buf = Buffer.alloc(DAEMON_EVENT_LOG_READ_LIMIT_BYTES);
+            try {
+                await fd.read(buf, 0, DAEMON_EVENT_LOG_READ_LIMIT_BYTES, offset);
+            }
+            finally {
+                await fd.close();
+            }
+            raw = buf.toString('utf8');
+        }
+        else {
+            raw = await fs.readFile(filePath, 'utf8');
+        }
+        let hasSeen = false;
+        let hasCompleted = false;
+        for (const line of raw.split('\n')) {
+            if (!line.trim())
+                continue;
+            try {
+                const event = JSON.parse(line);
+                if (event['workrailSessionId'] !== workrailSessionId)
+                    continue;
+                hasSeen = true;
+                if (event['kind'] === 'session_completed')
+                    hasCompleted = true;
+            }
+            catch {
+            }
+        }
+        return hasSeen && !hasCompleted;
+    }
+    catch {
+        return false;
+    }
+}
 async function readLiveActivity(workrailSessionId, maxEntries) {
     const date = new Date().toISOString().slice(0, 10);
     const filePath = path.join(DAEMON_EVENTS_DIR, `${date}.jsonl`);
@@ -87,12 +130,24 @@ async function readLiveActivity(workrailSessionId, maxEntries) {
                 continue;
             try {
                 const event = JSON.parse(line);
-                if (event['kind'] !== 'tool_called' ||
+                const isToolEvent = event['kind'] === 'tool_called' ||
+                    event['kind'] === 'tool_call_started' ||
+                    event['kind'] === 'agent_stuck';
+                if (!isToolEvent ||
                     event['workrailSessionId'] !== workrailSessionId ||
-                    typeof event['toolName'] !== 'string' ||
                     typeof event['ts'] !== 'number') {
                     continue;
                 }
+                if (event['kind'] === 'agent_stuck') {
+                    activities.push({
+                        toolName: 'agent_stuck',
+                        summary: `STUCK: ${String(event['reason'] ?? '?')} -- ${String(event['detail'] ?? '').slice(0, 80)}`,
+                        ts: event['ts'],
+                    });
+                    continue;
+                }
+                if (typeof event['toolName'] !== 'string')
+                    continue;
                 activities.push({
                     toolName: event['toolName'],
                     ...(typeof event['summary'] === 'string' ? { summary: event['summary'] } : {}),
@@ -136,7 +191,6 @@ class ConsoleService {
     }
     getSessionDetail(sessionIdStr) {
         const sessionId = (0, index_js_1.asSessionId)(sessionIdStr);
-        const nowMs = Date.now();
         return this.ports.sessionStore
             .load(sessionId)
             .mapErr((storeErr) => ({
@@ -157,17 +211,14 @@ class ConsoleService {
                     resolveWorkflowNames(dag, this.ports.pinnedWorkflowStore),
                 ]).map(([completionMap, stepLabels, workflowNames]) => projectSessionDetail(sessionId, truth, completionMap, stepLabels, workflowNames));
             })();
-            const registryEntry = this.ports.daemonRegistry?.snapshot().get(sessionId);
-            const isLive = registryEntry !== undefined
-                && (nowMs - registryEntry.lastHeartbeatMs) < AUTONOMOUS_HEARTBEAT_THRESHOLD_MS;
-            if (!isLive) {
-                return detailRA.map((detail) => ({ ...detail, liveActivity: null }));
-            }
-            const liveActivityRA = neverthrow_1.ResultAsync.fromSafePromise(readLiveActivity(sessionIdStr, LIVE_ACTIVITY_MAX_ENTRIES));
-            return neverthrow_1.ResultAsync.combine([detailRA, liveActivityRA]).map(([detail, liveActivity]) => ({
-                ...detail,
-                liveActivity,
-            }));
+            const isLiveRA = neverthrow_1.ResultAsync.fromSafePromise(isSessionLiveFromEventLog(sessionIdStr));
+            return neverthrow_1.ResultAsync.combine([detailRA, isLiveRA]).andThen(([detail, isLive]) => {
+                if (!isLive) {
+                    return (0, neverthrow_2.okAsync)({ ...detail, isLive: false, liveActivity: null });
+                }
+                const liveActivityRA = neverthrow_1.ResultAsync.fromSafePromise(readLiveActivity(sessionIdStr, LIVE_ACTIVITY_MAX_ENTRIES));
+                return liveActivityRA.map((liveActivity) => ({ ...detail, isLive: true, liveActivity }));
+            });
         });
     }
     getNodeDetail(sessionIdStr, nodeId) {

package/dist/v2/usecases/console-types.d.ts

@@ -87,6 +87,7 @@ export interface ConsoleSessionDetail {
     readonly sessionTitle: string | null;
     readonly health: ConsoleSessionHealth;
     readonly runs: readonly ConsoleDagRun[];
+    readonly isLive?: boolean;
     readonly liveActivity?: readonly ConsoleToolActivity[] | null;
 }
 export type ConsoleValidationOutcome = 'pass' | 'fail';

package/docs/design/bridge-removal-pr-a-candidates.md

@@ -0,0 +1,115 @@
+# PR-A Bridge Removal -- Design Candidates
+
+**Status:** Implementation ready. Single candidate; no genuine alternatives.
+**Date:** 2026-04-17
+**Scope:** PR-A only -- delete bridge-entry.ts, primary-tombstone.ts, bridge-events.ts; remove auto-bridge block from mcp-server.ts; remove tombstone/bridge-events call sites from stdio-entry.ts and http-entry.ts.
+
+---
+
+## Problem Understanding
+
+### Core Tensions
+
+**T1: Delete all bridge files vs keep http-entry transport intact**
+
+`http-entry.ts` imports from both `primary-tombstone.ts` and `bridge-events.ts`. The task requires keeping `http-entry.ts` transport logic untouched while removing tombstone/bridge-events call sites from it. The seam is at the import level, not the file level.
+
+**T2: Self-referential risk**
+
+This MCP server is the tool running this workflow. A broken build would kill the active session. All changes must be made, build verified, then committed.
+
+**T3: Logging loss**
+
+`logBridgeEvent({ kind: 'primary_started', ... })` in both stdio-entry.ts and http-entry.ts records primary server startup for bridge forensics. After bridge removal this logging has no consumer. Removing it is correct but it is the only usage of bridge-events.ts in those files -- confirms the module is entirely removable.
+
+### Likely Seam
+
+The `import` statements at the tops of `mcp-server.ts`, `stdio-entry.ts`, and `http-entry.ts`. Removing those imports is what makes the TypeScript compiler verify that nothing references the deleted files at compile time. Any missed call site produces a build error rather than silent breakage.
+
+### What Makes This Hard
+
+Nothing architecturally hard. The main risk is a missed import or call site in a file not in the primary list (e.g. `src/mcp/index.ts` re-exporting from bridge-entry.ts). Must grep for all three module names before deleting. The `mcp-server.test.ts` test reads the source file as a string and checks for specific export patterns -- need to verify no assertion references `startBridgeServer` or `detectHealthyPrimary`.
+
+---
+
+## Philosophy Constraints
+
+Sources: `/Users/etienneb/CLAUDE.md` and `docs/design/stdio-simplification-design-candidates.md` (philosophy table, lines 68-75).
+
+| Principle | Constraint |
+|---|---|
+| Architectural fixes over patches | Delete the root cause (bridge block). No feature flag. |
+| YAGNI with discipline | bridge-entry, primary-tombstone, bridge-events are all YAGNI once auto-bridge block is removed. |
+| Determinism over cleverness | 150ms stdin probe, reconnect backoff, spawn coordinator lock -- all eliminated. |
+| Make illegal states unrepresentable | `waitForStdinReadable` and `STDIO_CLIENT_PROBE_MS` have no callers after removal; deleting them prevents future misuse. |
+
+**No conflicts** between CLAUDE.md and design doc for PR-A.
+
+---
+
+## Impact Surface
+
+| Surface | Action | Reason |
+|---|---|---|
+| `src/mcp/transports/http-entry.ts` | Modify (tombstone/bridge-events call sites only) | Transport logic must stay intact |
+| `src/mcp/transports/http-listener.ts` | No change | No bridge imports |
+| `tests/integration/mcp-http-transport.test.ts` | No change | Tests MCP-over-HTTP, not bridge |
+| `tests/unit/mcp/http-listener.test.ts` | No change | Tests createHttpListener lifecycle |
+| `tests/unit/mcp-server.test.ts` | Review only | Checks source file exports as strings; confirmed no assertion for startBridgeServer |
+| `src/mcp/index.ts` | Review before deleting | May re-export from bridge-entry.ts |
+
+---
+
+## Candidates
+
+Only one genuine candidate. All alternatives (feature flag, deprecation shim, alias re-exports) contradict the explicit design doc and repo direct-deletion pattern. Noting this honestly.
+
+### Candidate 1 (Only Candidate): Direct deletion with targeted import removal
+
+**Summary:** Delete the 3 source files and 3 test files outright. Remove their imports and call sites from mcp-server.ts, stdio-entry.ts, and http-entry.ts. Update mcp-server.ts to call `startStdioServer()` unconditionally for stdio mode.
+
+**Tensions resolved:** T1 (boundary is at import level, not file level -- transport logic untouched), T2 (minimal blast radius -- 6 files deleted, 3 modified), T3 (logging call sites removed along with the module).
+
+**Tensions accepted:** None.
+
+**Boundary solved at:** Import statements at the tops of mcp-server.ts, stdio-entry.ts, http-entry.ts.
+
+**Why that boundary is best fit:** TypeScript compiler enforces correctness. Any missed call site produces a build error rather than silent breakage. The boundary is compiler-verified, not grep-and-hope.
+
+**Failure mode:** A missed import in an unexpected file (e.g. `mcp/index.ts` re-exporting `startBridgeServer`). Mitigation: grep `src/ tests/` for `bridge-entry`, `primary-tombstone`, `bridge-events` before deleting.
+
+**Repo-pattern relationship:** Follows direct-deletion pattern exactly. No flags, no shims. Every refactor in the git log (including PR #512) removes code directly.
+
+**Gains:** ~1127 lines of dead code deleted. Startup becomes deterministic. No 150ms probe delay. No spawn coordinator lock files. No tombstone files.
+
+**Gives up:** Nothing. Bridge functionality is superseded by standalone `worktrain console`.
+
+**Scope judgment:** Best-fit. Exactly matches PR-A specification from design doc.
+
+**Philosophy:** Honors architectural fixes over patches, YAGNI, determinism, make illegal states unrepresentable. No conflicts.
+
+---
+
+## Comparison and Recommendation
+
+**Recommendation: Candidate 1.**
+
+Only candidate. All alternative shapes add complexity without benefit and contradict the explicit design doc. Direct deletion is both the simplest and the architecturally correct approach.
+
+---
+
+## Self-Critique
+
+**Strongest argument against:** Could keep `bridge-events.ts` as a general-purpose process lifecycle log (it does log `primary_started` which is not bridge-specific). Counter: the file's event vocabulary is overwhelmingly bridge-specific (`reconnected`, `spawn_primary`, `budget_exhausted`, `waiting_for_primary`). Keeping it would require renaming, refactoring the event union, and updating callers. Scope creep. Delete it.
+
+**Narrower option:** Keep `primary-tombstone.ts` in place with no callers. Rejected: dead file with no callers is dead code; deleting makes the absence unrepresentable.
+
+**Broader option:** Also clean up `worktrain-spawn.ts` and `worktrain-await.ts` which have a `dashboard.lock` fallback that will now be permanently dead code. Evidence from design doc: "No behavioral change, but the fallback is now permanently dead code." The design doc explicitly calls this a follow-up chore PR. Not in PR-A scope.
+
+**Assumption that would invalidate:** `src/mcp/index.ts` re-exports `startBridgeServer` as part of public library API. Must verify with grep before deleting.
+
+---
+
+## Open Questions
+
+None requiring human decision. All scoped by design doc. The single pre-implementation check (grep for unexpected importers) is a mechanical verification step, not a decision.

package/docs/design/bridge-removal-pr-a-design-review.md

@@ -0,0 +1,79 @@
+# PR-A Bridge Removal -- Design Review Findings
+
+**Status:** Review complete. No blocking issues. Ready to implement.
+**Date:** 2026-04-17
+**Reviewed design:** Direct deletion of bridge-entry.ts, primary-tombstone.ts, bridge-events.ts; targeted import removal from mcp-server.ts, stdio-entry.ts, http-entry.ts.
+
+---
+
+## Tradeoff Review
+
+| Tradeoff | Verdict | Notes |
+|---|---|---|
+| Losing `logBridgeEvent` lifecycle logging | Acceptable | No consumer reads bridge.log except bridge forensics. Grep confirms no reader in `src/`. |
+| `waitForStdinReadable` removed from public API | Acceptable | Not re-exported from `mcp/index.ts`. Only consumer is `tests/unit/mcp/stdin-probe.test.ts` which is also deleted. |
+| `mcp-server.ts` JSDoc describes removed behavior | Requires fix | Top comment block (lines 11-24) explains auto-bridge mechanism that will no longer exist. Must update. |
+
+---
+
+## Failure Mode Review
+
+| Failure Mode | Coverage | Risk |
+|---|---|---|
+| Missed import in unexpected file | **Fully covered** -- grep of src/ and tests/ returned exactly the expected 7 files. TypeScript compiler provides second verification. | Low |
+| `mcp-server.test.ts` string assertions failing | **Covered** -- test file read in full; no assertion for startBridgeServer, detectHealthyPrimary, or waitForStdinReadable. | Low |
+| http-entry.ts transport logic accidentally broken | **Covered** -- tombstone call sites are isolated lines (32, 35, 111-113); transport registration (73-76, 78), health endpoint (89-91), and port binding (45) are untouched. | Low |
+
+No high-risk failure modes remain.
+
+---
+
+## Runner-Up / Simpler Alternative Review
+
+No genuine runner-up. All alternatives examined:
+- **Keep bridge-events.ts:** Leaves dead module with bridge-specific event vocabulary. Rejected (YAGNI).
+- **Skip test file deletion:** Would fail `npm test` at import resolution. Not simpler.
+- **Skip source file modification:** Build fails with cannot-find-module errors. Not viable.
+
+The selected design is already the minimal change that satisfies acceptance criteria.
+
+---
+
+## Philosophy Alignment
+
+| Principle | Status |
+|---|---|
+| Architectural fixes over patches | Satisfied -- root cause deleted, no flag |
+| YAGNI with discipline | Satisfied -- all 3 modules have zero callers post-removal |
+| Determinism over cleverness | Satisfied -- startup becomes one unconditional call |
+| Make illegal states unrepresentable | Satisfied -- waitForStdinReadable deleted |
+| Type safety as first line of defense | Satisfied -- compiler verifies completeness |
+| Document why, not what | Under tension -- JSDoc needs update (see Yellow finding) |
+
+---
+
+## Findings
+
+### Yellow -- JSDoc in mcp-server.ts describes removed behavior
+
+**File:** `src/mcp-server.ts` lines 11-24 (auto-bridge and zombie-bridge guard comments)
+
+**Issue:** After removal, the multi-line JSDoc comment at the top of the file describes behavior (auto-bridge detection, zombie-bridge guard, stdin probe) that no longer exists. Leaving it would mislead future readers.
+
+**Recommended fix:** Replace the existing comment block with a simplified description: "Resolves transport mode from environment and starts the appropriate server."
+
+**Severity:** Yellow -- not a correctness issue, but violates "Document why, not what" and misleads future readers.
+
+---
+
+## Recommended Revisions
+
+1. **Update JSDoc in mcp-server.ts** -- replace the auto-bridge explanation (lines 11-24) with a brief description of the simplified startup path. Required before commit.
+
+2. **Verify test passes for `mcp-server.test.ts`** -- after editing, run the test explicitly to confirm the string-match assertions still pass. This is a build-time check, but worth isolating.
+
+---
+
+## Residual Concerns
+
+None. Import surface verified. Failure modes covered. Philosophy alignment confirmed. The JSDoc revision is a required edit, not an unresolved concern.

package/docs/design/bridge-removal-pr-a-implementation-plan.md

@@ -0,0 +1,203 @@
+# PR-A Bridge Removal -- Implementation Plan
+
+**Date:** 2026-04-17
+**Branch:** `feat/mcp-simplify-remove-bridge`
+**Confidence:** High
+
+---
+
+## Problem Statement
+
+The WorkRail MCP server contains a bridge mechanism (bridge-entry.ts, primary-tombstone.ts, bridge-events.ts) that was built to solve port contention between multiple Claude Code windows. PR #512 (merged) introduced `worktrain console` as a standalone process that owns the dashboard UI independently of the MCP server. The bridge mechanism's reason for existence is gone. It adds ~1127 lines of reconnect state machine, spawn coordination, and tombstone logic that make startup non-deterministic (150ms probe delay) and increase operational complexity.
+
+---
+
+## Acceptance Criteria
+
+1. `src/mcp/transports/bridge-entry.ts` does not exist
+2. `src/mcp/transports/primary-tombstone.ts` does not exist
+3. `src/mcp/transports/bridge-events.ts` does not exist
+4. `src/mcp-server.ts` no longer imports or re-exports `startBridgeServer`, `detectHealthyPrimary`, or `waitForStdinReadable`
+5. `src/mcp-server.ts` no longer contains the auto-bridge detection block (lines 89-118 in current file)
+6. `src/mcp/transports/stdio-entry.ts` no longer imports from `primary-tombstone.ts` or `bridge-events.ts`
+7. `src/mcp/transports/http-entry.ts` no longer imports from `primary-tombstone.ts` or `bridge-events.ts`
+8. `npm run build` exits 0 with 0 TypeScript errors
+9. `npm test` exits 0
+
+---
+
+## Non-Goals
+
+- HttpServer removal (PR-B)
+- ToolContext changes
+- DI token cleanup
+- Any change to http-entry.ts transport logic (MCP-over-HTTP for bot services)
+- Modifying http-listener.ts or http-listener.test.ts
+- Removing `ctx.httpServer?.stop()` from shutdown hooks (HttpServer still starts in PR-A state)
+- Cleaning up `worktrain-spawn.ts` / `worktrain-await.ts` dashboard.lock fallback (chore PR later)
+
+---
+
+## Philosophy-Driven Constraints
+
+- **Architectural fixes over patches:** Delete root cause, not add flags
+- **YAGNI:** Delete all three modules with no callers
+- **Determinism:** Startup path must be a single unconditional call to `startStdioServer()` for stdio mode
+- **Commit type:** `chore(mcp)` -- internal cleanup, no user-visible behavior change, no tool contract change
+
+---
+
+## Invariants
+
+1. `WORKRAIL_TRANSPORT=http` path must still work -- `startHttpServer()` is called unconditionally for http mode
+2. `WORKRAIL_TRANSPORT=stdio` must call `startStdioServer()` directly after removal -- no probe, no bridge detection
+3. `mcp/index.ts` exports must be unchanged (confirmed: no bridge symbols were re-exported)
+4. `tests/integration/mcp-http-transport.test.ts` must pass -- MCP-over-HTTP transport is untouched
+
+---
+
+## Selected Approach
+
+**Direct deletion with targeted import removal.** Delete 3 source files and 3 test files. Remove their imports and call sites from 3 modified source files. TypeScript compiler verifies completeness at the import boundary.
+
+**Runner-up:** None. All alternatives were anti-patterns.
+
+---
+
+## Vertical Slices
+
+### Slice 1: Create branch and verify baseline
+
+- Create git branch `feat/mcp-simplify-remove-bridge` from current HEAD
+- Run `npm run build` to confirm baseline is green
+- Run `npm test` to confirm baseline test suite is green
+- **Done when:** Both commands exit 0
+
+### Slice 2: Delete source files
+
+- Delete `src/mcp/transports/bridge-entry.ts`
+- Delete `src/mcp/transports/primary-tombstone.ts`
+- Delete `src/mcp/transports/bridge-events.ts`
+- **Done when:** Files do not exist on disk
+
+### Slice 3: Delete test files
+
+- Delete `tests/unit/mcp/transports/bridge-entry.test.ts`
+- Delete `tests/unit/mcp/transports/primary-tombstone.test.ts`
+- Delete `tests/unit/mcp/stdin-probe.test.ts`
+- **Done when:** Files do not exist on disk
+
+### Slice 4: Modify mcp-server.ts
+
+Remove the following from `src/mcp-server.ts`:
+- Import of `startBridgeServer`, `detectHealthyPrimary` from `./mcp/transports/bridge-entry.js`
+- Re-export of `startBridgeServer`, `detectHealthyPrimary` from `./mcp/transports/bridge-entry.js`
+- `STDIO_CLIENT_PROBE_MS` constant (line 49)
+- `waitForStdinReadable` function (lines 62-82)
+- Auto-bridge detection block inside `main()` (lines 88-118)
+- Top JSDoc comment block (lines 11-24) describing auto-bridge -- replace with simplified description
+- **Done when:** File compiles with no errors referencing bridge-entry; `main()` calls `startStdioServer()` unconditionally for stdio mode
+
+### Slice 5: Modify stdio-entry.ts
+
+Remove from `src/mcp/transports/stdio-entry.ts`:
+- Import: `import { writeTombstone, clearTombstone } from './primary-tombstone.js'`
+- Import: `import { logBridgeEvent } from './bridge-events.js'`
+- Call: `logBridgeEvent({ kind: 'primary_started', transport: 'stdio' })` (line 39)
+- Call: `clearTombstone()` (line 44)
+- In `onBeforeTerminate`: remove the tombstone write block (lines 124-127: `const port = ctx.httpServer?.getPort(); if (port != null) { writeTombstone(port, process.pid); }`)
+- **Done when:** File compiles with no imports or references to deleted modules; `onBeforeTerminate` only calls `await ctx.httpServer?.stop()`
+
+### Slice 6: Modify http-entry.ts
+
+Remove from `src/mcp/transports/http-entry.ts` (tombstone/bridge-events call sites ONLY):
+- Import: `import { clearTombstone, writeTombstone } from './primary-tombstone.js'`
+- Import: `import { logBridgeEvent } from './bridge-events.js'`
+- Call: `logBridgeEvent({ kind: 'primary_started', transport: 'http', port })` (line 32)
+- Call: `clearTombstone()` (line 35)
+- In `onBeforeTerminate`: remove `if (boundPort != null) { writeTombstone(boundPort, process.pid); }` (lines 111-113)
+- **Done when:** File compiles with no imports or references to deleted modules; ALL transport logic (bindWithPortFallback, MCP endpoint registration, health endpoint) is unchanged
+
+### Slice 7: Build and test verification
+
+- Run `npm run build` -- must exit 0 with 0 errors
+- Run `npm test` -- must exit 0
+- **Done when:** Both commands succeed
+
+### Slice 8: Commit and push PR
+
+- `git add` the 6 deleted files and 3 modified source files
+- Commit with message: `chore(mcp): remove bridge, tombstone, and bridge-events from MCP server`
+- Push branch and create PR against main
+- **Done when:** PR is open
+
+---
+
+## Test Design
+
+**Tests to delete:**
+- `tests/unit/mcp/transports/bridge-entry.test.ts` -- imports deleted module
+- `tests/unit/mcp/transports/primary-tombstone.test.ts` -- imports deleted module
+- `tests/unit/mcp/stdin-probe.test.ts` -- imports `waitForStdinReadable` from mcp-server.ts (also deleted)
+
+**Tests that must stay green:**
+- `tests/unit/mcp-server.test.ts` -- reads mcp-server.ts as string; no assertion for bridge exports
+- `tests/unit/mcp/http-listener.test.ts` -- tests createHttpListener lifecycle; no bridge dependency
+- `tests/integration/mcp-http-transport.test.ts` -- tests MCP-over-HTTP; no bridge dependency
+- All other tests (no bridge imports in non-deleted test files)
+
+**No new tests required.** The change is a deletion; the verification is `npm run build` (0 errors) + `npm test`.
+
+---
+
+## Risk Register
+
+| Risk | Likelihood | Impact | Mitigation |
+|---|---|---|---|
+| Missed import in unexpected file | Low | Build failure | Grep sweep confirmed before implementation: only 7 files (5 src, 2 tests) import from deleted modules |
+| http-entry.ts transport logic broken | Low | Runtime failure | Tombstone call sites are isolated lines; transport logic verified as non-overlapping |
+| mcp-server.test.ts string assertions fail | Low | Test failure | Test file read in full; no assertion for bridge exports |
+
+---
+
+## PR Packaging Strategy
+
+Single PR. Branch: `feat/mcp-simplify-remove-bridge`. All 8 slices in one commit.
+
+Commit message: `chore(mcp): remove bridge, tombstone, and bridge-events from MCP server`
+
+PR description must include:
+- What was deleted and why (bridge mechanism superseded by standalone worktrain console)
+- What was NOT changed (http-entry.ts transport logic, http-listener.ts, ToolContext)
+- Link to design doc: `docs/design/stdio-simplification-design-candidates.md` PR-A section
+
+---
+
+## Philosophy Alignment Per Slice
+
+| Slice | Principle | Status |
+|---|---|---|
+| All slices | Architectural fixes over patches | Satisfied -- root cause deleted, no shim |
+| Slice 2-3 | YAGNI with discipline | Satisfied -- all 3 modules have zero callers post-removal |
+| Slice 4 | Determinism over cleverness | Satisfied -- startup becomes single unconditional call |
+| Slice 4 | Make illegal states unrepresentable | Satisfied -- waitForStdinReadable deleted |
+| Slice 4 | Document why, not what | Satisfied -- JSDoc updated to reflect simplified behavior |
+| Slice 5-6 | Type safety as first line | Satisfied -- compiler verifies no remaining references |
+| Slice 6 | Keep interfaces small | Satisfied -- http-entry.ts shrinks, transport logic unchanged |
+
+---
+
+## Follow-Up Tickets
+
+- `worktrain-spawn.ts` and `worktrain-await.ts` `dashboard.lock` fallback is now permanently dead code. Remove in a separate chore PR.
+- PR-B: Remove HttpServer from MCP server startup entirely.
+
+---
+
+## Unresolved Unknown Count
+
+**0.** All import surfaces verified, failure modes addressed, acceptance criteria measurable.
+
+## Plan Confidence Band
+
+**High.**