@cloudflare/sandbox 0.0.0-2b087c4 → 0.0.0-2f85e95

package/CHANGELOG.md

@@ -1,5 +1,43 @@
 # @cloudflare/sandbox
 
+## 0.0.8
+
+### Patch Changes
+
+- [`60af265`](https://github.com/cloudflare/sandbox-sdk/commit/60af265d834e83fd30a921a3e1be232f13fe24da) Thanks [@threepointone](https://github.com/threepointone)! - update dependencies
+
+## 0.0.7
+
+### Patch Changes
+
+- [`d1c7c99`](https://github.com/cloudflare/sandbox-sdk/commit/d1c7c99df6555eff71bcd59852e4b8eed2ad8cb6) Thanks [@threepointone](https://github.com/threepointone)! - fix file operations
+
+## 0.0.6
+
+### Patch Changes
+
+- [#9](https://github.com/cloudflare/sandbox-sdk/pull/9) [`24f5470`](https://github.com/cloudflare/sandbox-sdk/commit/24f547048d5a26137de4656cea13d83ad2cc0b43) Thanks [@ItsWendell](https://github.com/ItsWendell)! - fix baseUrl for stub and stub forwarding
+
+## 0.0.5
+
+### Patch Changes
+
+- [#5](https://github.com/cloudflare/sandbox-sdk/pull/5) [`7c15b81`](https://github.com/cloudflare/sandbox-sdk/commit/7c15b817899e4d9e1f25747aaf439e5e9e880d15) Thanks [@ghostwriternr](https://github.com/ghostwriternr)! - Make package ready for deployment
+
+## 0.0.4
+
+### Patch Changes
+
+- [`c0d9d33`](https://github.com/cloudflare/sandbox-sdk/commit/c0d9d3396badee1eab45e6b4a73d48957f31409b) Thanks [@threepointone](https://github.com/threepointone)! - actually work
+
+- [`444d2da`](https://github.com/cloudflare/sandbox-sdk/commit/444d2dafde9a0f190e50c879b0e768da1b289b51) Thanks [@threepointone](https://github.com/threepointone)! - add experimental label
+
+## 0.0.3
+
+### Patch Changes
+
+- [`2b087c4`](https://github.com/cloudflare/sandbox-sdk/commit/2b087c40a29697c20dad19b4e3b8512f5d404bd3) Thanks [@ghostwriternr](https://github.com/ghostwriternr)! - Fix worker unable to find container port
+
 ## 0.0.2
 
 ### Patch Changes

package/README.md

@@ -1,5 +1,7 @@
 ## @cloudflare/sandbox
 
+> **⚠️ Experimental** - This library is currently experimental and we're actively seeking feedback. Please try it out and let us know what you think!
+
 A library to spin up a sandboxed environment.
 
 First, setup your wrangler.json to use the sandbox:

package/container_src/index.ts

@@ -84,6 +84,8 @@ const server = serve({
     const url = new URL(req.url);
     const pathname = url.pathname;
 
+    console.log(`[Container] Incoming ${req.method} request to ${pathname}`);
+
     // Handle CORS
     const corsHeaders = {
       "Access-Control-Allow-Headers": "Content-Type, Authorization",
@@ -93,11 +95,13 @@ const server = serve({
 
     // Handle preflight requests
     if (req.method === "OPTIONS") {
+      console.log(`[Container] Handling CORS preflight for ${pathname}`);
       return new Response(null, { headers: corsHeaders, status: 200 });
     }
 
     try {
       // Handle different routes
+      console.log(`[Container] Processing ${req.method} ${pathname}`);
       switch (pathname) {
         case "/":
           return new Response("Hello from Bun server! 🚀", {
@@ -352,13 +356,14 @@ const server = serve({
           break;
 
         default:
+          console.log(`[Container] Route not found: ${pathname}`);
           return new Response("Not Found", {
             headers: corsHeaders,
             status: 404,
           });
       }
     } catch (error) {
-      console.error("[Server] Error handling request:", error);
+      console.error(`[Container] Error handling ${req.method} ${pathname}:`, error);
       return new Response(
         JSON.stringify({
           error: "Internal server error",
@@ -374,6 +379,7 @@ const server = serve({
       );
     }
   },
+  hostname: "0.0.0.0",
   port: 3000,
 } as any);
 
@@ -2874,7 +2880,7 @@ function executeMoveFile(
   });
 }
 
-console.log(`🚀 Bun server running on http://localhost:${server.port}`);
+console.log(`🚀 Bun server running on http://0.0.0.0:${server.port}`);
 console.log(`📡 HTTP API endpoints available:`);
 console.log(`   POST /api/session/create - Create a new session`);
 console.log(`   GET  /api/session/list - List all sessions`);

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@cloudflare/sandbox",
-  "version": "0.0.0-2b087c4",
+  "version": "0.0.0-2f85e95",
   "repository": {
     "type": "git",
     "url": "https://github.com/cloudflare/sandbox-sdk"
   },
   "description": "A sandboxed environment for running commands",
   "dependencies": {
-    "@cloudflare/containers": "^0.0.12"
+    "@cloudflare/containers": "^0.0.19"
   },
   "tags": [
     "sandbox",
@@ -17,7 +17,9 @@
     "durable objects"
   ],
   "scripts": {
-    "build": "tsup src/*.ts --outDir dist --dts --sourcemap --format esm"
+    "build": "rm -rf dist && tsup src/*.ts --outDir dist --dts --sourcemap --format esm",
+    "docker:build": "docker build -t ghostwriternr/cloudflare-sandbox:$npm_package_version .",
+    "docker:publish": "docker push docker.io/ghostwriternr/cloudflare-sandbox:$npm_package_version"
   },
   "exports": {
     ".": {

package/src/client.ts

@@ -203,10 +203,41 @@ export class HttpClient {
     path: string,
     options?: RequestInit
   ): Promise<Response> {
-    if (this.options.stub) {
-      return this.options.stub.containerFetch(path, options, this.options.port);
+    const url = this.options.stub
+      ? `http://localhost:${this.options.port}${path}`
+      : `${this.baseUrl}${path}`;
+    const method = options?.method || "GET";
+
+    console.log(`[HTTP Client] Making ${method} request to ${url}`);
+
+    try {
+      let response: Response;
+
+      if (this.options.stub) {
+        response = await this.options.stub.containerFetch(
+          url,
+          options,
+          this.options.port
+        );
+      } else {
+        response = await fetch(url, options);
+      }
+
+      console.log(
+        `[HTTP Client] Response: ${response.status} ${response.statusText}`
+      );
+
+      if (!response.ok) {
+        console.error(
+          `[HTTP Client] Request failed: ${method} ${url} - ${response.status} ${response.statusText}`
+        );
+      }
+
+      return response;
+    } catch (error) {
+      console.error(`[HTTP Client] Request error: ${method} ${url}`, error);
+      throw error;
     }
-    return fetch(this.baseUrl + path, options);
   }
   // Public methods to set event handlers
   setOnOutput(

package/src/index.ts

@@ -8,24 +8,31 @@ export function getSandbox(ns: DurableObjectNamespace<Sandbox>, id: string) {
 export class Sandbox<Env = unknown> extends Container<Env> {
   defaultPort = 3000; // The default port for the container to listen on
   sleepAfter = "3m"; // Sleep the sandbox if no requests are made in this timeframe
+  client: HttpClient;
 
-  client: HttpClient = new HttpClient({
-    onCommandComplete: (success, exitCode, stdout, stderr, command, args) => {
-      console.log(
-        `[Container] Command completed: ${command}, Success: ${success}, Exit code: ${exitCode}`
-      );
-    },
-    onCommandStart: (command, args) => {
-      console.log(`[Container] Command started: ${command} ${args.join(" ")}`);
-    },
-    onError: (error, command, args) => {
-      console.error(`[Container] Command error: ${error}`);
-    },
-    onOutput: (stream, data, command) => {
-      console.log(`[Container] [${stream}] ${data}`);
-    },
-    port: this.defaultPort,
-  });
+  constructor(ctx: DurableObjectState, env: Env) {
+    super(ctx, env);
+    this.client = new HttpClient({
+      onCommandComplete: (success, exitCode, stdout, stderr, command, args) => {
+        console.log(
+          `[Container] Command completed: ${command}, Success: ${success}, Exit code: ${exitCode}`
+        );
+      },
+      onCommandStart: (command, args) => {
+        console.log(
+          `[Container] Command started: ${command} ${args.join(" ")}`
+        );
+      },
+      onError: (error, command, args) => {
+        console.error(`[Container] Command error: ${error}`);
+      },
+      onOutput: (stream, data, command) => {
+        console.log(`[Container] [${stream}] ${data}`);
+      },
+      port: this.defaultPort,
+      stub: this,
+    });
+  }
 
   envVars = {
     MESSAGE: "I was passed in via the Sandbox class!",
@@ -69,7 +76,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
 
   async mkdir(
     path: string,
-    options: { recursive?: boolean; stream?: boolean }
+    options: { recursive?: boolean; stream?: boolean } = {}
   ) {
     if (options?.stream) {
       return this.client.mkdirStream(path, options.recursive);
@@ -80,7 +87,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
   async writeFile(
     path: string,
     content: string,
-    options: { encoding?: string; stream?: boolean }
+    options: { encoding?: string; stream?: boolean } = {}
   ) {
     if (options?.stream) {
       return this.client.writeFileStream(path, content, options.encoding);
@@ -88,7 +95,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
     return this.client.writeFile(path, content, options.encoding);
   }
 
-  async deleteFile(path: string, options: { stream?: boolean }) {
+  async deleteFile(path: string, options: { stream?: boolean } = {}) {
     if (options?.stream) {
       return this.client.deleteFileStream(path);
     }
@@ -98,7 +105,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
   async renameFile(
     oldPath: string,
     newPath: string,
-    options: { stream?: boolean }
+    options: { stream?: boolean } = {}
   ) {
     if (options?.stream) {
       return this.client.renameFileStream(oldPath, newPath);
@@ -109,7 +116,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
   async moveFile(
     sourcePath: string,
     destinationPath: string,
-    options: { stream?: boolean }
+    options: { stream?: boolean } = {}
   ) {
     if (options?.stream) {
       return this.client.moveFileStream(sourcePath, destinationPath);
@@ -119,7 +126,7 @@ export class Sandbox<Env = unknown> extends Container<Env> {
 
   async readFile(
     path: string,
-    options: { encoding?: string; stream?: boolean }
+    options: { encoding?: string; stream?: boolean } = {}
   ) {
     if (options?.stream) {
       return this.client.readFileStream(path, options.encoding);