Update to new npm fined grained deps targets

Closes #295

PiperOrigin-RevId: 215735317

Author: gregmagolan

BUILD.bazel

@@ -41,8 +41,8 @@ gazelle(
 js_library(
     name = "protobufjs_bootstrap_scripts",
     srcs = [
-        "@build_bazel_rules_typescript_protobufs_compiletime_deps//:node_modules/long/dist/long.js",
-        "@build_bazel_rules_typescript_protobufs_compiletime_deps//:node_modules/protobufjs/dist/minimal/protobuf.min.js",
+        "@build_bazel_rules_typescript_protobufs_compiletime_deps//node_modules/long:dist/long.js",
+        "@build_bazel_rules_typescript_protobufs_compiletime_deps//node_modules/protobufjs:dist/minimal/protobuf.min.js",
     ],
     # Make devmode loading work when it does require("protobufjs/minimal")
     # so this is shimmed to define it to equal global.protobuf
@@ -56,15 +56,15 @@ js_library(
 load("@build_bazel_rules_nodejs//:defs.bzl", "nodejs_binary")
 
 # A nodejs_binary for @bazel/typescript/tsc_wrapped to use by default in
-# ts_library that depends on @npm//:@bazel/typescript instead of the
+# ts_library that depends on @npm//@bazel/typescript instead of the
 # output of the //internal/tsc_wrapped ts_library rule. This
-# default is for downstream users that depend on the @bazel/karma npm
-# package. The generated @npm//:@bazel/typescript/tsc_wrapped target
+# default is for downstream users that depend on the @bazel/typescript npm
+# package. The generated @npm//@bazel/typescript/bin:tsc_wrapped target
 # does not work because it does not have the node `--expose-gc` flag
 # set which is required to support the call to `global.gc()`.
 nodejs_binary(
     name = "@bazel/typescript/tsc_wrapped",
-    data = ["@npm//:@bazel/typescript"],
+    data = ["@npm//@bazel/typescript"],
     entry_point = "@bazel/typescript/tsc_wrapped/tsc_wrapped.js",
     install_source_map_support = False,
     templated_args = ["--node_options=--expose-gc"],

README.md

@@ -183,15 +183,15 @@ ts_library(
     name = "my_code",
     srcs = glob(["*.ts"]),
     deps = [
-      "@npm//:@types/node",
-      "@npm//:@types/foo",
-      "@npm//:foo",
+      "@npm//@types/node",
+      "@npm//@types/foo",
+      "@npm//foo",
       "//path/to/other:library",
     ],
 )
 ```
 
-You can also you the `@npm//:@types` target which will include all
+You can also you the `@npm//@types` target which will include all
 packages in the `@types` scope as dependencies.
 
 If you are using self-managed npm dependencies, you can use the

WORKSPACE

@@ -42,21 +42,15 @@ yarn_install(
 # Install a hermetic version of node.
 node_repositories(preserve_symlinks = True)
 
+# Note: We depend on @bazel/typescript in package.json
+# so that the target @npm//@bazel/typescript is defined
+# as it is referenced in /BUILD.bazel for use downstream.
+# This target and package is not used locally so
+# the version of this dependency does not matter.
 yarn_install(
     name = "npm",
     package_json = "//:package.json",
     yarn_lock = "//:yarn.lock",
-    manual_build_file_contents = """
-filegroup(
-  name = "@bazel/typescript",
-  srcs = [],
-)
-
-filegroup(
-  name = "@bazel/karma",
-  srcs = [],
-)
-""",
 )
 
 load("@io_bazel_rules_go//go:def.bzl", "go_rules_dependencies", "go_register_toolchains")

examples/BUILD.bazel

@@ -40,14 +40,14 @@ ts_library(
         ":foo_ts_library",
         "//examples/generated_ts",
         "//examples/some_library:lib",
-        "@npm//:typescript",
-        # Example of using the `@npm//:@types` target to depend on all
+        "@npm//typescript",
+        # Example of using the `@npm//@types` target to depend on all
         # @types packages and with the types attribute of tsconfig not
         # specified. In this case, typescript will automatically discover
         # all types under node_modules/@types and included them in the compile.
         # See getAutomaticTypeDirectiveNames in
         # https://github.com/Microsoft/TypeScript/blob/master/src/compiler/moduleNameResolver.ts.
-        "@npm//:@types",
+        "@npm//@types",
     ],
 )
 

examples/app/BUILD.bazel

@@ -41,8 +41,8 @@ ts_library(
     srcs = ["app_e2e_test.ts"],
     tsconfig = "//examples:tsconfig-test",
     deps = [
-        "@npm//:@types/jasmine",
-        "@npm//:@types/node",
-        "@npm//:protractor",
+        "@npm//@types/jasmine",
+        "@npm//@types/node",
+        "@npm//protractor",
     ],
 )

examples/googmodule/BUILD.bazel

@@ -20,6 +20,6 @@ jasmine_node_test(
     srcs = ["googmodule_output_test.js"],
     data = [
         ":es5_output",
-        "@npm//:jasmine",
+        "@npm//jasmine",
     ],
 )

examples/protocol_buffers/BUILD.bazel

@@ -31,10 +31,10 @@ ts_library(
     tsconfig = "//examples:tsconfig-test",
     deps = [
         ":car",
-        "@npm//:@types/jasmine",
-        "@npm//:@types/long",
-        "@npm//:@types/node",
-        "@npm//:long",
+        "@npm//@types/jasmine",
+        "@npm//@types/long",
+        "@npm//@types/node",
+        "@npm//long",
     ],
 )
 
@@ -83,8 +83,8 @@ rollup_bundle(
 genrule(
     name = "protobufjs",
     srcs = [
-        "@build_bazel_rules_typescript_protobufs_compiletime_deps//:node_modules/protobufjs/dist/minimal/protobuf.min.js",
-        "@build_bazel_rules_typescript_protobufs_compiletime_deps//:node_modules/long/dist/long.js",
+        "@build_bazel_rules_typescript_protobufs_compiletime_deps//node_modules/protobufjs:dist/minimal/protobuf.min.js",
+        "@build_bazel_rules_typescript_protobufs_compiletime_deps//node_modules/long:dist/long.js",
     ],
     outs = [
         "protobuf.min.js",
@@ -108,8 +108,8 @@ ts_library(
     srcs = ["app_e2e_test.ts"],
     tsconfig = "//examples:tsconfig-test",
     deps = [
-        "@npm//:@types/jasmine",
-        "@npm//:@types/node",
-        "@npm//:protractor",
+        "@npm//@types/jasmine",
+        "@npm//@types/node",
+        "@npm//protractor",
     ],
 )

examples/testing/BUILD.bazel

@@ -12,8 +12,8 @@ ts_library(
     tsconfig = "//examples:tsconfig-test",
     deps = [
         ":lib",
-        "@npm//:@types/jasmine",
-        "@npm//:@types/node",
+        "@npm//@types/jasmine",
+        "@npm//@types/node",
     ],
 )
 

examples/tsconfig-bar.json

@@ -3,7 +3,7 @@
     "strict": true,
     "lib": ["es2015.promise", "dom", "es5"],
     // Auto-discover all types in this configuration since we don't
-    // specify `types = []` and the ts_library rule depends on `@npm//:@types`.
+    // specify `types = []` and the ts_library rule depends on `@npm//@types`.
     // Typescript will automatically discover all types under node_modules/@types
     // and included them in the build. See getAutomaticTypeDirectiveNames in
     // https://github.com/Microsoft/TypeScript/blob/master/src/compiler/moduleNameResolver.ts.

internal/BUILD.bazel

@@ -31,8 +31,8 @@ load("@build_bazel_rules_nodejs//:defs.bzl", "nodejs_binary", "jasmine_node_test
 nodejs_binary(
     name = "tsc",
     data = [
-        "@npm//:source-map-support",
-        "@npm//:typescript",
+        "@npm//source-map-support",
+        "@npm//typescript",
     ],
     entry_point = "typescript/lib/tsc.js",
     visibility = ["//internal:__subpackages__"],
@@ -65,25 +65,25 @@ ts_library(
     # work with vanilla tsc.
     # Workaround for https://github.com/Microsoft/TypeScript/issues/22208
     deps = [
-        "@npm//:@types/node",
-        "@npm//:tsickle",
-        "@npm//:tsutils",
-        "@npm//:typescript",
+        "@npm//@types/node",
+        "@npm//tsickle",
+        "@npm//tsutils",
+        "@npm//typescript",
     ],
 )
 
 # Other ts_library rules will use this custom compiler, which calls the
 # TypeScript APIs to act like tsc, but adds capabilities like Bazel workers.
-# TODO(gregmagolan): make @npm//:tsickle dependency optional
+# TODO(gregmagolan): make @npm//tsickle dependency optional
 nodejs_binary(
     name = "tsc_wrapped_bin",
     data = [
         ":tsc_wrapped",
-        "@npm//:protobufjs",
-        "@npm//:source-map-support",
-        "@npm//:tsickle",
-        "@npm//:tsutils",
-        "@npm//:typescript",
+        "@npm//protobufjs",
+        "@npm//source-map-support",
+        "@npm//tsickle",
+        "@npm//tsutils",
+        "@npm//typescript",
     ],
     entry_point = "build_bazel_rules_typescript/internal/tsc_wrapped/tsc_wrapped.js",
     templated_args = ["--node_options=--expose-gc"],
@@ -96,9 +96,9 @@ ts_library(
     tsconfig = "//internal:tsc_wrapped/tsconfig.json",
     deps = [
         ":tsc_wrapped",
-        "@npm//:@types/jasmine",
-        "@npm//:@types/node",
-        "@npm//:typescript",
+        "@npm//@types/jasmine",
+        "@npm//@types/node",
+        "@npm//typescript",
     ],
 )
 
@@ -107,8 +107,8 @@ jasmine_node_test(
     srcs = [],
     deps = [
         ":test_lib",
-        "@npm//:jasmine",
-        "@npm//:typescript",
+        "@npm//jasmine",
+        "@npm//typescript",
     ],
 )