just treat duration as nanoseconds, don't worry about the postgresql store as its deprecated

Author: ericallam

apps/webapp/app/routes/api.v1.runs.$runId.spans.$spanId.ts

@@ -54,10 +54,8 @@ export const loader = createLoaderApiRoute(
       return json({ error: "Span not found" }, { status: 404 });
     }
 
-    // Postgres eventRepository returns duration in ms, ClickHouse returns nanoseconds
-    const isClickhouse =
-      run.taskEventStore === "clickhouse" || run.taskEventStore === "clickhouse_v2";
-    const durationMs = isClickhouse ? span.duration / 1_000_000 : span.duration;
+    // Duration is nanoseconds from ClickHouse (Postgres store is deprecated)
+    const durationMs = span.duration / 1_000_000;
 
     const aiData =
       span.properties && typeof span.properties === "object"

packages/cli-v3/src/mcp/formatters.ts

@@ -236,11 +236,8 @@ function formatSpan(
 
   // Format span header
   const statusIndicator = getStatusIndicator(span.data);
-  // Trace durations from ClickHouse are nanoseconds, from Postgres are milliseconds.
-  // Normalize: values over 1e6 are nanoseconds (1e6 ns = 1ms; 1e6 ms = 16min).
-  const durationMs =
-    span.data.duration > 1_000_000 ? span.data.duration / 1_000_000 : span.data.duration;
-  const duration = formatDuration(durationMs);
+  // Trace durations are nanoseconds from ClickHouse
+  const duration = formatDuration(span.data.duration / 1_000_000);
   const startTime = formatDateTime(span.data.startTime);
 
   lines.push(`${indent}${prefix} [${span.id}] ${span.data.message} ${statusIndicator}`);