[cli] Fix false "data expired" warning in CLI for non-expired runs (#1736)

* fix cli expiredAt check

* fix cli expiredAt check

* fix cli expiredAt check

Author: karthikscale3

.changeset/many-peas-jog.md

@@ -0,0 +1,5 @@
+---
+"@workflow/cli": patch
+---
+
+Fix false "data expired" warning for runs with future expiredAt

packages/cli/src/lib/inspect/output.test.ts

@@ -0,0 +1,60 @@
+import { describe, expect, it } from 'vitest';
+import type { WorkflowRun } from '@workflow/world';
+import { formatTableValue, hasExpiredData } from './output.js';
+
+const makeRun = (overrides: Partial<WorkflowRun> = {}): WorkflowRun =>
+  ({
+    runId: 'run-1',
+    status: 'running',
+    deploymentId: 'dep-1',
+    workflowName: 'workflow//./src/workflows/test//myWorkflow',
+    input: undefined,
+    output: undefined,
+    error: undefined,
+    createdAt: new Date('2025-01-01'),
+    updatedAt: new Date('2025-01-01'),
+    completedAt: undefined,
+    startedAt: undefined,
+    expiredAt: undefined,
+    specVersion: 2,
+    executionContext: {},
+    ...overrides,
+  }) as unknown as WorkflowRun;
+
+describe('hasExpiredData', () => {
+  it('returns false when expiredAt is undefined', () => {
+    expect(hasExpiredData(makeRun({ expiredAt: undefined }))).toBe(false);
+  });
+
+  it('returns false when expiredAt is in the future', () => {
+    const future = new Date(Date.now() + 24 * 60 * 60 * 1000);
+    expect(hasExpiredData(makeRun({ expiredAt: future }))).toBe(false);
+  });
+
+  it('returns true when expiredAt is in the past', () => {
+    const past = new Date(Date.now() - 24 * 60 * 60 * 1000);
+    expect(hasExpiredData(makeRun({ expiredAt: past }))).toBe(true);
+  });
+});
+
+describe('formatTableValue expired data handling', () => {
+  it('returns input value when expiredAt is in the future', () => {
+    const future = new Date(Date.now() + 24 * 60 * 60 * 1000);
+    const item = { expiredAt: future.toISOString(), input: 'hello' };
+    const result = formatTableValue('input', 'hello', {}, undefined, item);
+    expect(result).not.toContain('expired');
+  });
+
+  it('returns expired message when expiredAt is in the past', () => {
+    const past = new Date(Date.now() - 24 * 60 * 60 * 1000);
+    const item = { expiredAt: past.toISOString(), output: 'hello' };
+    const result = formatTableValue('output', 'hello', {}, undefined, item);
+    expect(String(result)).toContain('data expired');
+  });
+
+  it('returns input value when expiredAt is not present', () => {
+    const item = { input: 'hello' };
+    const result = formatTableValue('input', 'hello', {}, undefined, item);
+    expect(String(result)).not.toContain('expired');
+  });
+});

packages/cli/src/lib/inspect/output.ts

@@ -276,7 +276,7 @@ const formatIdField = (
   return idStr;
 };
 
-const formatTableValue = (
+export const formatTableValue = (
   prop: string,
   value: unknown,
   opts: InspectCLIOptions = {},
@@ -318,8 +318,12 @@ const formatTableValue = (
   }
 
   if (prop === 'output' || prop === 'input' || prop === 'error') {
-    // Check if data has expired
-    if (item && 'expiredAt' in item && item.expiredAt != null) {
+    if (
+      item &&
+      'expiredAt' in item &&
+      item.expiredAt != null &&
+      new Date(item.expiredAt as string | number | Date) < new Date()
+    ) {
       return EXPIRED_DATA_MESSAGE;
     }
     return inlineFormatIO(value);
@@ -490,10 +494,15 @@ const showInspectInfoBox = (resource: string) => {
 const EXPIRED_DATA_MESSAGE = chalk.gray('<data expired>');
 
 /**
- * Checks if a run has expired data storage (run-level expiredAt field)
+ * Checks if a run has expired data storage (run-level expiredAt field).
+ * Only returns true when `expiredAt` is in the past.
  */
-const hasExpiredData = (run: WorkflowRun): boolean => {
-  return 'expiredAt' in run && run.expiredAt != null;
+export const hasExpiredData = (run: WorkflowRun): boolean => {
+  return (
+    'expiredAt' in run &&
+    run.expiredAt != null &&
+    new Date(run.expiredAt) < new Date()
+  );
 };
 
 /**