build(Sky): Standardize VS Code workbench loading with /vs/ URL paths

Replace @codeeditorland/output package specifiers with runtime /vs/ URL paths across all workbench entry points (Browser.astro, BrowserTest.astro, Mountain.astro, BrowserProxy/Workbench.ts).

Changes:
- Use /vs/code/browser/workbench/workbench.js (served from Sky/Public/vs → Output/Target/Microsoft/VSCode/vs symlink)
- Apply is:inline to bypass Vite/Rollup bundling entirely for inline scripts
- Use variable assignment (const WorkbenchUrl = "/vs/...") to make Rollup treat imports as truly dynamic (only string literals trigger analysis)
- Add try/catch error handling with console.error for graceful failure
- Update astro.config.ts external resolver to also match /vs/ prefix
- Comment out Electron workbench import (not yet compiled in Output)

This resolves Rollup's inability to resolve package specifiers at build time while maintaining dynamic loading at runtime.

Author: NikolaRHristov

Source/Workbench/Browser.astro

@@ -5,14 +5,14 @@ import NLS from "./NLS.astro";
 <Fragment>
 	<NLS />
 
-	<!-- Browser Workbench Script -->
-	<!-- Dynamic import matches the pattern used by Mountain.astro, BrowserTest.astro,
-	     BrowserProxy/Workbench.ts, and Electron/Workbench.ts. Static imports cause
-	     Rollup to hard-fail at build time (cannot resolve package specifiers).
-	     Dynamic imports only warn on resolution failure and allow the build to complete. -->
-	<script>
-		await import(
-			"@codeeditorland/output/vs/code/browser/workbench/workbench.js"
-		);
+	<!-- VS Code Browser Workbench — loaded via inline script to bypass Vite bundling.
+	     /vs/ is served from Sky/Public/vs → Output/Target/Microsoft/VSCode/vs (symlink).
+	     Relative imports inside workbench.js (../../../base/...) resolve against /vs/. -->
+	<script is:inline type="module">
+		try {
+			await import("/vs/code/browser/workbench/workbench.js");
+		} catch (error) {
+			console.error("[Browser] Failed to load VS Code browser workbench:", error);
+		}
 	</script>
 </Fragment>

Source/Workbench/BrowserProxy/Workbench.ts

@@ -12,8 +12,10 @@ interface MountainProxyWindow extends Window {
 console.log("[BrowserProxy] ===== Loading browser VSCode workbench =====");
 
 try {
-	// Import the browser workbench
-	await import("@codeeditorland/output/vs/code/browser/workbench/workbench.js");
+	// Variable URL: Rollup only analyzes string literals — an identifier
+	// reference is treated as truly dynamic and never resolved at build time.
+	const WorkbenchUrl = "/vs/code/browser/workbench/workbench.js";
+	await import(WorkbenchUrl);
 
 	console.log(
 		"[BrowserProxy] ✓ Browser workbench script loaded successfully",

Source/Workbench/BrowserTest.astro

@@ -73,17 +73,15 @@ import NLS from "./NLS.astro";
 			);
 	</script>
 
-	<!-- Browser Workbench Script -->
-	<script type="module">
-		await import("@codeeditorland/output/vs/code/browser/workbench/workbench.js");
-
-		console.log(
-			"[BrowserTest] ===== Loading browser VSCode workbench =====",
-		);
-		console.log(
-			"[BrowserTest] ✓ Browser workbench script loaded successfully",
-		);
-
+	<!-- Browser Workbench Script — is:inline bypasses Vite bundling entirely -->
+	<script is:inline type="module">
+		try {
+			await import("/vs/code/browser/workbench/workbench.js");
+			console.log("[BrowserTest] ===== Loading browser VSCode workbench =====");
+			console.log("[BrowserTest] ✓ Browser workbench script loaded successfully");
+		} catch (error) {
+			console.error("[BrowserTest] Failed to load VS Code browser workbench:", error);
+		}
 		console.log("[BrowserTest] ===== Workbench load attempted =====");
 	</script>
 </Fragment>

Source/Workbench/Electron/Workbench.ts

@@ -22,7 +22,9 @@ try {
 	// Import the Electron workbench (NOT browser workbench)
 	// Electron workbench uses Electron-specific APIs
 	// @ts-ignore - Dynamic import for side effects, .d.ts file is not a module but the .js file exists at runtime
-	await import("@codeeditorland/output/vs/code/electron-browser/workbench/workbench.js");
+	// electron-browser workbench.js is not compiled in Output — only .d.ts exists.
+	// When the Electron approach is activated, this path will need to be compiled first.
+	// await import("/vs/code/electron-browser/workbench/workbench.js");
 
 	console.log("[Electron] ✓ Electron workbench script loaded successfully");
 	console.log("[Electron] ===== Workbench load complete =====");

Source/Workbench/Mountain.astro

@@ -192,9 +192,10 @@ import NLS from "./NLS.astro";
 		);
 
 		try {
-			// Import the browser workbench (NOT Electron workbench)
-			// Browser workbench works without the vscode-file:// CSP errors
-			await import("@codeeditorland/output/vs/code/browser/workbench/workbench.js");
+			// Variable URL: Rollup only analyzes string literals — an identifier
+			// reference is treated as truly dynamic and never resolved at build time.
+			const WorkbenchUrl = "/vs/code/browser/workbench/workbench.js";
+			await import(WorkbenchUrl);
 
 			console.log(
 				"[Mountain] ✓ Browser workbench script loaded successfully",

astro.config.ts

@@ -72,9 +72,10 @@ export default defineConfig({
 				external: [
 					...External,
 					(id: string) =>
-						// Pre-resolved package specifier — catches @codeeditorland/output/vs/**
-						// before Rollup attempts resolution. Static imports of unresolved
-						// package specifiers are hard build errors; dynamic imports warn only.
+						// Absolute browser URL paths (/vs/...) — Rollup treats / as filesystem,
+						// but these are real browser URLs served at runtime. Mark external.
+						id.startsWith("/vs/") ||
+						// Package specifier — catches @codeeditorland/output/vs/**
 						id.startsWith("@codeeditorland/output/vs/") ||
 						// Resolved absolute path (after symlink + package.json exports map)
 						id.includes(