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

Author: ovflowd

.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

.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;

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);

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');

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]);

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>

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',

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 }}
         />
       );
     }

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]
   );
 

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']