Deployment management refactoring

Author: EhabY

src/commands.ts

@@ -13,7 +13,7 @@ import { type ContextManager } from "./core/contextManager";
 import { type MementoManager } from "./core/mementoManager";
 import { type PathResolver } from "./core/pathResolver";
 import { type SecretsManager } from "./core/secretsManager";
-import { type DeploymentManager } from "./deployment";
+import { type DeploymentManager } from "./deployment/deploymentManager";
 import { CertificateError } from "./error";
 import { type Logger } from "./logging/logger";
 import { type LoginCoordinator } from "./login/loginCoordinator";

src/core/secretsManager.ts

@@ -3,7 +3,7 @@ import { toSafeHost } from "../util";
 
 import type { Memento, SecretStorage, Disposable } from "vscode";
 
-import type { Deployment } from "../deployment";
+import type { Deployment } from "../deployment/types";
 
 // Each deployment has its own key to ensure atomic operations (multiple windows
 // writing to a shared key could drop data) and to receive proper VS Code events.

src/deployment/deploymentManager.ts

@@ -1,20 +1,16 @@
-import { type CoderApi } from "../api/coderApi";
-import { type ServiceContainer } from "../core/container";
-import { type ContextManager } from "../core/contextManager";
-import { type MementoManager } from "../core/mementoManager";
-import { type SecretsManager } from "../core/secretsManager";
-import { type Logger } from "../logging/logger";
-import { type WorkspaceProvider } from "../workspace/workspacesProvider";
-
-import {
-	type Deployment,
-	type DeploymentWithAuth,
-	isAuthenticated,
-} from "./types";
-
 import type { User } from "coder/site/src/api/typesGenerated";
 import type * as vscode from "vscode";
 
+import type { CoderApi } from "../api/coderApi";
+import type { ServiceContainer } from "../core/container";
+import type { ContextManager } from "../core/contextManager";
+import type { MementoManager } from "../core/mementoManager";
+import type { SecretsManager } from "../core/secretsManager";
+import type { Logger } from "../logging/logger";
+import type { WorkspaceProvider } from "../workspace/workspacesProvider";
+
+import type { Deployment, DeploymentWithAuth } from "./types";
+
 /**
  * Manages deployment state for the extension.
  *
@@ -72,7 +68,7 @@ export class DeploymentManager implements vscode.Disposable {
 	 * Check if we have an authenticated deployment (with a valid user).
 	 */
 	public isAuthenticated(): boolean {
-		return isAuthenticated(this.currentDeployment);
+		return this.currentDeployment?.user !== undefined;
 	}
 
 	/**

src/deployment/index.ts

@@ -17,13 +17,7 @@ export interface Deployment {
  * - Undefined token means that we should not override the existing token (if any).
  * - Undefined user means the deployment is set but not authenticated yet.
  */
-export type DeploymentWithAuth = Deployment & { token?: string; user?: User };
-
-/**
- * Type guard to check if a deployment has a valid authenticated user.
- */
-export function isAuthenticated(
-	deployment: DeploymentWithAuth | null,
-): deployment is Deployment & { user: User } {
-	return deployment?.user !== undefined;
-}
+export type DeploymentWithAuth = Deployment & {
+	readonly token?: string;
+	readonly user?: User;
+};

src/deployment/types.ts

@@ -12,7 +12,7 @@ import { needToken } from "./api/utils";
 import { Commands } from "./commands";
 import { ServiceContainer } from "./core/container";
 import { type SecretsManager } from "./core/secretsManager";
-import { DeploymentManager } from "./deployment";
+import { DeploymentManager } from "./deployment/deploymentManager";
 import { CertificateError, getErrorDetail } from "./error";
 import { maybeAskUrl } from "./promptUtils";
 import { Remote } from "./remote/remote";

src/extension.ts

@@ -3,15 +3,16 @@ import * as vscode from "vscode";
 
 import { CoderApi } from "../api/coderApi";
 import { needToken } from "../api/utils";
-import { type MementoManager } from "../core/mementoManager";
-import { type SecretsManager } from "../core/secretsManager";
-import { type Deployment } from "../deployment";
 import { CertificateError } from "../error";
-import { type Logger } from "../logging/logger";
 import { maybeAskUrl } from "../promptUtils";
 
 import type { User } from "coder/site/src/api/typesGenerated";
 
+import type { MementoManager } from "../core/mementoManager";
+import type { SecretsManager } from "../core/secretsManager";
+import type { Deployment } from "../deployment/types";
+import type { Logger } from "../logging/logger";
+
 interface LoginResult {
 	success: boolean;
 	user?: User;