refactor: create-wmr checks dir before installing (#365)

* refactor: create-wmr checks dir before installing * docs: Adding changeset
preactjs · Feb 23, 2021 · cce35d3 · cce35d3
1 parent ee75298
commit cce35d3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/.changeset/grumpy-pens-decide.md b/.changeset/grumpy-pens-decide.md
@@ -0,0 +1,5 @@
+---
+'create-wmr': minor
+---
+
+Blocks accidentally creating a new WMR project into a directory that already exists
diff --git a/packages/create-wmr/src/index.js b/packages/create-wmr/src/index.js
@@ -11,12 +11,19 @@ const { dim, bold, cyan, red } = kleur;
 
 sade('create-wmr [dir]', true)
 	.option('--eslint', 'Set up the Preact ESLint configuration (takes a lot longer)', false)
+	.option('--force', 'Force install into an existing directory', false)
 	.describe('Initialize a WMR project')
 	.example('npm init wmr ./some-directory')
 	.action(async (dir, opts) => {
 		const origCwd = process.cwd();
 		let cwd = process.cwd();
 		if (dir) {
+			if ((await fs.stat(dir)).isDirectory() && !opts.force) {
+				process.stderr.write(
+					`${red(`Refusing to overwrite directory! Please specify a different directory or use the '--force' flag`)}\n`
+				);
+				process.exit(1);
+			}
 			cwd = resolve(cwd, dir || '.');
 			await fs.mkdir(cwd, { recursive: true });
 			process.chdir(cwd);