Revert "feat: add --verbose flag to workflow status command"

This reverts commit c68608b.

Author: coleam00

packages/cli/src/commands/workflow.test.ts

@@ -934,6 +934,8 @@ describe('workflowStatusCommand', () => {
       },
     ]);
 
+    const startTime = new Date(Date.now() - 25 * 1000).toISOString();
+    const endTime = new Date(Date.now() - 15 * 1000).toISOString();
     (workflowEventsDb.listWorkflowEvents as ReturnType<typeof mock>).mockResolvedValueOnce([
       {
         id: 'e1',
@@ -942,16 +944,16 @@ describe('workflowStatusCommand', () => {
         step_name: 'plan',
         step_index: null,
         data: {},
-        created_at: new Date().toISOString(),
+        created_at: startTime,
       },
       {
         id: 'e2',
         workflow_run_id: 'run-verbose',
         event_type: 'node_completed',
         step_name: 'plan',
         step_index: null,
-        data: { node_output: 'Plan output here', duration_ms: 5000 },
-        created_at: new Date().toISOString(),
+        data: { node_output: 'Plan output here' },
+        created_at: endTime,
       },
     ]);
 
@@ -961,7 +963,6 @@ describe('workflowStatusCommand', () => {
     expect(calls.some(c => c.includes('Nodes:'))).toBe(true);
     expect(calls.some(c => c.includes('✓') && c.includes('plan'))).toBe(true);
     expect(calls.some(c => c.includes('Plan output here'))).toBe(true);
-    expect(calls.some(c => c.includes('Total:'))).toBe(true);
   });
 
   it('should show error message for failed node in verbose mode', async () => {
@@ -978,6 +979,8 @@ describe('workflowStatusCommand', () => {
       },
     ]);
 
+    const startTime = new Date(Date.now() - 20 * 1000).toISOString();
+    const endTime = new Date(Date.now() - 10 * 1000).toISOString();
     (workflowEventsDb.listWorkflowEvents as ReturnType<typeof mock>).mockResolvedValueOnce([
       {
         id: 'e3',
@@ -986,16 +989,16 @@ describe('workflowStatusCommand', () => {
         step_name: 'implement',
         step_index: null,
         data: {},
-        created_at: new Date().toISOString(),
+        created_at: startTime,
       },
       {
         id: 'e4',
         workflow_run_id: 'run-failed',
         event_type: 'node_failed',
         step_name: 'implement',
         step_index: null,
-        data: { error: 'Compilation failed', duration_ms: 3000 },
-        created_at: new Date().toISOString(),
+        data: { error: 'Compilation failed' },
+        created_at: endTime,
       },
     ]);
 
@@ -1006,51 +1009,6 @@ describe('workflowStatusCommand', () => {
     expect(calls.some(c => c.includes('Compilation failed'))).toBe(true);
   });
 
-  it('should truncate output at word boundary in verbose mode', async () => {
-    const workflowDb = await import('@archon/core/db/workflows');
-    const workflowEventsDb = await import('@archon/core/db/workflow-events');
-
-    (workflowDb.listWorkflowRuns as ReturnType<typeof mock>).mockResolvedValueOnce([
-      {
-        id: 'run-truncate',
-        workflow_name: 'implement',
-        working_path: '/path/to/worktree',
-        status: 'running',
-        started_at: new Date(Date.now() - 5 * 1000),
-      },
-    ]);
-
-    // Build a 210-char string with spaces every 5 chars: "aaaaa bbbbb ccccc ..."
-    const words = Array.from({ length: 42 }, (_, i) =>
-      String.fromCharCode(97 + (i % 26)).repeat(5)
-    );
-    const longOutput = words.join(' '); // 5*42 + 41 spaces = 251 chars total
-
-    (workflowEventsDb.listWorkflowEvents as ReturnType<typeof mock>).mockResolvedValueOnce([
-      {
-        id: 'e-trunc',
-        workflow_run_id: 'run-truncate',
-        event_type: 'node_completed',
-        step_name: 'plan',
-        step_index: null,
-        data: { node_output: longOutput, duration_ms: 1000 },
-        created_at: new Date().toISOString(),
-      },
-    ]);
-
-    await workflowStatusCommand(false, true);
-
-    const calls = consoleSpy.mock.calls.map((c: unknown[]) => String(c[0]));
-    const outputLine = calls.find(c => c.includes('Output:')) ?? '';
-    const preview = outputLine.replace(/^\s*Output:\s*/, '');
-    // Must end with '...' (was truncated)
-    expect(preview.endsWith('...')).toBe(true);
-    // The cut must be at a word boundary: char in original at cut position must be a space
-    const previewText = preview.slice(0, -3); // remove trailing '...'
-    const cutPos = previewText.length;
-    expect(longOutput[cutPos]).toBe(' '); // original char right after cut is a space
-  });
-
   it('should not show nodes section when no events in verbose mode', async () => {
     const workflowDb = await import('@archon/core/db/workflows');
     const workflowEventsDb = await import('@archon/core/db/workflow-events');

packages/cli/src/commands/workflow.ts

@@ -588,18 +588,6 @@ function formatDuration(ms: number): string {
   return `${String(mins)}m${String(remSecs)}s`;
 }
 
-/**
- * Truncate text at the last word boundary at or before maxLen, appending '...'.
- * Falls back to a hard cut at maxLen if no space is found in range.
- */
-function truncateAtWordBoundary(text: string, maxLen: number): string {
-  if (text.length <= maxLen) return text;
-  const slice = text.slice(0, maxLen);
-  const lastSpace = slice.lastIndexOf(' ');
-  const cut = lastSpace > 0 ? lastSpace : maxLen;
-  return text.slice(0, cut) + '...';
-}
-
 interface NodeSummary {
   nodeId: string;
   state: 'running' | 'completed' | 'failed' | 'skipped';
@@ -613,6 +601,7 @@ interface NodeSummary {
  * Processes node_started / node_completed / node_failed / node_skipped* events.
  */
 function buildNodeSummaries(events: WorkflowEventRow[]): NodeSummary[] {
+  const startTimes = new Map<string, number>();
   const summaries = new Map<string, NodeSummary>();
 
   for (const event of events) {
@@ -621,29 +610,35 @@ function buildNodeSummaries(events: WorkflowEventRow[]): NodeSummary[] {
 
     switch (event.event_type) {
       case 'node_started': {
+        startTimes.set(nodeId, new Date(event.created_at).getTime());
         if (!summaries.has(nodeId)) {
           summaries.set(nodeId, { nodeId, state: 'running' });
         }
         break;
       }
       case 'node_completed': {
+        const started = startTimes.get(nodeId);
+        const endTime = new Date(event.created_at).getTime();
         const rawOutput = event.data.node_output;
         const output = typeof rawOutput === 'string' ? rawOutput : undefined;
-        const rawDuration = event.data.duration_ms;
         summaries.set(nodeId, {
           nodeId,
           state: 'completed',
-          durationMs: typeof rawDuration === 'number' ? rawDuration : undefined,
-          outputPreview: output !== undefined ? truncateAtWordBoundary(output, 200) : undefined,
+          durationMs: started !== undefined ? endTime - started : undefined,
+          outputPreview:
+            output !== undefined
+              ? output.slice(0, 200) + (output.length > 200 ? '...' : '')
+              : undefined,
         });
         break;
       }
       case 'node_failed': {
-        const rawDuration = event.data.duration_ms;
+        const started = startTimes.get(nodeId);
+        const endTime = new Date(event.created_at).getTime();
         summaries.set(nodeId, {
           nodeId,
           state: 'failed',
-          durationMs: typeof rawDuration === 'number' ? rawDuration : undefined,
+          durationMs: started !== undefined ? endTime - started : undefined,
           error: typeof event.data.error === 'string' ? event.data.error : 'Unknown error',
         });
         break;
@@ -754,10 +749,6 @@ export async function workflowStatusCommand(json?: boolean, verbose?: boolean):
             console.log(`        Error:  ${node.error}`);
           }
         }
-        const totalMs = nodes.reduce((sum, n) => sum + (n.durationMs ?? 0), 0);
-        if (totalMs > 0) {
-          console.log(`  Total:  ${formatDuration(totalMs)}`);
-        }
       }
     }