fix(zone.js): add conditional exports to zone.js package (#51652)

This is needed to better support native ESM modules and avoid the otherwise necessary deep imports like `zone.js/fesm2015/zone-node.js` due to disallowed directory imports.

PR Close #51652

WORKSPACE

@@ -79,7 +79,7 @@ yarn_install(
         YARN_LABEL,
         "//:.yarnrc",
         "//tools:postinstall-patches.js",
-        "//tools/esm-interop:patches/npm/@angular+build-tooling+0.0.0-8d4803573edc70b90a1134ffa996303d1dcc18a9.patch",
+        "//tools/esm-interop:patches/npm/@angular+build-tooling+0.0.0-0109d498b0f6aae418ed4924a5e5c65695f0ac61.patch",
         "//tools/esm-interop:patches/npm/@bazel+concatjs+5.8.1.patch",
         "//tools/esm-interop:patches/npm/@bazel+esbuild+5.7.1.patch",
         "//tools/esm-interop:patches/npm/@bazel+protractor+5.7.1.patch",

integration/cli-elements-universal/angular.json

@@ -21,7 +21,7 @@
             "index": "src/index.html",
             "main": "src/main.ts",
             "polyfills": [
-              "zone.js/dist/zone"
+              "zone.js"
             ],
             "tsConfig": "tsconfig.app.json",
             "aot": true,
@@ -83,7 +83,7 @@
           "options": {
             "main": "src/test.ts",
             "polyfills": [
-              "zone.js/dist/zone"
+              "zone.js"
             ],
             "tsConfig": "tsconfig.spec.json",
             "karmaConfig": "karma.conf.js",

integration/cli-elements-universal/src/test.ts

@@ -1,6 +1,6 @@
 // This file is required by karma.conf.js and loads recursively all the .spec and framework files
 
-import 'zone.js/dist/zone-testing';
+import 'zone.js/testing';
 import { getTestBed } from '@angular/core/testing';
 import {
   BrowserDynamicTestingModule,

integration/injectable-def/src/main.ts

@@ -1,7 +1,4 @@
-// ZoneJS does not add any package exports right now, so we need to directly
-// reference the ESM entry-point for ZoneJS in NodeJS.
-// TODO: Replace this with a package import if ZoneJS sets the `exports` field.
-import 'zone.js/fesm2015/zone-node.js';
+import 'zone.js/node';
 
 // Load the Angular compiler as we will rely on JIT compilation for this test.
 // This test does not use the CLI and we are not processing the framework packages

packages/zone.js/BUILD.bazel

@@ -53,17 +53,12 @@ pkg_npm(
         "CHANGELOG.md",
         "README.md",
         "package.json",
-        "//packages/zone.js/mix:package.json",
-        "//packages/zone.js/node:package.json",
-        "//packages/zone.js/testing:package.json",
     ],
     visibility = ["//visibility:public"],
     deps = [
         ":LICENSE.wrapped",
         ":LICENSE_copy",
         ":zone_externs",
-        "//packages/zone.js/dist:dist_bundle_group",
-        "//packages/zone.js/plugins:plugin_bundle_group",
     ] + [
         "//packages/zone.js/bundles:" + b + "-es5.dist"
         for b in BUNDLES_ENTRY_POINTS.keys()

packages/zone.js/package.json

@@ -42,5 +42,41 @@
   "license": "MIT",
   "bugs": {
     "url": "https://github.com/angular/angular/issues"
+  },
+  "exports": {
+    "./package.json": {
+      "default": "./package.json"
+    },
+    ".": {
+      "typings": "./zone.d.ts",
+      "require": "./bundles/zone.umd.js",
+      "default": "./fesm2015/zone.js"
+    },
+    "./testing": {
+      "require": "./bundles/zone-testing.umd.js",
+      "default": "./fesm2015/zone-testing.js"
+    },
+    "./node": {
+      "require": "./bundles/zone-node.umd.js",
+      "default": "./fesm2015/zone-node.js"
+    },
+    "./mix": {
+      "require": "./bundles/zone-mix.umd.js",
+      "default": "./fesm2015/zone-mix.js"
+    },
+    "./dist/*.min": {
+      "default": "./bundles/*.umd.min.js"
+    },
+    "./dist/*": {
+      "default": "./bundles/*.umd.js"
+    },
+    "./plugins/*.min": {
+      "require": "./bundles/*.umd.min.js",
+      "default": "./fesm2015/*.min.js"
+    },
+    "./plugins/*": {
+      "require": "./bundles/*.umd.js",
+      "default": "./fesm2015/*.js"
+    }
   }
 }