fix(@angular/cli): populate path with working directory in nested schematics

With this change we change the how we handle `"format": "path"` schematic property option. We replace the formatter in favour of a `SmartDefaultProvider`, which ensures that nested schematics can access the `workingDirectory`.

Author: alan-agius4

packages/angular/cli/src/command-builder/command-module.ts

@@ -6,7 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import { analytics, logging, normalize, schema, strings } from '@angular-devkit/core';
+import { analytics, logging, schema, strings } from '@angular-devkit/core';
 import { readFileSync } from 'fs';
 import * as path from 'path';
 import {
@@ -197,8 +197,6 @@ export abstract class CommandModule<T extends {} = {}> implements CommandModuleI
    * **Note:** This method should be called from the command bundler method.
    */
   protected addSchemaOptionsToCommand<T>(localYargs: Argv<T>, options: Option[]): Argv<T> {
-    const workingDir = normalize(path.relative(this.context.root, process.cwd()));
-
     for (const option of options) {
       const {
         default: defaultVal,
@@ -211,7 +209,6 @@ export abstract class CommandModule<T extends {} = {}> implements CommandModuleI
         hidden,
         name,
         choices,
-        format,
       } = option;
 
       const sharedOptions: YargsOptions & PositionalOptions = {
@@ -224,11 +221,6 @@ export abstract class CommandModule<T extends {} = {}> implements CommandModuleI
         ...(this.context.args.options.help ? { default: defaultVal } : {}),
       };
 
-      // Special case for schematics
-      if (workingDir && format === 'path' && name === 'path' && hidden) {
-        sharedOptions.default = workingDir;
-      }
-
       if (positional === undefined) {
         localYargs = localYargs.option(strings.dasherize(name), {
           type,

packages/angular/cli/src/command-builder/schematics-command-module.ts

@@ -6,15 +6,15 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import { schema, tags } from '@angular-devkit/core';
+import { normalize as devkitNormalize, schema, tags } from '@angular-devkit/core';
 import { Collection, UnsuccessfulWorkflowExecution, formats } from '@angular-devkit/schematics';
 import {
   FileSystemCollectionDescription,
   FileSystemSchematicDescription,
   NodeWorkflow,
 } from '@angular-devkit/schematics/tools';
 import type { CheckboxQuestion, Question } from 'inquirer';
-import { resolve } from 'path';
+import { relative, resolve } from 'path';
 import { Argv } from 'yargs';
 import { getProjectByCwd, getSchematicDefaults } from '../utilities/config';
 import { memoize } from '../utilities/memoize';
@@ -133,8 +133,11 @@ export abstract class SchematicsCommandModule
     });
 
     workflow.registry.addPostTransform(schema.transforms.addUndefinedDefaults);
-    workflow.registry.addSmartDefaultProvider('projectName', () => this.getProjectName());
     workflow.registry.useXDeprecatedProvider((msg) => logger.warn(msg));
+    workflow.registry.addSmartDefaultProvider('projectName', () => this.getProjectName());
+    workflow.registry.addSmartDefaultProvider('workingDirectory', () =>
+      devkitNormalize(relative(this.context.root, process.cwd())),
+    );
 
     let shouldReportAnalytics = true;
     workflow.engineHost.registerOptionsTransform(async (schematic, options) => {
@@ -356,9 +359,9 @@ export abstract class SchematicsCommandModule
     if (typeof defaultProjectName === 'string' && defaultProjectName) {
       if (!this.defaultProjectDeprecationWarningShown) {
         logger.warn(tags.oneLine`
-            DEPRECATED: The 'defaultProject' workspace option has been deprecated.
-            The project to use will be determined from the current working directory.
-          `);
+             DEPRECATED: The 'defaultProject' workspace option has been deprecated.
+             The project to use will be determined from the current working directory.
+           `);
 
         this.defaultProjectDeprecationWarningShown = true;
       }

packages/schematics/angular/app-shell/schema.json

@@ -27,19 +27,16 @@
     },
     "main": {
       "type": "string",
-      "format": "path",
       "description": "The name of the main entry-point file.",
       "default": "main.server.ts"
     },
     "appDir": {
       "type": "string",
-      "format": "path",
       "description": "The name of the application directory.",
       "default": "app"
     },
     "rootModuleFileName": {
       "type": "string",
-      "format": "path",
       "description": "The name of the root module file",
       "default": "app.server.module.ts"
     },

packages/schematics/angular/class/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the class, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/component/index.ts

@@ -115,7 +115,7 @@ export default function (options: ComponentOptions): Rule {
       throw new SchematicsException(`Project "${options.project}" does not exist.`);
     }
 
-    if (options.path === undefined) {
+    if (!options.path) {
       options.path = buildDefaultPath(project);
     }
 

packages/schematics/angular/component/schema.json

@@ -8,7 +8,9 @@
   "properties": {
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the component file, relative to the current workspace. Default is a folder with the same name as the component in the project root.",
       "visible": false
     },

packages/schematics/angular/directive/index.ts

@@ -104,7 +104,7 @@ export default function (options: DirectiveOptions): Rule {
       throw new SchematicsException(`Project "${options.project}" does not exist.`);
     }
 
-    if (options.path === undefined) {
+    if (!options.path) {
       options.path = buildDefaultPath(project);
     }
 

packages/schematics/angular/directive/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the interface that defines the directive, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/enum/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the enum definition, relative to the current workspace.",
       "visible": false
     },

packages/schematics/angular/guard/schema.json

@@ -28,7 +28,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the interface that defines the guard, relative to the current workspace.",
       "visible": false
     },

packages/schematics/angular/interceptor/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the interceptor, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/interface/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the interface, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/library/schema.json

@@ -19,7 +19,9 @@
     },
     "entryFile": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the library's public API file, relative to the workspace root.",
       "default": "public-api"
     },

packages/schematics/angular/module/index.ts

@@ -131,7 +131,7 @@ function buildRoute(options: ModuleOptions, modulePath: string) {
 
 export default function (options: ModuleOptions): Rule {
   return async (host: Tree) => {
-    if (options.path === undefined) {
+    if (!options.path) {
       options.path = await createDefaultPath(host, options.project as string);
     }
 

packages/schematics/angular/module/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the NgModule, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/pipe/index.ts

@@ -84,7 +84,7 @@ function addDeclarationToNgModule(options: PipeOptions): Rule {
 
 export default function (options: PipeOptions): Rule {
   return async (host: Tree) => {
-    if (options.path === undefined) {
+    if (!options.path) {
       options.path = await createDefaultPath(host, options.project as string);
     }
 

packages/schematics/angular/pipe/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the pipe, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/resolver/schema.json

@@ -28,7 +28,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the interface that defines the resolver, relative to the current workspace.",
       "visible": false
     },

packages/schematics/angular/service/schema.json

@@ -17,7 +17,9 @@
     },
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the service, relative to the workspace root.",
       "visible": false
     },

packages/schematics/angular/universal/schema.json

@@ -21,19 +21,19 @@
     },
     "main": {
       "type": "string",
-      "format": "path",
       "description": "The name of the main entry-point file.",
       "default": "main.server.ts"
     },
     "appDir": {
       "type": "string",
-      "format": "path",
       "description": "The name of the application folder.",
       "default": "app"
     },
     "rootModuleFileName": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The name of the root NgModule file.",
       "default": "app.server.module.ts"
     },

packages/schematics/angular/web-worker/index.ts

@@ -30,7 +30,7 @@ function addSnippet(options: WebWorkerOptions): Rule {
   return (host: Tree, context: SchematicContext) => {
     context.logger.debug('Updating appmodule');
 
-    if (options.path === undefined) {
+    if (!options.path) {
       return;
     }
 
@@ -90,7 +90,7 @@ export default function (options: WebWorkerOptions): Rule {
       throw new SchematicsException(`Web Worker requires a project type of "application".`);
     }
 
-    if (options.path === undefined) {
+    if (!options.path) {
       options.path = buildDefaultPath(project);
     }
     const parsedPath = parseName(options.path, options.name);

packages/schematics/angular/web-worker/schema.json

@@ -8,7 +8,9 @@
   "properties": {
     "path": {
       "type": "string",
-      "format": "path",
+      "$default": {
+        "$source": "workingDirectory"
+      },
       "description": "The path at which to create the worker file, relative to the current workspace.",
       "visible": false
     },