1 files changed, 25 insertions, 18 deletions

diff --git a/src/client/util/LinkManager.ts b/src/client/util/LinkManager.ts
index 3f98ab3c4..344e2e4c0 100644
--- a/src/client/util/LinkManager.ts
+++ b/src/client/util/LinkManager.ts
@@ -136,7 +136,7 @@ export class LinkManager {
             true
         );
         FieldLoader.ServerLoadStatus.message = 'links';
-        this.addLinkDB(Doc.LinkDBDoc());
+        Doc.LinkDBDoc() && this.addLinkDB(Doc.LinkDBDoc()!);
     }
 
     public createlink_relationshipLists = () => {
@@ -148,16 +148,21 @@ export class LinkManager {
 
     public addLink(linkDoc: Doc, checkExists = false) {
         Doc.AddDocToList(Doc.UserDoc(), 'links', linkDoc);
-        if (!checkExists || !DocListCast(Doc.LinkDBDoc().data).includes(linkDoc)) {
-            Doc.AddDocToList(Doc.LinkDBDoc(), 'data', linkDoc);
-            // eslint-disable-next-line no-use-before-define
-            setTimeout(UPDATE_SERVER_CACHE, 100);
+        if (Doc.LinkDBDoc()) {
+            if (!checkExists || !DocListCast(Doc.LinkDBDoc()!.data).includes(linkDoc)) {
+                Doc.AddDocToList(Doc.LinkDBDoc()!, 'data', linkDoc);
+                // eslint-disable-next-line no-use-before-define
+                setTimeout(UPDATE_SERVER_CACHE, 100);
+            }
         }
     }
     public deleteLink(linkDoc: Doc) {
-        const ret = Doc.RemoveDocFromList(Doc.LinkDBDoc(), 'data', linkDoc);
-        linkDoc.$link_anchor_1 = linkDoc.$link_anchor_2 = undefined;
-        return ret;
+        if (Doc.LinkDBDoc()) {
+            const ret = Doc.RemoveDocFromList(Doc.LinkDBDoc()!, 'data', linkDoc);
+            linkDoc.$link_anchor_1 = linkDoc.$link_anchor_2 = undefined;
+            return ret;
+        }
+        return false;
     }
     public deleteAllLinksOnAnchor(anchor: Doc) {
         LinkManager.Instance.relatedLinker(anchor).forEach(LinkManager.Instance.deleteLink);
@@ -178,8 +183,8 @@ export class LinkManager {
         }
 
         const dirLinks = Array.from(anchor[DocData][DirectLinks]).filter(l => Doc.GetProto(anchor) === anchor[DocData] || ['1', '2'].includes(LinkManager.anchorIndex(l, anchor) as '0' | '1' | '2'));
-        const anchorRoot = DocCast(anchor.rootDocument, anchor); // template Doc fields store annotations on the topmost root of a template (not on themselves since the template layout items are only for layout)
-        const annos = DocListCast(anchorRoot[Doc.LayoutFieldKey(anchor) + '_annotations']);
+        const anchorRoot = DocCast(anchor.rootDocument, anchor)!; // template Doc fields store annotations on the topmost root of a template (not on themselves since the template layout items are only for layout)
+        const annos = DocListCast(anchorRoot[Doc.LayoutDataKey(anchor) + '_annotations']);
         return Array.from(
             annos.reduce((set, anno) => {
                 if (!processed.includes(anno)) {
@@ -242,11 +247,13 @@ export function UPDATE_SERVER_CACHE() {
         .filter(doc => doc instanceof Doc)
         .map(doc => doc as Doc);
     const references = new Set<Doc>(prototypes);
+    DocCast(Doc.UserDoc().myLinkDatabase) && references.add(DocCast(Doc.UserDoc().myLinkDatabase)!); // prevent crawling through link database here -- see below
     Doc.FindReferences(Doc.UserDoc(), references, undefined);
-    DocListCast(DocCast(Doc.UserDoc().myLinkDatabase).data).forEach(link => {
-        if (!references.has(DocCast(link.link_anchor_1)) && !references.has(DocCast(link.link_anchor_2))) {
-            Doc.RemoveDocFromList(DocCast(Doc.UserDoc().myLinkDatabase), 'data', link);
-            Doc.AddDocToList(Doc.MyRecentlyClosed, undefined, link);
+
+    DocListCast(DocCast(Doc.UserDoc().myLinkDatabase)?.data).forEach(link => {
+        if (DocCast(link.link_anchor_1) && !references.has(DocCast(link.link_anchor_1)!) && DocCast(link.link_anchor_2) && !references.has(DocCast(link.link_anchor_2)!)) {
+            DocCast(Doc.UserDoc().myLinkDatabase) && Doc.RemoveDocFromList(DocCast(Doc.UserDoc().myLinkDatabase)!, 'data', link);
+            Doc.MyRecentlyClosed && Doc.AddDocToList(Doc.MyRecentlyClosed, undefined, link);
         }
     });
     LinkManager.Instance.userLinkDBs.forEach(linkDb => Doc.FindReferences(linkDb, references, undefined));
@@ -257,10 +264,10 @@ export function UPDATE_SERVER_CACHE() {
     cacheDocumentIds = newCacheUpdate;
 
     // print out cached docs
-    //Doc.MyDockedBtns.linearView_IsOpen && console.log('Set cached docs = ');
-    // const isFiltered = filtered.filter(doc => !Doc.IsSystem(doc));
-    // const strings = isFiltered.map(doc => StrCast(doc.title) + ' ' + (Doc.IsDataProto(doc) ? '(data)' : '(embedding)'));
-    //Doc.MyDockedBtns.linearView_IsOpen && strings.sort().forEach((str, i) => console.log(i.toString() + ' ' + str));
+    Doc.MyDockedBtns?.linearView_IsOpen && console.log('Set cached docs = ');
+    const isFiltered = filtered.filter(doc => !Doc.IsSystem(doc));
+    const strings = isFiltered.map(doc => StrCast(doc.title) + ' ' + (Doc.IsDataProto(doc) ? '(data)' : '(embedding)'));
+    Doc.MyDockedBtns?.linearView_IsOpen && strings.sort().forEach((str, i) => console.log(i.toString() + ' ' + str));
 
     rp.post(ClientUtils.prepend('/setCacheDocumentIds'), {
         body: {