Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

- feature: mount in host runner #71

Merged
merged 11 commits into from
Nov 27, 2024
Merged

Conversation

agallardol
Copy link
Contributor

@agallardol agallardol commented Nov 27, 2024

Add support for mounting files and passing environment variables to Deno runner

This PR introduces several enhancements to the Deno runner in the shinkai-tools-runner library:

  1. Added support for mounting files with read/write and read-only permissions into the Deno execution environment. This allows tools to access specific files during execution.

  2. Implemented passing of environment variables to the Deno runner, including MOUNT, ASSETS, and HOME. These environment variables provide information about the mounted files and directories to the running tool.

  3. Enhanced the Deno permissions system to grant appropriate access to mounted files, assets, and necessary directories for Playwright/Chrome.

  4. Updated the example usage in the README to demonstrate how to set up and call a Shinkai tool from Rust, including passing input data, handling output, and specifying mounted files.

  5. Improved logging and error handling in the DenoExecutionStorage and DenoRunner modules.

  6. Renamed the assets field in the ExecutionContext struct to assets_files for clarity.

These changes improve the flexibility and usability of the shinkai-tools-runner library by allowing tools to access specific files and directories during execution. The updated example in the README provides a clear guide on how to leverage these new features when calling Shinkai tools from Rust.

@agallardol agallardol merged commit 71b87d5 into main Nov 27, 2024
1 check failed
@agallardol agallardol deleted the agallardol/mount-in-host-runner branch November 27, 2024 14:14
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.

1 participant