Skip to content

Conversation

@giuliastf
Copy link
Collaborator

@giuliastf giuliastf commented Sep 17, 2025

This pull request removes the use of the python-dotenv library and all calls to load_dotenv() from both sample scripts and the CLI utility code.

Development Package

  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath-mcp==0.0.103.dev1001480104",

  # Any version from PR
  "uipath-mcp>=0.0.103.dev1001480000,<0.0.103.dev1001490000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath-mcp = { index = "testpypi" }

@giuliastf giuliastf requested a review from Copilot September 17, 2025 15:23
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request removes the dependency on the python-dotenv library and eliminates all calls to load_dotenv() from the codebase, simplifying environment variable handling by relying on the standard library's os.environ.

  • Removes dotenv import and load_dotenv() calls from CLI and sample scripts
  • Updates function signatures and formatting for consistency
  • Increments package version from 0.0.102 to 0.0.103

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/uipath_mcp/_cli/cli_run.py Removes dotenv import and load_dotenv() call from CLI utility
samples/mcp-longrunning-server/server.py Code formatting improvements including trailing commas and line breaks
samples/mcp-functions-agent/builder.py Removes dotenv import and load_dotenv() call from sample agent
samples/github-slack-agent/main.py Removes dotenv import and load_dotenv() call from sample agent
samples/github-helper-agent/main.py Removes dotenv import and load_dotenv() call from sample agent
pyproject.toml Version bump to 0.0.103

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@giuliastf giuliastf force-pushed the fix/load_dotenv branch 2 times, most recently from 4a0fbfc to 49be506 Compare September 17, 2025 16:07
@giuliastf giuliastf self-assigned this Sep 17, 2025
@giuliastf giuliastf added the build:dev Create a dev build from the pr label Sep 18, 2025
@giuliastf giuliastf force-pushed the fix/load_dotenv branch 3 times, most recently from a2fde85 to 89c9009 Compare September 18, 2025 12:38
@ionmincu ionmincu merged commit 5730f31 into main Sep 19, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build:dev Create a dev build from the pr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants