Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new MCP (Model Context Protocol) server for flight search functionality using the fast-flights Python library. The server provides two tools for searching airports and flights, enabling AI agents to help users find flight information.
Key Changes:
- Added a new flight_tool MCP server with airport and flight search capabilities
- Integrated the server into the sample environments configuration
- Includes proper validation for dates, passenger counts, and search parameters
Reviewed Changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| sample-environments.yaml | Added mcp-flight configuration entry with MCP_URL endpoint |
| mcp/flight_tool/pyproject.toml | Project configuration with dependencies for fastmcp, fast_flights, and playwright |
| mcp/flight_tool/inspector.config.json | MCP Inspector configuration for local testing |
| mcp/flight_tool/flight_tool.py | Main server implementation with search_airports and search_flights tools |
| mcp/flight_tool/init.py | Package initialization with Apache 2.0 license header |
| mcp/flight_tool/README.md | Documentation for the MCP server, tools, parameters, and usage |
| mcp/flight_tool/Dockerfile | Container configuration using uv and Python 3.12 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
SiennaChien
force-pushed
the
flight-tool
branch
from
538bcfd to
30f6b85
Compare
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 7 out of 8 changed files in this pull request and generated 6 comments.
Comments suppressed due to low confidence (1)
mcp/flight_tool/flight_tool.py:274
- Except block directly handles BaseException.
except:
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
SiennaChien
force-pushed
the
flight-tool
branch
from
30f6b85 to
707f775
Compare
|
Made one comment/question in the code, but otherwise LGTM |
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
Signed-off-by: Sienna Chien <siennac@Siennas-MacBook-Air.local>
mrsabath
left a comment
mrsabath
left a comment
There was a problem hiding this comment.
Thank you Sienna for this useful contributions.
LGTM
Summary
This PR contains a simple MCP server that uses the fast-flights python library. It has been tested with the current weather demo agent (see image below)
The MCP server contains 2 tools
search_airports(): returns the airport code of airport(s) around a given citysearch_flights(): returns future flights matching the given parametersDemo Screenshot:
Edit (11/2): Additional testing completed with Peter's generic agent (will link his PR when he creates it, link to his fork). See image below:
Related issue(s)
Issue #78