Improve Docker reliability and document compose usage#138
Improve Docker reliability and document compose usage#138stevo1403 wants to merge 2 commits intoCaviraOSS:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves Docker reliability and usability by simplifying the build process and making the dashboard optional. The changes address issues from previous PRs (#4, #5) related to Docker build failures and streamline the container setup for production use.
Changes:
- Switched from Alpine + Bun to Bookworm-slim + npm for more reliable builds
- Made dashboard opt-in via Docker Compose profile to focus default startup on API/MCP
- Documented Docker usage patterns including optional dashboard profile and health check commands
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| packages/openmemory-js/Dockerfile | Replaced Alpine+Bun with Bookworm-slim+npm, added NODE_OPTIONS heap increase, simplified user creation for Debian-based image |
| docker-compose.yml | Removed deprecated version key, added ui profile to dashboard service for opt-in behavior |
| README.md | Added documentation for Docker usage with profile examples and health check verification commands |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Follow-up (clean formatting): addressed Copilot feedback in commit 4220bd1.
Validation rerun: docker compose up --build -d; service starts and healthcheck runs. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Validation