Improve Docker reliability and document compose usage#138
Improve Docker reliability and document compose usage#138nullure merged 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.
- Remove dashboard service (source files missing from main branch due to botched merge in b04bf6e - Dockerfile exists but no Next.js source) - Remove deprecated version: '3.8' key - Replace wget healthcheck with Node.js inline healthcheck (more reliable in slim images, same fix as proposed in PR CaviraOSS#138) - Clean up OM_IDE_ALLOWED_ORIGINS default (remove localhost refs)
|
hello ! still having this issue when running with the ui profile: |
|
I have gotten to the root of the bug. Will push soon. |
Summary
Validation