[infra] Replace fs-extra with node:fs calls (#19148)

Author: bernardobelchior

docs/scripts/buildServiceWorker.js

@@ -1,18 +1,19 @@
 /* eslint-disable no-console */
 const path = require('path');
 const fs = require('fs/promises');
-const fse = require('fs-extra');
 
 async function prepend(file, string) {
   const data = await fs.readFile(file, 'utf8');
   await fs.writeFile(file, string + data, 'utf8');
 }
 
 async function run() {
-  const swDest = path.join(__dirname, '../export/x/sw.js');
+  const swDestDir = path.join(__dirname, '../export/x');
+  const swDest = path.join(swDestDir, 'sw.js');
   const swSrc = path.join(__dirname, '../src/sw.js');
 
-  await fse.copy(swSrc, swDest);
+  await fs.mkdir(swDestDir, { recursive: true });
+  await fs.copyFile(swSrc, swDest);
   await prepend(
     swDest,
     `

docs/scripts/formattedTSDemos.js

@@ -11,7 +11,6 @@
  */
 const ignoreList = ['/pages.ts', 'styling.ts', 'styling.tsx', 'types.ts'];
 
-const fse = require('fs-extra');
 const fs = require('fs');
 const path = require('path');
 const babel = require('@babel/core');
@@ -91,14 +90,16 @@ const previewOverride = {
 async function transpileFile(tsxPath, program, ignoreCache = false) {
   const jsPath = tsxPath.replace(/\.tsx?$/, '.js');
   try {
-    if (!ignoreCache && (await fse.exists(jsPath))) {
-      const [jsStat, tsxStat] = await Promise.all([
-        fs.promises.stat(jsPath),
-        fs.promises.stat(tsxPath),
-      ]);
-      if (jsStat.mtimeMs > tsxStat.mtimeMs) {
-        // JavaScript version is newer, skip transpiling
-        return TranspileResult.Skipped;
+    if (!ignoreCache) {
+      const ignoreNotFound = (err) => (err.code === 'ENOENT' ? null : Promise.reject(err));
+      const jsStat = await fs.promises.stat(jsPath).catch(ignoreNotFound);
+
+      if (jsStat != null) {
+        const tsxStat = await fs.promises.stat(tsxPath);
+        if (jsStat.mtimeMs > tsxStat.mtimeMs) {
+          // JavaScript version is newer, skip transpiling
+          return TranspileResult.Skipped;
+        }
       }
     }
 
@@ -202,7 +203,7 @@ async function main(argv) {
   }
 
   tsxFiles.forEach((filePath) => {
-    fs.promises.watchFile(filePath, { interval: 500 }, async () => {
+    fs.watchFile(filePath, { interval: 500 }, async () => {
       if ((await transpileFile(filePath, program, true)) === 0) {
         console.log('Success - %s', filePath);
       }

package.json

@@ -142,7 +142,6 @@
     "eslint-plugin-mui-x": "workspace:^",
     "fast-glob": "catalog:",
     "format-util": "^1.0.5",
-    "fs-extra": "^11.3.1",
     "glob-gitignore": "^1.0.15",
     "globby": "^14.1.0",
     "html-webpack-plugin": "^5.6.3",

pnpm-lock.yaml

@@ -1,5 +1,5 @@
 import glob from 'fast-glob';
-import fse from 'fs-extra';
+import fs from 'node:fs/promises';
 
 async function run() {
   const importFiles = [
@@ -15,11 +15,14 @@ async function run() {
     const newFiles = glob.sync(`node_modules/@mui/monorepo/${pattern}`);
     return acc.concat(newFiles);
   }, []);
-  files.forEach((file) => {
-    fse.copySync(file, file.replace('node_modules/@mui/monorepo/docs/', 'docs/'));
+
+  const copyPromises = files.map(async (file) => {
+    await fs.copyFile(file, file.replace('node_modules/@mui/monorepo/docs/', 'docs/'));
     // eslint-disable-next-line no-console
     console.log(`copy ${file}`);
   });
+
+  await Promise.all(copyPromises);
 }
 
 run();

scripts/importDocsStatic.mjs

@@ -1,8 +1,8 @@
-import * as fse from 'fs-extra';
 import * as path from 'path';
 import * as childProcess from 'child_process';
 import { type Browser, chromium } from '@playwright/test';
 import { major } from '@mui/material/version';
+import fs from 'node:fs/promises';
 
 const isMaterialUIv6 = major === 6;
 const isMaterialUIv7 = major === 7;
@@ -69,7 +69,7 @@ async function main() {
   routes = routes.map((route) => route.replace(baseUrl, ''));
 
   // prepare screenshots
-  await fse.emptyDir(screenshotDir);
+  await emptyDir(screenshotDir);
 
   async function navigateToTest(route: string) {
     // Use client-side routing which is much faster than full page navigation via page.goto().
@@ -138,7 +138,6 @@ async function main() {
           }
 
           const screenshotPath = path.resolve(screenshotDir, `.${route}.png`);
-          await fse.ensureDir(path.dirname(screenshotPath));
 
           const testcase = await page.waitForSelector(
             `[data-testid="testcase"][data-testpath="${route}"]:not([aria-busy="true"])`,
@@ -189,7 +188,6 @@ async function main() {
     it('should position the headers matching the columns', async () => {
       const route = '/docs-data-grid-virtualization/ColumnVirtualizationGrid';
       const screenshotPath = path.resolve(screenshotDir, `.${route}ScrollLeft400px.png`);
-      await fse.ensureDir(path.dirname(screenshotPath));
 
       await navigateToTest(route);
 
@@ -216,9 +214,6 @@ async function main() {
     it('should position charts axis tooltip 8px away from the pointer', async () => {
       const route = '/docs-charts-tooltip/Interaction';
       const axisScreenshotPath = path.resolve(screenshotDir, `.${route}AxisTooltip.png`);
-      const itemScreenshotPath = path.resolve(screenshotDir, `.${route}ItemTooltip.png`);
-      await fse.ensureDir(path.dirname(axisScreenshotPath));
-      await fse.ensureDir(path.dirname(itemScreenshotPath));
 
       await navigateToTest(route);
 
@@ -243,7 +238,6 @@ async function main() {
     it('should export a chart as PNG', async () => {
       const route = '/docs-charts-export/ExportChartAsImage';
       const screenshotPath = path.resolve(screenshotDir, `.${route}PNG.png`);
-      await fse.ensureDir(path.dirname(screenshotPath));
 
       await navigateToTest(route);
 
@@ -274,7 +268,6 @@ async function main() {
       it('should take a screenshot of the data grid print preview', async () => {
         const route = '/docs-data-grid-export/ExportDefaultToolbar';
         const screenshotPath = path.resolve(screenshotDir, `.${route}Print.png`);
-        await fse.ensureDir(path.dirname(screenshotPath));
 
         await navigateToTest(route);
 
@@ -302,7 +295,6 @@ async function main() {
       it('should take a screenshot of the charts print preview', async () => {
         const route = '/docs-charts-export/PrintChart';
         const screenshotPath = path.resolve(screenshotDir, `.${route}Print.png`);
-        await fse.ensureDir(path.dirname(screenshotPath));
 
         await navigateToTest(route);
 
@@ -428,3 +420,14 @@ async function newTestPage(browser: Browser) {
 
   return page;
 }
+
+async function emptyDir(dir: string) {
+  let items;
+  try {
+    items = await fs.readdir(dir);
+  } catch {
+    return fs.mkdir(dir, { recursive: true });
+  }
+
+  return Promise.all(items.map((item) => fs.rm(path.join(dir, item), { recursive: true })));
+}