refactor: modify message api

Author: alexander-akait

src/index.js

@@ -123,7 +123,7 @@ export default function loader(content, map, meta) {
         newMap = JSON.stringify(newMap);
       }
 
-      let newContentObj = {
+      let moduleObj = {
         imports: '',
         runtime: `module.exports = exports = require(${stringifyRequest(
           this,
@@ -138,18 +138,18 @@ export default function loader(content, map, meta) {
       if (result.messages && result.messages.length > 0) {
         result.messages
           .filter(
-            (message) => (message.type === 'modify-code' ? message : false)
+            (message) => (message.type === 'modify-module' ? message : false)
           )
           .forEach((message) => {
             try {
-              newContentObj = message.modifyCode(this, newContentObj);
+              moduleObj = message.modifyModule(moduleObj, this);
             } catch (err) {
               this.emitError(err);
             }
           });
       }
 
-      const { imports, runtime, module, exports } = newContentObj;
+      const { imports, runtime, module, exports } = moduleObj;
 
       cb(
         null,

src/plugin.js

@@ -85,12 +85,12 @@ export default postcss.plugin(
           if (!alreadyImported[url]) {
             result.messages.push({
               pluginName,
-              type: 'modify-code',
-              modifyCode: (loaderContext, contentObj) => {
+              type: 'modify-module',
+              modifyModule: (moduleObj) => {
                 // eslint-disable-next-line no-param-reassign
-                contentObj.runtime = `${contentObj.runtime}${runtimeCode}\n`;
+                moduleObj.runtime = `${moduleObj.runtime}${runtimeCode}\n`;
 
-                return contentObj;
+                return moduleObj;
               },
             });
 
@@ -155,35 +155,35 @@ export default postcss.plugin(
 
               result.messages.push({
                 pluginName,
-                type: 'modify-code',
-                modifyCode: (loaderContext, contentObj) => {
+                type: 'modify-module',
+                modifyModule: (moduleObj, loaderContext) => {
                   if (!hasURLEscapeRuntimeCode) {
                     // eslint-disable-next-line no-param-reassign
-                    contentObj.imports = `var runtimeEscape = require(${stringifyRequest(
+                    moduleObj.imports = `var runtimeEscape = require(${stringifyRequest(
                       loaderContext,
                       runtimeEscapeFile
-                    )});\n${contentObj.imports}`;
+                    )});\n${moduleObj.imports}`;
 
                     hasURLEscapeRuntimeCode = true;
                   }
 
                   // eslint-disable-next-line no-param-reassign
-                  contentObj.imports = `${
-                    contentObj.imports
+                  moduleObj.imports = `${
+                    moduleObj.imports
                   }var ${placeholder} = require(${stringifyRequest(
                     loaderContext,
                     requestedURL
                   )});\n`;
 
                   // eslint-disable-next-line no-param-reassign
-                  contentObj.module = contentObj.module.replace(
+                  moduleObj.module = moduleObj.module.replace(
                     placeholder,
                     `" + runtimeEscape(${placeholder}) + "${
                       splittedURL[1] ? splittedURL[1] : ''
                     }${splittedURL[2] ? `#${splittedURL[2]}` : ''}`
                   );
 
-                  return contentObj;
+                  return moduleObj;
                 },
               });