@obsunified/collector 2.2.3 → 2.3.0

package/src/plugins/connected-routes/manifest.test.ts

@@ -1,40 +1,25 @@
+import { KNOWN_TABS, parseDashboardRouteHash } from "@obsunified/types";
 import { describe, expect, it } from "vitest";
 import {
 	hrefForEntity,
+	linksFromActions,
+	linksFromAi,
 	linksFromArtifacts,
 	linksFromEvalResults,
+	linksFromLogs,
+	linksFromMetricExemplars,
 	linksFromRetrievalEvents,
+	linksFromSpans,
 	linksFromToolCalls,
+	linksFromUsage,
+	linkToAction,
+	linkToAgentRun,
+	linkToSession,
+	linkToTrace,
 	MAX_LINKS_INLINE,
 	resolveFaultLocusTarget,
 } from "./manifest";
 
-// Tabs served by apps/web/src/app/router.ts (KNOWN_TABS). Generated hrefs
-// must start with one of these or the app falls back to Health.
-const ROUTER_TABS = new Set([
-	"health",
-	"investigate",
-	"traces",
-	"service-map",
-	"issues",
-	"logs",
-	"ai",
-	"evidence",
-	"usage",
-	"replay",
-	"alerts",
-	"users",
-	"agent-runs",
-	"actions",
-	"tool-calls",
-	"profiles",
-	"tool-reliability",
-	"evaluations",
-	"changes",
-	"findings",
-	"incidents",
-	"remediations",
-]);
 const tabOf = (href: string) =>
 	href.replace(/^#\//, "").split(/[/?#]/)[0] ?? "";
 
@@ -80,8 +65,105 @@ describe("connected manifest hrefs target real router tabs", () => {
 			"#/evaluations/e1",
 			"#/actions/a1",
 		]);
-		for (const href of hrefs)
-			expect(ROUTER_TABS.has(tabOf(href ?? ""))).toBe(true);
+		for (const href of hrefs) {
+			expect(KNOWN_TABS.has(tabOf(href ?? ""))).toBe(true);
+			expect(parseDashboardRouteHash(href ?? "").ok).toBe(true);
+		}
+	});
+
+	it("emits canonical routes from all link builders and round-trips through parseDashboardRouteHash", () => {
+		expect(linkToTrace("trace-1").href).toBe("#/traces?trace=trace-1");
+		expect(linkToSession("sess/1").href).toBe("#/replay?session=sess%2F1");
+		expect(linkToAction("act/1").href).toBe("#/actions/act%2F1");
+		expect(linkToAgentRun("run/1").href).toBe("#/agent-runs/run%2F1");
+
+		const inlineSpans = linksFromSpans(
+			[
+				{
+					traceId: "tr-1",
+					spanId: "sp-1",
+					serviceName: "api",
+					spanName: "GET /",
+				} as never,
+			],
+			"Spans",
+		);
+		expect(inlineSpans.links[0]?.href).toBe("#/traces?trace=tr-1&span=sp-1");
+		expect(parseDashboardRouteHash(inlineSpans.links[0]?.href ?? "")).toEqual({
+			ok: true,
+			route: { tab: "traces", traceId: "tr-1", spanId: "sp-1" },
+		});
+
+		const manySpans = linksFromSpans(
+			Array.from({ length: MAX_LINKS_INLINE + 1 }, (_, i) => ({
+				traceId: "tr-many",
+				spanId: `sp-${i}`,
+				serviceName: "api",
+				spanName: "op",
+			})) as never,
+			"Spans",
+		);
+		expect(manySpans.links[0]?.href).toBe("#/traces?q=tr-many");
+
+		const inlineLogs = linksFromLogs(
+			[
+				{
+					logId: "log-1",
+					traceId: "tr-1",
+					severity: "ERROR",
+					message: "boom",
+				} as never,
+			],
+			"Logs",
+		);
+		expect(inlineLogs.links[0]?.href).toBe("#/logs?id=log-1");
+
+		const inlineUsage = linksFromUsage(
+			[
+				{
+					eventId: "ev-1",
+					eventType: "click",
+					eventName: "buy",
+				} as never,
+			],
+			"Usage",
+		);
+		expect(inlineUsage.links[0]?.href).toBe("#/usage?id=ev-1");
+
+		const inlineExemplars = linksFromMetricExemplars(
+			[
+				{
+					traceId: "tr-1",
+					spanId: "sp-2",
+					serviceName: "api",
+					metricName: "latency",
+					value: 42,
+					tsNs: "100",
+				} as never,
+			],
+			"Exemplars",
+		);
+		expect(inlineExemplars.links[0]?.href).toBe(
+			"#/traces?trace=tr-1&span=sp-2",
+		);
+
+		const inlineAi = linksFromAi(
+			[
+				{
+					callId: "call/1",
+					provider: "openai",
+					modelName: "gpt-4o",
+				} as never,
+			],
+			"AI",
+		);
+		expect(inlineAi.links[0]?.href).toBe("#/ai/calls/call%2F1");
+
+		const inlineActions = linksFromActions(
+			[{ id: "act-1", actionKind: "step", name: "plan" } as never],
+			"Actions",
+		);
+		expect(inlineActions.links[0]?.href).toBe("#/actions/act-1");
 	});
 
 	it("hrefForEntity only emits router tabs for every evidence kind", () => {
@@ -104,12 +186,12 @@ describe("connected manifest hrefs target real router tabs", () => {
 			"replay",
 			"docs",
 		]) {
-			expect(ROUTER_TABS.has(tabOf(hrefForEntity(kind, "id-1"))), kind).toBe(
-				true,
-			);
+			const href = hrefForEntity(kind, "id-1");
+			expect(KNOWN_TABS.has(tabOf(href)), kind).toBe(true);
+			expect(parseDashboardRouteHash(href).ok, kind).toBe(true);
 		}
 		expect(hrefForEntity("trace", "abc")).toBe("#/traces?trace=abc");
-		expect(hrefForEntity("alert", "al 1")).toBe("#/alerts?id=al%201");
+		expect(hrefForEntity("alert", "al 1")).toBe("#/alerts?id=al+1");
 		expect(hrefForEntity("ai_call", "c1")).toBe("#/ai/calls/c1");
 	});
 });

package/src/plugins/connected-routes/manifest.ts

@@ -1,7 +1,10 @@
-import type {
-	CausalProofTier,
-	EvidenceEntityKind,
-	RelationshipCertainty,
+import {
+	type CausalProofTier,
+	dashboardRouteToHref,
+	type EvidenceEntityKind,
+	type RelationshipCertainty,
+	routeForEntity,
+	hrefForEntity as sharedHrefForEntity,
 } from "@obsunified/types";
 import type {
 	ActionRef,
@@ -19,6 +22,8 @@ import type {
 import type { CausalConfidence } from "../../lib/identity-index/types";
 import type { SqlDb } from "../../lib/sql-db";
 
+export { routeForEntity };
+
 export type ConnectedEntityKind =
 	| "span"
 	| "profile"
@@ -94,7 +99,7 @@ export const linkToTrace = (
 	label?: string,
 ): ConnectedLink => ({
 	label: label ?? `trace ${traceId.slice(0, 12)}`,
-	href: `#/traces/${traceId}`,
+	href: dashboardRouteToHref({ tab: "traces", traceId }),
 	entityKind: "trace",
 	entityId: traceId,
 });
@@ -108,7 +113,7 @@ export const linkToSession = (sessionId: string): ConnectedLink => ({
 		sessionId.length > 12
 			? `session …${sessionId.slice(-12)}`
 			: `session ${sessionId}`,
-	href: `#/replay?session=${encodeURIComponent(sessionId)}`,
+	href: dashboardRouteToHref({ tab: "replay", sessionId }),
 	sample: sessionId,
 	entityKind: "replay",
 	entityId: sessionId,
@@ -131,7 +136,10 @@ export const linksFromSpans = (
 			links: [
 				{
 					label: `${spans.length} spans`,
-					href: `#/traces?q=${encodeURIComponent(spans[0].traceId)}`,
+					href: dashboardRouteToHref({
+						tab: "traces",
+						query: spans[0].traceId,
+					}),
 					count: spans.length,
 					sample: truncate(
 						`${spans[0].serviceName ?? "?"} · ${spans[0].spanName}`,
@@ -144,7 +152,11 @@ export const linksFromSpans = (
 		label: prefix,
 		links: spans.map((s) => ({
 			label: `${s.serviceName ?? "?"} · ${truncate(s.spanName, 40)}`,
-			href: `#/traces/${s.traceId}#span=${s.spanId}`,
+			href: dashboardRouteToHref({
+				tab: "traces",
+				traceId: s.traceId,
+				spanId: s.spanId,
+			}),
 		})),
 	};
 };
@@ -167,7 +179,10 @@ export const linksFromLogs = (
 			links: [
 				{
 					label: `${logs.length} logs`,
-					href: `#/logs?q=${encodeURIComponent(sample.traceId ?? "")}`,
+					href: dashboardRouteToHref({
+						tab: "logs",
+						...(sample.traceId ? { query: sample.traceId } : {}),
+					}),
 					count: logs.length,
 					sample: truncate(sample.message),
 				},
@@ -178,7 +193,7 @@ export const linksFromLogs = (
 		label: prefix,
 		links: logs.map((l) => ({
 			label: `[${l.severity}] ${truncate(l.message, 60)}`,
-			href: `#/logs?id=${l.logId}`,
+			href: dashboardRouteToHref({ tab: "logs", logId: l.logId }),
 		})),
 	};
 };
@@ -200,7 +215,7 @@ export const linksFromUsage = (
 			links: [
 				{
 					label: `${events.length} usage events`,
-					href: `#/usage`,
+					href: dashboardRouteToHref({ tab: "usage" }),
 					count: events.length,
 					sample: truncate(`${events[0].eventType} · ${events[0].eventName}`),
 				},
@@ -211,7 +226,7 @@ export const linksFromUsage = (
 		label: prefix,
 		links: events.map((e) => ({
 			label: `${e.eventType} · ${truncate(e.eventName, 50)}`,
-			href: `#/usage?id=${e.eventId}`,
+			href: dashboardRouteToHref({ tab: "usage", usageId: e.eventId }),
 		})),
 	};
 };
@@ -235,8 +250,8 @@ export const linksFromMetricExemplars = (
 				{
 					label: `${exemplars.length} metric exemplars`,
 					href: sample.traceId
-						? `#/traces/${sample.traceId}`
-						: `#/traces?q=${encodeURIComponent(sample.metricName)}`,
+						? dashboardRouteToHref({ tab: "traces", traceId: sample.traceId })
+						: dashboardRouteToHref({ tab: "traces", query: sample.metricName }),
 					count: exemplars.length,
 					sample: truncate(
 						`${sample.serviceName ?? "?"} · ${sample.metricName}=${sample.value}`,
@@ -250,10 +265,12 @@ export const linksFromMetricExemplars = (
 		links: exemplars.map((e) => ({
 			label: `${e.serviceName ?? "?"} · ${truncate(e.metricName, 40)}=${e.value}`,
 			href: e.traceId
-				? e.spanId
-					? `#/traces/${e.traceId}#span=${e.spanId}`
-					: `#/traces/${e.traceId}`
-				: `#/traces?q=${encodeURIComponent(e.metricName)}`,
+				? dashboardRouteToHref({
+						tab: "traces",
+						traceId: e.traceId,
+						...(e.spanId ? { spanId: e.spanId } : {}),
+					})
+				: dashboardRouteToHref({ tab: "traces", query: e.metricName }),
 			sample: e.tsNs,
 		})),
 	};
@@ -313,7 +330,11 @@ export const profileLinksForTrace = async (
 			label: `${icon} ${type === "offcpu" ? "Off-CPU profiles" : `${type[0].toUpperCase()}${type.slice(1)} profiles`}`,
 			links: list.map((r) => ({
 				label: `${r.service_name ?? "?"} · ${r.duration_ms}ms`,
-				href: `#/profiles/${r.id}?trace_id=${encodeURIComponent(traceId)}`,
+				href: dashboardRouteToHref({
+					tab: "profiles",
+					profileId: r.id,
+					traceId,
+				}),
 			})),
 		});
 	}
@@ -337,7 +358,7 @@ export const linksFromAi = (
 			links: [
 				{
 					label: `${calls.length} AI calls`,
-					href: `#/ai`,
+					href: dashboardRouteToHref({ tab: "ai" }),
 					count: calls.length,
 					sample: truncate(`${calls[0].provider} · ${calls[0].modelName}`),
 				},
@@ -348,7 +369,7 @@ export const linksFromAi = (
 		label: prefix,
 		links: calls.map((c) => ({
 			label: `${c.provider} · ${c.modelName}`,
-			href: `#/ai/calls/${encodeURIComponent(c.callId)}`,
+			href: dashboardRouteToHref({ tab: "ai", aiCallId: c.callId }),
 			entityKind: "ai_call",
 			entityId: c.callId,
 		})),
@@ -360,7 +381,7 @@ export const linkToAction = (
 	label?: string,
 ): ConnectedLink => ({
 	label: label ?? `action ${actionId.slice(0, 12)}`,
-	href: `#/actions/${actionId}`,
+	href: dashboardRouteToHref({ tab: "actions", actionId }),
 	entityKind: "action",
 	entityId: actionId,
 });
@@ -370,7 +391,7 @@ export const linkToAgentRun = (
 	label?: string,
 ): ConnectedLink => ({
 	label: label ?? `agent run ${runId.slice(0, 12)}`,
-	href: `#/agent-runs/${runId}`,
+	href: dashboardRouteToHref({ tab: "agent-runs", agentRunId: runId }),
 	entityKind: "agent_run",
 	entityId: runId,
 });
@@ -413,7 +434,7 @@ export const linksFromActions = (
 			links: [
 				{
 					label: `${actions.length} actions`,
-					href: `#/actions`,
+					href: dashboardRouteToHref({ tab: "actions" }),
 					count: actions.length,
 					sample: truncate(
 						`${actions[0].actionKind} · ${actions[0].name ?? "unnamed"}`,
@@ -429,7 +450,7 @@ export const linksFromActions = (
 		label: prefix,
 		links: actions.map((a) => ({
 			label: `[${a.actionKind}] ${truncate(a.name ?? "unnamed", 40)}`,
-			href: `#/actions/${a.id}`,
+			href: dashboardRouteToHref({ tab: "actions", actionId: a.id }),
 			entityKind: "action",
 			entityId: a.id,
 			...actionContextMetadata(a, metadata),
@@ -457,7 +478,7 @@ export const linksFromToolCalls = (
 			links: [
 				{
 					label: `${calls.length} tool calls`,
-					href: `#/tool-calls`,
+					href: dashboardRouteToHref({ tab: "tool-calls" }),
 					count: calls.length,
 					sample: truncate(calls[0].toolName),
 					entityKind: "tool_call",
@@ -471,7 +492,7 @@ export const linksFromToolCalls = (
 		label: prefix,
 		links: calls.map((t) => ({
 			label: `tool: ${t.toolName}`,
-			href: `#/tool-calls/${t.id}`,
+			href: dashboardRouteToHref({ tab: "tool-calls", toolCallId: t.id }),
 			entityKind: "tool_call",
 			entityId: t.id,
 			...metadataForAction?.(t.actionId),
@@ -495,7 +516,7 @@ export const linksFromRetrievalEvents = (
 		links: events.map((r) => ({
 			label: `retrieve: ${r.retrieverName} (${r.totalResults} docs)`,
 			// No dedicated retrieval tab: the owning action's detail page lists it.
-			href: `#/actions/${encodeURIComponent(r.actionId)}`,
+			href: dashboardRouteToHref({ tab: "actions", actionId: r.actionId }),
 		})),
 	};
 };
@@ -520,7 +541,7 @@ export const linksFromEvalResults = (
 			label: `eval: ${e.evaluatorName} (${e.passed ? "passed" : "failed"})`,
 			// The Evaluations tab resolves this id to an eval case, falling back
 			// to the case saved from the same action.
-			href: `#/evaluations/${encodeURIComponent(e.id)}`,
+			href: dashboardRouteToHref({ tab: "evaluations", evaluationId: e.id }),
 			entityKind: "eval",
 			entityId: e.id,
 			...metadataForAction?.(e.actionId),
@@ -544,59 +565,21 @@ export const linksFromArtifacts = (
 		links: arts.map((a) => ({
 			label: `artifact: ${a.artifactName} (${a.artifactType})`,
 			// No dedicated artifact tab: the owning action's detail page lists it.
-			href: `#/actions/${encodeURIComponent(a.actionId)}`,
+			href: dashboardRouteToHref({ tab: "actions", actionId: a.actionId }),
 		})),
 	};
 };
 
 /**
- * Dashboard destination for an evidence entity, using only routes the web
- * router (`apps/web/src/app/router.ts`) actually serves. Unknown kinds fall
- * back to the evidence explorer so a link never lands on the Health tab.
+ * Dashboard destination for an evidence entity, delegating to the shared
+ * `@obsunified/types` route contract. Unknown kinds fall back to the
+ * evidence explorer so a link never lands on the Health tab.
  */
-export const hrefForEntity = (kind: string, id: string): string => {
-	const enc = encodeURIComponent;
-	switch (kind) {
-		case "trace":
-			return `#/traces?trace=${enc(id)}`;
-		case "span": {
-			const [traceId, spanId] = id.split(":");
-			return spanId && traceId
-				? `#/traces?trace=${enc(traceId)}&span=${enc(spanId)}`
-				: `#/evidence?kind=span&id=${enc(id)}`;
-		}
-		case "alert":
-			return `#/alerts?id=${enc(id)}`;
-		case "analysis":
-			return "#/health";
-		case "service":
-			return `#/service-map?service=${enc(id)}`;
-		case "agent_run":
-			return `#/agent-runs/${enc(id)}`;
-		case "action":
-			return `#/actions/${enc(id)}`;
-		case "tool_call":
-			return `#/tool-calls/${enc(id)}`;
-		case "ai_call":
-			return `#/ai/calls/${enc(id)}`;
-		case "profile":
-			return `#/profiles/${enc(id)}`;
-		case "change":
-			return `#/changes/${enc(id)}`;
-		case "finding":
-			return `#/findings/${enc(id)}`;
-		case "incident":
-			return `#/incidents/${enc(id)}`;
-		case "eval":
-			return `#/evaluations/${enc(id)}`;
-		case "log":
-			return `#/logs?id=${enc(id)}`;
-		case "replay":
-			return `#/replay?session=${enc(id)}`;
-		default:
-			return `#/evidence?kind=${enc(kind)}&id=${enc(id)}`;
-	}
-};
+export const hrefForEntity = (
+	kind: string,
+	id: string,
+	options?: { traceId?: string; spanId?: string },
+): string => sharedHrefForEntity(kind, id, options);
 
 export interface FaultLocusTarget {
 	entityKind: "service" | "span";

package/src/plugins/connected-routes.test.ts

@@ -335,7 +335,7 @@ describe("ConnectedRail manifest — span profile section (RFC 0009 #5)", () =>
 		expect(exemplarSection).toBeDefined();
 		expect(exemplarSection?.links[0].label).toContain("http.server.duration");
 		expect(exemplarSection?.links[0].href).toBe(
-			"#/traces/trace-metric#span=span-hot",
+			"#/traces?trace=trace-metric&span=span-hot",
 		);
 	});
 });
@@ -673,7 +673,7 @@ describe("ConnectedRail manifest — raw signal action back-links", () => {
 				?.links[0],
 		).toMatchObject({
 			label: "[agent.step] Fallback normalized action",
-			href: "#/actions/fallback:trace-fallback:agent.step:1",
+			href: "#/actions/fallback%3Atrace-fallback%3Aagent.step%3A1",
 			causalConfidence: ActionConfidence.Fallback,
 			source: "trace_id",
 			confidence: 0.55,
@@ -831,12 +831,12 @@ describe("ConnectedRail manifest — profile source entity", () => {
 		const traceSection = m.across.find((s) => s.label === "Sampled traces");
 		expect(traceSection?.links[0]).toMatchObject({
 			label: "trace trace-prof",
-			href: "#/traces/trace-prof",
+			href: "#/traces?trace=trace-prof",
 		});
 
 		const spanSection = m.across.find((s) => s.label === "Sampled spans");
 		expect(spanSection?.links[0].href).toBe(
-			"#/traces/trace-prof#span=span-hot",
+			"#/traces?trace=trace-prof&span=span-hot",
 		);
 
 		const actionSection = m.down.find((s) =>
@@ -1108,7 +1108,9 @@ describe("ConnectedRail scenario acceptance contracts", () => {
 		const sessionSpans = sessionManifest.across.find((s) =>
 			s.label.toLowerCase().includes("span"),
 		);
-		expect(sessionSpans?.links[0].href).toBe("#/traces/trace-a#span=span-hot");
+		expect(sessionSpans?.links[0].href).toBe(
+			"#/traces?trace=trace-a&span=span-hot",
+		);
 
 		const spanManifest = await fetch(
 			"/internal/connected/span/trace-a:span-hot",
@@ -1289,7 +1291,7 @@ describe("ConnectedRail scenario acceptance contracts", () => {
 			s.label.toLowerCase().includes("span"),
 		);
 		expect(sessionSpans?.links[0].href).toBe(
-			"#/traces/trace-ai-heavy#span=span-llm",
+			"#/traces?trace=trace-ai-heavy&span=span-llm",
 		);
 
 		const spanManifest = await fetch(
@@ -1464,7 +1466,7 @@ describe("ConnectedRail manifest — agent action graph entities (RFC 0010)", ()
 		// OTel Trace in related
 		const relatedTrace = m.related.find((s) => s.label === "OTel Trace");
 		expect(relatedTrace).toBeDefined();
-		expect(relatedTrace?.links[0].href).toBe("#/traces/tx-789");
+		expect(relatedTrace?.links[0].href).toBe("#/traces?trace=tx-789");
 	});
 
 	it("action detail raw manifest exposes causal confidence on action refs", async () => {
@@ -1650,7 +1652,7 @@ describe("ConnectedRail manifest — agent action graph entities (RFC 0010)", ()
 		// Up has Agent
 		const agentSec = m.up.find((s) => s.label === "Agent");
 		expect(agentSec).toBeDefined();
-		expect(agentSec?.links[0].href).toBe("#/agents/my-agent");
+		expect(agentSec?.links[0].href).toBe("#/evidence?kind=agent&id=my-agent");
 
 		// Down has Actions (Decision Spine)
 		const decisionSpine = m.down.find(

package/src/plugins/connected-routes.ts

@@ -15,7 +15,7 @@
  * RFC 0006 § Connected rail / Empty-state.
  */
 
-import type { CausalProofTier } from "@obsunified/types";
+import { type CausalProofTier, dashboardRouteToHref } from "@obsunified/types";
 import { ActionConfidence } from "@obsunified/types/constants";
 import type { CollectorPlugin } from "../framework/collector";
 import { getChangeEvent } from "../lib/autosre-change-correlation";
@@ -37,6 +37,7 @@ import type {
 import type { SqlDb } from "../lib/sql-db";
 import { sqlDbFor } from "../lib/sql-db";
 import { getProjectId } from "./_context";
+import { respondWithContract } from "./autosre-routes";
 
 import {
 	type ActionContextLinkMetadata,
@@ -456,7 +457,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 											? [
 													{
 														label: `${traceIds.length} sampled traces`,
-														href: `#/traces?trace=${encodeURIComponent(traceIds[0])}`,
+														href: dashboardRouteToHref({
+															tab: "traces",
+															traceId: traceIds[0],
+														}),
 														count: traceIds.length,
 														sample: traceIds[0],
 													},
@@ -556,7 +560,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 											.slice(0, MAX_LINKS_INLINE)
 											.map((toolCall) => ({
 												label: `tool: ${toolCall.tool_name}`,
-												href: `#/tool-calls/${toolCall.id}`,
+												href: dashboardRouteToHref({
+													tab: "tool-calls",
+													toolCallId: toolCall.id,
+												}),
 												entityKind: "tool_call",
 												entityId: toolCall.id,
 												...traceLevelMetadata("profile tool call"),
@@ -986,7 +993,7 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							links: [
 								{
 									label: `${run.agentName} (v${run.agentVersion})`,
-									href: `#/agents/${run.agentId}`,
+									href: hrefForEntity("agent", run.agentId),
 								},
 							],
 						},
@@ -1070,7 +1077,7 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							links: [
 								{
 									label: `${changeRow.targetNodeId} (${changeRow.environment})`,
-									href: `#/health?node=${encodeURIComponent(changeRow.targetNodeId)}`,
+									...resolveFaultLocusTarget(changeRow.targetNodeId),
 									relationshipCertainty: "exact",
 									reason: "Target node mutated by this change event.",
 								},
@@ -1099,7 +1106,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							label: "Attributed Findings",
 							links: findings.results.map((f) => ({
 								label: f.title,
-								href: `#/findings/${encodeURIComponent(f.id)}`,
+								href: dashboardRouteToHref({
+									tab: "findings",
+									findingId: f.id,
+								}),
 								entityKind: "finding",
 								entityId: f.id,
 								relationshipCertainty: "exact",
@@ -1137,7 +1147,11 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							label: "Failing Spans on Cohort Version",
 							links: spans.results.map((s) => ({
 								label: `${s.service_name} · ${s.span_name}`,
-								href: `#/traces/${s.trace_id}#span=${s.span_id}`,
+								href: dashboardRouteToHref({
+									tab: "traces",
+									traceId: s.trace_id,
+									spanId: s.span_id,
+								}),
 								entityKind: "span",
 								entityId: `${s.trace_id}:${s.span_id}`,
 								relationshipCertainty: "exact",
@@ -1167,7 +1181,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							links: [
 								{
 									label: `incident ${findingRow.incidentId}`,
-									href: `#/incidents/${encodeURIComponent(findingRow.incidentId)}`,
+									href: dashboardRouteToHref({
+										tab: "incidents",
+										incidentId: findingRow.incidentId,
+									}),
 									entityKind: "incident",
 									entityId: findingRow.incidentId,
 									relationshipCertainty: "exact",
@@ -1182,7 +1199,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							links: [
 								{
 									label: `change ${findingRow.contributingChangeId}`,
-									href: `#/changes/${encodeURIComponent(findingRow.contributingChangeId)}`,
+									href: dashboardRouteToHref({
+										tab: "changes",
+										changeId: findingRow.contributingChangeId,
+									}),
 									entityKind: "change",
 									entityId: findingRow.contributingChangeId,
 									relationshipCertainty: "exact",
@@ -1206,7 +1226,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 									? [
 											{
 												label: `contending trace ${findingRow.contendingTraceId}`,
-												href: `#/traces/${encodeURIComponent(findingRow.contendingTraceId)}`,
+												href: dashboardRouteToHref({
+													tab: "traces",
+													traceId: findingRow.contendingTraceId,
+												}),
 												entityKind: "trace" as const,
 												entityId: findingRow.contendingTraceId,
 												relationshipCertainty: "inferred" as const,
@@ -1280,7 +1303,10 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 							label: "Findings",
 							links: incFindings.results.map((f) => ({
 								label: f.title,
-								href: `#/findings/${encodeURIComponent(f.id)}`,
+								href: dashboardRouteToHref({
+									tab: "findings",
+									findingId: f.id,
+								}),
 								entityKind: "finding" as const,
 								entityId: f.id,
 								relationshipCertainty: "exact" as const,
@@ -1350,7 +1376,11 @@ export const connectedRoutesPlugin: CollectorPlugin = {
 				];
 			}
 
-			return c.json(manifest);
+			return respondWithContract(
+				c,
+				"GET /internal/connected/:kind/:id",
+				manifest,
+			);
 		});
 	},
 };