Skip to content

Conversation

FarDust
Copy link

@FarDust FarDust commented Apr 24, 2025

This pull request introduces Docker support, improves environment configuration, and enhances code documentation and maintainability. Key changes include adding Docker-related files, refining .env usage, and updating Python code with type annotations and docstrings.

Docker Support:

  • Added a Dockerfile to define the container environment, including dependencies and the command to run the MCP server.
  • Introduced a docker-compose.yml file to simplify container orchestration, allowing easy setup with environment variables and automatic restarts.
  • Updated README.md with instructions for running the MCP server using Docker, including steps for building, starting, and stopping the container.

Environment Configuration:

  • Updated .dockerignore to exclude unnecessary files and directories, such as __pycache__, .env, and node_modules, from being copied into the Docker image.
  • Removed the placeholder MEM0_API_KEY from the .env file and updated README.md with clearer instructions for setting up environment variables. [1] [2]

Code Improvements:

  • Enhanced main.py with Python >3.11 style type annotations and added numpy-style docstrings to improve code readability and maintainability. [1] [2]
  • Modified the server's CLI argument handling to use environment variables (HOST and PORT) as defaults, improving flexibility in deployment.

@sidarth164
Copy link

Hi, are there any plans to get this merged?

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