From fb85b7a793c0b8a325f7ee836eeb316fbe4dd394 Mon Sep 17 00:00:00 2001
From: Olivier Tassinari <olivier.tassinari@gmail.com>
Date: Sun, 4 Jun 2023 19:16:44 +0200
Subject: [PATCH] [docs-infra] Mandatory versions

---
 docs/pages/_app.js                          | 33 +++++++++++++++++----
 docs/src/modules/components/AppNavDrawer.js | 30 +++++++++++--------
 docs/src/modules/utils/useRouterExtra.ts    | 18 +++++++++--
 packages/markdown/parseMarkdown.js          | 16 +++++-----
 4 files changed, 70 insertions(+), 27 deletions(-)

diff --git a/docs/pages/_app.js b/docs/pages/_app.js
index 0b326438ef7362..d8f218cd318987 100644
--- a/docs/pages/_app.js
+++ b/docs/pages/_app.js
@@ -151,8 +151,8 @@ function AppWrapper(props) {
 
     if (product === 'material-ui') {
       return {
-        name: 'Material UI',
         metadata: 'MUI Core',
+        name: 'Material UI',
         versions: [
           { text: `v${materialPkgJson.version}`, current: true },
           {
@@ -169,16 +169,16 @@ function AppWrapper(props) {
 
     if (product === 'joy-ui') {
       return {
-        name: 'Joy UI',
         metadata: 'MUI Core',
+        name: 'Joy UI',
         versions: [{ text: `v${joyPkgJson.version}`, current: true }],
       };
     }
 
     if (product === 'system') {
       return {
-        name: 'MUI System',
         metadata: 'MUI Core',
+        name: 'MUI System',
         versions: [
           { text: `v${systemPkgJson.version}`, current: true },
           { text: 'v4', href: `https://v4.mui.com${languagePrefix}/system/basics/` },
@@ -192,13 +192,36 @@ function AppWrapper(props) {
 
     if (product === 'base') {
       return {
-        name: 'Base UI',
         metadata: 'MUI Core',
+        name: 'Base UI',
         versions: [{ text: `v${basePkgJson.version}`, current: true }],
       };
     }
 
-    return null; // The identifier for X and Toolpad is handled by their own site
+    if (product === 'core') {
+      return {
+        metadata: '',
+        name: 'MUI Core',
+        versions: [
+          { text: `v${materialPkgJson.version}`, current: true },
+          {
+            text: 'View all versions',
+            href: `https://mui.com${languagePrefix}/versions/`,
+          },
+        ],
+      };
+    }
+
+    return {
+      metadata: '',
+      name: 'Docs-infra',
+      versions: [
+        {
+          text: 'v0.0.0',
+          href: `https://mui.com${languagePrefix}/versions/`,
+        },
+      ],
+    };
   }, [pageProps.userLanguage, product]);
 
   const pageContextValue = React.useMemo(() => {
diff --git a/docs/src/modules/components/AppNavDrawer.js b/docs/src/modules/components/AppNavDrawer.js
index 89776a4ea18529..8074fc6b5f904a 100644
--- a/docs/src/modules/components/AppNavDrawer.js
+++ b/docs/src/modules/components/AppNavDrawer.js
@@ -89,7 +89,8 @@ ProductDrawerButton.propTypes = {
   productName: PropTypes.string,
 };
 
-function ProductIdentifier({ name, metadata, versionSelector }) {
+function ProductIdentifier(props) {
+  const { name, metadata, versionSelector } = props;
   return (
     <Box sx={{ flexGrow: 1 }}>
       <Typography
@@ -114,8 +115,8 @@ function ProductIdentifier({ name, metadata, versionSelector }) {
 
 ProductIdentifier.propTypes = {
   metadata: PropTypes.string,
-  name: PropTypes.string,
-  versionSelector: PropTypes.element,
+  name: PropTypes.string.isRequired,
+  versionSelector: PropTypes.element.isRequired,
 };
 
 // To match scrollMarginBottom
@@ -401,15 +402,11 @@ export default function AppNavDrawer(props) {
               <SvgMuiLogo width={30} />
             </Box>
           </NextLink>
-          {productIdentifier && (
-            <ProductIdentifier
-              name={productIdentifier.name}
-              metadata={productIdentifier.metadata}
-              versionSelector={
-                productIdentifier.versions ? renderVersionSelector(productIdentifier.versions) : []
-              }
-            />
-          )}
+          <ProductIdentifier
+            name={productIdentifier.name}
+            metadata={productIdentifier.metadata}
+            versionSelector={renderVersionSelector(productIdentifier.versions)}
+          />
         </ToolbarDiv>
         <Divider
           sx={(theme) => ({
@@ -425,6 +422,15 @@ export default function AppNavDrawer(props) {
     );
   }, [onClose, pages, activePageParents, t, productIdentifier, anchorEl]);
 
+  if (process.env.NODE_ENV !== 'production') {
+    if (!productIdentifier) {
+      throw new Error('docs-infra: missing productIdentifier in PageContext');
+    }
+    if (!productIdentifier.versions) {
+      throw new Error('docs-infra: missing productIdentifier.versions in PageContext');
+    }
+  }
+
   return (
     <nav className={className} aria-label={t('mainNavigation')}>
       {disablePermanent || mobile ? (
diff --git a/docs/src/modules/utils/useRouterExtra.ts b/docs/src/modules/utils/useRouterExtra.ts
index 5702b85321964a..73205d7ed87bcf 100644
--- a/docs/src/modules/utils/useRouterExtra.ts
+++ b/docs/src/modules/utils/useRouterExtra.ts
@@ -4,14 +4,26 @@ export default function useRouterExtra() {
   const router = useRouter();
 
   const asPathWithoutLang = router.asPath.replace(/^\/+[a-zA-Z]{2}\//, '/');
-  let product = asPathWithoutLang.replace(/^\/+([^/]+)\/.*/, '$1');
+  const firstFolder = asPathWithoutLang.replace(/^\/+([^/]+)\/.*/, '$1');
+  let product = firstFolder;
   if (asPathWithoutLang.startsWith('/x/')) {
     product = asPathWithoutLang.replace('/x/react-', '').replace(/\/.*/, '');
   }
+
+  if (asPathWithoutLang === '/versions/') {
+    product = 'core';
+  }
+
   return {
     ...router,
     asPathWithoutLang,
-    product: product as 'base' | 'material-ui' | 'joy-ui' | 'system' | 'data-grid' | 'date-pickers',
-    isMuiX: asPathWithoutLang.startsWith('/x'),
+    product: product as
+      | 'core'
+      | 'base'
+      | 'material-ui'
+      | 'joy-ui'
+      | 'system'
+      | 'data-grid'
+      | 'date-pickers',
   };
 }
diff --git a/packages/markdown/parseMarkdown.js b/packages/markdown/parseMarkdown.js
index 2265e88f2ae101..30c6ac799d3140 100644
--- a/packages/markdown/parseMarkdown.js
+++ b/packages/markdown/parseMarkdown.js
@@ -55,7 +55,7 @@ function checkUrlHealth(href, linkText, context) {
   if (url.pathname[url.pathname.length - 1] !== '/') {
     throw new Error(
       [
-        'Missing trailing slash. The following link:',
+        'docs-infra: Missing trailing slash. The following link:',
         `[${linkText}](${href}) in ${context.location} is missing a trailing slash, please add it.`,
         '',
         'See https://ahrefs.com/blog/trailing-slash/ for more details.',
@@ -76,7 +76,7 @@ function checkUrlHealth(href, linkText, context) {
     if (href[0] !== '/') {
       throw new Error(
         [
-          'Missing leading slash. The following link:',
+          'docs-infra: Missing leading slash. The following link:',
           `[${linkText}](${href}) in ${context.location} is missing a leading slash, please add it.`,
           '',
         ].join('\n'),
@@ -150,7 +150,9 @@ function getHeaders(markdown) {
 
     return headers;
   } catch (err) {
-    throw new Error(`${err.message} in getHeader(markdown) with markdown: \n\n${header}`);
+    throw new Error(
+      `docs-infra: ${err.message} in getHeader(markdown) with markdown: \n\n${header}`,
+    );
   }
 }
 
@@ -274,7 +276,7 @@ function createRender(context) {
         });
       } else if (level === 3) {
         if (!toc[toc.length - 1]) {
-          throw new Error(`Missing parent level for: ${headingText}`);
+          throw new Error(`docs-infra: Missing parent level for: ${headingText}`);
         }
 
         toc[toc.length - 1].children.push({
@@ -449,13 +451,13 @@ function prepareMarkdown(config) {
       const description = headers.description || getDescription(markdown);
 
       if (title == null || title === '') {
-        throw new Error(`Missing title in the page: ${location}`);
+        throw new Error(`docs-infra: Missing title in the page: ${location}`);
       }
 
       if (title.length > 70) {
         throw new Error(
           [
-            `The title "${title}" is too long (${title.length} characters).`,
+            `docs-infra: The title "${title}" is too long (${title.length} characters).`,
             'It needs to have fewer than 70 characters—ideally less than 60. For more details, see:',
             'https://developers.google.com/search/docs/advanced/appearance/title-link',
           ].join('\n'),
@@ -463,7 +465,7 @@ function prepareMarkdown(config) {
       }
 
       if (description == null || description === '') {
-        throw new Error(`Missing description in the page: ${location}`);
+        throw new Error(`docs-infra: Missing description in the page: ${location}`);
       }
 
       const contents = getContents(markdown);