build: remove npm_package rule for packages/bazel (#61826)

devversion · pkozlowski-opensource · commit 3af65af6a1fa · 2025-06-03T11:41:52.000+02:00
The bazel npm package is no longer needed and currently breaks as we started using a 1P pnpm linked package that would otherwise end up being pulled into this legacy rule. PR Close #61826
diff --git a/packages/bazel/BUILD.bazel b/packages/bazel/BUILD.bazel
@@ -1,38 +0,0 @@
-# BEGIN-DEV-ONLY
-load("//tools:defaults.bzl", "pkg_npm")
-
-pkg_npm(
-    name = "npm_package",
-    package_name = "@angular/bazel",
-    srcs = glob(
-        ["*"],
-        exclude = ["yarn.lock"],
-    ) + [
-        "//packages/bazel/src:package_assets",
-        "//packages/bazel/src/ng_module:package_assets",
-        "//packages/bazel/src/ng_package:package_assets",
-        "//packages/bazel/src/ngc-wrapped:package_assets",
-        "//packages/bazel/third_party/github.com/bazelbuild/bazel/src/main/protobuf:package_assets",
-    ],
-    substitutions = {
-        "(#|//)\\s+BEGIN-DEV-ONLY[\\w\\W]+?(#|//)\\s+END-DEV-ONLY": "",
-        # The partial compilation transition is applied in the user workspace. We do not want to require
-        # users having to define the build setting, so we directly point the flag label to the place where
-        # we expect `@angular/bazel` being located. Note: This expects the `@npm//` workspace to be used.
-        "//packages/bazel/src:partial_compilation": "@npm//@angular/bazel/src:partial_compilation",
-        # Substitutions to account for label changes when `@angular/bazel` is consumed externally.
-        # NodeJS binaries are pre-built and should be consumed through the NPM bin scripts of the package.
-        "//packages/bazel/": "//@angular/bazel/",
-        "@npm//@bazel/concatjs/internal:": "//@bazel/concatjs/internal:",
-    },
-    # Do not add more to this list.
-    # Dependencies on the full npm_package cause long re-builds.
-    visibility = [
-        "//integration:__subpackages__",
-    ],
-    deps = [
-        "//packages/bazel/src/ng_package:lib",
-        "//packages/bazel/src/ngc-wrapped:ngc_lib",
-    ],
-)
-# END-DEV-ONLY
diff --git a/scripts/build/package-builder.mts b/scripts/build/package-builder.mts
@@ -42,11 +42,7 @@ export function performNpmReleaseBuild(): BuiltPackage[] {
  * Git HEAD SHA is included in the version (for easier debugging and back tracing).
  */
 export function performDefaultSnapshotBuild(): BuiltPackage[] {
-  return buildReleasePackages(defaultDistPath, /* isSnapshotBuild */ true, [
-    // For snapshot builds, the Bazel package is still built. We want to have
-    // GitHub snapshot builds for it.
-    '//packages/bazel:npm_package',
-  ]);
+  return buildReleasePackages(defaultDistPath, /* isSnapshotBuild */ true);
 }
 
 /**
