fix(core): throw an error if generating a new workspace into a non-em…

…pty directory (#5296)

packages/workspace/src/generators/new/new.spec.ts

@@ -106,4 +106,19 @@ describe('new', () => {
     expect(readJson(tree, 'package.json')).toEqual(packageJson);
     expect(readJson(tree, '.eslintrc.json')).toEqual(eslintConfig);
   });
+
+  it('should throw an error when the directory is not empty', async () => {
+    tree.write('my-workspace/file.txt', '');
+
+    try {
+      await newGenerator(tree, {
+        ...defaultOptions,
+        name: 'my-workspace',
+        directory: 'my-workspace',
+        npmScope: 'npmScope',
+        appName: 'app',
+      });
+      fail('Generating into a non-empty directory should error.');
+    } catch (e) {}
+  });
 });

packages/workspace/src/generators/new/new.ts

@@ -171,6 +171,16 @@ export async function newGenerator(host: Tree, options: Schema) {
 
   options = normalizeOptions(options);
 
+  if (
+    host.exists(options.name) &&
+    !host.isFile(options.name) &&
+    host.children(options.name).length > 0
+  ) {
+    throw new Error(
+      `${join(host.root, options.name)} is not an empty directory.`
+    );
+  }
+
   const layout: 'packages' | 'apps-and-libs' =
     options.preset === 'oss' ? 'packages' : 'apps-and-libs';
   const workspaceOpts = {