From d1c1952a020e49b03911de3f924e2d952b55cc96 Mon Sep 17 00:00:00 2001
From: Jared Pereira <jared@awarm.space>
Date: Wed, 25 Mar 2026 01:56:37 +0000
Subject: [PATCH] fix bsky post ref and tag in update flow

---
 actions/publishToPublication.ts | 41 ++++++++++++++++++++++++++++++-----------
 app/[leaflet_id]/actions/PublishButton.tsx | 7 +++++--
 2 file(s) changed, 35 insertion(s)(+), 13 deletion(s)(-)

diff --git a/actions/publishToPublication.ts b/actions/publishToPublication.ts
--- a/actions/publishToPublication.ts
+++ b/actions/publishToPublication.ts
@@ -182,7 +182,7 @@
     credentialSession.did!,
   );
 
-  let existingRecord: Partial<PubLeafletDocument.Record> = {};
+  let existingRecord: Partial<SiteStandardDocument.Record> = {};
   const normalizedDoc = normalizeDocumentRecord(draft?.documents?.data);
   if (normalizedDoc) {
     // When reading existing data, use normalized format to extract fields
@@ -194,6 +194,7 @@
       tags: normalizedDoc.tags,
       coverImage: normalizedDoc.coverImage,
       theme: normalizedDoc.theme,
+      bskyPostRef: normalizedDoc.bskyPostRef,
     };
   }
 
@@ -249,6 +250,14 @@
   // Determine the rkey early since we need it for the path field
   const rkey = existingDocUri ? new AtUri(existingDocUri).rkey : TID.nextStr();
 
+  // Resolve fields: use new values if provided, otherwise preserve existing
+  const resolvedDescription =
+    description !== undefined ? description : existingRecord.description;
+  const resolvedTags = tags !== undefined ? tags : existingRecord.tags;
+  const resolvedCoverImage = coverImageBlob ?? existingRecord.coverImage;
+  const resolvedPublishedAt =
+    publishedAt || existingRecord.publishedAt || new Date().toISOString();
+
   // Create record based on the document type
   let record: PubLeafletDocument.Record | SiteStandardDocument.Record;
 
@@ -263,11 +272,15 @@
       title: title || "",
       site: siteUri,
       path: "/" + rkey,
-      publishedAt:
-        publishedAt || existingRecord.publishedAt || new Date().toISOString(),
-      ...(description && { description }),
-      ...(tags !== undefined && { tags }),
-      ...(coverImageBlob && { coverImage: coverImageBlob }),
+      publishedAt: resolvedPublishedAt,
+      ...(resolvedDescription !== undefined && {
+        description: resolvedDescription,
+      }),
+      ...(resolvedTags !== undefined && { tags: resolvedTags }),
+      ...(resolvedCoverImage && { coverImage: resolvedCoverImage }),
+      ...(existingRecord.bskyPostRef && {
+        bskyPostRef: existingRecord.bskyPostRef,
+      }),
       // Include theme for standalone documents (not for publication documents)
       ...(!publication_uri && theme && { theme }),
       ...(preferences && {
@@ -295,12 +308,14 @@
         },
       }),
       title: title || "",
-      description: description || "",
-      ...(tags !== undefined && { tags }),
-      ...(coverImageBlob && { coverImage: coverImageBlob }),
+      description: resolvedDescription || "",
+      ...(resolvedTags !== undefined && { tags: resolvedTags }),
+      ...(resolvedCoverImage && { coverImage: resolvedCoverImage }),
+      ...(existingRecord.bskyPostRef && {
+        postRef: existingRecord.bskyPostRef,
+      }),
       pages: pagesArray,
-      publishedAt:
-        publishedAt || existingRecord.publishedAt || new Date().toISOString(),
+      publishedAt: resolvedPublishedAt,
     } satisfies PubLeafletDocument.Record;
   }
 
@@ -332,6 +347,8 @@
         publication: publication_uri,
         title: title,
         description: description,
+        tags: resolvedTags ?? [],
+        cover_image: cover_image ?? null,
       }),
     ]);
   } else {
@@ -341,6 +358,8 @@
       document: result.uri,
       title: title || "",
       description: description || "",
+      tags: resolvedTags ?? [],
+      cover_image: cover_image ?? null,
     });
 
     // Heuristic: Remove title entities if this is the first time publishing standalone
diff --git a/app/[leaflet_id]/actions/PublishButton.tsx b/app/[leaflet_id]/actions/PublishButton.tsx
--- a/app/[leaflet_id]/actions/PublishButton.tsx
+++ b/app/[leaflet_id]/actions/PublishButton.tsx
@@ -65,7 +65,7 @@
 
 const UpdateButton = () => {
   let [isLoading, setIsLoading] = useState(false);
-  let { data: pub, mutate } = useLeafletPublicationData();
+  let { data: pub, mutate, normalizedDocument } = useLeafletPublicationData();
   let { permission_token, rootEntity, rep } = useReplicache();
   let { identity } = useIdentityData();
   let toaster = useToaster();
@@ -88,8 +88,11 @@
       : pub?.description || "";
 
   // Get tags from Replicache state (same as draft editor)
+  // Fall back to normalized document tags if Replicache hasn't pulled yet
   let tags = useSubscribe(rep, (tx) => tx.get<string[]>("publication_tags"));
-  const currentTags = Array.isArray(tags) ? tags : [];
+  const currentTags = Array.isArray(tags)
+    ? tags
+    : normalizedDocument?.tags ?? [];
 
   // Get cover image from Replicache state
   let coverImage = useSubscribe(rep, (tx) =>

--
tangled.sh