Review comments

Author: EhabY

package.json

@@ -289,7 +289,7 @@
 			},
 			{
 				"command": "coder.manageCredentials",
-				"title": "Manage Stored Credentials",
+				"title": "Manage Credentials",
 				"category": "Coder"
 			}
 		],

src/commands.ts

@@ -17,6 +17,7 @@ import { type PathResolver } from "./core/pathResolver";
 import { type SecretsManager } from "./core/secretsManager";
 import { type DeploymentManager } from "./deployment/deploymentManager";
 import { CertificateError } from "./error/certificateError";
+import { toError } from "./error/errorUtils";
 import { type Logger } from "./logging/logger";
 import { type LoginCoordinator } from "./login/loginCoordinator";
 import { maybeAskAgent, maybeAskUrl } from "./promptUtils";
@@ -258,7 +259,7 @@ export class Commands {
 			const items: Array<{
 				label: string;
 				description: string;
-				hostname: string | undefined;
+				hostname: string | null;
 			}> = hostnames.map((hostname) => ({
 				label: `$(key) ${hostname}`,
 				description: "Remove stored credentials",
@@ -270,7 +271,7 @@ export class Commands {
 				items.push({
 					label: "$(trash) Remove All",
 					description: `Remove credentials for all ${hostnames.length} deployments`,
-					hostname: undefined,
+					hostname: null,
 				});
 			}
 
@@ -285,6 +286,7 @@ export class Commands {
 
 			if (selected.hostname) {
 				await this.secretsManager.clearAllAuthData(selected.hostname);
+				this.logger.info("Removed credentials for", selected.hostname);
 				vscode.window.showInformationMessage(
 					`Removed credentials for ${selected.hostname}`,
 				);
@@ -302,6 +304,10 @@ export class Commands {
 					await Promise.all(
 						hostnames.map((h) => this.secretsManager.clearAllAuthData(h)),
 					);
+					this.logger.info(
+						"Removed credentials for all deployments:",
+						hostnames.join(", "),
+					);
 					vscode.window.showInformationMessage(
 						"Removed credentials for all deployments",
 					);
@@ -310,7 +316,7 @@ export class Commands {
 		} catch (error: unknown) {
 			this.logger.error("Failed to manage stored credentials", error);
 			vscode.window.showErrorMessage(
-				"Failed to manage stored credentials. Storage may be corrupted.",
+				`Failed to manage stored credentials: ${toError(error).message}`,
 			);
 		}
 	}