Merge pull request #1782 from augchan42/fix/twitter-profile-cache-1638

fix: remove twitter profile caching (#1638)
elizaOS · Jan 5, 2025 · 0e106cf · 0e106cf
2 parents ed16b0c + 6b0ae6e
commit 0e106cf
Showing 1 changed file with 1 addition and 22 deletions.
diff --git a/packages/client-twitter/src/base.ts b/packages/client-twitter/src/base.ts
@@ -729,28 +729,10 @@ export class ClientBase extends EventEmitter {
         );
     }
 
-    async getCachedProfile(username: string) {
-        return await this.runtime.cacheManager.get<TwitterProfile>(
-            `twitter/${username}/profile`
-        );
-    }
-
-    async cacheProfile(profile: TwitterProfile) {
-        await this.runtime.cacheManager.set(
-            `twitter/${profile.username}/profile`,
-            profile
-        );
-    }
-
     async fetchProfile(username: string): Promise<TwitterProfile> {
-        const cached = await this.getCachedProfile(username);
-
-        if (cached) return cached;
-
         try {
             const profile = await this.requestQueue.add(async () => {
                 const profile = await this.twitterClient.getProfile(username);
-                // console.log({ profile });
                 return {
                     id: profile.userId,
                     username,
@@ -767,13 +749,10 @@ export class ClientBase extends EventEmitter {
                 } satisfies TwitterProfile;
             });
 
-            this.cacheProfile(profile);
-
             return profile;
         } catch (error) {
             console.error("Error fetching Twitter profile:", error);
-
-            return undefined;
+            throw error;
         }
     }
 }