fix(gatsby-plugin-offline): prevent incorrect revisioning of static file by workbox (#10416)

Author: vtenfys

packages/gatsby-plugin-offline/.gitignore

@@ -1,3 +1,4 @@
 /*.js
 !index.js
+!sw-append.js
 yarn.lock

packages/gatsby-plugin-offline/README.md

@@ -40,12 +40,12 @@ const options = {
   cacheId: `gatsby-plugin-offline`,
   // Don't cache-bust JS or CSS files, and anything in the static directory,
   // since these files have unique URLs and their contents will never change
-  dontCacheBustUrlsMatching: /(\.js$|\.css$|\/static\/)/,
+  dontCacheBustUrlsMatching: /(\.js$|\.css$|static\/)/,
   runtimeCaching: [
     {
       // Use cacheFirst since these don't need to be revalidated (same RegExp
       // and same reason as above)
-      urlPattern: /(\.js$|\.css$|\/static\/)/,
+      urlPattern: /(\.js$|\.css$|static\/)/,
       handler: `cacheFirst`,
     },
     {

packages/gatsby-plugin-offline/src/gatsby-node.js

@@ -82,12 +82,12 @@ exports.onPostBuild = (args, pluginOptions) => {
     cacheId: `gatsby-plugin-offline`,
     // Don't cache-bust JS or CSS files, and anything in the static directory,
     // since these files have unique URLs and their contents will never change
-    dontCacheBustUrlsMatching: /(\.js$|\.css$|\/static\/)/,
+    dontCacheBustUrlsMatching: /(\.js$|\.css$|static\/)/,
     runtimeCaching: [
       {
         // Use cacheFirst since these don't need to be revalidated (same RegExp
         // and same reason as above)
-        urlPattern: /(\.js$|\.css$|\/static\/)/,
+        urlPattern: /(\.js$|\.css$|static\/)/,
         handler: `cacheFirst`,
       },
       {
@@ -126,7 +126,7 @@ exports.onPostBuild = (args, pluginOptions) => {
         .readFileSync(`${__dirname}/sw-append.js`, `utf8`)
         .replace(/%pathPrefix%/g, pathPrefix)
 
-      fs.appendFileSync(`public/sw.js`, swAppend)
+      fs.appendFileSync(`public/sw.js`, `\n` + swAppend)
       console.log(
         `Generated ${swDest}, which will precache ${count} files, totaling ${size} bytes.`
       )