Fix serialization issue and workspace provider refresh

Author: EhabY

src/core/secretsManager.ts

@@ -44,8 +44,14 @@ export class SecretsManager {
 	public async setCurrentDeployment(
 		deployment: Deployment | undefined,
 	): Promise<void> {
-		const state = {
-			deployment: deployment ?? null,
+		const state: CurrentDeploymentState & { timestamp: string } = {
+			// Extract the necessary fields before serializing
+			deployment: deployment
+				? {
+						url: deployment?.url,
+						safeHostname: deployment?.safeHostname,
+					}
+				: null,
 			timestamp: new Date().toISOString(),
 		};
 		await this.secrets.store(CURRENT_DEPLOYMENT_KEY, JSON.stringify(state));
@@ -132,7 +138,9 @@ export class SecretsManager {
 		auth: SessionAuth,
 	): Promise<void> {
 		const sessionKey = this.getSessionKey(safeHostname);
-		await this.secrets.store(sessionKey, JSON.stringify(auth));
+		// Extract only url and token before serializing
+		const state: SessionAuth = { url: auth.url, token: auth.token };
+		await this.secrets.store(sessionKey, JSON.stringify(state));
 		await this.recordDeploymentAccess(safeHostname);
 	}
 

src/deployment/deploymentManager.ts

@@ -80,7 +80,6 @@ export class DeploymentManager implements vscode.Disposable {
 	): Promise<void> {
 		this.setDeployment(deployment);
 
-		this.refreshWorkspaces();
 		await this.persistDeployment(deployment);
 	}
 
@@ -98,14 +97,15 @@ export class DeploymentManager implements vscode.Disposable {
 	}
 
 	private setDeployment(deployment: DeploymentWithAuth): void {
+		this.currentDeployment = { ...deployment };
 		if (deployment.token === undefined) {
 			this.client.setHost(deployment.url);
 		} else {
 			this.client.setCredentials(deployment.url, deployment.token);
 		}
 		this.registerAuthListener(deployment.safeHostname);
-		this.currentDeployment = { ...deployment };
 		this.updateAuthContexts(deployment.user);
+		this.refreshWorkspaces();
 	}
 
 	/**
@@ -137,8 +137,8 @@ export class DeploymentManager implements vscode.Disposable {
 						return;
 					}
 
-					this.logger.info("Deployment changed from another window");
 					if (deployment) {
+						this.logger.info("Deployment changed from another window");
 						const auth = await this.secretsManager.getSessionAuth(
 							deployment.safeHostname,
 						);