🐛 fix(core): Treat empty cache dirs as noise

packages/core/src/utils/isDirEmpty.ts

@@ -1,7 +1,15 @@
 import fs from 'fs-extra'
 
 export async function isDirEmpty(dirPath: string) {
-  const files = await fs.readdir(dirPath)
+  try {
+    const files = await fs.readdir(dirPath)
 
-  return files.length === 0
+    return files.length === 0
+  } catch (e) {
+    if (e instanceof Error && e.message.includes('ENOENT')) {
+      return true
+    }
+
+    throw e
+  }
 }

packages/core/src/utils/triggerCacheCreation.ts

@@ -3,6 +3,7 @@ import fs from 'fs-extra'
 import { ensureCacheDirExists } from '../ensureCacheDirExists'
 import { createCacheForWalletSetupFunction } from './createCacheForWalletSetupFunction'
 import { getUniqueWalletSetupFunctions } from './getUniqueWalletSetupFunctions'
+import { isDirEmpty } from './isDirEmpty'
 
 export async function triggerCacheCreation(
   setupFunctions: Awaited<ReturnType<typeof getUniqueWalletSetupFunctions>>,
@@ -16,14 +17,22 @@ export async function triggerCacheCreation(
 
   for (const [funcHash, { fileName, setupFunction }] of setupFunctions) {
     const cachePath = path.join(cacheDirPath, funcHash)
-    if (await fs.exists(cachePath)) {
-      if (!force) {
-        console.log(`Cache already exists for ${funcHash}. Skipping...`)
-        continue
-      }
+    const doesCacheDirExist = await fs.exists(cachePath)
+    const isCacheDirEmpty = await isDirEmpty(cachePath)
+
+    if (doesCacheDirExist) {
+      if (isCacheDirEmpty) {
+        // In case of incorrect Playwright setup, the cache dir will be empty. For now, we're just deleting it.
+        await fs.remove(cachePath)
+      } else {
+        if (!force) {
+          console.log(`Cache already exists for ${funcHash}. Skipping...`)
+          continue
+        }
 
-      console.log(`Cache already exists for ${funcHash} but force flag is set. Deleting cache...`)
-      await fs.remove(cachePath)
+        console.log(`Cache already exists for ${funcHash} but force flag is set. Deleting cache...`)
+        await fs.remove(cachePath)
+      }
     }
 
     const fileNameWithCorrectExtension = fileName.replace(/\.(ts|js|mjs)$/, '.{ts,js,mjs}')

packages/core/test/utils/isDirEmpty.test.ts

@@ -25,6 +25,11 @@ describe('isDirEmpty', () => {
     vol.reset() // Clear the in-memory file system after each test
   })
 
+  it('returns `true` if the directory does not exist', async () => {
+    const isEmpty = await isDirEmpty(path.join('empty_dir'))
+    expect(isEmpty).toEqual(true)
+  })
+
   it('returns `true` if the directory is empty', async () => {
     const isEmpty = await isDirEmpty(ROOT_DIR)
     expect(isEmpty).toEqual(true)

packages/core/test/utils/triggerCacheCreation.test.ts

@@ -39,6 +39,15 @@ vi.mock('../../src/utils/createCacheForWalletSetupFunction', async () => {
   }
 })
 
+// We're not adding a test for code that uses `isDirEmpty` because soon it will be removed.
+vi.mock('../../src/utils/isDirEmpty', async () => {
+  return {
+    isDirEmpty: vi.fn(async () => {
+      return false
+    })
+  }
+})
+
 describe('triggerCacheCreation', () => {
   const createCacheForWalletSetupFunctionSpy = vi.spyOn(
     CreateCacheForWalletSetupFunction,