fix(typescript): account for rootDir when predicting json output paths (

#3348)

Fixes #3330

packages/typescript/internal/ts_project.bzl

@@ -177,11 +177,12 @@ def _ts_project_impl(ctx):
     # tsc will only produce .json if it also produces .js
     if len(js_outs):
         pkg_len = len(ctx.label.package) + 1 if len(ctx.label.package) else 0
-        json_outs = [
-            ctx.actions.declare_file(_lib.join(ctx.attr.out_dir, src.short_path[pkg_len:]))
+        rootdir_replace_pattern = ctx.attr.root_dir + "/" if ctx.attr.root_dir else ""
+        json_outs = _declare_outputs(ctx, [
+            _lib.join(ctx.attr.out_dir, src.short_path[pkg_len:].replace(rootdir_replace_pattern, ""))
             for src in ctx.files.srcs
             if src.basename.endswith(".json") and src.is_source
-        ]
+        ])
     else:
         json_outs = []
 

packages/typescript/test/ts_project/rootdir_with_value/BUILD.bazel

@@ -4,11 +4,14 @@ load("//packages/typescript:index.bzl", "ts_project")
 # Ensure that subdir/a.ts produces outDir/a.js
 SRCS = [
     "subdir/a.ts",
+    "subdir/file.json",
 ]
 
 ts_project(
     name = "transpile",
     srcs = SRCS,
+    args = ["--listEmittedFiles"],
+    resolve_json_module = True,
     root_dir = "subdir",
 )
 

packages/typescript/test/ts_project/rootdir_with_value/subdir/a.ts

@@ -1 +1,2 @@
-export const a: string = 'hello';
+import data from './file.json';
+export const a: string = 'hello' + JSON.stringify(data);

packages/typescript/test/ts_project/rootdir_with_value/subdir/file.json

@@ -0,0 +1,4 @@
+[{
+    "a": "b"
+}]
+

packages/typescript/test/ts_project/rootdir_with_value/tsconfig.json

@@ -1,4 +1,6 @@
 {
     "compilerOptions": {
+        "resolveJsonModule": true,
+        "esModuleInterop": true,
     }
 }