Skip to content

feat: Add Model Context Protocol (MCP) Server Implementation #239

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

pipaber
Copy link

@pipaber pipaber commented Jul 2, 2025

Summary

This pull request introduces a Python-based server implementation for the Model Context Protocol (MCP). The primary goal of this addition is to extend the interoperability of the pointblank package, allowing it to communicate with and validate data contexts from Python environments.

This contribution is fully contained within a new mcp_folder, which includes the server logic, comprehensive unit tests, and documentation.

The main changes are:

  • Addition of a new mcp_folder (not sure where to put this implementation) containing the following files:

  • pointblank_server.py: The core MCP server that implements the MCP endpoints (for now just stdio but can be adapted to FastAPI as mcp python SDK shows on their docs.). It's designed to receive requests from a llm agent and return model-related context and validations.

  • test_mcp_server.py: A suite of pytest unit tests to ensure the server's functionality is correct and robust.

  • readme.md: Documentation explaining the purpose of the MCP server, how to set it up, and how to run it.

Related GitHub Issues and PRs

  • Ref: # N/A

Checklist

@rich-iannone
Copy link
Member

Thanks for submitting this PR and sorry it took awhile to acknowledge it. I'm going to review it shortly and provide some detailed comments for changes. Thanks again for the contribution and for your patience here!

@rich-iannone
Copy link
Member

rich-iannone commented Jul 16, 2025

In terms of changes I'd prefer right now (and this is before a more thorough review):

There's definitely more to change but I think these three things alone will get us off to a better start!

@pipaber
Copy link
Author

pipaber commented Jul 17, 2025

Hi @rich-iannone, great! I'll start doing these changes.

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