cloudflare · ghostwriternr · Nov 21, 2025 · Nov 19, 2025 · Nov 20, 2025 · Nov 20, 2025
diff --git a/PR-204-REVIEW-REPORT.md b/PR-204-REVIEW-REPORT.md
@@ -1870,6 +1870,7 @@ $ npm run test:e2e
    - Less flexible
 
 3. **Subshell with export** (chosen)
+
    ```bash
    (export VAR=val; command)
    ```

diff --git a/examples/typescript-validator/index.html b/examples/typescript-validator/index.html
@@ -1,4 +1,4 @@
-<!DOCTYPE html>
+<!doctype html>
 <html lang="en">
   <head>
     <meta charset="UTF-8" />

diff --git a/examples/typescript-validator/src/index.css b/examples/typescript-validator/src/index.css
@@ -1,4 +1,4 @@
-@import "tailwindcss";
+@import 'tailwindcss';
 
 @theme {
   /* Cloudflare Accent Colors */
@@ -31,14 +31,15 @@
 
 body {
   margin: 0;
-  font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Roboto', 'Oxygen',
-    'Ubuntu', 'Cantarell', 'Fira Sans', 'Droid Sans', 'Helvetica Neue',
-    sans-serif;
+  font-family:
+    -apple-system, BlinkMacSystemFont, 'Segoe UI', 'Roboto', 'Oxygen', 'Ubuntu',
+    'Cantarell', 'Fira Sans', 'Droid Sans', 'Helvetica Neue', sans-serif;
   -webkit-font-smoothing: antialiased;
   -moz-osx-font-smoothing: grayscale;
   background-color: var(--color-bg-cream);
 }
 
-code, pre {
+code,
+pre {
   font-family: 'Monaco', 'Courier New', 'Courier', monospace;
 }
diff --git a/examples/typescript-validator/wrangler.jsonc b/examples/typescript-validator/wrangler.jsonc
@@ -14,7 +14,7 @@
   "containers": [
     {
       "class_name": "Sandbox",
-      "image": "./Dockerfile",
+      "image": "./Dockerfile"
     }
   ],
   "durable_objects": {

diff --git a/package.json b/package.json
@@ -3,9 +3,10 @@
   "version": "0.0.0",
   "description": "an api for computers",
   "scripts": {
-    "typecheck": "turbo run typecheck",
+    "typecheck": "turbo run typecheck && npm run typecheck:e2e",
+    "typecheck:e2e": "tsc --noEmit --project tests/e2e/tsconfig.json",
     "format": "prettier --write .",
-    "check": "sherif && biome check && turbo run typecheck",
+    "check": "sherif && biome check && turbo run typecheck && npm run typecheck:e2e",
     "fix": "biome check --fix && turbo run typecheck",
     "build": "turbo run build",
     "build:clean": "turbo run build --force",

diff --git a/packages/sandbox-container/package.json b/packages/sandbox-container/package.json
@@ -5,7 +5,7 @@
   "description": "Container runtime for Cloudflare Sandbox SDK",
   "type": "module",
   "scripts": {
-    "build": "tsc --project tsconfig.json",
+    "build": "tsc --project tsconfig.build.json",
     "typecheck": "tsc --noEmit",
     "test": "bun test tests",
     "test:watch": "bun test --watch tests",

diff --git a/packages/sandbox-container/src/core/types.ts b/packages/sandbox-container/src/core/types.ts
@@ -183,6 +183,7 @@ export interface PortNotFoundResponse {
 export interface ProxyErrorResponse {
   error: string;
   message: string;
+  port: number;
 }
 
 export interface Middleware {

diff --git a/packages/sandbox-container/src/handlers/interpreter-handler.ts b/packages/sandbox-container/src/handlers/interpreter-handler.ts
@@ -99,21 +99,15 @@ export class InterpreterHandler extends BaseHandler<Request, Response> {
     } else {
       // Special handling for interpreter not ready - return 503 with Retry-After header
       if (result.error.code === 'INTERPRETER_NOT_READY') {
-        return new Response(
-          JSON.stringify({
-            success: false,
-            error: result.error,
-            timestamp: new Date().toISOString()
-          }),
-          {
-            status: 503,
-            headers: {
-              'Content-Type': 'application/json',
-              'Retry-After': String(result.error.details?.retryAfter || 5),
-              ...context.corsHeaders
-            }
+        const errorResponse = this.enrichServiceError(result.error);
+        return new Response(JSON.stringify(errorResponse), {
+          status: 503,
+          headers: {
+            'Content-Type': 'application/json',
+            'Retry-After': String(result.error.details?.retryAfter || 5),
+            ...context.corsHeaders
           }
-        );
+        });
       }
 
       return this.createErrorResponse(result.error, context);

diff --git a/packages/sandbox-container/src/handlers/misc-handler.ts b/packages/sandbox-container/src/handlers/misc-handler.ts
@@ -5,6 +5,12 @@ import { ErrorCode } from '@repo/shared/errors';
 import type { RequestContext } from '../core/types';
 import { BaseHandler } from './base-handler';
 
+export interface VersionResult {
+  success: boolean;
+  version: string;
+  timestamp: string;
+}
+
 export class MiscHandler extends BaseHandler<Request, Response> {
   async handle(request: Request, context: RequestContext): Promise<Response> {
     const url = new URL(request.url);
@@ -74,7 +80,7 @@ export class MiscHandler extends BaseHandler<Request, Response> {
   ): Promise<Response> {
     const version = process.env.SANDBOX_VERSION || 'unknown';
 
-    const response = {
+    const response: VersionResult = {
       success: true,
       version,
       timestamp: new Date().toISOString()

diff --git a/packages/sandbox-container/src/services/port-service.ts b/packages/sandbox-container/src/services/port-service.ts
@@ -8,7 +8,11 @@ import type {
   PortNotExposedContext
 } from '@repo/shared/errors';
 import { ErrorCode } from '@repo/shared/errors';
-import type { PortInfo, ServiceResult } from '../core/types';
+import type {
+  PortInfo,
+  ProxyErrorResponse,
+  ServiceResult
+} from '../core/types';
 import { PortManager } from '../managers/port-manager';
 
 export interface SecurityService {
@@ -281,17 +285,15 @@ export class PortService {
       // Check if port is exposed
       const portInfo = await this.store.get(port);
       if (!portInfo) {
-        return new Response(
-          JSON.stringify({
-            error: 'Port not found',
-            message: `Port ${port} is not exposed`,
-            port
-          }),
-          {
-            status: 404,
-            headers: { 'Content-Type': 'application/json' }
-          }
-        );
+        const errorResponse: ProxyErrorResponse = {
+          error: 'Port not found',
+          message: `Port ${port} is not exposed`,
+          port
+        };
+        return new Response(JSON.stringify(errorResponse), {
+          status: 404,
+          headers: { 'Content-Type': 'application/json' }
+        });
       }
 
       // Parse proxy path using manager
@@ -323,17 +325,15 @@ export class PortService {
         { port }
       );
 
-      return new Response(
-        JSON.stringify({
-          error: 'Proxy error',
-          message: `Failed to proxy request to port ${port}: ${errorMessage}`,
-          port
-        }),
-        {
-          status: 502,
-          headers: { 'Content-Type': 'application/json' }
-        }
-      );
+      const errorResponse: ProxyErrorResponse = {
+        error: 'Proxy error',
+        message: `Failed to proxy request to port ${port}: ${errorMessage}`,
+        port
+      };
+      return new Response(JSON.stringify(errorResponse), {
+        status: 502,
+        headers: { 'Content-Type': 'application/json' }
+      });
     }
   }
 

diff --git a/packages/sandbox-container/src/services/process-service.ts b/packages/sandbox-container/src/services/process-service.ts
@@ -18,6 +18,7 @@ import type { SessionManager } from './session-manager';
 
 // Re-export types for use by ProcessStore implementations
 export type { ProcessRecord, ProcessStatus } from '../core/types';
+export type { ProcessStore } from './process-store';
 
 export interface ProcessFilters {
   status?: ProcessStatus;

diff --git a/packages/sandbox-container/tests/handlers/execute-handler.test.ts b/packages/sandbox-container/tests/handlers/execute-handler.test.ts
@@ -1,14 +1,13 @@
 import { beforeEach, describe, expect, it, vi } from 'bun:test';
-import type { ExecResult, ProcessStartResult } from '@repo/shared';
+import type { ExecResult, Logger, ProcessStartResult } from '@repo/shared';
 import type { ErrorResponse } from '@repo/shared/errors';
 import type {
   ExecuteRequest,
-  Logger,
   RequestContext,
   ServiceResult
-} from '@sandbox-container/core/types.ts';
+} from '@sandbox-container/core/types';
 import { ExecuteHandler } from '@sandbox-container/handlers/execute-handler.js';
-import type { ProcessService } from '@sandbox-container/services/process-service.ts';
+import type { ProcessService } from '@sandbox-container/services/process-service';
 import { mocked } from '../test-utils';
 
 // Mock the service dependencies
@@ -21,12 +20,14 @@ const mockProcessService = {
   streamProcessLogs: vi.fn()
 } as unknown as ProcessService;
 
-const mockLogger: Logger = {
+const mockLogger = {
   info: vi.fn(),
   error: vi.fn(),
   warn: vi.fn(),
-  debug: vi.fn()
-};
+  debug: vi.fn(),
+  child: vi.fn()
+} as Logger;
+mockLogger.child = vi.fn(() => mockLogger);
 
 // Mock request context
 const mockContext: RequestContext = {

diff --git a/packages/sandbox-container/tests/handlers/file-handler.test.ts b/packages/sandbox-container/tests/handlers/file-handler.test.ts
@@ -2,14 +2,15 @@ import { beforeEach, describe, expect, it, vi } from 'bun:test';
 import type {
   DeleteFileResult,
   FileExistsResult,
+  Logger,
   MkdirResult,
   MoveFileResult,
   ReadFileResult,
   RenameFileResult,
   WriteFileResult
 } from '@repo/shared';
 import type { ErrorResponse } from '@repo/shared/errors';
-import type { Logger, RequestContext } from '@sandbox-container/core/types';
+import type { RequestContext } from '@sandbox-container/core/types';
 import { FileHandler } from '@sandbox-container/handlers/file-handler';
 import type { FileService } from '@sandbox-container/services/file-service';
 
@@ -33,12 +34,14 @@ const mockFileService = {
   // Remove private properties to avoid type conflicts
 } as unknown as FileService;
 
-const mockLogger: Logger = {
+const mockLogger = {
   info: vi.fn(),
   error: vi.fn(),
   warn: vi.fn(),
-  debug: vi.fn()
-};
+  debug: vi.fn(),
+  child: vi.fn()
+} as Logger;
+mockLogger.child = vi.fn(() => mockLogger);
 
 // Mock request context
 const mockContext: RequestContext = {

diff --git a/packages/sandbox-container/tests/handlers/git-handler.test.ts b/packages/sandbox-container/tests/handlers/git-handler.test.ts
@@ -1,7 +1,7 @@
 import { beforeEach, describe, expect, it, vi } from 'bun:test';
-import type { GitCheckoutResult } from '@repo/shared';
+import type { GitCheckoutResult, Logger } from '@repo/shared';
 import type { ErrorResponse } from '@repo/shared/errors';
-import type { Logger, RequestContext } from '@sandbox-container/core/types.ts';
+import type { RequestContext } from '@sandbox-container/core/types';
 import { GitHandler } from '@sandbox-container/handlers/git-handler';
 import type { GitService } from '@sandbox-container/services/git-service';
 
@@ -13,12 +13,14 @@ const mockGitService = {
   listBranches: vi.fn()
 } as unknown as GitService;
 
-const mockLogger: Logger = {
+const mockLogger = {
   info: vi.fn(),
   error: vi.fn(),
   warn: vi.fn(),
-  debug: vi.fn()
-};
+  debug: vi.fn(),
+  child: vi.fn()
+} as Logger;
+mockLogger.child = vi.fn(() => mockLogger);
 
 // Mock request context
 const mockContext: RequestContext = {
-Original file line number
+Diff line change
@@ Expand Up / @@ -1870,6 +1870,7 @@ $ npm run test:e2e @@
        - Less flexible
 . **Subshell with export** (chosen)
        ```bash
        (export VAR=val; command)
        ```
@@ Expand Down @@