refactor(schematics): remove ng-add skipPackageJson option (#13233)

* Removes the `skipPackageJson` option from the `ng-add` schematic. The option does not work properly because `ng add` by default saves the dependency to the `package.json` file. Also we have no way to disable saving when installing the CDK through the `NodePackageInstallTask`. See for reference: angular/angular-cli@729d407

Author: devversion

src/cdk/schematics/utils/index.ts

@@ -9,7 +9,6 @@
 export * from './ast';
 export * from './build-component';
 export * from './get-project';
-export * from './package-json';
 export * from './parse5-element';
 export * from './project-main-file';
 export * from './project-style-file';

src/lib/schematics/ng-add/index.spec.ts

@@ -44,17 +44,6 @@ describe('ng-add schematic', () => {
     expect(runner.tasks.some(task => task.name === 'run-schematic')).toBe(true);
   });
 
-  it('should not set up dependencies if skipPackageJson is specified', () => {
-    const tree = runner.runSchematic('ng-add', {skipPackageJson: true}, appTree);
-    const packageJson = JSON.parse(getFileContent(tree, '/package.json'));
-    const dependencies = packageJson.dependencies;
-
-    expect(dependencies['@angular/material']).toBeUndefined();
-    expect(dependencies['@angular/cdk']).toBeUndefined();
-
-    expect(runner.tasks.some(task => task.name === 'run-schematic')).toBe(true);
-  });
-
   it('should add hammerjs import to project main file', () => {
     const tree = runner.runSchematic('ng-add-setup-project', {}, appTree);
     const fileContent = getFileContent(tree, '/projects/material/src/main.ts');

src/lib/schematics/ng-add/index.ts

@@ -6,9 +6,9 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import {Rule, Tree, SchematicContext, TaskId} from '@angular-devkit/schematics';
+import {Rule, SchematicContext, Tree} from '@angular-devkit/schematics';
 import {NodePackageInstallTask, RunSchematicTask} from '@angular-devkit/schematics/tasks';
-import {addPackageToPackageJson, getPackageVersionFromPackageJson} from './package-json';
+import {addPackageToPackageJson, getPackageVersionFromPackageJson} from './package-config';
 import {Schema} from './schema';
 import {hammerjsVersion, materialVersion, requiredAngularVersionRange} from './version-names';
 
@@ -21,32 +21,24 @@ import {hammerjsVersion, materialVersion, requiredAngularVersionRange} from './v
  */
 export default function(options: Schema): Rule {
   return (host: Tree, context: SchematicContext) => {
-    // Since the Angular Material schematics depend on the schematic utility functions from the
-    // CDK, we need to install the CDK before loading the schematic files that import from the CDK.
-    let installTaskId: TaskId;
-
-    if (!options.skipPackageJson) {
-      // Version tag of the `@angular/core` dependency that has been loaded from the `package.json`
-      // of the CLI project. This tag should be preferred because all Angular dependencies should
-      // have the same version tag if possible.
-      const ngCoreVersionTag = getPackageVersionFromPackageJson(host, '@angular/core');
-
-      addPackageToPackageJson(host, '@angular/cdk', `^${materialVersion}`);
-      addPackageToPackageJson(host, '@angular/material', `^${materialVersion}`);
-      addPackageToPackageJson(host, '@angular/animations',
-          ngCoreVersionTag || requiredAngularVersionRange);
+    // Version tag of the `@angular/core` dependency that has been loaded from the `package.json`
+    // of the CLI project. This tag should be preferred because all Angular dependencies should
+    // have the same version tag if possible.
+    const ngCoreVersionTag = getPackageVersionFromPackageJson(host, '@angular/core');
 
-      if (options.gestures) {
-        addPackageToPackageJson(host, 'hammerjs', hammerjsVersion);
-      }
+    addPackageToPackageJson(host, '@angular/cdk', `^${materialVersion}`);
+    addPackageToPackageJson(host, '@angular/material', `^${materialVersion}`);
+    addPackageToPackageJson(host, '@angular/animations',
+        ngCoreVersionTag || requiredAngularVersionRange);
 
-      installTaskId = context.addTask(new NodePackageInstallTask());
-    } else {
-      installTaskId = context.addTask(new NodePackageInstallTask({
-        packageName: `@angular/cdk@^${materialVersion}`
-      }));
+    if (options.gestures) {
+      addPackageToPackageJson(host, 'hammerjs', hammerjsVersion);
     }
 
+    // Since the Angular Material schematics depend on the schematic utility functions from the
+    // CDK, we need to install the CDK before loading the schematic files that import from the CDK.
+    const installTaskId = context.addTask(new NodePackageInstallTask());
+
     context.addTask(new RunSchematicTask('ng-add-setup-project', options), [installTaskId]);
   };
 }

src/cdk/schematics/utils/package-json.ts renamed to src/lib/schematics/ng-add/package-config.ts

@@ -11,11 +11,6 @@
         "$source": "projectName"
       }
     },
-    "skipPackageJson": {
-      "type": "boolean",
-      "default": false,
-      "description": "Do not add materials dependencies to package.json (e.g., --skipPackageJson)"
-    },
     "theme": {
       "enum": ["indigo-pink", "deeppurple-amber", "pink-bluegrey", "purple-green", "custom"],
       "default": "indigo-pink",

src/lib/schematics/ng-add/package-json.ts

@@ -11,9 +11,6 @@ export interface Schema {
   /** Name of the project to target. */
   project: string;
 
-  /** Whether to skip package.json install. */
-  skipPackageJson: boolean;
-
   /** Whether gesture support should be set up or not. */
   gestures: boolean;