fix: fix unstable cache key

arianrhodsandlot · Oct 21, 2023 · 25948dd · 25948dd
1 parent 8fbec59
commit 25948dd
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/src/core/classes/file-system-providers/dropbox-provider.ts b/src/core/classes/file-system-providers/dropbox-provider.ts
@@ -34,7 +34,7 @@ export class DropboxProvider implements FileSystemProvider {
   }
 
   async getContentAndCache(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'DropboxProvider.peekContent', path }
     const blob = await this.getContent(path)
 
     const text = await blob.text()
@@ -45,7 +45,7 @@ export class DropboxProvider implements FileSystemProvider {
   }
 
   async peekContent(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'DropboxProvider.peekContent', path }
     const rawCache = await RequestCache.get(cacheKey)
     return rawCache?.value
   }
@@ -78,7 +78,7 @@ export class DropboxProvider implements FileSystemProvider {
     } while (listNextPage)
 
     if (children?.length) {
-      RequestCache.set({ name: `${this.constructor.name}.peek`, path }, children)
+      RequestCache.set({ name: 'DropboxProvider.peek', path }, children)
     }
 
     return children.map(
@@ -93,7 +93,7 @@ export class DropboxProvider implements FileSystemProvider {
   }
 
   async peek(path: string) {
-    const rawCache = await RequestCache.get({ name: `${this.constructor.name}.peek`, path })
+    const rawCache = await RequestCache.get({ name: 'DropboxProvider.peek', path })
     const children = rawCache?.value
     const fileAccessors: FileAccessor[] | undefined = children?.map(
       (item) =>

diff --git a/src/core/classes/file-system-providers/google-drive-provider.ts b/src/core/classes/file-system-providers/google-drive-provider.ts
@@ -50,7 +50,7 @@ export class GoogleDriveProvider implements FileSystemProvider {
   }
 
   async getContentAndCache(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'GoogleDriveProvider.peekContent', path }
     const blob = await this.getContent(path)
 
     const text = await blob.text()
@@ -61,7 +61,7 @@ export class GoogleDriveProvider implements FileSystemProvider {
   }
 
   async peekContent(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'GoogleDriveProvider.peekContent', path }
     const rawCache = await RequestCache.get(cacheKey)
     return rawCache?.value
   }
@@ -110,7 +110,7 @@ export class GoogleDriveProvider implements FileSystemProvider {
     } while (listNextPage)
 
     if (children?.length) {
-      RequestCache.set({ name: `${this.constructor.name}.peek`, path }, children)
+      RequestCache.set({ name: 'GoogleDriveProvider.peek', path }, children)
     }
 
     return children.map(
@@ -126,7 +126,7 @@ export class GoogleDriveProvider implements FileSystemProvider {
   }
 
   async peek(path: string) {
-    const rawCache = await RequestCache.get({ name: `${this.constructor.name}.peek`, path })
+    const rawCache = await RequestCache.get({ name: 'GoogleDriveProvider.peek', path })
     const children = rawCache?.value
     const fileAccessors: FileAccessor[] | undefined = children?.map(
       (item) =>
@@ -214,7 +214,7 @@ export class GoogleDriveProvider implements FileSystemProvider {
   private async getDirectoryWithCache(path: string) {
     const { cacheable } = await RequestCache.makeCacheable({
       func: (path: string) => this.getDirectory(path),
-      identifier: `${this.constructor.name}.getDirectory`,
+      identifier: 'GoogleDriveProvider.getDirectory',
     })
     return cacheable(path)
   }

diff --git a/src/core/classes/file-system-providers/onedrive-provider.ts b/src/core/classes/file-system-providers/onedrive-provider.ts
@@ -35,7 +35,7 @@ export class OnedriveProvider implements FileSystemProvider {
   }
 
   async getContentAndCache(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'OnedriveProvider.peekContent', path }
     const blob = await this.getContent(path)
 
     const text = await blob.text()
@@ -46,7 +46,7 @@ export class OnedriveProvider implements FileSystemProvider {
   }
 
   async peekContent(path: string) {
-    const cacheKey = { name: `${this.constructor.name}.peekContent`, path }
+    const cacheKey = { name: 'OnedriveProvider.peekContent', path }
     const rawCache = await RequestCache.get(cacheKey)
     return rawCache?.value
   }
@@ -80,7 +80,7 @@ export class OnedriveProvider implements FileSystemProvider {
     } while (listNextPage)
 
     if (children?.length) {
-      RequestCache.set({ name: `${this.constructor.name}.peek`, path }, children)
+      RequestCache.set({ name: 'OnedriveProvider.peek', path }, children)
     }
 
     return children.map(
@@ -95,7 +95,7 @@ export class OnedriveProvider implements FileSystemProvider {
   }
 
   async peek(path: string) {
-    const rawCache = await RequestCache.get({ name: `${this.constructor.name}.peek`, path })
+    const rawCache = await RequestCache.get({ name: 'OnedriveProvider.peek', path })
     const children = rawCache?.value
     const fileAccessors: FileAccessor[] | undefined = children?.map(
       (item) =>