Skip to content

docs: update sandbox docs for Daytona backend, backends CLI, and praisonai-sandbox package#2121

Merged
MervinPraison merged 1 commit into
mainfrom
claude/issue-2107-20260717-1022
Jul 17, 2026
Merged

docs: update sandbox docs for Daytona backend, backends CLI, and praisonai-sandbox package#2121
MervinPraison merged 1 commit into
mainfrom
claude/issue-2107-20260717-1022

Conversation

@praisonai-triage-agent

Copy link
Copy Markdown
Contributor

Fixes #2107

Summary

Updates sandbox documentation to reflect the merged PraisonAI PRs #3132 (real Daytona backend + sandbox backends CLI) and #3129 (standalone praisonai-sandbox package).

Changes

  • docs/features/sandbox-backends.mdx: replaced the daytona row "Not implemented" copy with "Daytona cloud sandboxes (via daytona-sdk)"; added a "Using the Daytona backend" subsection with an agent-centric example and DAYTONA_API_KEY callout; added a "Package layout" Info note about the standalone package and the praisonai.sandbox.* compatibility shim; added a Related card.
  • docs/cli/sandbox.mdx: added backends to the commands table and a dedicated "Backends" section with sample output and install-hint callout; added a note about the standalone praisonai-sandbox CLI.
  • docs/features/praisonai-sandbox-package.mdx (new): standalone package page with install commands, backward-compat guarantee, standalone CLI, and cross-links. Added to docs.json under Features.

SDK ground truth

All snippets verified against synced source and PR #3132 / #3129 diffs. Note: there is no SandboxConfig.daytona() factory in the SDK — Daytona is selected via SandboxConfig(sandbox_type="daytona") or --type daytona, so docs use those real paths instead of the factory suggested in the issue.

Generated with Claude Code

…sonai-sandbox package (fixes #2107)

Co-authored-by: MervinPraison <MervinPraison@users.noreply.github.com>
@MervinPraison

Copy link
Copy Markdown
Owner

@coderabbitai review

@MervinPraison

Copy link
Copy Markdown
Owner

/review

@MervinPraison

Copy link
Copy Markdown
Owner

@gemini review this PR

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@MervinPraison MervinPraison added pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:ci Blocked: CI not green on HEAD pipeline/merge-ready Eligible for merge gate auto-merge and removed pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:ci Blocked: CI not green on HEAD labels Jul 17, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces documentation for the new standalone praisonai-sandbox package, which houses the sandbox backends and can be installed independently of the main framework. It also documents the new backends CLI command and updates the Daytona backend status to implemented. The review feedback highlights a potential ModuleNotFoundError in the standalone code example due to an incorrect import path, and suggests using canonical documentation paths for several internal links to avoid redirects.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

### Standalone sandbox execution

```python
from praisonaiagents.sandbox import SandboxManager, SandboxConfig

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Since this section demonstrates standalone sandbox execution using the praisonai-sandbox package without the rest of the framework, importing from praisonaiagents.sandbox will fail with a ModuleNotFoundError if the user has only installed praisonai-sandbox. It should import from praisonai_sandbox instead.

from praisonai_sandbox import SandboxManager, SandboxConfig

Comment thread docs/cli/sandbox.mdx
The `--sandbox` flag enables secure command execution with validation and restrictions. The `praisonai sandbox` command manages sandbox containers.

<Info>
A dedicated `praisonai-sandbox` binary ships with the standalone [`praisonai-sandbox`](/features/praisonai-sandbox-package) package for when you want sandboxing without the rest of the framework. Both CLIs share the same commands.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Use the canonical path /docs/features/praisonai-sandbox-package instead of /features/praisonai-sandbox-package to maintain consistency with other links in the documentation and avoid relying on redirects.

A dedicated `praisonai-sandbox` binary ships with the standalone [`praisonai-sandbox`](/docs/features/praisonai-sandbox-package) package for when you want sandboxing without the rest of the framework. Both CLIs share the same commands.

Comment thread docs/cli/sandbox.mdx
```

<Tip>
Unavailable backends need their optional extra installed (for example `pip install "praisonai[daytona]"`). Selecting an unavailable backend with `--type X` exits with code 2 and prints an install hint — see [Installing Optional Backends](/features/sandbox-backends#installing-optional-backends).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Use the canonical path /docs/features/sandbox-backends instead of /features/sandbox-backends to maintain consistency with other links in the documentation and avoid relying on redirects.

Unavailable backends need their optional extra installed (for example `pip install "praisonai[daytona]"`). Selecting an unavailable backend with `--type X` exits with code 2 and prints an install hint — see [Installing Optional Backends](/docs/features/sandbox-backends#installing-optional-backends).

```

<Info>
**Package layout.** Sandbox backends live in the standalone **`praisonai-sandbox`** package. `pip install praisonai` pulls it in transitively; you can also install it alone with `pip install praisonai-sandbox` when you don't need the rest of the framework. All `praisonai.sandbox.*` imports keep working via a compatibility shim, so existing agent code does not need to change. See [The praisonai-sandbox Package](/features/praisonai-sandbox-package) for details.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Use the canonical path /docs/features/praisonai-sandbox-package instead of /features/praisonai-sandbox-package to maintain consistency with other links in the documentation and avoid relying on redirects.

**Package layout.** Sandbox backends live in the standalone **`praisonai-sandbox`** package. `pip install praisonai` pulls it in transitively; you can also install it alone with `pip install praisonai-sandbox` when you don't need the rest of the framework. All `praisonai.sandbox.*` imports keep working via a compatibility shim, so existing agent code does not need to change. See [The praisonai-sandbox Package](/docs/features/praisonai-sandbox-package) for details.

@MervinPraison MervinPraison added pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:ci Blocked: CI not green on HEAD and removed pipeline/merge-ready Eligible for merge gate auto-merge labels Jul 17, 2026
@MervinPraison

Copy link
Copy Markdown
Owner

Merge gate scan — eligible for assessment. Claude merge gate will assess and may auto-merge if MERGE_GATE_VERDICT: APPROVE.

@MervinPraison MervinPraison added pipeline/merge-ready Eligible for merge gate auto-merge claude-merge-gate-active and removed pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI pipeline/blocked:ci Blocked: CI not green on HEAD claude-merge-gate-active labels Jul 17, 2026
@MervinPraison

Copy link
Copy Markdown
Owner

Merge gate scan — eligible for assessment. Claude merge gate will assess and may auto-merge if MERGE_GATE_VERDICT: APPROVE.

@MervinPraison MervinPraison added claude-merge-gate-active pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI and removed pipeline/merge-ready Eligible for merge gate auto-merge labels Jul 17, 2026
@praisonai-triage-agent

Copy link
Copy Markdown
Contributor Author

MERGE_GATE_VERDICT: APPROVE

Docs-only PR, 4 files / 239 lines — within docs-primary limits. New page correctly placed in docs/features/. docs.json change is nav-only (single entry added). No blocking labels, no sensitive paths touched, no secrets in diff. Gemini left a COMMENTED review (not CHANGES_REQUESTED) flagging an import path concern and link canonicalization — minor editorial notes that do not block. No CI failures on HEAD e6ce23c1. Content is SDK-verified, agent-centric, and beginner-friendly.

@MervinPraison MervinPraison added pipeline/merge-ready Eligible for merge gate auto-merge and removed claude-merge-gate-active pipeline/awaiting-merge-gate FINAL done; waiting for merge gate / CI labels Jul 17, 2026
@MervinPraison
MervinPraison merged commit 0cd2c4b into main Jul 17, 2026
16 of 17 checks passed
@MervinPraison

Copy link
Copy Markdown
Owner

Merged by Claude PR merge gate (claude-merge-gate.yml).
Verdict: MERGE_GATE_VERDICT: APPROVE
SHA: e6ce23c
Method: merge

@MervinPraison

Copy link
Copy Markdown
Owner

Merge gate scan — not eligible for auto-merge.

  • not open
  • already merged by gate
  • mergeState=UNKNOWN

Actions: wait for CI and the Claude review chain, or add label needs-manual-review and merge manually.
Opt out: label no-auto-merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merged-by-gate pipeline/merge-ready Eligible for merge gate auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: update sandbox docs for Daytona backend, sandbox backends CLI, and praisonai-sandbox package (PraisonAI #3132, #3129)

1 participant