feat: integrate openclaw/crabbox as AgentENV sandbox client - #35
Conversation
Islo verification (openclaw/crabbox)Validated upstream crabbox against Islo while preparing this PR (no in-repo wrapper):
AgentENV usage remains |
Point the upstream Crabbox CLI at AgentENV's E2B-compatible API with a checked-in example config, docs, env-var notes, and CI that installs openclaw/crabbox (optional Islo smoke when ISLO_API_KEY is set). Co-authored-by: Cursor <cursoragent@cursor.com>
bab0c88 to
26be07d
Compare
|
✅ OpenCodeReview: No supported files changed. |
yingdi-shan
left a comment
There was a problem hiding this comment.
Thanks for your contribution! Integrations with other tools can improve AgentENV's usability, so this is a welcome addition. That said, I think we should keep this minimal: add a short section on how to configure it, plus a link, rather than updating the README and docs extensively or modifying the code for the integration test.
Reduce the Crabbox integration to what was asked for: a short configuration section plus links. Drops the dedicated CI workflow and the mock E2B contract server, removes the checked-in example config in favor of an inline snippet, and restores README, env-vars, overview, and common-issues to their upstream text.
|
Implemented the requested minimal shape on the current head: Crabbox is now a 19-line subsection of the existing |
Summary
e2bproviderconfig/crabbox.example.yamlcontaining only project-scoped template/workdir settings (no endpoint or credentials)crabboxdirectlyImportant deployment requirement
Crabbox's
e2bprovider sends file and process traffic tohttps://{port}-{sandboxID}.{domain}; it does not readE2B_SANDBOX_URL. AgentENV must therefore advertise[sandbox_proxy].domains[0](orAENV_SANDBOX_PROXY_DOMAINS) and expose wildcard DNS + TLS for that domain. A loopback-onlyE2B_API_URLis sufficient fordoctorandlist, but not forwarmuporrun.Validation
config/crabbox.example.yamlloaded by Crabbox with required0600permissionsactionlint -no-color .github/workflows/crabbox.ymlruff checkandruff format --checkfor the contract serverpyright scripts/tests/crabbox-e2b-contract-server.pymake fmtmake docscrabbox run --provider e2bagainst a KVM-backed AgentENV deployment with wildcard DNS/TLS (requires deployment infrastructure not available in this checkout)