Show Setup Wizard pop-up once setup is required

This ensures that pop-up appears if setup is required at the start. Instead of synchronizing changes at the start we decided to have pop-up appearing every time the setup is broken. This shouldn't happen normally, but when it happens then setup wizard pop-up in addition to painting setup bar red can actually be helpful.

Author: tiurin

src/plugins/setup.ts

@@ -368,17 +368,22 @@ export default createPlugin(
 			),
 		);
 
-		if (setupStatusTracker.status() === "setup_required") {
-			window
-				.showInformationMessage("Setup LocalStack to get started.", {
-					title: "Setup",
-					command: "localstack.setup",
-				})
-				.then((selected) => {
-					if (selected) {
-						commands.executeCommand(selected.command, "extension_startup");
-					}
-				});
-		}
+		setupStatusTracker.onChange((status) => {
+			if (status === "setup_required") {
+				void window
+					.showInformationMessage("Setup LocalStack to get started.", {
+						title: "Setup",
+						command: "localstack.setup",
+					})
+					.then((selected) => {
+						if (selected) {
+							void commands.executeCommand(
+								selected.command,
+								"extension_startup",
+							);
+						}
+					});
+			}
+		});
 	},
 );

src/utils/setup-status.ts

@@ -47,6 +47,26 @@ export async function createSetupStatusTracker(
 	);
 
 	const checkStatusNow = async () => {
+		const allStatusesInitialized = Object.values({
+			awsProfileTracker: awsProfileTracker.status(),
+			authTracker: localStackAuthenticationTracker.status(),
+			licenseTracker: licenseTracker.status(),
+		}).every((check) => check !== undefined);
+
+		if (!allStatusesInitialized) {
+			outputChannel.trace(
+				`[setup-status] File watchers not initialized yet, skipping status check : ${JSON.stringify(
+					{
+						awsProfileTracker: awsProfileTracker.status() ?? "undefined",
+						authTracker:
+							localStackAuthenticationTracker.status() ?? "undefined",
+						licenseTracker: licenseTracker.status() ?? "undefined",
+					},
+				)}`,
+			);
+			return;
+		}
+
 		statuses = await checkSetupStatus(outputChannel);
 
 		const setupRequired = [
@@ -255,7 +275,7 @@ function createLicenseStatusTracker(
 	return createFileStatusTracker(
 		outputChannel,
 		"[setup-status.license]",
-		[LOCALSTACK_AUTH_FILENAME, LICENSE_FILENAME], //TODO rewrite to depend on change in localStackAuthenticationTracker
+		[LOCALSTACK_AUTH_FILENAME, LICENSE_FILENAME], //TODO rewrite to depend on change in localStackAuthenticationTrackerl
 		async () =>
 			(await checkIsLicenseValid(outputChannel)) ? "ok" : "setup_required",
 	);