@putout/operator-filesystem 3.0.2 → 3.2.0

package/README.md

@@ -53,7 +53,7 @@ const [dirPath] = findFile(ast, 'hello');
 const newDirectoryPath = createDirectory(dirPath, 'world');
 ```
 
-### `findFile(path: Path, name: string | string[]): FilePath[]`
+### `findFile(filePath: Path|FilePath, name: string | string[]): FilePath[]`
 
 ```js
 const {operator} = require('putout');

package/lib/filesystem.js

@@ -11,6 +11,7 @@ const {
 } = require('@putout/operate');
 
 const maybeFS = require('./maybe-fs');
+const isString = (a) => typeof a === 'string';
 const {isArray} = Array;
 const maybeArray = (a) => isArray(a) ? a : [a];
 
@@ -43,6 +44,8 @@ module.exports.getParentDirectory = (filePath) => {
 module.exports.findFile = findFile;
 
 function findFile(node, name) {
+    checkName(name);
+    
     const filePaths = [];
     const names = maybeArray(name);
     
@@ -58,6 +61,11 @@ function findFile(node, name) {
     return filePaths;
 }
 
+function checkName(name) {
+    if (!isString(name) && !isArray(name))
+        throw Error(`☝️ Looks like you forget to pass the 'name' of a file to 'findFile(filePath: Path|FilePath, name: string | string[]): FilePath'`);
+}
+
 function getFilenamePath(filePath) {
     const filenamePath = getProperty(filePath, 'filename');
     return filenamePath.get('value');
@@ -68,10 +76,12 @@ function getFilename(filePath) {
     return value;
 }
 
-module.exports.getFileType = (filePath) => {
+module.exports.getFileType = getFileType;
+
+function getFileType(filePath) {
     const typePath = getProperty(filePath, 'type');
     return typePath.node.value.value;
-};
+}
 
 function getFileContent(filePath) {
     const content = getProperty(filePath, 'content');
@@ -227,6 +237,11 @@ const createContentProperty = (content) => {
 };
 
 module.exports.readFileContent = (filePath) => {
+    const fileType = getFileType(filePath);
+    
+    if (fileType === 'directory')
+        return '';
+    
     const [hasContent, content] = getFileContent(filePath);
     
     if (hasContent) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/operator-filesystem",
-  "version": "3.0.2",
+  "version": "3.2.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",