refactor: replace usage of ng_package with new ng_package from rules_angular

Author: josephperrott

WORKSPACE

@@ -274,3 +274,24 @@ esbuild_register_toolchains(
     name = "esbuild",
     esbuild_version = LATEST_ESBUILD_VERSION,
 )
+
+git_repository(
+    name = "rules_angular",
+    commit = "f6fa3a357d3c2999618be83b1350a9e11dee5588",
+    remote = "https://github.com/devversion/rules_angular.git",
+)
+
+load("@rules_angular//setup:step_1.bzl", "rules_angular_step1")
+
+rules_angular_step1()
+
+load("@rules_angular//setup:step_2.bzl", "rules_angular_step2")
+
+rules_angular_step2()
+
+load("@rules_angular//setup:step_3.bzl", "rules_angular_step3")
+
+rules_angular_step3(
+    angular_compiler_cli = "//:node_modules/@angular/compiler-cli",
+    typescript = "//:node_modules/typescript",
+)

package.json

@@ -125,6 +125,9 @@
     "listr2": "8.3.2",
     "lodash": "^4.17.21",
     "npm": "^11.0.0",
+    "magic-string": "0.30.17",
+    "rollup-plugin-dts": "6.2.1",
+    "rollup-plugin-sourcemaps2": "0.5.0",
     "prettier": "^3.0.0",
     "protractor": "~7.0.0",
     "puppeteer": "18.2.1",

packages/angular/ssr/BUILD.bazel

@@ -1,4 +1,3 @@
-load("@aspect_rules_js//npm:defs.bzl", "npm_package")
 load("@devinfra//bazel/api-golden:index_rjs.bzl", "api_golden_test_npm_package")
 load("@npm2//:defs.bzl", "npm_link_all_packages")
 load("@rules_pkg//:pkg.bzl", "pkg_tar")
@@ -40,8 +39,7 @@ ts_project(
 )
 
 ng_package(
-    name = "angular_package",
-    package_name = "@angular/ssr",
+    name = "pkg",
     srcs = [
         ":package.json",
         "//packages/angular/ssr/third_party/beasties:beasties_bundled",
@@ -54,34 +52,19 @@ ng_package(
     nested_packages = [
         "//packages/angular/ssr/schematics:pkg",
     ],
-    deps = [
-        ":ssr_legacy",
-        "//packages/angular/ssr/node:node_legacy",
+    package = "@angular/ssr",
+    rollup_runtime_deps = [
+        "//:node_modules/@rollup/plugin-commonjs",
+        "//:node_modules/@rollup/plugin-node-resolve",
+        "//:node_modules/magic-string",
+        "//:node_modules/rollup-plugin-dts",
+        "//:node_modules/rollup-plugin-sourcemaps2",
     ],
-)
-
-pkg_tar(
-    name = "npm_package_archive",
-    srcs = [":npm_package"],
-    extension = "tgz",
-    strip_prefix = "./npm_package",
-    # should not be built unless it is a dependency of another rule
-    tags = ["manual"],
-)
-
-# TODO: Replace when `ng_package` creates a valid `rules_js`-compliant npm package.
-npm_package(
-    name = "npm_package",
-    srcs = [":angular_package"],
-    replace_prefixes = {
-        "angular_package/": "",
-    },
     tags = ["release-package"],
-)
-
-alias(
-    name = "pkg",
-    actual = ":npm_package",
+    deps = [
+        ":ssr",
+        "//packages/angular/ssr/node",
+    ],
 )
 
 api_golden_test_npm_package(

packages/angular/ssr/schematics/BUILD.bazel

@@ -3,7 +3,7 @@
 # Use of this source code is governed by an MIT-style license that can be
 # found in the LICENSE file at https://angular.dev/license
 
-load("@aspect_rules_js//npm:defs.bzl", "npm_package")
+load("@aspect_rules_js//js:defs.bzl", "js_library")
 load("//tools:defaults2.bzl", "jasmine_test", "ts_project")
 load("//tools:ts_json_schema.bzl", "ts_json_schema")
 
@@ -91,8 +91,8 @@ jasmine_test(
     ],
 )
 
-# This package is intended to be combined into the main @angular/ssr package as a dep.
-npm_package(
+# This target is used as nested_package in the main @angular/ssr package as a dep.
+js_library(
     name = "pkg",
     srcs = [
         "package.json",

pnpm-lock.yaml

@@ -11,6 +11,7 @@ load("//tools:substitutions.bzl", "substitutions")
 def npm_package(
         name,
         deps = [],
+        srcs = [],
         visibility = None,
         pkg_deps = [],
         stamp_files = [],
@@ -96,7 +97,7 @@ def npm_package(
         name = "npm_package",
         visibility = visibility,
         # Note: Order matters here! Last file takes precedence after replaced prefixes.
-        srcs = deps + stamp_targets + [":final_package_json"],
+        srcs = deps + stamp_targets + [":final_package_json"] + srcs,
         replace_prefixes = dict({
             "substituted_final/": "",
             "substituted_with_tars/": "",

tools/bazel/npm_package.bzl

@@ -1,7 +1,7 @@
 load("@aspect_bazel_lib//lib:copy_to_bin.bzl", _copy_to_bin = "copy_to_bin")
 load("@aspect_rules_jasmine//jasmine:defs.bzl", _jasmine_test = "jasmine_test")
 load("@aspect_rules_js//js:defs.bzl", _js_binary = "js_binary")
-load("@npm//@angular/bazel:index.bzl", _ng_package = "ng_package")
+load("@rules_angular//src/ng_package:index.bzl", _ng_package = "ng_package")
 load("//tools:interop.bzl", _ts_project = "ts_project")
 load("//tools:substitutions.bzl", "substitutions")
 load("//tools/bazel:npm_package.bzl", _npm_package = "npm_package")
@@ -20,6 +20,7 @@ def js_binary(**kwargs):
 
 def ng_package(deps = [], **kwargs):
     _ng_package(
+        npm_package_impl = npm_package,
         deps = deps,
         license = "//:LICENSE",
         substitutions = select({