@salesforce/pwa-kit-dev 3.9.2 → 3.10.0-dev

package/configs/webpack/config.js

@@ -45,6 +45,9 @@ const INSPECT = process.execArgv.some(arg => /^--inspect(?:-brk)?(?:$|=)/.test(a
 const DEBUG = mode !== production && process.env.DEBUG === 'true';
 const CI = process.env.CI;
 const disableHMR = process.env.HMR === 'false';
+if (process.env.PWA_KIT_SSR_SOURCE_MAP) {
+  console.warn('PWA_KIT_SSR_SOURCE_MAP env variable will be deprecated for source-map in the next major release. Please use PWA_KIT_SOURCE_MAP');
+}
 if ([production, development].indexOf(mode) < 0) {
   throw new Error(`Invalid mode "${mode}"`);
 }
@@ -309,7 +312,7 @@ const client = entryPointExists(['app', 'main']) && baseConfig('web').extend(wit
     // Must be named "client". See - https://www.npmjs.com/package/webpack-hot-server-middleware#usage
     name: _configNames.CLIENT,
     // use source map to make debugging easier
-    devtool: mode === development ? 'source-map' : false,
+    devtool: mode === development || process.env.PWA_KIT_SOURCE_MAP === 'true' ? 'source-map' : false,
     entry: {
       main: getAppEntryPoint()
     },
@@ -332,7 +335,7 @@ const clientOptional = baseConfig('web').extend(config => {
     name: _configNames.CLIENT_OPTIONAL,
     entry: _objectSpread(_objectSpread(_objectSpread(_objectSpread({}, optional('loader', (0, _path.resolve)(projectDir, EXT_OVERRIDES_DIR, 'app', 'loader.js'))), optional('worker', (0, _path.resolve)(projectDir, 'worker', 'main.js'))), optional('core-polyfill', (0, _path.resolve)(projectDir, 'node_modules', 'core-js'))), optional('fetch-polyfill', (0, _path.resolve)(projectDir, 'node_modules', 'whatwg-fetch'))),
     // use source map to make debugging easier
-    devtool: mode === development ? 'source-map' : false,
+    devtool: mode === development || process.env.PWA_KIT_SOURCE_MAP === 'true' ? 'source-map' : false,
     plugins: [...config.plugins, analyzeBundle && getBundleAnalyzerPlugin(_configNames.CLIENT_OPTIONAL)].filter(Boolean)
   });
 }).build();
@@ -342,7 +345,7 @@ const renderer = _fsExtra.default.existsSync((0, _path.resolve)(projectDir, 'nod
     name: _configNames.SERVER,
     entry: '@salesforce/pwa-kit-react-sdk/ssr/server/react-rendering.js',
     // use eval-source-map for server-side debugging
-    devtool: mode === development && INSPECT ? 'eval-source-map' : false,
+    devtool: mode === development && INSPECT || process.env.PWA_KIT_SOURCE_MAP === 'true' ? 'eval-source-map' : false,
     output: {
       path: buildDir,
       // We want to split the build on local development to reduce memory usage.
@@ -364,7 +367,7 @@ const ssr = (() => {
   // Only compile the ssr file when we're building for prod.
   if (mode === production) {
     return baseConfig('node').extend(config => {
-      return _objectSpread(_objectSpread(_objectSpread({}, config), process.env.PWA_KIT_SSR_SOURCE_MAP === 'true' ? {
+      return _objectSpread(_objectSpread(_objectSpread({}, config), process.env.PWA_KIT_SSR_SOURCE_MAP === 'true' || process.env.PWA_KIT_SOURCE_MAP === 'true' ? {
         devtool: 'source-map'
       } : {}), {}, {
         // Must *not* be named "server". See - https://www.npmjs.com/package/webpack-hot-server-middleware#usage
@@ -393,7 +396,7 @@ const requestProcessor = entryPointExists(['app', 'request-processor']) && baseC
       libraryTarget: 'commonjs2'
     },
     // use eval-source-map for server-side debugging
-    devtool: mode === development && INSPECT ? 'eval-source-map' : false,
+    devtool: mode === development && INSPECT || process.env.PWA_KIT_SOURCE_MAP === 'true' ? 'eval-source-map' : false,
     plugins: [...config.plugins, analyzeBundle && getBundleAnalyzerPlugin(_configNames.REQUEST_PROCESSOR)].filter(Boolean)
   });
 }).build();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@salesforce/pwa-kit-dev",
-  "version": "3.9.2",
+  "version": "3.10.0-dev",
   "description": "Build tools for pwa-kit",
   "homepage": "https://github.com/SalesforceCommerceCloud/pwa-kit/tree/develop/packages/pwa-kit-dev#readme",
   "bugs": {
@@ -58,14 +58,14 @@
     "@loadable/server": "^5.15.3",
     "@loadable/webpack-plugin": "^5.15.2",
     "@pmmmwh/react-refresh-webpack-plugin": "^0.5.10",
-    "@salesforce/pwa-kit-runtime": "3.9.2",
+    "@salesforce/pwa-kit-runtime": "3.10.0-dev",
     "@typescript-eslint/eslint-plugin": "^5.57.0",
     "@typescript-eslint/parser": "^5.57.0",
     "archiver": "1.3.0",
     "babel-jest": "^26.6.3",
     "babel-loader": "^8.3.0",
     "babel-plugin-dynamic-import-node-babel-7": "^2.0.7",
-    "babel-plugin-formatjs": "10.4.0",
+    "babel-plugin-formatjs": "10.5.36",
     "chalk": "^4.1.2",
     "commander": "^9.5.0",
     "compression": "1.7.4",
@@ -121,7 +121,7 @@
     "@types/node": "~16.0.3",
     "@types/node-fetch": "~2.6.3",
     "@types/validator": "~13.7.14",
-    "internal-lib-build": "3.9.2",
+    "internal-lib-build": "3.10.0-dev",
     "nock": "^13.3.0",
     "nodemon": "^2.0.22",
     "superagent": "^6.1.0",
@@ -147,5 +147,5 @@
   "publishConfig": {
     "directory": "dist"
   },
-  "gitHead": "d982971e3a1cc1f65ce296e526c459b2ea5f83fe"
+  "gitHead": "d86dc7bea3fe03a0f475ecf1a0ab878374f07026"
 }

package/ssr/server/build-dev-server.test.js

@@ -560,7 +560,7 @@ describe('DevServer rendering', () => {
 });
 describe('DevServer service worker', () => {
   let tmpDir;
-  beforeEach( /*#__PURE__*/_asyncToGenerator(function* () {
+  beforeEach(/*#__PURE__*/_asyncToGenerator(function* () {
     tmpDir = yield _fsExtra.default.mkdtemp(_path.default.join(_os.default.tmpdir(), 'pwa-kit-test-'));
   }));
   afterEach(() => {

package/utils/script-utils.js

@@ -83,7 +83,7 @@ const walkDir = exports.walkDir = /*#__PURE__*/function () {
     const entries = yield (0, _fsExtra.readdir)(dir, {
       withFileTypes: true
     });
-    yield Promise.all(entries.map( /*#__PURE__*/function () {
+    yield Promise.all(entries.map(/*#__PURE__*/function () {
       var _ref4 = _asyncToGenerator(function* (entry) {
         const entryPath = _path.default.join(dir, entry.name);
         if (entry.isDirectory()) {
@@ -375,7 +375,7 @@ const createBundle = exports.createBundle = /*#__PURE__*/function () {
       archive.on('error', reject);
       output.on('finish', resolve);
       archive.finalize();
-    })).then( /*#__PURE__*/_asyncToGenerator(function* () {
+    })).then(/*#__PURE__*/_asyncToGenerator(function* () {
       const {
         dependencies = {},
         devDependencies = {},

package/utils/script-utils.test.js

@@ -41,7 +41,7 @@ jest.mock('child_process', () => {
 describe('scriptUtils', () => {
   const originalEnv = process.env;
   let tmpDir;
-  beforeEach( /*#__PURE__*/_asyncToGenerator(function* () {
+  beforeEach(/*#__PURE__*/_asyncToGenerator(function* () {
     process.env = _objectSpread({}, originalEnv);
     tmpDir = yield (0, _fsExtra.mkdtemp)(_path.default.join(_os.default.tmpdir(), 'scriptUtils-tests'));
     // This is a workaround for jest.spyOn(), since I guess it doesn't work with our imports?
@@ -49,7 +49,7 @@ describe('scriptUtils', () => {
     _fsExtra.readJson.mockReset().mockImplementation(actualReadJson);
     _child_process.execSync.mockReset().mockImplementation(actualExecSync);
   }));
-  afterEach( /*#__PURE__*/_asyncToGenerator(function* () {
+  afterEach(/*#__PURE__*/_asyncToGenerator(function* () {
     process.env = originalEnv;
     tmpDir && (yield (0, _fsExtra.rm)(tmpDir, {
       recursive: true
@@ -250,14 +250,14 @@ describe('scriptUtils', () => {
     }));
     test('should throw', /*#__PURE__*/_asyncToGenerator(function* () {
       const thePath = _path.default.join(tmpDir, '.mobify.test');
-      yield expect( /*#__PURE__*/_asyncToGenerator(function* () {
+      yield expect(/*#__PURE__*/_asyncToGenerator(function* () {
         return yield scriptUtils.readCredentials(thePath);
       })).rejects.toThrow(Error);
     }));
   });
   describe('createBundle', () => {
     test('should throw if ssr_only and ssr_shared is empty', /*#__PURE__*/_asyncToGenerator(function* () {
-      yield expect( /*#__PURE__*/_asyncToGenerator(function* () {
+      yield expect(/*#__PURE__*/_asyncToGenerator(function* () {
         return yield scriptUtils.createBundle({
           message: null,
           ssr_parameters: {},
@@ -269,7 +269,7 @@ describe('scriptUtils', () => {
       })).rejects.toThrow('no ssrOnly or ssrShared files are defined');
     }));
     test('should throw buildDir does not exist', /*#__PURE__*/_asyncToGenerator(function* () {
-      yield expect( /*#__PURE__*/_asyncToGenerator(function* () {
+      yield expect(/*#__PURE__*/_asyncToGenerator(function* () {
         return yield scriptUtils.createBundle({
           message: null,
           ssr_parameters: {},
@@ -357,7 +357,7 @@ describe('scriptUtils', () => {
           text,
           json
         };
-        const fetchMock = jest.fn( /*#__PURE__*/_asyncToGenerator(function* () {
+        const fetchMock = jest.fn(/*#__PURE__*/_asyncToGenerator(function* () {
           return responseMock;
         }));
         const client = new scriptUtils.CloudAPIClient({
@@ -410,7 +410,7 @@ describe('scriptUtils', () => {
       const json = () => Promise.resolve({
         token: 'token-value'
       });
-      const fetchMock = jest.fn( /*#__PURE__*/_asyncToGenerator(function* () {
+      const fetchMock = jest.fn(/*#__PURE__*/_asyncToGenerator(function* () {
         return {
           status: 200,
           text,
@@ -474,8 +474,8 @@ describe('scriptUtils', () => {
   });
   describe('walkDir', () => {
     const files = ['a', 'b/1', 'b/2', 'c/d/e'].map(_path.default.normalize);
-    beforeEach( /*#__PURE__*/_asyncToGenerator(function* () {
-      yield Promise.all(files.map( /*#__PURE__*/function () {
+    beforeEach(/*#__PURE__*/_asyncToGenerator(function* () {
+      yield Promise.all(files.map(/*#__PURE__*/function () {
         var _ref38 = _asyncToGenerator(function* (file) {
           return yield (0, _fsExtra.createFile)(_path.default.join(tmpDir, file));
         });