feat(@ngtools/webpack): move @angular/compiler-cli to a peer dependency

Now that the package is no longer installed globally, it can now properly use peer dependencies without issue.

Author: clydin

packages/ngtools/webpack/package.json

@@ -31,10 +31,13 @@
     "webpack-sources": "1.2.0"
   },
   "peerDependencies": {
+    "@angular/compiler-cli": ">=5.0.0 <8.0.0 || ^7.0.0-beta.0",
     "typescript": ">=2.4.0 < 3.1",
     "webpack": "^4.0.0"
   },
   "devDependencies": {
+    "@angular/compiler": "^7.0.0-beta.4",
+    "@angular/compiler-cli": "^7.0.0-beta.4",
     "typescript": "~3.0.1",
     "webpack": "^4.0.0"
   }

packages/ngtools/webpack/src/angular_compiler_plugin.ts

@@ -6,18 +6,7 @@
  * found in the LICENSE file at https://angular.io/license
  */
 import { Path, dirname, getSystemPath, normalize, resolve, virtualFs } from '@angular-devkit/core';
-import { ChildProcess, ForkOptions, fork } from 'child_process';
-import * as fs from 'fs';
-import * as path from 'path';
-import * as ts from 'typescript';
-import { Compiler, compilation } from 'webpack';
-import { time, timeEnd } from './benchmark';
-import { WebpackCompilerHost, workaroundResolve } from './compiler_host';
-import { resolveEntryModuleFromMain } from './entry_resolver';
-import { gatherDiagnostics, hasErrors } from './gather_diagnostics';
-import { LazyRouteMap, findLazyRoutes } from './lazy_routes';
 import {
-  CompilerCliIsSupported,
   CompilerHost,
   CompilerOptions,
   DEFAULT_ERROR_CODE,
@@ -32,7 +21,17 @@ import {
   createProgram,
   formatDiagnostics,
   readConfiguration,
-} from './ngtools_api';
+} from '@angular/compiler-cli';
+import { ChildProcess, ForkOptions, fork } from 'child_process';
+import * as fs from 'fs';
+import * as path from 'path';
+import * as ts from 'typescript';
+import { Compiler, compilation } from 'webpack';
+import { time, timeEnd } from './benchmark';
+import { WebpackCompilerHost, workaroundResolve } from './compiler_host';
+import { resolveEntryModuleFromMain } from './entry_resolver';
+import { gatherDiagnostics, hasErrors } from './gather_diagnostics';
+import { LazyRouteMap, findLazyRoutes } from './lazy_routes';
 import { resolveWithPaths } from './paths-plugin';
 import { WebpackResourceLoader } from './resource_loader';
 import {
@@ -152,7 +151,6 @@ export class AngularCompilerPlugin {
   }
 
   constructor(options: AngularCompilerPluginOptions) {
-    CompilerCliIsSupported();
     this._options = Object.assign({}, options);
     this._setupOptions(this._options);
   }

packages/ngtools/webpack/src/gather_diagnostics.ts

@@ -5,9 +5,9 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+import { Diagnostic, Diagnostics, Program } from '@angular/compiler-cli';
 import * as ts from 'typescript';
 import { time, timeEnd } from './benchmark';
-import { Diagnostic, Diagnostics, Program } from './ngtools_api';
 
 
 export class CancellationToken implements ts.CancellationToken {

packages/ngtools/webpack/src/ngtools_api.ts

@@ -7,18 +7,18 @@
  */
 import { terminal } from '@angular-devkit/core';
 import { NodeJsSyncHost } from '@angular-devkit/core/node';
-import * as ts from 'typescript';
-import { time, timeEnd } from './benchmark';
-import { WebpackCompilerHost } from './compiler_host';
-import { CancellationToken, gatherDiagnostics } from './gather_diagnostics';
 import {
   CompilerHost,
   CompilerOptions,
   Program,
   createCompilerHost,
   createProgram,
   formatDiagnostics,
-} from './ngtools_api';
+} from '@angular/compiler-cli';
+import * as ts from 'typescript';
+import { time, timeEnd } from './benchmark';
+import { WebpackCompilerHost } from './compiler_host';
+import { CancellationToken, gatherDiagnostics } from './gather_diagnostics';
 
 
 // This file should run in a child process with the AUTO_START_ARG argument