From 156b51d5371bc2d5d4a525fe8cdd051e24341851 Mon Sep 17 00:00:00 2001
From: Brad Harris <brad@studio.localdomain>
Date: Wed, 11 Mar 2026 09:36:03 -0600
Subject: [PATCH 1/3] Fix modals and settings pane on mobile

- Close mobile sidebar before opening dialogs (create agent, settings,
  delete, edit worktree) so they aren't hidden behind the slide panel
- Raise dialog/settings z-index to z-[70] above the mobile panel's z-[60]
- Make settings pane full-screen on mobile with drill-down section navigation
- Stack release manager layout vertically on mobile instead of side-by-side
- Add padding to the release log section

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 web/src/App.tsx                            |  8 +--
 web/src/components/app/release-manager.tsx |  6 +--
 web/src/components/app/settings-pane.tsx   | 59 ++++++++++++++++++----
 web/src/components/ui/dialog.tsx           |  4 +-
 4 files changed, 59 insertions(+), 18 deletions(-)

diff --git a/web/src/App.tsx b/web/src/App.tsx
index eab943a1..30e0593f 100644
--- a/web/src/App.tsx
+++ b/web/src/App.tsx
@@ -1429,12 +1429,12 @@ export function App(): JSX.Element {
             agents={agents}
             selectedAgentId={selectedAgentId}
             overflowAgentId={overflowAgentId}
-            onOpenCreateDialog={openCreateDialog}
-            onOpenEditWorktreeDialog={openEditWorktreeDialog}
-            onOpenSettings={() => setSettingsPaneOpen(true)}
+            onOpenCreateDialog={() => { setMobileLeftOpen(false); openCreateDialog(); }}
+            onOpenEditWorktreeDialog={(agent) => { setMobileLeftOpen(false); openEditWorktreeDialog(agent); }}
+            onOpenSettings={() => { setMobileLeftOpen(false); setSettingsPaneOpen(true); }}
             setOverflowAgentId={setOverflowAgentId}
             setDeleteTarget={setDeleteTarget}
-            setDeleteConfirmOpen={setDeleteConfirmOpen}
+            setDeleteConfirmOpen={(open) => { if (open) setMobileLeftOpen(false); setDeleteConfirmOpen(open); }}
             agentVisualState={agentVisualState}
             borderForAgentState={borderForAgentState}
             toggleAgentDetails={toggleAgentDetails}
diff --git a/web/src/components/app/release-manager.tsx b/web/src/components/app/release-manager.tsx
index 917e270b..75cb00c8 100644
--- a/web/src/components/app/release-manager.tsx
+++ b/web/src/components/app/release-manager.tsx
@@ -249,9 +249,9 @@ export function ReleaseManager(): JSX.Element {
   const phaseIndex = job ? PHASES_ORDER.indexOf(job.phase) : -1;
 
   return (
-    <div className="flex h-full min-h-0">
+    <div className="flex h-full min-h-0 flex-col md:flex-row">
       {/* Left column — controls */}
-      <div className="flex w-[360px] shrink-0 flex-col gap-6 overflow-y-auto border-r border-border p-6">
+      <div className="flex md:w-[360px] shrink-0 flex-col gap-6 overflow-y-auto border-b md:border-b-0 md:border-r border-border p-4 md:p-6">
 
         {/* Current version */}
         <div>
@@ -428,7 +428,7 @@ export function ReleaseManager(): JSX.Element {
       </div>
 
       {/* Right column — log */}
-      <div className="flex min-h-0 min-w-0 flex-1 flex-col">
+      <div className="flex min-h-0 min-w-0 flex-1 flex-col p-2">
         {showLog ? (
           <div
             ref={logRef}
diff --git a/web/src/components/app/settings-pane.tsx b/web/src/components/app/settings-pane.tsx
index 102ee66d..885af42e 100644
--- a/web/src/components/app/settings-pane.tsx
+++ b/web/src/components/app/settings-pane.tsx
@@ -1,5 +1,6 @@
+import { useState } from "react";
 import * as DialogPrimitive from "@radix-ui/react-dialog";
-import { Rocket, X } from "lucide-react";
+import { ChevronRight, ArrowLeft, Rocket, X } from "lucide-react";
 
 import { ReleaseManager } from "@/components/app/release-manager";
 import { cn } from "@/lib/utils";
@@ -16,19 +17,41 @@ type SettingsPaneProps = {
 };
 
 export function SettingsPane({ open, onClose }: SettingsPaneProps): JSX.Element {
-  const activeSection: SettingsSection = "release";
+  const [activeSection, setActiveSection] = useState<SettingsSection | null>("release");
 
   return (
-    <DialogPrimitive.Root open={open} onOpenChange={(v) => { if (!v) onClose(); }}>
+    <DialogPrimitive.Root
+      open={open}
+      onOpenChange={(v) => {
+        if (!v) {
+          onClose();
+          setActiveSection("release");
+        }
+      }}
+    >
       <DialogPrimitive.Portal>
-        <DialogPrimitive.Overlay className="fixed inset-0 z-50 bg-black/70 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0" />
-        <DialogPrimitive.Content className="fixed inset-4 z-50 flex flex-col overflow-hidden rounded-sm border border-border bg-card text-foreground shadow-2xl duration-200 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95">
+        <DialogPrimitive.Overlay className="fixed inset-0 z-[70] bg-black/70 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0" />
+        <DialogPrimitive.Content className="fixed inset-0 md:inset-4 z-[70] flex flex-col overflow-hidden rounded-none md:rounded-sm border border-border bg-card text-foreground shadow-2xl duration-200 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95">
           <DialogPrimitive.Title className="sr-only">Settings</DialogPrimitive.Title>
           <DialogPrimitive.Description className="sr-only">Dispatch settings and release manager</DialogPrimitive.Description>
 
           {/* Header */}
           <div className="flex h-12 shrink-0 items-center border-b border-border px-5">
-            <span className="text-sm font-semibold uppercase tracking-wide text-muted-foreground">Settings</span>
+            {/* Mobile back button when viewing a section */}
+            {activeSection !== null && (
+              <button
+                onClick={() => setActiveSection(null)}
+                className="mr-2 rounded-sm p-1 opacity-70 transition-opacity hover:opacity-100 md:hidden"
+              >
+                <ArrowLeft className="h-4 w-4" />
+              </button>
+            )}
+            <span className="text-sm font-semibold uppercase tracking-wide text-muted-foreground">
+              {activeSection !== null ? (
+                <span className="md:hidden">{SECTIONS.find((s) => s.id === activeSection)?.label ?? "Settings"}</span>
+              ) : null}
+              <span className={activeSection !== null ? "hidden md:inline" : ""}>Settings</span>
+            </span>
             <DialogPrimitive.Close className="ml-auto rounded-sm p-1 opacity-70 transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ring">
               <X className="h-4 w-4" />
               <span className="sr-only">Close</span>
@@ -37,11 +60,12 @@ export function SettingsPane({ open, onClose }: SettingsPaneProps): JSX.Element
 
           {/* Body */}
           <div className="flex min-h-0 flex-1">
-            {/* Nav */}
-            <nav className="flex w-40 shrink-0 flex-col border-r border-border py-2">
+            {/* Desktop nav — always visible */}
+            <nav className="hidden md:flex w-40 shrink-0 flex-col border-r border-border py-2">
               {SECTIONS.map(({ id, label, icon: Icon }) => (
                 <div
                   key={id}
+                  onClick={() => setActiveSection(id)}
                   className={cn(
                     "flex cursor-pointer items-center gap-2.5 px-4 py-2.5 text-sm transition-colors",
                     activeSection === id
@@ -55,8 +79,25 @@ export function SettingsPane({ open, onClose }: SettingsPaneProps): JSX.Element
               ))}
             </nav>
 
+            {/* Mobile nav — section list, shown when no section selected */}
+            {activeSection === null && (
+              <nav className="flex flex-1 flex-col md:hidden">
+                {SECTIONS.map(({ id, label, icon: Icon }) => (
+                  <button
+                    key={id}
+                    onClick={() => setActiveSection(id)}
+                    className="flex items-center gap-3 border-b border-border px-5 py-3.5 text-sm text-foreground transition-colors active:bg-muted"
+                  >
+                    <Icon className="h-4 w-4 shrink-0 text-muted-foreground" />
+                    {label}
+                    <ChevronRight className="ml-auto h-4 w-4 text-muted-foreground" />
+                  </button>
+                ))}
+              </nav>
+            )}
+
             {/* Content */}
-            <div className="min-h-0 min-w-0 flex-1">
+            <div className={cn("min-h-0 min-w-0 flex-1", activeSection === null && "hidden md:block")}>
               {activeSection === "release" && <ReleaseManager />}
             </div>
           </div>
diff --git a/web/src/components/ui/dialog.tsx b/web/src/components/ui/dialog.tsx
index 74d33c58..71d3839b 100644
--- a/web/src/components/ui/dialog.tsx
+++ b/web/src/components/ui/dialog.tsx
@@ -12,7 +12,7 @@ const DialogOverlay = React.forwardRef<
   React.ElementRef<typeof DialogPrimitive.Overlay>,
   React.ComponentPropsWithoutRef<typeof DialogPrimitive.Overlay>
 >(({ className, ...props }, ref) => (
-  <DialogPrimitive.Overlay ref={ref} className={cn("fixed inset-0 z-50 bg-black/70", className)} {...props} />
+  <DialogPrimitive.Overlay ref={ref} className={cn("fixed inset-0 z-[70] bg-black/70", className)} {...props} />
 ));
 DialogOverlay.displayName = DialogPrimitive.Overlay.displayName;
 
@@ -25,7 +25,7 @@ const DialogContent = React.forwardRef<
     <DialogPrimitive.Content
       ref={ref}
       className={cn(
-        "fixed left-1/2 top-1/2 z-50 grid w-[min(560px,calc(100vw-2rem))] -translate-x-1/2 -translate-y-1/2 gap-3 rounded-xl border border-border bg-card p-4 shadow-lg",
+        "fixed left-1/2 top-1/2 z-[70] grid w-[min(560px,calc(100vw-2rem))] -translate-x-1/2 -translate-y-1/2 gap-3 rounded-xl border border-border bg-card p-4 shadow-lg",
         className
       )}
       {...props}

From c3224294d54775dbe9139751ba3873fd1156ab53 Mon Sep 17 00:00:00 2001
From: Brad Harris <brad@studio.localdomain>
Date: Wed, 11 Mar 2026 09:46:54 -0600
Subject: [PATCH 2/3] Use random port for E2E tests in CI

The E2E_PORT was hardcoded to 8799 which causes collisions on the
self-hosted runner when concurrent jobs run or a previous run leaves
the port occupied. Use a random port in the 20000-30000 range, same
pattern as the Postgres container port.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 .github/workflows/ci.yml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index c801c4b8..e0cd8ba6 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -68,11 +68,17 @@ jobs:
           docker exec ${{ steps.postgres.outputs.container }} \
             psql -U dispatch -d postgres -c "CREATE DATABASE dispatch_ci;"
 
+      - name: Pick E2E port
+        id: e2e
+        run: |
+          PORT=$((RANDOM % 10000 + 20000))
+          echo "port=$PORT" >> "$GITHUB_OUTPUT"
+
       - name: E2E tests
         run: npx playwright test
         env:
           CI: "true"
-          E2E_PORT: "8799"
+          E2E_PORT: "${{ steps.e2e.outputs.port }}"
           DATABASE_URL: postgres://dispatch:dispatch@127.0.0.1:${{ steps.postgres.outputs.port }}/dispatch_ci
 
       - name: Upload Playwright report

From 2389664e9b70af0d0a1885c4a57f904ab7862d0b Mon Sep 17 00:00:00 2001
From: Brad Harris <brad@studio.localdomain>
Date: Wed, 11 Mar 2026 09:48:16 -0600
Subject: [PATCH 3/3] Fix settings E2E test for mobile nav changes

Scope the "Release" text selector to the navigation role to avoid
strict mode violation from the new mobile header span.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 e2e/settings.spec.ts | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/e2e/settings.spec.ts b/e2e/settings.spec.ts
index 7cb42d2d..19c3d066 100644
--- a/e2e/settings.spec.ts
+++ b/e2e/settings.spec.ts
@@ -10,12 +10,13 @@ test.describe("Settings pane", () => {
 
     // Settings overlay should appear with a "Settings" heading and "Release" nav item
     await expect(page.getByText("Settings").first()).toBeVisible({ timeout: 3_000 });
-    await expect(page.getByText("Release", { exact: true })).toBeVisible();
+    const releaseNav = page.getByRole("navigation").getByText("Release");
+    await expect(releaseNav).toBeVisible();
 
     // Close it via the X button (sr-only "Close")
     await page.getByRole("button", { name: "Close" }).click();
 
     // The Release nav item should no longer be visible (pane is closed)
-    await expect(page.getByText("Release", { exact: true })).not.toBeVisible({ timeout: 3_000 });
+    await expect(releaseNav).not.toBeVisible({ timeout: 3_000 });
   });
 });