Skip to content

Conversation

@T-rav
Copy link
Collaborator

@T-rav T-rav commented Oct 10, 2025

Summary

  • Removed automated Docker CI workflow (.github/workflows/docker-publish.yml)
  • Updated Dockerfile to run mcp_server.py directly instead of via module import
  • Simplifies deployment to manual Docker builds only

Changes

  • Removed: .github/workflows/docker-publish.yml (77 lines) - automated multi-platform Docker build workflow
  • Updated: docker/Dockerfile - Changed CMD from python -m mcp_server to python mcp_server.py

Rationale

Moving to manual Docker builds reduces CI complexity and gives more control over release timing.

Test Plan

  • Verified Dockerfile CMD change works with local build
  • Confirmed removal of workflow file doesn't affect other CI jobs

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Refactor
    • Docker image now starts the MCP server by default instead of demo mode.
    • Removed the demo-specific environment variable, simplifying container configuration.
  • Chores
    • Removed the CI workflow for building and publishing Docker images.

@coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025

Walkthrough

Removes the Docker publish GitHub Actions workflow and updates the Docker image’s default command by dropping WEBCAT_MODE and switching from running cli.py in demo mode to running mcp_server.py.

Changes

Cohort / File(s) Summary
CI/CD Workflow Removal
.github/workflows/docker-publish.yml
Deleted workflow that built and pushed multi-arch Docker images on tag or manual dispatch, including version derivation, metadata tagging, caching, login, push, and manifest inspect.
Docker Runtime Command Update
docker/Dockerfile
Removed WEBCAT_MODE=demo and changed CMD from invoking cli.py (demo mode) to invoking mcp_server.py; no other config changes.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant User
    participant Container
    participant mcp_server.py as MCP Server

    User->>Container: docker run image
    activate Container
    Note over Container: CMD now starts mcp_server.py
    Container->>MCP Server: Launch process
    activate MCP Server
    MCP Server-->>Container: Initialize server
    Container-->>User: Server running
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nudge the build clouds, watch them part—no more CI rain,
A lighter burrow, fewer gears in the chain.
New boots on startup: MCP takes the stage,
Demo hat off, we turn a sober page.
Thump-thump—ship shape! Now onward down the lane. 🐇🚀

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly conveys the primary change, namely the removal of the automated Docker CI workflow and the shift to manual builds for v2.3.2, following the conventional commit style. It accurately reflects the changes in the PR without extraneous details.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-docker-ci-workflow-v2.3.2

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e4c1603 and 8a530ed.

📒 Files selected for processing (2)
  • .github/workflows/docker-publish.yml (0 hunks)
  • docker/Dockerfile (1 hunks)
💤 Files with no reviewable changes (1)
  • .github/workflows/docker-publish.yml
🧰 Additional context used
📓 Path-based instructions (1)
docker/Dockerfile

📄 CodeRabbit inference engine (CLAUDE.md)

Use Python 3.11 base/runtime in Dockerfile to match the required interpreter version

Files:

  • docker/Dockerfile
🔇 Additional comments (1)
docker/Dockerfile (1)

34-34: Approve script entrypoint
Verified no relative imports in docker/mcp_server.py; switching to python mcp_server.py is safe.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@T-rav T-rav merged commit 175d42f into main Oct 10, 2025
11 checks passed
@T-rav T-rav deleted the chore/remove-docker-ci-workflow-v2.3.2 branch October 10, 2025 00:44
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.

2 participants