Skip to content

java sdk add codeInterpreter#33

Merged
furykerry merged 4 commits into
openkruise:masterfrom
ZhaoQing7892:java_add_runcode
Jun 24, 2026
Merged

java sdk add codeInterpreter#33
furykerry merged 4 commits into
openkruise:masterfrom
ZhaoQing7892:java_add_runcode

Conversation

@ZhaoQing7892

Copy link
Copy Markdown
Contributor

No description provided.

@kruise-bot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

public class SandboxCodeInterpreterExample {
private static final String API_KEY = "your-api-key";
private static final String SANDBOX_DOMAIN = "your.domain.com";
private static final String TEMPLATE = "code-interpreter";

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

consider read API_KEY and SANDBOX_DOMAIN from environment variables(E2B_DOMAIN, E2B_API_KEY). The go example also has the same problem, consider revise them too.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. Updated Java examples and documentation to read E2B_API_KEY and E2B_DOMAIN from environment variables.

|------------|--------------|--------------------------|
| `commands` | `Commands` | Command execution module |
| `files` | `Filesystem` | Filesystem module |
### Fields

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

duplicate ### Fields head

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done!

@furykerry furykerry merged commit fa34edf into openkruise:master Jun 24, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants