refactor(@angular/cli): remove ajv peer warning suppression

Author: clydin

package-lock.json

@@ -46,7 +46,6 @@
     "@angular-devkit/schematics": "0.5.0",
     "@schematics/angular": "0.5.0",
     "@schematics/update": "0.5.0",
-    "ajv": "^6.1.1",
     "chalk": "~2.2.0",
     "common-tags": "^1.3.1",
     "core-object": "^3.1.0",
@@ -74,6 +73,7 @@
     "@types/node": "^8.9.1",
     "@types/request": "~2.0.0",
     "@types/semver": "^5.3.30",
+    "ajv": "^6.1.1",
     "conventional-changelog": "1.1.0",
     "eslint": "^3.11.0",
     "express": "^4.14.0",

package.json

@@ -1,10 +1,3 @@
-// TODO: remove this commented AJV require.
-// We don't actually require AJV, but there is a bug with NPM and peer dependencies that is
-// whose workaround is to depend on AJV.
-// See https://github.com/angular/angular-cli/issues/9691#issuecomment-367322703 for details.
-// We need to add a require here to satisfy the dependency checker.
-// require('ajv');
-
 import * as path from 'path';
 
 const cli = require('../../ember-cli/lib/cli');

packages/@angular/cli/lib/cli/index.ts

@@ -32,7 +32,6 @@
     "@angular-devkit/schematics": "0.5.0",
     "@schematics/angular": "0.5.0",
     "@schematics/update": "0.5.0",
-    "ajv": "^6.1.1",
     "chalk": "~2.2.0",
     "common-tags": "^1.3.1",
     "core-object": "^3.1.0",