build: clean up BUILD files and update dependencies

This commit cleans up various `BUILD.bazel` files by removing unused file globs from `RUNTIME_ASSETS` and test sources. It also updates the `rules_pkg` dependency to v1.1.0 and removes the `multiple_version_override` for it in `MODULE.bazel`.

Additionally, the conditional `include_npm` setting in `tests/legacy-cli/e2e.bzl` has been removed.

Author: alan-agius4

MODULE.bazel

@@ -8,23 +8,7 @@ bazel_dep(name = "yq.bzl", version = "0.3.1")
 bazel_dep(name = "rules_nodejs", version = "6.5.2")
 bazel_dep(name = "aspect_rules_js", version = "2.6.0")
 bazel_dep(name = "aspect_rules_ts", version = "3.7.0")
-bazel_dep(name = "rules_pkg", version = "0.8.1")
-
-# Alow for usage of [email protected] even though other deps want a later verison.
-multiple_version_override(
-    module_name = "rules_pkg",
-    versions = [
-        "0.8.1",
-        "1.1.0",
-    ],
-)
-
-bazel_dep(name = "rules_python", version = "1.5.3")
-single_version_override(
-    module_name = "rules_python",
-    version = "1.5.3",
-)
-
+bazel_dep(name = "rules_pkg", version = "1.1.0")
 bazel_dep(name = "aspect_bazel_lib", version = "2.21.2")
 bazel_dep(name = "bazel_skylib", version = "1.8.2")
 bazel_dep(name = "aspect_rules_esbuild", version = "0.22.1")

MODULE.bazel.lock

@@ -48,8 +48,6 @@ RUNTIME_ASSETS = glob(
     include = [
         "src/**/schema.json",
         "src/**/*.js",
-        "src/**/*.mjs",
-        "src/**/*.html",
     ],
 ) + [
     "builders.json",

packages/angular/build/BUILD.bazel

@@ -7,12 +7,7 @@ load("//tools:defaults.bzl", "npm_package", "ts_project")
 
 licenses(["notice"])
 
-RUNTIME_ASSETS = glob(
-    include = [
-        "src/*.js",
-        "src/*.mjs",
-    ],
-) + [
+RUNTIME_ASSETS = [
     "package.json",
 ]
 

packages/angular/create/BUILD.bazel

@@ -15,8 +15,6 @@ npm_link_all_packages()
 
 RUNTIME_ASSETS = glob(
     include = [
-        "pwa/*.js",
-        "pwa/*.mjs",
         "pwa/files/**/*",
     ],
 ) + [

packages/angular/pwa/BUILD.bazel

@@ -14,7 +14,6 @@ ts_project(
         include = ["**/*.ts"],
         exclude = [
             "**/*_spec.ts",
-            "test/**/*.ts",
         ],
     ),
     data = ["package.json"],
@@ -35,7 +34,6 @@ ts_project(
     srcs = glob(
         include = [
             "**/*_spec.ts",
-            "test/**/*.ts",
         ],
     ),
     deps = [

packages/angular_devkit/schematics/tools/BUILD.bazel

@@ -21,7 +21,6 @@ ts_project(
         ],
         exclude = [
             "src/**/*_spec.ts",
-            "src/**/*_spec_helpers.ts",
         ],
     ) + [
         "index.ts",
@@ -43,7 +42,6 @@ ts_project(
     srcs = glob(
         include = [
             "src/**/*_spec.ts",
-            "src/**/*_spec_helpers.ts",
         ],
     ),
     deps = [

packages/ngtools/webpack/BUILD.bazel

@@ -51,7 +51,6 @@ RUNTIME_ASSETS = [
     include = [
         "*/schema.json",
         "*/files/**/*",
-        "*/other-files/**/*",
         "*/implements-files/**/*",
         "*/type-files/**/*",
         "*/functional-files/**/*",

packages/schematics/angular/BUILD.bazel

@@ -129,12 +129,7 @@ def _e2e_tests(name, runner, toolchain, **kwargs):
         tags = tags,
         toolchains = toolchains,
         node_toolchain = toolchain,
-        include_npm = select({
-            # For Windows testing mode, we use the real global NPM as otherwise this
-            # will be a lot of files that need to be brought from WSL to the host FS.
-            "@platforms//os:windows": False,
-            "//conditions:default": True,
-        }),
+        include_npm = True,
         **kwargs
     )