@putout/operator-filesystem 3.4.0 → 3.6.0

package/README.md

@@ -17,28 +17,12 @@ npm i putout @putout/operator-filesystem
 
 ```js
 const {operator} = require('putout');
-const {
-    createDirectory,
-    findFile,
-    createFile,
-} = operator;
+const {findFile, createFile} = operator;
 
 const [dirPath] = findFile(ast, 'hello');
 const filePath = createFile(dirPath, 'world.txt', 'hello world');
 ```
 
-```js
-const {operator} = require('putout');
-const {
-    createDirectory,
-    findFile,
-} = operator;
-
-const [dirPath] = findFile(ast, 'hello');
-
-const newDirectoryPath = createDirectory(dirPath, 'world');
-```
-
 ### `createDirectory(directoryPath: FilePath, name: string): FilePath`
 
 ```js
@@ -100,6 +84,19 @@ getFileType(filePath);
 'file';
 ```
 
+### `getFileContent(path: FilePath): [is: boolean, content: string]`
+
+Get `content` property if it exists, use [`readFileContent`](#read-file-content) to read file from **Filesystem**.
+
+```js
+const {operator} = require('putout');
+const {getFileContent} = operator;
+
+getFileContent(filePath);
+// returns
+[true, 'hello world'];
+```
+
 ### `removeFile(filePath: Path)`
 
 ```js

package/lib/filesystem.js

@@ -15,7 +15,15 @@ const isString = (a) => typeof a === 'string';
 const {isArray} = Array;
 const maybeArray = (a) => isArray(a) ? a : [a];
 
-const toBase64 = (a) => btoa(encodeURI(a));
+const toBase64 = (content) => {
+    const [e, decoded] = tryCatch(btoa, content);
+    
+    if (!e)
+        return encodeURI(decoded);
+    
+    return content;
+};
+
 const fromBase64 = (content) => {
     const [e, decoded] = tryCatch(atob, content);
     
@@ -93,6 +101,8 @@ function getFileType(filePath) {
     return typePath.node.value.value;
 }
 
+module.exports.getFileContent = getFileContent;
+
 function getFileContent(filePath) {
     const content = getProperty(filePath, 'content');
     
@@ -147,8 +157,8 @@ module.exports.moveFile = (filePath, dirPath) => {
     
     setLiteralValue(filenamePath.get('value'), newFilename);
     dirPathFiles.node.value.elements.push(filePath.node);
-    filePath.remove();
     
+    filePath.remove();
     maybeFS.renameFile(filename, newFilename);
 };
 
@@ -164,9 +174,9 @@ module.exports.copyFile = (filePath, dirPath) => {
     const [hasContent, content] = getFileContent(filePath);
     
     const copiedFile = ObjectExpression([
-        createType('file'),
-        createFilename(newFilename),
-        hasContent && createContent(content),
+        createTypeProperty('file'),
+        createFilenameProperty(newFilename),
+        hasContent && createContentProperty(content),
     ].filter(Boolean));
     
     maybeRemoveFile(dirPath, newFilename);
@@ -178,6 +188,13 @@ module.exports.copyFile = (filePath, dirPath) => {
 };
 
 function maybeRemoveFile(dirPath, filename) {
+    const type = getFileType(dirPath);
+    
+    if (type !== 'directory') {
+        const filename = getFilename(dirPath);
+        throw Error(`☝️ Looks like '${filename}' is not a directory, but: '${type}'. Rename to '${filename}/'`);
+    }
+    
     const dirPathFiles = getProperty(dirPath, 'files');
     const [fileToOverwrite] = findFile(dirPathFiles, filename);
     
@@ -187,28 +204,46 @@ function maybeRemoveFile(dirPath, filename) {
     fileToOverwrite.remove();
 }
 
-const createType = (type) => ObjectProperty(StringLiteral('type'), StringLiteral(type));
-const createFiles = (files) => ObjectProperty(StringLiteral('files'), ArrayExpression(files));
-const createFilename = (filename) => ObjectProperty(StringLiteral('filename'), StringLiteral(filename));
-const createContent = (content) => ObjectProperty(StringLiteral('content'), StringLiteral(content));
+const createTypeProperty = (type) => ObjectProperty(StringLiteral('type'), StringLiteral(type));
+
+module.exports.createTypeProperty = createTypeProperty;
 
-module.exports.createFile = (dirPath, name, content = '') => {
+const createFilesProperty = (files) => ObjectProperty(StringLiteral('files'), ArrayExpression(files));
+
+module.exports.createFilesProperty = createFilesProperty;
+
+const createFilenameProperty = (filename) => ObjectProperty(StringLiteral('filename'), StringLiteral(filename));
+
+module.exports.createFilenameProperty = createFilenameProperty;
+
+const createContentProperty = (content) => ObjectProperty(StringLiteral('content'), StringLiteral(content));
+
+module.exports.createContentProperty = createContentProperty;
+
+module.exports.createFile = (dirPath, name, content) => {
     maybeRemoveFile(dirPath, name);
     
     const dirPathFiles = getFiles(dirPath);
     const parentFilename = getFilename(dirPath);
     const filename = join(parentFilename, name);
     
-    const typeProperty = createType('file');
-    const filenameProperty = createFilename(filename);
+    const typeProperty = createTypeProperty('file');
+    const filenameProperty = createFilenameProperty(filename);
     
-    dirPathFiles.node.value.elements.push(ObjectExpression([typeProperty, filenameProperty, createContent(content)]));
+    const properties = [
+        typeProperty,
+        filenameProperty,
+        content && createContentProperty(content),
+    ].filter(Boolean);
+    
+    dirPathFiles.node.value.elements.push(ObjectExpression(properties));
     
     const filePath = dirPathFiles
         .get('value.elements')
         .at(-1);
     
-    writeFileContent(filePath, content);
+    if (isString(content))
+        writeFileContent(filePath, content);
     
     return filePath;
 };
@@ -222,9 +257,9 @@ module.exports.createDirectory = (dirPath, name) => {
     const parentFilename = getFilename(dirPath);
     const filename = join(parentFilename, name);
     
-    const typeProperty = createType('directory');
-    const filesProperty = createFiles([]);
-    const filenameProperty = createFilename(filename);
+    const typeProperty = createTypeProperty('directory');
+    const filesProperty = createFilesProperty([]);
+    const filenameProperty = createFilenameProperty(filename);
     
     dirPathFiles.node.value.elements.push(ObjectExpression([
         typeProperty,
@@ -239,13 +274,6 @@ module.exports.createDirectory = (dirPath, name) => {
         .at(-1);
 };
 
-const createContentProperty = (content) => {
-    const contentKey = StringLiteral('content');
-    const contentValue = StringLiteral(toBase64(content));
-    
-    return ObjectProperty(contentKey, contentValue);
-};
-
 module.exports.readFileContent = (filePath) => {
     const fileType = getFileType(filePath);
     
@@ -285,7 +313,7 @@ function writeFileContent(filePath, content) {
         return;
     }
     
-    const property = createContentProperty(content);
+    const property = createContentProperty(toBase64(content));
     filePath.node.properties.push(property);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/operator-filesystem",
-  "version": "3.4.0",
+  "version": "3.6.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "🐊Putout operator adds ability to filesystem referenced variables that was not defined",