Use typescript project references (#356)

- `yarn watch` works across packages
- goto definition works across packages
- faster incremental builds
- unify `tsconfig.json`s

See https://www.typescriptlang.org/docs/handbook/project-references.html

Author: fjakobs

.gitignore

@@ -15,3 +15,6 @@ out/
 **/.nyc_output/
 .databricks/**
 packages/databricks-sdk-js/src/config/testdata/azure/Library
+*.d.ts.map
+*.js.map
+tsconfig.tsbuildinfo

packages/databricks-sdk-js/package.json

@@ -17,8 +17,8 @@
         "/dist"
     ],
     "scripts": {
-        "build": "tsc -p tsconfig.json && cp src/fetch.* dist/",
-        "watch": "tsc -w -p tsconfig.json",
+        "build": "tsc --build",
+        "watch": "tsc --build --watch",
         "clean": "rm -rf dist node_modules",
         "openapi:fetch": "./scripts/fetch_openapi.sh 4609d709fc94a1b8dbe79ac78a1cece281fdaeff",
         "openapi:generate": "./scripts/generate_openapi.sh v0.2.0 && yarn run build",

packages/databricks-sdk-js/src/fetch.js

@@ -1,7 +1,7 @@
 // node-fetch@3 is ESM only. Need some trickery to make it work with CommonJS
-
-exports.fetch = async function (url, init) {
-    const {default: fetch} = await import("node-fetch");
+exports.fetch = async function fetch(url, init) {
+    // hide `import` statement from TypeScript compiler so it doesn't mess with it
+    const {default: fetch} = await eval(`import("node-fetch")`);
     const controller = new AbortController();
     const signal = controller.signal;
     return {

packages/databricks-sdk-js/tsconfig.json

@@ -1,20 +1,11 @@
 {
+    "extends": "../../tsconfig_base.json",
     "compilerOptions": {
-        "declaration": true,
+        "allowJs": true,
         "types": ["reflect-metadata", "mocha", "node"],
-        "module": "CommonJS",
-        "target": "ES2020",
-        "moduleResolution": "Node",
-        "esModuleInterop": true,
-        "isolatedModules": true,
         "lib": ["ES2020", "DOM", "ES2021.String"],
-        "sourceMap": true,
         "rootDir": "src",
-        "outDir": "dist",
-        "strict": true /* enable all strict type-checking options */,
-        "experimentalDecorators": true,
-        "emitDecoratorMetadata": true,
-        "forceConsistentCasingInFileNames": true
+        "outDir": "dist"
     },
     "include": ["src"]
 }

packages/databricks-vscode-types/package.json

@@ -14,8 +14,8 @@
     },
     "homepage": "https://github.com/databricks/databricks-vscode#readme",
     "scripts": {
-        "build": "tsc",
-        "watch": "tsc -w",
+        "build": "tsc --build",
+        "watch": "tsc --build --watch",
         "clean": "rm -rf dist node_modules",
         "fix": "eslint . --ext ts --fix && prettier . --write",
         "test:lint": "eslint . --ext ts && prettier . -c",

packages/databricks-vscode-types/tsconfig.json

@@ -1,13 +1,9 @@
 {
+    "extends": "../../tsconfig_base.json",
     "compilerOptions": {
-        "module": "CommonJS",
-        "target": "ES2020",
-        "declaration": true,
-        "esModuleInterop": true,
-        "isolatedModules": true,
         "lib": ["ES2020", "ES2021.String"],
-        "rootDir": ".",
-        "strict": true /* enable all strict type-checking options */
+        "rootDir": "."
     },
+    "references": [{"path": "../databricks-sdk-js"}],
     "include": ["*.ts"]
 }

packages/databricks-vscode/package.json

@@ -522,8 +522,8 @@
         "package:compile": "yarn run esbuild:base",
         "package:copy-webview-toolkit": "cp ./node_modules/@vscode/webview-ui-toolkit/dist/toolkit.js ./out/toolkit.js",
         "esbuild:base": "esbuild ./src/extension.ts --bundle --outfile=out/extension.js --external:vscode --format=cjs --platform=node --sourcemap --target=es2019",
-        "build": "tsc",
-        "watch": "yarn run package:copy-webview-toolkit && tsc -w",
+        "build": "tsc --build",
+        "watch": "yarn run package:copy-webview-toolkit && tsc --build --watch",
         "fix": "eslint src --ext ts --fix && prettier . --write",
         "test:lint": "eslint src --ext ts && prettier . -c",
         "test:unit": "yarn run build && node ./out/test/runTest.js",

packages/databricks-vscode/tsconfig.json

@@ -1,17 +1,14 @@
 {
+    "extends": "../../tsconfig_base.json",
     "compilerOptions": {
-        "module": "CommonJS",
-        "target": "ES2020",
         "outDir": "out",
-        "esModuleInterop": true,
-        "isolatedModules": true,
         "lib": ["ES2020", "ES2021.String"],
-        "sourceMap": true,
-        "rootDir": "src",
-        "strict": true /* enable all strict type-checking options */,
-        "forceConsistentCasingInFileNames": true,
-        "experimentalDecorators": true
+        "rootDir": "src"
     },
+    "references": [
+        {"path": "../databricks-sdk-js"},
+        {"path": "../databricks-vscode-types"}
+    ],
     "include": ["src"],
     "exclude": ["src/test/e2e/"]
 }

tsconfig_base.json

@@ -0,0 +1,18 @@
+{
+    "compilerOptions": {
+        "composite": true,
+        "declaration": true,
+        "declarationMap": true,
+        "sourceMap": true,
+        "allowJs": true,
+        "module": "CommonJS",
+        "target": "ES2020",
+        "moduleResolution": "Node",
+        "esModuleInterop": true,
+        "isolatedModules": true,
+        "strict": true,
+        "experimentalDecorators": true,
+        "emitDecoratorMetadata": true,
+        "forceConsistentCasingInFileNames": true
+    }
+}