Use stricter TS compiler settings for modules

packages/databricks-sdk-js/package.json

@@ -42,6 +42,7 @@
         "tmp-promise": "^3.0.3",
         "ts-loader": "^9.3.1",
         "ts-mocha": "^10.0.0",
+        "ts-node": "^10.9.1",
         "typescript": "^4.7.4",
         "uuid": "^8.3.2"
     }

packages/databricks-sdk-js/src/apis/executionContext.integ.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
 import {ExecutionContextService, CommandsService} from "..";
-import * as assert from "assert";
+import assert from "assert";
 
 import {IntegrationTestSetup, sleep} from "../test/IntegrationTestSetup";
 

packages/databricks-sdk-js/src/apis/jobs.integ.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
 import {JobsService, DbfsService} from "..";
-import assert = require("assert");
+import assert from "assert";
 
 import {IntegrationTestSetup, sleep} from "../test/IntegrationTestSetup";
 

packages/databricks-sdk-js/src/auth/configFile.test.ts

@@ -1,11 +1,11 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
-import assert = require("node:assert");
+import assert from "node:assert";
 import {loadConfigFile, resolveConfigFilePath} from "./configFile";
 import {writeFile} from "node:fs/promises";
 import {withFile} from "tmp-promise";
 import {homedir} from "node:os";
-import path = require("node:path");
+import path from "node:path";
 
 describe(__filename, () => {
     beforeEach(() => {

packages/databricks-sdk-js/src/auth/configFile.ts

@@ -1,4 +1,4 @@
-import path = require("node:path");
+import path from "node:path";
 import {readFile, stat} from "node:fs/promises";
 import {parse} from "ini";
 import {homedir} from "node:os";

packages/databricks-sdk-js/src/auth/fromConfigFile.test.ts

@@ -1,4 +1,4 @@
-import assert = require("node:assert");
+import assert from "node:assert";
 import {writeFile} from "node:fs/promises";
 import {withFile} from "tmp-promise";
 import {DEFAULT_PROFILE, fromConfigFile} from "./fromConfigFile";

packages/databricks-sdk-js/src/auth/fromEnv.test.ts

@@ -1,4 +1,4 @@
-import assert = require("node:assert");
+import assert from "node:assert";
 import {CredentialsProviderError} from "./types";
 import {fromEnv} from "./fromEnv";
 

packages/databricks-sdk-js/src/services/Cluster.integ.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
 import {Cluster} from "..";
-import assert = require("assert");
+import assert from "assert";
 
 import {IntegrationTestSetup} from "../test/IntegrationTestSetup";
 

packages/databricks-sdk-js/src/services/Command.ts

@@ -1,4 +1,4 @@
-import EventEmitter = require("node:events");
+import EventEmitter from "node:events";
 import {CommandsService, CommandsStatusResponse} from "../apis/commands";
 import {Language} from "../apis/executionContext";
 import {ExecutionContext} from "./ExecutionContext";

packages/databricks-sdk-js/src/services/ExecutionContext.integ.ts

@@ -1,7 +1,7 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
 import {ExecutionContext} from "..";
-import * as assert from "assert";
+import assert from "assert";
 
 import {IntegrationTestSetup} from "../test/IntegrationTestSetup";
 

packages/databricks-sdk-js/src/services/WorkflowRun.integ.ts

@@ -1,5 +1,5 @@
 import {DbfsService, Cluster, jobs, WorkflowRun, WorkspaceService} from "..";
-import * as assert from "assert";
+import assert from "assert";
 
 import {IntegrationTestSetup} from "../test/IntegrationTestSetup";
 

packages/databricks-sdk-js/tsconfig.json

@@ -3,6 +3,9 @@
         "declaration": true,
         "module": "CommonJS",
         "target": "ES6",
+        "moduleResolution": "Node",
+        "esModuleInterop": true,
+        "isolatedModules": true,
         "lib": ["ES2020"],
         "sourceMap": true,
         "rootDir": "src",

packages/databricks-vscode/src/cluster/ClusterListDataProvider.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
-import * as assert from "assert";
+import assert from "assert";
 import {mock, when, instance} from "ts-mockito";
 import {ClusterModel} from "./ClusterModel";
 import {Disposable, TreeItem} from "vscode";

packages/databricks-vscode/src/cluster/ClusterModel.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
-import * as assert from "assert";
+import assert from "assert";
 import {mock, when, anything, anyString, instance, verify} from "ts-mockito";
 import {ApiClient, cluster} from "@databricks/databricks-sdk";
 import {ClusterModel} from "./ClusterModel";

packages/databricks-vscode/src/configuration/ConfigurationDataProvider.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 
-import * as assert from "assert";
+import assert from "assert";
 import {mock, when, instance} from "ts-mockito";
 import {Disposable} from "vscode";
 import {ConfigurationDataProvider} from "./ConfigurationDataProvider";

packages/databricks-vscode/src/configuration/ConnectionManager.ts

@@ -4,7 +4,7 @@ import {
     fromConfigFile,
     ScimService,
 } from "@databricks/databricks-sdk";
-import {commands, EventEmitter, Uri, window, workspace} from "vscode";
+import {commands, EventEmitter, window, workspace} from "vscode";
 import {CliWrapper} from "../cli/CliWrapper";
 import {ProjectConfigFile} from "./ProjectConfigFile";
 import {selectProfile} from "./selectProfileWizard";

packages/databricks-vscode/src/configuration/ProjectConfigFile.ts

@@ -1,5 +1,5 @@
-import path = require("node:path");
-import fs = require("node:fs/promises");
+import path from "node:path";
+import fs from "node:fs/promises";
 
 export interface ProjectConfig {
     profile?: string;

packages/databricks-vscode/src/test/e2e/configure.e2e.ts

@@ -1,5 +1,5 @@
-import assert = require("node:assert");
-import path = require("node:path");
+import assert from "node:assert";
+import path from "node:path";
 import * as fs from "fs/promises";
 import * as tmp from "tmp-promise";
 import {

packages/databricks-vscode/src/test/e2e/run_on_cluster.e2e.ts

@@ -1,15 +1,9 @@
-import assert = require("node:assert");
-import path = require("node:path");
+import assert from "node:assert";
+import path from "node:path";
 import * as fs from "fs/promises";
 import * as tmp from "tmp-promise";
-import {
-    VSBrowser,
-    WebDriver,
-    ActivityBar,
-    InputBox,
-    Workbench,
-} from "vscode-extension-tester";
-import {openCommandPrompt, openFolder} from "./utils";
+import {VSBrowser, WebDriver} from "vscode-extension-tester";
+import {openFolder} from "./utils";
 
 describe("Run python on cluster", function () {
     // these will be populated by the before() function

packages/databricks-vscode/src/test/e2e/scripts/e2e.ts

@@ -1,8 +1,8 @@
 /* eslint-disable @typescript-eslint/naming-convention */
 import {spawn} from "child_process";
-import * as assert from "assert";
-import * as fs from "fs/promises";
-import * as tmp from "tmp-promise";
+import assert from "assert";
+import fs from "fs/promises";
+import tmp from "tmp-promise";
 
 /**
  * Create a temporary Databricks config file with values taken from environment variables

packages/databricks-vscode/src/test/runTest.ts

@@ -1,5 +1,5 @@
-import * as path from "path";
-import os = require("os");
+import path from "path";
+import os from "os";
 
 import {runTests} from "@vscode/test-electron";
 

packages/databricks-vscode/src/test/suite.ts

@@ -1,6 +1,6 @@
-import * as path from "path";
-import * as Mocha from "mocha";
-import * as glob from "glob";
+import path from "path";
+import Mocha from "mocha";
+import glob from "glob";
 
 export function run(): Promise<void> {
     // Create the mocha test

packages/databricks-vscode/tsconfig.json

@@ -1,7 +1,10 @@
 {
     "compilerOptions": {
-        "module": "commonjs",
-        "target": "ES2020",
+        "module": "CommonJS",
+        "target": "ES6",
+        "moduleResolution": "Node",
+        "esModuleInterop": true,
+        "isolatedModules": true,
         "lib": ["ES2020"],
         "sourceMap": true,
         "rootDir": "src",

yarn.lock

@@ -31,6 +31,7 @@ __metadata:
     tmp-promise: ^3.0.3
     ts-loader: ^9.3.1
     ts-mocha: ^10.0.0
+    ts-node: ^10.9.1
     typescript: ^4.7.4
     uuid: ^8.3.2
   languageName: unknown