Merge pull request #39 from AckeeCZ/feat/34-safe-ouput-dir-overwrite

Feat/34 safe ouput dir overwrite

Author: pavelsvagr

README.md

@@ -25,6 +25,7 @@ STARTER        Which template to setup (required)
 Options:
   --dir, -d DIR       Destination directory (default: ./node-app)
   --project-name, -n NAME     Google Cloud project name (default: directory basename)
+  --force, -f         Overwrite existing destination directory if it's not empty
   --help, -h          Show this help message
 
 Starters available:

lib/Bootstrap.js

@@ -35,6 +35,13 @@ export class Bootstrap {
         default: 'node-app',
         description: 'Google Cloud project name',
       })
+      .option('force', {
+        type: 'boolean',
+        alias: 'f',
+        default: false,
+        description:
+          "Overwrite existing destination directory if it's not empty",
+      })
       .version('1.0.0')
       .help()
 
@@ -73,7 +80,19 @@ export class Bootstrap {
       projectName: parsedArgs.projectName,
     })
     starter.setToolbelt(toolbelt)
-    toolbelt.mkdir(destination, { overwrite: true })
+
+    if (!toolbelt.isDirectoryEmpty(destination)) {
+      if (!parsedArgs.force) {
+        logger.info(
+          `Directory '${destination}' already exists and is not empty. Use --force or -f flag to overwrite the existing directory.`
+        )
+        process.exit(1)
+      } else {
+        logger.info(`Overwriting existing directory '${destination}'`)
+      }
+    }
+
+    toolbelt.mkdir(destination, { overwrite: parsedArgs.force })
     toolbelt.npm.init()
     starter.install()
   }

lib/Bootstrap.js.map

@@ -30,6 +30,23 @@ export class Toolbelt {
   public stringToPath(str: string) {
     return path.normalize(str) as Path
   }
+
+  public isDirectoryEmpty(dirpath: string): boolean {
+    const path = this.stringToPath(dirpath)
+
+    if (!fs.existsSync(path)) {
+      return true
+    }
+
+    const stat = fs.statSync(path)
+    if (!stat.isDirectory()) {
+      return true
+    }
+
+    const contents = fs.readdirSync(path)
+    return contents.length === 0
+  }
+
   public mkdir(
     dirpath: string,
     option?: {
@@ -43,7 +60,7 @@ export class Toolbelt {
       if (fs.existsSync(dirpath) && option?.overwrite) {
         fs.rmSync(dirpath, { recursive: true })
       }
-      fs.mkdirSync(dirpath)
+      fs.mkdirSync(dirpath, { recursive: true })
     }
   }
   /**