chore: move @types deps to the ts_config target

Source code doesn't have to include any indication these 'ambient' types are required.
However we can force users to be explicit in the tsconfig of which `@types` packages to include in compilation.
This follows the locality principle, since the types weren't referenced in the sources, the deps dont belong on the ts_project.

Author: alexeagle

frontend/next.js/BUILD.bazel

@@ -16,6 +16,12 @@ ts_config(
     name = "tsconfig",
     src = "tsconfig.json",
     visibility = ["//visibility:public"],
+    deps = [
+        ":node_modules/@types/is-even",
+        ":node_modules/@types/jest",
+        ":node_modules/@types/react",
+        ":node_modules/@types/react-dom",
+    ],
 )
 
 next(

frontend/next.js/pages/BUILD.bazel

@@ -20,9 +20,6 @@ ts_project(
     visibility = ["//next.js:__subpackages__"],
     deps = [
         "//next.js:node_modules/@bazel-example/one",
-        "//next.js:node_modules/@types/is-even",
-        "//next.js:node_modules/@types/react",
-        "//next.js:node_modules/@types/react-dom",
         "//next.js:node_modules/next",
         "//next.js/pages/api",
     ],
@@ -38,7 +35,7 @@ ts_project(
     deps = [
         "//next.js:node_modules/@testing-library/jest-dom",
         "//next.js:node_modules/@testing-library/react",
-        "//next.js:node_modules/@types/jest",
+        "//next.js:node_modules/@types",
         "//next.js/pages",
     ],
 )

frontend/next.js/tsconfig.json

@@ -11,7 +11,13 @@
     "moduleResolution": "node",
     "resolveJsonModule": true,
     "isolatedModules": true,
-    "jsx": "react-jsx"
+    "jsx": "react-jsx",
+    "types": [
+      "is-even",
+      "jest",
+      "react",
+      "react-dom"
+    ]
   },
   "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"],
   "exclude": ["node_modules"]

frontend/packages/one/BUILD.bazel

@@ -1,9 +1,19 @@
 load("@aspect_rules_js//npm:defs.bzl", "npm_package")
-load("@aspect_rules_ts//ts:defs.bzl", "ts_project")
+load("@aspect_rules_ts//ts:defs.bzl", "ts_config", "ts_project")
 load("@npm//:defs.bzl", "npm_link_all_packages")
 
 npm_link_all_packages(name = "node_modules")
 
+ts_config(
+    name = "tsconfig",
+    src = "tsconfig.json",
+    # transpile time direct dependencies
+    deps = [
+        ":node_modules/@types/is-odd",
+        #":node_modules/@types/node",
+    ],
+)
+
 ts_project(
     name = "one_ts",
     srcs = ["src/main.ts"],
@@ -13,11 +23,7 @@ ts_project(
     ],
     declaration = True,
     transpiler = "tsc",
-    # transpile time direct dependencies
-    deps = [
-        ":node_modules/@types/is-odd",
-        "//next.js:node_modules/@types/node",
-    ],
+    tsconfig = ":tsconfig",
 )
 
 # make this library available via node_modules

frontend/packages/one/tsconfig.json

@@ -2,6 +2,8 @@
     "compilerOptions": {
         "declaration": true,
         "esModuleInterop": true,
-        
+        "types": [
+            "is-odd"
+        ]
     }
 }

frontend/react/BUILD.bazel

@@ -21,6 +21,7 @@ ts_config(
     name = "tsconfig",
     src = "tsconfig.json",
     visibility = ["//visibility:public"],
+    deps = [":node_modules/@types/react"],
 )
 
 tsconfig_to_swcconfig.t2s(

frontend/react/src/BUILD.bazel

@@ -23,7 +23,6 @@ ts_project(
     tsconfig = "//react:tsconfig",
     visibility = ["//react:__subpackages__"],
     deps = [
-        "//react:node_modules/@types/react",
         "//react:node_modules/react",
         "//react:node_modules/react-dom",
         "//react:node_modules/vite-plugin-svgr",
@@ -43,7 +42,7 @@ ts_project(
         "//:node_modules/vitest",
         "//react:node_modules/@testing-library/jest-dom",
         "//react:node_modules/@testing-library/react",
-        "//react:node_modules/@types/jest",
+        "//react:node_modules/@types",
     ],
 )
 

frontend/react/tsconfig.json

@@ -14,7 +14,10 @@
     "resolveJsonModule": true,
     "isolatedModules": true,
     "declaration": true,
-    "jsx": "react-jsx"
+    "jsx": "react-jsx",
+    "types": [
+      "react"
+    ]
   },
   "include": ["src"]
 }