nits

Author: maparent

apps/obsidian/src/utils/publishNode.ts

@@ -39,11 +39,6 @@ export const publishNode = async ({
   }
   const contentId = idResponse.data.id;
   const lastModifiedDb = new Date(idResponse.data.last_modified + "Z");
-  console.log(
-    idResponse.data.last_modified,
-    lastModifiedDb.getTime(),
-    file.stat.mtime,
-  );
   if (
     existingPublish.includes(myGroup) &&
     file.stat.mtime <= lastModifiedDb.getTime()
@@ -124,10 +119,11 @@ export const publishNode = async ({
   // do not fail on cleanup
   if (cleanupResult.error) console.error(cleanupResult.error);
 
-  await plugin.app.fileManager.processFrontMatter(
-    file,
-    (fm: Record<string, unknown>) => {
-      fm.publishedToGroups = [...existingPublish, myGroup];
-    },
-  );
+  if (!existingPublish.includes(myGroup))
+    await plugin.app.fileManager.processFrontMatter(
+      file,
+      (fm: Record<string, unknown>) => {
+        fm.publishedToGroups = [...existingPublish, myGroup];
+      },
+    );
 };

packages/database/src/lib/files.ts

@@ -25,7 +25,7 @@ export const addFile = async ({
   if (!exists) {
     // we should use upsert here for sync issues, but we get obscure rls errors.
     const uploadResult = await client.storage.from(ASSETS_BUCKET_NAME).upload(hashvalue, content, {contentType: mimetype});
-    if (uploadResult.error && !uploadResult.error.message.endsWith(" already exists"))
+    if (uploadResult.error && (uploadResult.error as any).statusCode !== "409")
       throw uploadResult.error;
   }
   // not doing an upsert because it does not update on conflict