Skip to content

Closes #42: move all the logic from the fastapi server to within the mcp server container#47

Merged
BaoNguyen09 merged 6 commits intomainfrom
42-move-all-the-logic-from-the-fastapi-server-to-within-the-mcp-server-container
Jun 9, 2025
Merged

Closes #42: move all the logic from the fastapi server to within the mcp server container#47
BaoNguyen09 merged 6 commits intomainfrom
42-move-all-the-logic-from-the-fastapi-server-to-within-the-mcp-server-container

Conversation

@BaoNguyen09
Copy link
Owner

@BaoNguyen09 BaoNguyen09 commented Jun 9, 2025

Describe your changes (Include screenshots if neccessary)

  • move all tools into /mcp folder
  • created a main.py file in /mcp directory to act as mcp server and handle calling tools
  • update Dockerfile inside /mcp to build new container with new logic
  • remove /app with fastapi logic and mcp_server.py with old mcp server logic

Issue ticket number and link

Closes #42

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes
  • This is a major refactor so I have added a release and updated the README

…rver container

This commit moves all the logic from the FastAPI server to within the MCP server container. The following files have been added to the MCP tools directory:
- custom_errors.py: Contains custom error classes for handling specific exceptions.
- fetch_directory_tree.py: Implements the logic to fetch the directory tree structure.
- fetch_file_contents.py: Implements the logic to fetch file contents.
- fetch_issue_context.py: Implements the logic to fetch issue context based on a file path.
@BaoNguyen09 BaoNguyen09 self-assigned this Jun 9, 2025
@BaoNguyen09 BaoNguyen09 merged commit 91913ea into main Jun 9, 2025
@BaoNguyen09 BaoNguyen09 deleted the 42-move-all-the-logic-from-the-fastapi-server-to-within-the-mcp-server-container branch June 9, 2025 02:37
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.

Move all the logic from the fastapi server to within the mcp server container

1 participant