Refactor: code

src/Webpack4Cache.js

@@ -26,12 +26,23 @@ export default class Webpack4Cache {
     // eslint-disable-next-line no-param-reassign
     task.cacheIdent = task.cacheIdent || serialize(task.cacheKeys);
 
-    const { data } = await cacache.get(this.cacheDir, task.cacheIdent);
+    let cachedResult;
 
-    return JSON.parse(data);
+    try {
+      cachedResult = await cacache.get(this.cacheDir, task.cacheIdent);
+    } catch (ignoreError) {
+      // eslint-disable-next-line no-undefined
+      return undefined;
+    }
+
+    return JSON.parse(cachedResult.data);
   }
 
-  async store(task, data) {
-    return cacache.put(this.cacheDir, task.cacheIdent, JSON.stringify(data));
+  async store(task) {
+    return cacache.put(
+      this.cacheDir,
+      task.cacheIdent,
+      JSON.stringify(task.output)
+    );
   }
 }

src/Webpack5Cache.js

@@ -11,16 +11,12 @@ export default class Cache {
 
   async get(task) {
     // eslint-disable-next-line no-param-reassign
-    task.cacheIdent = task.cacheIdent || `${task.name}`;
+    task.eTag = task.eTag || this.cache.getLazyHashedEtag(task.assetSource);
 
-    // eslint-disable-next-line no-param-reassign
-    task.cacheETag =
-      task.cacheETag || this.cache.getLazyHashedEtag(task.assetSource);
-
-    return this.cache.getPromise(task.cacheIdent, task.cacheETag);
+    return this.cache.getPromise(task.assetName, task.eTag);
   }
 
-  async store(task, data) {
-    return this.cache.storePromise(task.cacheIdent, task.cacheETag, data);
+  async store(task) {
+    return this.cache.storePromise(task.assetName, task.eTag, task.output);
   }
 }