From 116a01575b0017dc605cb801eb1e47b7c67211b8 Mon Sep 17 00:00:00 2001
From: Julien Calixte <julien.calixte@theodo.com>
Date: Wed, 25 Feb 2026 17:08:18 +0100
Subject: [PATCH] feat: warn users when a public note no longer exists

Show an error toast and redirect to 404 when navigating to a deleted
ATProto note. In stacked panels, display an inline error message instead.
---
 src/components/StackedPublicNote.vue | 29 +++++++++++++++++++++-------
 src/views/PublicNoteView.vue         | 25 ++++++++++++++++++------
 2 files changed, 41 insertions(+), 13 deletions(-)

diff --git a/src/components/StackedPublicNote.vue b/src/components/StackedPublicNote.vue
index cef2619..50293a9 100644
--- a/src/components/StackedPublicNote.vue
+++ b/src/components/StackedPublicNote.vue
@@ -1,5 +1,6 @@
 <script lang="ts" setup>
-import { computed, nextTick, watch } from "vue"
+import { computed, nextTick, ref, watch } from "vue"
+import { errorMessage } from "@/utils/notif"
 import { useATProtoLinks } from "@/hooks/useATProtoLinks.hook"
 import { useNoteOverlay } from "@/hooks/useNoteOverlay.hook"
 import { useRouteQueryStackedNotes } from "@/hooks/useRouteQueryStackedNotes.hook"
@@ -34,11 +35,22 @@ const { scrollToFocusedNote } = useRouteQueryStackedNotes()
 const { listenToClick } = useATProtoLinks(className.value, didrkey)
 const { displayNoteOverlay } = useNoteOverlay(className.value, index)
 
-const noteRecord = computedAsync(async () =>
-  url.value
-    ? ((await fetch(url.value).then()).json() as Promise<PublicNoteRecord>)
-    : null,
-)
+const noteNotFound = ref(false)
+const noteRecord = computedAsync(async () => {
+  if (!url.value) return null
+  const response = await fetch(url.value)
+  if (!response.ok) {
+    noteNotFound.value = true
+    return null
+  }
+  return response.json() as Promise<PublicNoteRecord>
+})
+
+watch(noteNotFound, (notFound) => {
+  if (notFound) {
+    errorMessage("This note no longer exists.")
+  }
+})
 const { toHTML } = markdownBuilder()
 const title = computed(() => noteRecord.value?.value.title)
 const content = computed(() =>
@@ -83,7 +95,10 @@ watch(
       </div>
     </a>
     <section class="text-content">
-      <div class="note-content" v-html="content"></div>
+      <div v-if="noteNotFound" class="alert alert-error">
+        This note no longer exists.
+      </div>
+      <div class="note-content" v-else v-html="content"></div>
     </section>
   </div>
 </template>
diff --git a/src/views/PublicNoteView.vue b/src/views/PublicNoteView.vue
index 029b2f6..f3e4500 100644
--- a/src/views/PublicNoteView.vue
+++ b/src/views/PublicNoteView.vue
@@ -11,8 +11,9 @@ import { getUrl } from "@/modules/atproto/getUrl"
 import { downloadFont } from "@/utils/downloadFont"
 import { slugify } from "@/utils/slugify"
 import { computedAsync } from "@vueuse/core"
-import { computed, nextTick, watch } from "vue"
+import { computed, nextTick, ref, watch } from "vue"
 import { useRouter } from "vue-router"
+import { errorMessage } from "@/utils/notif"
 import { useResizeContainer } from "@/hooks/useResizeContainer.hook"
 import ThemeSwap from "@/components/ThemeSwap.vue"
 import { useTitle } from "@vueuse/core"
@@ -28,11 +29,23 @@ const url = computedAsync(
   null,
 )
 
-const noteRecord = computedAsync(async () =>
-  url.value
-    ? ((await fetch(url.value).then()).json() as Promise<PublicNoteRecord>)
-    : null,
-)
+const noteNotFound = ref(false)
+const noteRecord = computedAsync(async () => {
+  if (!url.value) return null
+  const response = await fetch(url.value)
+  if (!response.ok) {
+    noteNotFound.value = true
+    return null
+  }
+  return response.json() as Promise<PublicNoteRecord>
+})
+
+watch(noteNotFound, (notFound) => {
+  if (notFound) {
+    errorMessage("This note no longer exists.")
+    router.replace({ name: "SpaceCowboy" })
+  }
+})
 
 watch(noteRecord, () => {
   if (