fix(enrichment): show Cancelled in details panel for aborted runs

Author: TheodoreSpeaks

apps/sim/app/api/table/[tableId]/rows/[rowId]/enrichment/[groupId]/route.test.ts

@@ -55,6 +55,7 @@ const detail: EnrichmentRunDetail = {
   durationMs: 1000,
   totalCost: 0.05,
   matchedProvider: 'hunter',
+  aborted: false,
   providers: [
     {
       id: 'hunter',

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/enrichment-details/enrichment-details.tsx

@@ -19,7 +19,7 @@ import { MAX_LOG_DETAILS_WIDTH_RATIO, MIN_LOG_DETAILS_WIDTH } from '@/stores/log
 
 type EnrichmentDetailsTab = 'result' | 'cascade'
 
-type ResultStatus = 'matched' | 'no_match' | 'error' | 'not_run'
+type ResultStatus = 'matched' | 'no_match' | 'error' | 'not_run' | 'cancelled'
 
 const RESULT_STATUS_CONFIG: Record<
   ResultStatus,
@@ -29,6 +29,7 @@ const RESULT_STATUS_CONFIG: Record<
   no_match: { variant: 'gray', label: 'No match' },
   error: { variant: 'red', label: 'Error' },
   not_run: { variant: 'gray', label: 'Not run' },
+  cancelled: { variant: 'orange', label: 'Cancelled' },
 }
 
 /** Minimum bar width so a sub-millisecond provider still shows on the timeline. */
@@ -76,11 +77,13 @@ function didRun(p: EnrichmentProviderOutcome): boolean {
 }
 
 /**
- * Derives the cell-level outcome from the cascade — mirrors the executor: a run
- * is `error` only when every provider that ran errored; `not_run` when nothing
- * executed (missing inputs / cancelled early); otherwise a clean `no_match`.
+ * Derives the cell-level outcome from the cascade — mirrors the executor: a
+ * cancelled run is `cancelled` regardless of how far the cascade got; otherwise
+ * `error` only when every provider that ran errored, `not_run` when nothing
+ * executed (missing inputs), else a clean `no_match`.
  */
 function deriveResultStatus(detail: EnrichmentRunDetail): ResultStatus {
+  if (detail.aborted) return 'cancelled'
   if (detail.matchedProvider) return 'matched'
   const ran = detail.providers.filter(didRun)
   if (ran.length === 0) return 'not_run'

apps/sim/background/workflow-column-execution.ts

@@ -335,7 +335,7 @@ async function runWorkflowAndWriteTerminal(
             jobId: null,
             workflowId: statusId,
             error: 'Cancelled',
-            enrichmentDetails: skippedEnrichmentDetail(enrichment),
+            enrichmentDetails: skippedEnrichmentDetail(enrichment, { aborted: true }),
           })
           return 'error'
         }

apps/sim/enrichments/run.test.ts

@@ -137,6 +137,7 @@ describe('runEnrichment cascade detail', () => {
       }
     )
     expect(mockExecuteTool).not.toHaveBeenCalled()
+    expect(outcome.detail.aborted).toBe(true)
     expect(outcome.detail.providers.map((p) => p.status)).toEqual(['not_run', 'not_run'])
   })
 

apps/sim/enrichments/run.ts

@@ -30,14 +30,18 @@ export interface EnrichmentRunOutcome {
  * marked `skipped` so the details panel stays informative (shows the configured
  * cascade) instead of empty.
  */
-export function skippedEnrichmentDetail(enrichment: EnrichmentConfig): EnrichmentRunDetail {
+export function skippedEnrichmentDetail(
+  enrichment: EnrichmentConfig,
+  opts: { aborted?: boolean } = {}
+): EnrichmentRunDetail {
   const now = new Date().toISOString()
   return {
     startedAt: now,
     completedAt: now,
     durationMs: 0,
     totalCost: 0,
     matchedProvider: null,
+    aborted: opts.aborted ?? false,
     providers: enrichment.providers.map((provider) => ({
       id: provider.id,
       label: provider.label,
@@ -204,6 +208,7 @@ export async function runEnrichment(
     durationMs: completedAt - startedAt,
     totalCost: cost,
     matchedProvider,
+    aborted: Boolean(ctx.signal?.aborted),
     providers,
   }
 

apps/sim/lib/table/types.ts

@@ -180,6 +180,9 @@ export interface EnrichmentRunDetail {
   totalCost: number
   /** Provider id that produced the match, or `null` on no match. */
   matchedProvider: string | null
+  /** True when the run was cancelled (stop / signal abort) — drives a
+   *  "Cancelled" result rather than inferring no-match/not-run from the cascade. */
+  aborted: boolean
   /** Every configured provider, in cascade order (including `not_run` ones). */
   providers: EnrichmentProviderOutcome[]
 }