Skip to content

Conversation

@skosijer
Copy link
Contributor

@skosijer skosijer commented Sep 23, 2025

📋 Pull Request Summary

  • Created a shared Container interface for WebContainer and DockerContainer
  • Implemented the interface for WebContainers
  • Updated the usage in the whole project to switch to the new Container interface
  • Tested the app to make sure it works as expected

IMPORTANT NOTES:

  • Keeping both WebContainers and Docker will become hard to maintain over time
  • The components using WebContainers are all client side components, while Docker is purely backend service
  • If we continued down this path, we would probably have a suboptimal performance when using Docker; The LLM calls are executed on the backend, when the response is returned to the frontend the ActionRunner is parsing the messages and executing actions directly on the client side using webcontainer library (action examples: create file, update file, run shell command etc.). With Docker this would mean that we have an unnecessary hop to the backend once again to execute same actions in Docker containers. The rest of the project components are also configured to work purely on the client side making this integration suboptimal in case of Docker containers.

Test after making the changes. The app works well using WebContainers ✅
image

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.

2 participants