Merge pull request #48532 from callstack-internal/hur/chore/fix-secur…

…ity-vulnerabilities chore: fix security vulnerabilities
Expensify · Sep 17, 2024 · 266ad46 · 266ad46
2 parents 2190f62 + aa5ddab
commit 266ad46
Show file tree

Hide file tree

Showing 10 changed files with 1,673 additions and 4,558 deletions.
diff --git a/.github/actions/javascript/authorChecklist/index.js b/.github/actions/javascript/authorChecklist/index.js
diff --git a/.storybook/webpack.config.ts b/.storybook/webpack.config.ts
@@ -94,6 +94,11 @@ const webpackConfig = ({config}: {config: Configuration}) => {
         loader: require.resolve('@svgr/webpack'),
     });
 
+    config.module.rules?.push({
+        test: /pdf\.worker\.mjs$/,
+        type: 'asset/source',
+    });
+
     config.plugins.push(
         new DefinePlugin({
             __DEV__: process.env.NODE_ENV === 'development',

diff --git a/babel.config.js b/babel.config.js
@@ -17,9 +17,9 @@ const defaultPlugins = [
     '@babel/transform-runtime',
     '@babel/plugin-proposal-class-properties',
 
-    // We use `transform-class-properties` for transforming ReactNative libraries and do not use it for our own
+    // We use `@babel/plugin-transform-class-properties` for transforming ReactNative libraries and do not use it for our own
     // source code transformation as we do not use class property assignment.
-    'transform-class-properties',
+    '@babel/plugin-transform-class-properties',
 
     // Keep it last
     'react-native-reanimated/plugin',

diff --git a/config/webpack/webpack.common.ts b/config/webpack/webpack.common.ts
@@ -123,7 +123,7 @@ const getCommonConfiguration = ({file = '.env', platform = 'web'}: Environment):
                 {from: 'node_modules/pdfjs-dist/cmaps/', to: 'cmaps/'},
             ],
         }),
-        new EnvironmentPlugin({JEST_WORKER_ID: null}),
+        new EnvironmentPlugin({JEST_WORKER_ID: ''}),
         new IgnorePlugin({
             resourceRegExp: /^\.\/locale$/,
             contextRegExp: /moment$/,
@@ -165,7 +165,7 @@ const getCommonConfiguration = ({file = '.env', platform = 'web'}: Environment):
             // We are importing this worker as a string by using asset/source otherwise it will default to loading via an HTTPS request later.
             // This causes issues if we have gone offline before the pdfjs web worker is set up as we won't be able to load it from the server.
             {
-                test: new RegExp('node_modules/pdfjs-dist/legacy/build/pdf.worker.js'),
+                test: new RegExp('node_modules/pdfjs-dist/legacy/build/pdf.worker.mjs'),
                 type: 'asset/source',
             },