Skip to content

feat: update dockerfile to support headless browsing and run better in a container #1055

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

samsheff
Copy link

Features

  • An updated Dockerfile that supports headless browsing and a fully working environment

Feature Docs
To Build the docker image:
docker build -t manus .

To run it:
docker run -v ${PWD}:/app/OpenManus/workspace -it manus

The -v flag makes it so the workspace files can be retrieved after the task is complete. It is not required for use if its not needed in your use case. Also, by default it runs main.py, but this can be overridden using docker cli flags too (for running bash or the other scripts).

Influence
Dockerfile is the only file updated. It just adds support for browsing and fixes other issues using manus in a container. It also lets OpenManus run easily on windows.

Result
image

Other
Nothing else, unless there are questions.

@samsheff samsheff changed the title feat: update dockerfile feat: update dockerfile to support headless browsing and run better in a container Apr 11, 2025
@samsheff
Copy link
Author

Hey guys, just wanted to bump this. I know you have a lot of PR's open but figured this might be a quick one to review!

Copy link

@hfabio hfabio left a comment

Choose a reason for hiding this comment

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

it may be worth decreasing the number of layers grouping the commands like:

RUN apt-get update && apt-get install -y && pip install playwright && playwright install --with-deps chromium

and also running it with alpine might optimize memory consumption.

Great PR btw, just few notes

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