Use universal info logging since existing logging is expected to be seen without config

Author: jaggederest

src/error.ts

@@ -55,7 +55,7 @@ export class CertificateError extends Error {
 							await CertificateError.determineVerifyErrorCause(address);
 						return new CertificateError(err.message, cause);
 					} catch (error) {
-						logger.debug(
+						logger.info(
 							`Failed to parse certificate from ${address}: ${error}`,
 						);
 						break;

src/inbox.ts

@@ -64,7 +64,7 @@ export class Inbox implements vscode.Disposable {
 		});
 
 		this.#socket.on("open", () => {
-			logger.debug("Listening to Coder Inbox");
+			logger.info("Listening to Coder Inbox");
 		});
 
 		this.#socket.on("error", (error) => {
@@ -87,7 +87,7 @@ export class Inbox implements vscode.Disposable {
 
 	dispose() {
 		if (!this.#disposed) {
-			logger.debug("No longer listening to Coder Inbox");
+			logger.info("No longer listening to Coder Inbox");
 			this.#socket.close();
 			this.#disposed = true;
 		}
@@ -98,6 +98,6 @@ export class Inbox implements vscode.Disposable {
 			error,
 			"Got empty error while monitoring Coder Inbox",
 		);
-		logger.debug(message);
+		logger.info(message);
 	}
 }

src/remote.ts

@@ -118,7 +118,7 @@ export class Remote {
 							case "starting":
 							case "stopping":
 								writeEmitter = initWriteEmitterAndTerminal();
-								logger.debug(`Waiting for ${workspaceName}...`);
+								logger.info(`Waiting for ${workspaceName}...`);
 								workspace = await waitForBuild(
 									restClient,
 									writeEmitter,
@@ -170,7 +170,7 @@ export class Remote {
 								);
 							}
 						}
-						logger.debug(
+						logger.info(
 							`${workspaceName} status is now ${workspace.latest_build.status}`,
 						);
 					}
@@ -239,7 +239,7 @@ export class Remote {
 		}
 
 		logger.info(`Using deployment URL: ${baseUrlRaw}`);
-		logger.debug(`Using deployment label: ${parts.label || "n/a"}`);
+		logger.info(`Using deployment label: ${parts.label || "n/a"}`);
 
 		// We could use the plugin client, but it is possible for the user to log
 		// out or log into a different deployment while still connected, which would
@@ -305,7 +305,7 @@ export class Remote {
 		// Next is to find the workspace from the URI scheme provided.
 		let workspace: Workspace;
 		try {
-			logger.debug(`Looking for workspace ${workspaceName}...`);
+			logger.info(`Looking for workspace ${workspaceName}...`);
 			workspace = await workspaceRestClient.getWorkspaceByOwnerAndName(
 				parts.username,
 				parts.workspace,
@@ -393,7 +393,7 @@ export class Remote {
 		this.commands.workspace = workspace;
 
 		// Pick an agent.
-		logger.debug(`Finding agent for ${workspaceName}...`);
+		logger.info(`Finding agent for ${workspaceName}...`);
 		const gotAgent = await this.commands.maybeAskAgent(workspace, parts.agent);
 		if (!gotAgent) {
 			// User declined to pick an agent.
@@ -404,7 +404,7 @@ export class Remote {
 		logger.info(`Found agent ${agent.name} with status ${agent.status}`);
 
 		// Do some janky setting manipulation.
-		logger.debug("Modifying settings...");
+		logger.info("Modifying settings...");
 		const remotePlatforms = this.vscodeProposed.workspace
 			.getConfiguration()
 			.get<Record<string, string>>("remote.SSH.remotePlatform", {});
@@ -504,7 +504,7 @@ export class Remote {
 
 		// Wait for the agent to connect.
 		if (agent.status === "connecting") {
-			logger.debug(`Waiting for ${workspaceName}/${agent.name}...`);
+			logger.info(`Waiting for ${workspaceName}/${agent.name}...`);
 			await vscode.window.withProgress(
 				{
 					title: "Waiting for the agent to connect...",
@@ -563,7 +563,7 @@ export class Remote {
 		// If we didn't write to the SSH config file, connecting would fail with
 		// "Host not found".
 		try {
-			logger.debug("Updating SSH config...");
+			logger.info("Updating SSH config...");
 			await this.updateSSHConfig(
 				workspaceRestClient,
 				parts.label,

src/workspaceMonitor.ts

@@ -43,7 +43,7 @@ export class WorkspaceMonitor implements vscode.Disposable {
 		this.name = `${workspace.owner_name}/${workspace.name}`;
 		const url = this.restClient.getAxiosInstance().defaults.baseURL;
 		const watchUrl = new URL(`${url}/api/v2/workspaces/${workspace.id}/watch`);
-		logger.debug(`Monitoring ${this.name}...`);
+		logger.info(`Monitoring ${this.name}...`);
 
 		const eventSource = new EventSource(watchUrl.toString(), {
 			fetch: createStreamingFetchAdapter(this.restClient.getAxiosInstance()),
@@ -86,7 +86,7 @@ export class WorkspaceMonitor implements vscode.Disposable {
 	 */
 	dispose() {
 		if (!this.disposed) {
-			logger.debug(`Unmonitoring ${this.name}...`);
+			logger.info(`Unmonitoring ${this.name}...`);
 			this.statusBarItem.dispose();
 			this.eventSource.close();
 			this.disposed = true;
@@ -203,7 +203,7 @@ export class WorkspaceMonitor implements vscode.Disposable {
 			error,
 			"Got empty error while monitoring workspace",
 		);
-		logger.debug(message);
+		logger.info(message);
 	}
 
 	private updateContext(workspace: Workspace) {

src/workspacesProvider.ts

@@ -97,7 +97,7 @@ export class WorkspaceProvider
 	 */
 	private async fetch(): Promise<WorkspaceTreeItem[]> {
 		if (vscode.env.logLevel <= vscode.LogLevel.Debug) {
-			logger.debug(
+			logger.info(
 				`Fetching workspaces: ${this.getWorkspacesQuery || "no filter"}...`,
 			);
 		}

vitest.config.ts

@@ -13,5 +13,8 @@ export default defineConfig({
 			"./src/test/**",
 		],
 		environment: "node",
+		env: {
+			NODE_ENV: "test",
+		  },
 	},
 });