fix: null-safe span_id comparison in findSpanById

BYK · BYK · commit 753acc4f5a08 · 2026-03-16T11:04:20.000Z
Use optional chaining (span.span_id?.toLowerCase()) to avoid crash
when a span from the API has neither span_id nor event_id.
diff --git a/src/lib/formatters/trace.ts b/src/lib/formatters/trace.ts
@@ -347,7 +347,7 @@ export function findSpanById(
     depth: number,
     ancestors: TraceSpan[]
   ): FoundSpan | null {
-    if (span.span_id.toLowerCase() === spanId) {
+    if (span.span_id?.toLowerCase() === spanId) {
       return { span, depth, ancestors };
     }
     for (const child of span.children ?? []) {

Original file line number	Diff line number	Diff line change
`@@ -347,7 +347,7 @@ export function findSpanById(`
`347`	`347`	`depth: number,`
`348`	`348`	`ancestors: TraceSpan[]`
`349`	`349`	`): FoundSpan \| null {`
`350`		`- if (span.span_id.toLowerCase() === spanId) {`
	`350`	`+ if (span.span_id?.toLowerCase() === spanId) {`
`351`	`351`	`return { span, depth, ancestors };`
`352`	`352`	`}`
`353`	`353`	`for (const child of span.children ?? []) {`