Skip to content

Comments

Improve Docker reliability and document compose usage#138

Open
stevo1403 wants to merge 2 commits intoCaviraOSS:mainfrom
stevo1403:docs/docker-setup-fixes
Open

Improve Docker reliability and document compose usage#138
stevo1403 wants to merge 2 commits intoCaviraOSS:mainfrom
stevo1403:docs/docker-setup-fixes

Conversation

@stevo1403
Copy link

@stevo1403 stevo1403 commented Feb 20, 2026

Summary

  • make dashboard opt-in via compose profile (ui) so default startup focuses on API/MCP
  • remove obsolete version key from docker-compose.yml
  • switch packages/openmemory-js/Dockerfile to node:20-bookworm-slim + npm ci build flow
  • increase Node heap during TS build to avoid Docker OOM (NODE_OPTIONS=--max-old-space-size=4096)
  • document Docker usage in README (default API/MCP command, optional dashboard profile, health checks)

Validation

  • docker compose up --build -d
  • verified container starts on :8080 and server boots successfully

Copilot AI review requested due to automatic review settings February 20, 2026 18:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

@stevo1403
Copy link
Author

Follow-up (clean formatting): addressed Copilot feedback in commit 4220bd1.

  • Dockerfile: scoped build memory option to the build command only: RUN NODE_OPTIONS="--max-old-space-size=4096" npm run build
  • docker-compose.yml: replaced openmemory wget healthcheck with a Node-based healthcheck so it does not depend on wget in the runtime image

Validation rerun: docker compose up --build -d; service starts and healthcheck runs.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

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.

1 participant