feat: provided some file structural changes (nodejs#5464)

Author: ovflowd

Diff for: .husky/pre-commit

@@ -3,15 +3,9 @@
 
 DIR=$(cd `dirname $0` && pwd -P)
 
-# resets the node release data file
-echo "[]" > $DIR/../public/node-releases-data.json
-
-# resets the blog post data file
-echo "{ \"pagination\": [], \"categories\": [], \"posts\": [] }" > $DIR/../public/blog-posts-data.json
-
 # adds these changes to be part of the current commit
-git add --sparse $DIR/../public/node-releases-data.json
-git add --sparse $DIR/../public/blog-posts-data.json
+git checkout $DIR/../public/node-releases-data.json
+git checkout $DIR/../public/blog-posts-data.json
 
 # lint and format staged files
 npx lint-staged

Diff for: .storybook/test-runner.ts

@@ -1,7 +1,6 @@
-// TODO: Convert this to ESM once the bug is resolved
-// https://github.com/storybookjs/test-runner/issues/293
+import type { TestRunnerConfig } from '@storybook/test-runner';
 
-const config = {
+const config: TestRunnerConfig = {
   async postRender(page, _context) {
     const rootElementId = '[data-test-id="story-root"]';
     const rootElement = await page.locator(rootElementId);
@@ -11,4 +10,4 @@ const config = {
   },
 };
 
-module.exports = config;
+export default config;

Diff for: app/en/feed/[feed]/route.ts

@@ -1,6 +1,6 @@
 import { NextResponse } from 'next/server';
 import * as nextJson from '@/next.json.mjs';
-import * as nextData from '@/next-data/index.mjs';
+import * as nextData from '@/next.data.mjs';
 
 // loads all the data from the blog-posts-data.json file
 const websiteFeeds = nextData.generateWebsiteFeeds(nextJson.blogData);

Diff for: components/Article/Codebox/__tests__/index.test.tsx

@@ -4,12 +4,6 @@ import { IntlProvider } from 'react-intl';
 
 import Codebox, { replaceLabelLanguages, replaceLanguages } from '../index';
 
-jest.mock('isomorphic-dompurify', () => {
-  return {
-    sanitize: jest.fn().mockImplementation(source => source),
-  };
-});
-
 describe('Replacer tests', (): void => {
   it('replaceLabelLanguages', (): void => {
     expect(replaceLabelLanguages('language-console')).toBe('language-bash');

Diff for: components/Article/Codebox/index.tsx

@@ -1,7 +1,6 @@
 import { useEffect, useState } from 'react';
 import { FormattedMessage } from 'react-intl';
 import { highlight, languages } from 'prismjs';
-import { sanitize } from 'isomorphic-dompurify';
 import classnames from 'classnames';
 import styles from './index.module.scss';
 import { useCopyToClipboard } from '../../../hooks/useCopyToClipboard';
@@ -46,7 +45,7 @@ const Codebox: FC<CodeBoxProps> = ({ children: { props } }) => {
     const prismLanguage = languages[parsedLanguage] || languages.text;
 
     setParsedCode(
-      sanitize(highlight(codeArray[langIndex], prismLanguage, parsedLanguage))
+      highlight(codeArray[langIndex], prismLanguage, parsedLanguage)
     );
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [langIndex]);

Diff for: components/Common/Banner/__snapshots__/index.stories.tsx.snap

@@ -19,8 +19,8 @@ exports[`Common/Banner WithHTMLImage smoke-test 1`] = `
      target="_blank"
      rel="noopener noreferrer"
   >
-    <img alt="Banner Image"
-         src="/static/images/nodejs-training.png"
+    <img src="/static/images/nodejs-training.png"
+         alt="Banner Image"
     >
   </a>
 </div>

Diff for: components/Common/Banner/__tests__/index.test.tsx

@@ -3,10 +3,6 @@ import { IntlProvider } from 'react-intl';
 import Banner from '../index';
 import type { WebsiteBanner } from '../../../../types';
 
-jest.mock('isomorphic-dompurify', () => ({
-  sanitize: jest.fn((html: string) => html),
-}));
-
 const bannersIndex: WebsiteBanner = {
   endDate: '',
   link: 'test/banner/link',

Diff for: components/Common/Banner/index.tsx

@@ -1,6 +1,5 @@
 import { useMemo } from 'react';
 import { useIntl } from 'react-intl';
-import { sanitize } from 'isomorphic-dompurify';
 import styles from './index.module.scss';
 import { dateIsBetween } from '../../../util/dateIsBetween';
 import { isAbsoluteUrl } from '../../../util/isAbsoluteUrl';
@@ -36,7 +35,7 @@ const useHtmlContent = ({ html, link }: WebsiteBanner) =>
           href={link}
           target="_blank"
           rel="noopener noreferrer"
-          dangerouslySetInnerHTML={{ __html: sanitize(html) }}
+          dangerouslySetInnerHTML={{ __html: html }}
         />
       );
     }

Diff for: layouts/DownloadReleasesLayout.tsx

@@ -1,5 +1,4 @@
 import { useMemo } from 'react';
-import { sanitize } from 'isomorphic-dompurify';
 import BaseLayout from './BaseLayout';
 import { useLayoutContext } from '../hooks/useLayoutContext';
 import DownloadReleasesTable from '../components/Downloads/DownloadReleasesTable';
@@ -14,7 +13,7 @@ const DownloadReleasesLayout: FC<PropsWithChildren> = ({ children }) => {
   // @TODO: Remove this once we migrate to `nodejs/nodejs.dev` codebase as this is unsafe
   // And completely not recommended
   const extraModulesContent = useMemo(
-    () => sanitize(`[<a href="#backref-1">1</a>]: ${modules}`),
+    () => `[<a href="#backref-1">1</a>]: ${modules}`,
     [modules]
   );
 

Diff for: next-data/generateBlogPostsData.mjs

@@ -1,9 +1,9 @@
 'use strict';
 
 import { readFile, writeFile } from 'node:fs/promises';
-import { basename, dirname, extname, join } from 'node:path';
+import { basename, extname, join } from 'node:path';
 import graymatter from 'gray-matter';
-import * as helpers from './helpers.mjs';
+import * as nextHelpers from '../next.helpers.mjs';
 
 // gets the current blog path based on local module path
 const blogPath = join(process.cwd(), 'pages/en/blog');
@@ -51,7 +51,7 @@ const getFrontMatter = (filename, source) => {
  */
 const generateBlogPostsData = async () => {
   // we retrieve all the filenames of all blog posts
-  const filenames = await helpers.getMarkdownFiles(
+  const filenames = await nextHelpers.getMarkdownFiles(
     process.cwd(),
     'pages/en/blog',
     ['**/index.md', '**/pagination.md']

Diff for: next-data/index.mjs

@@ -13,9 +13,15 @@ const nextConfig = {
   distDir: nextConstants.ENABLE_STATIC_EXPORT ? 'build' : '.next',
   output: nextConstants.ENABLE_STATIC_EXPORT ? 'export' : undefined,
   experimental: {
+    swcMinify: true,
+    legacyBrowsers: false,
     nextScriptWorkers: true,
+    webpackBuildWorker: true,
     largePageDataBytes: 128 * 100000,
     swcPlugins: [['next-superjson-plugin', {}]],
+    outputFileTracingExcludes: {
+      '*': ['./public/**', 'node_modules/**/@swc/core*'],
+    },
   },
 };
 

Diff for: next.config.mjs

@@ -0,0 +1,11 @@
+'use strict';
+
+import generateWebsiteFeeds from './next-data/generateWebsiteFeeds.mjs';
+import generateBlogPostsData from './next-data/generateBlogPostsData.mjs';
+import generateNodeReleasesJson from './next-data/generateNodeReleasesJson.mjs';
+
+export {
+  generateWebsiteFeeds,
+  generateBlogPostsData,
+  generateNodeReleasesJson,
+};

Diff for: next.data.mjs

@@ -6,14 +6,13 @@ import remarkGfm from 'remark-gfm';
 import { serialize } from 'next-mdx-remote/serialize';
 import * as nextLocales from './next.locales.mjs';
 import * as nextConstants from './next.constants.mjs';
-import * as nextData from './next-data/index.mjs';
+import * as nextHelpers from './next.helpers.mjs';
 
 // allows us to run a glob to get markdown files based on a language folder
 const getPathsByLanguage = async (
   locale = nextConstants.DEFAULT_LOCALE_CODE,
   ignored = []
-) =>
-  nextData.helpers.getMarkdownFiles(process.cwd(), `pages/${locale}`, ignored);
+) => nextHelpers.getMarkdownFiles(process.cwd(), `pages/${locale}`, ignored);
 
 /**
  * This method is responsible for generating a Collection of all available paths that

Diff for: next.dynamic.mjs

@@ -1,6 +1,5 @@
 'use strict';
 
-import { existsSync } from 'node:fs';
 import { fileURLToPath } from 'node:url';
 import { glob } from 'glob';