fix: update tsconfig.json and convert pretty-bytes imports to standard ES6

jaggederest · claude · jaggederest · commit d9b543ad443a · 2025-06-17T12:37:51.000-07:00
- Updated tsconfig.json to use CommonJS module system with proper ES module interop - Converted dynamic imports of pretty-bytes to standard ES6 imports in remote.ts and storage.ts - Added integration test command to CLAUDE.md documentation 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -10,6 +10,7 @@
 - Run all tests: `yarn test`
 - Run specific test: `vitest ./src/filename.test.ts`
 - CI test mode: `yarn test:ci`
+- Integration tests: `yarn test:integration`
 
 ## Code Style Guidelines
 
diff --git a/src/remote.ts b/src/remote.ts
@@ -6,9 +6,7 @@ import * as fs from "fs/promises";
 import * as jsonc from "jsonc-parser";
 import * as os from "os";
 import * as path from "path";
-// Dynamic import for ESM module
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let prettyBytes: any;
+import prettyBytes from "pretty-bytes";
 import * as semver from "semver";
 import * as vscode from "vscode";
 import {
@@ -852,10 +850,6 @@ export class Remote {
 			download_bytes_sec: number;
 			using_coder_connect: boolean;
 		}) => {
-			// Load ESM module if not already loaded
-			if (!prettyBytes) {
-				prettyBytes = (await import("pretty-bytes")).default;
-			}
 			let statusText = "$(globe) ";
 
 			// Coder Connect doesn't populate any other stats
diff --git a/src/storage.ts b/src/storage.ts
@@ -3,9 +3,7 @@ import { createWriteStream } from "fs";
 import fs from "fs/promises";
 import { IncomingMessage } from "http";
 import path from "path";
-// Dynamic import for ESM module
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let prettyBytes: any;
+import prettyBytes from "pretty-bytes";
 import * as vscode from "vscode";
 import { errToStr } from "./api-helper";
 import * as cli from "./cliManager";
@@ -124,10 +122,6 @@ export class Storage {
 	 * downloads being disabled.
 	 */
 	public async fetchBinary(restClient: Api, label: string): Promise<string> {
-		// Load ESM module if not already loaded
-		if (!prettyBytes) {
-			prettyBytes = (await import("pretty-bytes")).default;
-		}
 		const baseUrl = restClient.getAxiosInstance().defaults.baseURL;
 
 		// Settings can be undefined when set to their defaults (true in this case),
diff --git a/tsconfig.json b/tsconfig.json
@@ -1,13 +1,17 @@
 {
 	"compilerOptions": {
-		"module": "Node16",
+		"module": "commonjs",
 		"target": "ES2022",
+		"moduleResolution": "node",
 		"outDir": "out",
 		// "dom" is required for importing the API from coder/coder.
 		"lib": ["ES2022", "dom"],
 		"sourceMap": true,
 		"strict": true,
-		"skipLibCheck": true
+		"esModuleInterop": true,
+		"skipLibCheck": true,
+		"forceConsistentCasingInFileNames": true
 	},
-	"exclude": ["vitest.config.ts"]
+	"exclude": ["node_modules", "vitest.config.ts"],
+	"include": ["src/**/*"]
 }
