fix(Wind): Use Effect.either for proper fiber-level failure handling in bootstrap

Replace JavaScript try/catch with Effect.either in BootstrapImplementation to properly catch Effect failures from yield*. JavaScript try/catch cannot intercept fiber-level failures from suspended Effects - only Effect.either or Effect.catchAll can handle these.

Also simplify logsPath handling in ResolveConfiguration to use Mountain's pre-timestamped directory (e.g., .../logs/20260410T105248/window1/) directly, removing the redundant timestamp nesting that was being added client-side.

These changes ensure bootstrap stage failures are correctly captured and reported, and prevent duplicate log directory nesting.

Author: NikolaRHristov

Source/ESBuild.js

@@ -1,61 +1,59 @@
 export const Clean = process.env["Clean"] === "true";
 export const Meta = process.env["Meta"] === "true";
-export const On =
-	process.env["NODE_ENV"] === "development" ||
-	process.env["TAURI_ENV_DEBUG"] === "true";
+export const On = process.env["NODE_ENV"] === "development" ||
+    process.env["TAURI_ENV_DEBUG"] === "true";
 /**
  * @module ESBuild
  *
  */
 export default {
-	color: true,
-	format: "esm",
-	logLevel: On ? "debug" : "silent",
-	metafile: Meta,
-	minify: !On,
-	outbase: "Source/Configuration",
-	outdir: "Configuration",
-	platform: "node",
-	target: "esnext",
-	tsconfig: "tsconfig.json",
-	write: true,
-	legalComments: On ? "inline" : "none",
-	bundle: false,
-	assetNames: "Asset/[name]-[hash]",
-	sourcemap: On,
-	drop: On ? [] : ["debugger"],
-	ignoreAnnotations: !On,
-	keepNames: On,
-	plugins: [
-		{
-			name: "Target",
-			// @ts-ignore
-			setup({ onStart, initialOptions: { outdir } }) {
-				switch (true) {
-					case Clean === true:
-						onStart(async () => {
-							try {
-								outdir
-									? await (
-											await import("node:fs/promises")
-										).rm(outdir, {
-											recursive: true,
-										})
-									: {};
-							} catch (_Error) {
-								console.log(_Error);
-							}
-						});
-						break;
-					default:
-						break;
-				}
-			},
-		},
-	],
-	loader: {
-		".json": "copy",
-		".sh": "copy",
-	},
+    color: true,
+    format: "esm",
+    logLevel: On ? "debug" : "silent",
+    metafile: Meta,
+    minify: !On,
+    outbase: "Source/Configuration",
+    outdir: "Configuration",
+    platform: "node",
+    target: "esnext",
+    tsconfig: "tsconfig.json",
+    write: true,
+    legalComments: On ? "inline" : "none",
+    bundle: false,
+    assetNames: "Asset/[name]-[hash]",
+    sourcemap: On,
+    drop: On ? [] : ["debugger"],
+    ignoreAnnotations: !On,
+    keepNames: On,
+    plugins: [
+        {
+            name: "Target",
+            // @ts-ignore
+            setup({ onStart, initialOptions: { outdir } }) {
+                switch (true) {
+                    case Clean === true:
+                        onStart(async () => {
+                            try {
+                                outdir
+                                    ? await (await import("node:fs/promises")).rm(outdir, {
+                                        recursive: true,
+                                    })
+                                    : {};
+                            }
+                            catch (_Error) {
+                                console.log(_Error);
+                            }
+                        });
+                        break;
+                    default:
+                        break;
+                }
+            },
+        },
+    ],
+    loader: {
+        ".json": "copy",
+        ".sh": "copy",
+    },
 };
 export const { sep, posix } = await import("node:path");

Source/Effect/Bootstrap/Implementation/BootstrapImplementation.ts

@@ -8,7 +8,7 @@
  * @category Implementation
  */
 
-import { Effect, Layer } from "effect";
+import { Effect, Either, Layer } from "effect";
 
 import { Telemetry } from "../../Telemetry.js";
 import type { BootstrapService } from "../Interface/BootstrapService.js";
@@ -76,22 +76,27 @@ const makeBootstrap = (): BootstrapService => ({
 			for (const Stage of Stages) {
 				const StageStartTime = Date.now();
 				let Result: StageResult;
-				try {
-					// @ts-expect-error - Effect stages have different requirements that runtime handles correctly
-					const StageResult = yield* Effect.suspend(
-						() => Stage,
-					) as any;
+				// Use Effect.either to catch fiber-level failures (missing services, etc.)
+				// JavaScript try/catch does NOT catch Effect failures from yield*.
+				const Outcome = yield* Effect.either(
+					Effect.suspend(() => Stage) as unknown as Effect.Effect<StageResult, unknown>,
+				);
+				if (Either.isRight(Outcome)) {
 					Result = {
-						...StageResult,
+						...Outcome.right,
 						duration: Date.now() - StageStartTime,
 					};
-				} catch (E) {
-					const Error = E instanceof Error ? E : new Error(String(E));
+				} else {
+					const FailCause = Outcome.left;
+					const ErrorObj =
+						FailCause instanceof Error
+							? FailCause
+							: new Error(String(FailCause));
 					Result = {
 						stageName: "Unknown",
 						success: false as boolean,
 						duration: Date.now() - StageStartTime,
-						error: Error,
+						error: ErrorObj,
 					} satisfies StageResult;
 				}
 				Results.push(Result);

Source/Function/Install/Function/ResolveConfiguration.ts

@@ -81,20 +81,10 @@ export async function ResolveConfiguration(): Promise<ISandboxConfiguration> {
 	DevLog("config", "folderUri:", JSON.stringify(FolderUri));
 	DevLog("config", "workspace:", JSON.stringify(Workspace));
 
-	// Session timestamp for logs subdirectory
-	const Now = new Date();
-	const SessionTimestamp = [
-		Now.getFullYear(),
-		String(Now.getMonth() + 1).padStart(2, "0"),
-		String(Now.getDate()).padStart(2, "0"),
-		"T",
-		String(Now.getHours()).padStart(2, "0"),
-		String(Now.getMinutes()).padStart(2, "0"),
-		String(Now.getSeconds()).padStart(2, "0"),
-	].join("");
-	const LogsLocation = Paths.logsPath
-		? `${Paths.logsPath}/${SessionTimestamp}`
-		: undefined;
+	// Mountain returns logsPath as a session-timestamped directory
+	// (e.g., .../logs/20260410T105248) with window1/ already created.
+	// Use it directly — no additional timestamp nesting needed.
+	const LogsLocation = Paths.logsPath || undefined;
 
 	return {
 		windowId: 1,