Additional comments

Author: jelbourn

tools/bazel-tools-tsconfig.json

@@ -1,7 +1,5 @@
 {
   "compilerOptions": {
-    "module": "commonjs",
-    "target": "es5",
     "lib": ["es2015"],
     "types": ["node"]
   }

tools/sass_bundle.bzl

@@ -54,9 +54,8 @@ sass_bundle = rule(
     # rule (and will thus be available to the nodejs_binary in the sandbox).
     "entry_point": attr.label(mandatory = True, allow_single_file = True),
 
-    # The executable (bundler) for this rule. The user will typically never
-    # specify this and always use the default.
-    "sass_bundle": attr.label(
+    # The executable (bundler) for this rule (private).
+    "_sass_bundle": attr.label(
       default = Label("//tools:sass_bundle"),
       executable = True,
       cfg = "host"

tools/sass_bundle.ts

@@ -8,6 +8,11 @@ const minimist = require('minimist');
 /** The root of the bazel workspace (sandbox). */
 const workspaceRoot = process.cwd();
 
+/**
+ * Bundles sass partials into a single file using the `scss-bundle` package. This does not use the
+ * default CLI for `scss-bundle` to give us a place to introspect / debug values coming from
+ * bazel if necessary.
+ */
 export async function main(args: string[]): Promise<number> {
   const parsedArgs: {srcs: string, output: string, entry: string} = minimist(args);
   const inputFiles = parsedArgs.srcs.split(',');