From 2d38b7b4b2990f3ca67a17aa6ef9cdd58e2aa3c9 Mon Sep 17 00:00:00 2001
From: "Ryan Johnson (ntninja)" <ryan.v.johnson3@gmail.com>
Date: Sun, 5 Jul 2026 00:30:29 +0000
Subject: [PATCH] =?UTF-8?q?feat(marshaling):=20the=20page=20reads=20as=20t?=
 =?UTF-8?q?wo=20scopes=20=E2=80=94=20pilot=20marshaling=20vs=20heat=20ruli?=
 =?UTF-8?q?ngs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Design call with the RD: protests/rulings STAY on the marshaling page (rulings
belong next to the evidence — a DQ or a protest resolution is made looking at
the laps, the trace, and the audit, and a separate page would make every
adjudication a context switch on the same heat). Revisit a dedicated cross-heat
protest queue if/when pilot-filed protests arrive.

What changes is the SEAM: the per-pilot surfaces (trace, tune, laps &
corrections) and the heat-scoped surfaces (penalties, protests, result
lifecycle, void) now sit under two labeled region heads with a deliberately
heavy divider between them — sunlit-laptop legible, and the heat region reads
as a distinct zone rather than more pilot UI.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 .../rd-console/src/screens/Marshaling.svelte  | 45 +++++++++++++++++++
 .../rd-console/tests/MarshalingScreen.test.ts | 11 +++++
 2 files changed, 56 insertions(+)

diff --git a/frontend/apps/rd-console/src/screens/Marshaling.svelte b/frontend/apps/rd-console/src/screens/Marshaling.svelte
index 0ed2f0e..8ac1a9e 100644
--- a/frontend/apps/rd-console/src/screens/Marshaling.svelte
+++ b/frontend/apps/rd-console/src/screens/Marshaling.svelte
@@ -933,6 +933,14 @@
         </div>
       {/if}
 
+      <!-- ── REGION: pilot marshaling ─────────────────────────────────────────────────
+           Everything from here to the heat-rulings divider is scoped to the SHOWN PILOT:
+           their trace, their detection levels, their laps and corrections. -->
+      <h3 class="region-head">
+        Pilot marshaling
+        <span class="region-hint">the shown pilot's laps, trace &amp; corrections</span>
+      </h3>
+
       {#if hasShownTrace && shownTrace}
         {@const tuning = canControl && tuneTrace != null && tuneKeyCurrent}
         <!-- Signal-as-evidence (Slice 4): the RSSI graph for the selected pilot's captured trace. A
@@ -1264,6 +1272,15 @@
       {/if}
 
       {#if canControl}
+        <!-- ── REGION: heat rulings & protests ────────────────────────────────────────────
+             A deliberate hard break from the per-pilot surfaces above (RD-requested): from
+             here down the scope is the WHOLE HEAT — penalties, protests, the result
+             lifecycle, and the void — independent of which pilot is shown. -->
+        <hr class="region-divider" />
+        <h3 class="region-head heat">
+          Heat rulings &amp; protests
+          <span class="region-hint">whole-heat scope — penalties, protests &amp; the result</span>
+        </h3>
         {#if resultLocked}
           <!-- The official-result lock: the Director rejects every result-changing ruling on a
                Final heat, so the correction surfaces below are withheld entirely and the ONE way
@@ -1819,6 +1836,34 @@
     list-style: none;
     padding: var(--gf-space-1) 0;
   }
+  /* Region heads: the two scopes of this page (pilot vs heat), RD-requested after the
+     lap-editor consolidation made the seam invisible. The divider is deliberately HEAVY —
+     sunlit-laptop legible — and the heat region reads as a distinct zone, not more pilot UI. */
+  .region-head {
+    display: flex;
+    align-items: baseline;
+    gap: var(--gf-space-3);
+    margin: var(--gf-space-4) 0 var(--gf-space-2);
+    font-size: var(--gf-font-size-lg);
+    letter-spacing: var(--gf-tracking-caps);
+    text-transform: uppercase;
+  }
+  .region-head .region-hint {
+    font-size: var(--gf-font-size-sm);
+    letter-spacing: normal;
+    text-transform: none;
+    color: var(--gf-text-muted);
+    font-weight: var(--gf-font-weight-normal);
+  }
+  .region-divider {
+    margin: var(--gf-space-7) 0 var(--gf-space-2);
+    border: 0;
+    border-top: 3px solid color-mix(in srgb, var(--gf-accent, #7aa2ff) 45%, var(--gf-border));
+  }
+  .region-head.heat {
+    color: color-mix(in srgb, var(--gf-accent, #7aa2ff) 70%, var(--gf-text));
+  }
+
   .lap-restore {
     flex: none;
     font-size: var(--gf-font-size-sm);
diff --git a/frontend/apps/rd-console/tests/MarshalingScreen.test.ts b/frontend/apps/rd-console/tests/MarshalingScreen.test.ts
index b06f5ca..bf25ba2 100644
--- a/frontend/apps/rd-console/tests/MarshalingScreen.test.ts
+++ b/frontend/apps/rd-console/tests/MarshalingScreen.test.ts
@@ -50,6 +50,17 @@ describe('Marshaling (Slice 3)', () => {
     expect(sendSpy).toHaveBeenCalledWith({ SplitLap: { target: 13, at: 21_500_000 } });
   });
 
+  it('the page reads as TWO scopes: pilot marshaling above, heat rulings below the divider', () => {
+    const { session } = makeTestSession({ live: liveRunning, laps: lapList });
+    render(Marshaling, { session });
+    expect(screen.getByRole('heading', { name: /Pilot marshaling/ })).toBeInTheDocument();
+    expect(screen.getByRole('heading', { name: /Heat rulings & protests/ })).toBeInTheDocument();
+    // The pilot region precedes the heat region in document order (the divider separates them).
+    const heads = screen.getAllByRole('heading', { name: /Pilot marshaling|Heat rulings/ });
+    expect(heads[0]).toHaveTextContent('Pilot marshaling');
+    expect(heads[1]).toHaveTextContent('Heat rulings & protests');
+  });
+
   it('marshals a different heat WITHOUT moving Race Control’s current heat', async () => {
     const heats: HeatSummary[] = [
       {