perf(clickhouse): stop selecting span attributes for the run trace API tree

The run trace API (`GET /api/v1/runs/:runId/trace`, unpaged) no longer
reads span attributes from ClickHouse when building the span tree. The
tree never used them, so responses are unchanged while ClickHouse reads
and the data sent to the API shrink, most for large traces.

Mono-RevId: cb52ce09b11e29ad36f09b11caaaf9582e5cbfbf

Author: Mariefay

internal-packages/clickhouse/src/taskEvents.ts

@@ -86,7 +86,6 @@ export const TaskEventDetailedSummaryV1Result = z.object({
   kind: z.string(),
   metadata: z.string(),
   message: z.string(),
-  attributes_text: z.string(),
 });
 
 export type TaskEventDetailedSummaryV1Result = z.output<typeof TaskEventDetailedSummaryV1Result>;
@@ -108,7 +107,6 @@ export function getTraceDetailedSummaryQueryBuilder(
       "kind",
       "metadata",
       { name: "message", expression: "LEFT(message, 256)" },
-      "attributes_text",
     ],
     settings,
   });
@@ -306,7 +304,6 @@ export function getTraceDetailedSummaryQueryBuilderV2(
       "kind",
       "metadata",
       { name: "message", expression: "LEFT(message, 256)" },
-      "attributes_text",
     ],
     settings,
   });

internal-packages/clickhouse/src/taskEventsDetailedSummary.test.ts

@@ -0,0 +1,18 @@
+import { describe, expect, it } from "vitest";
+import { NoopClient } from "./client/noop.js";
+import {
+  getTraceDetailedSummaryQueryBuilder,
+  getTraceDetailedSummaryQueryBuilderV2,
+} from "./taskEvents.js";
+
+describe("trace detailed summary query builders", () => {
+  it.each([
+    ["task_events_v1", getTraceDetailedSummaryQueryBuilder],
+    ["task_events_v2", getTraceDetailedSummaryQueryBuilderV2],
+  ])("%s does not select attributes_text (the tree builder never reads it)", (table, builder) => {
+    const { query } = builder(new NoopClient())().build();
+
+    expect(query).toMatch(new RegExp(`^SELECT .+ FROM trigger_dev\\.${table}$`));
+    expect(query).not.toContain("attributes_text");
+  });
+});