Add Docker support with build and publish workflow#2
Conversation
There was a problem hiding this comment.
Pull request overview
Adds containerization and automated Docker publishing to complement the existing release workflow, enabling distribution via GHCR alongside npm.
Changes:
- Introduces a multi-stage Dockerfile for building and running the Node/TS service.
- Adds a
docker-publishjob to the release workflow to build and push images to GHCR with semver tagging. - Adds
.dockerignoreand updates the changelog to reflect Docker/CI additions.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| Dockerfile | Multi-stage build and runtime image definition for the MCP server |
| .github/workflows/release.yml | Adds a GHCR build-and-push job to the release pipeline |
| .dockerignore | Reduces Docker build context and excludes dev-only files |
| CHANGELOG.md | Documents GHCR image publishing and updated release automation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # Default: stdio transport. Override CMD for HTTP: | ||
| # docker run -p 3000:3000 -e PLATFONE_API_KEY=… <image> node dist/index.js --transport=http | ||
| EXPOSE 3000 |
There was a problem hiding this comment.
The Dockerfile comments say the default transport is stdio, but the image still declares EXPOSE 3000. This can be misleading for users and tooling (it suggests the container listens on 3000 by default). Consider either removing EXPOSE (and documenting the HTTP option), or switching the default command to run with --transport=http if HTTP is the expected container mode.
| # Default: stdio transport. Override CMD for HTTP: | |
| # docker run -p 3000:3000 -e PLATFONE_API_KEY=… <image> node dist/index.js --transport=http | |
| EXPOSE 3000 | |
| # Default: stdio transport (no ports exposed). For HTTP, override CMD: | |
| # docker run -p 3000:3000 -e PLATFONE_API_KEY=… <image> node dist/index.js --transport=http |
…l command in Dockerfile - Address @copilot PR comments
No description provided.