@myrialabs/clopen 0.0.3 → 0.0.5

package/.github/workflows/{release.yml → ci.yml}

@@ -1,17 +1,17 @@
-name: Release & Publish
+name: CI & Release
 
 on:
   push:
+    branches: [main, dev]
     tags:
       - 'v*.*.*'
+  pull_request:
+    branches: [main]
 
 jobs:
-  release:
-    name: Release Package
+  ci:
+    name: Test & Build
     runs-on: ubuntu-latest
-    permissions:
-      contents: write
-      id-token: write
 
     steps:
       - name: Checkout
@@ -22,12 +22,6 @@ jobs:
         with:
           bun-version: latest
 
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: '22'
-          registry-url: 'https://registry.npmjs.org'
-
       - name: Install
         run: bun install
 
@@ -47,8 +41,40 @@ jobs:
             exit 1
           fi
 
+  publish:
+    name: Publish & Release
+    needs: ci
+    if: startsWith(github.ref, 'refs/tags/v')
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write
+      id-token: write
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Bun
+        uses: oven-sh/setup-bun@v1
+        with:
+          bun-version: latest
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '22'
+          registry-url: 'https://registry.npmjs.org'
+
+      - name: Install
+        run: bun install
+
+      - name: Build
+        run: bun run build
+
       - name: Publish to npm
         run: npm publish --provenance --access public
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
       - name: Create GitHub release
         uses: softprops/action-gh-release@v2

package/backend/index.ts

@@ -1,6 +1,5 @@
 #!/usr/bin/env bun
 import { Elysia } from 'elysia';
-import { staticPlugin } from '@elysiajs/static';
 import { corsMiddleware } from './middleware/cors';
 import { errorHandlerMiddleware } from './middleware/error-handler';
 import { loggerMiddleware } from './middleware/logger';
@@ -11,6 +10,8 @@ import { disposeAllEngines } from './lib/engine';
 import { debug } from '$shared/utils/logger';
 import { isPortInUse } from './lib/shared/port-utils';
 import { networkInterfaces } from 'os';
+import { resolve } from 'node:path';
+import { statSync } from 'node:fs';
 
 // Import WebSocket router
 import { wsRouter } from './ws';
@@ -66,18 +67,26 @@ if (isDevelopment) {
 	// Store cleanup function for graceful shutdown
 	(globalThis as any).__closeViteDev = closeViteDev;
 } else {
-	// Production: Serve static frontend files
-	app.use(
-		staticPlugin({
-			assets: './dist',
-			prefix: '/'
-		})
-	);
-
-	// SPA fallback: serve index.html for all non-file routes
-	// Pass BunFile directly (not .stream()) so Bun sets Content-Length automatically
-	app.all('/*', async () => {
-		return new Response(Bun.file('./dist/index.html'), {
+	// Production: Read index.html once at startup as a string so Content-Length
+	// is always correct. Bun.file() streaming can hang on some platforms.
+	const distDir = resolve(process.cwd(), 'dist');
+	const indexHtml = await Bun.file(resolve(distDir, 'index.html')).text();
+
+	app.all('/*', ({ path }) => {
+		// Serve static files from dist/
+		if (path !== '/' && !path.includes('..')) {
+			const filePath = resolve(distDir, path.slice(1));
+			if (filePath.startsWith(distDir)) {
+				try {
+					if (statSync(filePath).isFile()) {
+						return new Response(Bun.file(filePath));
+					}
+				} catch {}
+			}
+		}
+
+		// SPA fallback: serve cached index.html
+		return new Response(indexHtml, {
 			headers: { 'Content-Type': 'text/html; charset=utf-8' }
 		});
 	});

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.3",
+  "version": "0.0.5",
   "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/shared/utils/logger.ts

@@ -3,19 +3,6 @@
  *
  * Provides filtered logging capabilities with label-based categorization.
  * All logs can be filtered by label, method type, and text content.
- *
- * Label Categories (22 total):
- * - Communication & Real-time: WebSocket, MCP operations
- * - File Operations: File I/O operations
- * - Chat & Notifications: Messages and notifications
- * - Terminal: Shell/PTY operations
- * - Preview: Browser preview and video encoding
- * - Data Persistence: Database, migrations, snapshots, checkpoints
- * - Engine & Processing: Engine operations
- * - User & Session: User and session management
- * - Configuration & Settings: Settings
- * - Infrastructure & Utilities: Server, git, project ops
- * - Frontend/UI State: Workspace and checkpoint UI state
  */
 
 export type LogLabel =
@@ -71,8 +58,7 @@ interface LoggerConfig {
 // enabled = true in development, false in production
 const config: LoggerConfig = {
 	enabled: process.env.NODE_ENV !== 'production',
-	filterLabels: ['snapshot', 'checkpoint', 'chat'],
-	// filterLabels: ['file'],
+	filterLabels: null,
 	filterMethods: null,
 	filterText: null
 };

package/.github/workflows/test.yml

@@ -1,40 +0,0 @@
-name: Test & Build
-
-on:
-  push:
-    branches: [main, dev]
-  pull_request:
-    branches: [main]
-
-jobs:
-  test:
-    name: Run Tests
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Setup Bun
-        uses: oven-sh/setup-bun@v1
-        with:
-          bun-version: latest
-
-      - name: Install
-        run: bun install
-
-      - name: Type check
-        run: bun run check
-
-      - name: Lint
-        run: bun run lint
-
-      - name: Build
-        run: bun run build
-
-      - name: Verify build
-        run: |
-          if [ ! -d "dist" ]; then
-            echo "Build failed - dist directory not found"
-            exit 1
-          fi