Add Docker one-command setup (bundled Ollama) + bind-host and Windows-path support - #4
Add Docker one-command setup (bundled Ollama) + bind-host and Windows-path support#4ariavhayempour wants to merge 3 commits into
Conversation
|
Some branch version issues which is why there are a couple repeated commits from earlier in the og main branch |
|
Thanks a lot @ariavhayempour for the PR, can you squash the commits to keep the history clean and you could also take a look at git rebase to know more about it. Once it's done I'll review the PR. |
Provide a `docker compose up` workflow that runs open-paxel with an optional bundled Ollama service (--profile ollama). Provider selection (OpenAI / OpenRouter / Ollama) is driven entirely by env vars in .env.example with no image rebuild, and the Docker Ollama base URL is isolated from the non-Docker dev variable so the two never collide.
Add a --host option (and OPEN_PAXEL_HOST) for `serve` so it can bind beyond 127.0.0.1, needed when running in a container. Make session discovery work on POSIX paths (macOS/Linux) and Windows. Default the model to empty so OPEN_PAXEL_MODEL is the single source of truth across app and bundled-Ollama pull.
Drop unused imports and dead metrics/helpers flagged by lint, untrack a stray frontend tsbuildinfo, and prune orphaned package-lock entries.
|
@staru09 Should be good now! Let me know if there is anything else that needs to be fixed. |
|
@staru09 updates? |
|
Hii @ariavhayempour thanks for the contribution, would you mind squashing the commits for a clean commit history. |
|
can you also attach a demo video with the changes that you've made? |
| # Host Claude sessions, read-only. Mounted at the container user's | ||
| # $HOME/.claude (HOME=/home/app) because discovery reads | ||
| # Path.home()/.claude/projects. (~ is NOT expanded by compose — use ${HOME}.) | ||
| - "${HOME}/.claude:/home/app/.claude:ro" |
There was a problem hiding this comment.
Windows use a different file-format for storage
| - "openpaxel-data:/data" | ||
|
|
||
| # Bundled local LLM — only starts with `docker compose --profile ollama up`. | ||
| ollama: |
There was a problem hiding this comment.
Make Ollama profile ready (depends_on / wait) or drop bundled Ollama from compose and document host LLM only.
2 participants
Summary
Adds a one-command Docker setup so Open-Paxel runs without a local Python/Node
toolchain, plus a few portability and config improvements found while
containerizing.
What's included
Dockerfile,docker-compose.yml,.dockerignore,.env.example, and README docs. Includes a bundled-Ollamacompose profile that pulls and serves the same model the app requests.
serve(default127.0.0.1, env-overridable)so the server can bind
0.0.0.0inside a container.modeldefaults to empty ("provider default") so
OPEN_PAXEL_MODELis the singlesource of truth across the app and the container's model pull.
tsbuildinfo, drop dead lines).Tests
Adds
tests/test_cli_serve.pyandtests/test_config.pycovering thebind-host and config changes.