@shipfox/client-workflows 22.0.1 → 22.0.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shipfox/client-workflows",
   "license": "MIT",
-  "version": "22.0.1",
+  "version": "22.0.3",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/ShipfoxHQ/shipfox.git",
@@ -29,14 +29,14 @@
     "date-fns": "^4.1.0",
     "@shipfox/annotations-dto": "12.3.0",
     "@shipfox/api-definitions-dto": "12.3.0",
-    "@shipfox/api-workflows-dto": "13.0.0",
+    "@shipfox/api-workflows-dto": "13.1.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.3",
+    "@shipfox/client-projects": "22.0.3",
+    "@shipfox/client-shell": "22.0.3",
+    "@shipfox/client-triggers": "22.0.3",
+    "@shipfox/client-ui": "22.0.3",
+    "@shipfox/react-ui": "2.2.0"
   },
   "peerDependencies": {
     "@tanstack/react-query": "^5.101.0",

package/src/components/job-detail/agent-config-failure-callout.stories.tsx

@@ -83,6 +83,19 @@ const errorCases: Array<{
       category: 'user',
     },
   },
+  {
+    label: 'Managed-only policy',
+    error: {
+      message: 'This instance only supports provider `shipfox`.',
+      code: 'workspace-providers-disabled',
+      managedProviderId: 'shipfox',
+      exitCode: null,
+      signal: undefined,
+      reason: 'agent_config_invalid',
+      agentConfigIssue: 'provider_unsupported',
+      category: 'user',
+    },
+  },
 ];
 
 export const Playground: Story = {};
@@ -113,6 +126,22 @@ export const TestProviderUnsupported: Story = {
   play: assertCallout('Choose a supported model provider', false),
 };
 
+export const TestManagedOnlyPolicy: Story = {
+  args: {
+    error: {
+      message: 'This instance only supports provider `shipfox`.',
+      code: 'workspace-providers-disabled',
+      managedProviderId: 'shipfox',
+      exitCode: null,
+      signal: undefined,
+      reason: 'agent_config_invalid',
+      agentConfigIssue: 'provider_unsupported',
+      category: 'user',
+    },
+  },
+  play: assertCallout('Use shipfox for this instance', false),
+};
+
 function makeError(agentConfigIssue: AgentConfigIssueValue): StepError {
   return {
     message: 'Agent configuration is invalid',

package/src/components/job-detail/agent-config-failure-callout.tsx

@@ -9,6 +9,8 @@ import {Button} from '@shipfox/react-ui/button';
 import {Link} from '@tanstack/react-router';
 import type {AgentStepConfig, StepError} from '#core/workflow-run.js';
 
+const MANAGED_ONLY_PROVIDER_CODE = 'workspace-providers-disabled';
+
 export function AgentConfigFailureCallout({
   workspaceSlug,
   config,
@@ -47,6 +49,15 @@ function agentConfigFailureCopy(
   config: AgentStepConfig | null,
   error: StepError | null,
 ): {title: string; description: string; showProviderCta: boolean} {
+  const managedOnlyProvider = managedOnlyProviderFromError(error);
+  if (managedOnlyProvider) {
+    return {
+      title: `Use ${managedOnlyProvider} for this instance`,
+      description: `This instance only supports provider \`${managedOnlyProvider}\`. Update this step to use \`${managedOnlyProvider}\`, or remove its provider field to use the managed default.`,
+      showProviderCta: false,
+    };
+  }
+
   const provider = configValue(config?.provider, 'the selected provider');
   const model = configValue(config?.model, 'the selected model');
 
@@ -92,6 +103,13 @@ function agentConfigFailureCopy(
   }
 }
 
+function managedOnlyProviderFromError(error: StepError | null): string | undefined {
+  if (error?.code !== MANAGED_ONLY_PROVIDER_CODE && error?.managedProviderId === undefined) {
+    return undefined;
+  }
+  return error.managedProviderId ?? 'the managed provider';
+}
+
 function configValue(value: string | null | undefined, fallback: string): string {
   return value ?? fallback;
 }

package/src/components/job-detail/job-empty-states.test.ts

@@ -1,3 +1,4 @@
+import type {Step} from '#core/workflow-run.js';
 import {
   workflowJob,
   workflowJobExecutionDto,
@@ -9,6 +10,7 @@ import {
   jobSucceededSummary,
   outputFailureDescriptionForExecution,
   skippedJobDescription,
+  toSelectedAttemptError,
 } from './job-empty-states.js';
 
 describe('jobSucceededSummary', () => {
@@ -32,6 +34,25 @@ describe('jobSucceededSummary', () => {
   });
 });
 
+describe('toSelectedAttemptError', () => {
+  test('preserves managed-provider metadata from a historical attempt', () => {
+    const error = toSelectedAttemptError({type: 'agent'} as Step, {
+      message: 'This instance only supports provider `shipfox`.',
+      code: 'workspace-providers-disabled',
+      managedProviderId: 'shipfox',
+      reason: 'agent_config_invalid',
+      agentConfigIssue: 'provider_unsupported',
+    });
+
+    expect(error).toMatchObject({
+      code: 'workspace-providers-disabled',
+      managedProviderId: 'shipfox',
+      reason: 'agent_config_invalid',
+      agentConfigIssue: 'provider_unsupported',
+    });
+  });
+});
+
 describe('skippedJobDescription', () => {
   test('explains when materialized output exceeds the configured size limit', () => {
     expect(skippedJobDescription('output_too_large')).toBe(

package/src/components/job-detail/job-empty-states.tsx

@@ -292,8 +292,18 @@ export function toSelectedAttemptError(
 
   if (resolvedReason === undefined) return null;
 
+  const code = typeof error.code === 'string' ? error.code : undefined;
+  const managedProviderId =
+    typeof error.managedProviderId === 'string'
+      ? error.managedProviderId
+      : typeof error.managed_provider_id === 'string'
+        ? error.managed_provider_id
+        : undefined;
+
   return {
     message: typeof error.message === 'string' ? error.message : '',
+    ...(code === undefined ? {} : {code}),
+    ...(managedProviderId === undefined ? {} : {managedProviderId}),
     exitCode: exitCode === null || typeof exitCode === 'number' ? exitCode : null,
     signal: typeof error.signal === 'string' ? error.signal : undefined,
     reason: resolvedReason,

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>
   );
 }
 

package/src/core/entities/step.ts

@@ -51,6 +51,8 @@ export interface StepSourceLocation {
 
 export interface StepError {
   message: string;
+  code?: string | undefined;
+  managedProviderId?: string | undefined;
   exitCode: number | null;
   signal: string | undefined;
   reason: StepErrorReason | undefined;

package/src/hooks/api/workflow-run-mapper.ts

@@ -217,6 +217,10 @@ export function toStep(dto: WorkflowRunStepDetailDto): Step {
     error: dto.error
       ? {
           message: dto.error.message,
+          ...(dto.error.code === undefined ? {} : {code: dto.error.code}),
+          ...(dto.error.managed_provider_id === undefined
+            ? {}
+            : {managedProviderId: dto.error.managed_provider_id}),
           exitCode: dto.error.exit_code ?? null,
           signal: dto.error.signal,
           reason: dto.error.reason,