Skip to content

Added template and tweaked e2b logic #2

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

Merged
merged 1 commit into from
May 19, 2025

Conversation

juancastano
Copy link
Contributor

@juancastano juancastano commented May 16, 2025

TL;DR

Improved sandbox creation for Vue applications using E2B's code interpreter.

What changed?

  • Replaced the complex sandbox setup with a more streamlined approach using E2B's code interpreter
  • Created a custom Vue template for E2B with pre-installed dependencies
  • Simplified the sandbox creation process by writing all files at once
  • Added automatic detection and installation of new dependencies from package.json
  • Updated the preview URL generation to use the correct port (5173)
  • Added Docker configuration for the Vue template

How to test?

  1. Generate a Vue application through the API
  2. Verify that the sandbox is created successfully
  3. Check that the preview URL works and displays the generated application
  4. Test with applications that require additional dependencies to ensure they're installed correctly

Why make this change?

The previous implementation had a complex setup process with multiple steps that could fail. This new approach is more reliable and efficient, using a pre-configured template that speeds up sandbox creation. It also handles dependency management more intelligently by only installing new dependencies that aren't already pre-installed in the template.

Copy link
Contributor Author

juancastano commented May 19, 2025

Merge activity

  • May 19, 6:16 AM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 19, 6:18 AM EDT: Graphite rebased this pull request as part of a merge.
  • May 19, 6:19 AM EDT: @juancastano merged this pull request with Graphite.

@juancastano juancastano changed the base branch from 05-15-code_generation_working._need_to_figure_out_e2b to graphite-base/2 May 19, 2025 10:16
@juancastano juancastano changed the base branch from graphite-base/2 to main May 19, 2025 10:16
@juancastano juancastano force-pushed the 05-15-added_template_and_tweaked_e2b_logic branch from c7903ae to dec7cb1 Compare May 19, 2025 10:17
@juancastano juancastano merged commit f93f20a into main May 19, 2025
1 check passed
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