feat(core): update prompt for installed template to present available options (#436)

Author: AgentEnder

docs/core/generators/application.md

@@ -18,7 +18,7 @@ Generate a dotnet project under the application directory.
 
 - (string): A directory where the project is placed
 
-### <span className="required">template</span>
+### template
 
 - (string): The template to instantiate when the command is invoked. Each template might have specific options you can pass.
 

docs/core/generators/library.md

@@ -18,7 +18,7 @@ Generate a dotnet project under the library directory.
 
 - (string): A directory where the project is placed
 
-### <span className="required">template</span>
+### template
 
 - (string): The template to instantiate when the command is invoked. Each template might have specific options you can pass.
 

package.json

@@ -62,6 +62,7 @@
     "@types/node": "17.0.18",
     "@types/react": "17",
     "@types/rimraf": "^3.0.2",
+    "@types/semver": "^7.3.9",
     "@types/tmp": "^0.2.3",
     "@types/yargs-parser": "^20.2.1",
     "@typescript-eslint/eslint-plugin": "5.12.1",

packages/core/src/generators/app/schema.json

@@ -27,8 +27,7 @@
     },
     "template": {
       "type": "string",
-      "description": "The template to instantiate when the command is invoked. Each template might have specific options you can pass.",
-      "x-prompt": "What template should the project be initialized with? (https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-new#template-options)"
+      "description": "The template to instantiate when the command is invoked. Each template might have specific options you can pass."
     },
     "language": {
       "type": "string",
@@ -80,5 +79,5 @@
       "aliases": ["solution", "s"]
     }
   },
-  "required": ["name", "template", "language"]
+  "required": ["name", "language"]
 }

packages/core/src/generators/init/generator.ts

@@ -8,7 +8,7 @@ import {
 } from '@nrwl/devkit';
 
 import { DotNetClient, dotnetFactory } from '@nx-dotnet/dotnet';
-import { CONFIG_FILE_PATH, NxDotnetConfig } from '@nx-dotnet/utils';
+import { CONFIG_FILE_PATH, isDryRun, NxDotnetConfig } from '@nx-dotnet/utils';
 
 export async function initGenerator(
   host: Tree,
@@ -73,7 +73,7 @@ function updateNxJson(host: Tree) {
 
 function initToolManifest(host: Tree, dotnetClient: DotNetClient) {
   const initialized = host.exists('.config/dotnet-tools.json');
-  if (!initialized) {
+  if (!initialized && !isDryRun()) {
     console.log('Tool Manifest created for managing local .NET tools');
     dotnetClient.new('tool-manifest');
   }

packages/core/src/generators/lib/schema.json

@@ -27,8 +27,7 @@
     },
     "template": {
       "type": "string",
-      "description": "The template to instantiate when the command is invoked. Each template might have specific options you can pass.",
-      "x-prompt": "What template should the project be initialized with? (https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-new#template-options)"
+      "description": "The template to instantiate when the command is invoked. Each template might have specific options you can pass."
     },
     "language": {
       "type": "string",
@@ -74,5 +73,5 @@
       "aliases": ["solution", "s"]
     }
   },
-  "required": ["name", "template", "language", "testTemplate"]
+  "required": ["name", "language", "testTemplate"]
 }

packages/core/src/generators/test/generator.ts

@@ -7,7 +7,7 @@ import { normalizeOptions } from '../utils/generate-project';
 import { GenerateTestProject } from '../utils/generate-test-project';
 import { NxDotnetGeneratorSchema } from './schema';
 
-export default function (
+export default async function (
   host: Tree,
   options: NxDotnetGeneratorSchema,
   dotnetClient = new DotNetClient(dotnetFactory()),
@@ -34,7 +34,10 @@ export default function (
     projectType: project.projectType ?? 'library',
   };
 
-  const normalizedOptions = normalizeOptions(host, projectGeneratorOptions);
+  const normalizedOptions = await normalizeOptions(
+    host,
+    projectGeneratorOptions,
+  );
 
   return GenerateTestProject(host, normalizedOptions, dotnetClient);
 }

packages/core/src/generators/utils/generate-project.spec.ts

@@ -32,6 +32,15 @@ describe('nx-dotnet project generator', () => {
       standalone: false,
       projectType: 'application',
     };
+
+    jest.spyOn(dotnetClient, 'listInstalledTemplates').mockReturnValue([
+      {
+        shortNames: ['classlib'],
+        templateName: 'Class Library',
+        languages: ['C#'],
+        tags: [],
+      },
+    ]);
   });
 
   afterEach(async () => {

packages/core/src/generators/utils/generate-project.ts

@@ -17,7 +17,11 @@ import { readFileSync, writeFileSync } from 'fs';
 import { dirname, relative } from 'path';
 import { XmlDocument } from 'xmldoc';
 
-import { DotNetClient, dotnetNewOptions } from '@nx-dotnet/dotnet';
+import {
+  DotNetClient,
+  dotnetNewOptions,
+  KnownDotnetTemplates,
+} from '@nx-dotnet/dotnet';
 import { findProjectFileInPath, isDryRun, resolve } from '@nx-dotnet/utils';
 
 import {
@@ -29,24 +33,27 @@ import {
 import initSchematic from '../init/generator';
 import { GenerateTestProject } from './generate-test-project';
 import { addToSolutionFile } from './add-to-sln';
+import { promptForTemplate } from './prompt-for-template';
 
-export interface NormalizedSchema extends NxDotnetProjectGeneratorSchema {
+export interface NormalizedSchema
+  extends Omit<NxDotnetProjectGeneratorSchema, 'template'> {
   projectName: string;
   projectRoot: string;
   projectDirectory: string;
   projectLanguage: string;
-  projectTemplate: string;
+  projectTemplate: KnownDotnetTemplates;
   parsedTags: string[];
   className: string;
   namespaceName: string;
   projectType?: ProjectType;
 }
 
-export function normalizeOptions(
+export async function normalizeOptions(
   host: Tree,
   options: NxDotnetProjectGeneratorSchema,
+  client?: DotNetClient,
   projectType?: ProjectType,
-): NormalizedSchema {
+): Promise<NormalizedSchema> {
   const name = names(options.name).fileName;
   const className = names(options.name).className;
   const projectDirectory = options.directory
@@ -62,6 +69,10 @@ export function normalizeOptions(
     ? options.tags.split(',').map((s) => s.trim())
     : [];
 
+  const template = client
+    ? await promptForTemplate(client, options.template, options.language)
+    : options.template;
+
   const npmScope = names(
     readWorkspaceConfiguration(host).npmScope || '',
   ).className;
@@ -81,7 +92,7 @@ export function normalizeOptions(
     projectDirectory,
     parsedTags,
     projectLanguage: options.language,
-    projectTemplate: options.template,
+    projectTemplate: template as KnownDotnetTemplates,
     namespaceName,
     projectType: projectType ?? options.projectType ?? 'library',
   };
@@ -113,7 +124,12 @@ export async function GenerateProject(
 
   options.testTemplate = options.testTemplate ?? 'none';
 
-  const normalizedOptions = normalizeOptions(host, options, projectType);
+  const normalizedOptions = await normalizeOptions(
+    host,
+    options,
+    dotnetClient,
+    projectType,
+  );
 
   const projectConfiguration: ProjectConfiguration = {
     root: normalizedOptions.projectRoot,
@@ -146,7 +162,7 @@ export async function GenerateProject(
     newParams['dryRun'] = true;
   }
 
-  dotnetClient.new(normalizedOptions.template, newParams);
+  dotnetClient.new(normalizedOptions.projectTemplate, newParams);
   if (!isDryRun()) {
     addToSolutionFile(
       host,

packages/core/src/generators/utils/generate-test-project.spec.ts

@@ -74,7 +74,6 @@ describe('nx-dotnet test project generator', () => {
 
     options = {
       name: 'domain-existing-app',
-      template: 'xunit',
       testTemplate: 'xunit',
       language: 'C#',
       skipOutputPathManipulation: true,