From 0630a14d99f65561f95e2bcf1e2883789eb2e260 Mon Sep 17 00:00:00 2001 From: himaniraghav3 Date: Wed, 18 Dec 2024 16:22:25 +0530 Subject: [PATCH 1/2] PM-218 Fix DOS issues with fs --- config/env.js | 2 +- config/paths.js | 2 +- config/webpack.config.js | 5 +---- scripts/build.js | 2 +- scripts/start.js | 2 +- 5 files changed, 5 insertions(+), 8 deletions(-) diff --git a/config/env.js b/config/env.js index afe5d437..fba3ab29 100644 --- a/config/env.js +++ b/config/env.js @@ -49,7 +49,7 @@ dotenvFiles.forEach(dotenvFile => { // Otherwise, we risk importing Node.js core modules into an app instead of Webpack shims. // https://github.com/facebook/create-react-app/issues/1023#issuecomment-265344421 // We also resolve them to make sure all tools using them work consistently. -const appDirectory = fs.realpathSync(process.cwd()) +const appDirectory = process.cwd() process.env.NODE_PATH = (process.env.NODE_PATH || '') .split(path.delimiter) .filter(folder => folder && !path.isAbsolute(folder)) diff --git a/config/paths.js b/config/paths.js index dbd6246b..a55133df 100644 --- a/config/paths.js +++ b/config/paths.js @@ -6,7 +6,7 @@ const url = require('url') // Make sure any symlinks in the project folder are resolved: // https://github.com/facebook/create-react-app/issues/637 -const appDirectory = fs.realpathSync(process.cwd()) +const appDirectory = process.cwd() const resolveApp = relativePath => path.resolve(appDirectory, relativePath) const envPublicUrl = process.env.PUBLIC_URL diff --git a/config/webpack.config.js b/config/webpack.config.js index d77bd35c..c5d2c025 100644 --- a/config/webpack.config.js +++ b/config/webpack.config.js @@ -27,9 +27,6 @@ const shouldUseSourceMap = process.env.GENERATE_SOURCEMAP !== 'false' // makes for a smoother build process. const shouldInlineRuntimeChunk = process.env.INLINE_RUNTIME_CHUNK !== 'false' -// Check if TypeScript is setup -const useTypeScript = fs.existsSync(paths.appTsConfig) - // style files regexes const cssRegex = /\.css$/ const cssModuleRegex = /\.module\.css$/ @@ -257,7 +254,7 @@ module.exports = function (webpackEnv) { // for React Native Web. extensions: paths.moduleFileExtensions .map(ext => `.${ext}`) - .filter(ext => useTypeScript || !ext.includes('ts')), + .filter(ext => !ext.includes('ts')), alias: { // Support React Native Web // https://www.smashingmagazine.com/2016/08/a-glimpse-into-the-future-with-react-native-for-web/ diff --git a/scripts/build.js b/scripts/build.js index eef55b65..60909f24 100644 --- a/scripts/build.js +++ b/scripts/build.js @@ -28,7 +28,7 @@ const printBuildError = require('react-dev-utils/printBuildError') const measureFileSizesBeforeBuild = FileSizeReporter.measureFileSizesBeforeBuild const printFileSizesAfterBuild = FileSizeReporter.printFileSizesAfterBuild -const useYarn = fs.existsSync(paths.yarnLockFile) +const useYarn = false // These sizes are pretty large. We'll warn for bundles exceeding them. const WARN_AFTER_BUNDLE_GZIP_SIZE = 512 * 1024 diff --git a/scripts/start.js b/scripts/start.js index 71d668f7..47f9f854 100644 --- a/scripts/start.js +++ b/scripts/start.js @@ -32,7 +32,7 @@ const paths = require('../config/paths') const configFactory = require('../config/webpack.config') const createDevServerConfig = require('../config/webpackDevServer.config') -const useYarn = fs.existsSync(paths.yarnLockFile) +const useYarn = true const isInteractive = process.stdout.isTTY // Warn and crash if required files are missing From afe1c7cd507a26463e42ee4905520f6c55156598 Mon Sep 17 00:00:00 2001 From: himaniraghav3 Date: Wed, 18 Dec 2024 16:25:23 +0530 Subject: [PATCH 2/2] Typo --- scripts/start.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/start.js b/scripts/start.js index 47f9f854..57864185 100644 --- a/scripts/start.js +++ b/scripts/start.js @@ -32,7 +32,7 @@ const paths = require('../config/paths') const configFactory = require('../config/webpack.config') const createDevServerConfig = require('../config/webpackDevServer.config') -const useYarn = true +const useYarn = false const isInteractive = process.stdout.isTTY // Warn and crash if required files are missing