feat(misc): introduce a way to set the project name/root format for a…

…ll generators
nrwl · Sep 1, 2023 · ab66c70 · ab66c70
1 parent ad2d1e8
commit ab66c70
Show file tree

Hide file tree

Showing 13 changed files with 140 additions and 34 deletions.
diff --git a/docs/generated/devkit/NxJsonConfiguration.md b/docs/generated/devkit/NxJsonConfiguration.md
@@ -191,7 +191,8 @@ Where new apps + libs should be placed
 
 #### Type declaration
 
-| Name      | Type     |
-| :-------- | :------- |
-| `appsDir` | `string` |
-| `libsDir` | `string` |
+| Name                        | Type                           |
+| :-------------------------- | :----------------------------- |
+| `appsDir?`                  | `string`                       |
+| `libsDir?`                  | `string`                       |
+| `projectNameAndRootFormat?` | `"as-provided"` \| `"derived"` |
diff --git a/docs/generated/devkit/Workspace.md b/docs/generated/devkit/Workspace.md
@@ -267,10 +267,11 @@ Where new apps + libs should be placed
 
 #### Type declaration
 
-| Name      | Type     |
-| :-------- | :------- |
-| `appsDir` | `string` |
-| `libsDir` | `string` |
+| Name                        | Type                           |
+| :-------------------------- | :----------------------------- |
+| `appsDir?`                  | `string`                       |
+| `libsDir?`                  | `string`                       |
+| `projectNameAndRootFormat?` | `"as-provided"` \| `"derived"` |
 
 #### Inherited from
 

diff --git a/e2e/utils/create-project-utils.ts b/e2e/utils/create-project-utils.ts
@@ -28,7 +28,7 @@ import {
   RunCmdOpts,
   runCommand,
 } from './command-utils';
-import { output } from '@nx/devkit';
+import { NxJsonConfiguration, output } from '@nx/devkit';
 import { readFileSync } from 'fs';
 import { join } from 'path';
 
@@ -41,6 +41,7 @@ let projName: string;
 export function newProject({
   name = uniq('proj'),
   packageManager = getSelectedPackageManager(),
+  unsetProjectNameAndRootFormat = true,
 } = {}): string {
   try {
     const projScope = 'proj';
@@ -51,6 +52,13 @@ export function newProject({
         packageManager,
       });
 
+      if (unsetProjectNameAndRootFormat) {
+        updateJson<NxJsonConfiguration>('nx.json', (nxJson) => {
+          delete nxJson.workspaceLayout;
+          return nxJson;
+        });
+      }
+
       // Temporary hack to prevent installing with `--frozen-lockfile`
       if (isCI && packageManager === 'pnpm') {
         updateFile(

diff --git a/packages/devkit/src/generators/project-name-and-root-utils.spec.ts b/packages/devkit/src/generators/project-name-and-root-utils.spec.ts
@@ -348,10 +348,21 @@ describe('determineProjectNameAndRootOptions', () => {
 
       expect(promptSpy).toHaveBeenCalledTimes(2);
 
-      expect(readNxJson(tree).generators['@nx/some-plugin:app']).toEqual({
+      expect(readNxJson(tree).workspaceLayout).toEqual({
         projectNameAndRootFormat: 'as-provided',
       });
 
+      promptSpy.mockReset();
+
+      await determineProjectNameAndRootOptions(tree, {
+        name: 'libName',
+        projectType: 'library',
+        directory: 'shared',
+        callingGenerator: '@nx/some-plugin:app',
+      });
+
+      expect(promptSpy).not.toHaveBeenCalled();
+
       // restore original interactive mode
       restoreOriginalInteractiveMode();
     });

diff --git a/packages/devkit/src/generators/project-name-and-root-utils.ts b/packages/devkit/src/generators/project-name-and-root-utils.ts
@@ -75,7 +75,9 @@ export async function determineProjectNameAndRootOptions(
   const formats = getProjectNameAndRootFormats(tree, options);
   const format =
     options.projectNameAndRootFormat ??
-    (await determineFormat(tree, formats, options.callingGenerator));
+    (getDefaultProjectNameAndRootFormat(tree) === 'as-provided'
+      ? 'as-provided'
+      : await determineFormat(tree, formats, options.callingGenerator));
 
   return {
     ...formats[format],
@@ -167,11 +169,7 @@ async function determineFormat(
       initial: true,
     });
     if (saveDefault) {
-      const nxJson = readNxJson(tree);
-      nxJson.generators ??= {};
-      nxJson.generators[callingGenerator] ??= {};
-      nxJson.generators[callingGenerator].projectNameAndRootFormat = result;
-      updateNxJson(tree, nxJson);
+      setProjectNameAndRootFormatDefault(tree);
     } else {
       logger.warn(deprecationWarning);
     }
@@ -183,6 +181,18 @@ async function determineFormat(
   return result;
 }
 
+function setProjectNameAndRootFormatDefault(tree: Tree) {
+  const nxJson = readNxJson(tree);
+  nxJson.workspaceLayout ??= {};
+  nxJson.workspaceLayout.projectNameAndRootFormat = 'as-provided';
+  updateNxJson(tree, nxJson);
+}
+
+function getDefaultProjectNameAndRootFormat(tree: Tree) {
+  const nxJson = readNxJson(tree);
+  return nxJson.workspaceLayout?.projectNameAndRootFormat ?? 'derived';
+}
+
 function getProjectNameAndRootFormats(
   tree: Tree,
   options: ProjectGenerationOptions

diff --git a/packages/nx/schemas/nx-schema.json b/packages/nx/schemas/nx-schema.json
@@ -56,6 +56,11 @@
         "appsDir": {
           "type": "string",
           "description": "Default folder name for apps."
+        },
+        "projectNameAndRootFormat": {
+          "type": "string",
+          "description": "Default method of handling arguments for generating projects",
+          "enum": ["as-provided", "derived"]
         }
       },
       "additionalProperties": false

diff --git a/packages/nx/src/config/nx-json.ts b/packages/nx/src/config/nx-json.ts
@@ -86,8 +86,9 @@ export interface NxJsonConfiguration<T = '*' | string[]> {
    * Where new apps + libs should be placed
    */
   workspaceLayout?: {
-    libsDir: string;
-    appsDir: string;
+    libsDir?: string;
+    appsDir?: string;
+    projectNameAndRootFormat?: 'as-provided' | 'derived';
   };
   /**
    * Available Task Runners

diff --git a/packages/workspace/src/generators/new/__snapshots__/new.spec.ts.snap b/packages/workspace/src/generators/new/__snapshots__/new.spec.ts.snap
@@ -53,5 +53,8 @@ exports[`new should generate an empty nx.json 1`] = `
       "runner": "nx/tasks-runners/default",
     },
   },
+  "workspaceLayout": {
+    "projectNameAndRootFormat": "as-provided",
+  },
 }
 `;
diff --git a/packages/workspace/src/generators/new/generate-workspace-files.spec.ts b/packages/workspace/src/generators/new/generate-workspace-files.spec.ts
@@ -122,6 +122,9 @@ describe('@nx/workspace:generateWorkspaceFiles', () => {
             "runner": "nx/tasks-runners/default",
           },
         },
+        "workspaceLayout": {
+          "projectNameAndRootFormat": "as-provided",
+        },
       }
     `);
     const validateNxJson = ajv.compile(nxSchema);
@@ -174,6 +177,9 @@ describe('@nx/workspace:generateWorkspaceFiles', () => {
             "runner": "nx/tasks-runners/default",
           },
         },
+        "workspaceLayout": {
+          "projectNameAndRootFormat": "as-provided",
+        },
       }
     `);
   });

diff --git a/packages/workspace/src/generators/new/generate-workspace-files.ts b/packages/workspace/src/generators/new/generate-workspace-files.ts
@@ -98,6 +98,9 @@ function createNxJson(
         },
       },
     },
+    workspaceLayout: {
+      projectNameAndRootFormat: 'as-provided',
+    },
   };
 
   nxJson.targetDefaults = {

diff --git a/packages/workspace/src/generators/new/new.ts b/packages/workspace/src/generators/new/new.ts
@@ -41,27 +41,28 @@ export interface NormalizedSchema extends Schema {
   isCustomPreset: boolean;
 }
 
-export async function newGenerator(host: Tree, opts: Schema) {
+export async function newGenerator(tree: Tree, opts: Schema) {
   const options = normalizeOptions(opts);
-  validateOptions(options, host);
+  validateOptions(options, tree);
 
-  await generateWorkspaceFiles(host, { ...options, nxCloud: undefined } as any);
+  await generateWorkspaceFiles(tree, { ...options, nxCloud: undefined } as any);
 
-  addPresetDependencies(host, options);
-  addCloudDependencies(host, options);
+  addPresetDependencies(tree, options);
+
+  addCloudDependencies(tree, options);
 
   return async () => {
     const pmc = getPackageManagerCommand(options.packageManager);
     if (pmc.preInstall) {
       execSync(pmc.preInstall, {
-        cwd: joinPathFragments(host.root, options.directory),
+        cwd: joinPathFragments(tree.root, options.directory),
         stdio: process.env.NX_GENERATE_QUIET === 'true' ? 'ignore' : 'inherit',
       });
     }
-    installPackagesTask(host, false, options.directory, options.packageManager);
+    installPackagesTask(tree, false, options.directory, options.packageManager);
     // TODO: move all of these into create-nx-workspace
     if (options.preset !== Preset.NPM && !options.isCustomPreset) {
-      await generatePreset(host, options);
+      await generatePreset(tree, options);
     }
   };
 }

diff --git a/packages/workspace/src/generators/preset/__snapshots__/preset.spec.ts.snap b/packages/workspace/src/generators/preset/__snapshots__/preset.spec.ts.snap
@@ -36,6 +36,41 @@ exports[`preset should create files (preset = angular-monorepo) 3`] = `
 ]
 `;
 
+exports[`preset should create files (preset = react-monorepo) 1`] = `
+[
+  "src",
+  "tsconfig.app.json",
+  "webpack.config.js",
+  ".babelrc",
+  "tsconfig.json",
+  "project.json",
+  ".eslintrc.json",
+  "jest.config.ts",
+  "tsconfig.spec.json",
+]
+`;
+
+exports[`preset should create files (preset = react-monorepo) 2`] = `
+[
+  "app",
+  "assets",
+  "environments",
+  "favicon.ico",
+  "index.html",
+  "main.tsx",
+  "styles.css",
+]
+`;
+
+exports[`preset should create files (preset = react-monorepo) 3`] = `
+[
+  "app.spec.tsx",
+  "nx-welcome.tsx",
+  "app.module.css",
+  "app.tsx",
+]
+`;
+
 exports[`preset should create files (preset = react-standalone bundler = vite) 1`] = `
 {
   "configurations": {