fix: address second round of BugBot feedback

BYK · BYK · commit 051830afd49f · 2026-03-16T10:56:28.000Z
- Restore null fallback for span_id in formatSpanSimple: span.span_id ?? ''
  (defensive against runtime undefined from normalizeTraceSpan edge cases)
- Unexport SPAN_TABLE_COLUMNS — only used internally by formatSpanTable,
  no external consumers
diff --git a/src/lib/formatters/human.ts b/src/lib/formatters/human.ts
@@ -1117,7 +1117,7 @@ function formatSpanSimple(span: TraceSpan, opts: FormatSpanOptions): void {
     line += `  ${plainSafeMuted(`(${prettyMs(durationMs)})`)}`;
   }
 
-  line += `  ${plainSafeMuted(span.span_id)}`;
+  line += `  ${plainSafeMuted(span.span_id ?? "")}`;
 
   lines.push(line);
 
diff --git a/src/lib/formatters/trace.ts b/src/lib/formatters/trace.ts
@@ -416,7 +416,7 @@ export function spanListItemToFlatSpan(item: SpanListItem): FlatSpan {
 }
 
 /** Column definitions for the flat span table */
-export const SPAN_TABLE_COLUMNS: Column<FlatSpan>[] = [
+const SPAN_TABLE_COLUMNS: Column<FlatSpan>[] = [
   {
     header: "Span ID",
     value: (s) => `\`${s.span_id}\``,

Original file line number	Diff line number	Diff line change
`@@ -1117,7 +1117,7 @@ function formatSpanSimple(span: TraceSpan, opts: FormatSpanOptions): void {`
`1117`	`1117`	line += ` ${plainSafeMuted(`(${prettyMs(durationMs)})`)}`;
`1118`	`1118`	`}`
`1119`	`1119`
`1120`		- line += ` ${plainSafeMuted(span.span_id)}`;
	`1120`	+ line += ` ${plainSafeMuted(span.span_id ?? "")}`;
`1121`	`1121`
`1122`	`1122`	`lines.push(line);`
`1123`	`1123`
Original file line number	Diff line number	Diff line change
`@@ -416,7 +416,7 @@ export function spanListItemToFlatSpan(item: SpanListItem): FlatSpan {`
`416`	`416`	`}`
`417`	`417`
`418`	`418`	`/** Column definitions for the flat span table */`
`419`		`-export const SPAN_TABLE_COLUMNS: Column<FlatSpan>[] = [`
	`419`	`+const SPAN_TABLE_COLUMNS: Column<FlatSpan>[] = [`
`420`	`420`	`{`
`421`	`421`	`header: "Span ID",`
`422`	`422`	value: (s) => `\`${s.span_id}\``,