Mcp sample for long running #119
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
radu-mocanu
approved these changes
Jul 9, 2025
edis-uipath
force-pushed
the
feature/mcp_samples_env_longrunning
branch
2 times, most recently
from
0561543 to
65eea99
Compare
edis-uipath
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR adds a sample MCP server for long-running operations with real-time progress reporting, including configuration and documentation.
- Introduces
server.pywith async tools for fixed, custom, and random-duration blocking tasks - Adds MCP and UiPath configuration files (
mcp.json,uipath.json,pyproject.toml) - Supplies comprehensive
README.mdfor setup and usage
Reviewed Changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| samples/mcp-longrunning-server/uipath.json | Defines UiPath entry point (check filePath) |
| samples/mcp-longrunning-server/server.py | Implements async blocking tasks with progress and logging |
| samples/mcp-longrunning-server/pyproject.toml | Project metadata and dependency on uipath-mcp |
| samples/mcp-longrunning-server/mcp.json | Configures the MCP server transport and command |
| samples/mcp-longrunning-server/README.md | Instructions and examples for using the long-running server |
Comments suppressed due to low confidence (1)
samples/mcp-longrunning-server/server.py:12
- Add unit tests for
run_blocking_operationand the tool functions to verify progress reporting, error handling, and result structure.
async def run_blocking_operation(
| ctx: Context | ||
| ) -> Dict[str, Any]: | ||
| """Run a blocking operation with progress notifications.""" | ||
| start_time = time.time() |
There was a problem hiding this comment.
[nitpick] Consider using time.monotonic() instead of time.time() for measuring elapsed durations to avoid issues with system clock changes.
Suggested change
| start_time = time.time() | |
| start_time = time.monotonic() |
| } | ||
|
|
||
| # Generate random duration between 0 and max_duration_seconds | ||
| random_duration = random.uniform(0, max_duration_seconds) |
There was a problem hiding this comment.
[nitpick] Currently you cast random_duration to int, which truncates the fractional part and may lead to inconsistent logs versus actual runtime. Consider using math.ceil or document the truncation behavior.
edis-uipath
force-pushed
the
feature/mcp_samples_env_longrunning
branch
from
65eea99 to
9062bdb
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a sample MCP server for long-running operations with real-time progress reporting, including configuration and documentation.
Introduces server.py with async tools for fixed, custom, and random-duration blocking tasks
Adds MCP and UiPath configuration files (mcp.json, uipath.json, pyproject.toml)
Supplies comprehensive README.md for setup and usage