fix localstack instance status tracker

Author: skyrpex

src/utils/emitter.ts

@@ -32,9 +32,10 @@ export function createValueEmitter<T>(): ValueEmitter<T> {
 		},
 		onChange(callback) {
 			callbacks.push(callback);
-			if (currentValue) {
-				void Promise.resolve(callback(currentValue)).catch(() => {});
-			}
+			void Promise.resolve(callback(currentValue)).catch(() => {});
+			// if (currentValue !== undefined) {
+			// 	void Promise.resolve(callback(currentValue)).catch(() => {});
+			// }
 		},
 	};
 }

src/utils/localstack-container.ts

@@ -36,12 +36,19 @@ export function createLocalStackContainerStatusTracker(
 
 	void timeTracker.run("container-status.getContainerStatus", async () => {
 		await getContainerStatus(containerName).then((newStatus) => {
-			if (status.value() !== undefined) {
+			outputChannel.trace(
+				`[localstack-container-status] getContainerStatus=${newStatus} currentStatus=${status.value()}`,
+			);
+			if (status.value() === undefined) {
 				status.setValue(newStatus);
 			}
 		});
 	});
 
+	status.onChange((status) => {
+		outputChannel.trace(`[localstack-container-status] container=${status}`);
+	});
+
 	return {
 		status() {
 			return status.value();

src/utils/localstack-instance.ts

@@ -40,12 +40,17 @@ export function createLocalStackInstanceStatusTracker(
 	};
 
 	const deriveStatus = () => {
+		outputChannel.trace(
+			`[localstack-instance-status] containerStatus=${containerStatus} healthCheckStatusTracker=${healthCheckStatusTracker.status()} previousStatus=${status.value()}`,
+		);
 		const newStatus = getLocalStackStatus(
 			containerStatus,
 			healthCheckStatusTracker.status(),
 			status.value(),
 		);
-		setStatus(newStatus);
+		if (newStatus) {
+			setStatus(newStatus);
+		}
 	};
 
 	containerStatusTracker.onChange((newContainerStatus) => {
@@ -65,7 +70,7 @@ export function createLocalStackInstanceStatusTracker(
 
 	containerStatusTracker.onChange((newContainerStatus) => {
 		outputChannel.trace(
-			`[localstack-status] container=${newContainerStatus} (localstack=${status.value()})`,
+			`[localstack-instance-status] container=${newContainerStatus} (localstack=${status.value()})`,
 		);
 
 		if (newContainerStatus === "running" && status.value() !== "running") {