Do not show dead guides for superseded extensions

Author: holly-cummins

gatsby-node.js

@@ -165,8 +165,6 @@ exports.sourceNodes = async ({
     node.metadata.minimumJavaVersion = node.metadata["minimum-java-version"]
     delete node.metadata["minimum-java-version"]
 
-    node.metadata.guide = await rewriteGuideUrl(extension)
-
     // Look for extensions which are not the same, but which have the same artifact id
     // (artifactId is just the 'a' part of the gav, artifact is the whole gav string)
 
@@ -210,6 +208,8 @@ exports.sourceNodes = async ({
 
     }
 
+    node.metadata.guide = await rewriteGuideUrl(node)
+
     if (node.metadata) {
       // Do the link to the download data
       node.metadata.downloads = node.metadata?.maven?.artifactId ? `${node.metadata.maven.groupId}:${node.metadata.maven.artifactId}` : undefined

src/components/util/guide-url-rewriter.js

@@ -1,4 +1,4 @@
-const rewriteGuideUrl = async ({ name, metadata }) => {
+const rewriteGuideUrl = async ({ name, metadata, isSuperseded }) => {
   // We have to access the url exist as a dynamic import (because CJS), await it because dynamic imports give a promise, and then destructure it to get the default
   // A simple property read won't work
   const {
@@ -21,6 +21,14 @@ const rewriteGuideUrl = async ({ name, metadata }) => {
         originalLink
       )
       return undefined
+    } else if (isSuperseded) {
+      console.warn(
+        "Stripping dead guide link from superseded extension. Extension is:",
+        name,
+        "and guide link is",
+        originalLink
+      )
+      return undefined
     } else if (metadata?.guide) {
       const transforms = [
         metadata => metadata.guide.replace("guides", "version/main/guides"),