Use containerized Astro build for cross-platform consistency

- Build Astro in Docker container (node:22-slim) instead of native npm
- Removes Windows-specific npm dependency issues
- Remove manually added Windows packages from package.json
- Regenerate package-lock.json in container for Linux compatibility
- Increase Node.js memory limit (4GB) to handle large builds

Author: damianh

astro/package.json

@@ -30,8 +30,6 @@
     "@fontsource/roboto": "^5.2.9",
     "@pasqal-io/starlight-client-mermaid": "^0.1.0",
     "@resvg/resvg-js": "^2.6.2",
-    "@resvg/resvg-js-win32-x64-msvc": "^2.6.2",
-    "@rollup/rollup-win32-x64-msvc": "^4.55.1",
     "astro": "^5.17.1",
     "astro-opengraph-images": "^1.14.2",
     "astro-redirect-from": "^1.3.4",

build.cs

@@ -13,13 +13,11 @@
         ?? throw new InvalidOperationException("Could not find repository root");
 }
 
-var astroDir = Path.Combine(repoRoot, "astro");
 var serverDir = Path.Combine(repoRoot, "server");
 var wwwrootDir = Path.Combine(serverDir, "Docs.Web", "wwwroot");
 
 // Target names
 const string Restore = "restore";
-const string AstroInstall = "astro-install";
 const string AstroBuild = "astro-build";
 const string DotnetBuild = "dotnet-build";
 const string DotnetPublish = "dotnet-publish";
@@ -31,18 +29,33 @@
 const string Clean = "clean";
 const string Default = "default";
 
-// Restore/Install
+// Restore
 Target(Restore, () =>
     RunAsync("dotnet", "restore Docs.slnx", workingDirectory: serverDir));
 
-// Use 'npm install' instead of 'npm ci' due to npm bug with optional dependencies on Windows
-// See: https://github.com/npm/cli/issues/4828
-Target(AstroInstall, () =>
-    RunAsync("npm", "install", workingDirectory: astroDir));
-
-// Build
-Target(AstroBuild, dependsOn: [AstroInstall], () =>
-    RunAsync("npm", "run build -- --outDir ../server/Docs.Web/wwwroot", workingDirectory: astroDir));
+// Astro build in container - avoids Windows npm issues with platform-specific dependencies
+Target(AstroBuild, async () =>
+{
+    // Ensure output directory exists
+    Directory.CreateDirectory(wwwrootDir);
+    
+    // Convert Windows paths to Docker-compatible format (forward slashes)
+    var astroPath = Path.Combine(repoRoot, "astro").Replace('\\', '/');
+    var outputPath = wwwrootDir.Replace('\\', '/');
+    
+    // Use Docker to build Astro - consistent across all platforms
+    // Use Debian-based image (not Alpine) for better compatibility with native modules
+    // Increase Node.js memory limit to handle large builds
+    await RunAsync("docker", 
+        "run --rm " +
+        $"-v \"{astroPath}:/app\" " +
+        $"-v \"{outputPath}:/output\" " +
+        "-w /app " +
+        "-e NODE_OPTIONS=\"--max-old-space-size=4096\" " +
+        "node:22-slim " +
+        "sh -c \"npm ci && npm run build -- --outDir /output\"",
+        configureEnvironment: env => env.Add("MSYS_NO_PATHCONV", "1"));
+});
 
 Target(DotnetBuild, dependsOn: [Restore], () =>
     RunAsync("dotnet", "build Docs.slnx --no-restore", workingDirectory: serverDir));