refactor: replace usage of ng_package with new ng_package from rules_angular

Author: josephperrott

Diff for: 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",
+)

Diff for: goldens/public-api/angular/ssr/node/index.api.md

@@ -5,10 +5,10 @@
 ```ts
 
 import { ApplicationRef } from '@angular/core';
-import type { Http2ServerRequest } from 'node:http2';
-import type { Http2ServerResponse } from 'node:http2';
-import type { IncomingMessage } from 'node:http';
-import type { ServerResponse } from 'node:http';
+import { Http2ServerRequest } from 'node:http2';
+import { Http2ServerResponse } from 'node:http2';
+import { IncomingMessage } from 'node:http';
+import { ServerResponse } from 'node:http';
 import { StaticProvider } from '@angular/core';
 import { Type } from '@angular/core';
 

Diff for: 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",

Diff for: 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 = "npm_package",
     srcs = [
         ":package.json",
         "//packages/angular/ssr/third_party/beasties:beasties_bundled",
@@ -53,10 +51,21 @@ ng_package(
     ],
     nested_packages = [
         "//packages/angular/ssr/schematics:pkg",
+        # Included directly as the generated types reference the types file in this location.
+        "//packages/angular/ssr/third_party/beasties:beasties_dts",
     ],
+    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",
+    ],
+    tags = ["release-package"],
     deps = [
-        ":ssr_legacy",
-        "//packages/angular/ssr/node:node_legacy",
+        ":ssr",
+        "//packages/angular/ssr/node",
     ],
 )
 
@@ -69,16 +78,6 @@ pkg_tar(
     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",

Diff for: packages/angular/ssr/schematics/BUILD.bazel

@@ -3,8 +3,8 @@
 # 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("//tools:defaults.bzl", "jasmine_test", "ts_project")
+load("@aspect_rules_js//js:defs.bzl", "js_library")
 load("//tools:ts_json_schema.bzl", "ts_json_schema")
 
 licenses(["notice"])
@@ -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",

Diff for: 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/": "",

Diff for: 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")