From 9b59b7b50f5e439a53dbf29583c7ada702117af8 Mon Sep 17 00:00:00 2001
From: Anirudh Oppiliappan <anirudh@tangled.org>
Date: Thu, 20 Aug 2026 22:20:35 +0300
Subject: [PATCH] web/lib/components/notifications: fix notification links to
 use the full at-uri

Signed-off-by: Anirudh Oppiliappan <anirudh@tangled.org>
---
 web/src/lib/components/notifications/NotificationItem.svelte | 4 ++--
 web/src/lib/components/notifications/notification.ts         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/web/src/lib/components/notifications/NotificationItem.svelte b/web/src/lib/components/notifications/NotificationItem.svelte
index 8523605dd..f8d5e9612 100644
--- a/web/src/lib/components/notifications/NotificationItem.svelte
+++ b/web/src/lib/components/notifications/NotificationItem.svelte
@@ -34,7 +34,6 @@
 </script>
 
 <script lang="ts">
-	import { resolve } from "$app/paths";
 	import Mail from "$icon/mail";
 	import MailCheck from "$icon/mail-check";
 	import ColorBall from "$lib/components/ui/ColorBall.svelte";
@@ -58,7 +57,8 @@
 
 	const style = $derived(notificationItem({ read: notification.read }));
 	const Icon = $derived(notificationIcon(notification.type));
-	const href = $derived(resolve(notificationHref(notification) as "/"));
+	// resolve() would mangle the raw at:// uri here (collapses "//"), so it's used as-is
+	const href = $derived(notificationHref(notification));
 	const target = $derived(notificationTarget(notification));
 
 	// the row is one big link, so the read toggle inside it must not navigate
diff --git a/web/src/lib/components/notifications/notification.ts b/web/src/lib/components/notifications/notification.ts
index 8b2b99210..811eff10c 100644
--- a/web/src/lib/components/notifications/notification.ts
+++ b/web/src/lib/components/notifications/notification.ts
@@ -127,7 +127,7 @@ export const notificationHref = (notification: NotificationSummary): string => {
 	const base = `/${repo.ownerHandle}/${repo.name}`;
 	const subject = notification.subject;
 	if (!subject) return base;
-	return `${base}/${subject.kind === "issue" ? "issues" : "pulls"}/${subject.number}`;
+	return `${base}/${subject.kind === "issue" ? "issues" : "pulls"}/${subject.uri}`;
 };
 
 // today, the six days before it, then everything older, same buckets the go page
-- 
2.51.2