Skip to content

feat: add all-in-one sandbox toolset#105

Merged
OhYee merged 1 commit into
mainfrom
codex/aio-sandbox-toolset-sdk
May 25, 2026
Merged

feat: add all-in-one sandbox toolset#105
OhYee merged 1 commit into
mainfrom
codex/aio-sandbox-toolset-sdk

Conversation

@hhuangzhen
Copy link
Copy Markdown
Collaborator

Thank you for creating a pull request to contribute to Serverless Devs agentrun-sdk-python code! Before you open the request please answer the following questions to help it be more easily integrated. Please check the boxes "[ ]" with "[x]" when done too.
Please select one of the PR types below to complete


Fix bugs

Bug detail

The specific manifestation of the bug or the associated issue.

Pull request tasks

  • Add test cases for the changes
  • Passed the CI test

Update docs

Reason for update

Why do you need to update your documentation?

Pull request tasks

  • Update Chinese documentation
  • Update English documentation

Add contributor

Contributed content

  • Code
  • Document

Content detail

if content_type == 'code' || content_type == 'document':
    please tell us `PR url`,like: https://github.com/Serverless-Devs/agentrun-sdk-python/pull/1
else:
    please describe your contribution in detail

Others

Reason for update

Why do you need to update your documentation?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds an “all-in-one” (AIO) sandbox toolset to the built-in integration layer so a single sandbox can expose browser + code-interpreter capabilities, including file transfer/recordings and remote-access URL helpers, and threads the new local_artifact_dir option through the public integration wrappers.

Changes:

  • Introduce AioToolSet in the built-in sandbox integration and extend the sandbox_toolset(...) factory to support TemplateType.AIO (and explicitly reject TemplateType.CUSTOM).
  • Add AIO-specific tools for upload/download, recordings management, and remote CDP/VNC URL retrieval with headers intentionally not exposed.
  • Add integration tests validating AIO factory behavior, tool exposure, local artifact dir constraints, and wrapper parameter forwarding.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
agentrun/integration/builtin/sandbox.py Adds AioToolSet, capability checks, local artifact dir handling, and updates the toolset factory for AIO/CUSTOM.
agentrun/integration/langchain/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
agentrun/integration/langgraph/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
agentrun/integration/google_adk/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
agentrun/integration/crewai/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
agentrun/integration/pydantic_ai/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
agentrun/integration/agentscope/builtin.py Threads local_artifact_dir through sandbox_toolset(...) wrapper into built-in factory.
tests/unittests/integration/test_aio_toolset.py Adds test coverage for AIO toolset behavior, path safety checks, and wrapper forwarding.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread agentrun/integration/builtin/sandbox.py
Comment thread agentrun/integration/builtin/sandbox.py
@OhYee OhYee merged commit 73c45d4 into main May 25, 2026
3 of 4 checks passed
@OhYee OhYee deleted the codex/aio-sandbox-toolset-sdk branch May 25, 2026 09:16
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.

3 participants