Skip to content

Add Tensorlake sandbox - #1

Open
cooleel wants to merge 8 commits into
mainfrom
tl/sandbox
Open

Add Tensorlake sandbox#1
cooleel wants to merge 8 commits into
mainfrom
tl/sandbox

Conversation

@cooleel

@cooleel cooleel commented Jun 18, 2026

Copy link
Copy Markdown

Background

Summary

Manual Verification

Checklist

  • All commits are signed (PRs with unsigned commits cannot be merged)
  • Tests have been added / updated (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • A patch changeset for relevant packages has been added (for bug fixes / features - run pnpm changeset in the project root)
  • I have reviewed this pull request (self-review)

Future Work

Related Issues

@cooleel
cooleel force-pushed the tl/sandbox branch 3 times, most recently from a01b04e to e78b9bc Compare June 19, 2026 02:15
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
cooleel and others added 2 commits June 19, 2026 13:37
- Honor `workingDirectory` for bare commands: thread the resolved default
  working dir into the base session and `restricted()` so `run`/`spawn`
  without a per-call dir resolve against the advertised
  `defaultWorkingDirectory` instead of the image default.
- Surface genuine resume failures: gate `resume()` on the matched sandbox
  status (only SUSPENDED/SUSPENDING) and let errors propagate, rather than
  swallowing every resume error and returning a session that fails opaquely.
- Avoid cross-provider snapshot reuse: fold a stable settings/credentials
  hash into the template name so providers sharing a harness identity but
  differing in settings don't reuse each other's snapshot.
- Don't leak sandboxes on failed initialization: terminate the sandbox (and
  named template) when setup/onFirstCreate/checkpoint throws.
- Evict tunnels from the cache on failure so a transient error doesn't
  permanently poison `getPortUrl`.

Adds regression tests for each.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
cooleel pushed a commit that referenced this pull request Jul 22, 2026
…6235)

<!--
Welcome to contributing to AI SDK! We're excited to see your changes.

We suggest you read the following contributing guide we've created
before submitting:

https://github.com/vercel/ai/blob/main/CONTRIBUTING.md
-->

## Background

<!-- Why was this change necessary? -->

Updating Laminar integration docs for native integration with v7 instead
of the legacy OTEL path.

## Summary

<!-- What did you change? -->

Documentation only. Update examples for AI SDK v7, update outdated doc
links and information about laminar

## Manual Verification

<!--
For features & bugfixes.
Please explain how you *manually* verified that the change works
end-to-end as expected (excluding automated tests).
Remove the section if it's not needed (e.g. for docs).
-->

Proofreading

## Checklist

<!--
Do not edit this list. Leave items unchecked that don't apply. If you
need to track subtasks, create a new "## Tasks" section

Please check if the PR fulfills the following requirements:
-->

- [x] All commits are signed (PRs with unsigned commits cannot be
merged)
- [ ] Tests have been added / updated (for bug fixes / features)
- [ ] Documentation has been added / updated (for bug fixes / features)
- [ ] A _patch_ changeset for relevant packages has been added (for bug
fixes / features - run `pnpm changeset` in the project root)
- [X] I have reviewed this pull request (self-review)

## Future Work

<!--
Feel free to mention things not covered by this PR that can be done in
future PRs.
Remove the section if it's not needed.
 -->

## Related Issues

<!--
List related issues here, e.g. "Fixes vercel#1234".
Remove the section if it's not needed.
-->
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