@myrialabs/clopen 0.0.2 → 0.0.4

package/backend/lib/vite-dev.ts

@@ -1,49 +1,25 @@
 /**
- * Vite Dev Server Integration (Bun-optimized)
+ * Vite Dev Server Integration
  *
  * Embeds Vite as middleware inside the Elysia/Bun server.
- *
- * Performance optimizations:
- * - HTML requests use vite.transformIndexHtml() directly (bypass Node compat layer)
- * - Module requests use vite.transformRequest() directly (bypass Node compat layer)
- * - Only edge cases (Vite internals, pre-bundled deps) go through middleware adapter
- *
- * Reliability:
- * - All async operations have timeouts to prevent hanging promises
- * - HTML has raw fallback if Vite transform hangs
- * - Middleware adapter has safety timeout
+ * Uses Vite's direct APIs (transformIndexHtml, transformRequest) for speed,
+ * with Node.js compat middleware adapter only for Vite internals (HMR, pre-bundled deps).
  */
 
 import { createServer as createViteServer, type ViteDevServer, type Connect } from 'vite';
 import { IncomingMessage, ServerResponse } from 'node:http';
 import { resolve } from 'node:path';
+import { statSync } from 'node:fs';
 
 let vite: ViteDevServer | null = null;
 
-// Resolved paths (computed once at startup)
 const PUBLIC_DIR = resolve(process.cwd(), 'static');
 const INDEX_PATH = resolve(process.cwd(), 'index.html');
 
-// Timeouts (ms)
-const HTML_TRANSFORM_TIMEOUT = 5000;
-const MODULE_TRANSFORM_TIMEOUT = 10000;
+// Safety timeout for the Node.js compat middleware adapter (ms).
+// Prevents hanging if Vite middleware never calls res.end() or next().
 const MIDDLEWARE_TIMEOUT = 10000;
 
-// ============================================================================
-// Utilities
-// ============================================================================
-
-/**
- * Race a promise against a timeout. Returns null on timeout (never rejects).
- */
-function withTimeout<T>(promise: Promise<T>, ms: number): Promise<T | null> {
-	let timer: ReturnType<typeof setTimeout>;
-	return Promise.race([
-		promise,
-		new Promise<null>((resolve) => { timer = setTimeout(() => resolve(null), ms); })
-	]).finally(() => clearTimeout(timer));
-}
-
 // ============================================================================
 // Lifecycle
 // ============================================================================
@@ -73,45 +49,26 @@ export async function closeViteDev(): Promise<void> {
 // Main Request Handler
 // ============================================================================
 
-/**
- * Handle an HTTP request in dev mode.
- * Uses Vite's direct APIs for speed, middleware only as fallback.
- * All paths have timeouts to guarantee a response — never hangs.
- */
 export async function handleDevRequest(viteServer: ViteDevServer, request: Request): Promise<Response> {
 	const url = new URL(request.url);
 	const pathname = url.pathname;
 
-	// 1. Static public files — fast Bun.file() (no Vite overhead)
-	const publicFile = await servePublicFile(pathname);
+	// 1. Static public files
+	const publicFile = servePublicFile(pathname);
 	if (publicFile) return publicFile;
 
-	// 2. HTML / SPA routes — direct vite.transformIndexHtml() with timeout + fallback
+	// 2. HTML / SPA routes
 	if (isHtmlRequest(pathname)) {
-		const htmlResponse = await withTimeout(serveHtml(viteServer, pathname), HTML_TRANSFORM_TIMEOUT);
-		if (htmlResponse) return htmlResponse;
-
-		// Fallback: serve raw HTML without Vite transforms (HMR won't work but page loads)
-		try {
-			const rawHtml = await Bun.file(INDEX_PATH).text();
-			return new Response(rawHtml, {
-				headers: { 'Content-Type': 'text/html; charset=utf-8' }
-			});
-		} catch {
-			// INDEX_PATH doesn't exist — fall through
-		}
+		return serveHtml(viteServer, pathname);
 	}
 
-	// 3. Module requests — direct vite.transformRequest() with timeout
+	// 3. Module requests (skip Vite internals like /__vite_hmr)
 	if (!pathname.startsWith('/__')) {
-		const moduleResponse = await withTimeout(
-			serveModule(viteServer, pathname + url.search, request),
-			MODULE_TRANSFORM_TIMEOUT
-		);
+		const moduleResponse = await serveModule(viteServer, pathname + url.search, request);
 		if (moduleResponse) return moduleResponse;
 	}
 
-	// 4. Fallback: pipe through Vite's connect middleware (with safety timeout)
+	// 4. Fallback: Vite connect middleware (for HMR, pre-bundled deps, etc.)
 	const middlewareResponse = await pipeViteMiddleware(viteServer.middlewares, request);
 	if (middlewareResponse) return middlewareResponse;
 
@@ -119,23 +76,26 @@ export async function handleDevRequest(viteServer: ViteDevServer, request: Reque
 }
 
 // ============================================================================
-// Fast Path: Public Files
+// Static Public Files
 // ============================================================================
 
-/**
- * Try serving a static file from the public directory (static/).
- * Uses BunFile directly (not .stream()) so Bun sets Content-Length automatically,
- * preventing endless loading when the browser can't detect stream end.
- */
-async function servePublicFile(pathname: string): Promise<Response | null> {
-	if (pathname.includes('..')) return null;
+function servePublicFile(pathname: string): Response | null {
+	if (pathname === '/' || pathname.includes('..')) return null;
 
 	const filePath = resolve(PUBLIC_DIR, pathname.slice(1));
 	if (!filePath.startsWith(PUBLIC_DIR)) return null;
 
-	const file = Bun.file(filePath);
-	if (!(await file.exists())) return null;
+	// Use statSync to verify the path is a regular file, not a directory.
+	// Bun.file().exists() returns inconsistent results for directories across
+	// platforms (Linux/macOS/Windows) and Bun versions, which can cause
+	// Response(Bun.file(directory)) to hang indefinitely.
+	try {
+		if (!statSync(filePath).isFile()) return null;
+	} catch {
+		return null;
+	}
 
+	const file = Bun.file(filePath);
 	return new Response(file, {
 		headers: {
 			'Content-Type': file.type || 'application/octet-stream',
@@ -145,12 +105,9 @@ async function servePublicFile(pathname: string): Promise<Response | null> {
 }
 
 // ============================================================================
-// Fast Path: HTML / SPA
+// HTML / SPA
 // ============================================================================
 
-/**
- * Check if a request is for an HTML page (root or SPA client-side route).
- */
 function isHtmlRequest(pathname: string): boolean {
 	if (pathname === '/') return true;
 	if (pathname.startsWith('/@') || pathname.startsWith('/__')) return false;
@@ -159,9 +116,6 @@ function isHtmlRequest(pathname: string): boolean {
 	return !lastSegment.includes('.');
 }
 
-/**
- * Serve transformed HTML directly via Vite's API.
- */
 async function serveHtml(viteServer: ViteDevServer, pathname: string): Promise<Response> {
 	const rawHtml = await Bun.file(INDEX_PATH).text();
 	const html = await viteServer.transformIndexHtml(pathname, rawHtml);
@@ -171,23 +125,19 @@ async function serveHtml(viteServer: ViteDevServer, pathname: string): Promise<R
 }
 
 // ============================================================================
-// Fast Path: Module Requests
+// Module Requests
 // ============================================================================
 
 function getModuleContentType(url: string): string {
 	const pathname = url.split('?')[0];
-	// Only serve as raw CSS when explicitly requested with ?direct (e.g. <link> tags).
-	// CSS imported via JS is always transformed to a JavaScript module by Vite
-	// (for HMR / style injection), so it must be served as application/javascript.
+	// Raw CSS only when explicitly requested with ?direct (e.g. <link> tags).
+	// CSS imported via JS is transformed to a JS module by Vite (for HMR).
 	if (pathname.endsWith('.css') && url.includes('direct')) {
 		return 'text/css';
 	}
 	return 'application/javascript';
 }
 
-/**
- * Try to serve a module using Vite's transformRequest API.
- */
 async function serveModule(viteServer: ViteDevServer, url: string, request: Request): Promise<Response | null> {
 	try {
 		const result = await viteServer.transformRequest(url);
@@ -214,7 +164,7 @@ async function serveModule(viteServer: ViteDevServer, url: string, request: Requ
 }
 
 // ============================================================================
-// Fallback: Vite Connect Middleware Adapter (with safety timeout)
+// Vite Connect Middleware Adapter
 // ============================================================================
 
 const MOCK_SOCKET = {
@@ -242,9 +192,8 @@ const MOCK_SOCKET = {
 };
 
 /**
- * Pipe a request through Vite's connect middleware.
- * Has a safety timeout to prevent promises that never resolve
- * (e.g., middleware errors without calling res.end() or next()).
+ * Bridges Web API Request → Node.js IncomingMessage/ServerResponse → Web API Response.
+ * Required because Vite's connect middleware uses the Node.js HTTP API.
  */
 function pipeViteMiddleware(
 	middleware: Connect.Server,
@@ -263,8 +212,6 @@ function pipeViteMiddleware(
 		let ended = false;
 		const chunks: Uint8Array[] = [];
 
-		// Safety timeout — if middleware never calls res.end() or next(),
-		// resolve with null to prevent hanging the HTTP response forever.
 		const safetyTimer = setTimeout(() => {
 			if (!ended) {
 				ended = true;
@@ -280,12 +227,9 @@ function pipeViteMiddleware(
 			const nodeHeaders = res.getHeaders();
 			const h = new Headers();
 			for (const key in nodeHeaders) {
-				// Skip hop-by-hop headers — these are transport-level headers that
-				// must NOT be forwarded through the adapter. Bun will set the correct
-				// Content-Length and Transfer-Encoding based on the actual body.
-				// Forwarding stale Content-Length from the Node.js ServerResponse
-				// can cause a mismatch with the collected body, making the browser
-				// wait for more data that never arrives (infinite loading spinner).
+				// Skip hop-by-hop headers — Bun sets correct Content-Length/Transfer-Encoding
+				// based on the actual body. Forwarding stale values from ServerResponse
+				// causes browser to wait for data that never arrives.
 				const lower = key.toLowerCase();
 				if (lower === 'transfer-encoding' || lower === 'content-length' ||
 					lower === 'connection' || lower === 'keep-alive') continue;
@@ -301,9 +245,7 @@ function pipeViteMiddleware(
 
 			const body = chunks.length === 0
 				? null
-				: chunks.length === 1
-					? chunks[0]
-					: Buffer.concat(chunks);
+				: Buffer.concat(chunks);
 
 			resolve(new Response(body, { status: res.statusCode, headers: h }));
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myrialabs/clopen",
-  "version": "0.0.2",
+  "version": "0.0.4",
   "description": "All-in-one web workspace for Claude Code & OpenCode — chat, terminal, git, browser preview, checkpoints, and real-time collaboration",
   "author": "Myria Labs",
   "license": "MIT",

package/vite.config.ts

@@ -4,7 +4,7 @@ import tailwindcss from '@tailwindcss/vite';
 import { resolve } from 'path';
 
 export default defineConfig({
-	plugins: [svelte(), tailwindcss()],
+	plugins: [tailwindcss(), svelte()],
 	publicDir: 'static',
 	build: {
 		outDir: 'dist',