Skip to content

test(worker): bind sandbox mock statically - #355

Merged
outof-place merged 1 commit into
mainfrom
fix/poll-agent-static-sandbox-mock-20260828
Aug 28, 2026
Merged

test(worker): bind sandbox mock statically#355
outof-place merged 1 commit into
mainfrom
fix/poll-agent-static-sandbox-mock-20260828

Conversation

@outof-place

Copy link
Copy Markdown
Collaborator

Follow-up to #354 / AWP-125.

Main CI run 33173321742 showed that the top-level dynamic test import introduced in #354 could still lose the Vitest mock boundary under the full suite: one timeout test never reached its stdout spy and the following replay test reached the real Vercel Sandbox OIDC path.

This surgical test-only patch restores the static Sandbox import while retaining #354's deferred rejection/drain cleanup. Vitest hoists vi.mock, so runtime dynamic imports in poll-agent.ts resolve against the statically established mocked module.

Verification:

  • poll-agent.test.ts: 21/21, repeated 20/20 sequentially (420 tests)
  • worker TypeScript: pass
  • git diff --check: pass
  • task-scoped Sol profile exactness check: pass
  • fresh Sol/High audit: SHIP

No production runtime file changes.

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-workflow-app-dashboard Ready Ready Preview Aug 28, 2026 1:29pm

Request Review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d1d9822e-24ca-4f81-a34d-06246ae3bca5


Comment @coderabbitai help to get the list of available commands.

@outof-place
outof-place merged commit 6c461fd into main Aug 28, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant