@shipfox/client-workflows 20.0.0 → 22.0.0

package/src/components/workflow-run-list/workflow-run-list-view.test.tsx

@@ -1,4 +1,5 @@
 import type {JobStatusDto} from '@shipfox/api-workflows-dto';
+import {WORKFLOW_RUN_JOB_PREVIEW_LIMIT} from '@shipfox/api-workflows-dto';
 import {screen, within} from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import type {WorkflowRunListItem, WorkflowRunStatus} from '#core/workflow-run.js';
@@ -8,12 +9,14 @@ import {
   workflowRunListItem,
 } from '#test/fixtures/workflow-run.js';
 import {renderWithRouter} from '#test/render.js';
+import {JOB_STATUS_STRIP_WIDTH, jobStatusStripWidthForPreview} from './job-status-strip.js';
 import type {WorkflowRunListQuery, WorkflowRunListViewProps} from './types.js';
 import {WorkflowRunListView} from './workflow-run-list-view.js';
 
 const PROJECT_ID = '44444444-4444-4444-8444-444444444444';
 const JOB_STRIP_LABEL_RE = /jobs:/u;
 const JOB_SUMMARY_NAME_RE = /queued-build/u;
+const OVERFLOW_COUNT_RE = /^\+/u;
 const WORKSPACE_SLUG = 'acme';
 const PROJECT_SLUG = 'checkout-api';
 
@@ -35,6 +38,38 @@ describe('WorkflowRunListView', () => {
     vi.restoreAllMocks();
   });
 
+  describe('panel composition', () => {
+    test('keeps the controls in the panel header and states in its body', async () => {
+      renderListView([run('succeeded', 'build-image')]);
+      await screen.findByText('build-image');
+
+      const panel = document.querySelector<HTMLElement>('[data-slot="panel"]');
+      const header = panel?.querySelector<HTMLElement>('[data-slot="panel-header"]');
+      const body = panel?.querySelector<HTMLElement>('[data-slot="panel-body"]');
+
+      expect(panel).not.toBeNull();
+      expect(header).not.toBeNull();
+      expect(body).not.toBeNull();
+      expect(within(header as HTMLElement).getByLabelText('Search runs')).toBeInTheDocument();
+      expect(
+        within(header as HTMLElement).getByRole('button', {name: filterTrigger('Status')}),
+      ).toBeInTheDocument();
+      expect(
+        within(header as HTMLElement).getByRole('button', {name: filterTrigger('Branch')}),
+      ).toBeInTheDocument();
+      expect(
+        within(header as HTMLElement).getByRole('button', {name: filterTrigger('Actor')}),
+      ).toBeInTheDocument();
+      expect(
+        within(header as HTMLElement).getByRole('button', {name: filterTrigger('Event')}),
+      ).toBeInTheDocument();
+      expect(
+        within(header as HTMLElement).getByLabelText('Filter runs by creation date'),
+      ).toBeInTheDocument();
+      expect(within(body as HTMLElement).getByText('build-image')).toBeInTheDocument();
+    });
+  });
+
   describe('filtering', () => {
     test('narrows the list to the selected status', async () => {
       const user = userEvent.setup();
@@ -380,10 +415,24 @@ describe('WorkflowRunListView', () => {
     test('counts the jobs it had to hide beyond the strip threshold', async () => {
       renderListView([run('running', 'deploy-web', 'run-1', {...workflowRunJobsOfStatus(40)})]);
 
-      expect(await screen.findByText('+33')).toBeInTheDocument();
+      expect(await screen.findByText('+24')).toBeInTheDocument();
       expect(screen.getByRole('img', {name: '40 jobs: 40 succeeded'})).toBeInTheDocument();
     });
 
+    test('renders every job in the API preview without a client-side cap', async () => {
+      renderListView([run('running', 'deploy-web', 'run-1', {...workflowRunJobsOfStatus(16)})]);
+
+      const strip = await screen.findByRole('img', {name: '16 jobs: 16 succeeded'});
+
+      expect(within(strip).getAllByLabelText('Succeeded')).toHaveLength(16);
+      expect(within(strip).queryByText(OVERFLOW_COUNT_RE)).not.toBeInTheDocument();
+    });
+
+    test('reserves the full API preview and overflow marker width', () => {
+      expect(jobStatusStripWidthForPreview(WORKFLOW_RUN_JOB_PREVIEW_LIMIT)).toBe(294);
+      expect(JOB_STATUS_STRIP_WIDTH).toBe(294);
+    });
+
     // The API sends a bounded preview, so a run whose only failure sits past it has no failed
     // job in the payload at all. The counts still carry it, and the overflow glyph has to say
     // so rather than showing an unbroken row of green.
@@ -398,7 +447,7 @@ describe('WorkflowRunListView', () => {
         name: '21 jobs: 1 failed, 20 succeeded',
       });
       expect(strip).toBeInTheDocument();
-      expect(screen.getByText('+14')).toBeInTheDocument();
+      expect(screen.getByText('+5')).toBeInTheDocument();
       expect(within(strip).getByLabelText('Failed')).toBeInTheDocument();
     });
 
@@ -420,8 +469,8 @@ describe('WorkflowRunListView', () => {
       ]);
 
       const strip = await screen.findByRole('img', {name: '20 jobs: 20 listening'});
-      expect(screen.getByText('+13')).toBeInTheDocument();
-      expect(within(strip).getAllByLabelText('Listening')).toHaveLength(8);
+      expect(screen.getByText('+4')).toBeInTheDocument();
+      expect(within(strip).getAllByLabelText('Listening')).toHaveLength(17);
 
       const user = userEvent.setup();
       await user.hover(strip);

package/src/components/workflow-run-list/workflow-run-list-view.tsx

@@ -1,3 +1,4 @@
+import {Panel, PanelBody, PanelHeader} from '@shipfox/react-ui/panel';
 import {TimeTickerProvider} from '@shipfox/react-ui/time-ticker';
 import {Header} from '@shipfox/react-ui/typography';
 import {cn} from '@shipfox/react-ui/utils';
@@ -58,7 +59,7 @@ export function WorkflowRunListView({
     <TimeTickerProvider intervalMs={1000} reducedMotionIntervalMs={10_000}>
       <section
         aria-labelledby={headingId}
-        className={cn('flex min-h-0 min-w-0 flex-1 flex-col gap-cluster', className)}
+        className={cn('flex min-h-0 min-w-0 flex-1 flex-col', className)}
       >
         {/* The tab strip above already reads "Runs", so the page heading is for structure and
             assistive tech rather than a second visible title competing with it. */}
@@ -66,27 +67,32 @@ export function WorkflowRunListView({
           Workflow runs
         </Header>
 
-        <WorkflowRunFilters
-          search={currentSearch}
-          facets={facets}
-          onChange={handleFiltersChange}
-          onClear={handleClearFilters}
-          hasActiveFilters={hasActiveFilters}
-        />
-
-        <WorkflowRunListContent
-          query={query}
-          totalRuns={runs.length}
-          runs={filteredRuns}
-          workspaceSlug={workspaceSlug}
-          projectSlug={projectSlug}
-          hasActiveFilters={hasActiveFilters}
-          onClearFilters={handleClearFilters}
-          hasNextPage={hasNextPage}
-          isFetchingNextPage={isFetchingNextPage}
-          isFetchNextPageError={isFetchNextPageError}
-          {...(onLoadMore ? {onLoadMore} : {})}
-        />
+        <Panel className="min-h-0 flex-1">
+          <PanelHeader className="flex-wrap">
+            <WorkflowRunFilters
+              search={currentSearch}
+              facets={facets}
+              onChange={handleFiltersChange}
+              onClear={handleClearFilters}
+              hasActiveFilters={hasActiveFilters}
+            />
+          </PanelHeader>
+          <PanelBody className="min-h-0 flex-1">
+            <WorkflowRunListContent
+              query={query}
+              totalRuns={runs.length}
+              runs={filteredRuns}
+              workspaceSlug={workspaceSlug}
+              projectSlug={projectSlug}
+              hasActiveFilters={hasActiveFilters}
+              onClearFilters={handleClearFilters}
+              hasNextPage={hasNextPage}
+              isFetchingNextPage={isFetchingNextPage}
+              isFetchNextPageError={isFetchNextPageError}
+              {...(onLoadMore ? {onLoadMore} : {})}
+            />
+          </PanelBody>
+        </Panel>
       </section>
     </TimeTickerProvider>
   );

package/src/components/workflow-run-list/workflow-run-list.stories.tsx

@@ -75,15 +75,15 @@ const SAMPLE_RUNS: WorkflowRunListItem[] = [
 
 // One story on the full-width page. The data states (loading / empty / errors / runs) are
 // driven by args; search, the filter menus, "clear filters" and "no matches" are live in the
-// rendered toolbar since the view owns that state. The decorator gives the list the 1120px
-// content cap and a real height, so `flex-1` scrolls the way it does in the app shell.
+// rendered toolbar since the view owns that state. The decorator gives the list a full-width
+// subtle canvas and a real height, so `flex-1` scrolls the way it does in the app shell.
 const meta = {
   title: 'Workflows/WorkflowRunList',
   component: WorkflowRunListView,
   parameters: {layout: 'centered'},
   decorators: [
     (Story) => (
-      <div className="flex h-600 w-[1120px] max-w-full bg-background-neutral-base p-24">
+      <div className="flex h-600 w-full bg-background-subtle-base p-24">
         <Story />
       </div>
     ),
@@ -137,27 +137,18 @@ export const NarrowLayout: Story = {
 };
 
 /**
- * Row density against the three strip sizes the threshold has to survive: comfortably under
- * it, comfortably over it, and far enough over that the overflow count carries the failure.
+ * Row density against the three strip sizes the preview bound has to survive: comfortably under
+ * it, just over it, and far enough over that the overflow count carries the failure.
  */
 export const JobStripDensity: Story = {
   args: {
     runs: [
       makeRun('succeeded', 'two-jobs', 2, {jobs: ['succeeded', 'succeeded']}),
-      makeRun('failed', 'twelve-jobs', 6, {
+      makeRun('failed', 'seventeen-jobs', 6, {
         jobs: [
-          'succeeded',
-          'succeeded',
-          'succeeded',
+          ...(Array.from({length: 12}, () => 'succeeded') as JobStatusDto[]),
           'failed',
-          'succeeded',
-          'succeeded',
-          'skipped',
-          'skipped',
-          'succeeded',
-          'succeeded',
-          'succeeded',
-          'succeeded',
+          ...(Array.from({length: 4}, () => 'skipped') as JobStatusDto[]),
         ],
       }),
       makeRun('running', 'forty-jobs', 9, {
@@ -229,9 +220,7 @@ function StateExample({label, children}: {label: string; children: ReactNode}) {
       <Code variant="label" className="text-foreground-neutral-subtle">
         {label}
       </Code>
-      <div className="flex h-560 rounded-8 border border-border-neutral-base bg-background-neutral-base p-16">
-        {children}
-      </div>
+      <div className="flex h-560 min-w-0 bg-background-subtle-base p-16">{children}</div>
     </div>
   );
 }

package/src/components/workflow-run-list/workflow-run-row.tsx

@@ -1,5 +1,6 @@
 import {TriggerSourceIcon} from '@shipfox/client-triggers';
 import {Icon, type IconName} from '@shipfox/react-ui/icon';
+import {Panel} from '@shipfox/react-ui/panel';
 import {RelativeTime} from '@shipfox/react-ui/relative-time';
 import {Tooltip, TooltipContent, TooltipTrigger} from '@shipfox/react-ui/tooltip';
 import {Code, Text} from '@shipfox/react-ui/typography';
@@ -22,7 +23,7 @@ import {
   workflowRunCommitLabel,
 } from '#core/workflow-run.js';
 import {withoutWorkflowRunSelectionSearch} from '#core/workflow-run-url-state.js';
-import {JobStatusStrip, jobStatusSummary} from './job-status-strip.js';
+import {JOB_STATUS_STRIP_WIDTH, JobStatusStrip, jobStatusSummary} from './job-status-strip.js';
 
 export function WorkflowRunRowList({
   runs,
@@ -34,16 +35,18 @@ export function WorkflowRunRowList({
   projectSlug?: string | undefined;
 }) {
   return (
-    // A container, not the viewport, drives the row's breakpoints. The list is capped at
-    // 1120px inside a wider page, so what a row can afford is its own width; keying off the
-    // viewport would keep the job strip hidden on a wide screen or crush it on a narrow one.
-    <ul className="@container divide-y divide-border-neutral-base border-t border-border-neutral-base">
-      {runs.map((run) => (
-        <li key={run.id}>
-          <WorkflowRunRow run={run} workspaceSlug={workspaceSlug} projectSlug={projectSlug} />
-        </li>
-      ))}
-    </ul>
+    // A container, not the viewport, drives the row's breakpoints. What a row can afford is its
+    // own width; keying off the viewport would keep the job strip hidden on a wide screen or
+    // crush it on a narrow one.
+    <Panel className="@container">
+      <ul className="divide-y divide-border-neutral-base">
+        {runs.map((run) => (
+          <li key={run.id}>
+            <WorkflowRunRow run={run} workspaceSlug={workspaceSlug} projectSlug={projectSlug} />
+          </li>
+        ))}
+      </ul>
+    </Panel>
   );
 }
 
@@ -55,9 +58,9 @@ export function WorkflowRunRowList({
  * drops beneath. The numeric columns are fixed-width so duration and time form real columns
  * down the list rather than tracking each row's name length.
  *
- * The thresholds are the container widths the spec's viewport breakpoints produce, given a
- * column capped at 1120px with 24px of page padding on each side: a 1280px viewport leaves
- * the row 1072px, and a 1024px viewport leaves it 976px.
+ * The 976px threshold keeps identity and provenance on one line; below it, provenance drops
+ * beneath the identity. The 1200px threshold keeps the full job preview and numeric columns
+ * readable together; below it, the strip is hidden so the numeric columns remain aligned.
  */
 export function WorkflowRunRow({
   run,
@@ -99,10 +102,14 @@ export function WorkflowRunRow({
       </div>
 
       <span className="flex shrink-0 items-center gap-cluster">
-        {/* Fixed at the strip's worst case (see MAX_VISIBLE_JOBS) so the overflow count cannot
-            paint over the duration, and kept even when a run has no jobs planned yet, so the
-            numerics stay in line down the list instead of stepping in and out. */}
-        <span className="hidden w-160 @min-[1040px]:flex">
+        {/* Reserve the API preview's full width so the overflow count cannot paint over the
+            duration, and keep it even when a run has no jobs planned yet so the numerics stay in
+            line down the list instead of stepping in and out. The max-content width expands if
+            the payload ever exceeds the current API preview bound. */}
+        <span
+          className="hidden w-max shrink-0 @min-[1200px]:flex"
+          style={{minWidth: JOB_STATUS_STRIP_WIDTH}}
+        >
           <JobStatusStrip jobs={run.jobs} />
         </span>
         <span className="flex w-64 justify-end">
@@ -121,7 +128,7 @@ export function WorkflowRunRow({
   // Rows run edge to edge inside the list's scroll container, which would clip the standard
   // outset focus ring, so this one is inset per the design system's focus-ring rule.
   const rowClassName =
-    'flex w-full min-w-0 items-center gap-inline px-row py-row text-left transition-colors hover:bg-background-components-hover focus-visible:shadow-[inset_0_0_0_2px_var(--color-primary-500)] focus-visible:outline-none @min-[976px]:h-44 @min-[976px]:py-0';
+    'flex w-full min-w-0 items-center gap-inline px-row py-row text-left transition-colors hover:bg-background-neutral-hover focus-visible:shadow-focus-inset focus-visible:outline-none @min-[976px]:h-44 @min-[976px]:py-0';
 
   // Optimistic manual runs (temp-<uuid>) have no detail page until the canonical row
   // replaces them on the next poll, so they render non-interactively instead of as a link

package/src/components/workflow-run-summary/workflow-run-summary.stories.tsx

@@ -56,10 +56,10 @@ const RUN_ATTEMPTS_RESPONSE = runAttemptsResponseDto({
 });
 
 const withFrame: Decorator = (Story) => (
-  <div className="min-h-screen bg-background-neutral-background">
-    <div className="mx-auto flex min-h-screen w-full max-w-[1120px] flex-col overflow-hidden border-x border-border-neutral-base bg-background-subtle-base">
+  <div className="min-h-screen bg-background-subtle-base">
+    <div className="flex min-h-screen w-full flex-col px-frame py-frame">
       <Story />
-      <div className="min-h-0 flex-1 bg-background-neutral-background p-16" />
+      <div className="min-h-0 flex-1 bg-background-subtle-base p-16" />
     </div>
   </div>
 );

package/src/components/workflow-run-summary/workflow-run-summary.test.tsx

@@ -38,7 +38,8 @@ describe('WorkflowRunSummary', () => {
 
     const summary = await screen.findByRole('region', {name: 'deploy-web'});
 
-    expect(summary).toHaveClass('bg-background-neutral-background', 'px-row', 'py-row');
+    expect(summary).toHaveClass('px-row', 'py-row');
+    expect(summary).not.toHaveClass('bg-background-subtle-base');
     expect(summary.firstElementChild).not.toHaveClass('max-w-[1120px]');
     expect(within(summary).getByRole('heading', {name: 'deploy-web'})).toBeInTheDocument();
     expect(within(summary).getAllByText('Running')).not.toHaveLength(0);
@@ -298,7 +299,15 @@ describe('WorkflowRunSummary', () => {
     const onCancel = vi.fn();
     renderSummary({}, {onCancel});
 
-    await user.click(await screen.findByRole('button', {name: 'Cancel workflow'}));
+    const button = await screen.findByRole('button', {name: 'Cancel workflow'});
+    expect(button).toHaveClass(
+      'bg-background-button-danger-default',
+      'text-foreground-neutral-on-color',
+      'shadow-button-danger',
+    );
+    expect(button).not.toHaveClass('bg-background-neutral-base');
+
+    await user.click(button);
 
     expect(onCancel).toHaveBeenCalledTimes(1);
   });

package/src/components/workflow-run-summary/workflow-run-summary.tsx

@@ -29,7 +29,7 @@ import {WorkflowRunAttemptSwitcher} from './workflow-run-attempt-switcher.js';
 const STATUS_BADGE_LABEL_WIDTH_CH = Math.max(
   ...WORKFLOW_RUN_STATUSES.map((status) => getWorkflowStatusVisual(status).label.length),
 );
-const RERUN_BUTTON_SURFACE_CLASS_NAME =
+const NEUTRAL_ACTION_SURFACE_CLASS_NAME =
   'bg-background-neutral-base hover:bg-background-neutral-hover active:bg-background-neutral-pressed';
 
 type WorkflowRunAction = 'cancel' | 'rerun-all' | 'rerun-menu' | 'none';
@@ -70,10 +70,7 @@ export function WorkflowRunSummary({
 
   return (
     <TimeTickerProvider intervalMs={1000} reducedMotionIntervalMs={10_000}>
-      <section
-        aria-labelledby={headingId}
-        className="bg-background-neutral-background px-row py-row"
-      >
+      <section aria-labelledby={headingId} className="px-row py-row">
         <div className="grid min-w-0 grid-cols-[minmax(0,1fr)_auto] items-center gap-x-cluster gap-y-inline overflow-hidden max-[480px]:grid-cols-1">
           <div className="col-start-1 row-start-1 min-w-0 max-[480px]:col-start-auto max-[480px]:row-start-auto">
             <div className="flex min-w-0 items-center gap-inline">
@@ -228,7 +225,7 @@ function WorkflowRunActionSlot({
         type="button"
         variant="secondary"
         size="xs"
-        className={RERUN_BUTTON_SURFACE_CLASS_NAME}
+        className={NEUTRAL_ACTION_SURFACE_CLASS_NAME}
         isLoading={rerunPending}
         disabled={rerunPending}
         onClick={() => onRerun('all')}
@@ -247,7 +244,7 @@ function WorkflowRunActionSlot({
           type="button"
           variant="secondary"
           size="xs"
-          className={RERUN_BUTTON_SURFACE_CLASS_NAME}
+          className={NEUTRAL_ACTION_SURFACE_CLASS_NAME}
           iconRight="arrowDownSLine"
           isLoading={rerunPending}
           disabled={rerunPending}

package/src/components/workflow-run-view/run-workspace-nav.test.tsx

@@ -13,6 +13,7 @@ const RUN_ID = '66666666-6666-4666-8666-666666666666';
 const CURRENT_JOB_ID = '88888888-8888-4888-8888-888888888888';
 const BUILD_LINK_PATTERN = /build/;
 const DEPLOY_LINK_PATTERN = /deploy/;
+const SETUP_LINK_PATTERN = /setup/;
 const NOW = Date.parse('2026-06-26T12:00:00.000Z');
 
 describe('RunWorkspaceNav', () => {
@@ -92,7 +93,8 @@ describe('RunWorkspaceNav', () => {
       />,
     );
 
-    expect(await screen.findByRole('link', {name: 'Summary'})).toBeInTheDocument();
+    const summary = await screen.findByRole('link', {name: 'Summary'});
+    expect(summary).toBeInTheDocument();
     expect(screen.getByRole('heading', {name: 'Jobs'})).toBeInTheDocument();
     expect(screen.getByRole('heading', {name: 'Run details'})).toBeInTheDocument();
     expect(screen.getByRole('link', {name: 'Annotations'})).toBeInTheDocument();
@@ -112,6 +114,13 @@ describe('RunWorkspaceNav', () => {
 
     const current = within(jobs).getByRole('link', {name: BUILD_LINK_PATTERN});
     expect(current).toHaveAttribute('aria-current', 'page');
+    expect(current.querySelector('[data-run-workspace-active-bar]')).not.toBeNull();
+    expect(
+      within(jobs)
+        .getByRole('link', {name: SETUP_LINK_PATTERN})
+        .querySelector('[data-run-workspace-active-bar]'),
+    ).toBeNull();
+    expect(jobs.querySelectorAll('[data-run-workspace-active-bar]')).toHaveLength(1);
     expect(current).toHaveAttribute(
       'href',
       expect.stringContaining(`/runs/${RUN_ID}/jobs/${CURRENT_JOB_ID}`),
@@ -137,7 +146,36 @@ describe('RunWorkspaceNav', () => {
 
     expect(await screen.findByRole('heading', {name: 'Jobs'})).toBeInTheDocument();
     expect(screen.getByRole('link', {name: BUILD_LINK_PATTERN})).toBeInTheDocument();
-    expect(screen.getByRole('link', {name: 'Summary'})).toHaveAttribute('aria-current', 'page');
+    const summary = screen.getByRole('link', {name: 'Summary'});
+    expect(summary).toHaveAttribute('aria-current', 'page');
+    expect(summary.querySelector('[data-run-workspace-active-bar]')).not.toBeNull();
+  });
+
+  test('falls back to the first job when the current job id is stale', async () => {
+    const run = workflowRunDetail({
+      id: RUN_ID,
+      jobs: [
+        workflowJobDto({id: 'job-deploy', name: 'deploy', position: 1}),
+        workflowJobDto({id: 'job-setup', name: 'setup', position: 0}),
+      ],
+    });
+
+    renderWithRouter(
+      <RunWorkspaceNav
+        workspaceSlug="acme"
+        projectSlug="project"
+        run={run}
+        activeSection="summary"
+        currentJobId="stale-job-id"
+      />,
+    );
+
+    const jobs = (await screen.findByRole('heading', {name: 'Jobs'})).closest('section');
+    if (!jobs) throw new Error('Missing jobs section');
+
+    const fallback = within(jobs).getByRole('link', {name: SETUP_LINK_PATTERN});
+    expect(fallback).toHaveAttribute('aria-current', 'page');
+    expect(jobs.querySelectorAll('[data-run-workspace-active-bar]')).toHaveLength(1);
   });
 
   test('scrolls the current job into view when the mobile rail opens', async () => {

package/src/components/workflow-run-view/run-workspace-nav.tsx

@@ -23,6 +23,9 @@ import {JobExecutionTimeText} from '../job-detail/job-execution-time-text.js';
 
 type RunWorkspaceSection = Exclude<WorkflowRunTab, 'jobs'>;
 
+const RUN_WORKSPACE_LINK_CLASS_NAME =
+  'relative flex min-h-32 items-center gap-inline rounded-4 px-tight outline-none transition-colors hover:bg-background-neutral-hover focus-visible:shadow-border-interactive-with-active @max-[767px]:min-h-44 [@media(pointer:coarse)]:min-h-44';
+
 export interface RunWorkspaceNavProps {
   workspaceSlug: string;
   projectSlug: string;
@@ -48,12 +51,14 @@ export function RunWorkspaceNav({
 }: RunWorkspaceNavProps) {
   const jobs = useMemo(() => [...run.jobs].sort(compareJobs), [run.jobs]);
   const [mobileOpen, setMobileOpen] = useState(false);
-  const currentJob = jobs.find((job) => job.id === currentJobId);
+  const currentJob =
+    jobs.find((job) => job.id === currentJobId) ?? (currentJobId ? jobs[0] : undefined);
+  const resolvedCurrentJobId = currentJob?.id;
   const currentLabel = currentJob?.displayName ?? sectionLabel(activeSection);
 
   return (
     <TimeTickerProvider intervalMs={1000} reducedMotionIntervalMs={10_000}>
-      <aside className="flex min-h-0 w-full shrink-0 flex-col border-b border-border-neutral-base bg-background-neutral-background min-[768px]:w-240 min-[768px]:border-b-0 min-[768px]:border-r">
+      <aside className="flex min-h-0 w-full shrink-0 flex-col border-b border-border-neutral-base min-[768px]:w-240 min-[768px]:border-b-0">
         <Collapsible
           open={mobileOpen}
           onOpenChange={setMobileOpen}
@@ -96,7 +101,7 @@ export function RunWorkspaceNav({
               run={run}
               jobs={jobs}
               activeSection={activeSection}
-              currentJobId={currentJobId}
+              currentJobId={resolvedCurrentJobId}
               jobSearch={jobSearch}
               annotationSummary={annotationSummary}
               mobileOpen={mobileOpen}
@@ -140,8 +145,8 @@ function RunWorkspaceNavContent({
   }, [currentJobId, mobileOpen]);
 
   return (
-    <nav aria-label="Run workspace" className="flex min-h-0 min-w-0 flex-1 flex-col gap-group">
-      <ul>
+    <nav aria-label="Run workspace" className="flex min-h-0 min-w-0 flex-1 flex-col">
+      <ul className="border-b border-border-neutral-base pb-row">
         <li>
           <RunSectionLink
             workspaceSlug={workspaceSlug}
@@ -157,7 +162,7 @@ function RunWorkspaceNavContent({
 
       <section
         aria-labelledby="run-workspace-jobs-heading"
-        className="flex min-h-0 min-w-0 flex-1 flex-col"
+        className="flex min-h-0 min-w-0 flex-1 flex-col border-b border-border-neutral-base py-row"
       >
         <div className="flex items-center justify-between gap-inline px-tight pb-[6px]">
           <Text
@@ -189,10 +194,12 @@ function RunWorkspaceNavContent({
                   onClick={onNavigate}
                   aria-current={current ? 'page' : undefined}
                   className={cn(
-                    'flex min-h-32 min-w-0 items-center gap-inline rounded-4 px-tight text-left outline-none transition-colors hover:bg-background-neutral-hover focus-visible:shadow-border-interactive-with-active @max-[767px]:min-h-44 [@media(pointer:coarse)]:min-h-44',
+                    RUN_WORKSPACE_LINK_CLASS_NAME,
+                    'min-w-0 text-left',
                     current && 'bg-background-neutral-hover',
                   )}
                 >
+                  {current ? <RunWorkspaceActiveBar /> : null}
                   <WorkflowStatusIcon
                     status={deriveJobDisplayStatus(job)}
                     size={12}
@@ -211,7 +218,7 @@ function RunWorkspaceNavContent({
         </ol>
       </section>
 
-      <section aria-labelledby="run-workspace-details-heading">
+      <section aria-labelledby="run-workspace-details-heading" className="pt-row">
         <Text
           as="h2"
           id="run-workspace-details-heading"
@@ -292,10 +299,12 @@ function RunSectionLink({
           : undefined
       }
       className={cn(
-        'flex min-h-32 items-center gap-inline rounded-4 px-tight text-xs font-medium text-foreground-neutral-base outline-none transition-colors hover:bg-background-neutral-hover focus-visible:shadow-border-interactive-with-active @max-[767px]:min-h-44 [@media(pointer:coarse)]:min-h-44',
+        RUN_WORKSPACE_LINK_CLASS_NAME,
+        'text-xs font-medium text-foreground-neutral-base',
         current && 'bg-background-neutral-hover',
       )}
     >
+      {current ? <RunWorkspaceActiveBar /> : null}
       {sectionLabel(section)}
       {count ? (
         <span
@@ -310,6 +319,16 @@ function RunSectionLink({
   );
 }
 
+function RunWorkspaceActiveBar() {
+  return (
+    <span
+      aria-hidden="true"
+      data-run-workspace-active-bar
+      className="absolute inset-y-0 left-0 w-2 rounded-l-4 bg-border-highlights-interactive"
+    />
+  );
+}
+
 function sectionLabel(section: RunWorkspaceSection): string {
   if (section === 'annotations') return 'Annotations';
   if (section === 'source') return 'Source';

package/src/components/workflow-run-view/workflow-run-states.tsx

@@ -7,10 +7,7 @@ import {Text} from '@shipfox/react-ui/typography';
 
 export function WorkflowRunSkeleton() {
   return (
-    <section
-      aria-label="Loading workflow run"
-      className="bg-background-neutral-background px-row py-row"
-    >
+    <section aria-label="Loading workflow run" className="px-row py-row">
       <div className="flex min-w-0 flex-wrap items-center gap-x-cluster gap-y-inline">
         <div className="flex min-w-0 items-center gap-inline">
           <Skeleton className="size-8 rounded-full" />