fix writeRedirectFiles

packages/docusaurus-plugin-client-redirects/src/collectRedirects.ts

@@ -5,7 +5,6 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-import path from 'path';
 import {flatten, uniqBy} from 'lodash';
 import {
   RedirectsCreator,
@@ -13,8 +12,6 @@ import {
   RedirectMetadata,
   PluginOptions,
 } from './types';
-import createRedirectPageContent from './createRedirectPageContent';
-import {addTrailingSlash, getFilePathForRoutePath} from './utils';
 import {
   fromExtensionsRedirectCreator,
   toExtensionsRedirectCreator,

packages/docusaurus-plugin-client-redirects/src/index.ts

@@ -31,7 +31,7 @@ export default function pluginClientRedirectsPages(
       const redirects: RedirectMetadata[] = collectRedirects(pluginContext);
 
       // Write files only at the end: make code more easy to test without IO
-      await writeRedirectFiles(redirects);
+      await writeRedirectFiles(redirects, pluginContext);
     },
   };
 }

packages/docusaurus-plugin-client-redirects/src/writeRedirectFiles.ts

@@ -6,23 +6,58 @@
  */
 
 import fs from 'fs-extra';
-import {RedirectMetadata} from './types';
+import path from 'path';
+import {memoize} from 'lodash';
 
-type RedirectFile = Pick<
+import {PluginContext, RedirectMetadata} from './types';
-  RedirectMetadata,
+import createRedirectPageContent from './createRedirectPageContent';
-  'redirectAbsoluteFilePath' | 'redirectPageContent'
+import {addTrailingSlash, getFilePathForRoutePath} from './utils';
->;
 
-export default async function writeRedirectFiles(redirects: RedirectFile[]) {
+type FileMetadata = RedirectMetadata & {
-  async function writeRedirectFile(redirect: RedirectFile) {
+  fileAbsolutePath: string;
+  fileContent: string;
+};
+
+function toFileMetadata(
+  redirects: RedirectMetadata[],
+  pluginContext: PluginContext,
+): FileMetadata[] {
+  // Perf: avoid rendering the template twice with the exact same "props"
+  // We might create multiple redirect pages for the same destination url
+  // note: the first fn arg is the cache key!
+  const createPageContentMemoized = memoize((toUrl: string) => {
+    return createRedirectPageContent({toUrl});
+  });
+
+  return redirects.map((redirect) => {
+    const fileAbsolutePath = path.join(
+      pluginContext.outDir,
+      getFilePathForRoutePath(redirect.fromRoutePath),
+    );
+    const toUrl = addTrailingSlash(
+      `${pluginContext.baseUrl}${redirect.toRoutePath}`,
+    );
+    const fileContent = createPageContentMemoized(toUrl);
+    return {
+      ...redirect,
+      fileAbsolutePath,
+      fileContent,
+    };
+  });
+}
+
+export default async function writeRedirectFiles(
+  redirects: RedirectMetadata[],
+  pluginContext: PluginContext,
+) {
+  async function writeFile(file: FileMetadata) {
     try {
-      await fs.writeFile(
+      await fs.writeFile(file.fileAbsolutePath, file.fileContent);
-        redirect.redirectAbsoluteFilePath,
-        redirect.redirectPageContent,
-      );
     } catch (err) {
       throw new Error(`Redirect file creation error: ${err}`);
     }
   }
-  await Promise.all(redirects.map(writeRedirectFile));
+
+  const files = toFileMetadata(redirects, pluginContext);
+  await Promise.all(files.map(writeFile));
 }