fix(gatsby-transformer-remark): correctly pass cache to sub plugins

e2e-tests/development-runtime/content/2018-12-14-hello-world.md

@@ -6,3 +6,5 @@ date: 2018-12-14
 This is a truly meaningful blog post
 
 <h2 data-testid="sub-title">%SUB_TITLE%</h2>
+
+<h2 data-testid="gatsby-remark-subcache-value"></h2>

e2e-tests/development-runtime/cypress/integration/functionality/sub-plugin-caching.js

@@ -0,0 +1,16 @@
+/*
+ * This e2e test validates that the cache structure
+ * is unique per plugin (even sub-plugins)
+ * and can interact between Gatsby lifecycles and a plugin
+ */
+describe(`sub-plugin caching`, () => {
+  beforeEach(() => {
+    cy.visit(`/2018-12-14-hello-world/`).waitForAPI(`onRouteUpdate`)
+  })
+
+  it(`has access to custom sub-plugin cache`, () => {
+    cy.getTestElement(`gatsby-remark-subcache-value`)
+      .invoke(`text`)
+      .should(`eq`, `Hello World`)
+  })
+})

e2e-tests/development-runtime/gatsby-config.js

@@ -24,7 +24,7 @@ module.exports = {
     {
       resolve: `gatsby-transformer-remark`,
       options: {
-        plugins: [],
+        plugins: [`gatsby-remark-subcache`],
       },
     },
     `gatsby-plugin-sharp`,

e2e-tests/development-runtime/plugins/gatsby-remark-subcache/constants.js

@@ -0,0 +1 @@
+exports.id = `gatsby-remark-subcache-value`

e2e-tests/development-runtime/plugins/gatsby-remark-subcache/gatsby-node.js

@@ -0,0 +1,5 @@
+const { id } = require(`./constants`)
+
+exports.onPreBootstrap = async ({ cache }) => {
+  await cache.set(id, `Hello World`)
+}

e2e-tests/development-runtime/plugins/gatsby-remark-subcache/index.js

@@ -0,0 +1,11 @@
+const visit = require(`unist-util-visit`)
+const { id } = require(`./constants`)
+
+module.exports = function remarkPlugin({ cache, markdownAST }) {
+  visit(markdownAST, `html`, async node => {
+    if (node.value.match(id)) {
+      const value = await cache.get(id)
+      node.value = node.value.replace(/></, () => `>${value}<`)
+    }
+  })
+}

e2e-tests/development-runtime/plugins/gatsby-remark-subcache/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "gatsby-remark-subcache"
+}

packages/gatsby-transformer-remark/src/extend-node-type.js

@@ -58,6 +58,26 @@ const tableOfContentsCacheKey = node =>
 const withPathPrefix = (url, pathPrefix) =>
   (pathPrefix + url).replace(/\/\//, `/`)
 
+// TODO: remove this check with next major release
+// the getCache feature was implemented in gatsby@2.0.88
+const safeGetCache = ({ getCache, cache, reporter }) => {
+  const GATSBY_VERSION_WITH_GET_CACHE = `2.0.88`
+  let warned = false
+  return id => {
+    if (!getCache) {
+      if (!warned) {
+        warned = true
+        reporter.warn(reporter.stripIndent`
+          It looks like you're using a slightly outdated version of gatsby with gatsby-transformer-remark.
+          To update, run 'npm install gatsby@^${GATSBY_VERSION_WITH_GET_CACHE} --save' or 'yarn add gatsby@^${GATSBY_VERSION_WITH_GET_CACHE}'
+        `)
+      }
+      return cache
+    }
+    return getCache(id)
+  }
+}
+
 /**
  * Map that keeps track of generation of AST to not generate it multiple
  * times in parallel.
@@ -67,7 +87,16 @@ const withPathPrefix = (url, pathPrefix) =>
 const ASTPromiseMap = new Map()
 
 module.exports = (
-  { type, store, pathPrefix, getNode, getNodesByType, cache, reporter },
+  {
+    type,
+    pathPrefix,
+    getNode,
+    getNodesByType,
+    cache,
+    getCache: possibleGetCache,
+    reporter,
+    ...rest
+  },
   pluginOptions
 ) => {
   if (type.name !== `MarkdownRemark`) {
@@ -76,6 +105,8 @@ module.exports = (
   pluginsCacheStr = pluginOptions.plugins.map(p => p.name).join(``)
   pathPrefixCacheStr = pathPrefix || ``
 
+  const getCache = safeGetCache({ cache, getCache: possibleGetCache, reporter })
+
   return new Promise((resolve, reject) => {
     // Setup Remark.
     const {
@@ -150,7 +181,9 @@ module.exports = (
               files: fileNodes,
               getNode,
               reporter,
-              cache,
+              cache: getCache(plugin.name),
+              getCache,
+              ...rest,
             },
             plugin.pluginOptions
           )
@@ -218,7 +251,9 @@ module.exports = (
               files: fileNodes,
               pathPrefix,
               reporter,
-              cache,
+              cache: getCache(plugin.name),
+              getCache,
+              ...rest,
             },
             plugin.pluginOptions
           )

packages/gatsby-transformer-remark/src/on-node-create.js

@@ -3,7 +3,7 @@ const crypto = require(`crypto`)
 const _ = require(`lodash`)
 
 module.exports = async function onCreateNode(
-  { node, getNode, loadNodeContent, actions, createNodeId, reporter },
+  { node, loadNodeContent, actions, createNodeId, reporter },
   pluginOptions
 ) {
   const { createNode, createParentChildLink } = actions