Fix/precache cleanupresult compat

packages/workbox-precaching/src/PrecacheController.ts

@@ -264,7 +264,10 @@ class PrecacheController {
         printCleanupDetails(deletedURLs);
       }
 
-      return {deletedURLs};
+      return {
+        deletedURLs,
+        deletedCacheRequests: deletedURLs,
+      };
     });
   }
 

packages/workbox-precaching/src/_types.ts

@@ -14,6 +14,11 @@ export interface InstallResult {
 }
 
 export interface CleanupResult {
+  deletedURLs: string[];
+  /**
+   * @deprecated Use `deletedURLs` instead.
+   * TODO: Remove in the next major release.
+   */
   deletedCacheRequests: string[];
 }
 
@@ -51,8 +56,10 @@ export type urlManipulation = ({url}: {url: URL}) => URL[];
 
 /**
  * @typedef {Object} CleanupResult
- * @property {Array<string>} deletedCacheRequests List of URLs that were deleted
- * while cleaning up the cache.
+ * @property {Array<string>} deletedURLs List of URLs that were deleted while
+ * cleaning up the cache.
+ * @property {Array<string>} deletedCacheRequests Deprecated alias for
+ * deletedURLs.
  *
  * @memberof workbox-precaching
  */

test/workbox-precaching/sw/test-PrecacheController.mjs

@@ -834,9 +834,13 @@ describe(`PrecacheController`, function () {
         activateEvent,
       );
       expect(cleanupDetailsTwo.deletedURLs.length).to.equal(1);
+      expect(cleanupDetailsTwo.deletedCacheRequests.length).to.equal(1);
       expect(cleanupDetailsTwo.deletedURLs[0]).to.eql(
         `${location.origin}/scripts/index.js?__WB_REVISION__=1234`,
       );
+      expect(cleanupDetailsTwo.deletedCacheRequests[0]).to.eql(
+        `${location.origin}/scripts/index.js?__WB_REVISION__=1234`,
+      );
 
       const keysTwo = await cache.keys();
       expect(keysTwo.length).to.equal(cacheListTwo.length);
@@ -894,6 +898,10 @@ describe(`PrecacheController`, function () {
         `${location.origin}/index.1234.html`,
         `${location.origin}/scripts/stress.js?test=search&foo=bar&__WB_REVISION__=1234`,
       ]);
+      expect(cleanupDetailsTwo.deletedCacheRequests).to.have.members([
+        `${location.origin}/index.1234.html`,
+        `${location.origin}/scripts/stress.js?test=search&foo=bar&__WB_REVISION__=1234`,
+      ]);
 
       const keysTwo = await cache.keys();
       expect(keysTwo.length).to.equal(cacheListTwo.length);