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/smoke/BUILD

@@ -3,7 +3,7 @@ load("@bazel_skylib//rules:build_test.bzl", "build_test")
 
 esbuild(
     name = "lib",
-    srcs = ["main.js"],
+    srcs = ["main.js", "tsconfig.json"],
     entry_point = "main.js",
 )
 

e2e/smoke/tsconfig.json

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

e2e/tsconfig/.bazelrc

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

e2e/tsconfig/BUILD.bazel

@@ -0,0 +1,34 @@
+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",
+)
+
+assert_contains(
+    name = "config-a",
+    actual = "a.js",
+    expected = "library: A"
+)
+
+assert_contains(
+    name = "config-b",
+    actual = "b.js",
+    expected = "library: B"
+)
+

e2e/tsconfig/MODULE.bazel

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

e2e/tsconfig/WORKSPACE

@@ -0,0 +1,36 @@
+# Override http_archive for local testing
+local_repository(
+    name = "aspect_rules_esbuild",
+    path = "../..",
+)
+
+#---SNIP--- Below here is re-used in the workspace snippet published on releases
+
+######################
+# rules_esbuild setup #
+######################
+
+# Fetches the rules_esbuild dependencies.
+# If you want to have a different version of some dependency,
+# you should fetch it *before* calling this.
+# Alternatively, you can skip calling this function, so long as you've
+# already fetched all the dependencies.
+load("@aspect_rules_esbuild//esbuild:dependencies.bzl", "rules_esbuild_dependencies")
+
+rules_esbuild_dependencies()
+
+# If you didn't already register a toolchain providing nodejs, do that:
+load("@rules_nodejs//nodejs:repositories.bzl", "DEFAULT_NODE_VERSION", "nodejs_register_toolchains")
+
+nodejs_register_toolchains(
+    name = "node",
+    node_version = DEFAULT_NODE_VERSION,
+)
+
+# Register a toolchain containing esbuild npm package and native bindings
+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"]
+        }
+    }
+}

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,

esbuild/repositories.bzl

@@ -39,6 +39,10 @@ def _esbuild_repo_impl(repository_ctx):
         Label("@aspect_rules_esbuild//esbuild/private:launcher.js"),
         "launcher.js",
     )
+    repository_ctx.symlink(
+        Label("@aspect_rules_esbuild//esbuild/private:empty.json"),
+        "empty.json",
+    )
     build_content = """#Generated by esbuild/repositories.bzl
 load("@aspect_rules_esbuild//esbuild:toolchain.bzl", "esbuild_toolchain")
 load("@aspect_rules_js//js:defs.bzl", "js_binary")
@@ -52,7 +56,7 @@ npm_link_package(
 js_binary(
     name = "launcher",
     entry_point = "launcher.js",
-    data = [":node_modules/esbuild"],
+    data = [":node_modules/esbuild", ":empty.json"],
 )
 
 esbuild_toolchain(