binhend 1.5.0 → 1.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "binhend",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "",
   "main": "index.js",
   "author": "Nguyen Duc Binh",

package/src/component.build.js

@@ -1,6 +1,6 @@
 
 const { parse } = require('path');
-const { scanNestedFiles, cloneFile, writeToFile, makeFullDirPath } = require('./component.file');
+const { scanNestedFiles, cloneFileIfNew, writeToFileIfNew, makeFullDirPath } = require('./component.file');
 const CodeFormat = require('./code');
 const Component = require('./component');
 const UglifyJS = require('uglify-js');
@@ -19,23 +19,24 @@ function processEachFile({ source: sourceRootPath, web: outputRootPath, module:
         makeFullDirPath(fileOutputPath);
 
         if (parse(file.name).ext !== '.js') {
-            return cloneFile(fileSourcePath, fileOutputPath);
+            return cloneFileIfNew(fileSourcePath, fileOutputPath);
         }
 
         try {
             var component = require(file.path);
         }
         catch (error) {
-            return cloneFile(fileSourcePath, fileOutputPath);
+            return cloneFileIfNew(fileSourcePath, fileOutputPath);
         }
 
         if (!(component instanceof Function) || component.constructor !== Component) {
-            return cloneFile(fileSourcePath, fileOutputPath);
+            return cloneFileIfNew(fileSourcePath, fileOutputPath);
         }
 
         var code = buildCodeMethod(component, stageRootPath);
         code = Component.minification ? minifyCode(code) : beautifyCode(code);
-        writeToFile(fileOutputPath, code);
+
+        writeToFileIfNew(fileOutputPath, code);
     });
 }
 

package/src/component.file.js

@@ -1,10 +1,7 @@
 
-const { readdir, statSync, existsSync, copyFile, writeFile, mkdirSync } = require('fs');
+const { readdir, statSync, existsSync, copyFile, writeFile, writeFileSync, readFileSync, mkdirSync } = require('fs');
 const { join, dirname } = require('path');
 
-// TODO
-// [-] Enhance code by removing sync logics (except for existsSync, mkdirSync): readdir, readFileSync, (statSync?)
-
 function getItemInfo(itemPath) {
     var info = {};
 
@@ -91,6 +88,22 @@ function cloneFile(filepath, outpath) {
     });
 }
 
+function cloneFileIfNew(filepath, outpath) {
+    if (isSameFile(filepath, outpath)) return;
+    cloneFile(filepath, outpath);
+}
+
+function isSameFile(filePathA, filePathB) {
+    var sameFile = false;
+
+    if (existsSync(filePathA) && existsSync(filePathB)) {
+        var fileA = readFileSync(filePathA), fileB = readFileSync(filePathB);
+        sameFile = fileA.equals(fileB);
+    }
+
+    return sameFile;
+}
+
 function makeFullDirPath(filePath) {
     var dirPath = dirname(filePath);
     if (existsSync(dirPath)) return true;
@@ -106,6 +119,29 @@ function writeToFile(fileOutputPath, content) {
     });
 }
 
+function writeToFileIfNew(fileOutputPath, content) {
+    try {
+        if (isSameContent(fileOutputPath, content)) return false;
+        writeFileSync(fileOutputPath, content, { encoding: 'utf8', flag: 'w' });
+        return true;
+    }
+    catch (error) {
+        printError('Failed writing file', fileOutputPath, error);
+        return false;
+    }
+}
+
+function isSameContent(filePath, content) {
+    var sameContent = false;
+
+    if (existsSync(filePath)) {
+        var current = readFileSync(filePath, { encoding: 'utf8', flag: 'r' });
+        sameContent = content === current;
+    }
+
+    return sameContent;
+}
+
 function printError(message, id, error) {
     console.error(`[BINHEND][COMPONENT] ${message}:`, id);
     console.error('[BINHEND][COMPONENT] Error details:', error);
@@ -117,7 +153,10 @@ module.exports = {
     scanFiles,
     scanNestedFiles,
     cloneFile,
+    cloneFileIfNew,
     makeFullDirPath,
     writeToFile,
+    writeToFileIfNew,
+    isSameContent,
     printError
 };

package/src/component.format.js

@@ -1,10 +1,7 @@
 
-const { writeFileSync, readFileSync, existsSync } = require('fs');
+const { readFileSync } = require('fs');
 const { parse } = require('path');
-const { scanNestedFiles, cloneFile, printError, makeFullDirPath } = require('./component.file');
-
-// TODO
-// [-] Enhance code by removing sync logics (except for existsSync, mkdirSync): readdir, readFileSync, (statSync?)
+const { scanNestedFiles, cloneFileIfNew, printError, makeFullDirPath, writeToFileIfNew } = require('./component.file');
 
 const PREFIX_CODE =
 `var { context, tag, svg, script, require, css } = binh.context(module, require);
@@ -29,19 +26,17 @@ function generate(sourceRootPath, outputRootPath, callbackDone) {
     
             if (fileExtension === '.css') {
                 var cssModuleCode = 'binh.css(module);';
-                writeFileSync(fileOutputPath + '.js', cssModuleCode, { encoding: 'utf8', flag: 'w' });
+                writeToFileIfNew(fileOutputPath + '.js', cssModuleCode);
             }
     
             if (fileExtension !== '.js') {
-                return cloneFile(fileSourcePath, fileOutputPath);
+                return cloneFileIfNew(fileSourcePath, fileOutputPath);
             }
 
             var content = readFileSync(fileSourcePath, { encoding: 'utf8', flag: 'r' });
             var code = PREFIX_CODE + content.trim() + '\r\n\r\n;binh.final(module);';
 
-            if (!isSameContent(code, fileSourcePath)) {
-                writeFileSync(fileOutputPath, code, { encoding: 'utf8', flag: 'w' });
-            }
+            writeToFileIfNew(fileOutputPath, code);
         }
         catch (error) {
             printError('Failed formatting file:', fileSourcePath, error);
@@ -49,17 +44,6 @@ function generate(sourceRootPath, outputRootPath, callbackDone) {
     });
 }
 
-function isSameContent(code, filePath) {
-    var sameContent = false;
-
-    if (existsSync(filePath)) {
-        var current = readFileSync(filePath, { encoding: 'utf8', flag: 'r' });
-        sameContent = code === current;
-    }
-
-    return sameContent;
-}
-
 module.exports = {
     generate
 };