Update build_bazel_rules_nodejs to fix closure compiles (#912)

* Update build_bazel_rules_nodejs to fix closure compiles

* Fix spacing in update_bazel_workspace.sh script

* space

bazel/test_external/BUILD

@@ -10,16 +10,37 @@ wasm_cc_binary(
     cc_target = ":hello-world",
 )
 
+BASE_LINKOPTS = [
+    "--bind",  # Enable embind
+    "-sMODULARIZE",
+]
+
+RELEASE_OPTS = [
+    "--closure 1",  # Run the closure compiler
+]
+
+DEBUG_OPTS = [
+    "--closure 0",  # Do not use closure
+]
+
+config_setting(
+    name = "release_opts",
+    values = {"compilation_mode": "opt"},
+)
+
+config_setting(
+    name = "debug_opts",
+    values = {"compilation_mode": "dbg"},
+)
+
 cc_binary(
     name = "hello-embind",
     srcs = ["hello-embind.cc"],
-    # It's probably a good idea to use select() and config_setting
-    # to add different linkopts to BASE_LINKOPTS depending on things
-    # like a "opt" or "dbg" compilation mode.
-    linkopts = [
-        "--bind",  # Enable embind
-        "-sMODULARIZE",
-    ],
+    linkopts = select({
+        ":debug_opts": BASE_LINKOPTS + DEBUG_OPTS,
+        ":release_opts": BASE_LINKOPTS + RELEASE_OPTS,
+        "//conditions:default": BASE_LINKOPTS + RELEASE_OPTS,
+    }),
     # This target won't build successfully on its own because of missing emscripten
     # headers etc. Therefore, we hide it from wildcards.
     tags = ["manual"],