@cloudflare/sandbox 0.12.10 → 0.12.12

package/dist/bridge/index.js

@@ -1,6 +1,6 @@
-import "../dist-B_eXrP83.js";
-import "../errors-CpkKPbWC.js";
-import { S as validateTunnelName, r as getSandbox, w as streamFile, x as validatePort, y as SandboxSecurityError } from "../sandbox-Co2NYKhX.js";
+import "../dist-CqPJfREN.js";
+import "../errors-CHhwDJNh.js";
+import { S as validateTunnelName, r as getSandbox, w as streamFile, x as validatePort, y as SandboxSecurityError } from "../sandbox-BYPanejK.js";
 import { DurableObject, env, tracing } from "cloudflare:workers";
 import { Hono } from "hono";
 
@@ -115,12 +115,23 @@ function validateSessionId(id) {
 /**
 * Convert an SSE stream from readFileStream() into a raw byte stream.
 * Decodes base64 chunks for binary files and UTF-8-encodes text chunks.
+* Reads up to the first chunk before returning. A missing file arrives as an
+* error event inside a successful stream, so it rejects here, while the caller
+* can still choose the HTTP status.
 */
-function sseToByteStream(sse) {
+async function sseToByteStream(sse) {
 	const encoder = new TextEncoder();
+	const enqueue = (controller, chunk) => {
+		controller.enqueue(chunk instanceof Uint8Array ? chunk : encoder.encode(chunk));
+	};
+	const chunks = streamFile(sse);
+	const first = await chunks.next();
 	return new ReadableStream({ async start(controller) {
 		try {
-			for await (const chunk of streamFile(sse)) controller.enqueue(chunk instanceof Uint8Array ? chunk : encoder.encode(chunk));
+			if (!first.done) {
+				enqueue(controller, first.value);
+				for await (const chunk of chunks) enqueue(controller, chunk);
+			}
 			controller.close();
 		} catch (err) {
 			controller.error(err);
@@ -2056,14 +2067,16 @@ function createBridgeApp(config) {
 			executor = await sandbox.getSession(sessionId);
 		}
 		try {
-			const stream = await executor.readFileStream(resolvedPath);
-			return new Response(sseToByteStream(stream), {
+			const byteStream = await sseToByteStream(await executor.readFileStream(resolvedPath));
+			return new Response(byteStream, {
 				status: 200,
 				headers: { "Content-Type": "application/octet-stream" }
 			});
 		} catch (err) {
-			if (err.code === "FILE_NOT_FOUND") return errorJson(`File not found: ${resolvedPath}`, "workspace_read_not_found", 404);
-			return errorJson(`read failed: ${err instanceof Error ? err.message : String(err)}`, "exec_transport_error", 502);
+			const code = err.code;
+			const msg = err instanceof Error ? err.message : String(err);
+			if (code === "FILE_NOT_FOUND" || msg.startsWith("File streaming error: File not found: ")) return errorJson(`File not found: ${resolvedPath}`, "workspace_read_not_found", 404);
+			return errorJson(`read failed: ${msg}`, "exec_transport_error", 502);
 		}
 	}));
 	app.put(`${apiPrefix}/sandbox/:id/file/*`, traced("write_file", async (c) => {
@@ -2185,7 +2198,7 @@ function createBridgeApp(config) {
 			if (result.exitCode !== 0) return errorJson(`tar failed (exit ${result.exitCode}): ${result.stderr}`, "workspace_archive_read_error", 502);
 			const stream = await sandbox.readFileStream(tmpPath);
 			sandbox.exec(`rm -f ${shellQuote(tmpPath)}`).catch(() => {});
-			return new Response(sseToByteStream(stream), {
+			return new Response(await sseToByteStream(stream), {
 				status: 200,
 				headers: { "Content-Type": "application/octet-stream" }
 			});