@zohodesk/react-cli 1.1.7 → 1.1.8-exp.1

package/lib/configs/webpack.prod.config.js

@@ -44,7 +44,8 @@ const {
     selectorReplace,
     removePropTypes,
     devConsoleExculde,
-    crossorigin
+    crossorigin,
+    postCssPluginOrder
   }
 } = options;
 let {
@@ -160,7 +161,8 @@ module.exports = {
         mediaQueryHoverActiveString,
         classNameBlob: '[local]',
         cssUniqueness: false,
-        selectorReplace: null
+        selectorReplace: null,
+        postCssPluginOrder
       })
     } : null, {
       test: seperateCssModules ? /\.module\.css$/ : /\.css$/,
@@ -175,7 +177,8 @@ module.exports = {
         cssUniqueness,
         selectorReplace,
         cssHashSelectors,
-        classNamePrefix
+        classNamePrefix,
+        postCssPluginOrder
       })
     }, {
       test: /\.jpe?g$|\.gif$|\.png$/,

package/lib/deprecationLogger.js

@@ -0,0 +1,41 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.deprecateMessage = deprecateMessage;
+exports.deprecateOption = deprecateOption;
+exports.deprecationLoggerEnd = deprecationLoggerEnd;
+exports.deprecationLoggerStart = deprecationLoggerStart;
+
+var _logger = require("./logger");
+
+function deprecateMessage(message) {
+  (0, _logger.messageLogger)('\x1b[33m%s\x1b[0m', message);
+}
+
+function printLine() {
+  deprecateMessage('\n!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! \n ');
+}
+
+let isFirstDeprecation = false;
+
+function deprecateOption(previousObjPath = '', newObjPath = '', message = '') {
+  // eslint-disable-next-line no-use-before-define
+  deprecationLoggerStart();
+  deprecateMessage(`Please move "${previousObjPath.split('.').join(' > ')}" to "${newObjPath.split('.').join(' > ')}" following option ${message}`);
+}
+
+function deprecationLoggerStart() {
+  if (!isFirstDeprecation) {
+    isFirstDeprecation = true;
+    printLine();
+  }
+}
+
+function deprecationLoggerEnd() {
+  if (isFirstDeprecation) {
+    isFirstDeprecation = false;
+    printLine();
+  }
+}

package/lib/loaderUtils/configsAssetsLoaders.js

@@ -79,39 +79,39 @@ function createNameTemplate(enableChunkHash) {
   const ext = `${enableChunkHash ? '.[hash:20]' : ''}.[ext]`;
   return `[name]${ext}`;
 }
-/*
-export function createImageAndFontsAndSVGLoaders(enableChunkHash) {
-  const nameTemplate = createNameTemplate(enableChunkHash);
-  return [
-    configImageLoader(nameTemplate),
-    configFontLoader(nameTemplate),
-    configSVGLoader(nameTemplate),
-    configAudioLoader(nameTemplate)
-  ];
-}
+/*
+export function createImageAndFontsAndSVGLoaders(enableChunkHash) {
+  const nameTemplate = createNameTemplate(enableChunkHash);
+  return [
+    configImageLoader(nameTemplate),
+    configFontLoader(nameTemplate),
+    configSVGLoader(nameTemplate),
+    configAudioLoader(nameTemplate)
+  ];
+}
 */
 
-/*
-export function createLoaderOptionObject(
-  loaderName,
-  nameTemplate,
-  fallback,
-  limit = 1000
-) {
-  return {
-    loader: loaderName,
-    options: {
-      limit,
-      name: nameTemplate,
-      fallback
-    }
-  };
-}
-
-function configLoaderObject(filter, loaderAndOptions) {
-  return {
-    test: filter,
-    use: loaderAndOptions
-  };
-}
+/*
+export function createLoaderOptionObject(
+  loaderName,
+  nameTemplate,
+  fallback,
+  limit = 1000
+) {
+  return {
+    loader: loaderName,
+    options: {
+      limit,
+      name: nameTemplate,
+      fallback
+    }
+  };
+}
+
+function configLoaderObject(filter, loaderAndOptions) {
+  return {
+    test: filter,
+    use: loaderAndOptions
+  };
+}
 */

package/lib/loaderUtils/getCSSLoaders.js

@@ -18,6 +18,24 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 const fs = require('fs');
 
 const options = (0, _utils.getOptions)();
+const defaultPostCssPluginOrder = ['valueReplacer', 'selectorReplace', 'hasRTL', 'hoverActive', 'combinerMediaQuery', 'cssVariableReplacement', 'selectorWeight', 'minifier', 'composeMinification'];
+
+function calculatePostCssPluginOrder(postCssPluginOrder, pluginOrder) {
+  // if(typeof postCssPluginOrder === Boolean)
+  if (typeof postCssPluginOrder === 'boolean') {
+    if (postCssPluginOrder) {
+      return pluginOrder;
+    }
+
+    return defaultPostCssPluginOrder.filter(value => pluginOrder.includes(value));
+  }
+
+  if (Array.isArray(postCssPluginOrder)) {
+    return postCssPluginOrder;
+  }
+
+  return [];
+}
 
 const getCSSLoaders = optionsObj => {
   const {
@@ -29,7 +47,8 @@ const getCSSLoaders = optionsObj => {
     cssUniqueness,
     selectorReplace,
     cssHashSelectors,
-    classNamePrefix
+    classNamePrefix,
+    postCssPluginOrder
   } = optionsObj;
   const {
     devCssFileBountry
@@ -54,7 +73,7 @@ const getCSSLoaders = optionsObj => {
   } // console.log('selector weight config : ', selectorWeightConfig);
 
 
-  let postcssPlugins = [plugins.valueReplacer && require('../postcss-plugins/ValueReplacer')(valueReplacer), plugins.selectorReplace && require('../postcss-plugins/SelectorReplace')(selectorReplace), plugins.hasRTL && require('@zohodesk/postcss-rtl')({
+  const postcssPlugins = [plugins.valueReplacer && require('../postcss-plugins/ValueReplacer')(valueReplacer), plugins.hasRTL && require('@zohodesk/postcss-rtl')({
     addPrefixToSelector: function addPrefixToSelector(selector, prefix) {
       if (prefix === '[dir]') {
         return selector;
@@ -62,7 +81,7 @@ const getCSSLoaders = optionsObj => {
 
       return `${prefix} ${selector}`; // Make selectors like [dir=rtl] > .selector
     }
-  }), plugins.combinerMediaQuery && require('postcss-combine-media-query')(), plugins.hoverActive && require('../postcss-plugins/hoverActivePlugin')(mediaQueryHoverActiveString), plugins.cssVariableReplacement && fs.existsSync(cssVariableReplacementConfig) && require('../postcss-plugins/variableModificationPlugin/index').plugin(cssVariableReplacementConfig) // ,
+  }), plugins.selectorReplace && require('../postcss-plugins/SelectorReplace')(selectorReplace), plugins.combinerMediaQuery && require('postcss-combine-media-query')(), plugins.hoverActive && require('../postcss-plugins/hoverActivePlugin')(mediaQueryHoverActiveString), plugins.cssVariableReplacement && fs.existsSync(cssVariableReplacementConfig) && require('../postcss-plugins/variableModificationPlugin/index').plugin(cssVariableReplacementConfig) // ,
   // plugins.composeMinification &&
   //   require('../postcss-plugins/composePlugin')()
   ].filter(Boolean);
@@ -89,65 +108,67 @@ const getCSSLoaders = optionsObj => {
         // console.log('check here : ', params.resourcePath);
         // getCSSLoaders for all postcss filtering
 
-        /*
-        input : 
-        params.resourcePath : 'D:/MyWork/..../desk_client_app/supportapp/src/components/Avatar/Avatar.module.css,
-        patterns : [
-            // include all files
-           "valueReplacer": [ 
-              "*"
-            ],
-            // include all files
-            "selectorReplace": [
-              "*"
-            ],
-            // include src folder, include deskapp folder, exclude node modules
-            "hoverActive": [
-              "src",
-              "deskapp",
-              "!node_modules"
-            ],
-            // include src folder, include deskapp folder, exclude node modules
-            "combinerMediaQuery": [
-              "src",
-              "deskapp",
-              "!node_modules"
-            ],
-            // include src folder, include deskapp folder, exclude node modules
-            "hasRTL": [
-              "src",
-              "deskapp",
-              "!node_modules"
-            ],
-            // include src folder, include deskapp folder, exclude node modules
-            "cssVariableReplacement": [
-              "src",
-              "deskapp",
-              "!node_modules"
-            ],
-            // include src folder, include deskapp folder, exclude node modules
-            "selectorWeight": [
-              "src",
-              "deskapp",
-              "!node_modules"
-            ],
-            // include all files
-            "cssUniqueness": [
-              "*"
-            ]
-        ],
-        
-        [ postcssPlugins that are implemented ]
-          */
+        /*
+        input : 
+        params.resourcePath : 'D:/MyWork/..../desk_client_app/supportapp/src/components/Avatar/Avatar.module.css,
+        patterns : [
+            // include all files
+           "valueReplacer": [ 
+              "*"
+            ],
+            // include all files
+            "selectorReplace": [
+              "*"
+            ],
+            // include src folder, include deskapp folder, exclude node modules
+            "hoverActive": [
+              "src",
+              "deskapp",
+              "!node_modules"
+            ],
+            // include src folder, include deskapp folder, exclude node modules
+            "combinerMediaQuery": [
+              "src",
+              "deskapp",
+              "!node_modules"
+            ],
+            // include src folder, include deskapp folder, exclude node modules
+            "hasRTL": [
+              "src",
+              "deskapp",
+              "!node_modules"
+            ],
+            // include src folder, include deskapp folder, exclude node modules
+            "cssVariableReplacement": [
+              "src",
+              "deskapp",
+              "!node_modules"
+            ],
+            // include src folder, include deskapp folder, exclude node modules
+            "selectorWeight": [
+              "src",
+              "deskapp",
+              "!node_modules"
+            ],
+            // include all files
+            "cssUniqueness": [
+              "*"
+            ]
+        ],
+        
+        [ postcssPlugins that are implemented ]
+            */
         //  console.log(params.resourcePath);
         //  console.log(postcssPlugins);
-        const finalpostcssPlugins = (0, _fileHandling.isFileNameMatchingPattern)({
+        const pluginOrder = Object.keys(plugins).filter(x => plugins[x] !== false);
+        const finalPostcssPlugins = (0, _fileHandling.isFileNameMatchingPattern)({
           filename: params.resourcePath,
           filterObject: patterns,
-          plugins: postcssPlugins
+          plugins: postcssPlugins,
+          order: calculatePostCssPluginOrder(postCssPluginOrder, pluginOrder)
         }); // postcssPlugins that are allowed
 
-        return finalpostcssPlugins.length > 0 ? finalpostcssPlugins : [require('../postcss-plugins/EmptyPlugin')()];
+        return finalPostcssPlugins.length > 0 ? finalPostcssPlugins : [require('../postcss-plugins/EmptyPlugin')()];
       }
     }
   } : null, plugins.composeMinification ? {

package/lib/loaders/workerLoader.js

@@ -16,7 +16,7 @@ var _SingleEntryPlugin = _interopRequireDefault(require("webpack/lib/SingleEntry
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-/* import WebWorkerTemplatePlugin from 'webpack/lib/webworker/WebWorkerTemplatePlugin';
+/* import WebWorkerTemplatePlugin from 'webpack/lib/webworker/WebWorkerTemplatePlugin';
 import ExternalsPlugin from 'webpack/lib/ExternalsPlugin'; */
 const schema = {
   'type': 'object',
@@ -80,14 +80,14 @@ function pitch(request) {
     globalObject: 'self'
   };
   workerContext.compiler = this._compilation.createChildCompiler(`worker-loader ${request}`, workerContext.options);
-  /* 
-  new WebWorkerTemplatePlugin().apply(workerContext.compiler);
-  
-  if (compilerOptions.externals) {
-    new ExternalsPlugin(
-      getExternalsType(compilerOptions),
-      compilerOptions.externals
-    ).apply(workerContext.compiler);
+  /* 
+  new WebWorkerTemplatePlugin().apply(workerContext.compiler);
+  
+  if (compilerOptions.externals) {
+    new ExternalsPlugin(
+      getExternalsType(compilerOptions),
+      compilerOptions.externals
+    ).apply(workerContext.compiler);
   } */
 
   new _SingleEntryPlugin.default(this.context, `!!${request}`, _path.default.parse(this.resourcePath).name).apply(workerContext.compiler);

package/lib/pluginUtils/getDevPlugins.js

@@ -72,7 +72,8 @@ const getDevPlugins = (options, publicPath) => {
       enableRTLSplit
     },
     i18n,
-    unusedFiles
+    unusedFiles,
+    resourceHints
   } = options;
   const hasEFC = newOptionForEnableEFC || prevOptionForEnableEFC;
   const cssLTRFileNameTempalte = enableRTLSplit ? 'css/[name].ltr.css' : 'css/[name].css';
@@ -94,7 +95,7 @@ const getDevPlugins = (options, publicPath) => {
     filename: cssLTRFileNameTempalte,
     // ignoreOrder: true,
     chunkFilename: cssLTRFileNameTempalte
-  }), new _plugins.ResourceHintsPlugin()];
+  }), new _plugins.ResourceHintsPlugin(resourceHints)];
 
   if (enableRTLSplit) {
     pluginsArr.push(new _RtlCssPlugin.RtlCssPlugin({
@@ -114,11 +115,11 @@ const getDevPlugins = (options, publicPath) => {
   }
 
   const i18nManifestFileName = 'i18n-manifest.json';
-  /**
-   * NOTE:
-   *  this file name ext .i18n.js added
-   *  Because, in service worker they maintain cache as file name based
-   *  So, to make differce we added .i18n.js ext.
+  /**
+   * NOTE:
+   *  this file name ext .i18n.js added
+   *  Because, in service worker they maintain cache as file name based
+   *  So, to make differce we added .i18n.js ext.
    */
 
   const i18nFileNameTemplate = 'i18n-chunk/[locale]/[name].i18n.js';

package/lib/pluginUtils/getProdPlugins.js

@@ -107,7 +107,7 @@ const getProdPlugins = (options, publicPath = '') => {
     // ignoreOrder: true,
     filename: cssLTRFileNameTempalte,
     chunkFilename: cssLTRFileNameTempalte
-  }), new _plugins.ResourceHintsPlugin(), new _plugins.MinifyPlugin()];
+  }), new _plugins.ResourceHintsPlugin(options.resourceHints), new _plugins.MinifyPlugin()];
 
   if (enableRTLSplit) {
     pluginsArr.push(new _RtlCssPlugin.RtlCssPlugin({
@@ -121,11 +121,11 @@ const getProdPlugins = (options, publicPath = '') => {
   }
 
   const i18nManifestFileName = 'i18n-manifest.json';
-  /**
-   * NOTE:
-   *  this file name ext .i18n.js added
-   *  Because, in service worker they maintain cache as file name based
-   *  So, to make differce we added .i18n.js ext.
+  /**
+   * NOTE:
+   *  this file name ext .i18n.js added
+   *  Because, in service worker they maintain cache as file name based
+   *  So, to make differce we added .i18n.js ext.
    */
 
   const i18nFileNameTemplate = 'i18n-chunk/[locale]/[name].[chunkhash].i18n.js';

package/lib/plugins/CustomAttributePlugin.md

@@ -1,35 +1,35 @@
-# CustomAttributePlugin
- In this plugin we added all dynamic chunks tags (like js script tags , css script tags, i18n split script tags) to extra custom attribute
-
-## Options
-
-```json
-{
-  "name": "some",
-  "react-cli": {
-    "app": {
-      "customAttributes": {// this is the all options
-        "enable": false,
-        "attributes": null,
-        "jsAttributes": null,
-        "cssAttributes": null,
-        "i18nAttributes": null
-      }
-  }
-}
-```
-
-### enable   [Boolean]
-this decides is this plugin or feature needs to be enabled of not.
-
-
-### attributes   [Object] 
-It  was object of key value pairs Example Given Option {"data-app-name":"myapp"}  
-Then all dynamically (js script tags , css script tags, i18n split script tags) added tags via webpack 
-
-### jsAttributes   [Object] 
-same as [attributes] but only for js script tags
-### cssAttributes   [Object] 
-same as [attributes] but only for css script tags
-### i18nAttributes   [Object] 
-same as [attributes] but only for i18n split script tags
+# CustomAttributePlugin
+ In this plugin we added all dynamic chunks tags (like js script tags , css script tags, i18n split script tags) to extra custom attribute
+
+## Options
+
+```json
+{
+  "name": "some",
+  "react-cli": {
+    "app": {
+      "customAttributes": {// this is the all options
+        "enable": false,
+        "attributes": null,
+        "jsAttributes": null,
+        "cssAttributes": null,
+        "i18nAttributes": null
+      }
+  }
+}
+```
+
+### enable   [Boolean]
+this decides is this plugin or feature needs to be enabled of not.
+
+
+### attributes   [Object] 
+It  was object of key value pairs Example Given Option {"data-app-name":"myapp"}  
+Then all dynamically (js script tags , css script tags, i18n split script tags) added tags via webpack 
+
+### jsAttributes   [Object] 
+same as [attributes] but only for js script tags
+### cssAttributes   [Object] 
+same as [attributes] but only for css script tags
+### i18nAttributes   [Object] 
+same as [attributes] but only for i18n split script tags

package/lib/plugins/EFCPlugin.md

@@ -1,6 +1,6 @@
-# EFC Plugin
-
-In EFC plugin we are create sdk file for efc related purpose
-
-
-# document will be soon
+# EFC Plugin
+
+In EFC plugin we are create sdk file for efc related purpose
+
+
+# document will be soon

package/lib/plugins/I18NInjectIntoIndexPlugin.js

@@ -115,10 +115,10 @@ class I18NInjectIntoIndexPlugin {
         i18nObj = i18nFiles.reduce((res, next) => {
           const fileName = next.replace(`i18n${_path.default.sep}`, '');
           const splittedFileName = fileName.split('.');
-          /* if (this.isDevelopment) {
-            res[splittedFileName[0]] = `${this.publicPath}/i18n/${fileName}`;
-          } else {
-            res[splittedFileName[0]] = fileName;
+          /* if (this.isDevelopment) {
+            res[splittedFileName[0]] = `${this.publicPath}/i18n/${fileName}`;
+          } else {
+            res[splittedFileName[0]] = fileName;
           } */
 
           res[splittedFileName[0]] = `${this.publicPath}/i18n/${fileName}`;

package/lib/plugins/I18nSplitPlugin/I18nDownlodLogic.js

@@ -13,23 +13,23 @@ var _hashUtils = require("./utils/hashUtils");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-/**
- * this plugin for the download i18n files
- * So, if need to downoad i18n in browser this plugin handle that
- *
- *
- * this plugin works
- * how it works?
- *  this plugin hook the requireEnsure method and we write logic for current compiled file download logic
- *
- * hinds:-
- *  requireEnsure method was loader for all js chunk's ,
- *  which means if we use dynamic import then this function will create script tag for that dynamic resource
- *  so every dynamic js chunk will load by this function,
- *  So we update this function with when  dynamic jschunk call the nour code also execute
- *
- * some thing  missing  in this definetion
- *
+/**
+ * this plugin for the download i18n files
+ * So, if need to downoad i18n in browser this plugin handle that
+ *
+ *
+ * this plugin works
+ * how it works?
+ *  this plugin hook the requireEnsure method and we write logic for current compiled file download logic
+ *
+ * hinds:-
+ *  requireEnsure method was loader for all js chunk's ,
+ *  which means if we use dynamic import then this function will create script tag for that dynamic resource
+ *  so every dynamic js chunk will load by this function,
+ *  So we update this function with when  dynamic jschunk call the nour code also execute
+ *
+ * some thing  missing  in this definetion
+ *
  */
 // import { REGEXP_HASH, REGEXP_CHUNKHASH } from './utils/hashUtils';
 const pluginName = 'DownloadLogicOfI18n'; // const pluginName = 'i18n-plugin';
@@ -126,20 +126,20 @@ class I18nDownlodLogic {
       return _webpack.Template.asString([source, '', '// object to store loaded I18N chunks', 'var installedI18nChunks = {', _webpack.Template.indent(mainChunk.ids.map(id => `${JSON.stringify(id)}: 0`).join(',\n')), '};']);
     });
     mainTemplate.hooks.requireEnsure.tap(pluginName, (source, mainChunk, hash) => {
-      /**
-       * for Information this is tapped in mainTemplate ,
-       * So this hooks argument chunk is main chunk means entry chunk mostly.
-       * if any chunk has i18n then we must write our download  i18n logic,
+      /**
+       * for Information this is tapped in mainTemplate ,
+       * So this hooks argument chunk is main chunk means entry chunk mostly.
+       * if any chunk has i18n then we must write our download  i18n logic,
        */
       const chunkMap = this.getI18nChunkObject(mainChunk);
 
       if (!Object.keys(chunkMap).length) {
         return source;
       }
-      /**
-       *   chunkMaps has
-       *  @property  {Object} hash  [it has key  as chunk id and value as chunkHash ]
-       *  @property  {Object} name  [it has key  as chunk id and value as chunk name ]
+      /**
+       *   chunkMaps has
+       *  @property  {Object} hash  [it has key  as chunk id and value as chunkHash ]
+       *  @property  {Object} name  [it has key  as chunk id and value as chunk name ]
        */
 
 
@@ -176,22 +176,22 @@ class I18nDownlodLogic {
         },
         contentHashType: MODULE_TYPE
       });
-      /** IDEA:
-       *      as my suggestion ignore this `jsop` approch,
-       *      Because json parse is faster then js parse.
-       *      my suggestion  is make ajax or fetch request
+      /** IDEA:
+       *      as my suggestion ignore this `jsop` approch,
+       *      Because json parse is faster then js parse.
+       *      my suggestion  is make ajax or fetch request
        */
 
       const buf = [];
-      /*
-        if (REGEXP_HASH.test(filenameTemplate)) {
-          buf.push(
-            `var hash = ${mainTemplate.renderCurrentHashCode(hash)};`
-          );
-        }
-        if (REGEXP_CHUNKHASH.test(filenameTemplate)) {
-          buf.push(`var chunkHashes = ${JSON.stringify(chunkMaps.hash)};`);
-        }
+      /*
+        if (REGEXP_HASH.test(filenameTemplate)) {
+          buf.push(
+            `var hash = ${mainTemplate.renderCurrentHashCode(hash)};`
+          );
+        }
+        if (REGEXP_CHUNKHASH.test(filenameTemplate)) {
+          buf.push(`var chunkHashes = ${JSON.stringify(chunkMaps.hash)};`);
+        }
         */
 
       if ((0, _hashUtils.hasContentHash)(filenameTemplate)) {