@cloudflare/sandbox 0.0.0-fddccfd → 0.0.0-ff2fa91

package/src/clients/interpreter-client.ts

@@ -6,16 +6,20 @@ import {
   type ExecutionError,
   type OutputMessage,
   type Result,
-  ResultImpl,
+  ResultImpl
 } from '@repo/shared';
 import type { ErrorResponse } from '../errors';
-import { createErrorFromResponse, ErrorCode, InterpreterNotReadyError } from '../errors';
+import {
+  createErrorFromResponse,
+  ErrorCode,
+  InterpreterNotReadyError
+} from '../errors';
 import { BaseHttpClient } from './base-client.js';
 import type { HttpClientOptions } from './types.js';
 
 // Streaming execution data from the server
 interface StreamingExecutionData {
-  type: "result" | "stdout" | "stderr" | "error" | "execution_complete";
+  type: 'result' | 'stdout' | 'stderr' | 'error' | 'execution_complete';
   text?: string;
   html?: string;
   png?: string; // base64
@@ -27,13 +31,13 @@ interface StreamingExecutionData {
   json?: unknown;
   chart?: {
     type:
-      | "line"
-      | "bar"
-      | "scatter"
-      | "pie"
-      | "histogram"
-      | "heatmap"
-      | "unknown";
+      | 'line'
+      | 'bar'
+      | 'scatter'
+      | 'pie'
+      | 'histogram'
+      | 'heatmap'
+      | 'unknown';
     data: unknown;
     options?: unknown;
   };
@@ -62,14 +66,14 @@ export class InterpreterClient extends BaseHttpClient {
     options: CreateContextOptions = {}
   ): Promise<CodeContext> {
     return this.executeWithRetry(async () => {
-      const response = await this.doFetch("/api/contexts", {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
+      const response = await this.doFetch('/api/contexts', {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
-          language: options.language || "python",
-          cwd: options.cwd || "/workspace",
-          env_vars: options.envVars,
-        }),
+          language: options.language || 'python',
+          cwd: options.cwd || '/workspace',
+          env_vars: options.envVars
+        })
       });
 
       if (!response.ok) {
@@ -87,7 +91,7 @@ export class InterpreterClient extends BaseHttpClient {
         language: data.language,
         cwd: data.cwd || '/workspace',
         createdAt: new Date(data.timestamp),
-        lastUsed: new Date(data.timestamp),
+        lastUsed: new Date(data.timestamp)
       };
     });
   }
@@ -100,18 +104,18 @@ export class InterpreterClient extends BaseHttpClient {
     timeoutMs?: number
   ): Promise<void> {
     return this.executeWithRetry(async () => {
-      const response = await this.doFetch("/api/execute/code", {
-        method: "POST",
+      const response = await this.doFetch('/api/execute/code', {
+        method: 'POST',
         headers: {
-          "Content-Type": "application/json",
-          Accept: "text/event-stream",
+          'Content-Type': 'application/json',
+          Accept: 'text/event-stream'
         },
         body: JSON.stringify({
           context_id: contextId,
           code,
           language,
           ...(timeoutMs !== undefined && { timeout_ms: timeoutMs })
-        }),
+        })
       });
 
       if (!response.ok) {
@@ -120,7 +124,7 @@ export class InterpreterClient extends BaseHttpClient {
       }
 
       if (!response.body) {
-        throw new Error("No response body for streaming execution");
+        throw new Error('No response body for streaming execution');
       }
 
       // Process streaming response
@@ -132,9 +136,9 @@ export class InterpreterClient extends BaseHttpClient {
 
   async listCodeContexts(): Promise<CodeContext[]> {
     return this.executeWithRetry(async () => {
-      const response = await this.doFetch("/api/contexts", {
-        method: "GET",
-        headers: { "Content-Type": "application/json" },
+      const response = await this.doFetch('/api/contexts', {
+        method: 'GET',
+        headers: { 'Content-Type': 'application/json' }
       });
 
       if (!response.ok) {
@@ -152,7 +156,7 @@ export class InterpreterClient extends BaseHttpClient {
         language: ctx.language,
         cwd: ctx.cwd || '/workspace',
         createdAt: new Date(data.timestamp),
-        lastUsed: new Date(data.timestamp),
+        lastUsed: new Date(data.timestamp)
       }));
     });
   }
@@ -160,8 +164,8 @@ export class InterpreterClient extends BaseHttpClient {
   async deleteCodeContext(contextId: string): Promise<void> {
     return this.executeWithRetry(async () => {
       const response = await this.doFetch(`/api/contexts/${contextId}`, {
-        method: "DELETE",
-        headers: { "Content-Type": "application/json" },
+        method: 'DELETE',
+        headers: { 'Content-Type': 'application/json' }
       });
 
       if (!response.ok) {
@@ -201,7 +205,7 @@ export class InterpreterClient extends BaseHttpClient {
       }
     }
 
-    throw lastError || new Error("Execution failed after retries");
+    throw lastError || new Error('Execution failed after retries');
   }
 
   private isRetryableError(error: unknown): boolean {
@@ -211,8 +215,8 @@ export class InterpreterClient extends BaseHttpClient {
 
     if (error instanceof Error) {
       return (
-        error.message.includes("not ready") ||
-        error.message.includes("initializing")
+        error.message.includes('not ready') ||
+        error.message.includes('initializing')
       );
     }
 
@@ -221,7 +225,7 @@ export class InterpreterClient extends BaseHttpClient {
 
   private async parseErrorResponse(response: Response): Promise<Error> {
     try {
-      const errorData = await response.json() as ErrorResponse;
+      const errorData = (await response.json()) as ErrorResponse;
       return createErrorFromResponse(errorData);
     } catch {
       // Fallback if response isn't JSON
@@ -240,7 +244,7 @@ export class InterpreterClient extends BaseHttpClient {
     stream: ReadableStream<Uint8Array>
   ): AsyncGenerator<string> {
     const reader = stream.getReader();
-    let buffer = "";
+    let buffer = '';
 
     try {
       while (true) {
@@ -250,11 +254,11 @@ export class InterpreterClient extends BaseHttpClient {
         }
         if (done) break;
 
-        let newlineIdx = buffer.indexOf("\n");
+        let newlineIdx = buffer.indexOf('\n');
         while (newlineIdx !== -1) {
           yield buffer.slice(0, newlineIdx);
           buffer = buffer.slice(newlineIdx + 1);
-          newlineIdx = buffer.indexOf("\n");
+          newlineIdx = buffer.indexOf('\n');
         }
       }
 
@@ -282,25 +286,25 @@ export class InterpreterClient extends BaseHttpClient {
       const data = JSON.parse(jsonData) as StreamingExecutionData;
 
       switch (data.type) {
-        case "stdout":
+        case 'stdout':
           if (callbacks.onStdout && data.text) {
             await callbacks.onStdout({
               text: data.text,
-              timestamp: data.timestamp || Date.now(),
+              timestamp: data.timestamp || Date.now()
             });
           }
           break;
 
-        case "stderr":
+        case 'stderr':
           if (callbacks.onStderr && data.text) {
             await callbacks.onStderr({
               text: data.text,
-              timestamp: data.timestamp || Date.now(),
+              timestamp: data.timestamp || Date.now()
             });
           }
           break;
 
-        case "result":
+        case 'result':
           if (callbacks.onResult) {
             // Create a ResultImpl instance from the raw data
             const result = new ResultImpl(data);
@@ -308,17 +312,17 @@ export class InterpreterClient extends BaseHttpClient {
           }
           break;
 
-        case "error":
+        case 'error':
           if (callbacks.onError) {
             await callbacks.onError({
-              name: data.ename || "Error",
-              message: data.evalue || "Unknown error",
-              traceback: data.traceback || [],
+              name: data.ename || 'Error',
+              message: data.evalue || 'Unknown error',
+              traceback: data.traceback || []
             });
           }
           break;
 
-        case "execution_complete":
+        case 'execution_complete':
           // Signal completion - callbacks can handle cleanup if needed
           break;
       }

package/src/clients/port-client.ts

@@ -1,17 +1,13 @@
 import type {
   PortCloseResult,
   PortExposeResult,
-  PortListResult,
+  PortListResult
 } from '@repo/shared';
 import { BaseHttpClient } from './base-client';
 import type { HttpClientOptions } from './types';
 
 // Re-export for convenience
-export type {
-  PortExposeResult,
-  PortCloseResult,
-  PortListResult,
-};
+export type { PortExposeResult, PortCloseResult, PortListResult };
 
 /**
  * Request interface for exposing ports
@@ -32,7 +28,6 @@ export interface UnexposePortRequest {
  * Client for port management and preview URL operations
  */
 export class PortClient extends BaseHttpClient {
-
   /**
    * Expose a port and get a preview URL
    * @param port - Port number to expose
@@ -69,9 +64,14 @@ export class PortClient extends BaseHttpClient {
    * @param port - Port number to unexpose
    * @param sessionId - The session ID for this operation
    */
-  async unexposePort(port: number, sessionId: string): Promise<PortCloseResult> {
+  async unexposePort(
+    port: number,
+    sessionId: string
+  ): Promise<PortCloseResult> {
     try {
-      const url = `/api/exposed-ports/${port}?session=${encodeURIComponent(sessionId)}`;
+      const url = `/api/exposed-ports/${port}?session=${encodeURIComponent(
+        sessionId
+      )}`;
       const response = await this.delete<PortCloseResult>(url);
 
       this.logSuccess('Port unexposed', `${port}`);
@@ -102,4 +102,4 @@ export class PortClient extends BaseHttpClient {
       throw error;
     }
   }
-}
+}

package/src/clients/process-client.ts

@@ -5,7 +5,7 @@ import type {
   ProcessListResult,
   ProcessLogsResult,
   ProcessStartResult,
-  StartProcessRequest,
+  StartProcessRequest
 } from '@repo/shared';
 import { BaseHttpClient } from './base-client';
 import type { HttpClientOptions } from './types';
@@ -18,15 +18,13 @@ export type {
   ProcessInfoResult,
   ProcessKillResult,
   ProcessLogsResult,
-  ProcessCleanupResult,
+  ProcessCleanupResult
 };
 
-
 /**
  * Client for background process management
  */
 export class ProcessClient extends BaseHttpClient {
-
   /**
    * Start a background process
    * @param command - Command to execute as a background process
@@ -42,7 +40,7 @@ export class ProcessClient extends BaseHttpClient {
       const data: StartProcessRequest = {
         command,
         sessionId,
-        processId: options?.processId,
+        processId: options?.processId
       };
 
       const response = await this.post<ProcessStartResult>(
@@ -70,7 +68,10 @@ export class ProcessClient extends BaseHttpClient {
       const url = `/api/process/list`;
       const response = await this.get<ProcessListResult>(url);
 
-      this.logSuccess('Processes listed', `${response.processes.length} processes`);
+      this.logSuccess(
+        'Processes listed',
+        `${response.processes.length} processes`
+      );
       return response;
     } catch (error) {
       this.logError('listProcesses', error);
@@ -157,11 +158,13 @@ export class ProcessClient extends BaseHttpClient {
    * Stream logs from a specific process (sandbox-scoped, not session-scoped)
    * @param processId - ID of the process to stream logs from
    */
-  async streamProcessLogs(processId: string): Promise<ReadableStream<Uint8Array>> {
+  async streamProcessLogs(
+    processId: string
+  ): Promise<ReadableStream<Uint8Array>> {
     try {
       const url = `/api/process/${processId}/stream`;
       const response = await this.doFetch(url, {
-        method: 'GET',
+        method: 'GET'
       });
 
       const stream = await this.handleStreamResponse(response);

package/src/clients/sandbox-client.ts

@@ -24,7 +24,7 @@ export class SandboxClient {
     // Ensure baseUrl is provided for all clients
     const clientOptions: HttpClientOptions = {
       baseUrl: 'http://localhost:3000',
-      ...options,
+      ...options
     };
 
     // Initialize all domain clients with shared options
@@ -36,6 +36,4 @@ export class SandboxClient {
     this.interpreter = new InterpreterClient(clientOptions);
     this.utils = new UtilityClient(clientOptions);
   }
-
-
-}
+}

package/src/clients/types.ts

@@ -4,7 +4,11 @@ import type { Logger } from '@repo/shared';
  * Minimal interface for container fetch functionality
  */
 export interface ContainerStub {
-  containerFetch(url: string, options: RequestInit, port?: number): Promise<Response>;
+  containerFetch(
+    url: string,
+    options: RequestInit,
+    port?: number
+  ): Promise<Response>;
 }
 
 /**
@@ -81,4 +85,4 @@ export type ResponseHandler<T> = (response: Response) => Promise<T>;
  */
 export interface SessionRequest {
   sessionId?: string;
-}
+}

package/src/clients/utility-client.ts

@@ -17,6 +17,13 @@ export interface CommandsResponse extends BaseApiResponse {
   count: number;
 }
 
+/**
+ * Response interface for getting container version
+ */
+export interface VersionResponse extends BaseApiResponse {
+  version: string;
+}
+
 /**
  * Request interface for creating sessions
  */
@@ -38,14 +45,13 @@ export interface CreateSessionResponse extends BaseApiResponse {
  * Client for health checks and utility operations
  */
 export class UtilityClient extends BaseHttpClient {
-
   /**
    * Ping the sandbox to check if it's responsive
    */
   async ping(): Promise<string> {
     try {
       const response = await this.get<PingResponse>('/api/ping');
-      
+
       this.logSuccess('Ping successful', response.message);
       return response.message;
     } catch (error) {
@@ -60,7 +66,7 @@ export class UtilityClient extends BaseHttpClient {
   async getCommands(): Promise<string[]> {
     try {
       const response = await this.get<CommandsResponse>('/api/commands');
-      
+
       this.logSuccess(
         'Commands retrieved',
         `${response.count} commands available`
@@ -77,7 +83,9 @@ export class UtilityClient extends BaseHttpClient {
    * Create a new execution session
    * @param options - Session configuration (id, env, cwd)
    */
-  async createSession(options: CreateSessionRequest): Promise<CreateSessionResponse> {
+  async createSession(
+    options: CreateSessionRequest
+  ): Promise<CreateSessionResponse> {
     try {
       const response = await this.post<CreateSessionResponse>(
         '/api/session/create',
@@ -91,4 +99,25 @@ export class UtilityClient extends BaseHttpClient {
       throw error;
     }
   }
-}
+
+  /**
+   * Get the container version
+   * Returns the version embedded in the Docker image during build
+   */
+  async getVersion(): Promise<string> {
+    try {
+      const response = await this.get<VersionResponse>('/api/version');
+
+      this.logSuccess('Version retrieved', response.version);
+      return response.version;
+    } catch (error) {
+      // If version endpoint doesn't exist (old container), return 'unknown'
+      // This allows for backward compatibility
+      this.logger.debug(
+        'Failed to get container version (may be old container)',
+        { error }
+      );
+      return 'unknown';
+    }
+  }
+}