Skip to content

test(sandbox): run the isolate driver for real in compat CI - #742

Merged
LinuxDevil merged 4 commits into
mainfrom
obs/o28-isolate-ci
Oct 24, 2026
Merged

LinuxDevil merged 4 commits into
mainfrom
obs/o28-isolate-ci

Conversation

@LinuxDevil

Copy link
Copy Markdown
Owner

The isolate sandbox driver was only ever tested against an injected fake; compat CI now installs isolated-vm 6.2.0 and runs a real timeout/OOM/tool block (LOUSHO_ISOLATE_TESTS=1 makes a missing module fail, not skip).

Ticket: O28 (Forge Observe vs LangSmith evaluation follow-up)

Changes

  • fix(sandbox): classify isolated-vm's own eval timeout; one compat install
  • ci(compat): pin isolated-vm to 6.2.0 (7.x needs Node 24)
  • test(sandbox): run the isolate driver for real in compat CI

Verification: focused vitest, tsc --noEmit, eslint on changed files (details in the review package); full suite and fallow run in CI.

🤖 Generated with Claude Code

LinuxDevil and others added 4 commits October 25, 2026 01:30
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…tall

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…plexity)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@LinuxDevil
LinuxDevil merged commit a492510 into main Oct 24, 2026
2 checks passed
@LinuxDevil
LinuxDevil deleted the obs/o28-isolate-ci branch October 24, 2026 22:42
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