@shipfox/client-workflows 22.0.1 → 22.0.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/client-workflows",
   "license": "MIT",
-  "version": "22.0.1",
+  "version": "22.0.2",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -31,12 +31,12 @@
     "@shipfox/api-definitions-dto": "12.3.0",
     "@shipfox/api-workflows-dto": "13.0.0",
     "@shipfox/client-api": "6.0.1",
-    "@shipfox/client-logs": "22.0.0",
-    "@shipfox/client-projects": "22.0.1",
-    "@shipfox/client-shell": "22.0.1",
-    "@shipfox/client-triggers": "22.0.1",
-    "@shipfox/react-ui": "2.1.0",
-    "@shipfox/client-ui": "22.0.0"
+    "@shipfox/client-logs": "22.0.2",
+    "@shipfox/client-projects": "22.0.2",
+    "@shipfox/client-shell": "22.0.2",
+    "@shipfox/client-triggers": "22.0.2",
+    "@shipfox/client-ui": "22.0.2",
+    "@shipfox/react-ui": "2.1.1"
   },
   "peerDependencies": {
     "@tanstack/react-query": "^5.101.0",

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

@@ -1,5 +1,4 @@
 import {QueryLoadError} from '@shipfox/client-ui';
-import {Panel} from '@shipfox/react-ui/panel';
 import type {WorkflowRunListItem} from '#core/workflow-run.js';
 import type {WorkflowRunListQuery} from './types.js';
 import {
@@ -53,18 +52,14 @@ export function WorkflowRunListContent({
   const showNoMatches = hasLoaded && runs.length === 0 && !showEmptyState;
 
   return (
-    <div className="min-h-0 flex-1 overflow-y-auto">
+    <>
       {isPending ? <WorkflowRunListSkeleton /> : null}
       {!isPending ? (
-        <Panel>
-          <QueryLoadError query={query} subject="workflow runs" icon="pulseLine" variant="panel" />
-        </Panel>
+        <QueryLoadError query={query} subject="workflow runs" icon="pulseLine" variant="panel" />
       ) : null}
       {!isPending && refreshFailed ? <WorkflowRunListStaleError query={query} /> : null}
       {showEmptyState ? (
-        <Panel>
-          <WorkflowRunListEmpty workspaceSlug={workspaceSlug} projectSlug={projectSlug} />
-        </Panel>
+        <WorkflowRunListEmpty workspaceSlug={workspaceSlug} projectSlug={projectSlug} />
       ) : null}
       {showNoMatches ? (
         <WorkflowRunListNoMatches
@@ -86,6 +81,6 @@ export function WorkflowRunListContent({
           />
         </>
       ) : null}
-    </div>
+    </>
   );
 }

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

@@ -2,7 +2,6 @@ import type {QueryLoadErrorQuery} from '@shipfox/client-ui';
 import {Button} from '@shipfox/react-ui/button';
 import {Callout} from '@shipfox/react-ui/callout';
 import {EmptyState} from '@shipfox/react-ui/empty-state';
-import {Panel} from '@shipfox/react-ui/panel';
 import {Skeleton} from '@shipfox/react-ui/skeleton';
 import {Text} from '@shipfox/react-ui/typography';
 import {Link} from '@tanstack/react-router';
@@ -16,7 +15,7 @@ const SKELETON_ROW_COUNT = 8;
  */
 export function WorkflowRunListSkeleton() {
   return (
-    <Panel role="status" aria-label="Loading runs" className="@container divide-y">
+    <div role="status" aria-label="Loading runs" className="@container divide-y">
       {Array.from({length: SKELETON_ROW_COUNT}).map((_, index) => (
         <div
           // biome-ignore lint/suspicious/noArrayIndexKey: skeleton row, stable position
@@ -30,7 +29,7 @@ export function WorkflowRunListSkeleton() {
           <Skeleton className="h-12 w-48" />
         </div>
       ))}
-    </Panel>
+    </div>
   );
 }
 
@@ -114,43 +113,41 @@ export function WorkflowRunListNoMatches({
   onLoadMore?: () => void;
 }) {
   return (
-    <Panel>
-      <div className="flex flex-col gap-inline">
-        {isFetchNextPageError ? (
-          <Callout role="alert" type="error">
-            Could not load more workflow runs. Try again to continue searching older runs.
-          </Callout>
-        ) : null}
-        <EmptyState
-          icon="filterOffLine"
-          title={hasNextPage ? 'No matches in loaded history' : 'No matching runs'}
-          description={
-            hasNextPage
-              ? 'No loaded run matches these filters. Load more to search further back.'
-              : 'No run matches these filters.'
-          }
-          action={
-            <div className="flex flex-wrap justify-center gap-inline">
-              {hasNextPage && onLoadMore ? (
-                <Button
-                  type="button"
-                  size="sm"
-                  variant="primary"
-                  isLoading={isFetchingNextPage}
-                  onClick={onLoadMore}
-                >
-                  Load more runs
-                </Button>
-              ) : null}
-              <Button type="button" size="sm" variant="secondary" onClick={onClear}>
-                Show all runs
+    <div className="flex flex-col gap-inline">
+      {isFetchNextPageError ? (
+        <Callout role="alert" type="error">
+          Could not load more workflow runs. Try again to continue searching older runs.
+        </Callout>
+      ) : null}
+      <EmptyState
+        icon="filterOffLine"
+        title={hasNextPage ? 'No matches in loaded history' : 'No matching runs'}
+        description={
+          hasNextPage
+            ? 'No loaded run matches these filters. Load more to search further back.'
+            : 'No run matches these filters.'
+        }
+        action={
+          <div className="flex flex-wrap justify-center gap-inline">
+            {hasNextPage && onLoadMore ? (
+              <Button
+                type="button"
+                size="sm"
+                variant="primary"
+                isLoading={isFetchingNextPage}
+                onClick={onLoadMore}
+              >
+                Load more runs
               </Button>
-            </div>
-          }
-          variant="panel"
-        />
-      </div>
-    </Panel>
+            ) : null}
+            <Button type="button" size="sm" variant="secondary" onClick={onClear}>
+              Show all runs
+            </Button>
+          </div>
+        }
+        variant="panel"
+      />
+    </div>
   );
 }
 

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

@@ -48,6 +48,7 @@ describe('WorkflowRunListView', () => {
       const body = panel?.querySelector<HTMLElement>('[data-slot="panel-body"]');
 
       expect(panel).not.toBeNull();
+      expect(document.querySelectorAll('[data-slot="panel"]')).toHaveLength(1);
       expect(header).not.toBeNull();
       expect(body).not.toBeNull();
       expect(within(header as HTMLElement).getByLabelText('Search runs')).toBeInTheDocument();
@@ -204,6 +205,7 @@ describe('WorkflowRunListView', () => {
       renderListView([], {query: loadedQuery({isPending: true, data: undefined})});
 
       expect(await screen.findByLabelText('Loading runs')).toBeInTheDocument();
+      expect(document.querySelectorAll('[data-slot="panel"]')).toHaveLength(1);
       expect(screen.queryByText('No runs yet')).not.toBeInTheDocument();
     });
 
@@ -217,6 +219,7 @@ describe('WorkflowRunListView', () => {
       renderListView([]);
 
       expect(await screen.findByText('No runs yet')).toBeInTheDocument();
+      expect(document.querySelectorAll('[data-slot="panel"]')).toHaveLength(1);
       expect(screen.getByRole('link', {name: 'View workflows'})).toBeInTheDocument();
       expect(screen.queryByRole('button', {name: 'Show all runs'})).not.toBeInTheDocument();
     });
@@ -228,6 +231,7 @@ describe('WorkflowRunListView', () => {
       await user.type(await screen.findByLabelText('Search runs'), 'no-such-run');
 
       expect(screen.getByText('No matching runs')).toBeInTheDocument();
+      expect(document.querySelectorAll('[data-slot="panel"]')).toHaveLength(1);
       expect(screen.queryByText('No runs yet')).not.toBeInTheDocument();
       expect(screen.getByRole('button', {name: 'Show all runs'})).toBeInTheDocument();
     });
@@ -248,6 +252,7 @@ describe('WorkflowRunListView', () => {
       });
 
       expect(await screen.findByLabelText('Search runs')).toBeInTheDocument();
+      expect(document.querySelectorAll('[data-slot="panel"]')).toHaveLength(1);
       expect(screen.queryByText('No runs yet')).not.toBeInTheDocument();
       expect(screen.queryByText('No matching runs')).not.toBeInTheDocument();
     });

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

@@ -77,7 +77,7 @@ export function WorkflowRunListView({
               hasActiveFilters={hasActiveFilters}
             />
           </PanelHeader>
-          <PanelBody className="min-h-0 flex-1">
+          <PanelBody className="min-h-0 flex-1 overflow-y-auto">
             <WorkflowRunListContent
               query={query}
               totalRuns={runs.length}

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

@@ -1,6 +1,5 @@
 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';
@@ -38,15 +37,13 @@ export function WorkflowRunRowList({
     // 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>
+    <ul className="@container divide-y divide-border-neutral-base">
+      {runs.map((run) => (
+        <li key={run.id}>
+          <WorkflowRunRow run={run} workspaceSlug={workspaceSlug} projectSlug={projectSlug} />
+        </li>
+      ))}
+    </ul>
   );
 }