From 98a180d47314e4b71c22ca2c938945f41805e0a2 Mon Sep 17 00:00:00 2001
From: Allan Chain <36528777+AllanChain@users.noreply.github.com>
Date: Tue, 26 Nov 2024 22:16:30 +0800
Subject: [PATCH] fix: keep extra in journal item & remove arXiv key

Thanks @falbarelli from #24
---
 src/modules/arxiv-merge.ts | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/src/modules/arxiv-merge.ts b/src/modules/arxiv-merge.ts
index 6740462..8a68446 100644
--- a/src/modules/arxiv-merge.ts
+++ b/src/modules/arxiv-merge.ts
@@ -4,7 +4,7 @@ import { catchError } from "./error";
 import { getPref } from "../utils/prefs";
 
 export class arXivMerge {
-  static reservedKeys = ["collections", "dateAdded", "dateModified", "extra"];
+  static reservedKeys = ["collections", "dateAdded", "dateModified"];
 
   @catchError
   static registerRightClickMenuItem() {
@@ -82,6 +82,21 @@ export class arXivMerge {
       // @ts-ignore some fields are not listed in zotero-type
       journalJSON[field] = preprintJSON[field];
     });
+    // `extra` field need more care
+    const preprintExtra = Zotero.Utilities.Internal.extractExtraFields(
+      preprintJSON.extra as string,
+    );
+    journalJSON.extra = Zotero.Utilities.Internal.combineExtraFields(
+      journalJSON.extra as string,
+      preprintExtra.fields,
+    );
+    // We generally want to keep extra information, but remove arXiv info
+    const notes = preprintExtra.extra
+      .split("\n")
+      .filter((line) => !line.startsWith("arXiv:"))
+      .join("\n");
+    if (notes) journalJSON.extra += "\n" + notes;
+
     /* Avoid citation key collision after preprint item updates (say year)
      * For example, no collision:
      * - Published item: li_wang_2024-1
@@ -96,7 +111,11 @@ export class arXivMerge {
      * The workaround here is to set a random citation key for the published item.
      * TODO: What if the user wants to keep the citation key (not fixed in BBT)?
      */
-    publishedItem.setField("extra", `Citation Key: ${crypto.randomUUID()}`);
+    const tempExtra = Zotero.Utilities.Internal.combineExtraFields(
+      journalJSON.extra as string,
+      new Map([["Citation Key", crypto.randomUUID()]]),
+    );
+    publishedItem.setField("extra", tempExtra);
     publishedItem.saveTx();
     preprintItem.fromJSON(journalJSON);
     preprintItem.saveTx();