gulp-tasks/build-node-packages.js

@@ -71,10 +71,19 @@ async function generateWorkboxBuildJSONSchema(packagePath) {
       schema.properties.include.items = {};
     }
 
+    // See https://github.com/GoogleChrome/workbox/issues/2910
+    if (schema.definitions.OnSyncCallback) {
+      schema.definitions.OnSyncCallback = {};
+    }
+
     if (schema.definitions.RouteMatchCallback) {
       schema.definitions.RouteMatchCallback = {};
     }
 
+    if (schema.definitions.RouteHandlerCallback) {
+      schema.definitions.RouteHandlerCallback = {};
+    }
+
     // See https://github.com/GoogleChrome/workbox/issues/2901
     if (schema.definitions.WorkboxPlugin) {
       for (const plugin of Object.keys(

lerna.json

@@ -3,5 +3,5 @@
   "packages": [
     "packages/*"
   ],
-  "version": "6.2.3"
+  "version": "6.2.4"
 }

packages/workbox-background-sync/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workbox-background-sync",
-  "version": "6.2.3",
+  "version": "6.2.4",
   "license": "MIT",
   "author": "Google's Web DevRel Team",
   "description": "Queues failed requests and uses the Background Sync API to replay them when the network is available",
@@ -25,6 +25,6 @@
   "types": "index.d.ts",
   "dependencies": {
     "idb": "^6.0.0",
-    "workbox-core": "6.2.3"
+    "workbox-core": "6.2.4"
   }
 }

packages/workbox-background-sync/src/_version.ts

@@ -1,2 +1,2 @@
 // @ts-ignore
-try{self['workbox:background-sync:6.2.3']&&_()}catch(e){}
+try{self['workbox:background-sync:6.2.4']&&_()}catch(e){}

packages/workbox-broadcast-update/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workbox-broadcast-update",
-  "version": "6.2.3",
+  "version": "6.2.4",
   "license": "MIT",
   "author": "Google's Web DevRel Team",
   "description": "A service worker helper library that uses the Broadcast Channel API to announce when a cached response has updated",
@@ -22,6 +22,6 @@
   "module": "index.mjs",
   "types": "index.d.ts",
   "dependencies": {
-    "workbox-core": "6.2.3"
+    "workbox-core": "6.2.4"
   }
 }

packages/workbox-broadcast-update/src/_version.ts

@@ -1,2 +1,2 @@
 // @ts-ignore
-try{self['workbox:broadcast-update:6.2.3']&&_()}catch(e){}
+try{self['workbox:broadcast-update:6.2.4']&&_()}catch(e){}

packages/workbox-build/package.json

@@ -1,6 +1,6 @@
 {
   "name": "workbox-build",
-  "version": "6.2.3",
+  "version": "6.2.4",
   "description": "A module that integrates into your build process, helping you generate a manifest of local files that workbox-sw should precache.",
   "keywords": [
     "workbox",
@@ -43,21 +43,21 @@
     "strip-comments": "^2.0.1",
     "tempy": "^0.6.0",
     "upath": "^1.2.0",
-    "workbox-background-sync": "6.2.3",
-    "workbox-broadcast-update": "6.2.3",
-    "workbox-cacheable-response": "6.2.3",
-    "workbox-core": "6.2.3",
-    "workbox-expiration": "6.2.3",
-    "workbox-google-analytics": "6.2.3",
-    "workbox-navigation-preload": "6.2.3",
-    "workbox-precaching": "6.2.3",
-    "workbox-range-requests": "6.2.3",
-    "workbox-recipes": "6.2.3",
-    "workbox-routing": "6.2.3",
-    "workbox-strategies": "6.2.3",
-    "workbox-streams": "6.2.3",
-    "workbox-sw": "6.2.3",
-    "workbox-window": "6.2.3"
+    "workbox-background-sync": "6.2.4",
+    "workbox-broadcast-update": "6.2.4",
+    "workbox-cacheable-response": "6.2.4",
+    "workbox-core": "6.2.4",
+    "workbox-expiration": "6.2.4",
+    "workbox-google-analytics": "6.2.4",
+    "workbox-navigation-preload": "6.2.4",
+    "workbox-precaching": "6.2.4",
+    "workbox-range-requests": "6.2.4",
+    "workbox-recipes": "6.2.4",
+    "workbox-routing": "6.2.4",
+    "workbox-strategies": "6.2.4",
+    "workbox-streams": "6.2.4",
+    "workbox-sw": "6.2.4",
+    "workbox-window": "6.2.4"
   },
   "main": "build/index.js",
   "workbox": {

packages/workbox-build/src/cdn-details.json

@@ -2,5 +2,5 @@
   "origin": "https://storage.googleapis.com",
   "bucketName": "workbox-cdn",
   "releasesDir": "releases",
-  "latestVersion": "6.2.3"
+  "latestVersion": "6.2.4"
 }

packages/workbox-build/src/lib/errors.ts

@@ -120,4 +120,5 @@ export const errors = {
     configure a custom cacheName.`,
   'manifest-transforms': ol`When using manifestTransforms, you must provide
     an array of functions.`,
+  'invalid-handler-string': ol`The handler name provided is not valid: `,
 };

packages/workbox-build/src/lib/validate-options.ts

@@ -40,7 +40,12 @@ export class WorkboxConfigError extends Error {
   }
 }
 
-function validate<T>(input: unknown, methodName: MethodNames): T {
+// Some methods need to do follow-up validation using the JSON schema,
+// so return both the validated options and then schema.
+function validate<T>(
+  input: unknown,
+  methodName: MethodNames,
+): [T, JSONSchemaType<T>] {
   // Don't mutate input: https://github.com/GoogleChrome/workbox/issues/2158
   const inputCopy = Object.assign({}, input);
   // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
@@ -49,7 +54,7 @@ function validate<T>(input: unknown, methodName: MethodNames): T {
   if (validate(inputCopy)) {
     // All methods support manifestTransforms, so validate it here.
     ensureValidManifestTransforms(inputCopy);
-    return inputCopy;
+    return [inputCopy, jsonSchema];
   }
 
   const betterErrors = betterAjvErrors({
@@ -126,23 +131,65 @@ function ensureValidRuntimeCachingOrGlobDirectory(
   }
 }
 
+// This is... messy, because we can't rely on the built-in ajv validation for
+// runtimeCaching.handler, as it needs to accept {} (i.e. any) due to
+// https://github.com/GoogleChrome/workbox/pull/2899
+// So we need to perform validation when a string (not a function) is used.
+function ensureValidStringHandler(
+  options: GenerateSWOptions | WebpackGenerateSWOptions,
+  jsonSchema: JSONSchemaType<GenerateSWOptions | WebpackGenerateSWOptions>,
+): void {
+  let validHandlers: Array<string> = [];
+  /* eslint-disable */
+  for (const handler of jsonSchema.definitions?.RuntimeCaching?.properties
+    ?.handler?.anyOf || []) {
+    if ('enum' in handler) {
+      validHandlers = handler.enum;
+      break;
+    }
+  }
+  /* eslint-enable */
+
+  for (const runtimeCaching of options.runtimeCaching || []) {
+    if (
+      typeof runtimeCaching.handler === 'string' &&
+      !validHandlers.includes(runtimeCaching.handler)
+    ) {
+      throw new WorkboxConfigError(
+        errors['invalid-handler-string'] + runtimeCaching.handler,
+      );
+    }
+  }
+}
+
 export function validateGenerateSWOptions(input: unknown): GenerateSWOptions {
-  const validatedOptions = validate<GenerateSWOptions>(input, 'GenerateSW');
+  const [validatedOptions, jsonSchema] = validate<GenerateSWOptions>(
+    input,
+    'GenerateSW',
+  );
   ensureValidNavigationPreloadConfig(validatedOptions);
   ensureValidCacheExpiration(validatedOptions);
   ensureValidRuntimeCachingOrGlobDirectory(validatedOptions);
+  ensureValidStringHandler(validatedOptions, jsonSchema);
 
   return validatedOptions;
 }
 
 export function validateGetManifestOptions(input: unknown): GetManifestOptions {
-  return validate<GetManifestOptions>(input, 'GetManifest');
+  const [validatedOptions] = validate<GetManifestOptions>(input, 'GetManifest');
+
+  return validatedOptions;
 }
 
 export function validateInjectManifestOptions(
   input: unknown,
 ): InjectManifestOptions {
-  return validate<InjectManifestOptions>(input, 'InjectManifest');
+  const [validatedOptions] = validate<InjectManifestOptions>(
+    input,
+    'InjectManifest',
+  );
+
+  return validatedOptions;
 }
 
 // The default `exclude: [/\.map$/, /^manifest.*\.js$/]` value can't be
@@ -157,13 +204,14 @@ export function validateWebpackGenerateSWOptions(
     },
     input,
   );
-  const validatedOptions = validate<WebpackGenerateSWOptions>(
+  const [validatedOptions, jsonSchema] = validate<WebpackGenerateSWOptions>(
     inputWithExcludeDefault,
     'WebpackGenerateSW',
   );
 
   ensureValidNavigationPreloadConfig(validatedOptions);
   ensureValidCacheExpiration(validatedOptions);
+  ensureValidStringHandler(validatedOptions, jsonSchema);
 
   return validatedOptions;
 }
@@ -178,8 +226,10 @@ export function validateWebpackInjectManifestOptions(
     },
     input,
   );
-  return validate<WebpackInjectManifestOptions>(
+  const [validatedOptions] = validate<WebpackInjectManifestOptions>(
     inputWithExcludeDefault,
     'WebpackInjectManifest',
   );
+
+  return validatedOptions;
 }

packages/workbox-build/src/schema/GenerateSWOptions.json

@@ -407,11 +407,7 @@
         "urlPattern"
       ]
     },
-    "RouteHandlerCallback": {
-      "description": "The \"handler\" callback is invoked whenever a `Router` matches a URL/Request\nto a `Route` via its `RouteMatchCallback`. This handler callback should\nreturn a `Promise` that resolves with a `Response`.\n\nIf a non-empty array or object is returned by the `RouteMatchCallback` it\nwill be passed in as this handler's `options.params` argument.",
-      "type": "object",
-      "additionalProperties": false
-    },
+    "RouteHandlerCallback": {},
     "RouteHandlerObject": {
       "description": "An object with a `handle` method of type `RouteHandlerCallback`.\n\nA `Route` object can be created with either an `RouteHandlerCallback`\nfunction or this `RouteHandler` object. The benefit of the `RouteHandler`\nis it can be extended (as is done by the `workbox-strategies` package).",
       "type": "object",
@@ -437,10 +433,7 @@
       },
       "additionalProperties": false
     },
-    "OnSyncCallback": {
-      "type": "object",
-      "additionalProperties": false
-    },
+    "OnSyncCallback": {},
     "BroadcastCacheUpdateOptions": {
       "type": "object",
       "properties": {

packages/workbox-build/src/schema/GetManifestOptions.json

@@ -299,11 +299,7 @@
         "urlPattern"
       ]
     },
-    "RouteHandlerCallback": {
-      "description": "The \"handler\" callback is invoked whenever a `Router` matches a URL/Request\nto a `Route` via its `RouteMatchCallback`. This handler callback should\nreturn a `Promise` that resolves with a `Response`.\n\nIf a non-empty array or object is returned by the `RouteMatchCallback` it\nwill be passed in as this handler's `options.params` argument.",
-      "type": "object",
-      "additionalProperties": false
-    },
+    "RouteHandlerCallback": {},
     "RouteHandlerObject": {
       "description": "An object with a `handle` method of type `RouteHandlerCallback`.\n\nA `Route` object can be created with either an `RouteHandlerCallback`\nfunction or this `RouteHandler` object. The benefit of the `RouteHandler`\nis it can be extended (as is done by the `workbox-strategies` package).",
       "type": "object",
@@ -329,10 +325,7 @@
       },
       "additionalProperties": false
     },
-    "OnSyncCallback": {
-      "type": "object",
-      "additionalProperties": false
-    },
+    "OnSyncCallback": {},
     "BroadcastCacheUpdateOptions": {
       "type": "object",
       "properties": {

packages/workbox-build/src/schema/InjectManifestOptions.json

@@ -311,11 +311,7 @@
         "urlPattern"
       ]
     },
-    "RouteHandlerCallback": {
-      "description": "The \"handler\" callback is invoked whenever a `Router` matches a URL/Request\nto a `Route` via its `RouteMatchCallback`. This handler callback should\nreturn a `Promise` that resolves with a `Response`.\n\nIf a non-empty array or object is returned by the `RouteMatchCallback` it\nwill be passed in as this handler's `options.params` argument.",
-      "type": "object",
-      "additionalProperties": false
-    },
+    "RouteHandlerCallback": {},
     "RouteHandlerObject": {
       "description": "An object with a `handle` method of type `RouteHandlerCallback`.\n\nA `Route` object can be created with either an `RouteHandlerCallback`\nfunction or this `RouteHandler` object. The benefit of the `RouteHandler`\nis it can be extended (as is done by the `workbox-strategies` package).",
       "type": "object",
@@ -341,10 +337,7 @@
       },
       "additionalProperties": false
     },
-    "OnSyncCallback": {
-      "type": "object",
-      "additionalProperties": false
-    },
+    "OnSyncCallback": {},
     "BroadcastCacheUpdateOptions": {
       "type": "object",
       "properties": {

packages/workbox-build/src/schema/WebpackGenerateSWOptions.json

@@ -386,11 +386,7 @@
         "urlPattern"
       ]
     },
-    "RouteHandlerCallback": {
-      "description": "The \"handler\" callback is invoked whenever a `Router` matches a URL/Request\nto a `Route` via its `RouteMatchCallback`. This handler callback should\nreturn a `Promise` that resolves with a `Response`.\n\nIf a non-empty array or object is returned by the `RouteMatchCallback` it\nwill be passed in as this handler's `options.params` argument.",
-      "type": "object",
-      "additionalProperties": false
-    },
+    "RouteHandlerCallback": {},
     "RouteHandlerObject": {
       "description": "An object with a `handle` method of type `RouteHandlerCallback`.\n\nA `Route` object can be created with either an `RouteHandlerCallback`\nfunction or this `RouteHandler` object. The benefit of the `RouteHandler`\nis it can be extended (as is done by the `workbox-strategies` package).",
       "type": "object",
@@ -416,10 +412,7 @@
       },
       "additionalProperties": false
     },
-    "OnSyncCallback": {
-      "type": "object",
-      "additionalProperties": false
-    },
+    "OnSyncCallback": {},
     "BroadcastCacheUpdateOptions": {
       "type": "object",
       "properties": {

packages/workbox-build/src/schema/WebpackInjectManifestOptions.json

@@ -298,11 +298,7 @@
         "urlPattern"
       ]
     },
-    "RouteHandlerCallback": {
-      "description": "The \"handler\" callback is invoked whenever a `Router` matches a URL/Request\nto a `Route` via its `RouteMatchCallback`. This handler callback should\nreturn a `Promise` that resolves with a `Response`.\n\nIf a non-empty array or object is returned by the `RouteMatchCallback` it\nwill be passed in as this handler's `options.params` argument.",
-      "type": "object",
-      "additionalProperties": false
-    },
+    "RouteHandlerCallback": {},
     "RouteHandlerObject": {
       "description": "An object with a `handle` method of type `RouteHandlerCallback`.\n\nA `Route` object can be created with either an `RouteHandlerCallback`\nfunction or this `RouteHandler` object. The benefit of the `RouteHandler`\nis it can be extended (as is done by the `workbox-strategies` package).",
       "type": "object",
@@ -328,10 +324,7 @@
       },
       "additionalProperties": false
     },
-    "OnSyncCallback": {
-      "type": "object",
-      "additionalProperties": false
-    },
+    "OnSyncCallback": {},
     "BroadcastCacheUpdateOptions": {
       "type": "object",
       "properties": {