langchain-ai/deepagentsjs
1
0
Fork 0
mirror of https://github.com/langchain-ai/deepagentsjs.git synced 2026-07-18 16:04:44 -04:00
Code Issues 42 Packages Projects Releases 10 Wiki Activity

[PR #194] [MERGED] fix(deepagents): polish sandbox interfaces #197

New Issue
Closed
opened 2026-02-16 06:17:25 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 06:17:25 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/deepagentsjs/pull/194
Author: @christian-bromann
Created: 2/6/2026
Status: ✅ Merged
Merged: 2/6/2026
Merged by: @christian-bromann

Base: main ← Head: cb/polish-sandbox-interfaces

📝 Commits (8)

📊 Changes

17 files changed (+329 additions, -757 deletions)

View changed files

➕ .changeset/slimy-sites-shout.md (+9 -0)
📝 libs/deepagents/README.md (+3 -3)
📝 libs/deepagents/src/backends/index.ts (+4 -3)
📝 libs/deepagents/src/backends/protocol.ts (+65 -101)
➖ libs/deepagents/src/backends/sandbox-provider.test.ts (+0 -330)
📝 libs/deepagents/src/index.ts (+3 -1)
📝 libs/providers/daytona/src/sandbox.ts (+46 -15)
📝 libs/providers/daytona/src/types.ts (+24 -12)
📝 libs/providers/deno/src/sandbox.int.test.ts (+2 -2)
📝 libs/providers/deno/src/sandbox.ts (+11 -19)
📝 libs/providers/deno/src/types.ts (+5 -12)
📝 libs/providers/modal/src/index.ts (+7 -2)
📝 libs/providers/modal/src/sandbox.ts (+93 -4)
📝 libs/providers/modal/src/types.ts (+4 -12)
📝 libs/providers/node-vfs/src/index.ts (+2 -6)
📝 libs/providers/node-vfs/src/sandbox.ts (+28 -208)
📝 libs/providers/node-vfs/src/types.ts (+23 -27)

📄 Description

This PR ensures we:

  • have a fromId and fromName static method on provider packages (if applicable)
  • have a instance getter to return the provider sandbox instance
  • have a client getter to provider the provider client (if applicable)
  • have a main SandboxError class as well as provider specific sandbox error classes for error handling
  • removes any SandboxProvider classes

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/deepagentsjs/pull/194 **Author:** [@christian-bromann](https://github.com/christian-bromann) **Created:** 2/6/2026 **Status:** ✅ Merged **Merged:** 2/6/2026 **Merged by:** [@christian-bromann](https://github.com/christian-bromann) **Base:** `main` ← **Head:** `cb/polish-sandbox-interfaces` --- ### 📝 Commits (8) - [`b2ebe2d`](https://github.com/langchain-ai/deepagentsjs/commit/b2ebe2d119cb75901e026b34ffee5807b8b4c69a) fix(deepagents): polish sandbox interfaces - [`1a35b7b`](https://github.com/langchain-ai/deepagentsjs/commit/1a35b7bbaaac67c7032e4018b25e0858af6f8e65) revert - [`8d8cb02`](https://github.com/langchain-ai/deepagentsjs/commit/8d8cb023e4d5d77101dd5df0d764be62ad6d058d) global ref - [`268b5fd`](https://github.com/langchain-ai/deepagentsjs/commit/268b5fd003910d8b1808969b1159043996501e10) format - [`994e57b`](https://github.com/langchain-ai/deepagentsjs/commit/994e57b5506267f2e198ba7f50eb8d48ce57e586) changesets - [`4f007f9`](https://github.com/langchain-ai/deepagentsjs/commit/4f007f9b3bafa93e8816c7131c707c5e0eb72877) tweak CI - [`59b2564`](https://github.com/langchain-ai/deepagentsjs/commit/59b2564b8c67e734db89daafc308bafb25779000) Merge branch 'main' into cb/polish-sandbox-interfaces - [`a7a7f5d`](https://github.com/langchain-ai/deepagentsjs/commit/a7a7f5d9f7c42b03a7b09ac3de6f89e2a3ba077d) fix ### 📊 Changes **17 files changed** (+329 additions, -757 deletions) <details> <summary>View changed files</summary> ➕ `.changeset/slimy-sites-shout.md` (+9 -0) 📝 `libs/deepagents/README.md` (+3 -3) 📝 `libs/deepagents/src/backends/index.ts` (+4 -3) 📝 `libs/deepagents/src/backends/protocol.ts` (+65 -101) ➖ `libs/deepagents/src/backends/sandbox-provider.test.ts` (+0 -330) 📝 `libs/deepagents/src/index.ts` (+3 -1) 📝 `libs/providers/daytona/src/sandbox.ts` (+46 -15) 📝 `libs/providers/daytona/src/types.ts` (+24 -12) 📝 `libs/providers/deno/src/sandbox.int.test.ts` (+2 -2) 📝 `libs/providers/deno/src/sandbox.ts` (+11 -19) 📝 `libs/providers/deno/src/types.ts` (+5 -12) 📝 `libs/providers/modal/src/index.ts` (+7 -2) 📝 `libs/providers/modal/src/sandbox.ts` (+93 -4) 📝 `libs/providers/modal/src/types.ts` (+4 -12) 📝 `libs/providers/node-vfs/src/index.ts` (+2 -6) 📝 `libs/providers/node-vfs/src/sandbox.ts` (+28 -208) 📝 `libs/providers/node-vfs/src/types.ts` (+23 -27) </details> ### 📄 Description This PR ensures we: - have a `fromId` and `fromName` static method on provider packages (if applicable) - have a `instance` getter to return the provider sandbox instance - have a `client` getter to provider the provider client (if applicable) - have a main SandboxError class as well as provider specific sandbox error classes for error handling - removes any SandboxProvider classes --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 06:17:25 -05:00
yindo closed this issue 2026-02-16 06:17:25 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
help wanted
needs-response
pull-request
Mirrored from GitHub Pull Request
 stale
triage: high-impact
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/deepagentsjs#197
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?