Update Bazel to 7.1.1, slight cleanup and compatibility improvements

7.1 (https://github.com/bazelbuild/bazel/releases/tag/7.1.0) introduces
a number of bug fixes and performance improvements, 7.1.1 has some
additional bug fixes.

- Pass all parameters in kj_test to ensure tags will actually be
  applied, avoiding a considerable footgun.
- Clean up a few globs, allowing us to enable incompatible_disallow_empty_glob
- Flip --legacy_external_runfiles early to avoid generating superfluous
  runfile trees.
- Update wd_ts_bundle to fix --legacy_external_runfiles=false for
  dependents.

.bazelrc

@@ -17,6 +17,12 @@ build --remote_download_all
 # Enable webgpu
 build --//src/workerd/io:enable_experimental_webgpu=True
 
+# Avoid generating duplicate runfile trees. This will become the default in a future bazel version.
+build --legacy_external_runfiles=false
+
+# Flip this early to avoid breaking compatibility once it becomes the default.
+build --incompatible_disallow_empty_glob
+
 # Dawn tint build flags
 build --@dawn//src/tint:tint_build_glsl_writer=False
 build --@dawn//src/tint:tint_build_glsl_validator=False

.bazelversion

@@ -1 +1 @@
-7.0.2
+7.1.1

build/kj_test.bzl

@@ -17,4 +17,6 @@ def kj_test(
  "@//:use_dead_strip": ["-Wl,-dead_strip", "-Wl,-no_exported_symbols"],
  "//conditions:default": [""],
  }),
+ data = data,
+ tags = tags,
  )

build/wd_ts_bundle.bzl

@@ -77,8 +77,8 @@ def wd_ts_bundle(
  eslint_bin.eslint_test(
  name = name + "@eslint",
  args = [
- "--config $(execpath {})".format(eslintrc_json),
- "--parser-options project:$(execpath {})".format(tsconfig_json),
+ "--config $(location {})".format(eslintrc_json),
+ "--parser-options project:$(location {})".format(tsconfig_json),
  "-f stylish",
  "--report-unused-disable-directives",
  ] + ["$(location " + src + ")" for src in ts_srcs],

src/cloudflare/BUILD.bazel

@@ -9,14 +9,15 @@ wd_ts_bundle(
  "internal/*.ts",
  "internal/*.js",
  ],
- exclude = [
- "internal/test/*",
+ allow_empty = True,
+ ),
+ modules = glob(
+ [
+ "*.ts",
+ "*.js",
  ],
+ allow_empty = True,
  ),
- modules = glob([
- "*.ts",
- "*.js",
- ]),
  schema_id = "0xbcc8f57c63814006",
  tsconfig_json = "tsconfig.json",
 )

src/cloudflare/internal/test/d1/BUILD.bazel

@@ -3,8 +3,5 @@ load("//:build/wd_test.bzl", "wd_test")
 wd_test(
  src = "d1-api-test.wd-test",
  args = ["--experimental"],
- data = glob([
- "*.js",
- "*.capnp",
- ]),
+ data = glob(["*.js"]),
 )

src/cloudflare/internal/test/vectorize/BUILD.bazel

@@ -2,8 +2,5 @@ load("//:build/wd_test.bzl", "wd_test")
 
 wd_test(
  src = "vectorize-api-test.wd-test",
- data = glob([
- "*.js",
- "*.capnp",
- ]),
+ data = glob(["*.js"]),
 )

src/workerd/server/tests/unsafe-module/BUILD.bazel

@@ -3,8 +3,5 @@ load("//:build/wd_test.bzl", "wd_test")
 wd_test(
  src = "unsafe-module-test.wd-test",
  args = ["--experimental"],
- data = glob([
- "*.js",
- "*.capnp",
- ]),
+ data = glob(["*.js"]),
 )