@mastra/observability 1.17.2 → 1.17.3-alpha.0

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @mastra/observability
 
+## 1.17.3-alpha.0
+
+### Patch Changes
+
+- Fixed the "Span not found." error when opening the span linked from a log or metric in Studio. ([#22286](https://github.com/mastra-ai/mastra/pull/22286))
+
+  Logs and metrics emitted inside an internal or excluded span were stamped with that span's own id. Those spans are dropped before export, so the id referenced a span no exporter ever received, and clicking through from a log detail panel returned a 404.
+
+  Logs and metrics now resolve to the nearest ancestor span that actually reaches exporters, and omit `spanId` entirely when no such ancestor exists. This applies to newly emitted signals; already-stored logs and metrics keep the ids they were written with.
+
+- Updated dependencies [[`7176362`](https://github.com/mastra-ai/mastra/commit/717636281a3339911a05ea2cc8ae38afe4fd2cef), [`e3b796d`](https://github.com/mastra-ai/mastra/commit/e3b796d29a63f0d5c97dd815aadec40687346d70), [`49ccd14`](https://github.com/mastra-ai/mastra/commit/49ccd142268a61fb55ea75bc76287643a21f3677), [`3855b38`](https://github.com/mastra-ai/mastra/commit/3855b38c4c25af32ab8e298e148becc963abe92c)]:
+  - @mastra/core@1.63.0-alpha.0
+
 ## 1.17.2
 
 ### Patch Changes

package/dist/index.cjs

@@ -2879,6 +2879,23 @@ var BaseSpan = class BaseSpan {
 		if (parent.isExcluded || !includeInternalSpans && parent.isInternal) return parent.getParentSpan(includeInternalSpans);
 		return this.parent;
 	}
+	/**
+	* Get the spanId that observability signals (logs, metrics, scores) should
+	* reference for this span. If this span itself reaches exporters, that is
+	* its own id. If it is excluded from export (internal span, excludeSpanTypes,
+	* or NoOpSpan), resolves to the nearest exportable ancestor instead.
+	* Returns undefined when no exportable ancestor exists.
+	*
+	* Only covers exclusions known when the span is created (`isExcluded`). A
+	* `spanFilter` or a span output processor can still drop a span at export
+	* time — both run on the exported span data after the span ends, so their
+	* verdict isn't knowable while signals are being stamped. Signals emitted
+	* inside a span dropped that way still reference its own id.
+	*/
+	getExportedSpanId() {
+		if (!this.isExcluded) return this.id;
+		return this.getParentSpanId(this.observabilityInstance.getConfig().includeInternalSpans);
+	}
 	/** Get the closest parent spanId that will reach exporters (unless includeInternalSpans) */
 	getParentSpanId(includeInternalSpans) {
 		if (!this.parent) return this.parentSpanId;
@@ -2920,7 +2937,7 @@ var BaseSpan = class BaseSpan {
 		const rootTags = rootSpan.tags?.length ? [...rootSpan.tags] : void 0;
 		this.correlationContext = {
 			traceId: this.traceId,
-			spanId: this.id,
+			spanId: this.getExportedSpanId(),
 			tags: rootTags,
 			entityType: this.entityType,
 			entityId: this.entityId,
@@ -3439,7 +3456,7 @@ var BaseObservabilityInstance = class extends _mastra_core_base.MastraBase {
 		const metadata = span?.metadata ? structuredClone(span.metadata) : void 0;
 		return new LoggerContextImpl({
 			traceId: span?.traceId,
-			spanId: span?.id,
+			spanId: (0, _mastra_core_observability.resolveExportedSpanId)(span),
 			correlationContext,
 			metadata,
 			observabilityBus: this.observabilityBus,
@@ -3456,7 +3473,7 @@ var BaseObservabilityInstance = class extends _mastra_core_base.MastraBase {
 		const metadata = span?.metadata ? structuredClone(span.metadata) : void 0;
 		return new MetricsContextImpl({
 			traceId: span?.traceId,
-			spanId: span?.id,
+			spanId: (0, _mastra_core_observability.resolveExportedSpanId)(span),
 			correlationContext,
 			metadata,
 			cardinalityFilter: this.cardinalityFilter,