From 3eeeee4b064051c20926ed3b043fdca4c892a0a7 Mon Sep 17 00:00:00 2001
From: Martti Malmi <sirius@iki.fi>
Date: Tue, 9 Jan 2024 14:56:20 +0200
Subject: [PATCH] use queueMicrotask instead of setTimeout 0

---
 packages/app/src/Db/FuzzySearch.ts            |  4 ++--
 .../system/src/SocialGraph/SocialGraph.ts     | 20 +------------------
 2 files changed, 3 insertions(+), 21 deletions(-)

diff --git a/packages/app/src/Db/FuzzySearch.ts b/packages/app/src/Db/FuzzySearch.ts
index 7efe0a66..b8921223 100644
--- a/packages/app/src/Db/FuzzySearch.ts
+++ b/packages/app/src/Db/FuzzySearch.ts
@@ -20,7 +20,7 @@ export const addEventToFuzzySearch = ev => {
   if (ev.kind !== 0) {
     return;
   }
-  setTimeout(() => {
+  queueMicrotask(() => {
     const existing = profileTimestamps.get(ev.pubkey);
     if (existing) {
       if (existing > ev.created_at) {
@@ -42,7 +42,7 @@ export const addEventToFuzzySearch = ev => {
 };
 
 export const addCachedMetadataToFuzzySearch = (profile: CachedMetadata) => {
-  setTimeout(() => {
+  queueMicrotask(() => {
     const existing = profileTimestamps.get(profile.pubkey);
     if (existing) {
       if (existing > profile.created) {
diff --git a/packages/system/src/SocialGraph/SocialGraph.ts b/packages/system/src/SocialGraph/SocialGraph.ts
index 7db24e11..767ad2be 100644
--- a/packages/system/src/SocialGraph/SocialGraph.ts
+++ b/packages/system/src/SocialGraph/SocialGraph.ts
@@ -51,7 +51,7 @@ export default class SocialGraph {
     if (event.kind !== 3) {
       return;
     }
-    setTimeout(() => {
+    queueMicrotask(() => {
       try {
         const author = ID(event.pubkey);
         const timestamp = event.created_at;
@@ -116,15 +116,6 @@ export default class SocialGraph {
     if (!this.usersByFollowDistance.has(distance)) {
       this.usersByFollowDistance.set(distance, new Set());
     }
-    if (distance <= 2) {
-      /*
-      let unsub;
-      // get also profile events for profile search indexing
-      // eslint-disable-next-line prefer-const
-      unsub = PubSub.subscribe({ authors: [STR(user)], kinds: [0] }, () => unsub?.(), true);
-      // TODO subscribe once param?
-       */
-    }
     this.usersByFollowDistance.get(distance)?.add(user);
     // remove from higher distances
     for (const d of this.usersByFollowDistance.keys()) {
@@ -166,13 +157,6 @@ export default class SocialGraph {
     }
 
     this.followedByUser.get(follower)?.add(followedUser);
-    if (this.followedByUser.get(this.root)?.has(follower)) {
-      /*
-      setTimeout(() => {
-          PubSub.subscribe({ authors: [STR(followedUser)], kinds: [0, 3] }, undefined, true);
-        }, 0);
-       */
-    }
   }
 
   removeFollower(unfollowedUser: UID, follower: UID) {
@@ -236,7 +220,6 @@ export default class SocialGraph {
     if (includeSelf) {
       set.add(user);
     }
-    //return PubSub.subscribe({ kinds: [3], authors: [user] }, callback);
     return set;
   }
 
@@ -246,7 +229,6 @@ export default class SocialGraph {
     for (const id of this.followersByUser.get(userId) || []) {
       set.add(STR(id));
     }
-    //return PubSub.subscribe({ kinds: [3], '#p': [address] }, callback); // TODO this doesn't fire when a user is unfollowed
     return set;
   }
 }