build: no longer install snapshot build of @angular/bazel (#23429)

* build: no longer install snapshot build of `@angular/bazel`

No longer uses a snapshot build for `@angular/bazel`. We
previously used a snapshot build in order to update to
v4 of `rules_nodejs`.

* build: add missing dependency for jsonc-parser

Sets up the jsonc-parser dependency in the BUILD bazel file.

Author: devversion

package.json

@@ -73,7 +73,7 @@
     "@angular-devkit/build-optimizer": "0.1201.0",
     "@angular-devkit/core": "12.1.2",
     "@angular-devkit/schematics": "12.1.2",
-    "@angular/bazel": "https://github.com/angular/bazel-builds.git#2566cc859235399edc72749a957f1cdcba2dc121",
+    "@angular/bazel": "12.1.2",
     "@angular/compiler-cli": "12.1.2",
     "@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#56655ed6ccf24b7f3ad6b9ccdd1ef2896c7fa9f4",
     "@angular/localize": "12.1.2",
@@ -183,6 +183,7 @@
     "inquirer": "^8.0.0",
     "jasmine": "^3.6.0",
     "jasmine-core": "^3.6.0",
+    "jsonc-parser": "^3.0.0",
     "kagekiri": "^1.4.1",
     "karma": "^4.4.1",
     "karma-browserstack-launcher": "^1.6.0",

src/cdk/schematics/testing/BUILD.bazel

@@ -21,6 +21,7 @@ ts_library(
         "@npm//@types/jasmine",
         "@npm//@types/node",
         "@npm//fs-extra",
+        "@npm//jsonc-parser",
         "@npm//rxjs",
     ],
 )

src/cdk/schematics/testing/test-case-setup.ts

@@ -5,7 +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 {getSystemPath, JsonParseMode, parseJson, Path} from '@angular-devkit/core';
+
+import {parse} from 'jsonc-parser';
+import {getSystemPath,  Path} from '@angular-devkit/core';
 import {HostTree, Tree} from '@angular-devkit/schematics';
 import {SchematicTestRunner, UnitTestTree} from '@angular-devkit/schematics/testing';
 import {readFileSync} from 'fs-extra';
@@ -82,10 +84,10 @@ export async function createTestCaseSetup(migrationName: string, collectionPath:
   });
 
   const testAppTsconfigPath = 'projects/cdk-testing/tsconfig.app.json';
-  // Parse TypeScript configuration files with JSON5 as they could contain comments or
-  // unquoted properties.
-  const testAppTsconfig =
-      parseJson(appTree.readContent(testAppTsconfigPath), JsonParseMode.Json5) as any;
+  // Parse TypeScript configuration files with JSONC (like the CLI does) as the
+  // config files could contain comments or trailing commas
+  const testAppTsconfig = parse(appTree.readContent(testAppTsconfigPath), [],
+      {allowTrailingComma: true});
 
   // include all TypeScript files in the project. Otherwise all test input
   // files won't be part of the program and cannot be migrated.

yarn.lock

@@ -70,11 +70,12 @@
   dependencies:
     tslib "^2.2.0"
 
-"@angular/bazel@https://github.com/angular/bazel-builds.git#2566cc859235399edc72749a957f1cdcba2dc121":
-  version "12.2.0-next.1"
-  resolved "https://github.com/angular/bazel-builds.git#2566cc859235399edc72749a957f1cdcba2dc121"
+"@angular/[email protected]":
+  version "12.1.2"
+  resolved "https://registry.yarnpkg.com/@angular/bazel/-/bazel-12.1.2.tgz#be3219cb7a573b95c47184cb5904ada4ffb7b135"
+  integrity sha512-cSP2j2tZxQPw8YYQ7SkNCz7iUkz8I3tyUX2VZO717vLBDGdYgjtgJ3+A6sgLraCL0e+4pj8xRa5m8G8ovNLVrQ==
   dependencies:
-    "@microsoft/api-extractor" "7.18.1"
+    "@microsoft/api-extractor" "7.17.1"
     shelljs "0.8.4"
     tsickle "^0.38.0"
     tslib "^2.2.0"
@@ -1395,10 +1396,10 @@
     "@microsoft/tsdoc-config" "~0.15.2"
     "@rushstack/node-core-library" "3.40.0"
 
-"@microsoft/api-extractor@7.18.1":
-  version "7.18.1"
-  resolved "https://registry.yarnpkg.com/@microsoft/api-extractor/-/api-extractor-7.18.1.tgz#61b39f972b646261dd49f2de9f5d448aa6497e7a"
-  integrity sha512-qljUF2Q0zAx1vJrjKkJVGN7OVbsXki+Pji99jywyl6L/FK3YZ7PpstUJYE6uBcLPy6rhNPWPAsHNTMpG/kHIsg==
+"@microsoft/api-extractor@7.17.1":
+  version "7.17.1"
+  resolved "https://registry.yarnpkg.com/@microsoft/api-extractor/-/api-extractor-7.17.1.tgz#df6b7aa7cadcb35e2483048ebe5dbaf7af932446"
+  integrity sha512-NqdN627QoDdDtvfqEoX+Z6WbF2yid5URUZVYefC9KuisgwrsfvbSj5QwzOZlMYcufoafTq9jfeVyFAXr4PqGig==
   dependencies:
     "@microsoft/api-extractor-model" "7.13.3"
     "@microsoft/tsdoc" "0.13.2"
@@ -1585,7 +1586,7 @@
   resolved "https://registry.yarnpkg.com/@octokit/openapi-types/-/openapi-types-9.2.0.tgz#46bbfe6a85bfd2987e69216955fcd04df7d025bb"
   integrity sha512-c4A1Xm0At+ypvBfEETREu519wLncJYQXvY+dBGg/V5YA51eg5EwdDsPPfcOMG0cuXscqRvsIgIySTmTJUdcTNA==
 
-"@octokit/openapi-types@^9.5.0":
+"@octokit/openapi-types@^9.4.0", "@octokit/openapi-types@^9.5.0":
   version "9.7.0"
   resolved "https://registry.yarnpkg.com/@octokit/openapi-types/-/openapi-types-9.7.0.tgz#9897cdefd629cd88af67b8dbe2e5fb19c63426b2"
   integrity sha512-TUJ16DJU8mekne6+KVcMV5g6g/rJlrnIKn7aALG9QrNpnEipFc1xjoarh0PKaAWf2Hf+HwthRKYt+9mCm5RsRg==
@@ -8549,7 +8550,7 @@ json5@^2.1.0, json5@^2.1.2:
   dependencies:
     minimist "^1.2.5"
 
-[email protected]:
+[email protected], jsonc-parser@^3.0.0:
   version "3.0.0"
   resolved "https://registry.yarnpkg.com/jsonc-parser/-/jsonc-parser-3.0.0.tgz#abdd785701c7e7eaca8a9ec8cf070ca51a745a22"
   integrity sha512-fQzRfAbIBnR0IQvftw9FJveWiHp72Fg20giDrHz6TdfB12UH/uue0D3hm57UB5KgAVuniLMCaS8P1IMj9NR7cA==