Handle review comments

Author: EhabY

src/commands.ts

@@ -147,12 +147,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"))
@@ -171,7 +171,7 @@ export class Commands {
 			});
 
 			if (selected) {
-				await this.openFile(path.join(logDir, selected));
+				await openFile(path.join(logDir, selected));
 			}
 		} catch (error) {
 			vscode.window.showErrorMessage(
@@ -180,11 +180,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;
+	}
+}