chore: minor failsafes and cleanup

Author: ovflowd

apps/site/app/[locale]/next-data/api-data/route.ts

@@ -32,31 +32,40 @@ export const GET = async () => {
     authorizationHeaders
   );
 
-  return gitHubApiResponse.json().then((apiDocsFiles: Array<GitHubApiFile>) => {
-    // maps over each api file and get the download_url, fetch the content and deflates it
-    const mappedApiFiles = apiDocsFiles.map(
-      async ({ name, path: filename, download_url }) => {
-        const apiFileResponse = await fetch(download_url);
-
-        // Retrieves the content as a raw text string
-        const source = await apiFileResponse.text();
-
-        // Removes empty/blank lines or lines just with spaces and trims each line
-        // from leading and trailing paddings/spaces
-        const cleanedContent = parseRichTextIntoPlainText(source);
-
-        const deflatedSource = deflateSync(cleanedContent).toString('base64');
-
-        return {
-          filename: filename,
-          pathname: getPathnameForApiFile(name, versionWithPrefix),
-          content: deflatedSource,
-        };
-      }
-    );
-
-    return Promise.all(mappedApiFiles).then(Response.json);
-  });
+  // transforms the response into an array of GitHubApiFile
+  const apiDocsFiles: Array<GitHubApiFile> = await gitHubApiResponse.json();
+
+  // prevent the route from crashing if the response is not an array of GitHubApiFile
+  // and return an empty array instead. This is a fallback for when the GitHub API is not available.
+  if (!Array.isArray(apiDocsFiles)) {
+    return Response.json([]);
+  }
+
+  // maps over each api file and get the download_url, fetch the content and deflates it
+  const mappedApiFiles = apiDocsFiles.map(
+    async ({ name, path: filename, download_url }) => {
+      const apiFileResponse = await fetch(download_url);
+
+      // Retrieves the content as a raw text string
+      const source = await apiFileResponse.text();
+
+      // Removes empty/blank lines or lines just with spaces and trims each line
+      // from leading and trailing paddings/spaces
+      const cleanedContent = parseRichTextIntoPlainText(source);
+
+      const deflatedSource = deflateSync(cleanedContent).toString('base64');
+
+      return {
+        filename: filename,
+        pathname: getPathnameForApiFile(name, versionWithPrefix),
+        content: deflatedSource,
+      };
+    }
+  );
+
+  const data = await Promise.all(mappedApiFiles);
+
+  return Response.json(data);
 };
 
 // This function generates the static paths that come from the dynamic segments

apps/site/app/[locale]/next-data/page-data/route.ts

@@ -10,44 +10,52 @@ import { parseRichTextIntoPlainText } from '#site/util/string';
 // for a digest and metadata of all existing pages on Node.js Website
 // @see https://nextjs.org/docs/app/building-your-application/routing/router-handlers
 export const GET = async () => {
+  // Retrieves all available routes for the default locale
   const allAvailbleRoutes = await dynamicRouter.getRoutesByLanguage(
     defaultLocale.code
   );
 
-  const availablePagesMetadata = allAvailbleRoutes
-    .filter(route => !route.startsWith('blog'))
-    .map(async pathname => {
-      const { source, filename } = await dynamicRouter.getMarkdownFile(
-        defaultLocale.code,
-        pathname
-      );
+  // We exclude the blog routes from the available pages metadata
+  // as they are generated separately and are not part of the static pages
+  // and are not part of the static pages metadata
+  const routesExceptBlog = allAvailbleRoutes.filter(
+    route => !route.startsWith('blog')
+  );
+
+  const availablePagesMetadata = routesExceptBlog.map(async pathname => {
+    const { source, filename } = await dynamicRouter.getMarkdownFile(
+      defaultLocale.code,
+      pathname
+    );
+
+    // Gets the title and the Description from the Page Metadata
+    const { title, description } = await dynamicRouter.getPageMetadata(
+      defaultLocale.code,
+      pathname
+    );
 
-      // Gets the title and the Description from the Page Metadata
-      const { title, description } = await dynamicRouter.getPageMetadata(
-        defaultLocale.code,
-        pathname
-      );
+    // Parser the Markdown source with `gray-matter` and then only
+    // grabs the markdown content and cleanses it by removing HTML/JSX tags
+    // removing empty/blank lines or lines just with spaces and trims each line
+    // from leading and trailing paddings/spaces
+    const cleanedContent = parseRichTextIntoPlainText(matter(source).content);
 
-      // Parser the Markdown source with `gray-matter` and then only
-      // grabs the markdown content and cleanses it by removing HTML/JSX tags
-      // removing empty/blank lines or lines just with spaces and trims each line
-      // from leading and trailing paddings/spaces
-      const cleanedContent = parseRichTextIntoPlainText(matter(source).content);
+    // Deflates a String into a base64 string-encoded (zlib compressed)
+    const content = deflateSync(cleanedContent).toString('base64');
 
-      // Deflates a String into a base64 string-encoded (zlib compressed)
-      const content = deflateSync(cleanedContent).toString('base64');
+    // Returns metadata of each page available on the Website
+    return {
+      filename,
+      pathname,
+      title,
+      description,
+      content,
+    };
+  });
 
-      // Returns metadata of each page available on the Website
-      return {
-        filename,
-        pathname,
-        title,
-        description,
-        content,
-      };
-    });
+  const data = await Promise.all(availablePagesMetadata);
 
-  return Response.json(await Promise.all(availablePagesMetadata));
+  return Response.json(data);
 };
 
 // This function generates the static paths that come from the dynamic segments