Revert "Block workflow traffic until bootstrap blockers clear"

This reverts commit d4622f4.

Author: rmcdaniel

README.md

@@ -705,12 +705,6 @@ of pretending to be interchangeable HTTP peers. `GET /api/cluster/info`,
 `/api/health`, and `/api/ready` stay available for discovery and liveness even
 on scheduler-only, execution-only, or matching-only nodes.
 
-Those runtime-serving workflow routes also fail closed on bootstrap blockers.
-If database connectivity or workflow-table migrations are not ready, hosted
-workflow and worker-protocol routes return `503` with
-`reason: "workflow_v2_blocked"` plus `blocked_by` and `remediation` instead of
-accepting traffic that depends on an incomplete rollout state.
-
 The same `GET /api/cluster/info` response now includes a versioned
 `coordination_health` manifest for rollout-safety coordination risk. It
 summarizes the current server-wide workflow v2 health status, warning and error

app/Http/Middleware/RequireWorkflowBootstrapReady.php

@@ -16,7 +16,6 @@ final class RemoteScheduleStarter implements ScheduleWorkflowStarter
 {
     public function __construct(
         private readonly WorkflowStartService $startService,
-        private readonly ServerReadiness $readiness,
     ) {}
 
     public function start(
@@ -25,20 +24,6 @@ public function start(
         string $outcome,
         ?string $effectiveOverlapPolicy = null,
     ): ScheduleStartResult {
-        $workflowStatus = $this->readiness->workflowStatus();
-        $blockedBy = is_array($workflowStatus['blocked_by'] ?? null)
-            ? array_values(array_filter($workflowStatus['blocked_by'], static fn (mixed $value): bool => is_string($value) && $value !== ''))
-            : [];
-
-        if (($workflowStatus['status'] ?? null) === 'blocked' && $blockedBy !== []) {
-            throw new WorkflowExecutionUnavailableException(
-                'schedule_start',
-                $schedule->schedule_id,
-                'workflow_v2_blocked',
-                'Workflow v2 bootstrap blockers must clear before scheduled workflows can start.',
-            );
-        }
-
         $action = WorkflowSchedule::normalizeActionTimeouts(
             is_array($schedule->action) ? $schedule->action : [],
         );

app/Support/RemoteScheduleStarter.php

@@ -18,7 +18,6 @@
 use App\Http\Middleware\ControlPlaneVersionResolver;
 use App\Http\Middleware\NamespaceResolver;
 use App\Http\Middleware\RequireRole;
-use App\Http\Middleware\RequireWorkflowBootstrapReady;
 use App\Http\Middleware\RequireTopologyRoles;
 use App\Http\Middleware\WorkerProtocolVersionResolver;
 use Illuminate\Support\Facades\Route;
@@ -52,11 +51,6 @@
 // NamespaceResolver on hosted routes so wrong-node requests fail closed with a
 // machine-readable topology reason without leaking namespace existence.
 //
-// RequireWorkflowBootstrapReady sits in the same slot for runtime-serving
-// workflow routes. It only blocks on explicit database/migration bootstrap
-// blockers so routes fail closed during rollout/schema drift without locking
-// out recovery paths such as compatible worker registration.
-//
 // WorkerProtocolVersionResolver follows the same ordering for worker-plane
 // routes, keeping protocol skew and namespace errors in the worker envelope.
 Route::middleware([Authenticate::class])->group(function () {
@@ -68,7 +62,6 @@
     $cpv = ControlPlaneVersionResolver::class;
     $httpControl = RequireTopologyRoles::class.':api_ingress,control_plane';
     $httpWorker = RequireTopologyRoles::class.':api_ingress,control_plane';
-    $workflowBootstrap = RequireWorkflowBootstrapReady::class;
     $wpv = WorkerProtocolVersionResolver::class;
 
     // ── System ───────────────────────────────────────────────────────
@@ -89,7 +82,7 @@
     });
 
     // ── Workflows ────────────────────────────────────────────────────
-    Route::prefix('workflows')->middleware([$operator, $cpv, $httpControl, $workflowBootstrap, $ns])->group(function () {
+    Route::prefix('workflows')->middleware([$operator, $cpv, $httpControl, $ns])->group(function () {
         Route::get('/', [WorkflowController::class, 'index']);
         Route::post('/', [WorkflowController::class, 'start']);
         Route::get('/{workflowId}', [WorkflowController::class, 'show']);
@@ -120,12 +113,12 @@
     });
 
     // ── Bridge Adapters ──────────────────────────────────────────────
-    Route::prefix('bridge-adapters')->middleware([$operator, $cpv, $httpControl, $workflowBootstrap, $ns])->group(function () {
+    Route::prefix('bridge-adapters')->middleware([$operator, $cpv, $httpControl, $ns])->group(function () {
         Route::post('/webhook/{adapter}', [BridgeAdapterController::class, 'webhook']);
     });
 
     // ── Worker Task Polling ──────────────────────────────────────────
-    Route::prefix('worker')->middleware([$worker, $wpv, $httpWorker, $workflowBootstrap, $ns])->group(function () {
+    Route::prefix('worker')->middleware([$worker, $wpv, $httpWorker, $ns])->group(function () {
         // Registration
         Route::post('/register', [WorkerController::class, 'register']);
         Route::post('/heartbeat', [WorkerController::class, 'heartbeat']);

routes/api.php

@@ -223,34 +223,6 @@ public function test_it_records_rollout_safety_start_rejections_as_skips(): void
         $this->assertSame([], $schedule->recent_actions ?? []);
     }
 
-    public function test_it_skips_due_schedules_when_workflow_bootstrap_is_blocked(): void
-    {
-        DB::table('migrations')
-            ->where('migration', '2026_04_21_000300_add_workflow_definition_fingerprints_to_worker_registrations')
-            ->delete();
-
-        WorkflowSchedule::create([
-            'schedule_id' => 'skip-bootstrap-blocked',
-            'namespace' => 'default',
-            'spec' => ['cron_expressions' => ['* * * * *']],
-            'action' => ['workflow_type' => 'TestWorkflow'],
-            'next_fire_at' => now()->subMinute(),
-        ]);
-
-        $this->artisan('schedule:evaluate')
-            ->assertExitCode(0)
-            ->expectsOutputToContain('skipped');
-
-        $schedule = WorkflowSchedule::where('schedule_id', 'skip-bootstrap-blocked')->firstOrFail();
-        $this->assertSame('workflow_v2_blocked', $schedule->last_skip_reason);
-        $this->assertSame(1, (int) $schedule->skipped_trigger_count);
-        $this->assertSame(0, (int) $schedule->fires_count);
-        $this->assertSame(0, (int) $schedule->failures_count);
-        $this->assertNull($schedule->last_fired_at);
-        $this->assertNull($schedule->latest_workflow_instance_id);
-        $this->assertSame([], $schedule->recent_actions ?? []);
-    }
-
     // ── next_fire_at advancement ────────────────────────────────────
 
     public function test_it_advances_next_fire_at_after_successful_fire(): void