feat: refactor client + containers to be testable, and add tests

.changeset/fruity-banks-add.md

@@ -0,0 +1,7 @@
+---
+"@cloudflare/sandbox": patch
+---
+
+comprehensive testing infrastructure and client architecture improvements
+
+Establishes complete testing suite (476 tests) with unit, integration, container, and e2e coverage. Refactors monolithic HttpClient into domain-specific clients (Command, File, Process, Port, Git, Utility) with enhanced error handling. Fixes critical port access control vulnerability and enhances preview URL security with mandatory tokens. Solves Build ID problem enabling container testing. Maintains 100% backward compatibility.

.github/workflows/prerelease.yml

@@ -20,17 +20,26 @@ jobs:
           node-version: 20
           cache: "npm"
 
-      - run: npm install
+      - name: Install dependencies
+        run: npm install
 
       - name: Modify package.json version
         run: npx tsx .github/version-script.ts
 
       - name: Resolve workspace dependencies
         run: npx tsx .github/resolve-workspace-versions.ts
 
-      - run: npm run build
-      - run: npm run check
-      - run: CI=true npm run test
+      - name: Run type checking
+        run: npm run typecheck
+
+      - name: Run linting
+        run: npm run check
+
+      - name: Build packages
+        run: npm run build
+
+      - name: Run full test suite
+        run: npm test
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3

.github/workflows/pullrequest.yml

@@ -4,26 +4,45 @@
 
 jobs:
   check:
-    timeout-minutes: 5
+    timeout-minutes: 15
     strategy:
       matrix:
         os: [
             ubuntu-24.04,
             # windows-latest,
             # macos-latest,
           ]
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
         with:
           fetch-depth: 1
 
       - uses: actions/setup-node@v4
         with:
           node-version: 20
           cache: "npm"
 
-      - run: npm install
-      - run: npm run build
-      - run: npm run check
-      - run: CI=true npm run test
+      - name: Install dependencies
+        run: npm install
+
+      - name: Run type checking
+        run: npm run typecheck
+
+      - name: Run linting
+        run: npm run check
+
+      - name: Build packages
+        run: npm run build
+
+      - name: Run unit tests
+        run: npm run test:unit
+
+      - name: Run integration tests  
+        run: npm run test:integration
+
+      - name: Run container tests
+        run: npm run test:container
+
+      - name: Run E2E tests
+        run: npm run test:e2e

.github/workflows/release.yml

@@ -23,8 +23,20 @@ jobs:
           node-version: 20
           cache: "npm"
 
-      - run: npm install
-      - run: npm run build
+      - name: Install dependencies
+        run: npm install
+
+      - name: Run type checking
+        run: npm run typecheck
+
+      - name: Run linting
+        run: npm run check
+
+      - name: Build packages
+        run: npm run build
+
+      - name: Run full test suite
+        run: npm test
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3

.github/workflows/test.yml

@@ -0,0 +1,109 @@
+name: Test Suite
+permissions:
+  contents: read
+
+on:
+  push:
+    branches: [main, develop]
+  pull_request:
+    branches: [main]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [20]
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run type checking
+        run: npm run typecheck
+
+      - name: Run linting
+        run: npm run check
+
+      - name: Run unit tests
+        run: npm run test:unit
+
+      - name: Run integration tests
+        run: npm run test:integration
+
+      - name: Run container tests
+        run: npm run test:container
+
+      - name: Run E2E tests
+        run: npm run test:e2e
+
+      - name: Generate coverage report
+        run: npm run test:coverage
+
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v3
+        with:
+          file: ./packages/sandbox/coverage/lcov.info
+          fail_ci_if_error: true
+
+  build:
+    runs-on: ubuntu-latest
+    needs: test
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build sandbox package
+        run: npm run build
+        working-directory: packages/sandbox
+
+      - name: Build Docker image
+        run: npm run docker:local
+        working-directory: packages/sandbox
+
+  example-test:
+    runs-on: ubuntu-latest
+    needs: build
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build sandbox package
+        run: npm run build
+        working-directory: packages/sandbox
+
+      - name: Test example application
+        run: |
+          npm ci
+          npm run build
+        working-directory: examples/basic