Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

refactor: split monolithic server.py (800+ lines) into separate tool modules

Đang mở
  • Có gì không ổn?
#68 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
python
Lĩnh vực
backend

Hướng nghiên cứu

Bắt đầu bằng cách đọc minimax_mcp/server.py và các bài kiểm thử hiện có của dự án để lập bản đồ các định nghĩa công cụ hiện tại, cấu hình và thiết lập MCP. Tách các công cụ vào các module được đề xuất bên dưới minimax_mcp/tools/ trong khi vẫn giữ nguyên hành vi, sau đó chạy tất cả các bài kiểm thử hiện có và xác minh rằng uvx minimax-mcp hoạt động giống hệt; được xem là hoàn tất khi server.py có dưới 100 dòng và mỗi công cụ có module riêng.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Problem

minimax_mcp/server.py is a single 800+ line file containing all tool definitions, configuration, and the MCP server setup. This makes it hard to navigate, test, and maintain.

Current structure

minimax_mcp/
├── __init__.py      # 2 lines
├── __main__.py      # config generator
├── client.py        # API client (good, already separated)
├── const.py         # constants (good, already separated)
├── exceptions.py    # exceptions (good, already separated)
├── server.py        # 800+ lines — ALL tools + config + server setup
└── utils.py         # utilities (good, already separated)

Proposed structure

minimax_mcp/
├── __init__.py
├── __main__.py
├── client.py
├── const.py
├── exceptions.py
├── server.py        # ~50 lines — MCP setup, config, imports tools
├── utils.py
└── tools/
    ├── __init__.py  # exports all tool functions
    ├── audio.py     # text_to_audio, play_audio, list_voices
    ├── video.py     # generate_video, query_video_generation
    ├── image.py     # text_to_image
    ├── music.py     # music_generation
    └── voice.py     # voice_clone, voice_design

Benefits

  • Each tool module is independently testable
  • Easier to add new tools without touching a massive file
  • Clear separation of concerns
  • Reduces merge conflicts when multiple contributors work on different tools

Implementation notes

  • Each tool module receives mcp, api_client, base_path, resource_mode as module-level or injected dependencies
  • server.py imports and registers all tools
  • No behavior change — purely structural refactor

Acceptance criteria

  • server.py reduced to <100 lines
  • Each tool in its own module
  • All existing tests pass
  • uvx minimax-mcp works identically
Ngôn ngữ chính
Python
Star
1.6k
Fork
283
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của MiniMax-AI/MiniMax-MCP

Tất cả issue của MiniMax-AI/MiniMax-MCP

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.