fixup!

Author: avivkeller

apps/site/app/[locale]/download/archive/[version]/page.tsx

@@ -1,12 +1,11 @@
-// eslint-disable-next-line @typescript-eslint/ban-ts-comment
-// @ts-nocheck
-// TODO(@AVIVKELLER): THIS FILE
 import { defaultLocale } from '@node-core/website-i18n';
 import { notFound, redirect } from 'next/navigation';
 
 import provideReleaseData from '#site/next-data/providers/releaseData';
 import provideReleaseVersions from '#site/next-data/providers/releaseVersions';
 import { ENABLE_STATIC_EXPORT } from '#site/next.constants.mjs';
+import { getMarkdownFile } from '#site/router';
+import { renderPage } from '#site/router/render';
 
 import type { DynamicParams } from '#site/types';
 import type { FC } from 'react';
@@ -37,10 +36,7 @@ export const generateStaticParams = async () => {
 // finally it returns (if the locale and route are valid) the React Component with the relevant context
 // and attached context providers for rendering the current page
 const getPage: FC<PageParams> = async props => {
-  const { version, locale: routeLocale } = await props.params;
-
-  // Gets the current full pathname for a given path
-  const [locale, pathname] = basePage.getLocaleAndPath(version, routeLocale);
+  const { version, locale } = await props.params;
 
   if (version === 'current') {
     const releaseData = await provideReleaseData();
@@ -53,22 +49,15 @@ const getPage: FC<PageParams> = async props => {
   const versions = await provideReleaseVersions();
 
   // Verifies if the current route is a dynamic route
-  const isDynamicRoute = versions.some(r => r.includes(pathname));
-
-  // Gets the Markdown content and context for Download Archive pages
-  const [content, context] = await basePage.getMarkdownContext({
-    locale,
-    pathname: 'download/archive',
-  });
+  const isDynamicRoute = versions.some(r => r.includes(version));
 
   // If this isn't a valid dynamic route for archive version or there's no markdown
   //  file for this, then we fail as not found as there's nothing we can do.
-  if (isDynamicRoute && context.filename) {
-    return basePage.renderPage({
-      content,
-      layout: context.frontmatter.layout!,
-      context: { ...context, pathname: `/download/archive/${pathname}` },
-    });
+  if (isDynamicRoute) {
+    const markdown = (await getMarkdownFile(locale, 'download/archive'))!;
+    markdown.pathname = `/download/archive/${version}`;
+
+    return renderPage(markdown);
   }
 
   return notFound();

apps/site/next.config.mjs

@@ -23,8 +23,8 @@ const getDeploymentId = async () => {
 
 /** @type {import('next').NextConfig} */
 const nextConfig = {
-  // Configure `pageExtensions` to include markdown and MDX files
-  pageExtensions: ['md', 'mdx', 'tsx'],
+  // We need to include _all_ our page extensions here
+  pageExtensions: ['md', 'mdx', 'ts', 'tsx'],
   // Full Support of React 18 SSR and Streaming
   reactCompiler: true,
   // We don't want to redirect with trailing slashes

packages/content/eslint.config.js

@@ -4,21 +4,13 @@ import baseConfig from '../../eslint.config.js';
 
 export default baseConfig.concat([
   {
-    ignores: ['content/en/blog/**/*.{md,mdx}/**'],
+    ignores: ['src/en/blog/**/*.{md,mdx}/**'],
   },
 
-  mdx.flatCodeBlocks,
-
-  // Type-checking
   {
-    ignores: ['**/*.{md,mdx}', '**/*.{md,mdx}/**'],
-    languageOptions: {
-      parserOptions: {
-        tsconfigRootDir: import.meta.dirname,
-      },
-    },
+    ...mdx.flatCodeBlocks,
     rules: {
-      '@typescript-eslint/consistent-type-imports': 'error',
+      ...mdx.flatCodeBlocks.rules,
       '@typescript-eslint/no-require-imports': 'off',
     },
   },