Handle review comments

Author: EhabY

CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Unreleased
 
+### Added
+
+- Proxy log directory now defaults to the extension's global storage when `coder.proxyLogDirectory`
+  is not set, so SSH connection logs are always captured without manual configuration. Also respects
+  the `CODER_SSH_LOG_DIR` environment variable as a fallback.
+
 ## [v1.14.0-pre](https://github.com/coder/vscode-coder/releases/tag/v1.14.0-pre) 2026-03-06
 
 ### Added

src/commands.ts

@@ -148,12 +148,12 @@ export class Commands {
 	public async viewLogs(): Promise<void> {
 		if (this.workspaceLogPath) {
 			// Return the connected deployment's log file.
-			return this.openFile(this.workspaceLogPath);
+			return openFile(this.workspaceLogPath);
 		}
 
 		const logDir = this.pathResolver.getProxyLogPath();
 		try {
-			const files = await fs.readdir(logDir);
+			const files = await readdirOrEmpty(logDir);
 			// Sort explicitly since fs.readdir order is platform-dependent
 			const logFiles = files
 				.filter((f) => f.endsWith(".log"))
@@ -172,7 +172,7 @@ export class Commands {
 			});
 
 			if (selected) {
-				await this.openFile(path.join(logDir, selected));
+				await openFile(path.join(logDir, selected));
 			}
 		} catch (error) {
 			vscode.window.showErrorMessage(
@@ -181,11 +181,6 @@ export class Commands {
 		}
 	}
 
-	private async openFile(filePath: string): Promise<void> {
-		const uri = vscode.Uri.file(filePath);
-		await vscode.window.showTextDocument(uri);
-	}
-
 	/**
 	 * Log out and clear stored credentials, requiring re-authentication on next login.
 	 */
@@ -768,3 +763,22 @@ export class Commands {
 		});
 	}
 }
+
+async function openFile(filePath: string): Promise<void> {
+	const uri = vscode.Uri.file(filePath);
+	await vscode.window.showTextDocument(uri);
+}
+
+/**
+ * Read a directory's entries, returning an empty array if it does not exist.
+ */
+async function readdirOrEmpty(dirPath: string): Promise<string[]> {
+	try {
+		return await fs.readdir(dirPath);
+	} catch (err) {
+		if (err instanceof Error && "code" in err && err.code === "ENOENT") {
+			return [];
+		}
+		throw err;
+	}
+}