feat: set explicit and configurable tsconfig

By default esbuild uses any local tsconfig.json file. With sandboxing issues (#58)
this may unexpectedly cause `esbuild_bundle` to pickup any local tsconfig.json.

A custom tsconfig can now be set using `esbuild_bundle(tsconfig)`. By default an empty tsconfig is used to prevent unexpected sandboxing issues.

.github/workflows/ci.yaml

@@ -71,6 +71,7 @@ jobs:
           - 'e2e/smoke'
           - 'e2e/npm-links'
           - 'e2e/sourcemaps'
+          - 'e2e/tsconfig'
         exclude:
           # Don't test macos with Bazel 5 to minimize macOS minutes (billed at 10X)
           # https://docs.github.com/en/billing/managing-billing-for-github-actions/about-billing-for-github-actions#included-storage-and-minutes

e2e/tsconfig/.bazelrc

@@ -0,0 +1,2 @@
+build --enable_runfiles
+common:bzlmod --enable_bzlmod

e2e/tsconfig/BUILD.bazel

@@ -0,0 +1,47 @@
+load("@aspect_rules_esbuild//esbuild:defs.bzl", "esbuild")
+load("@aspect_bazel_lib//lib:testing.bzl", "assert_contains")
+
+SRCS = ["main.js", "libs/a.js", "libs/b.js"]
+ENTRY = "main.js"
+
+esbuild(
+    name = "target-a",
+    srcs = SRCS,
+    entry_point = ENTRY,
+    tsconfig = "tsconfig-a.json",
+    output = "a.js",
+)
+
+esbuild(
+    name = "target-b",
+    srcs = SRCS,
+    entry_point = ENTRY,
+    tsconfig = "tsconfig-b.json",
+    output = "b.js",
+)
+
+esbuild(
+    name = "target-none",
+    srcs = [ENTRY],
+    entry_point = ENTRY,
+    output = "none.js",
+    external = ["var-lib"],
+)
+
+assert_contains(
+    name = "config-a",
+    actual = "a.js",
+    expected = "library: A"
+)
+
+assert_contains(
+    name = "config-b",
+    actual = "b.js",
+    expected = "library: B"
+)
+
+assert_contains(
+    name = "config-none",
+    actual = "none.js",
+    expected = "var-lib"
+)

e2e/tsconfig/MODULE.bazel

@@ -0,0 +1,7 @@
+"Bazel dependencies"
+bazel_dep(name = "aspect_rules_esbuild", dev_dependency = True, version = "0.0.0")
+
+local_path_override(
+    module_name = "aspect_rules_esbuild",
+    path = "../..",
+)

e2e/tsconfig/WORKSPACE

@@ -0,0 +1,22 @@
+local_repository(
+    name = "aspect_rules_esbuild",
+    path = "../..",
+)
+
+load("@aspect_rules_esbuild//esbuild:dependencies.bzl", "rules_esbuild_dependencies")
+
+rules_esbuild_dependencies()
+
+load("@rules_nodejs//nodejs:repositories.bzl", "DEFAULT_NODE_VERSION", "nodejs_register_toolchains")
+
+nodejs_register_toolchains(
+    name = "node",
+    node_version = DEFAULT_NODE_VERSION,
+)
+
+load("@aspect_rules_esbuild//esbuild:repositories.bzl", "LATEST_ESBUILD_VERSION", "esbuild_register_toolchains")
+
+esbuild_register_toolchains(
+    name = "esbuild",
+    esbuild_version = LATEST_ESBUILD_VERSION,
+)

e2e/tsconfig/libs/a.js

@@ -0,0 +1 @@
+export const ANSWER = "library: A"

e2e/tsconfig/libs/b.js

@@ -0,0 +1 @@
+export const ANSWER = "library: B"

e2e/tsconfig/main.js

@@ -0,0 +1,3 @@
+import { ANSWER } from "var-lib"
+
+console.log(ANSWER)

e2e/tsconfig/tsconfig-a.json

@@ -0,0 +1,7 @@
+{
+    "compilerOptions": {
+        "paths": {
+            "var-lib": ["./libs/a.js"]
+        }
+    }
+}

e2e/tsconfig/tsconfig-b.json

@@ -0,0 +1,7 @@
+{
+    "compilerOptions": {
+        "paths": {
+            "var-lib": ["./libs/b.js"]
+        }
+    }
+}

e2e/tsconfig/tsconfig.json

@@ -0,0 +1 @@
+with invalid syntax, esbuild shouldn't use this

esbuild/private/BUILD.bazel

@@ -1,3 +1,4 @@
+load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
 
 exports_files(
@@ -10,6 +11,12 @@ exports_files(
     visibility = ["//visibility:public"],
 )
 
+copy_to_bin(
+    name = "empty-json",
+    srcs = ["empty.json"],
+    visibility = ["//visibility:public"],
+)
+
 bzl_library(
     name = "esbuild",
     srcs = ["esbuild.bzl"],

esbuild/private/empty.json

@@ -0,0 +1 @@
+{}

esbuild/private/esbuild.bzl

@@ -163,6 +163,15 @@ See https://esbuild.github.io/api/#target for more details
         TODO: show how to write a config file that depends on plugins, similar to the esbuild_config macro in rules_nodejs.
     """,
     ),
+    "tsconfig": attr.label(
+        mandatory = False,
+        allow_single_file = True,
+        default = Label("@aspect_rules_esbuild//esbuild/private:empty-json"),
+        doc = """TypeScript configuration file used by esbuild. Default to an empty file with no configuration.
+        
+        See https://esbuild.github.io/api/#tsconfig for more details
+    """
+    )
 }
 
 def _bin_relative_path(ctx, file):
@@ -181,6 +190,7 @@ def _esbuild_impl(ctx):
 
     entry_points = desugar_entry_point_names(ctx.file.entry_point, ctx.files.entry_points)
     entry_points_bin_copy = copy_files_to_bin_actions(ctx, entry_points)
+    tsconfig_bin_copy = copy_file_to_bin_action(ctx, ctx.file.tsconfig)
 
     args = dict({
         "bundle": True,
@@ -199,6 +209,7 @@ def _esbuild_impl(ctx):
         # Also disable the log limit and show all logs
         "logLevel": "warning",
         "logLimit": 0,
+        "tsconfig": _bin_relative_path(ctx, tsconfig_bin_copy),
         "metafile": ctx.attr.metafile,
         "platform": ctx.attr.platform,
         # Don't preserve symlinks since doing so breaks node_modules resolution
@@ -311,7 +322,7 @@ def _esbuild_impl(ctx):
             file
             for file in ctx.files.srcs
             if not (file.path.endswith(".d.ts") or file.path.endswith(".tsbuildinfo"))
-        ]) + entry_points_bin_copy + other_inputs + node_toolinfo.tool_files + esbuild_toolinfo.tool_files,
+        ]) + entry_points_bin_copy + [tsconfig_bin_copy] + other_inputs + node_toolinfo.tool_files + esbuild_toolinfo.tool_files,
         transitive = [js_lib_helpers.gather_files_from_js_providers(
             targets = ctx.attr.srcs + ctx.attr.deps,
             include_transitive_sources = True,